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

Generated Data in Block formate using promtool for now. #746

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

Conversation

kushalShukla-web
Copy link

@kushalShukla-web kushalShukla-web commented Sep 15, 2024

This pull request aims to confirm whether I am proceeding on the right Direction Specifically, it addresses the following:

  • Generation of data for PromBench.
  • Configuration of the Loadgen Querier to utilize the newly generated data.
    @bboreham @kakkoyun

Signed-off-by: Kushal Shukla <[email protected]>
@kushalShukla-web
Copy link
Author

kushalShukla-web commented Sep 17, 2024

Update 2.0.0

  1. In this PR, I have commented out the prometheus.yml file as it is not required at the moment.
  2. I have used hostPath volumes to retrieve data from the local system. Hi @bboreham, can we use Persistent Volumes instead of hostPath volumes ? but as the prometheus pods are running on a separate nodes so if pods terminated still they are going to be created on the same node .
  3. Currently, the path is hardcoded for my system. I need to make it more dynamic (I tried but did not came up with the answer) or use Persistent Volumes instead.
  4. Hi @bboreham i need little bit your guidance here .

@bboreham
Copy link
Member

Thanks for this.

I don't think we want large amounts of binary data in the git repo. We can set up a Google Cloud block volume and put the data there, or maybe a Google Cloud Storage bucket.

Your hostPath volume is fine to illustrate; depending on the point above we will mount the Google Cloud volume or not.

I don't think we want large amounts of config commented out either.

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