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

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

gbenhaim
Copy link
Contributor

  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.

@@ -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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

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 <[email protected]>"
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 <[email protected]>"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

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 <[email protected]>
Copy link

@gbenhaim
Copy link
Contributor Author

Once we will get this PR merged we will update #1076 with the correct path to the Containerfile.

@mnecas mnecas merged commit bf792d7 into kubev2v:main Sep 30, 2024
5 of 6 checks 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.

2 participants