kubermatic / mla

MLA (Monitoring, logging, alerting) solution for KKP.
Apache License 2.0
8 stars 9 forks source link

fix mla consul #70

Closed SimonTheLeg closed 2 years ago

SimonTheLeg commented 2 years ago

This PR solves the failing CI for https://github.com/kubermatic/kubermatic/pull/8189

It has been tested locally with a single-node cluster and should also work with the e2e tests smaller cluster.

❯ kc get nodes
NAME                     STATUS   ROLES                  AGE    VERSION
mla-test-control-plane   Ready    control-plane,master   3h2m   v1.21.1
❯ kc get pods
NAME                     READY   STATUS    RESTARTS   AGE
consul-consul-server-0   1/1     Running   0          20m
consul-consul-server-1   1/1     Running   0          20m
consul-consul-server-2   1/1     Running   0          21m

It has also been tested on the seed-dev cluster

❯ kc get pods | grep consul
consul-consul-server-0                             1/1     Running     0          9m26s
consul-consul-server-1                             1/1     Running     0          9m26s
consul-consul-server-2                             1/1     Running     0          9m26s

Additionally I also removed the consul client, as it is not needed for our mla use-case

kubermatic-bot commented 2 years ago

LGTM label has been added.

Git tree hash: 4f9ba2b2a91a6a6619a521bc6b8cec16e4346937

kubermatic-bot commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jiachengxu, SimonTheLeg

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubermatic/mla/blob/main/OWNERS)~~ [jiachengxu] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment