Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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 <[email protected]>
@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