UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Moved the excluded namespaces to the new regex method for excluded resources.

Merged Eric Goode requested to merge opa-add-regexp into master
All threads resolved!

Merge Request

No package changes. Scope is in Big Bang only.

Excluded namespaces converted to excluded resources where the exclusion could be narrowed down to namespace and pod/container name using regex.

Values added to k3d-dev-values.yaml and ci/k3d/values.yaml to accommodate load balancer pod/container exceptions.

Additional Details

For testing, all packages should be deployed and no denies should occur. Use kubectl get events -n <namespace> to check for denies.

closes #693 (closed)

closes #776 (closed)

Edited by Michael McLeroy

Merge request reports

Loading
Loading

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
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • Micah Nagel
  • @egoode a lot of comments but they all boil down to the same couple of things:

    • make sure our conditionals are on the proper packages (mostly fluentbit, these were pre-existing issues but we should clean them up while we're modifying these values)
    • make sure that we're using the excludedResources as intended: the goal is to restrict our exceptions down further than just the namespace. As is, most of these just add a whole namespace exception in a different way.

    What I'd suggest doing to get the pieces:

    • remove the exceptions
    • deploy the offending packages
    • view the errors that happen (typically a describe on the deployments/daemonsets)
    • the errors will output something like "gatekeeper denied this-resource-name" or "gatekeeper denied this-resource-name-xhyd"
    • use that name as the excludedResources, sometimes we can do an exact match and sometimes we need regex due to the random strings.
  • removed statusreview label

  • Eric Goode added 71 commits

    added 71 commits

    Compare with previous version

  • Michael McLeroy added 53 commits

    added 53 commits

    Compare with previous version

  • added 1 commit

    • 2cccdbc0 - fix(gatekeeper): narrowed exclusions

    Compare with previous version

  • added 1 commit

    • ac191ab8 - fix(gatekeeper): node exporter missing dot

    Compare with previous version

  • Michael McLeroy changed the description

    changed the description

  • Michael McLeroy added 28 commits

    added 28 commits

    Compare with previous version

  • added 1 commit

    • 27d633fc - fix(gatekeeper): fluentbit conditional updated

    Compare with previous version

  • Michael McLeroy resolved all threads

    resolved all threads

  • added 1 commit

    • 7859b52e - docs: updated development env

    Compare with previous version

  • Michael McLeroy approved this merge request

    approved this merge request

  • Michael McLeroy added 3 commits

    added 3 commits

    Compare with previous version

  • Michael McLeroy approved this merge request

    approved this merge request

  • Michael McLeroy enabled an automatic merge when the pipeline for 36d16708 succeeds

    enabled an automatic merge when the pipeline for 36d16708 succeeds

  • Michael McLeroy mentioned in commit e3eb8bbf

    mentioned in commit e3eb8bbf

  • Please register or sign in to reply
    Loading