Skip to content

Check in chart dependencies and stop calling dep updates in CI

DJ Mountney requested to merge checkin-chart-dependencies into master

What does this MR do?

Checks in the chart dependencies so they only get pulled in via git, and won't require all our CI jobs to keep reaching out to external repos.

I don't like checking these in with the chart repo source code, but if we go with an alternative mirror, we would also want to mirror the mirror so that gitlab.com wasn't depending on itself. And to get this working today, it just makes more sense to allow the existing git mirroring to do the work for us.

Followups (#3334):

  • Add back the dependency update, but for the dependencies update pipeline
  • Document the process of manually updating and cheching in the chart bundles when updated the requirements.lock
  • Discuss alternative approaches of mirroring

Related issues

Fixes #2980 (closed)

And related to incident: gitlab-com/gl-infra/production#7165 (closed)

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

  • Merge Request Title and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes
  • Documentation created/updated
  • Tests added
  • Integration tests added to GitLab QA
  • Equivalent MR/issue for omnibus-gitlab opened
Edited by Mitchell Nielsen

Merge request reports