-
Notifications
You must be signed in to change notification settings - Fork 52
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
incorporate build condition for raft-cpu-image in multi-arch job #706
incorporate build condition for raft-cpu-image in multi-arch job #706
Conversation
41701f8
to
5669110
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one suggestion for your consideration.
Reviewing #706, I noticed the following on the Files tab (https://github.com/rapidsai/docker/pull/706/files): > FromAsCasing: 'as' and 'FROM' keywords' casing do not match > More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/ <img width="1285" alt="image" src="https://github.com/user-attachments/assets/66191c7f-77d9-415c-b965-b7e039df49fa"> This resolves those warnings. They don't change anything functionally, but I do slightly agree with the style suggestion, and it'd be nice to remove the visual noise of all those warnings in PR reviews. Authors: - James Lamb (https://github.com/jameslamb) Approvers: - Bradley Dice (https://github.com/bdice) URL: #707
e50b55c
to
d168627
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! Now that I see this happening down in a .jq
script, proposed one more change for your consideration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Changes look good to me, hopefully we'll get a successful CI run to verify that the filter worked as expected. Looks like the recent builds mostly failed with network-related issues.
/merge |
Accounts for this condition for the multiach-manifest job.