-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: Add alerts support #218
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #218 +/- ##
=======================================
Coverage 68.50% 68.50%
=======================================
Files 3 3
Lines 181 181
=======================================
Hits 124 124
Misses 57 57 ☔ View full report in Codecov by Sentry. |
[citest] |
@richm, regarding failing checks: Most of fails are because of In other cases, it looks like issues of CI itself. So can someone responsible for CI look into this, please?
|
Yes, ignore this. ansible 2.17 does not support centos-8 managed nodes. We will be removing this test shortly. @spetrosi Looks like we need to enable the HA (and ResilientStorage) repos on centos e.g. something like what we do in tox-lsr - https://github.com/linux-system-roles/linux-system-roles.github.io/blob/main/download/linux-system-roles.json#L105
Yes, sounds good. At any rate, none of the test failures are due to this PR. |
A pr with workaround that installs passlib for now is it linux-system-roles/tft-tests#13 |
@idevat please rebase and I'll rerun the ci |
6695801
to
533e9be
Compare
@spetrosi rebased. |
[citest] |
Enhancement:
Add support for configuring Pacemaker Alerts
Reason:
This is necessary so that users can configure alerts to take some external action when a cluster event occurs (node failure, resource starting or stopping, etc.)
Result:
It is now possible to configure Pacemaker Alerts using the role
Issue Tracker Tickets (Jira or BZ if any):
https://issues.redhat.com/browse/RHEL-45285