Skip to content

Bump chart version to 5.8.0 on gstg and pre for mailroom

John Skarbek requested to merge jts/ingress-confuse into master

What does this MR do?

For gitlab-com/gl-infra/production#6344 (closed)

This MR bumps chart to 5.8.0 on gstg and pre. The main target of this bump is MailRoom webhook delivery method

Although the number of file changes is huge, they are automatically updated by the following steps, as guided in README.md:

Due to this upgrade, it was noted that KAS was receiving a bad Ingress configuration, therefore the latest revision of this MR brings in gitlab-org/charts/gitlab!2385 (merged) with the goal of preventing an incident for the KAS service.

Due to this, we are setting the ...kas.nginx-ingress.class to a value of none. This is the recommendation by distribution from issue: gitlab-org/charts/gitlab#3130 (closed) Therefore, we've set this value to each environment to be explicit. This can later be sent back to the main values file after the chart is upgraded in all environments.

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