Skip to content

Update gpuCI RAPIDS_VER to 22.04#8600

Merged
jrbourbeau merged 1 commit intomainfrom
upgrade-gpuci-rapids
Jan 25, 2022
Merged

Update gpuCI RAPIDS_VER to 22.04#8600
jrbourbeau merged 1 commit intomainfrom
upgrade-gpuci-rapids

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Jan 21, 2022

A new cuDF nightly version has been detected.

Updated axis.yaml to use 22.04.

@jsignell
Copy link
Copy Markdown
Member

jsignell commented Jan 21, 2022

Ping @dask/gpu

@jsignell jsignell marked this pull request as ready for review January 21, 2022 16:58
@jsignell
Copy link
Copy Markdown
Member

jsignell commented Jan 21, 2022

So I should merge?

@jakirkham
Copy link
Copy Markdown
Member

jakirkham commented Jan 21, 2022

Let's wait for @charlesbluca

jrbourbeau
jrbourbeau reviewed Jan 21, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Let's wait for @charlesbluca

+1. Also just wanted to point out that gpuCI is currently failing due to 22.04 docker images not being available yet. I recall there being a similar delay with previous similar update PRs. Regardless, Charles should know when this is good to merge

@github-actions github-actions bot force-pushed the upgrade-gpuci-rapids branch 2 times, most recently from 9936eb3 to eb63ed7 Compare January 23, 2022 00:08
@charlesbluca
Copy link
Copy Markdown
Member

charlesbluca commented Jan 24, 2022

Thanks for the ping @jakirkham; yeah this should wait until 22.04 docker images are built, which is currently waiting on rapidsai/dask-build-environment#25 to get merged in - once that is in and the new images built, we can try the tests out again and make sure everything is passing

@charlesbluca
Copy link
Copy Markdown
Member

charlesbluca commented Jan 24, 2022

rerun tests

@github-actions github-actions bot force-pushed the upgrade-gpuci-rapids branch from eb63ed7 to 40f0575 Compare January 25, 2022 00:09
jrbourbeau
jrbourbeau approved these changes Jan 25, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Looks like 22.04 is being used and gpuCI is passing -- thanks @charlesbluca

@jrbourbeau jrbourbeau merged commit 4df8cfe into main Jan 25, 2022
@jsignell jsignell deleted the upgrade-gpuci-rapids branch January 25, 2022 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrbourbeau jrbourbeau jrbourbeau approved these changes

+1 more reviewer

@galipremsagar galipremsagar galipremsagar approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jsignell @jakirkham @charlesbluca @jrbourbeau @galipremsagar