Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The operator should check that the Thanos sidecar's version supports the --prometheus.http-client flag #5223

Closed
kssanthosh opened this issue Dec 11, 2022 · 8 comments · Fixed by #6448

Comments

@kssanthosh
Copy link

What happened?
When applying the manifest after generating from thanos sidecar.sonnet

error parsing commandline arguments: [/bin/thanos sidecar --prometheus.url=http://localhost:9090/ --prometheus.http-client={"tls_config": {"insecure_skip_verify":true}} --grpc-address=:10901 --http-address=:10902 --objstore.config=type: s3
config:
bucket: prometheus-data
endpoint: s3.us-east-5.amazonaws.com
access_key: xxxxx
secret_key: xxxx--tsdb.path=/prometheus]: unknown long flag '--prometheus.http-client'
thanos: error: unknown long flag '--prometheus.http-client'

Did you expect to see some different?

How to reproduce it (as minimally and precisely as possible):

Environment
Kops 1.25

  • Prometheus Operator version:

    v0.61.1

  • Kubernetes version information:

    Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.0"

  • Kubernetes cluster kind:

    insert how you created your cluster: kops, bootkube, tectonic-installer, etc.

  • Manifests:

insert manifests relevant to the issue
  • Prometheus Operator Logs:
Insert Prometheus Operator logs relevant to the issue here
  • Prometheus Logs:
Insert Prometheus logs relevant to the issue here

Anything else we need to know?:

@Sindvero
Copy link

Sindvero commented Dec 11, 2022

Same error here with the thanos sidecar jsonnet file that generated the k8s manifests for k8s v1.24.6 deployed with kops

@simonpasquier
Copy link
Contributor

what is the version of the Thanos image?

@Sindvero
Copy link

thanos version: v0.8.1

@simonpasquier
Copy link
Contributor

@Sindvero this version is way too old, you need at least v0.24.0. Having said we need a version check in prometheus-operator.

@simonpasquier simonpasquier transferred this issue from prometheus-operator/kube-prometheus Dec 13, 2022
@simonpasquier simonpasquier changed the title thanos sidecar not working The operator should check that the Thanos sidecar's version supports the --prometheus.http-client flag Dec 13, 2022
@mouad-eh
Copy link
Contributor

mouad-eh commented Mar 9, 2024

I will work on this

@MateSousa
Copy link
Contributor

hey @mouad-eh , are you working on this??

@mouad-eh
Copy link
Contributor

@MateSousa
Unfortunately, I didn't have time to work on it. Maybe I can take a look at it in the weekend.
If you are intrested you can go head and work on it. You don't need to ask for assignement from the maintainers, just showing interest is enough.

@MateSousa
Copy link
Contributor

@mouad-eh thanks, I will work on this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants