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

Graphite HTTP Request Error #385

Closed
sammcj opened this issue May 5, 2014 · 7 comments
Closed

Graphite HTTP Request Error #385

sammcj opened this issue May 5, 2014 · 7 comments

Comments

@sammcj
Copy link

sammcj commented May 5, 2014

My graphiteUrl is configured as:

graphiteUrl: "http://graphite.mydomain.com",

visiting this url returns the standard graphite API page of 'no data' which is correct if no parameters have been passed to it.

@torkelo
Copy link
Member

torkelo commented May 5, 2014

Do you not get the standard graphite web page? Or are you using graphite - api ?

@torkelo
Copy link
Member

torkelo commented May 5, 2014

Please show result from chrome network tab, so I can see what error graphite render request returns

@lefthand
Copy link

lefthand commented May 6, 2014

Hello, it appears I'm hitting the same error. When I access the page in the browser it comes up correctly. Here is the response I got from the network tab.

https://gist.github.com/lefthand/e024da2eeaece6edf881

I'm running grafana in a vagrant instance locally while the graphite host is in a real server.

@lefthand
Copy link

lefthand commented Jun 7, 2014

I updated that gist with a little more / clearer information. The error is:

Exception Value: 'NoneType' object has no attribute 'startswith'

I noticed that I'm able to edit queries and it pulls in all the correct data so I can construct queries but I still get the 500 error when rendering the actual graph.

@torkelo
Copy link
Member

torkelo commented Jun 7, 2014

@lefthand what is the graphite query you are trying this with? Graphite seems to think it is not a valid query.

@lefthand
Copy link

lefthand commented Jun 8, 2014

This happens with any query:

randomWalk('random walk')

@lefthand
Copy link

lefthand commented Jun 8, 2014

Thanks for looking at at this @torkelo (and thanks again for the shirt and Monitorama!)

I discovered that my graphite install only seems to like GET requests and not POST. Once I added

render_method: 'GET'

to my graphite settings that seems to have solved the problem for me. I tried added POST to Access-Control-Allow-Methods on my graphite apache config file but that didn't seem to help. I'd love to fully understand it but I'm happy for now.

@sammcj Does this solve your problem as well? Reading over issue #345 helped me sort it out.

@torkelo torkelo closed this as completed Jun 8, 2014
kminehart added a commit that referenced this issue Jul 15, 2022
* Fix XSS in runbook URL (#383)

* "Release: Updated versions in package to 8.3.8" (#385)

* Fix: Choose Lookup params per auth module v8.3.x (#401)

* Fix: Choose Lookup params per auth module

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: Prefer pointer to struct in lookup

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: user email for ldap

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: Use only login for lookup in LDAP

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: use user email for ldap

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

fix remaining test

fix nit picks

* update lock

* fix integration tests

* [v8.3.x] Merge 'release-8.3.9` branch (#404)

* Fix: Choose Lookup params per auth module

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: Prefer pointer to struct in lookup

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: user email for ldap

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: Use only login for lookup in LDAP

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

Fix: use user email for ldap

Co-authored-by: Karl Persson <kalle.persson@grafana.com>

fix remaining test

fix nit picks

(cherry picked from commit 09ca54d4665583b5648fea5726a19e0b58df4ec0)

* fix integration tests

(cherry picked from commit 16c3077bb39f41acacbef4b13c65c7c907c54de7)

* Release: Bump version to 8.3.9 (#403)

* Change bump-version.yml

* "Release: Updated versions in package to 8.3.9"

Co-authored-by: dsotirakis <dimitrios.sotirakis@grafana.com>

* Update yarn.lock

Co-authored-by: jguer <joao.guerreiro@grafana.com>
Co-authored-by: Grot (@grafanabot) <43478413+grafanabot@users.noreply.github.com>

* "Release: Updated versions in package to 8.3.10" (#409)

* Update grabpl

* update grabpl

* CI: Update `grabpl` version - remove `--no-pull-enterprise` flag (#47013)

* Update grabpl version

* Sign drone

* Remove --no-pull-enterprise flag

* Sign drone

* Update grabpl

* update drone, cherry-pick some ci updates

* update PR pipeline

Co-authored-by: George Robinson <george.robinson@grafana.com>
Co-authored-by: Grot (@grafanabot) <43478413+grafanabot@users.noreply.github.com>
Co-authored-by: Jguer <joao.guerreiro@grafana.com>
Co-authored-by: Dimitris Sotirakis <dimitrios.sotirakis@grafana.com>
Co-authored-by: Kevin Minehart <kmineh0151@gmail.com>
Co-authored-by: Dimitris Sotirakis <sotirakis.dim@gmail.com>
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

No branches or pull requests

3 participants