Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Pause the e2e-nvidia-l40s-x8.yml workflow#3166

Merged
mergify[bot] merged 1 commit intoinstructlab:maininstructlab/instructlab:mainfrom
kami619:pause-xl-e2e-workflowkami619/instructlab:pause-xl-e2e-workflowCopy head branch name to clipboard
Feb 19, 2025
Merged

Pause the e2e-nvidia-l40s-x8.yml workflow#3166
mergify[bot] merged 1 commit intoinstructlab:maininstructlab/instructlab:mainfrom
kami619:pause-xl-e2e-workflowkami619/instructlab:pause-xl-e2e-workflowCopy head branch name to clipboard

Conversation

@kami619
Copy link
Contributor

@kami619 kami619 commented Feb 18, 2025

Until we fix the underlying root cause for this #2999 and/or add more time if that becomes the inevitable path.

The change in this PR comments the schedule block of the workflow yaml and left a comment to reflect the change. This becomes easier to turn it back on, than removing the schedule block altogether.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

Until we fix the underlying root cause or add more time if that becomes the inevitable path - instructlab#2999 

Signed-off-by: Kamesh Akella <kakella@redhat.com>
@mergify mergify bot added the CI/CD Affects CI/CD configuration label Feb 18, 2025
@mergify mergify bot added the one-approval PR has one approval from a maintainer label Feb 18, 2025
Copy link
Contributor

@booxter booxter left a comment

Choose a reason for hiding this comment

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

I hope you have a plan to fix it because we can't let it disabled for a long time. (Can the job be split into pieces that are shorter than 6h?)

@mergify mergify bot merged commit dad90ae into instructlab:main Feb 19, 2025
14 checks passed
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Feb 19, 2025
@kami619 kami619 deleted the pause-xl-e2e-workflow branch February 19, 2025 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments

Close sidebar
Morty Proxy This is a proxified and sanitized view of the page, visit original site.