scaletest: Move Prometheus to addon nodes - #18048
Conversation
|
/approve |
|
/retest |
|
/test pull-kops-gce-master-scale-performance-100 |
810625b to
37ade34
Compare
|
/test pull-kops-gce-master-scale-performance-100 |
37ade34 to
581810b
Compare
|
Heh, looks like kops never create addons node. That's why the test passed :( |
22f590a to
cfc2596
Compare
|
Ok, found the error |
cfc2596 to
b8b29d5
Compare
|
Prometheus logs |
|
Still getting Don't know why |
|
Direct proxy API also works |
2dbf6c2 to
4a3596a
Compare
|
/retest |
|
For some reason it picked old commit 9d4a1281e038e65e85caffa20e76c2d67870edfc instead of 9ab1dc73e51a5cdbfada3d2437f4d4972e59aee9 that includes kubernetes/perf-tests#3885 |
|
/test pull-kops-gce-master-scale-performance-100 |
|
It didn't work. Still we don't know why CL2 cannot connect to Prometheus and there are no logs. Close to giving up. |
4a3596a to
05535d9
Compare
|
/test pull-kops-gce-master-scale-performance-100 |
|
@serathius @upodroid anything else left to do here? |
|
Yeah. Let's test the aws one /test pull-kops-gce-master-scale-performance-100 |
|
Oops wrong one /test pull-kops-ec2-master-scale-performance-100 |
|
Can we skip AWS? release blocking tests are on GCE, the AWS is a followup. |
AWS is missing firewall rules to allow Prometheus to scrape control plane.
05535d9 to
f7c4cf6
Compare
Yes |
| # this is used as a label to select kube-proxy pods on kops for kube-proxy service | ||
| # used by CL2 Prometheus here https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/pkg/prometheus/manifests/default/kube-proxy-service.yaml#L2 | ||
| export PROMETHEUS_KUBE_PROXY_SELECTOR_KEY="k8s-app" | ||
| export PROMETHEUS_SCRAPE_APISERVER_ONLY="true" |
There was a problem hiding this comment.
FYI PROMETHEUS_SCRAPE_APISERVER_ONLY doesn't exist on CL2 side.
|
You can't because the aws jobs are release-informing https://testgrid.k8s.io/sig-release-master-informing#ec2-master-scale-performance If we merge it, we'll need to fix it within a day or two. |
|
By skipping AWS I didn't mean break their testing. Just skip fixing addon nodes. |
|
/retest |
|
/test pull-kops-ec2-master-scale-performance-100 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman, upodroid The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |


/cc @mborsz @upodroid @hakman
With #18036 we can move prometheus to addons node.