Skip to content

ci: Update workflow to use pre-built docker containers - Part 2#895

Merged
akkart-aws merged 1 commit intoaws:masterfrom
akkart-aws:ci/containerize-ci
Jun 20, 2025
Merged

ci: Update workflow to use pre-built docker containers - Part 2#895
akkart-aws merged 1 commit intoaws:masterfrom
akkart-aws:ci/containerize-ci

Conversation

@akkart-aws
Copy link
Copy Markdown
Contributor

Description of changes:

Currently the github actions workflow installs all the packages during runtime. Instead we want to install all the packages except aws-ofi-nccl plugin within a docker container and use this container during runtime.

This commit updates the PR-CI to use the pre-built containers instead of installing the stack during runtime.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@akkart-aws akkart-aws requested a review from rajachan as a code owner May 21, 2025 19:42
Comment thread .github/workflows/distcheck.yaml Outdated
@akkart-aws akkart-aws force-pushed the ci/containerize-ci branch 4 times, most recently from f9b7125 to 342a453 Compare May 28, 2025 19:15
rauteric
rauteric previously approved these changes May 29, 2025
Comment thread .github/workflows/distcheck.yaml Outdated
@rauteric
Copy link
Copy Markdown
Contributor

Also, please add the obligatory "Signed-off-by" to your commit message.

@akkart-aws
Copy link
Copy Markdown
Contributor Author

Also, please add the obligatory "Signed-off-by" to your commit message.

Done

@akkart-aws
Copy link
Copy Markdown
Contributor Author

bot:aws:retest

@akkart-aws
Copy link
Copy Markdown
Contributor Author

bot:aws:retest

Comment thread .github/workflows/distcheck.yaml
Currently the github actions workflow installs all the packages
during runtime. Instead we want to install all the packages except
aws-ofi-nccl plugin within a docker container and use this
container during runtime.

This commit updates the PR-CI to use the pre-built containers
instead of installing the stack during runtime.

Signed-off-by: Arun Karthik <akkart@amazon.com>
@akkart-aws akkart-aws force-pushed the ci/containerize-ci branch from 220cad2 to a752255 Compare June 19, 2025 22:20
@akkart-aws akkart-aws requested a review from a team as a code owner June 19, 2025 22:20
@akkart-aws akkart-aws merged commit 2d693bb into aws:master Jun 20, 2025
31 of 46 checks passed
@akkart-aws akkart-aws deleted the ci/containerize-ci branch January 27, 2026 11:17
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.

5 participants