Metrics server running on application CPU cores

Bug #1964503 reported by Thiago Oliveira Miranda
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Thiago Oliveira Miranda

Bug Description

Brief Description
-----------------
Metrics Server is running on application CPU cores.

Severity
--------
Minor - this is more of a performance/optimization issue; not a functional issue

Steps to Reproduce
------------------
- Apply label kube-cpu-mgr-policy=static
- install metrics-server
- Check if it is running on platform CPU cores or application CPU cores

Expected Behavior
------------------
metrics-server should execute only on platform cores

Actual Behavior
----------------
metrics-server shifting on application and platform cores

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

System Configuration
--------------------
any - not specific to any system configuration

Branch/Pull Time/Commit
-----------------------
stx master

Timestamp/Logs
--------------
Not needed. Issue is easily reproducible.

Test Activity
-------------
Other - Evaluation

Workaround
----------
N/A

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (master)

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

Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to monitoring (master)

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

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

Reviewed: https://review.opendev.org/c/starlingx/integ/+/833150
Committed: https://opendev.org/starlingx/integ/commit/636e5352d9445080fce66de14673e90804f4e6e1
Submitter: "Zuul (22348)"
Branch: master

commit 636e5352d9445080fce66de14673e90804f4e6e1
Author: Thiago Miranda <email address hidden>
Date: Thu Mar 10 12:04:30 2022 -0500

    Add metrics-server to list of platform namespaces

    Testing:
    * Deployed ISO with changes.
    * Configured kube-cpu-mgr-policy=static.
    * Verified that metrics-server were running on platform CPUs.

    Partial-Bug: 1964503

    Signed-off-by: Thiago Miranda <email address hidden>
    Change-Id: I9bebe9ec27fcd70e89a4cae52bfacde993f958eb

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (master)

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

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

Reviewed: https://review.opendev.org/c/starlingx/monitoring/+/833153
Committed: https://opendev.org/starlingx/monitoring/commit/42eebe5025e7c81fd747c90b60723203d9822294
Submitter: "Zuul (22348)"
Branch: master

commit 42eebe5025e7c81fd747c90b60723203d9822294
Author: Thiago Miranda <email address hidden>
Date: Thu Mar 10 12:24:06 2022 -0500

    Add metrics-server to list of platform namespaces

    Partial-Bug: 1964503
    Depends-On: https://review.opendev.org/c/starlingx/integ/+/833150

    Signed-off-by: Thiago Miranda <email address hidden>
    Change-Id: I9d2517e9cfff1f7da27651bab1473bcfc1a2667b

Ghada Khalil (gkhalil)
Changed in starlingx:
assignee: nobody → Thiago Oliveira Miranda (thiagooliveiramiranda)
tags: added: stx.apps
Changed in starlingx:
importance: Undecided → Low
tags: added: stx.7.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to integ (master)

Reviewed: https://review.opendev.org/c/starlingx/integ/+/834215
Committed: https://opendev.org/starlingx/integ/commit/9b3c3b55664933407d41c18999dd3f01ac4289f0
Submitter: "Zuul (22348)"
Branch: master

commit 9b3c3b55664933407d41c18999dd3f01ac4289f0
Author: Thiago Miranda <email address hidden>
Date: Thu Mar 17 14:44:35 2022 -0400

    Add metrics-server to list of platform namespaces

    Testing:
    * Deployed ISO with changes.
    * Configured kube-cpu-mgr-policy=static.
    * Verified that metrics-server were running on platform CPUs.

    Partial-Bug: 1964503

    Signed-off-by: Thiago Miranda <email address hidden>
    Change-Id: I447e4ccc113a4d0cc34a73bd71ac961305987c06

Revision history for this message
Ghada Khalil (gkhalil) wrote :

Are further changes required for this LP? If not, please mark it as Fix Released.

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/starlingx/integ/+/849157
Committed: https://opendev.org/starlingx/integ/commit/500adad67ee2da00b94767c9eb284127f752cd7f
Submitter: "Zuul (22348)"
Branch: master

commit 500adad67ee2da00b94767c9eb284127f752cd7f
Author: Kaustubh Dhokte <email address hidden>
Date: Fri Jul 8 23:20:13 2022 +0000

    Debian: Add metrics-server to platform namespaces

    This change https://review.opendev.org/c/starlingx/integ/+/834215
    adds metrics-server to the list of platform namespaces for
    k8s 1.23.1. Apparently, Debian package for k8s 1.23.1 was not
    added when above change was created.

    Note: The patch was copied from centos/files.

    Test Plan:
    Pass: Package builds successfully
    PASS: Image built and deployed successfully.
    PASS: Apply metrics-server and verify that metrics-server is
          running on platform CPUs.

    Closes-Bug: 1964503

    Signed-off-by: Kaustubh Dhokte <email address hidden>
    Change-Id: I989be27416f388dc0ff46a820b8fa3a276600737

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.