grafana-toolbox / grafana-wtf

Grep through all Grafana entities in the spirit of git-wtf.
GNU Affero General Public License v3.0
144 stars 13 forks source link

CI: Update to Grafana 8.5.21 and 9.4.3 #44

Closed amotl closed 10 months ago

amotl commented 1 year ago

What the title says.

amotl commented 1 year ago

It looks like Grafana 9.3.0 introduced a small regression here.

amotl commented 1 year ago

1. Regression with HTTP status on HTTP DELETE

Grafana 8.5.21

It looks like the issue has apparently been fixed on the Grafana 8 series, specifically with Grafana 8.5.20, where the previous release 8.5.16 had the problem, while 8.5.15 was still ok.

image

https://github.com/grafana/grafana/compare/v8.5.16...v8.5.20 has many changes, so it was not possible to identify the improving one yet.

Grafana 9.4.3

However, the problem is still present on Grafana 9.4.

-- https://github.com/panodata/grafana-wtf/actions/runs/4333757677/jobs/7567151287#step:5:2881

/cc @zuchka, @mildwonkey

amotl commented 1 year ago

2. Regression: user attribute in response changed

Grafana 9.4.3

When upgrading to Grafana 9.4, it looks like there has been another change. Where the API returned a user name within the user attribute beforehand, it now apparently returns the user's id.

Differing items:
{'user': 1} != {'user': 'admin'}

Full diff:
  {
   'folder': 'Testdrive',
   'message': '',
   'title': 'luftdaten.info generic trend v27',
   'url': 'http://localhost:33333/d/ioUrPwQiz/luftdaten-info-generic-trend-v27',
-  'user': 'admin',
+  'user': 1,
   'version': 1,
  }

-- https://github.com/panodata/grafana-wtf/actions/runs/4333757677/jobs/7567151287#step:5:3227