Skip to content
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.

go-modules: fix build, module name and local imports #12

Merged

Conversation

neolit123
Copy link
Contributor

@neolit123 neolit123 commented Jun 22, 2019

fix the build in current master:

  • update module to github.com/kubernetes-sigs/cluster-api-provider-docker
  • modify github.com/chuckha/cluster-api-provider-docker imports to
    github.com/kubernetes-sigs/cluster-api-provider-docker/kind/actions
    run 'go mod tidy'
  • rename DeleteClusterNode to DeleteNode
    the existing functions don't have Cluster in the name:
    DeleteControlPlane, DeleteWorker

/kind bug
/priority backlog
/assign @chuckha

@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jun 22, 2019
@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 22, 2019
Copy link

@pablochacin pablochacin left a comment

Choose a reason for hiding this comment

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

Would prefer two separated PR as this addresses two unrelated issues. But if others agree in this PR as is, I won't block it for merging.

@neolit123
Copy link
Contributor Author

updated OP to include the reason for the rename.
without it the build is broken for kind-test.

@neolit123 neolit123 changed the title go-modules: fix module name and local imports go-modules: fix build, module name and local imports Jun 22, 2019
Copy link
Contributor

@chuckha chuckha left a comment

Choose a reason for hiding this comment

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

There's a PR out for the DeleteNode fix

I also swear I thought I modified all the imports but git don't lie!

Thanks for this fix, just undo the Delete changes and it's good to go

kind/actions/cluster_actions.go Outdated Show resolved Hide resolved
- update module to github.com/kubernetes-sigs/cluster-api-provider-docker
- modify github.com/chuckha/cluster-api-provider-docker imports to
github.com/kubernetes-sigs/cluster-api-provider-docker/kind/actions
run 'go mod tidy'
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 24, 2019
@neolit123
Copy link
Contributor Author

updated to exclude the Delete* rename.

Copy link
Contributor

@chuckha chuckha left a comment

Choose a reason for hiding this comment

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

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chuckha, neolit123

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 24, 2019
@ashish-amarnath
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 24, 2019
@k8s-ci-robot k8s-ci-robot merged commit 10b854d into kubernetes-retired:master Jun 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. priority/backlog Higher priority than priority/awaiting-more-evidence. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants