Code review comment for ~andreserl/maas:2.5_prometheus_integration
Revision history for this message
| Blake Rouse (blake-rouse) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Looks good.
You do have one issues. The interval will not be updated automatically, it will only be updated one the old interval is reached. So on startup it will work fine, but changing it to 30 minutes, will not update until 60 minutes is reached. Then it will update from that point.