Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch ci-sdxl github runner and refactor workflow. #644

Merged
merged 4 commits into from
Dec 4, 2024

Conversation

ScottTodd
Copy link
Member

@ScottTodd ScottTodd commented Dec 4, 2024

The mi300-sdxl-kernel runner has been offline for a few weeks, so runs of this workflow have been queued: https://github.com/nod-ai/shark-ai/actions/workflows/ci-sdxl.yaml. This mi300x-4 runner is probably fit to run this workflow.

Also refactored the workflow to not use explicit build steps, which loosens the requirements on installed software and helps make progress on #584.

@monorimet
Copy link
Contributor

if we update apt before installing cmake, perhaps this won't happen? or we can use the pip install.

@ScottTodd
Copy link
Member Author

or we can use the pip install.

Yeah, we won't need CMake in integration test workflows once #584 is completed. The package build will be completed on one runner (either as workflow dependency or from a prior nightly build), then test runners will just pip install. Since this workflow is running on pull_request, I'd prefer to use the workflow dependency.

@ScottTodd ScottTodd changed the title Switch ci-sdxl to a github runner that is online. Switch ci-sdxl github runner and refactor workflow. Dec 4, 2024
@ScottTodd ScottTodd marked this pull request as ready for review December 4, 2024 18:52
@ScottTodd
Copy link
Member Author

Successful run of the workflow (phew, no regressions!): https://github.com/nod-ai/shark-ai/actions/runs/12166327005/job/33932314986?pr=644

@ScottTodd ScottTodd merged commit 2304a56 into nod-ai:main Dec 4, 2024
9 checks passed
@ScottTodd ScottTodd deleted the ci-sdxl-runner branch December 4, 2024 22:43
monorimet pushed a commit that referenced this pull request Dec 13, 2024
The `mi300-sdxl-kernel` runner has been offline for a few weeks, so runs
of this workflow have been queued:
https://github.com/nod-ai/shark-ai/actions/workflows/ci-sdxl.yaml. This
`mi300x-4` runner is probably fit to run this workflow.

Also refactored the workflow to not use explicit build steps, which
loosens the requirements on installed software and helps make progress
on #584.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants