DO NOT CLOSE IT / GitHub workflows / Access Rules

pytorch / test-infra

This repository hosts code that supports the testing infrastructure for the main PyTorch repo. For example, this repo hosts the logic to track disabled tests and slow tests, as well as our continuation integration jobs HUD/dashboard.

https://hud.pytorch.org/

Other

72 stars 68 forks source link

DO NOT CLOSE IT / GitHub workflows / Access Rules #5132

Open DanilBaibak opened 2 months ago

DanilBaibak commented 2 months ago

In the FIRST comment you can specify who will use the linux foundation (LF) runners:

Add a GitHub username to use LF runners.
Add "*" at the beginning to switch ALL users to LF runners.
Add "!" at the beginning to switch ALL users to old runners.

DanilBaibak commented 2 months ago

@atalman @botmethere @clee2000 @huydhn @izaitsevfb @jeanschmidt @osalpekar @seemethere @ZainRizvi @zxiiro

kit1980 commented 1 month ago

Should we close this?

DanilBaibak commented 1 month ago

No, pls don't close the issue. It will be used for dynamic rollout while we are migrating from our runners to the linux foundation runners.

malfet commented 6 days ago

Removed myself from the list, as LF runners queue is huge.. Also, why everyone need to be mentioned twice?

zxiiro commented 6 days ago

Removed myself from the list, as LF runners queue is huge.. Also, why everyone need to be mentioned twice?

PR pytorch/pytorch#129246 changed it so that we are using @ prefixed usernames. So I think the non-@ prefix ones no longer need to be there.

zxiiro commented 6 days ago

Looks like pytorch/pytorch#129462 introduced a regression where it no longer properly checked the @ prefixed usernames. I have a fix on that but can't open a PR as it depends on pytorch/pytorch#129500 getting merged first. Just waiting for PR checks to complete and I'll try to get it in and open a PR for the @ fix.