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

Terminology: LVM, LVM2, and Cluster LVM #352

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Conversation

tahliar
Copy link
Collaborator

@tahliar tahliar commented Oct 18, 2023

PR creator: Description

  • Used the existing clvm entity in the remaining places where it wasn't used.
  • Changed all instances of LVM2 (except package names and command output) to LVM. As far as I can tell, LVM1 is obsolete, so I think it's safe to just use the term LVM without causing confusion.

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

  • jsc#DOCTEAM-111

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 SP5
    • 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

@jfaltenbacher
Copy link
Contributor

thank you @tahliar . I'll take care to reflect this in TermWeb as well

@tahliar
Copy link
Collaborator Author

tahliar commented Oct 18, 2023

thank you @tahliar . I'll take care to reflect this in TermWeb as well

Thanks @jfaltenbacher! I saw that TermWeb already says LVM and not LVM2 so that's all good, but it does need to be updated to say Logical Volume Manager instead of Logical Volume Management.
Google shows that both are used depending on where you look, so I was a bit frustrated trying to figure out if the HA docs use the right term! But the description from rpm -qi lvm2 says "Programs and man pages for configuring and using the LVM2 Logical Volume Manager" so that's definitive enough for me.

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.

LGTM!

@tahliar tahliar merged commit 9dc1306 into main Oct 19, 2023
4 checks passed
tahliar added a commit that referenced this pull request Oct 19, 2023
tahliar added a commit that referenced this pull request Oct 19, 2023
tahliar added a commit that referenced this pull request Oct 19, 2023
tahliar added a commit that referenced this pull request Oct 19, 2023
tahliar added a commit that referenced this pull request Oct 19, 2023
@tahliar tahliar deleted the DOCTEAM-111-lvm-terminology branch October 19, 2023 03:29
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.

3 participants