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

Add node ranges config in InfrastructureConfig #82

Open
defo89 opened this issue Nov 21, 2024 · 1 comment
Open

Add node ranges config in InfrastructureConfig #82

defo89 opened this issue Nov 21, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@defo89
Copy link
Contributor

defo89 commented Nov 21, 2024

Summary

Current Shoot .spec.networking.nodes is a string which means only one CIDR can be configured.

However, in on-prem networking there are case where worker nodes reside in different subnets which are not easy to summarise.

We as provider extension can allow configuring array of CIDRs in InfrastructureConfig and update InfrastructureStatusNetworking Nodes field which will be later updated to Shoot status and used as reference e.g. in Calico's IP_AUTODETECTION_METHOD.

@defo89 defo89 added the enhancement New feature or request label Nov 21, 2024
@defo89 defo89 self-assigned this Nov 21, 2024
@defo89
Copy link
Contributor Author

defo89 commented Nov 22, 2024

Proposal is to follow aws extension approach and add the following to metal Infrastructure:

apiVersion: extensions.gardener.cloud/v1alpha1
kind: Infrastructure
metadata:
  name: infrastructure
  namespace: shoot--foobar--metal
spec:
  providerConfig:
    apiVersion: aws.provider.extensions.gardener.cloud/v1alpha1
    kind: InfrastructureConfig
    networks:
    - name: pod-a
      workers: 10.250.10.0/24
      vlan: 500
    - name: pod-b
      workers: 10.250.11.0/24
      vlan: 501

This will:

  • today: update ranges in Infrastructure.status.networking.nodes, so that is copied to Shoot.status.networking.nodes.
  • later: trigger some network API to provision required networks.

These networks could also be referenced in WorkerConfig to signal that worker pool needs IP assignment from respective IP address pool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

1 participant