Skip to content

Fix `check-vendored-charts.sh` to actually work

Graeme Gillies requested to merge ggillies/fix-check-vendored-charts.sh into master

What does this MR do?

Part of gitlab-com/gl-infra/delivery#2147 (closed)

The script in question had flawed logic where it was pushd into the temp directory and running from there, meaning it was diffing everything against the same copy (not the copy in this repo). This mean that the script would always succeed, and thus not be useful.

This change fixes it to work correctly (and fixes an issue found when testing the vendored chart for gitlab-runner).

Author Check-list

Please read the Contributing document and once you do, complete the following:

  • Assign to the correct reviewer per the contributing document
  • Apply the correct metadata per the contributing document
  • Link to related MRs for applying the changes on other environments
  • Link to related Chef changes
  • If necessary link to a Criticality 4 Change Request issue

Reviewer Check-list

  • Reviewed the diff jobs to confirm changes are as expected
  • No changes shown in the diffs not associated with this MR - This may require a rebase or further investigation

Applier Check-list

  • Make sure there is no ongoing deployment for the affected envs before merging (see #announcements slack channel)
Edited by John Skarbek

Merge request reports

Loading