- Notifications
You must be signed in to change notification settings - Fork 1.8k
(manifests/v2) add catalog targets to Makefile #4406
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
(manifests/v2) add catalog targets to Makefile #4406
Conversation
403b99c
to 9ffbb58
Compare @camilamacedo86 this is the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a few questions and nits. Otherwise, it shows great. 🥇
9ffbb58
to f8a08b7
Compare 8853d20
to 215f778
Compare There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing changelog fragment.
Overall, it looks great.
215f778
to 21b7891
Compare targets to Makefile Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>
21b7891
to a075a66
Compare There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
# https://github.com/operator-framework/community-operators/blob/7f1438c/docs/packaging-operator.md#updating-your-existing-operator | ||
.PHONY: catalog-build | ||
catalog-build: opm ## Build a catalog image. | ||
$(OPM) index add --mode semver --tag $(CATALOG_IMG) --bundles $(BUNDLE_IMGS) $(FROM_INDEX_OPT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ make catalog-build
./bin/opm index add --mode semver --tag my.domain/testcas-catalog:v0.0.1 --bundles my.domain/testcas-bundle:v0.0.1
INFO[0000] building the index bundles="[my.domain/testcas-bundle:v0.0.1]"
Error: Binary was compiled with 'CGO_ENABLED=0', go-sqlite3 requires cgo to work. This is a stub
Usage:
opm index add [flags]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did you install opm
? It looks like you installed it from source locally, since the released binaries are all installed with CGO_ENABLED=1
.
Description of the change:
catalog-build
,catalog-push
, andopm
targets to Makefile, and improve variables/comments for ease of use.Motivation for the change: this addition to the Makefile allows users to create catalog images with
opm index add
. They can string togethermake
rules like so:or if
IMAGE_TAG_BASE
is set to an existing repo:export IMAGE_TAG_BASE=quay.io/example/my-operator make bundle-build bundle-push catalog-build catalog-push
which will build and push
quay.io/example/my-operator-bundle:v0.0.1
andquay.io/example/my-operator-catalog:v0.0.1
.Enhancement: enhancements/operator-sdk-builds-catalogs.md
Signed-off-by: Eric Stroczynski ericstroczynski@gmail.com
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs