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

Shard spaces (rebased variant of PR #23) #40

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

Conversation

Civil
Copy link

@Civil Civil commented Feb 10, 2015

Sorry, I've lost the repo where it was and I don't know how to rebase PR from deleted repo, so I've decided to create new one.

@Civil
Copy link
Author

Civil commented Feb 10, 2015

Please note, that I don't have InfluxDB setup right now, so I was unable to test the code. The only tests I've performed - run pylint and check that there are no syntax errors.

@Dieterbe
Copy link
Contributor

@liyichao you were a big fan of the previous PR, could you test this? i'm super busy right now :(

@liyichao
Copy link

I setup a test environment, where I use another docker image with Civil's pull request, but it reports 500 when I direct grafana to use that as a backend.

graphite-api's log seems useless, so I try to run app.py manually using Flask's app.run(debug=True),
but failed to do so because #41

Also, influxdb 0.9's shard spaces will be changed, which I'd like to wait for before setuping retention policies.

Currently, influxdb can not have multiple graphite-servers making it difficult to set up a totally separate testing environment for me. Because we use a forever shard space without any down-sampling and retention policies, what I can test is also limited.

@liyichao
Copy link

@Civil I use graphite-api and here is my schema:
schema:
- ['', [['', 10, '30d'],['10_mins_rollup.', 600, '700d']]]

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.

4 participants