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

Cherrypicker plugin: Find GitHub account and deploy #23469

Closed
alvaroaleman opened this issue Sep 3, 2021 · 13 comments
Closed

Cherrypicker plugin: Find GitHub account and deploy #23469

alvaroaleman opened this issue Sep 3, 2021 · 13 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/testing Categorizes an issue or PR as relevant to SIG Testing.
Milestone

Comments

@alvaroaleman
Copy link
Member

We added manifests and config for the cherrypicker plugin in #16329

However this is not quite enough because:

  • It needs to use a different GitHub account, because Prow ignores events from itself
  • It doesn't get deployed into the cluster yet

/cc @spiffxp @dims @vincepri

@alvaroaleman alvaroaleman added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 3, 2021
@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Sep 3, 2021
@alvaroaleman
Copy link
Member Author

/sig testing

@k8s-ci-robot k8s-ci-robot added sig/testing Categorizes an issue or PR as relevant to SIG Testing. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Sep 3, 2021
@cjwagner
Copy link
Member

cjwagner commented Sep 3, 2021

cc @mpherman2
/assign @droslean
The invalid config is spamming the Prow logs: "Post "http://cherrypicker": dial tcp: lookup cherrypicker on 10.111.240.10:53: no such host"

@spiffxp
Copy link
Member

spiffxp commented Sep 8, 2021

I'm gonna make an external cherry picker bot user for this

@spiffxp
Copy link
Member

spiffxp commented Sep 8, 2021

/assign

@alvaroaleman
Copy link
Member Author

We added the deployment in #23471 and from the manifests and config it looks to me like it should work, however when trying it out it does not: kubernetes-sigs/controller-runtime#1640 (comment)

Could someone with access to the prow cluster check if its indeed deployed and running?

@cjwagner
Copy link
Member

cjwagner commented Sep 8, 2021

Could someone with access to the prow cluster check if its indeed deployed and running?

The deployment exists, but there are no associated pods. The deployment status includes this:

  - lastTransitionTime: "2021-09-08T15:05:51Z"
    lastUpdateTime: "2021-09-08T15:05:51Z"
    message: 'pods "cherrypicker-5d78d679f8-" is forbidden: error looking up service
      account default/cherrypicker: serviceaccount "cherrypicker" not found'
    reason: FailedCreate
    status: "True"
    type: ReplicaFailure

It doesn't look like we actually need a service account so I made a PR to remove the reference: #23517

@alvaroaleman
Copy link
Member Author

It doesn't look like we actually need a service account so I made a PR to remove the reference: #23517

It still doesn't appear to be working unfortunately: kubernetes-sigs/controller-runtime#1640 (comment)

@spiffxp
Copy link
Member

spiffxp commented Sep 9, 2021

Still working on getting an account for you, it was flagged by abuse prevention on creation, still working on getting that addressed

@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2021

@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2021

Not quite. Found a typo, #23556 should address

@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2021

/priority important-soon
/milestone v1.23

@k8s-ci-robot k8s-ci-robot added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Sep 10, 2021
@k8s-ci-robot k8s-ci-robot added this to the v1.23 milestone Sep 10, 2021
@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2021

Worked: kubernetes-sigs/controller-runtime#1661

I'd like to get the bot added as an org member before calling this done, is there anything else you can think of @alvaroaleman ?

@alvaroaleman
Copy link
Member Author

I'd like to get the bot added as an org member before calling this done, is there anything else you can think of @alvaroaleman ?

Nope, that should be it. Thanks a lot for picking this up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/testing Categorizes an issue or PR as relevant to SIG Testing.
Projects
None yet
Development

No branches or pull requests

5 participants