Skip to content

Document stance on static IP addresses #1740

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

Closed
ncdc opened this issue Nov 8, 2019 · 15 comments
Closed

Document stance on static IP addresses #1740

ncdc opened this issue Nov 8, 2019 · 15 comments
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/documentation Categorizes issue or PR as related to documentation. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/backlog Higher priority than priority/awaiting-more-evidence.

Comments

@ncdc
Copy link
Contributor

ncdc commented Nov 8, 2019

User Story

As a user/operator, I would like to understand Cluster API's stance on static IP addresses so that I can understand if it will affect my ability to use Cluster API in my environment.

Detailed Description

We should document that individual infrastructure providers may, at their discretion, add support for static IP address assignment at the individual machine level, but, when you move up to higher-level abstractions such as MachineSet, MachineDeployment, KubeadmControlPlane, there is no support for customizing each replica. If someone wanted to use static IPs with these abstractions, there would need to be a coordinated effort between the infrastructure provider and some sort of IPAM controller (outside the scope of Cluster API itself).

/kind documentation
/priority important-soon
/milestone v0.3.0

cc @akutz

@k8s-ci-robot k8s-ci-robot added this to the v0.3.0 milestone Nov 8, 2019
@k8s-ci-robot k8s-ci-robot added kind/documentation Categorizes issue or PR as related to documentation. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Nov 8, 2019
@akutz
Copy link
Contributor

akutz commented Nov 8, 2019

While it may be implied/inferred, it also (I think) bears explicit mention in such documentation that the upgrade capabilities for MachineDeployments and the upcoming control plane upgrades won't work without DHCP or some sort of IPAM (as @ncdc mentioned above).

@ncdc
Copy link
Contributor Author

ncdc commented Dec 20, 2019

cc @rosskukulinski

@ncdc ncdc self-assigned this Jan 10, 2020
@ncdc ncdc added priority/backlog Higher priority than priority/awaiting-more-evidence. and removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Jan 22, 2020
@sheriefhammam
Copy link

A relevant observation
I haven't debugged it yet but for some reason i can assign static IPs to worker nodes but it affects somehow the control plane

apiVersion: infrastructure.cluster.x-k8s.io/v1alpha2
kind: VSphereMachineTemplate
metadata:
  name: workload-cluster-02-md-0
  namespace: default
spec:
  template:
    spec:
      datacenter: NIC-Darmstadt
      diskGiB: 20
      memoryMiB: 2048
      network:
        devices:
        - dhcp4: false
          dhcp6: false
          gateway4: 10.100.146.1
          ipAddrs:
          - 10.100.146.50/24
          networkName: ds146
      numCPUs: 2
      template: T_centos-7-kube-v1.16.3.a4
kubectl get nodes -o wide
NAME                                        STATUS   ROLES    AGE   VERSION   INTERNAL-IP     EXTERNAL-IP     OS-IMAGE                KERNEL-VERSION               CONTAINER-RUNTIME
workload-cluster-02-controlplane-0          Ready    master   36m   v1.16.3   10.100.146.50   10.100.146.50   CentOS Linux 7 (Core)   3.10.0-1062.9.1.el7.x86_64   containerd://1.3.0
workload-cluster-02-md-0-5b7df745f6-bgs2z   Ready    <none>   35m   v1.16.3   10.100.146.84   10.100.146.84   CentOS Linux 7 (Core)   3.10.0-1062.9.1.el7.x86_64   containerd://1.3.0

@vincepri
Copy link
Member

/help
/milestone Next

@k8s-ci-robot
Copy link
Contributor

@vincepri:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help
/milestone Next

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot modified the milestones: v0.3.0, Next Feb 12, 2020
@k8s-ci-robot k8s-ci-robot added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Feb 12, 2020
@luniHw
Copy link

luniHw commented Feb 16, 2020

Issue:
Used capv 0.4.2 and centos-7-kube-v1.15.6 to create centos k8s cluster on nimbus env, it failed to assign static ip, a dynamical ip is assigned to vm. (Nimbus env, some ips are dhcp, some ips are static ips)

Help:
I need to upgrade capv the version or do others? thanks

@MnrGreg
Copy link

MnrGreg commented Feb 16, 2020

@luniHw static IP assignment in CAPV is fixed with: kubernetes-sigs/cluster-api-provider-vsphere#707 but there hasn't been a CAPV release since.

@yastij any idea on timing for the next release?

@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 16, 2020
@vincepri
Copy link
Member

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 19, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 17, 2020
@fabriziopandini
Copy link
Member

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 17, 2020
@vincepri
Copy link
Member

/lifecycle frozen

@k8s-ci-robot k8s-ci-robot added the lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. label Aug 18, 2020
@vincepri
Copy link
Member

/close

Cluster API doesn't do ip management today, this is left for infrastructure providers

@k8s-ci-robot
Copy link
Contributor

@vincepri: Closing this issue.

In response to this:

/close

Cluster API doesn't do ip management today, this is left for infrastructure providers

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jessehu
Copy link
Contributor

jessehu commented Jul 30, 2022

update: the IPAM Proposal is added in CAPI 1.2.0:
#6000
#6313

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/documentation Categorizes issue or PR as related to documentation. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Development

No branches or pull requests

10 participants