eraser icon indicating copy to clipboard operation
eraser copied to clipboard

add helm upgrade test to ci

Open sozercan opened this issue 3 years ago • 6 comments

Describe the solution you'd like after #108 is merged, test that installs helm chart using a specific version then upgrades to latest chart is manifest_staging

example: https://github.com/open-policy-agent/gatekeeper/blob/master/.github/workflows/upgrade.yaml

sozercan avatar May 09 '22 04:05 sozercan

@sozercan Should this run e2e tests, or just deploy the Helm chart then upgrade? Right now the e2e tests looked to be pretty reliant on a YAML deployment of Eraser.

If the upgrade is successful, should this workflow promote the helm chart from staging to the charts dir? Or will that be a manual step?

lee0c avatar May 18 '22 22:05 lee0c

Also, you can assign me or @joheec

lee0c avatar May 18 '22 22:05 lee0c

this is for testing if a PR breaks compatibility with a previous Helm chart version. Once we release a tagged release and helm chart is available in the helm repo, this PR test should:

  • deploy a chart from helm repo (#108), it's pinned to a specific version (base helm version that comes from Makefile). this is the version we want to be backwards compatible
  • optionally, run tests
  • upgrade the helm chart version using chart in manifest_staging in the PR
  • run tests

example: https://github.com/open-policy-agent/gatekeeper/blob/master/.github/workflows/upgrade.yaml

sozercan avatar May 19 '22 02:05 sozercan

@lee0c i also opened #199 for a regular helm test

sozercan avatar May 19 '22 15:05 sozercan

@sozercan Has a chart been published to the github pages repo yet? To fully build this out that is required.

lee0c avatar May 19 '22 20:05 lee0c

not yet, that will require a new release to be cut. #199 might be easier one to get started with

sozercan avatar May 19 '22 20:05 sozercan