-
Notifications
You must be signed in to change notification settings - Fork 1.8k
POWERMON-578 0.5 Installation Doc Updates #95307
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
POWERMON-578 0.5 Installation Doc Updates #95307
Conversation
|
🤖 Wed Jul 30 16:29:20 - Prow CI generated the docs preview: https://95307--ocpdocs-pr.netlify.app/ |
bbaa659 to
119ec29
Compare
|
power-monitoring-0.5 rebased against main 07/22/2025 by Brian D This PR rebased against power-monitoring-0.5 07/23/2025 |
sthaha
left a comment
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.
Lets retain the warning to remove previously installed versions.
358bd21 to
55ec678
Compare
|
power-monitoring-0.5 rebased against main by Brian D. 07/29/2025 PR rebased against power-monitoring-0.5 07/29/2025 AM. |
|
/lgtm |
sthaha
left a comment
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.
/lgtm
vprashar2929
left a comment
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.
/lgtm
55ec678 to
b3ed58d
Compare
ed75bf5 to
1a2d078
Compare
vprashar2929
left a comment
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.
/lgtm
gabriel-rh
left a comment
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.
LGTM
| [id="power-monitoring-deploying-power-monitor-custom-resource_{context}"] | ||
| = Deploying PowerMonitor custom resource | ||
|
|
||
| You can deploy {PM-kepler} by creating an instance of the `PowerMonitor` custom resource (CR) by using the {PM-operator}. |
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.
I know the style guide says to use "by", but the 2 by's in a row doesn't read well - I'd drop the last one
|
New changes are detected. LGTM label has been removed. |
7ec8190 to
e521c02
Compare
|
@gwynnemonahan: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
TECHNOLOGY PREVIEW
POWERMON-578 0.5 Installation Doc Updates
Version(s):
Merge to only the
power-monitoring-0.5branch - no cherrypicks are required.I will open one PR against main to incorporate all of the Power Monitoring content just before its GA.
Note to self: applies to 4.17+
Issue:
https://issues.redhat.com/browse/POWERMON-578
Link to docs preview:
https://95307--ocpdocs-pr.netlify.app/openshift-enterprise/latest/observability/power_monitoring/installing-power-monitoring#power-monitoring-deploying-power-monitor-crd_installing-power-monitoring
QE review:
Additional information:
Also updates attribute to be "Power Monitoring Operator" so it is inline with OCP style.