Skip to content

Grant kubernetes-release-test GCB GCR admin on k8s-staging-kubernetes #1042

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

Merged
merged 1 commit into from
Jul 20, 2020

Conversation

justaugustus
Copy link
Member

This is required for VDF as we need to continue running stages/releases
from the old project while publishing container images to new project.

Also, we drop membership for that svc acct to the releng-editors group.
It was a previous experiment that didn't actually work (hence this
commit).

Signed-off-by: Stephen Augustus [email protected]

/assign @dims @thockin @listx
cc: @kubernetes/release-engineering

ref: kubernetes/release#1230, kubernetes/release#270, kubernetes/release#911

Example failure: https://console.cloud.google.com/cloud-build/builds/e7beb8a7-f06b-4db9-aff5-00de05757dfb;step=2?project=kubernetes-release-test

================================================================================
CHECK PREREQUISITES  (1/6)
================================================================================

Checking required system packages: OK
Checking required PIP packages: OK
Skipping security_layer::auth_check...
Checking Docker version: OK
Verifying Docker CLI Experimental status: OK
Checking write access to registry gcr.io/k8s-staging-kubernetes: FAILED
[2020-Jul-18 22:56:02 UTC] check_prerequisites in 5s
FAILED in check_prerequisites.

RELEASE INCOMPLETE! Exiting...


anago: DONE main on 9002e5d1184f Sat Jul 18 22:56:02 UTC 2020 in 5s

This is required for VDF as we need to continue running stages/releases
from the old project while publishing container images to new project.

Also, we drop membership for that svc acct to the releng-editors group.
It was a previous experiment that didn't actually work (hence this
commit).

Signed-off-by: Stephen Augustus <[email protected]>
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 19, 2020
@k8s-ci-robot k8s-ci-robot added area/access Define who has access to what via IAM bindings, role bindings, policy, etc. wg/k8s-infra labels Jul 19, 2020
Copy link
Contributor

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@dims
Copy link
Member

dims commented Jul 20, 2020

/approve
/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dims, hasheddan, justaugustus

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 20, 2020
@k8s-ci-robot k8s-ci-robot merged commit 018a5fe into kubernetes:master Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/access Define who has access to what via IAM bindings, role bindings, policy, etc. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants