timosalm / tap-install

43 stars 22 forks source link

Update installation scripts to install all resources using Carvel tools #19

Closed emalm closed 2 years ago

emalm commented 2 years ago

Applying all additional resources using kapp deploy instead of kubectl apply makes them easier to track, to update, and to uninstall, and structures them to reveal more of the intent behind them.

Note: these changes add yq as a dependency in the scripts to simplify extraction of fields from the values.yaml files.

Additional changes:

emalm commented 2 years ago

FYI @alexbarbato

emalm commented 2 years ago

@tsalm-pivotal Thank you!

Also, my apologies for forgetting to mention that I reverted the TAP install-script behavior to use the default namespace for Grype to manipulate, and thank you for restoring the previous behavior in https://github.com/tsalm-pivotal/tap-install/commit/c4a0fce3e6b3d022be86ecbfc7675f865f10009a.