Skip to content

Commit

Permalink
Drop requirement for project_id field in cloud credentials
Browse files Browse the repository at this point in the history
  • Loading branch information
sd109 committed Dec 6, 2024
1 parent 4a096c8 commit 7f97230
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
5 changes: 0 additions & 5 deletions charts/openstack-cluster/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,16 +108,11 @@ For ease of use, this chart is written so that a `clouds.yaml` file can be given
to the chart as a configuration file. When an application credential is created in Horizon,
the corresponding `clouds.yaml` file can be downloaded, and should look something like this:

> [!WARNING]
> The Cluster API OpenStack provider currently requires that the `project_id` is present,
> which you will need to add manually.
```yaml
clouds:
openstack:
auth:
auth_url: https://my.cloud:5000
project_id: "<project id>"
application_credential_id: "<app cred id>"
application_credential_secret: "<app cred secret>"
region_name: "RegionOne"
Expand Down
3 changes: 0 additions & 3 deletions charts/openstack-cluster/templates/secret-cloud-config.yaml
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
{{- if not .Values.cloudCredentialsSecretName }}
{{- if .Values.clouds }}
{{- $cloud := index .Values.clouds .Values.cloudName }}
{{- if not (dig "auth" "project_id" nil $cloud) }}
{{- fail "clouds.yaml must contain the project ID" }}
{{- end }}
---
apiVersion: v1
kind: Secret
Expand Down

0 comments on commit 7f97230

Please sign in to comment.