Skip to content

Commit

Permalink
Add Provider Network Support (#16)
Browse files Browse the repository at this point in the history
Sadly, CAPO isn't allowed to create "non-standar" network types, so we
need to provision one externally and pass it in.  Hopefully, according
to the code, it'll add the L3 subnet and router for us.
  • Loading branch information
spjmurray authored Aug 21, 2024
1 parent 81802e0 commit 1acc2ea
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/cluster-api-cluster-openstack/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@ apiVersion: v2
name: cluster-api-cluster-openstack
description: A Helm chart to deploy a Kubernetes Cluster
type: application
version: v0.5.1
version: v0.5.2
icon: https://raw.githubusercontent.com/unikorn-cloud/helm-cluster-api/main/icons/default.png
2 changes: 1 addition & 1 deletion charts/cluster-api-cluster-openstack/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ spec:
source:
repoURL: https://unikorn-cloud.github.io/helm-cluster-api
chart: cluster-api-cluster-openstack
targetRevision: v0.5.1
targetRevision: v0.5.2
helm:
releaseName: foo
# Remove the default work queue.
Expand Down
4 changes: 4 additions & 0 deletions charts/cluster-api-cluster-openstack/templates/cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,10 @@ spec:
{{- end }}
managedSecurityGroups:
allowAllInClusterTraffic: true
{{- if .Values.network.networkID }}
network:
id: {{ .Values.network.networkID }}
{{- end }}
managedSubnets:
- cidr: {{ .Values.network.nodeCIDR }}
dnsNameservers: {{ toJson .Values.network.dnsNameservers }}
Expand Down
5 changes: 4 additions & 1 deletion charts/cluster-api-cluster-openstack/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,10 @@
},
"dnsNameservers": {
"$ref": "#/$defs/nonEmptyIPV4List"
}
},
"networkID": {
"type": "string"
}
}
}
}
Expand Down
5 changes: 5 additions & 0 deletions charts/cluster-api-cluster-openstack/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -188,3 +188,8 @@ network:
# DNS nameservers to use.
dnsNameservers:
- 8.8.8.8

# If specified defines an existing nwtork to use, if not defined a network
# will be created by CAPO. This allows you to use non-standard network
# types e.g. a VLAN to be used for baremetal nodes.
# networkID: 8f526b54-fab3-435d-b4b3-f65fd8474b8a

0 comments on commit 1acc2ea

Please sign in to comment.