prow icon indicating copy to clipboard operation
prow copied to clipboard

Prow seems to assign Assignees who are not in OWNERS file

Open adelton opened this issue 1 year ago • 3 comments

In https://github.com/opendatahub-io/notebooks/pull/587, after adding my LGTM review, openshift-ci (bot) came and "assigned adelton" as Assignee.

I'm not even listed in https://github.com/opendatahub-io/notebooks/blob/main/OWNERS so this is quite surprising and does not seem to match the referenced pull request process at https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#the-code-review-process.

Should the prow behaviour (perhaps in https://github.com/kubernetes-sigs/prow/tree/main/pkg/plugins/assign) be modified, or should this be brought for clarification to https://github.com/kubernetes/community/?

adelton avatar Jun 26 '24 14:06 adelton

Right, Prow does that for approvers after they approve a PR. I am actually not sure why it does that :D I'm not sure how your Prow is configured, it can be configured to consider GH approves as Prow approves, which I think is the case here; your review was a GH approve.

petr-muller avatar Jun 26 '24 16:06 petr-muller

For completeness, here's our Prow configs for the repo.

First, https://github.com/openshift/release/tree/master/ci-operator/config/opendatahub-io/notebooks (that's just CI jobs, probably not relevant).

And second, here's our global prow config https://github.com/openshift/release/blob/master/core-services/prow/02_config/_config.yaml and here's the repo specific config https://github.com/openshift/release/blob/master/core-services/prow/02_config/opendatahub-io/notebooks/_prowconfig.yaml and plugin config https://github.com/openshift/release/blob/master/core-services/prow/02_config/opendatahub-io/notebooks/_pluginconfig.yaml

jiridanek avatar Jun 26 '24 18:06 jiridanek

I am actually not sure why it does that :D

It's just Kubernetes's workflow, to assign people that take over the review / approval. It gives more visibility.

NOTE: anyone can /assign anyhow.

BenTheElder avatar Jul 25 '24 20:07 BenTheElder

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Oct 23 '24 20:10 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle rotten
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Nov 22 '24 21:11 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

k8s-triage-robot avatar Dec 22 '24 21:12 k8s-triage-robot

@k8s-triage-robot: Closing this issue, marking it as "Not Planned".

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

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-sigs/prow repository.

k8s-ci-robot avatar Dec 22 '24 21:12 k8s-ci-robot