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

[Proposal] Add support to OLM bundle format #703

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

Conversation

varshaprasad96
Copy link

This commit intends to add a proposal to introduce support for installing OLM bundles in registry+v1 format to kapp-controller.

This commit intends to add a proposal to introduce
support for installing OLM bundles in registry+v1 format
to kapp-controller.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
Copy link

netlify bot commented Nov 15, 2023

Deploy Preview for carvel canceled.

Name Link
🔨 Latest commit 6a788ff
🔍 Latest deploy log https://app.netlify.com/sites/carvel/deploys/6555465e43b2440008277712

@varshaprasad96
Copy link
Author

cc: @ncdc

Copy link
Member

@praveenrewar praveenrewar left a comment

Choose a reason for hiding this comment

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

Could we also add a desired workflow for users to be able to install operators?

}
```

With the help of the following inputs, kapp-controller can then convert the registry+v1 contents into a set of plain kubernetes manifests.
Copy link
Member

Choose a reason for hiding this comment

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

I think a little bit more details need to be added on how kapp-controller can convert this into plain Kubernetes manifests.

Copy link
Author

Choose a reason for hiding this comment

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

@praveenrewar The conversion between OLM's registryV1 to plain k8s manifest is based on a yaml named "ClusterServiceVersion" - CSV (https://olm.operatorframework.io/docs/concepts/crds/clusterserviceversion/). There is a direct 1 on 1 translation - which means that based on the info given specified in a CSV yaml (like deployment spec and permissions) we create individual ClusterRoles, service accounts, Bindings, Deployments etc.

@varshaprasad96
Copy link
Author

varshaprasad96 commented Nov 30, 2023

@praveenrewar The e2e workflow remains the same as it is now with any other bundle using kapp-ctrl with the introduction of this templating step. With this change, we intend to enable the installation of operators using an OLM specific format. That being said, kapp-ctrl as is can still deploy operators - if it is packaged using Helm chart, or just through a bunch of plain k8s manifests.

@renuy
Copy link
Contributor

renuy commented Mar 5, 2024

@varshaprasad96 ,
Are you still working on this, or can we close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants