-
Notifications
You must be signed in to change notification settings - Fork 827
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
Add Artifact Registry to Image Manifests - Part Eight #4308
Conversation
- name: eu.gcr.io/k8s-test-infra | ||
service-account: [email protected] | ||
- name: asia.gcr.io/k8s-test-infra | ||
service-account: [email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This manifest was promoting to a non-existent destination. @BenTheElder, are you happy with the rename?
REDACTED MCW0CDP3YY tmp test $ gcrane ls us.gcr.io/k8s-test-infra
2022/10/04 20:03:09 No matching credentials were found for "us.gcr.io"
Error: GET https://us.gcr.io/v2/token?scope=repository%3Ak8s-test-infra%3Apull&service=us.gcr.io: UNKNOWN: Project 'project:k8s-test-infra' not found or deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@upodroid this should be k8s-infra-kubernetes-io
perhaps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it needs to be *.gcr.io/k8s-artifacts-prod/*/foobar
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these images are not being promoted currently, only pushed to gcr.io/k8s-staging-test-infra
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know, there are others as well.
https://github.com/kubernetes/k8s.io/tree/main/k8s.gcr.io#creating-staging-repos these instructions need to be reconsidered. Everyone is following them even if they don't intend on promoting images from staging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
I think this name is fine, and we can tweak it later if we go to promote anything.
EDIT: also surfacing in #sig-testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 test-infra
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
/hold
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
remove /hold when you're ready for this to merge
- name: eu.gcr.io/k8s-test-infra | ||
service-account: [email protected] | ||
- name: asia.gcr.io/k8s-test-infra | ||
service-account: [email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 test-infra
@@ -10,9 +10,35 @@ change. | |||
|
|||
Reviewers should ensure that: | |||
* The source registry is one that we trust. | |||
* There is one non-source registry for each of {us, eu, asia}. | |||
* The non-source registries include an appropriate "subdir" for the | |||
* The following destination registries must be specified: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems like something a presubmit test should do for us to reduce review burden?
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, saschagrunert, spiffxp, upodroid, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold cancel |
Part of #3976
Fixes #1343
/cc @kubernetes/release-engineering @dims @BenTheElder
This is the last one. Merging this and the other PRs should complete the GCR to Artifact Registry(AR) Migration from sig-release's perspective. The last activity for sig-k8s-infra is to update oci-proxy/registry.k8s.io to serve images from AR and delete the jobs from kubernetes/test-infra#26916