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

internal/plugins: move go/v2-alpha to its own package #3181

Merged
merged 1 commit into from
Jun 18, 2020

Conversation

estroz
Copy link
Member

@estroz estroz commented Jun 5, 2020

Description of the change:

  • cmd/operator-sdk/cli: update import path for Go plugin
  • go.mod,go.sum: bump kubebuilder version

Motivation for the change: the go.operator-sdk.io/v2-alpha plugin is now in its own package. See kubernetes-sigs/kubebuilder#1550 for a reference as to why this was done.

/cc @camilamacedo86 @hasbro17 @joelanford

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 13, 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 Jun 18, 2020
@estroz
Copy link
Member Author

estroz commented Jun 18, 2020

cmd/operator-sdk/cli: update import path for Go plugin

go.mod,go.sum: bump kubebuilder version
@@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package golang
package v2
Copy link
Contributor

@camilamacedo86 camilamacedo86 Jun 18, 2020

Choose a reason for hiding this comment

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

Can we keep golang and add v2?
See that we will have helm and ansible as well so the golang name make clear.

Copy link
Member Author

Choose a reason for hiding this comment

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

The package path contains golang.

Copy link
Contributor

Choose a reason for hiding this comment

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

far enough

Copy link
Contributor

@camilamacedo86 camilamacedo86 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 Jun 18, 2020
@estroz estroz changed the title internal/plugins: move go/v2-alpha to its own package. internal/plugins: move go/v2-alpha to its own package Jun 18, 2020
@estroz estroz merged commit 6dac68b into operator-framework:master Jun 18, 2020
@estroz estroz deleted the refactor/add-plugin-v2 branch June 18, 2020 17:23
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.

None yet

3 participants