-
Notifications
You must be signed in to change notification settings - Fork 32
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
Remove outdated sdkbase directory and docs #353
Conversation
Download the local KBase SDK base Docker image: | ||
|
||
docker pull kbase/sdkbase2:latest | ||
|
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.
This is not the correct image for python apps, which would presumably be what the majority of people will be building. The correct image will be pulled automatically when building a new app, so it's easier to omit this step completely.
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.
So have we tried to rebuild some images with this setup? I'm somewhat leery when we remove a while directory.
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.
The automated tests for this repo all use this setup (i.e. ignoring the sdkbase
directory), as does anyone who runs docker pull kbase/kb-sdk
.
The only way that people might use the code is if they ran make sdkbase
, which builds an app base image with the ID kbase/kbase:sdkbase2.latest
. There were some vague/incomplete instructions in the docs about building kb-sdk
from source about running make sdkbase
, but I found that whenever I ran the command and then used kb-sdk
to generate apps, the apps were on a weird base image, and some additional tests that I'd put in failed. I believe this was the precursor to the modern sdkbase images, which are now held in https://github.com/kbase/sdkbase2.
Where does the SDK base image Dockerfile live these days? Edit:
|
|
Wait - so dockerized kb-sdk isn't released? So am I right in thinking that the sdkbase directory is completely unused everywhere, based on the answers above? |
@MrCreosote Yup! The sdkbase directory is unused. |
And dockerized kb-sdk isn't released? |
Dockerized kb-sdk is available via |
But the dockerfile is on the develop branch, hence not released, right? |
This is the current status of https://hub.docker.com/r/kbase/kb-sdk/tags
|
The
sdkbase
directory is no longer used as new KBase apps are built onsdkbase2
. This PR removes the old sdkbase directory and mentions of it in the documentation.Closes #351.