PSAC labels missing from flux-helm namespace

Bug #1995717 reported by Jerry Sun
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Jerry Sun

Bug Description

Brief Description
-----------------
flux-helm namespace was added after the PSAC feature and as a result, the flux-helm namespace did not have any PSAC labels

Severity
--------
Minor: System/Feature is usable with minor issue

Steps to Reproduce
------------------
Bootstrap any system and run "kubectl describe namespace flux-helm"

Expected Behavior
------------------
PSAC labels exist for flux-helm namespace

Actual Behavior
----------------
PSAC labels missing

Reproducibility
---------------
Reproducible

System Configuration
--------------------
multi node system

Branch/Pull Time/Commit
-----------------------
2022-11-04

Jerry Sun (jerry-sun-u)
Changed in starlingx:
assignee: nobody → Jerry Sun (jerry-sun-u)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to config (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/863650

Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ansible-playbooks (master)
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Low
tags: added: stx.8.0 stx.config stx.containers
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/c/starlingx/config/+/863650
Committed: https://opendev.org/starlingx/config/commit/fe13e1f8cf12143cc4591c6bae885ad2dd4ff2b2
Submitter: "Zuul (22348)"
Branch: master

commit fe13e1f8cf12143cc4591c6bae885ad2dd4ff2b2
Author: Jerry Sun <email address hidden>
Date: Fri Nov 4 10:07:46 2022 -0400

    Add missing PSAC labels to flux-helm namespace

    Pod Security Admission Controller (PSAC)
    labels were missing from flux-helm namespace. This commit
    adds PSAC labels to the flux-helm namespace. Privileged and latest
    were chosen to keep consistent with existing labels with other
    namespaces.

    PASS: Bootstrap system, check that flux-helm namespace has the same
          PSAC labels as other namespaces like kube-system and
          cert-manager

    Change-Id: I72f21f03db39c7879626a083bd1e0827c80aad7d
    Partial-Bug: 1995717
    Signed-off-by: Jerry Sun <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ansible-playbooks (master)

Reviewed: https://review.opendev.org/c/starlingx/ansible-playbooks/+/863651
Committed: https://opendev.org/starlingx/ansible-playbooks/commit/9f79d4e0ef35c2a1c30ffcfc362cb7f0441b4b67
Submitter: "Zuul (22348)"
Branch: master

commit 9f79d4e0ef35c2a1c30ffcfc362cb7f0441b4b67
Author: Jerry Sun <email address hidden>
Date: Fri Nov 4 10:05:46 2022 -0400

    Add missing PSAC labels to flux-helm namespace

    Pod Security Admission Controller (PSAC)
    labels were missing from flux-helm namespace. This commit
    adds PSAC labels to the flux-helm namespace. Privileged and latest
    were chosen to keep consistent with existing labels with other
    namespaces.

    Test Cases:

    PASS: Bootstrap system, check that flux-helm namespace has the same
          PSAC labels as other namespaces like kube-system and
          cert-manager

    Change-Id: I3c0db98b13f1a73a61698032d4b7a1d48d1ca4f9
    Closes-Bug: 1995717
    Signed-off-by: Jerry Sun <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.