Skip to content

add a grace for Jobs that may start in Unknown#291

Merged
roclark merged 5 commits intoNVIDIA-NeMo:mainfrom
prekshivyas:main
Aug 22, 2025
Merged

add a grace for Jobs that may start in Unknown#291
roclark merged 5 commits intoNVIDIA-NeMo:mainfrom
prekshivyas:main

Conversation

@prekshivyas
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>
Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>
Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>
zoeyz101
zoeyz101 previously approved these changes Jul 15, 2025
Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>
zoeyz101
zoeyz101 previously approved these changes Jul 16, 2025
Copy link
Copy Markdown
Contributor

@zoeyz101 zoeyz101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Signed-off-by: prekshivyas <prekhsivyas@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 8, 2025

This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or update or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 8, 2025
@prekshivyas
Copy link
Copy Markdown
Contributor Author

comment to remove stale

Copy link
Copy Markdown
Contributor

@roclark roclark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for putting this together!

@roclark roclark merged commit 19f7cab into NVIDIA-NeMo:main Aug 22, 2025
18 of 21 checks passed
zoeyz101 pushed a commit to zoeyz101/NeMo-Run that referenced this pull request Nov 12, 2025
* add a grace for Jobs that may start in Unknown

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>

* add a grace for Jobs that may start in Unknown

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>

* add a grace for Jobs that may start in Unknown

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>

* fix linting

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>

* make the handling of Unknown job status better by polling

Signed-off-by: prekshivyas <prekhsivyas@gmail.com>

---------

Signed-off-by: Prekshi Vyas <prekshivyas@gmail.com>
Signed-off-by: prekshivyas <prekhsivyas@gmail.com>
Co-authored-by: prekshivyas <prekhsivyas@gmail.com>
Signed-off-by: Zoey Zhang <zozhang@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants