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 op monitor to primitives #431

Merged
merged 3 commits into from
Feb 12, 2025
Merged

Conversation

tahliar
Copy link
Collaborator

@tahliar tahliar commented Feb 7, 2025

PR creator: Description

op monitor is no longer automatically applied to primitives. I've added it where it was missing, with the exception of some examples where it didn't seem necessary.

The timeout and interval values are the defaults from crm ra info.

PR creator: Are there any relevant issues/feature requests?

  • bsc#1231386
  • jsc#DOCTEAM-1698

PR creator: Which product versions do the changes apply to?

When opening a PR, check all versions of the documentation that your PR applies to.

  • SLE-HA 15
    • 15 next (current main, no backport necessary)
    • 15 SP6
    • 15 SP5
    • 15 SP4
    • 15 SP3
  • SLE-HA 12
    • 12 SP5

PR reviewer only: Have all backports been applied?

The doc team member merging your PR will take care of backporting to older documents.
When opening a PR, do not set the following check box.

  • all necessary backports are done

It is no longer applied automatically
bsc#1231386
jsc#DOCTEAM-1698
@tahliar
Copy link
Collaborator Author

tahliar commented Feb 7, 2025

Hi @zzhou, I've added op monitor where it was missing, with the exception of some examples where it didn't seem necessary. Let me know if I've missed anything or if I've added it somewhere it isn't needed. :)

Copy link
Contributor

@zzhou1 zzhou1 left a comment

Choose a reason for hiding this comment

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

Great thx!

@tahliar tahliar merged commit 71482be into main Feb 12, 2025
4 checks passed
@tahliar tahliar deleted the DOCTEAM-1698-op-monitor-all-guides branch February 12, 2025 05:47
tahliar added a commit that referenced this pull request Feb 12, 2025
* Add op monitor to primitives

It is no longer applied automatically
bsc#1231386
jsc#DOCTEAM-1698

* Adjust op monitors based on tech review

* Small edit
tahliar added a commit that referenced this pull request Feb 12, 2025
* Add op monitor to primitives

It is no longer applied automatically
bsc#1231386
jsc#DOCTEAM-1698

* Adjust op monitors based on tech review

* Small edit
tahliar added a commit that referenced this pull request Feb 12, 2025
* Add op monitor to primitives

It is no longer applied automatically
bsc#1231386
jsc#DOCTEAM-1698

* Adjust op monitors based on tech review

* Small edit
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.

2 participants