-
Notifications
You must be signed in to change notification settings - Fork 54
Added build args for codeflare-sdk in notebook image build #167
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
Added build args for codeflare-sdk in notebook image build #167
Conversation
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.
Looks good. One comment.
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.
Did a quick test and the image seems to be built as expected
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.
LGTM!
Thanks for making the changes
Added a build arg for specifying the codeflare-sdk version.
References Issue #152
To test this run
docker build . --build-arg="SDK_VERSION=<VERSION>"
in thecustom-nb-image
folder.The default version is set to
0.4.4