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

add private method to reinitialize alerts #208

Merged
merged 5 commits into from
Jan 26, 2022
Merged

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented Jan 24, 2022

This PR adds a private method for refreshing (re-initializing) relation data for alerts.
This is needed for cos-config to be able to refresh alerts when alerts show up on disk after the last hook fired.

cf. canonical/loki-k8s-operator#76, canonical/grafana-k8s-operator#53

dstathis
dstathis previously approved these changes Jan 24, 2022
Copy link
Contributor

@mmanciop mmanciop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code change LGTM, but I think this warrants a unit test or two :-)

Copy link
Contributor

@rbarry82 rbarry82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename and make the method private?

rbarry82
rbarry82 previously approved these changes Jan 25, 2022
Copy link
Contributor

@mmanciop mmanciop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description is outdated, the rest LGTM.

@sed-i sed-i changed the title add refresh method add private method to reinitialize alerts Jan 25, 2022
@simskij simskij merged commit 884d8a4 into main Jan 26, 2022
@simskij simskij deleted the feature/refresh_alerts branch January 26, 2022 14:06
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.

5 participants