Skip to content

Commit

Permalink
propagate additional tags from awsCluster to ebs-csi-driver (#509)
Browse files Browse the repository at this point in the history
  • Loading branch information
Berk Dehrioglu authored Feb 11, 2024
1 parent c193a00 commit 75389c2
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Added

- Add propagating tags from `cluster-aws` to resources managed my `ebs-csi-driver`.
- CI: trigger automated e2e tests on Renovate PRs.
- Add new annotation for vintage irsa domain which is only used for migrating vintage clusters.
- Use 443 as the default api-server Load Balancer port.
Expand Down
7 changes: 7 additions & 0 deletions helm/cluster-aws/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -371,3 +371,10 @@ sts.amazonaws.com{{ if hasPrefix "cn-" (include "aws-region" .) }}.cn{{ end }}
{{- define "awsNoProxyList" }}
- {{ $.Values.global.connectivity.network.vpcCidr }}
{{- end }}

{{- define "resource.default.additionalTags" -}}
{{- if .Values.global.providerSpecific.additionalResourceTags }}
{{ toYaml .Values.global.providerSpecific.additionalResourceTags }}
{{- end }}
giantswarm.io/cluster: {{ include "resource.default.name" $ }}
{{- end -}}
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{{/* Default Helm values for the app */}}
{{/* See schema for the appropriate app version here https://github.com/giantswarm/aws-ebs-csi-driver-app/blob/master/helm/aws-ebs-csi-driver-app/values.schema.json */}}
{{- define "defaultAwsEbsCsiDriverHelmValues" }}
extraVolumeTags: {{ include "resource.default.additionalTags" . | nindent 2 }}
{{- if (.Values.global.connectivity.proxy).enabled }}
proxy:
noProxy: "{{ include "cluster.connectivity.proxy.noProxy" (dict "global" $.Values.global "providerIntegration" $.Values.cluster.providerIntegration) }}"
Expand Down

0 comments on commit 75389c2

Please sign in to comment.