search

thanos-community / thanos-operator

WIP: Operator to manage Thanos installation
Apache License 2.0
22 stars 3 forks source link

Add service monitor to Thanos Query #100

Closed coleenquadros closed 3 weeks ago

coleenquadros commented 1 month ago

@philipgough @saswatamcode its ready for review. I cannot add you as reviewers.

coleenquadros commented 3 weeks ago

@philipgough @saswatamcode I refactored the service monitor config to a struct.

For the integration tests I need to have a service monitor CRD definition installed as we are expecting service monitor to be enabled by default in the Thanos operator.

I need to also add a check to if the Service Monitor CRD is present in the cluster and only then proceed with creating and not block the operator. Which I will take care of in the next PR. Rebasing was not fun 😅