Skip to content

Set timeout value for remaining lint jobs#142444

Closed
huydhn wants to merge 1 commit into
pytorch:mainfrom
huydhn:increase-max-timeout-remaining-lint
Closed

Set timeout value for remaining lint jobs#142444
huydhn wants to merge 1 commit into
pytorch:mainfrom
huydhn:increase-max-timeout-remaining-lint

Conversation

@huydhn

@huydhn huydhn commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

Some lint jobs are using the default 30 minutes timeout, but the jobs could wait up to 90 minutes now for the Docker image to become available after pytorch/test-infra#6013

@huydhn huydhn added test-config/default no-runner-experiments Bypass Meta/LF runner determinator labels Dec 10, 2024
@huydhn huydhn requested a review from wdvr December 10, 2024 01:00
@huydhn huydhn requested a review from a team as a code owner December 10, 2024 01:00
@pytorch-bot

pytorch-bot Bot commented Dec 10, 2024 •
edited
Loading

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/142444

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit 97f3c81 with merge base a108b28 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot Bot added the topic: not user facing topic category label Dec 10, 2024
@huydhn

huydhn commented Dec 10, 2024

Copy link
Copy Markdown
Contributor Author

@pytorchbot merge -f 'Lint jobs have passed'

@pytorchmergebot

pytorchmergebot commented Dec 10, 2024

Copy link
Copy Markdown
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@jeanschmidt

jeanschmidt commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

It seems that lint jobs are still failing :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wdvr wdvr wdvr approved these changes

Assignees

No one assigned

Labels

Merged no-runner-experiments Bypass Meta/LF runner determinator test-config/default topic: not user facing topic category

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@huydhn @pytorchmergebot @jeanschmidt @wdvr