Check for Microshift - Githubissues

cloud-bulldozer / k8s-netperf

Running Networking Performance Tests against K8s

Apache License 2.0

28 stars 17 forks source link

Check for Microshift #128

Closed jtaleric closed 8 months ago

jtaleric commented 8 months ago

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] Optimization
[ ] Documentation Update

Description

Attempting to fix prom queries w/ microshift

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

sjug commented 8 months ago

It seems that OpenShift only relabels the node_cpu_seconds_total query and that requires the check in QueryNodeCPU() however the container_cpu_usage_seconds_total instance isn't relabeled and has the IP address instead.

I will just change my MicroShift prometheus config to use IP addresses only rather than the hostname. Appreciate the support @jtaleric!