UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

identify namespaces that dont use istio via label. Enable istio in twistlock

Merged runyontr requested to merge istio-namespace-labels into master
All threads resolved!

Summary

Identify each applications use of istio

also enable it in Twistlock

Merge request reports

Merge request pipeline #345328 passed

Merge request pipeline passed for 24e81c77

Approved by

Merged by Michael McLeroyMichael McLeroy 3 years ago (Aug 11, 2021 1:01pm UTC)

Merge details

  • Changes merged into master with 234c3674 (commits were squashed).
  • Deleted the source branch.

Pipeline #418688 passed

Pipeline passed for 234c3674 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel added 1 commit

    added 1 commit

    • 24e81c77 - Apply 3 suggestion(s) to 3 file(s)

    Compare with previous version

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel unapproved this merge request

    unapproved this merge request

    • Resolved by Michael McLeroy

      Before this can be merged, we need to make sure that none of these will be denied by OPA Gatekeeper due to "namespacesHaveIstio". That constraint matches namespaces without the label "admission.gatekeeper.sh/ignore". So, either that label needs to be present or the namespace needs to be ignored in the constraint.

  • Will merge this at the beginning of next sprint. Istio injection policy is not set to deny yet.

  • Michael McLeroy resolved all threads

    resolved all threads

  • Michael McLeroy changed milestone to %1.14.0

    changed milestone to %1.14.0

  • mentioned in issue #611 (closed)

  • Michael McLeroy approved this merge request

    approved this merge request

  • Michael McLeroy mentioned in commit 234c3674

    mentioned in commit 234c3674

  • Please register or sign in to reply
    Loading