-
Notifications
You must be signed in to change notification settings - Fork 82
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
Can't create folder on Grafana 7.3.7 stable (tried 7.1.3 as well, same issue) #147
Comments
Tested locally through docker and integration tests.
And the folder is properly created.
I'm failing at reproducing @gacallea. Can you post the output of a curl command on |
@rrey it returns only this:
|
well that's unexpected ... |
do I need to enable anything api related in the grafana.ini perhaps? while this is unexpected every other ansible grafana task works. I'm puzzled. |
Not all modules checks the version. Only the one that are addressing feature introduced after a specific version. |
I'm investigating and apparently one needs to be authenticated to get more info from the health api. If I run the curl locally or visit the page, it only returns the dashboard. I'll try authenticating the request and come back to you. That being said, it's weird because the tasks should clearly be authenticated with url_user and url_password. Regarding the compatible version, I tried downgrading to 7.1.3 because the readme on community grafana lists it as supported. Same issues. |
Ok, I found the settings. Under [auth.anonymous] there's a hide_version = true that will not output version and commit. I set this not to disclose info to non-authenticated users for security reasons, and tools should probably not rely on this info as many could choose to hide this info as well as a best practice. However, I have enabled it to debug the issue and now it returns the following:
Also, I tried running the tasks now that this is in place and:
EDIT: it works as expected with the configuration change to hide_version = false I still think this tools should not rely on such brittle piece of info that people may want to hide for security reasons. |
perhaps a possible solution could be for grafana_folder to actually use url_user and url_password to use authentication when checking the health api for version instead of relying on non auth queries? |
The request already use the the authentication headers to access this endpoint. |
sounds like a plan :) should I open a new request for that to happen? you take care of it? thank you for all your help, I appreciate you. |
@gacallea I'll keep the version check but I'll add a parameter that allow to skip the control. |
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
Parameter allows to skip version check and execute module anyway. Closes: #147
thank you so much @rrey!!! I appreciate you :) about using it, will the documentation update automatically (like with a doc generator from code?) an example would be nice otherwise. please and thank you |
ps: this should work?
|
Doc update is in the hand of ansible team. |
it worked fine :) (I re-enabled the hide version thingie) |
SUMMARY
Trying to create "named folders" and upload "named dashboards" to them.
ISSUE TYPE
COMPONENT NAME
community.grafana.grafana_folder
ANSIBLE VERSION
CONFIGURATION
OS / ENVIRONMENT
Ubuntu 20.04
Grafana 7.3.7 stable -- installed from the official Grafana repository (oss stable)
STEPS TO REPRODUCE
Posting the relevant tasks only. All variables work in other tasks. The upload to /tmp/dashboard is omitted here.
EXPECTED RESULTS
a folder with a ticker name should be created.
a ticker named dashboard should be uploaded to the above folder.
ACTUAL RESULTS
The text was updated successfully, but these errors were encountered: