Skip to content
This repository has been archived by the owner on Mar 17, 2023. It is now read-only.

Add Makefile to build and push the JupyterHub image #52

Conversation

vpavlin
Copy link

@vpavlin vpavlin commented May 22, 2020

No description provided.

@sefkhet-abwy sefkhet-abwy bot added the size/M label May 22, 2020
@vpavlin vpavlin requested a review from LaVLaS May 22, 2020 15:36
@goern
Copy link

goern commented May 23, 2020

@harshad16 could you please check if this is covered by our Tekton pipelines? I think a 'tag-release' workflow would be better than building the image on some random laptop...

Makefile Outdated
VERSION=3.0.7
FINAL_IMG_VERSION=$(VERSION)-$(COMMIT)
FINAL_IMG_URL=$(REGISTRY)/$(REPO)/jupyterhub-img:$(FINAL_IMG_VERSION)
S2I_ARGS=--with-builder podman
Copy link

Choose a reason for hiding this comment

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

This is dependent on openshift/source-to-image#996 being merged, correct?

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, my bad, guess we should have docker by default till that PR is merged. So I changed it around - it will use docker now and there is a comment which suggest how to build with podman

@vpavlin vpavlin force-pushed the feature/add-build-makefile branch from 7b62296 to e6a7aa9 Compare May 25, 2020 08:20
@vpavlin vpavlin force-pushed the feature/add-build-makefile branch from 37dee0c to c322d47 Compare June 4, 2020 13:16
@goern
Copy link

goern commented Jun 4, 2020

is this something used for local development? Can we wrap the build and publish up in one of our pipelines?

@AICoE/devops

@harshad16
Copy link

harshad16 commented Jun 4, 2020

is this something used for local development? Can we wrap the build and publish up in one of our pipelines?

@AICoE/devops

we can enable the pipeline to execute the makefile, however we would need to provide the bots (quay push bot) details to pipeline as the makefile it pushes the image to quay itself.
or we can modify the pipeline declarative file so that the pipeline can build the image and push teh image with the requirement of makefile.

@vpavlin
Copy link
Author

vpavlin commented Jun 4, 2020

You can call the makefile just to build the images, but I am not sure it is necessary to use it. I made it as an interim solution so that we at least have the build process documented and reproducible (so far it was just in our heads). I am happy to drop this in favour of something like Tekton Pipeline definition.

@harshad16
Copy link

You can call the makefile just to build the images, but I am not sure it is necessary to use it. I made it as an interim solution so that we at least have the build process documented and reproducible (so far it was just in our heads). I am happy to drop this in favour of something like Tekton Pipeline definition.

ack.

@vpavlin
Copy link
Author

vpavlin commented Aug 31, 2020

We now have Thoth pipelines setup, so we do not need this anymore

@vpavlin vpavlin closed this Aug 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants