Skip to content

Commit

Permalink
Merge pull request #591 from rstudio/test-more-install-values
Browse files Browse the repository at this point in the history
Test the license-file-secret-values linting file with basic-values.yaml additions
  • Loading branch information
jforest authored Oct 18, 2024
2 parents 8e994aa + c0d1bcb commit b578ab9
Show file tree
Hide file tree
Showing 4 changed files with 29 additions and 11 deletions.
22 changes: 17 additions & 5 deletions .github/workflows/chart-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -130,26 +130,38 @@ jobs:
- name: Create kind cluster
uses: helm/[email protected]
if: ${{ steps.list-changed.outputs.changed == 'true' || github.ref == 'refs/heads/main' }}

- name: Install SealedSecrets Helm Chart
if: ${{ steps.list-changed.outputs.changed == 'true' || github.ref == 'refs/heads/main' }}
run: |
helm repo add sealed-secrets https://bitnami-labs.github.io/sealed-secrets
helm install sealed-secrets sealed-secrets/sealed-secrets
- name: Create posit-test namespace
run: kubectl create namespace posit-test

- name: Create License File Secrets
run: |
echo "${{ secrets.PWB_LICENSE_FILE }}" > pwb.lic
kubectl create secret generic pwb-license --from-file=pwb.lic --namespace posit-test
rm pwb.lic
echo "${{ secrets.PCT_LICENSE_FILE }}" > pct.lic
kubectl create secret generic pct-license --from-file=pct.lic --namespace posit-test
rm pct.lic
echo "${{ secrets.PPM_LICENSE_FILE }}" > ppm.lic
kubectl create secret generic ppm-license --from-file=ppm.lic --namespace posit-test
rm ppm.lic
# no allow-failure until https://github.com/actions/toolkit/issues/399
- name: Run chart-testing (install changed)
id: ct-install
if: ${{ github.ref != 'refs/heads/main' && steps.list-changed.outputs.changed == 'true' }}
run: ct install --target-branch main --chart-dirs charts --chart-dirs other-charts --excluded-charts rstudio-library
run: ct install --target-branch main --chart-dirs charts --chart-dirs other-charts --excluded-charts rstudio-library --namespace posit-test
continue-on-error: true

# no allow-failure until https://github.com/actions/toolkit/issues/399
- name: Run chart-testing (install all)
id: ct-install-all
if: ${{ github.ref == 'refs/heads/main' }}
run: ct install --target-branch main --all --chart-dirs charts --chart-dirs other-charts --excluded-charts rstudio-library
run: ct install --target-branch main --all --chart-dirs charts --chart-dirs other-charts --excluded-charts rstudio-library --namespace posit-test
continue-on-error: true

- name: Notify Slack of chart install failure
Expand Down
6 changes: 4 additions & 2 deletions ci/rstudio-connect/install/basic-values.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
readinessProbe:
enabled: false
license:
file:
secret: pct-license
secretKey: pct.lic
6 changes: 4 additions & 2 deletions ci/rstudio-pm/install/basic-values.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
readinessProbe:
enabled: false
license:
file:
secret: ppm-license
secretKey: ppm.lic
6 changes: 4 additions & 2 deletions ci/rstudio-workbench/install/basic-values.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
readinessProbe:
enabled: false
license:
file:
secret: pwb-license
secretKey: pwb.lic

0 comments on commit b578ab9

Please sign in to comment.