rstudio / helm

Helm Resources for RStudio Products
MIT License
36 stars 28 forks source link

Move ci directory outside of the chart directory structure, and break it into two. One set of values for linting, one for installation testing #582

Closed jforest closed 1 month ago

jforest commented 1 month ago

The separation of values files into ci-lint and ci-install is required for now, because the current values files in the ci directory do not pass install tests. We need to create a full set of values files that both pass linting AND install tests. Once that is complete we can move those universal files back into ci and remove this silly process. https://github.com/rstudio/helm/issues/585

Update the workflow to create the symlinks as needed

I had to create some basic-values.yaml files for the install testing of the charts. The values added were the ones I needed in order for the charts to pass the install tests. Empty values didn't work for those ones: https://github.com/rstudio/helm/issues/587

closes #586

EDIT Also moving the values files out of the chart directories themselves. That way we can modify test values without having to increment the chart versions