bitops icon indicating copy to clipboard operation
bitops copied to clipboard

helm copy default before and after scripts

Open mickmcgrath13 opened this issue 4 years ago • 5 comments

https://github.com/bitovi/bitops/blob/master/scripts/helm/copy-defaults.sh

should copy default/helm/<chart>/bitops.before-deploy.d and default/helm/<chart>/bitops.after-deploy.d

mickmcgrath13 avatar Dec 28 '20 23:12 mickmcgrath13

Two things;

  1. When does default/helm/*charts*/bitops.<before|after>-deploy.d get used? I haven't seen this pattern before..
  2. Does this line in the copy-defaults.sh file not recursively copy anything in the OpsRepo chart?

PhillypHenning avatar Sep 10 '21 16:09 PhillypHenning

https://github.com/bitovi/bitops/pull/126

PhillypHenning avatar Sep 10 '21 16:09 PhillypHenning

When does default/helm/charts/bitops.<before|after>-deploy.d get used? I haven't seen this pattern before..

similar to before/after scripts for ansible. they get used during the deployment of a chart.

here: https://github.com/bitovi/bitops/blob/master/scripts/helm/helm_handle_chart.sh#L33 and here: https://github.com/bitovi/bitops/blob/master/scripts/helm/helm_handle_chart.sh#L102

Does this line in the copy-defaults.sh file not recursively copy anything in the OpsRepo chart?

that's for subcharts (i.e. opsrepo/<env>/helm/chartA/charts) which is different from before/after scripts (i.e. opsrepo/<env>/helm/chartA/bitops.(before|after)-deploy.d

mickmcgrath13 avatar Sep 15 '21 13:09 mickmcgrath13

@shyamrayaprolu I believe you fixed this in the V2 plugin?

PhillypHenning avatar Jul 13 '22 22:07 PhillypHenning

@PhillypHenning - yes, this is fixed

shyamrayaprolu avatar Jul 13 '22 22:07 shyamrayaprolu