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

Added Bundle Image #1877

Merged
merged 2 commits into from
Jul 3, 2023
Merged

Conversation

yuriolisa
Copy link
Contributor

Resolves #1823

Signed-off-by: Yuri Sa <[email protected]>
@yuriolisa yuriolisa requested a review from a team June 28, 2023 15:36
Signed-off-by: Yuri Sa <[email protected]>
Copy link
Contributor

@jaronoff97 jaronoff97 left a comment

Choose a reason for hiding this comment

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

How is this going to be used in CI?

@pavolloffay
Copy link
Member

How is this going to be used in CI?

The same question, what are we planning with the bundle image? Who is consuming it? Is the plan to publish is af part of the #1879 (cc) @iblancasa )

@IshwarKanse are you planning to use the bundle image?

@IshwarKanse
Copy link
Contributor

IshwarKanse commented Jul 3, 2023

@pavolloffay @jaronoff97 This PR adds steps to build the catalogsource image using the operator bundle image. We were planning to use #1823 as part of our CI in which we build the images on a nightly basis and update the index and use another job which runs e2e tests consumes the updated builds via the catalogsource. However we decided to add a job which builds the latest bundle, installs it using operator-sdk and use it to run e2e tests.

Considering our efforts to build and publish the operator bundle outlined in #1879, it would be advantageous to also build and publish the catalogsource image. CatalogSources are widely recognized and accepted as a distribution mechanism for operators, providing compatibility and allowing for usage in disconnected environments. By utilizing the catalogsource in conjunction with imageContentSourcePolicy, we can use mirrored images without altering the references to the operator images in the CSV. This approach offers a more streamlined and standardized installation and management process, as users can leverage the published CatalogSource index image instead of manually configuring and running the operator bundle with operator-sdk.

@pavolloffay pavolloffay merged commit 878a402 into open-telemetry:main Jul 3, 2023
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Added Bundle Image

Signed-off-by: Yuri Sa <[email protected]>

* Added Bundle Image

Signed-off-by: Yuri Sa <[email protected]>

---------

Signed-off-by: Yuri Sa <[email protected]>
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.

Add steps to build an index image containing operator bundle
6 participants