Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

ci: Add e2e-nvidia-l40s-x4 job variant for python 3.12 - #3296

Merged
mergify[bot] merged 1 commit into
instructlab:mainfrom
booxter:create-py312-large-workflow
Apr 18, 2025
Merged

ci: Add e2e-nvidia-l40s-x4 job variant for python 3.12#3296
mergify[bot] merged 1 commit into
instructlab:mainfrom
booxter:create-py312-large-workflow

Conversation

@booxter

@booxter booxter commented Apr 17, 2025

Copy link
Copy Markdown
Contributor

We will need this job to validate that the switch to python 3.12 doesn't
break anything, particularly for dolomite training.

Signed-off-by: Ihar Hrachyshka ihar.hrachyshka@gmail.com

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 the CI/CD Affects CI/CD configuration label Apr 17, 2025
@booxter
booxter force-pushed the create-py312-large-workflow branch from 892796f to 5146844 Compare April 17, 2025 15:56
We will need this job to validate that the switch to python 3.12 doesn't
break anything, particularly for dolomite training.

This is a verbatim copy of the original python3.11 job, but with the
following sed rule applied:

sed -i 's/python3.11/python3.12/g' {}

Signed-off-by: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>
@booxter
booxter force-pushed the create-py312-large-workflow branch from 5146844 to a6acdbd Compare April 17, 2025 15:56
@booxter
booxter requested review from RobotSail and ktdreyer April 17, 2025 16:05
@ktdreyer

Copy link
Copy Markdown
Contributor

If you push this branch to https://github.com/instructlab/instructlab , you can test it before merging

@booxter

booxter commented Apr 17, 2025

Copy link
Copy Markdown
Contributor Author

I would like to merge it before the rest of python 3.12 work happens. We'll need this workflow during the development and it's less messy if we merge it now and unblock remaining work to happen in forks not in the main repo, which would pollute branch namespaces for all other users of the repo.

@booxter
booxter requested review from a team and nathan-weinberg April 17, 2025 21:01
@booxter

booxter commented Apr 18, 2025

Copy link
Copy Markdown
Contributor Author

@instructlab/core-maintainers I will need this workflow merged to continue working on python3.12 enablement. I'd like to see this merged before all remaining issues with python 3.12 are resolved. It should be safe to merge this because the workflow is not triggered automatically and is not required to merge patches. Please.

@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 18, 2025
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Apr 18, 2025
@booxter
booxter removed the request for review from ktdreyer April 18, 2025 22:24
@mergify
mergify Bot merged commit 4478c6c into instructlab:main Apr 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants