Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate Helm Integration Testing Strategy #49

Open
sabre1041 opened this issue Apr 12, 2022 · 1 comment
Open

Consolidate Helm Integration Testing Strategy #49

sabre1041 opened this issue Apr 12, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@sabre1041
Copy link
Contributor

Currently, new capabilities have been added to test the Helm charts associated with operators. Included to facilitate the testing are a series of supporting Helm charts. These charts have been included in each of the operator repositories.

In order to simplify the configuration and to consolidate the work, the following enhancements can be implemented:

  1. Move the charts from individual repositories and store them within the CoP Helm Charts repository
  2. Update each chart to reference the remote repository
@sabre1041 sabre1041 added the enhancement New feature or request label Apr 12, 2022
@trevorbox
Copy link
Contributor

trevorbox commented Jun 16, 2022

Parameterize the ingress object to support any port needed per integration test (https://github.com/redhat-cop/vault-config-operator/blob/main/integration/helm/ingress-nginx/templates/ingress-vault.yaml#L1-L16), and publish the chart in CoP Helm Charts repository. Keep the curl request in individual Makefiles for kind ingress-nginx, but standardize it across the operators (see https://github.com/redhat-cop/vault-config-operator/blob/main/Makefile#L112).

Secondly, a chart for prometheus https://github.com/redhat-cop/vault-config-operator/tree/main/integration/helm/prometheus-rbac with a dependency on https://prometheus-community.github.io/helm-charts should also get published to CoP Helm Charts repository

@sabre1041 this sound right? anything else?

@trevorbox trevorbox self-assigned this Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants