PyTorch depends on custom actions stored in contributor personal repos

[zxiiro]

There's a few GitHub actions that PyTorch depends on that appear to be stored in personal repos of some contributors. This is a risk as the team cannot easily update these actions if the contributor is not available for whatever reason. We should standardize on storing any custom actions in a PyTorch controlled repo.

Here's a list of actions that I believe maybe personal repos:

• malfet/checkout@silent-checkout
• nick-fields/retry@3e91a01
• nick-fields/[email protected]
• seemethere/download-artifact-s3
• seemethere/upload-artrifact-s3@v5
• necojackarc/auto-request-review@e08cdff

[zxiiro]

Discussed this with @seemethere, @ZainRizvi, and @jeanschmidt over Slack. There was agreement that this should be moved to a more central location and we decided that putting these actions into test-infra is probably the best place at the moment.

This is low priority but I'll look into moving some of these repos.

[jwagantall]

@zxiiro Working on this, so if I understood correctly:

These actions are going to be moved from the private repos into test-infra right?
Any workflows using these actions will also need to be updated which that would include test-infra itself and the main pytorch repos? Do I need to cover other repos?

[zxiiro]

I don't know what other repos use these. I've only checked pytorch/pytorch. At one of the CI Sync meetings though we did agree that pytorch/test-infra was likely the best place.

[jwagantall]

Follow up on this:

• malfet/checkout@silent-checkout
  • There is a similar feature in a later version of the original checkout action. Checking with Nikita as it would be a better option to
    just follow a later version of the forked checkout action
• nick-fields/retry@3e91a01
• nick-fields/[email protected]
  • This can potentially be left like it is since there are few contributors behind this action which is beneficial
• seemethere/download-artifact-s3
• seemethere/upload-artrifact-s3@v5
  • Looking on what are the changes added to this forked action to see if we can go back to using the original
• necojackarc/auto-request-review@e08cdff