hassio-addons / addon-grafana

Grafana - Home Assistant Community Add-ons
https://addons.community
MIT License
220 stars 61 forks source link

โฌ†๏ธ Upgrades Grafana to v9.1.5 #275

Closed QuentinBtd closed 1 year ago

QuentinBtd commented 2 years ago

Proposed Changes

Upgrades Grafana to v9.1.5

frenck commented 2 years ago

How did you test this?

QuentinBtd commented 2 years ago

How did you test this?

Hi ! You are fast !

I think I am in the wrong way. I am in the process of discovering how to make/update/test addons ๐Ÿ˜Š

Tests will come. Could you add "WIP" label on this PR ?

frenck commented 2 years ago

Tests will come. Could you add "WIP" label on this PR ?

It's your PR. Feel free to mark it as a draft and ready for review again when ready.

QuentinBtd commented 2 years ago

It's running on my HAss instance.

(installed via https://github.com/QuentinBtd/hass-repository)

Do you have a check list of specifics tests to make ?

fti7 commented 2 years ago

Any progress on this PR? :-)

QuentinBtd commented 2 years ago

Any progress on this PR? :-)

Hi,

It run well on my HAss instance since I create this PR.

I have my dashboard and some plugins installed on it. :)

cedricziel commented 2 years ago

@QuentinBtd I think you need to apply a label to have it reviewed again :)

ready for review

cedricziel commented 2 years ago

Maybe update to 9.1.2 while you're at it ;-)

QuentinBtd commented 1 year ago

Maybe update to 9.1.2 while you're at it ;-)

Hi I'm testing it ๐Ÿ‘Œ

cedricziel commented 1 year ago

Hi @QuentinBtd - the PR is missing one of the labels and the verify workflow fails.

Error! This pull request does not contain any of the valid labels: ['breaking-change', 'bugfix', 'documentation', 'enhancement', 'refactor', 'performance', 'new-feature', 'maintenance', 'ci', 'dependencies', 'translations']
QuentinBtd commented 1 year ago

Hi @QuentinBtd - the PR is missing one of the labels and the verify workflow fails.

Error! This pull request does not contain any of the valid labels: ['breaking-change', 'bugfix', 'documentation', 'enhancement', 'refactor', 'performance', 'new-feature', 'maintenance', 'ci', 'dependencies', 'translations']

Hello @cedricziel I know that, but I didnโ€™t have the permission to add label ๐Ÿ˜…

I saw Frenck remove the request review that I asked him. Maybe could you review this PR ?

cedricziel commented 1 year ago

Iโ€™d love to - however I donโ€™t have permission to.

@frenck we seem to have a locking situation here. Can you guide us how to proceed?

frenck commented 1 year ago

Yes, have patience. Thanks ๐Ÿ™

maexrakete commented 1 year ago

Hey!

Curious what the progress is here? I'd really love to use the new Grafana-Version for the better Flux-Query integration :)

Thanks for your work!

frenck commented 1 year ago

See my message above. I returned from my holiday this weekend and currently catching up with all work and notifications. As per above, please be patient. Thanks

QuentinBtd commented 1 year ago

You're welcome ! I'll follow next updates of Grafana and if it's relevant, I'll open new PR ๐Ÿ™‚