hubot-grafana does not work for Grafana 8

[superbrothers]

In Grafana 8, GET /dashboards/db/:slug has been removed. hubot-grafana has used that API endpoint, so it does not work for Grafana 8 😢

hubot-grafana/src/grafana.coffee

Lines 185 to 187 in 1f94916

	# Call the API to get information about this dashboard
	callGrafana endpoint, "dashboards/db/#{slug}", (dashboard) ->
	robot.logger.debug dashboard

• https://grafana.com/docs/grafana/latest/release-notes/release-notes-8-0-0/#breaking-changes

[imyuliz]

me too,

[imyuliz]

slug deprecated in Grafana v5.0,
https://grafana.com/docs/grafana/latest/http_api/dashboard/

What should we do

[superbrothers]

I have created a patch to support Grafana 8. This patch may not be compatible with older Grafana, so it will be difficult to merge this in.

• superbrothers@a12cb3e

[stephenyeargin]

/cc #68

So with this change, what should the user interaction expectation be for working with a specific dashboard? Can we no longer have a command that shorthands it out to !graf db my-great-dashboard instead of !graf db d3adbe3f?

[stephenyeargin]

Reviewing the search API, I don't see a way to guarantee that we always return a single dashboard if we were to try and transform an existing user-friendly URI. It seems like a lot to ask for folks to learn a nine-character, case-sensitive UID to pull up their dashboard.

I see a handful of ways forward:

1. Declare that this Hubot package supports only Grafana 7.x and below, effectively sun-setting this project (I don't like this option)
2. Release v3.0.0 that merges @superbrothers's branch with instructions for users of < 8 to remain on the 2.x release
3. Add an environment variable that enables Grafana 8 support (switch statement on how to query/return data). This allows folks to opt-in, but requires a manual step.
4. Continue seeing if there's a way to handle the changes "under the hood" using the Search API in an attempt mitigate changes in user space

/cc @torkelo if you'd like to weigh in.

[superbrothers]

2. Release v3.0.0 that merges @superbrothers branch with instructions for users of < 8 to remain on the 2.x release

I think this is the better way.

[stephenyeargin]

Making a note that the test coverage is rather extensive for this package. I started the process of updating it, but it will take a few sessions to get it all done.