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

feat: add helm plugin scaffolding new layout #3421

Merged
merged 1 commit into from
Jul 15, 2020
Merged

feat: add helm plugin scaffolding new layout #3421

merged 1 commit into from
Jul 15, 2020

Conversation

camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented Jul 14, 2020

Description of the change:
This PR adds the helm plugin scaffolding the files with the new layout and using the current sdk helm pkg implementation (image)

  • added e2e tests for helm new layout. See:/test/e2e-helm-new/e2e_suite.go and ./hack/tests/e2e-helm-new.sh
  • centralize the method ReplaceInFile in the test/internal/utils.go since it is useful for all tests
  • Impl e2e tests in shell for the new layout. See hack/tests/e2e-helm.sh

Motivation

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.

This PR adds the Helm Plugin which will provide the Helm project in the new layout by the command operator-sdk init --plugins=helm.operator-sdk.io/v1`. For a further understanding see the WIP: #329 and the doc Extensible CLI and Scaffolding Plugins.

Follow-ups

@camilamacedo86
Copy link
Contributor Author

Started in; #3295 ( I close to test the CI in my fork and I could not re-open )

hack/tests/e2e-helm-new.sh Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@camilamacedo86 camilamacedo86 requested review from asmacdo and estroz July 14, 2020 21:12
hack/tests/e2e-helm.sh Outdated Show resolved Hide resolved
Copy link
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 15, 2020
hack/tests/e2e-helm.sh Outdated Show resolved Hide resolved
@joelanford joelanford added this to the v1.0.0 milestone Jul 15, 2020
@camilamacedo86
Copy link
Contributor Author

camilamacedo86 commented Jul 15, 2020

/retest

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 15, 2020
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 15, 2020
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 15, 2020
Copy link
Member

@joelanford joelanford left a comment

Choose a reason for hiding this comment

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

/lgtm

🎉 🎉 🎉

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 15, 2020
@camilamacedo86 camilamacedo86 merged commit 0fe8cf4 into operator-framework:master Jul 15, 2020
@camilamacedo86 camilamacedo86 deleted the helm-external-plugin branch July 15, 2020 21:34
@joelanford joelanford mentioned this pull request Jul 16, 2020
92 tasks
camilamacedo86 added a commit that referenced this pull request Jul 18, 2020
**Description of the change:**

Provide the same documentation and steps using the helm plugin. See the PR #3421 which adds the plugin. 

- Update all related info in the Helm documentation  
- Create a new sub-directory called Legacy to keep the previous docs

**Motivation**

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](https://github.com/kubernetes-sigs/kubebuilder/blob/master/designs/integrating-kubebuilder-and-osdk.md) and [Extensible CLI and Scaffolding Plugins](https://github.com/kubernetes-sigs/kubebuilder/blob/master/designs/extensible-cli-and-scaffolding-plugins-phase-1.md).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants