-
Notifications
You must be signed in to change notification settings - Fork 0
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
Upgrade Rook to 1.11.11 #124
Conversation
4cef1bb
to
4b377a2
Compare
568ba34
to
3820edb
Compare
Remove configuration for removed machine disruption config on OCP
3820edb
to
df22d42
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please go over the runbooks again. The PR currently significantly reduces runbook quality:
Overall guidelines:
- Don't blindly commit the output of the runbook generator
- Label templates have no place in the runbook
- Compare changes carefully and think about what you'd like to see when you click on the runbook link in an alert, it's totally ok to rewrite the alert message to go more into the "why" of the alert instead of replicating the "what" from the alert description.
docs/modules/ROOT/pages/runbooks/CephMgrPrometheusModuleInactive.adoc
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/runbooks/CephMgrPrometheusModuleInactive.adoc
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes mostly LGTM now, one remaining runbook fix inline.
Note that I can't explicitly approve the PR since I originally created it 🙈
36731a1
to
7743d5d
Compare
Replaces #115
Checklist
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
bug
,enhancement
,documentation
,change
,breaking
,dependency
as they show up in the changelog.