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

thanos: remove deprecated features and support for older versions #2629

Merged
merged 1 commit into from Jun 6, 2019
Merged

thanos: remove deprecated features and support for older versions #2629

merged 1 commit into from Jun 6, 2019

Conversation

abursavich
Copy link
Contributor

Thanos support is explicitly flagged as experimental in the documentation which states that it may change significantly without deprecation notice or backward compatibility in any release.

Thanos configuration has changed significantly throughout its early iterations. Most notably, object store configuration was replaced in v0.2.0 and gossip was deprecated in v0.4.0 (removed in v0.5.0). Retaining complete backwards compatibility in prometheus-operator for all version of thanos has unnecessarily complicated its API and implementation.

Per #2623 (comment), this drops support for older versions and simplifies the CRDs.

@abursavich
Copy link
Contributor Author

abursavich commented Jun 5, 2019

I'll bump the dependencies and DefaultThanosVersion to v0.5.0 once it's released, which may happen before or after this is merged.

cc @brancz

@brancz
Copy link
Contributor

brancz commented Jun 6, 2019

Thank you for the awesome work! Lgtm 👍

Copy link
Member

@metalmatze metalmatze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
Awesome, thanks!

@metalmatze metalmatze merged commit bbfed8c into prometheus-operator:master Jun 6, 2019
RiRa12621 pushed a commit to RiRa12621/prometheus-operator that referenced this pull request Jul 10, 2019
paulfantom added a commit to paulfantom/prometheus-operator that referenced this pull request Jul 22, 2019
* upstream/master:
  pkg/alertmanager: change podManagement policy to parallel to prevent statefulset reconciliation from hanging
  support KUBECONFIG environment
  fix namespaceSelector: any
  Adjust thanos doc accordinly to prometheus-operator#2629
  regenerate
  Add more recent prometheus versions to compatibility matrix
  make --always-make vendor
  Adding label selector for AlertManager objects discovery filtering
  Create docuementation for ingress monitoring
  pkg/alertmanager: increase terminationGracePeriod to 120
  Update Kubernetes client libraries to 1.15.0
  alertmanager/statefulset: move gossip port to 9094
yahaa pushed a commit to talcoding/prometheus-operator that referenced this pull request Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants