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

Clarify and improve STONITH requirement #315

Merged
merged 5 commits into from
Mar 1, 2023
Merged

Conversation

tahliar
Copy link
Collaborator

@tahliar tahliar commented Feb 24, 2023

PR creator: Description

I've addressed two JIRA issues in this PR:

DOCTEAM-144

  • Clarified the STONITH item in the NFS quick start guide. This section is a usage scenario rather than a list of requirements, so not an appropriate place to go into a lot of detail.
  • Added more detail to the STONITH requirement in the Installation quick start guide.

DOCTEAM-107

  • Added an appendix with simple procedures to add iSCSI storage for use with SBD. I've tried to clearly mark it as just a simple option for testing purposes, but I think it's really useful to include as an easy way to get SBD for testing out an HA cluster, even if you know nothing about shared storage.

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

  • bsc#1158162
  • jsc#DOCTEAM-144
  • jsc#DOCTEAM-107

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 SP4
    • 15 SP3
    • 15 SP2
    • 15 SP1
  • SLE-HA 12
    • 12 SP5
    • 12 SP4

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

Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

Hi, just minor corrections referring to this update, otherwise LGTM. Thanks!

Co-authored-by: Daria Vladykina <[email protected]>
@tahliar tahliar merged commit 24c0ae6 into main Mar 1, 2023
@tahliar tahliar deleted the DOCTEAM-107-iscsi-for-sbd branch March 1, 2023 00:42
tahliar added a commit that referenced this pull request Mar 1, 2023
* Add basic iscsi storage steps for testing

jsc#DOCTEAM-107

* Clarify STONITH requirement

jsc#DOCTEAM-144
bsc#1158162

* Add xref to quickstart

* Fix old typo

* Apply suggestions from review

Co-authored-by: Daria Vladykina <[email protected]>

---------

Co-authored-by: Daria Vladykina <[email protected]>
tahliar added a commit that referenced this pull request Mar 1, 2023
* Add basic iscsi storage steps for testing

jsc#DOCTEAM-107

* Clarify STONITH requirement

jsc#DOCTEAM-144
bsc#1158162

* Add xref to quickstart

* Fix old typo

* Apply suggestions from review

Co-authored-by: Daria Vladykina <[email protected]>

---------

Co-authored-by: Daria Vladykina <[email protected]>
tahliar added a commit that referenced this pull request Mar 1, 2023
* Add basic iscsi storage steps for testing

jsc#DOCTEAM-107

* Clarify STONITH requirement

jsc#DOCTEAM-144
bsc#1158162

* Add xref to quickstart

* Fix old typo

* Apply suggestions from review

Co-authored-by: Daria Vladykina <[email protected]>

---------

Co-authored-by: Daria Vladykina <[email protected]>
tahliar added a commit that referenced this pull request Mar 1, 2023
* Add basic iscsi storage steps for testing

jsc#DOCTEAM-107

* Clarify STONITH requirement

jsc#DOCTEAM-144
bsc#1158162

* Add xref to quickstart

* Fix old typo

* Apply suggestions from review

Co-authored-by: Daria Vladykina <[email protected]>

---------

Co-authored-by: Daria Vladykina <[email protected]>
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