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
release: Use DockerHub for all images #6548
release: Use DockerHub for all images #6548
Conversation
It seems that the above PR failed for 2 reasons:
|
@apo-ger had worked on updating all the Makefiles and also revamped the build process of the notebook-servers. But since we are tight on time I'll just revert the dockerhub change in the Dockerfiles of the notebooks, for the build process to success. But to fix this properly we must rework on the problematic build process of the notebook servers. @apo-ger let's do this after the RC0 is cut
@NickLoukas has a PR for this in flight #6549 |
d1496ba
to
639cbee
Compare
The GH Actions will pick up the change in releasing/version/VERSION and rebuild all the images. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
We'll need to have proper makefiles for building these images and not expect the base images to be already committed. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
67572a1
to
0664424
Compare
@elikatsis @StefanoFioravanzo @thesuperzapper this should be ready to merge. All tests pass, images can be build and manifests get applied in a K8s 1.22 cluster successfully |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: StefanoFioravanzo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* release: Update the tag to v1.6.0-alpha and reubild images The GH Actions will pick up the change in releasing/version/VERSION and rebuild all the images. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * Update all images to use DockerHub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * notebooks: Reuse ECR images as base We'll need to have proper makefiles for building these images and not expect the base images to be already committed. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * releasing: Update releasing script for dockerhub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * release: Update manifests for v1.6.0-rc.0 Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
* Update the tag to v1.6.0-alpha * Update all images to use DockerHub * notebooks: Reuse ECR images as base - We'll need to have proper makefiles for building these images and not expect the base images to be already committed. * Update releasing script for dockerhub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> Cherry-picked-from: kubeflow#6548
cherry-picking: kubeflow#6548 * Update all images to use DockerHub * Update releasing script for dockerhub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> Cherry-picked-by: Apostolos Gerakaris <apoger@arrikto.com>
cherry-picking: #6548 * Update all images to use DockerHub * Update releasing script for dockerhub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> Cherry-picked-by: Apostolos Gerakaris <apoger@arrikto.com> Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> Co-authored-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
* release: Update the tag to v1.6.0-alpha and reubild images The GH Actions will pick up the change in releasing/version/VERSION and rebuild all the images. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * Update all images to use DockerHub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * notebooks: Reuse ECR images as base We'll need to have proper makefiles for building these images and not expect the base images to be already committed. Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * releasing: Update releasing script for dockerhub Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com> * release: Update manifests for v1.6.0-rc.0 Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
I'm making the PR for the
v1.6-branch
initially and we can cherry-pick tomaster
later on.This PR:
docker.io/kubeflownotebookswg
releasing/version/VERSION
file to build all images with the tagv1.6.0-alpha
, to ensure all images can be build/git cc @elikatsis @StefanoFioravanzo @thesuperzapper