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

Skip to content

Conversation

@jaideepr97
Copy link
Member

@jaideepr97 jaideepr97 commented Apr 14, 2025

Adds a new large-sized job specifically to run the llama pipeline. This is also an optional job that should be trigger-able when needed

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.

@mergify mergify bot added CI/CD Affects CI/CD configuration testing Relates to testing dependencies Relates to dependencies labels Apr 14, 2025
@mergify mergify bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 14, 2025
@github-actions
Copy link

E2E (NVIDIA L40S x4) workflow launched on this PR: View run

@github-actions
Copy link

e2e workflow succeeded on this PR: View run, congrats!

@mergify mergify bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 15, 2025
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Apr 15, 2025
@github-actions
Copy link

E2E (NVIDIA L40S x4) workflow launched on this PR: View run

@github-actions
Copy link

e2e workflow failed on this PR: View run, please investigate.

@mergify mergify bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 15, 2025
@github-actions
Copy link

E2E (NVIDIA L40S x4) workflow launched on this PR: View run

@github-actions
Copy link

e2e workflow succeeded on this PR: View run, congrats!

Copy link
Contributor

@ktdreyer ktdreyer left a comment

Choose a reason for hiding this comment

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

The CI is green now

@mergify mergify bot added the one-approval PR has one approval from a maintainer label Apr 15, 2025
@ktdreyer
Copy link
Contributor

Can we take this out of Draft and merge? It's not clear to me why this is in Draft.

@jaideepr97
Copy link
Member Author

Can we take this out of Draft and merge? It's not clear to me why this is in Draft.

I'm not 100% confident this is achieving what it is supposed to -i.e, running a large job with the llama config

@jaideepr97 jaideepr97 marked this pull request as ready for review April 16, 2025 14:48
@jaideepr97 jaideepr97 force-pushed the llama-e2e-large branch 3 times, most recently from d03b5f6 to 2f445f1 Compare April 16, 2025 14:59
@jaideepr97 jaideepr97 changed the title Llama e2e large ci: add new Llama e2e large sized job Apr 16, 2025
@mergify mergify bot added the ci-failure PR has at least one CI failure label Apr 16, 2025
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Apr 16, 2025
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Apr 16, 2025
@courtneypacheco courtneypacheco removed the request for review from RobotSail April 16, 2025 16:48
@mergify mergify bot merged commit b09063f into main Apr 16, 2025
31 checks passed
@mergify mergify bot deleted the llama-e2e-large branch April 16, 2025 16:50
courtneypacheco added a commit that referenced this pull request Jun 24, 2025
We created this job in #3288 to run the llama pipeline, but the job has persistently failed and we no longer monitor the llama pipeline in this repo. Therefore, I am removing this git workflow.

Signed-off-by: Courtney Pacheco <[email protected]>
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 dependencies Relates to dependencies testing Relates to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants