starlingx/monitoring
Code Issues Proposed changes

Add sriov-fec-system namespace to the platform infra list in collectd

Browse Source 
Currently the pods installed in the sriov-fec-operator namespace run
on application cores. The sriov-fec-operator App is seen
as a platform app and therefore its pods need to run on platform
cores.

Accordingly, add the sriov-fec-system namespace to the list of
platform namespaces in collectd.

Test Plan:
PASS: Execute kube-memory tool and ensure its output contains fec
      namespace data.
PASS: Edit collectd tool source code to display the list of namespaces
      classified as K8S_NAMESPACE_SYSTEM and verify if the
      sriov-fec-operator namespace was found.

Story: 2010826
Task: 48639

Change-Id: I85a07e7a30018b28ea49e96f0100294d40ce4433
Signed-off-by: Marcos Paulo Oliveira Silva <Marcos.PauloOliveiraSilva@windriver.com>
This commit is contained in:
Marcos Paulo Oliveira Silva 2023-08-16 15:16:56 -03:00
parent 0d1ac162dc
commit ccb106deec
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
collectd-extensions/src/plugin_common.py
View File

@ -105,7 +105,7 @@ BASE_GROUPS_EXCLUDE = [CGROUP_K8S, CGROUP_MACHINE]
K8S_NAMESPACE_SYSTEM = ['kube-system', 'armada', 'cert-manager', 'portieris',
'vault', 'notification', 'platform-deployment-manager',
'flux-helm', 'metrics-server', 'node-feature-discovery',
'intel-power', 'power-metrics']
'intel-power', 'power-metrics', 'sriov-fec-system']
K8S_NAMESPACE_ADDON = ['monitor', 'openstack']
# Pod parent cgroup name based on annotation.

2
kube-memory/kube-memory/kube_memory/kube_memory.py
View File

@ -43,7 +43,7 @@ BASE_GROUPS = ['docker', 'system.slice', 'user.slice']
K8S_NAMESPACE_SYSTEM = ['kube-system', 'armada', 'cert-manager', 'portieris',
'vault', 'notification', 'platform-deployment-manager',
'flux-helm', 'metrics-server', 'node-feature-discovery',
'intel-power', 'power-metrics']
'intel-power', 'power-metrics', 'sriov-fec-system']
K8S_NAMESPACE_ADDON = ['monitor', 'openstack']
# Used commands