prow: add a OWNERS file

[jbtrystram]

This allows us to have permissions to /override prow tests among other things. Currently this is limited to repo administrator but an OWNERS files makes it easier to manage.
This is a copy from the openshift/os repo.

[c4rt0]

Other than the members, I find this to be a great idea. /lgtm

[c4rt0]

Rishab isn't part of our team at the moment, I think we can remove his credentials for now.

[c4rt0]

Same here.

[travier]

Hum, I think it's in https://github.com/openshift/release/blob/master/ci-operator/config/coreos/coreos-assembler/OWNERS

[travier]

which is https://github.com/openshift/release/blob/master/OWNERS_ALIASES#L99 & https://github.com/openshift/release/blob/master/OWNERS_ALIASES#L119

Is something not working as expected?

[jbtrystram]

@travier I created that because I hit #3826 (comment)

[travier]

OK, let's get this list cleaned-up here (and in openshift/os as needed as well).

[travier]

and openshift/release as well

[jbtrystram]

@travier updated .
related PRs:

• release : NO-JIRA: OWNERS: cleanup coreos group openshift/release#54463
• os : NO-JIRA: OWNERS: cleanup coreos group openshift/os#1553

[jlebon]

LGTM, though note that we don't actually use Prow for approving or merging things in this repo. It's totally fine IMO to just manually merge things if one of the Prow tests is flaking and you know the change isn't covered by it anyway.

[jlebon]

Hmm, but actually instead of this, don't we just want OWNERS_ALIASES with the coreos-approvers alias? That way, we maintain a single list of users in openshift/release.

[jbtrystram]

It's totally fine IMO to just manually merge things if one of the Prow tests is flaking and you know the change isn't covered by it anyway.

I am pretty sure I am not able to. I think only repository admins can merge ignoring checks.

I can't do anything more than this

[travier]

Note that you need to disable auto-merge to get the UI to bypass the checks. But I agree that it's only available to repo admins as it comes with risks.

So overall it's better to have those Prow commands work.

[jbtrystram]

@jlebon from what I understand of the documentation, OWNER-ALIASES only work

Hmm, but actually instead of this, don't we just want OWNERS_ALIASES with the coreos-approvers alias? That way, we maintain a single list of users in openshift/release.

yeah, did some more reading and testing, this does not work