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

add cache configuration via file for query-frontend and store-gateway #1206

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

bueti
Copy link
Contributor

@bueti bueti commented Jan 4, 2021

Q A
Bug fix? no
New feature? yes
API breaks? no
Deprecations? no
Related tickets -
License Apache 2.0

What's in this PR?

Added ability to provide a cache configuration to thanos query-frontend and thanos store gateway. This enables caching via Memcached

Why?

Using Memcached can reduce load times of big queries greatly

Additional context

  • for easier navigation I moved all store and query-gateway related template files into their folder.
  • added artifacthub.io annotations

Checklist

  • Code meets the Developer Guide
  • User guide and development docs updated (if needed)

To Do

@bueti
Copy link
Contributor Author

bueti commented Jan 4, 2021

@wiardvanrij you asked about this in the other PR (#1195), would be great if you could test this. It works on our setup but would be good to get some "external" validation.

@bueti
Copy link
Contributor Author

bueti commented Jan 19, 2021

@tarokkk, @ahma would you have some time to look at this PR? 🙇

@bueti
Copy link
Contributor Author

bueti commented Feb 10, 2021

is this project dead?

@laghoule
Copy link

laghoule commented May 4, 2021

I'm interested in these feature, is there plan to add it somehow?

@bueti
Copy link
Contributor Author

bueti commented May 5, 2021

I'm interested in these feature, is there plan to add it somehow?

tbh I would not count on this chart to be updated anymore. We are running on our own fork but will migrate to the Thanos Chart from Bitnami

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.

2 participants