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

Sets cuda_default for p Linux and removes cuda_9 #19933

Merged

Conversation

mahdipub
Copy link
Contributor

@mahdipub mahdipub commented Jul 30, 2024

Now that plinux build moved to docker, we can set cuda level to cuda_default as we use for others like xlinux. With that we can also remove cuda_9 which only plinux is using.

Signed-off-by: mahdi@ibm.com

@mahdipub mahdipub force-pushed the sets_cuda_to_default_for_all branch from ce98c02 to ee081d6 Compare July 30, 2024 14:24
Now that plinux build moved to docker, we can set cuda level to `cuda_default` as we use for others like xlinux. With that we can also remove `cuda_9` which only plinux is using.

Signed-off-by: mahdi@ibm.com
@mahdipub mahdipub force-pushed the sets_cuda_to_default_for_all branch from ee081d6 to 17894a5 Compare July 30, 2024 14:29
Copy link
Contributor

@AdamBrousseau AdamBrousseau left a comment

Choose a reason for hiding this comment

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

lgtm. I believe we still have machine issues running docker builds on the osu machines so a pr build should still fail.

@AdamBrousseau AdamBrousseau merged commit 3b3874e into eclipse-openj9:master Jul 30, 2024
2 checks passed
@AdamBrousseau
Copy link
Contributor

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.

2 participants