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

Initial changes to addon-package docs #15

Merged
merged 2 commits into from
Jul 18, 2023

Conversation

maryfrances01
Copy link
Contributor

MTSRE-890

@maryfrances01 maryfrances01 changed the title DRAFT: initial changes to addon-package docs Initial changes to addon-package docs Jul 11, 2023
@maryfrances01 maryfrances01 marked this pull request as ready for review July 11, 2023 17:59
[merge request](https://gitlab.cee.redhat.com/service/managed-tenants-bundles/-/tree/main/addons/reference-addon/package)
for an example.

2. The MTB CI creates the packageImage and the Operator Lifecycle Manager (OLM) Index Image as part of the team's
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is the best example to describe how the MTB CI creates the packageImage and OLM Index Image?

[package operator](https://package-operator.run/docs/guides/packaging-an-application/#go-templates)
documentation for more information.

## Tenants Onboarding Steps
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What is the connection between the ClusterPackage manifest and Tenant Onboarding? Put another way, how do we go from having a ClusterPackage manifest to onboarding steps?

@maryfrances01
Copy link
Contributor Author

@garrettthomaskth I have a couple of questions for clarification. Thanks!

@maryfrances01 maryfrances01 self-assigned this Jul 11, 2023
Copy link
Member

@apahim apahim left a comment

Choose a reason for hiding this comment

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

/lgtm

@maryfrances01 maryfrances01 removed the request for review from garrettthomaskth July 18, 2023 14:52
@maryfrances01 maryfrances01 merged commit 04afd07 into main Jul 18, 2023
2 checks passed
@maryfrances01 maryfrances01 deleted the mhull-mtsre-890-addon-package-docs branch July 18, 2023 14:52
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