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

Add build images CI job #32

Merged
merged 2 commits into from
Feb 21, 2022
Merged

Conversation

mikenairn
Copy link
Member

Adds a new GitHub workflow that builds and pushes the operators images (manager/bundle/catalog) to the kuadrant quay.io repo.

Updates the Makefile to bring it in line with the rest of the kuadrant operators allowing the same GH workflow to be used in all kuadrant operator repos and uses kustomizations to generate authorino and deploy manifests using the upstream operand kustomization as input.

Test runs here:

Notes:

Adds a new GitHub workflow that builds and pushes the operators images
(manager/bundle/catalog) to the kuadrant quay.io repo.
Updates the Makefile to bring it inline with the rest of the kuadrant
operators allowing the same GH workflow to be used in all kuadrant
operator repos.
Adds authorino and deploy kustomizations.
Nothing has chnaged here only the order of the resources in the manifest.
Copy link
Collaborator

@guicassolato guicassolato left a comment

Choose a reason for hiding this comment

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

Nice cleanup on the Makefile, @mikenairn! Thanks!

run: make bundle REGISTRY=${{ env.IMG_REGISTRY_HOST }} ORG=${{ env.IMG_REGISTRY_ORG }} IMAGE_TAG=latest VERSION=0.0.0
- name: Git diff
run: git diff
# Uncomment this when ORG in the Makefile to be updated to "kuadrant"
Copy link
Collaborator

Choose a reason for hiding this comment

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

It will require some coordination to update existing references to quay.io/3scale/authorino-operator.

https://github.com/Kuadrant/authorino-operator/blob/main/.github/workflows/build-latest.yaml should also go away.

cd $(PROJECT_DIR)/config/manager && $(KUSTOMIZE) edit set image controller=$(OPERATOR_IMAGE) ;\
cd $(PROJECT_DIR) && $(KUSTOMIZE) build config/default >> $(DEPLOYMENT_FILE)
cd $(PROJECT_DIR) && $(KUSTOMIZE) build config/deploy > $(DEPLOYMENT_FILE)
Copy link
Collaborator

Choose a reason for hiding this comment

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

A lot safer this way.

TBH, I didn't know kustomize allowed to point to remote resources. Apparently since March 2020.

@mikenairn mikenairn merged commit 5ada185 into Kuadrant:main Feb 21, 2022
@mikenairn mikenairn deleted the add_image_builds branch February 21, 2022 11:11
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.

2 participants