Skip to content

Improve the default .gitlab-ci.yml

Hordur Freyr Yngvason requested to merge improve-default-gitlab-ci-yml into master

Several proposed improvements based on my experience migrating internal projects to a cluster management project template:

  • Remove the Helm 2 release detection. It adds noise and is unlikely to benefit anyone at this point.
  • Condense the documentation comment for the KUBE_CONTEXT variable.
  • Use sync instead of apply because it is more robust and aligns better with IaC principles (the code repository is the source of truth).
  • Add an explicit diff stage that runs on branches, and make the sync manual. That adds two degrees of safety: (1) in merge requests, we can review the proposed changes, and (2) have a second chance to review them prior to actually applying on the default branch.
Edited by Hordur Freyr Yngvason

Merge request reports