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

fix(thanos) Enable standalone thanos query deployment #567

Merged
merged 13 commits into from
Feb 5, 2025

Conversation

trouaux
Copy link
Contributor

@trouaux trouaux commented Jan 9, 2025

Pull Request Details

Enable standalone thanos-query deployment

Adding deployment conditions to store and compactor components
and fixing tests accordingly

Proper use of .Values.thanos.query.standalone option

! CI will be fixed in thanos ruler PR !
#534

Breaking Changes

Issues Fixed

Other Relevant Information

@trouaux trouaux requested review from a team as code owners January 9, 2025 09:52
@trouaux trouaux changed the title fix(thanos) add conditions for store and compactor testing + deploy fix(thanos) Enable standalone thanos query deployment Jan 9, 2025
Copy link
Contributor

@viennaa viennaa left a comment

Choose a reason for hiding this comment

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

Thanks for taking this. Some small things inside.

thanos/charts/templates/store-svc.yaml Outdated Show resolved Hide resolved
thanos/charts/values.yaml Show resolved Hide resolved
thanos/charts/values.yaml Show resolved Hide resolved
thanos/charts/Chart.yaml Outdated Show resolved Hide resolved
Copy link
Member

@richardtief richardtief left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks.

Copy link
Member

@richardtief richardtief left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks.

Copy link
Member

@richardtief richardtief left a comment

Choose a reason for hiding this comment

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

Looks good, thanks. You need to update the plugindefinition.yaml accordingly.

@trouaux
Copy link
Contributor Author

trouaux commented Jan 30, 2025

Looks good, thanks. You need to update the plugindefinition.yaml accordingly.

done

Copy link
Member

@richardtief richardtief left a comment

Choose a reason for hiding this comment

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

All good. Thanks.

@richardtief
Copy link
Member

This PR needs to be rebased onto #614

Copy link

⚠️ Chart oci://ghcr.io/cloudoperators/greenhouse-extensions/charts/thanos:0.1.5 already exists in OCI registry. Please increment the chart version.

@trouaux trouaux requested a review from viennaa February 3, 2025 08:07
Copy link
Contributor

@viennaa viennaa left a comment

Choose a reason for hiding this comment

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

Thank you and let's go!

Copy link

github-actions bot commented Feb 5, 2025

⚠️ Chart oci://ghcr.io/cloudoperators/greenhouse-extensions/charts/thanos:0.1.7 already exists in OCI registry. Please increment the chart version.

Copy link

github-actions bot commented Feb 5, 2025

⚠️ Chart oci://ghcr.io/cloudoperators/greenhouse-extensions/charts/thanos:0.1.7 already exists in OCI registry. Please increment the chart version.

@trouaux trouaux merged commit 9fa8e3b into main Feb 5, 2025
4 checks passed
@trouaux trouaux deleted the thanos-compactor-store-tests-fix branch February 5, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants