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

block thanos agent upgrades and suggest fedetl #2998

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

jessegoodier
Copy link
Collaborator

@jessegoodier jessegoodier commented Jan 27, 2024

What does this PR change?

Blocking thanos agents until 2.x supports multi-cluster prometheus.

Does this PR rely on any other PRs?

No

How does this PR impact users? (This is the kind of thing that goes in release notes!)

Thanos based agents should remain on 1.x versions until 2.x supports multi-cluster prometheus.

What risks are associated with merging this PR? What is required to fully test this PR?

NA

How was this PR tested?

--set agent=true

Error: UPGRADE FAILED: execution error at (cost-analyzer/templates/NOTES.txt:4:4): 

Kubecost 2.0 Does not support Thanos based agents. For Thanos, please continue to use 1.108.x.
Consider moving to Kubecost Federated ETL based agents.
Refer to the following documentation for more information: https://docs.kubecost.com/install-and-configure/install/kubecostv2
Support for Thanos agents is under consideration.
If you have any questions or concerns, please reach out to us at product@kubecost.com

Have you made an update to documentation? If so, please provide the corresponding PR.

Need to add statement in v2 doc.

@chipzoller
Copy link
Collaborator

Conflicts here.

@jessegoodier
Copy link
Collaborator Author

Conflicts here.

fixed

@jessegoodier jessegoodier merged commit eabd33b into develop Jan 29, 2024
12 checks passed
@jessegoodier jessegoodier deleted the block-agent-upgrades-20 branch January 29, 2024 16:21
@jessegoodier
Copy link
Collaborator Author

/cherry-pick v2.0

jessegoodier added a commit that referenced this pull request Jan 29, 2024
Co-authored-by: Jesse Goodier <31039225+jessegoodier@users.noreply.github.com>
ameijer pushed a commit that referenced this pull request Jan 31, 2024
…host-1141-yaml-actions

* commit '9abf05499eab352d9a10590c282e8e9491939804':
  updated release note (#3027)
  Bumped image tag with tighter prediction interval (#3023)
  Bumped image tag with retry changes (#3021)
  Bump thanosio/thanos from v0.32.5 to v0.34.0 in /cost-analyzer (#3015)
  Bump grafana/grafana from 10.2.3 to 10.3.1 in /cost-analyzer (#3013)
  Bump prometheus/alertmanager from v0.25.0 to v0.26.0 in /cost-analyzer (#2914)
  Bump prometheus-operator/prometheus-config-reloader in /cost-analyzer (#3014)
  Fix 2.0 precondition for nightly environment (#3016)
  Enable forecasting by default, fix "early spike" forecast behavior (#3010)
  [GTM-199] take global annotations setting into etlutils (#3006)
  Update to check if saml is enabled, not just if the object exists. (#3005)
  block thanos agent upgrades and suggest fedetl (#2998)
  Move 2.0 precondition checks into a function (#2986)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants