Skip to content

chore: consolidate backup config after moving prod config to sdp-pipelines - #6645

Open
Tim Steffens (tmstff) wants to merge 1 commit into
Azure:mainfrom
tmstff:consolidate_backup_config
Open

chore: consolidate backup config after moving prod config to sdp-pipelines#6645
Tim Steffens (tmstff) wants to merge 1 commit into
Azure:mainfrom
tmstff:consolidate_backup_config

Conversation

@tmstff

@tmstff Tim Steffens (tmstff) commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

https://redhat.atlassian.net/browse/ARO-29247

What

We want to have to prod specific backup config in sdp-pipelines

Why

The move is done to be able to see the changes in the rendered configuration for verification. Also you're able to change the config a bit more quickly and independent from ARO-HCP.

This is just cleaning up leftovers.

Testing

make -C config materialize and check results

Special notes for your reviewer

PR Checklist

  • PR is scoped to a single task (no mixed concerns)
  • Title follows Conventional Commits format
  • Summary explains the "Why" behind the change
  • Linked to relevant ticket/issue
  • Self-reviewed the diff
  • CI/CD checks are passing (ignore Tide)
  • Commit history is clean (rebased/squashed)
  • All comment threads resolved before merge

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes leftover Microsoft-cloud-specific backend backup scheduling overrides from config/config.msft.clouds-overlay.yaml, aligning this repo’s config with the stated move of production backup configuration into sdp-pipelines.

Changes:

  • Removed backend.backupScheduleState overrides from the MSFT int and stg environment defaults.
  • Removed the backend.backupScheduleState: Disabled override from the MSFT prod environment defaults.
  • Removed the per-region australiaeast.backend.backupScheduleState override.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tmstff

Copy link
Copy Markdown
Collaborator Author

@geoberle

Copy link
Copy Markdown
Collaborator

/approve
/lgtm

@openshift-ci

openshift-ci Bot commented Aug 24, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geoberle, tmstff

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:

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

@tmstff

Copy link
Copy Markdown
Collaborator Author

/unhold

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD e11deed and 2 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 7add20e and 1 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 0eba6aa and 0 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/hold

Revision 3c81c74 was retested 3 times: holding

@tmstff

Copy link
Copy Markdown
Collaborator Author

/unhold

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD a500a9d and 2 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD cb92a98 and 1 for PR HEAD 3c81c74 in total

@tmstff

Copy link
Copy Markdown
Collaborator Author

/test e2e-parallel

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 61af7a5 and 0 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/hold

Revision 3c81c74 was retested 3 times: holding

@tmstff

Copy link
Copy Markdown
Collaborator Author

/test e2e-parallel
/unhold

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 0c0a9d3 and 2 for PR HEAD 3c81c74 in total

@tmstff

Copy link
Copy Markdown
Collaborator Author

/test e2e-parallel

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 9cc2e1d and 1 for PR HEAD 3c81c74 in total

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 2d7a67b and 0 for PR HEAD 3c81c74 in total

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants