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

Only support alert rules if grafana is above version 9.4.0 #205

Merged

Conversation

mt3593
Copy link
Contributor

@mt3593 mt3593 commented Apr 6, 2023

Follow on from #201

This ensures that it only saves and restores alert rules if grafana's version is equal to or above 9.4.0.

I have tested this by running up a docker image of the various version and trying to restore and back up from each version.

How does this look @acjohnson @michyliao @ysde

This does add a dependency on to the packaging library, hope that's ok.

@acjohnson
Copy link
Collaborator

I actually like this method but I'll be curious what @ysde thinks. Also please add packaging to the requires list in setup.py @mt3593

@ysde
Copy link
Owner

ysde commented Apr 11, 2023

Hi @mt3593 , @acjohnson
Thanks!

It's great, I also tested versions like '9.4.0-beta1', the version.parse also worked.
Can you also help add description on README under the "Supported components" block ?
something like "Alert Rules ( >= 9.4.0)"

Many thanks

@mt3593
Copy link
Contributor Author

mt3593 commented Apr 11, 2023

@ysde oh didn't think about beta version, good to see the library has already thought of this.

Updated readme 👍

@acjohnson acjohnson merged commit 1b5c8e2 into ysde:master Apr 11, 2023
@mt3593 mt3593 deleted the only-save-alert-rules-if-grafana-supports-it branch April 11, 2023 13:12
@acjohnson
Copy link
Collaborator

Thank you for this @mt3593 🎉

@mt3593
Copy link
Contributor Author

mt3593 commented Apr 11, 2023

No worries @acjohnson, anything more we need to get a release out?

@acjohnson
Copy link
Collaborator

Not that I'm aware of @ysde if you want I can cut a release but I think with the new features it should be 1.3.0

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

Successfully merging this pull request may close these issues.

3 participants