Skip to content

doc: add migration guide for helm new layout#3342

Merged
camilamacedo86 merged 3 commits intooperator-framework:masterfrom
camilamacedo86:doc-helm-new-layout-migration
Jul 8, 2020
Merged

doc: add migration guide for helm new layout#3342
camilamacedo86 merged 3 commits intooperator-framework:masterfrom
camilamacedo86:doc-helm-new-layout-migration

Conversation

@camilamacedo86
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 commented Jul 3, 2020

Description of the change:

  • Add doc with the steps required that need be followed by the users migrate their project for the new layout.

Motivation for the change:
SDK is in a process to be integrated with KB which means that its project layouts will be aligned. More info : Integrating Kubebuilder and Operator SDK.

As part of this process, we are creating the Helm plugin. See PR #3295. Also, we are updating the steps in the doc in the PR #3326 and here we are adding a new doc which will help users to migrate their projects.

@camilamacedo86 camilamacedo86 added language/helm Issue is related to a Helm operator project kind/documentation Categorizes issue or PR as related to documentation. kubebuilder-integration Relates to rewriting the SDK in Kubebuilder plugin form labels Jul 3, 2020
@camilamacedo86 camilamacedo86 reopened this Jul 3, 2020
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/hold not merge before the new helm plugin be merged
However, we can review it already.

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 6, 2020
Copy link
Copy Markdown
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2020
Comment thread website/content/en/docs/helm/migration.md Outdated
…t public to users


Removed the header for we are able to merge this doc without making it public to users and then, make it visible in the PR #3326
@openshift-ci-robot
Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 8, 2020
@camilamacedo86 camilamacedo86 removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 8, 2020
@camilamacedo86 camilamacedo86 merged commit b7bc77b into operator-framework:master Jul 8, 2020
@camilamacedo86 camilamacedo86 deleted the doc-helm-new-layout-migration branch July 8, 2020 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/documentation Categorizes issue or PR as related to documentation. kubebuilder-integration Relates to rewriting the SDK in Kubebuilder plugin form language/helm Issue is related to a Helm operator project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants