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

enable trigger for k/release #9518

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

BenTheElder
Copy link
Member

/priority critical-urgent

this should not have been disabled, will investigate how it was disabled later

@k8s-ci-robot k8s-ci-robot added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. area/prow Issues or PRs related to prow labels Sep 21, 2018
@BenTheElder
Copy link
Member Author

/assign @cblecker @krzyzacy

@cblecker
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 21, 2018
Copy link
Member

@krzyzacy krzyzacy left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BenTheElder, cblecker, krzyzacy

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

BenTheElder referenced this pull request Sep 21, 2018
@BenTheElder
Copy link
Member Author

verify gofmt seems to be stuck
/override pull-test-infra-verify-gofmt

@k8s-ci-robot
Copy link
Contributor

@BenTheElder: Overrode contexts on behalf of BenTheElder: pull-test-infra-verify-gofmt

In response to this:

verify gofmt seems to be stuck
/override pull-test-infra-verify-gofmt

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@BenTheElder
Copy link
Member Author

podlogs:

Cloning into 'test-infra'...
I0921 19:00:33.815] Args: --job=pull-test-infra-verify-gofmt --service-account=/etc/service-account/service-account.json --repo=k8s.io/test-infra=master:8839d1ddf73f26911434454d268d333688970548,9518:c480863320fe47ab8f484dbec08a892f3c2bb74c --root=/go/src --upload=gs://kubernetes-jenkins/pr-logs --scenario=execute -- ./hack/verify-gofmt.sh
I0921 19:00:33.815] Bootstrap pull-test-infra-verify-gofmt...
I0921 19:00:33.820] Builder: gke-prow-default-pool-3c8994a8-3f4h
I0921 19:00:33.824] Gubernator results at https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/pr-logs/pull/test-infra/9518/pull-test-infra-verify-gofmt/15612
I0921 19:00:33.824] Call:  gcloud auth activate-service-account --key-file=/etc/service-account/service-account.json
W0921 19:00:34.678] Activated service account credentials for: [pr-kubekins@kubernetes-jenkins-pull.iam.gserviceaccount.com]
I0921 19:00:35.690] process 18 exited with code 0 after 0.0m
I0921 19:00:35.691] Call:  gcloud config get-value account
W0921 19:00:36.246] Your active configuration is: [default]
W0921 19:00:36.250] 
I0921 19:00:36.840] process 30 exited with code 0 after 0.0m
I0921 19:00:36.841] Will upload results to gs://kubernetes-jenkins/pr-logs using pr-kubekins@kubernetes-jenkins-pull.iam.gserviceaccount.com
I0921 19:00:36.843] Call:  kubectl get -oyaml pods/3c163c49-bdd0-11e8-9f70-0a580a6c03a7
E0921 19:00:36.864] unable to upload podspecs: [Errno 2] No such file or directory
I0921 19:00:36.865] Root: /go/src
I0921 19:00:36.865] cd to /go/src
I0921 19:00:36.866] Checkout: /go/src/k8s.io/test-infra master:8839d1ddf73f26911434454d268d333688970548,9518:c480863320fe47ab8f484dbec08a892f3c2bb74c to /go/src/k8s.io/test-infra
I0921 19:00:36.866] Call:  git init k8s.io/test-infra
I0921 19:00:36.886] Initialized empty Git repository in /go/src/k8s.io/test-infra/.git/
I0921 19:00:36.888] process 43 exited with code 0 after 0.0m
I0921 19:00:36.889] Call:  git config --local user.name 'K8S Bootstrap'
I0921 19:00:36.910] process 44 exited with code 0 after 0.0m
I0921 19:00:36.911] Call:  git config --local user.email k8s_bootstrap@localhost
I0921 19:00:36.923] process 45 exited with code 0 after 0.0m
I0921 19:00:36.924] Call:  git fetch --quiet --tags https://github.com/kubernetes/test-infra master +refs/pull/9518/head:refs/pr/9518

This should have completed in under a minute.

@BenTheElder
Copy link
Member Author

hmm I don't think that worked properly @fejta

/test pull-test-infra-verify-gofmt

@BenTheElder
Copy link
Member Author

/test pull-test-infra-verify-gofmt

@k8s-ci-robot k8s-ci-robot merged commit 77ff7a5 into kubernetes:master Sep 21, 2018
@k8s-ci-robot
Copy link
Contributor

@BenTheElder: Updated the plugins configmap using the following files:

  • key plugins.yaml using file prow/plugins.yaml

In response to this:

/priority critical-urgent

this should not have been disabled, will investigate how it was disabled later

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@BenTheElder BenTheElder deleted the release-trigger branch September 21, 2018 19:07
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/prow Issues or PRs related to prow 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. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants