Allow maintainers to opt-out of perf workflow approval requests

[peternied]

Description

Tweak the logic for the maintainer list that becomes the assigned approvers on GitHub issues so that maintainers can opt out of this approval process.

Related Issues

• Related to Fix logic for getting maintainer ids for benchmark workflow #15236

Check List

• Functionality includes testing.
• API changes companion pull request created, if applicable.
• Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[peternied]

FYI - @hyandell Looks like you were also included as an org admin - I'll make sure you aren't included on as an assignee on issues like #15458 in the future.

[reta]

@peternied thanks for opening it, do you think we could rely on the same approach as you did for CODEOWNERS , for example:

/performance-benchmarks <list>

By default, we could include all maintainers in the <list but people could opt-out any time, what do you think?

[dblock]

@peternied thanks for opening it, do you think we could rely on the same approach as you did for CODEOWNERS , for example:

/performance-benchmarks <list>

By default, we could include all maintainers in the <list but people could opt-out any time, what do you think?

I like this better too!

[peternied]

Good idea @reta to reuse the codeowners. I've made this update, but I need to validate that I didn't break this workflow with the new action.

[peternied]

It has been over a week since I thought about getting myself removed from the perf workflow approvals - and even with code complete coming up this hasn't been on the front of my mind so I'm going to set this down. Mostly this is because I don't have a good way to be sure that I'm not breaking the workflow until after this change is merged :/

If this is bugging folks other than myself maybe we can have someone take up this PR or we can collaborate on getting this out, created an issue [1] to make sure this notion isn't lost when I close out this PR.

• [1] [Feature Request] The perf workflow requires maintainer approval - it could be useful to provide a way to opt out of these requests #15770