UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Add labels to authservice namespace in compliance with charter

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

Summary

Authservice namespace doesn't have labels consistent with the charter an other namespaces

Relevant logs/screenshots

❯ k get ns --show-labels authservice
NAME          STATUS   AGE    LABELS
authservice   Active   3d2h   app.kubernetes.io/managed-by=Helm,istio-injection=enabled

/cc @micah.nagel

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
  • runyontr resolved all threads

    resolved all threads

  • runyontr mentioned in issue #417 (closed)

    mentioned in issue #417 (closed)

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel unapproved this merge request

    unapproved this merge request

  • Micah Nagel approved this merge request

    approved this merge request

  • LGTM, #417 (closed) can handle the follow up to clean up grouping.

  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel added 59 commits

    added 59 commits

    Compare with previous version

  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel added 1 commit

    added 1 commit

    Compare with previous version

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel enabled an automatic merge when the pipeline for 239bee8f succeeds

    enabled an automatic merge when the pipeline for 239bee8f succeeds

  • Micah Nagel canceled the automatic merge

    canceled the automatic merge

  • Micah Nagel unapproved this merge request

    unapproved this merge request

  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel added 1 commit

    added 1 commit

    Compare with previous version

  • LGTM, but I did the rebase / resolve of conflicts...want to make sure no one else seems something I missed.

  • Micah Nagel approved this merge request

    approved this merge request

  • merged

  • Micah Nagel mentioned in commit 68e125ce

    mentioned in commit 68e125ce

  • mentioned in issue #424 (closed)

  • evan.rush mentioned in issue #634 (closed)

    mentioned in issue #634 (closed)

  • Please register or sign in to reply
    Loading