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 configs for building a bundle with Konflux #1075

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gbenhaim
Copy link

  1. Add a script that Konflux will update with the latest references to the operands. This script is also used during the build for producing the variables that will be used when generating the manifests from templates.

  2. Add a Containerfile for building the bundle.

1. Add a script that Konflux will update with the latest
references to the operands. This script is also used during the build
for producing the variables that will be used when generating
the manifests from templates.

2. Add a Containerfile for building the bundle.

Signed-off-by: gbenhaim <gbenhaim@redhat.com>
Copy link

sonarcloud bot commented Sep 25, 2024

@@ -0,0 +1,63 @@
FROM quay.io/konflux-ci/operator-sdk-builder@sha256:08f084228288d6314573d7742671fc7ddf84c19e4cf8eeb7623f261a514d0e4e as builder
Copy link
Member

Choose a reason for hiding this comment

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

Please move this under the /build/ dir

io.openshift.tags="migration" \
io.k8s.description="Migration Toolkit for Virtualization - Operator Bundle" \
summary="Migration Toolkit for Virtualization - Operator Bundle" \
maintainer="Migration Toolkit for Virtualization Team <migtoolkit-virt@redhat.com>"
Copy link
Member

Choose a reason for hiding this comment

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

Please switch to

        maintainer="Forklift by Konveyor Community <forklift-dev@googlegroups.com>"

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