UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Resolve "Update Gitlab to work with Istio sidecar injection"

Merged kevin.wilder requested to merge 60-istio-sidecar-injection into main

Add istio sidecar injection. To make istio injection work, annotations are conditionally added to certain jobs (init containers) to disable injection. The annotations are only added if istio is enabled.
All changes are backwards compatible with BigBang. No breaking changes because istio injection is already disabled by default.
Note that some annotations can be added by chart values (migrations and shared-secrets) and are therefore set in the BigBang template. Some init jobs (minio-create-buckets and upgrades) cannot be configured with chart values. Those are implemented by modifying the gitlab sub-charts.
Note: the gitlab issue #60 will be closed by the BigBang MR. Istio injection for Gitlab will not actually happen until the BigBang MR gets merged.

Also, bump resources for gitaly
closes #106 (closed)

Edited by kevin.wilder

Merge request reports

Merge request pipeline #547695 passed with warnings

Merge request pipeline passed with warnings for c65f424d

Merged by kevin.wilderkevin.wilder 3 years ago (Oct 29, 2021 8:30pm UTC)

Merge details

  • Changes merged into main with e58ea3d2 (commits were squashed).
  • Deleted the source branch.

Pipeline #547703 passed with warnings

Pipeline passed with warnings for e58ea3d2 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading