Open
Conversation
Contributor
|
I like query_explorer.py (and agree with your todo's for it) |
It'd be good if we could be nice and trim with our queries since it makes it easier to debug, but g-e is using POST to get the actual data so it's not that relevant.
Author
|
Query explorer now loads the proper configuration and I tidied up build_from_targets a little. I think that's enough to merge, but there's a few things which could be nice to consider in other patches:
|
Contributor
|
i'm sorry i've been dropping the ball on this. i've been so busy. |
Author
|
No problem. If you have some things that you think I might be able to do to make it easier then let me know. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a "first thing that works" implementation for a query
percent by <tag>as discussed in #121 which will result in graph targets roughly meaning (forpercent by tag2):This also works when you do something like
percent by type sum by core, you get a metric like:Additionally I added a utility to introspect the process of generating a query and give a very simple graphite url for testing purposes. Could come in handy for other people.
I expect this pull request needs some refactoring but I wanted to create it early so you can get an idea of where I went with it.
A couple of things I intend to fix:
I think in general
build_from_targetswould be easier to change if it had a bit more of an object language, rather than just dicts everywhere. In particular I found it quite hard to work out if I was computing something which had already been done elsewhere.