Skip to content
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

Building the docker image using GitHub actions and registry #1523

Merged
merged 3 commits into from
Aug 25, 2023

Conversation

pushyamig
Copy link
Contributor

@pushyamig pushyamig commented Aug 22, 2023

The PR add a workflow where each commit to master we will have a Docker build via github action and final images are pushed to Github Container registry.

Openshift Pull these images from Github Container registry
https://docs.openshift.com/container-platform/4.11/openshift_images/image-streams-manage.html#images-imagestream-import_image-streams-managing

Full working model is here (Note: The POC is not from this code since the push variable need to target to a branch instead of master

  1. Github actions https://github.com/pushyamig/my-learning-analytics/actions
  2. Github registry https://github.com/pushyamig/my-learning-analytics/pkgs/container/my-learning-analytics
  3. Github branch used for POC https://github.com/pushyamig/my-learning-analytics/blob/ghactions/.github/workflows/main.yml
  4. Openshift pulling from Github registry https://console-openshift-console.apps.containersprod.art2.p1.openshiftapps.com/k8s/ns/myla-beta/imagestreamtags/pu-test%3Alatest/history
  5. Myla running from this workflow https://umich.beta.instructure.com/courses/562259/external_tools/54703

@pushyamig
Copy link
Contributor Author

This branch has latest changes from master now

Copy link
Member

@jonespm jonespm left a comment

Choose a reason for hiding this comment

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

This looks good to me. I think when we do the release we'll either update the instructions or add another workflow to do when we release but we can use this before that!

@pushyamig pushyamig merged commit 1b4bbbe into tl-its-umich-edu:master Aug 25, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants