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

ARO-9420: *: add acrpull controller #1065

Merged
merged 2 commits into from
Jan 9, 2025
Merged

ARO-9420: *: add acrpull controller #1065

merged 2 commits into from
Jan 9, 2025

Conversation

stevekuznetsov
Copy link
Contributor


*: add acrpull controller

Signed-off-by: Steve Kuznetsov [email protected]


various fixes and updates

  • updated msi pull controller to 0.1.7
  • updated pull binding CRD for the additional workload identity fields
  • updated podmonitor to use the az apigroup (we don't have core prometheus CRDs installed)
  • updated VAP parameterNotFoundAction to Allow to mitigate a potential bug
  • updated servicegroup name in pipeline.yaml for acrpull
  • serialize image-puller MI federation setup (only one update at a time is supported)

Signed-off-by: Gerd Oberlechner [email protected]


Copy link

github-actions bot commented Jan 8, 2025

Please rebase pull request.

stevekuznetsov and others added 2 commits January 8, 2025 12:52
Signed-off-by: Steve Kuznetsov <[email protected]>
- updated msi pull controller to 0.1.7
- updated pull binding CRD for the additional workload identity fields
- updated podmonitor to use the az apigroup (we don't have core prometheus CRDs installed)
- updated VAP parameterNotFoundAction to `Allow` to mitigate a potential bug
- updated servicegroup name in pipeline.yaml for acrpull
- serialize image-puller MI federation setup (only one update at a time is supported)

Signed-off-by: Gerd Oberlechner <[email protected]>
Signed-off-by: Steve Kuznetsov <[email protected]>
@geoberle geoberle merged commit 28a3958 into main Jan 9, 2025
21 checks passed
@geoberle geoberle deleted the skuznets/acrpull-only branch January 9, 2025 08:12
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