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

feat: update CS image in deployment environments to include new changes #1418

Merged

Conversation

miguelsorianod
Copy link
Collaborator

@miguelsorianod miguelsorianod commented Feb 27, 2025

What this PR does

Updates the CS image, corresponding to tag 2a03722.

A non exhaustive list of new changes are:

  • Fix ARO-HCP Node Pools reconciling when using OCM Commercial
    endpoints
  • Limit and validate CS environment attribute length at
    CS startup time
  • Validate the Managed Resource Group of a given ARO-HCP
    cluster being created is unique within an Azure Subscription
  • Validate that the Managed Resource Group of a given ARO-HCP
    cluster being created does not have the same name as
    the Resource Group of that same cluster

Jira:
Link to demo recording:

Special notes for your reviewer

janboll
janboll previously approved these changes Feb 27, 2025
@miguelsorianod
Copy link
Collaborator Author

Although the tests passed I am going to update this MR with a new SHA

@miguelsorianod miguelsorianod marked this pull request as draft February 27, 2025 12:53
A non exhaustive list of new changes are:
- Fix ARO-HCP Node Pools reconciling when using OCM Commercial
  endpoints
- Limit and validate CS environment attribute length at
  CS startup time
- Validate the Managed Resource Group of a given ARO-HCP
  cluster being created is unique within an Azure Subscription
- Validate that the Managed Resource Group of a given ARO-HCP
  cluster being created does not have the same name as
  the Resource Group of that same cluster
@miguelsorianod miguelsorianod force-pushed the msoriano-update-cs-image-nprepl-reconciling-fix branch from 520446a to 9211335 Compare February 27, 2025 14:00
@miguelsorianod miguelsorianod marked this pull request as ready for review February 27, 2025 14:00
@miguelsorianod miguelsorianod merged commit 6033aef into main Feb 27, 2025
23 checks passed
@miguelsorianod miguelsorianod deleted the msoriano-update-cs-image-nprepl-reconciling-fix branch February 27, 2025 14:17
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.

3 participants