Skip to content

RAPIDS on IBM Cloud#37

Merged
jacobtomlinson merged 32 commits intorapidsai:mainfrom
marifse:main
Oct 20, 2022
Merged

RAPIDS on IBM Cloud#37
jacobtomlinson merged 32 commits intorapidsai:mainfrom
marifse:main

Conversation

@marifse
Copy link
Copy Markdown
Contributor

@marifse marifse commented Sep 7, 2022

Closes #20

@jacobtomlinson, hope you be doing great.

I have included the IBM docs in cloud section of this repository as per discussed with you. You can check and verify and let me know or do it yourself if there is any correction required.

Thank you.

@marifse marifse mentioned this pull request Sep 8, 2022
Copy link
Copy Markdown
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

Many thanks for raising this, it looks great.

I'm currently waiting for NVIDIA to provide me access to an IBM Cloud account so I can test these instructions out. But once I have it I'll review things here and merge.

The failure is due to linter errors. We have instructions on linting in the README and you can also see the errors in the log here.

Once I get access to an account and test things out I'm happy to push any linter corrections myself though.

@jacobtomlinson jacobtomlinson added doc Improvements or additions to documentation cloud/ibm IBM Cloud labels Sep 9, 2022
Copy link
Copy Markdown
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

I've done an edit pass on this PR and tweaked things like headings, links, etc.

I've also removed much of the Kubernetes install instructions and instead linked over to the generic Kubernetes instructions. This just leaves the IBM Cloud specific steps and will help us maintain this documentation going forwards.

I've managed to test the single node instructions with the IBM Cloud account I have access to but it doesn't seem to have enough Kubernetes permissions for me to test those yet. I hope to get that resolved and finish up review and get this merged in the next couple of days.

@jacobtomlinson
Copy link
Copy Markdown
Member

I've been unable to get the IKS steps working so I'm going to split those docs out into a separate PR so that we can at least get the single-node steps merged.

@jacobtomlinson jacobtomlinson merged commit 5ece226 into rapidsai:main Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cloud/ibm IBM Cloud doc Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IBM Cloud + RAPIDS

2 participants