Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provision shard generation with cx keyvaults #819

Merged
merged 1 commit into from
Nov 20, 2024
Merged

provision shard generation with cx keyvaults #819

merged 1 commit into from
Nov 20, 2024

Conversation

geoberle
Copy link
Collaborator

@geoberle geoberle commented Nov 8, 2024

What this PR does

adapts the provision shard configuration to the new format with the dedicated azure section, referencing the the DNS zone, and MGMT cluster KVs to use (CX secrets + CX MSI).

⚠️ only merge together with a CS promotion that implements the new provision shard structure

Jira: ARO-10856
Link to demo recording:

Special notes for your reviewer

Copy link

Please rebase pull request.

adapts the provision shard configuration to the new format with the dedicated
azure section, referencing the the DNS zone, and MGMT cluster KVs to use (CX secrets + CX MSI).

Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
@machi1990
Copy link
Collaborator

Hi @geoberle we can undraft the PR and move it ready to review. While doing so, the cs image tag needs to be bumped to 5528e96 which contains the change. Thanks

@machi1990
Copy link
Collaborator

Hi @geoberle we can undraft the PR and move it ready to review. While doing so, the cs image tag needs to be bumped to 5528e96 which contains the change. Thanks

Maybe he change in 5528e96 isn't the one we need as it'll require the env to be recreated 🤔

@geoberle geoberle marked this pull request as ready for review November 20, 2024 12:57
@geoberle geoberle merged commit 7ed2e61 into main Nov 20, 2024
5 checks passed
@geoberle geoberle deleted the cs-cx-kv branch November 20, 2024 12:57
katherinelc321 pushed a commit to katherinelc321/ARO-HCP that referenced this pull request Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants