From f8650e4ddf0225695653314d48bb2ee3bdf112f7 Mon Sep 17 00:00:00 2001 From: Nuru Date: Tue, 9 Jul 2024 17:17:57 -0700 Subject: [PATCH] [eks/actions-runner-controller] Fix misconfigured document separators in Helm chart template (#1077) --- modules/eks/actions-runner-controller/CHANGELOG.md | 13 ++++++++++++- .../charts/actions-runner/Chart.yaml | 2 +- .../actions-runner/templates/runnerdeployment.yaml | 3 ++- modules/eks/karpenter-node-pool/CHANGELOG.md | 4 +++- modules/eks/karpenter/CHANGELOG.md | 4 +++- 5 files changed, 21 insertions(+), 5 deletions(-) diff --git a/modules/eks/actions-runner-controller/CHANGELOG.md b/modules/eks/actions-runner-controller/CHANGELOG.md index d0ca5cd41..d3c2cc338 100644 --- a/modules/eks/actions-runner-controller/CHANGELOG.md +++ b/modules/eks/actions-runner-controller/CHANGELOG.md @@ -1,4 +1,15 @@ -## PR [#1075](https://github.com/cloudposse/terraform-aws-components/pull/1075) +## Release 1.470.1 + +Components PR [#1077](https://github.com/cloudposse/terraform-aws-components/pull/1077) + +Bugfix: + +- Fix templating of document separators in Helm chart template. Affects users who are not using + `running_pod_annotations`. + +## Release 1.470.0 + +Components PR [#1075](https://github.com/cloudposse/terraform-aws-components/pull/1075) New Features: diff --git a/modules/eks/actions-runner-controller/charts/actions-runner/Chart.yaml b/modules/eks/actions-runner-controller/charts/actions-runner/Chart.yaml index 95f7916b1..d4a340fa3 100644 --- a/modules/eks/actions-runner-controller/charts/actions-runner/Chart.yaml +++ b/modules/eks/actions-runner-controller/charts/actions-runner/Chart.yaml @@ -15,7 +15,7 @@ type: application # This is the chart version. This version number should be incremented each time you make changes # to the chart and its templates, including the app version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 0.3.0 +version: 0.3.1 # This chart only deploys Resources for actions-runner-controller, so app version does not really apply. # We use Resource API version instead. diff --git a/modules/eks/actions-runner-controller/charts/actions-runner/templates/runnerdeployment.yaml b/modules/eks/actions-runner-controller/charts/actions-runner/templates/runnerdeployment.yaml index 27077abae..dbdfd1a84 100644 --- a/modules/eks/actions-runner-controller/charts/actions-runner/templates/runnerdeployment.yaml +++ b/modules/eks/actions-runner-controller/charts/actions-runner/templates/runnerdeployment.yaml @@ -12,6 +12,7 @@ we explicitly convert to boolean based on the string value */}} We keep the logic here in case we need to revert to the sidecar option. */}} {{- $use_dind_in_runner := $use_dind }} {{- if $use_pvc }} +--- # Persistent Volumes can be used for image caching apiVersion: v1 kind: PersistentVolumeClaim @@ -95,8 +96,8 @@ data: printf "Annotated pod at %s with annotations:\n '%s'\n" "$AT" '{{ . | toJson }}' fi ---- {{ end }} +--- apiVersion: actions.summerwind.dev/v1alpha1 kind: RunnerDeployment metadata: diff --git a/modules/eks/karpenter-node-pool/CHANGELOG.md b/modules/eks/karpenter-node-pool/CHANGELOG.md index 2a110392c..c8402e80e 100644 --- a/modules/eks/karpenter-node-pool/CHANGELOG.md +++ b/modules/eks/karpenter-node-pool/CHANGELOG.md @@ -1,4 +1,6 @@ -## Components [PR #1076](https://github.com/cloudposse/terraform-aws-components/pull/1076) +## Release 1.470.0 + +Components PR [#1076](https://github.com/cloudposse/terraform-aws-components/pull/1076) - Allow specifying elements of `spec.template.spec.kubelet` - Make taint values optional diff --git a/modules/eks/karpenter/CHANGELOG.md b/modules/eks/karpenter/CHANGELOG.md index 6d7a8f2d2..bd5b3bc24 100644 --- a/modules/eks/karpenter/CHANGELOG.md +++ b/modules/eks/karpenter/CHANGELOG.md @@ -1,4 +1,6 @@ -## Components [PR #1076](https://github.com/cloudposse/terraform-aws-components/pull/1076) +## Release 1.470.0 + +Components PR [#1076](https://github.com/cloudposse/terraform-aws-components/pull/1076) #### Bugfix