opendatahub-io-contrib / jupyterhub-odh

Example JupyterHub deployment using OpenShift OAuth authenticator.
16 stars 31 forks source link

Remove jupyterhub publish service #99

Closed maulikjs closed 3 years ago

maulikjs commented 3 years ago

Related Issues and Dependencies

https://issues.redhat.com/browse/ODH-405

This Pull Request implements

This MR deprecates the Jupyterhub Publish service which was deprecated downstream

vpavlin commented 3 years ago

/retest

vpavlin commented 3 years ago

@maulikjs You are changing teh Pipfile, so you need to re-lock the deps (that's probably why the CI is failing)

crobby commented 3 years ago

Deprecate or remove?

maulikjs commented 3 years ago

Deprecate or remove?

is there a difference? 🤔

crobby commented 3 years ago

Deprecate or remove?

is there a difference?

Deprecate means we're leaving it alone for now, but will remove in the future. Remove means....remove :)

maulikjs commented 3 years ago

@vpavlin can you check now?

VaishnaviHire commented 3 years ago

@maulikjs @vpavlin Tested the updated PR. No longer getting error due to Pipfile.lock file.

However the build failed with following error:

Successfully pushed image-registry.openshift-image-registry.svc:5000/opendatahub/jupyterhub-img:latest
Warning: Push failed, retrying in 5s ...
Registry server Address: 
Registry server User Name: serviceaccount
Registry server Email: serviceaccount@example.org
Registry server Password: <<non-empty>>
error: build error: Failed to push image: error copying layers and metadata from "containers-storage:[overlay@/var/lib/containers/storage+/var/run/containers/storage:overlay.imagestore=/var/lib/shared]image-registry.openshift-image-registry.svc:5000/opendatahub/jupyterhub-img:latest" to "docker://image-registry.openshift-image-registry.svc:5000/opendatahub/jupyterhub-img:latest": Error writing blob: Error initiating layer upload to /v2/opendatahub/jupyterhub-img/blobs/uploads/ in image-registry.openshift-image-registry.svc:5000: received unexpected HTTP status: 500 Internal Server Error
VaishnaviHire commented 3 years ago

/lgtm

I think the issue mentioned above was specific to my test cluster. Successfully tested the updated image on new cluster.

vpavlin commented 3 years ago

@maulikjs Looks good..can you please rebase thought? We just merged another PR and there are conflicts..sorry about it

vpavlin commented 3 years ago

lgtm