Skip datasources sync for prow

[janboll]

What/Why

Add skipDatasourceSync to skip sync for certain environments.
Also add the skip for prow environments.
We commonly see 409 caused by concurrent updates

[Copilot]

Pull request overview

This PR introduces an environment-configurable flag to skip Grafana datasource synchronization, and enables it by default for prow environments to avoid frequent 409s from concurrent updates.

Changes:

• Add a SkipSync short-circuit to the Grafana datasources pipeline step.
• Wire monitoring.skipDatasourceSync into the region pipeline as skipSync.
• Extend config defaults/schema and set prow rendered config to skip datasource sync.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tooling/templatize/pkg/pipeline/grafana_datasources.go	Adds early-return when SkipSync is set, with a log message.
dev-infrastructure/region-pipeline.yaml	Passes config-driven skipDatasourceSync into the GrafanaDatasources step as skipSync.
config/rendered/dev/prow/westus3.yaml	Enables skipDatasourceSync: true for prow.
config/rendered/dev/pers/westus3.yaml	Adds explicit skipDatasourceSync: false in rendered config.
config/rendered/dev/perf/westus3.yaml	Adds explicit skipDatasourceSync: false in rendered config.
config/rendered/dev/dev/westus3.yaml	Adds explicit skipDatasourceSync: false in rendered config.
config/rendered/dev/cspr/westus3.yaml	Adds explicit skipDatasourceSync: false in rendered config.
config/rendered/dev/ci01/westus3.yaml	Adds explicit skipDatasourceSync: false in rendered config.
config/config.yaml	Adds default monitoring.skipDatasourceSync: false and sets prow defaults to true.
config/config.schema.json	Extends the monitoring schema with skipDatasourceSync boolean.

[Copilot]

Pull request overview

Copilot reviewed 18 out of 26 changed files in this pull request and generated no new comments.

[raelga]

/lgtm
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: janboll, raelga

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• config/OWNERS [janboll,raelga]
• dev-infrastructure/OWNERS [janboll,raelga]
• test/OWNERS [janboll,raelga]
• tooling/OWNERS [janboll,raelga]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-merge-bot]

/retest-required

Remaining retests: 0 against base HEAD 6c7c554 and 2 for PR HEAD dd7e9f6 in total

[openshift-ci]

New changes are detected. LGTM label has been removed.

[Copilot]

Pull request overview

Copilot reviewed 18 out of 26 changed files in this pull request and generated no new comments.

[openshift-ci]

@janboll: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-parallel	554a429	link	true	/test e2e-parallel
ci/prow/verify	1a5c6ff	link	true	/test verify

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.