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, binding #1011

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

stevekuznetsov
Copy link
Contributor

Supersedes #986

@stevekuznetsov
Copy link
Contributor Author

Error: failed linting charts: failed processing charts
 ✖︎ frontend => (version: "0.1.0", path: "frontend/deploy/helm/frontend") > failed waiting for process: exit status 1

??

@stevekuznetsov stevekuznetsov force-pushed the skuznets/acrpull branch 3 times, most recently from a234856 to a6d978d Compare December 19, 2024 21:02
Copy link

Please rebase pull request.

Copy link

github-actions bot commented Jan 8, 2025

Please rebase pull request.

Signed-off-by: Steve Kuznetsov <[email protected]>
geoberle and others added 4 commits January 8, 2025 12:55
- 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]>
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]>
@stevekuznetsov
Copy link
Contributor Author

@geoberle not sure if GitHub will be happy with a fast-forward but this PR is rebased on top of #1065 now

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