Skip to content

OSDOCS-18757#LSO Symlinks updating#109365

Open
lpettyjo wants to merge 1 commit intoopenshift:mainfrom
lpettyjo:OSDOCS-18757
Open

OSDOCS-18757#LSO Symlinks updating#109365
lpettyjo wants to merge 1 commit intoopenshift:mainfrom
lpettyjo:OSDOCS-18757

Conversation

@lpettyjo
Copy link
Copy Markdown
Contributor

@lpettyjo lpettyjo commented Mar 31, 2026

@lpettyjo lpettyjo added this to the Planned for 4.22 GA milestone Mar 31, 2026
@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 31, 2026
@lpettyjo lpettyjo requested a review from gnufied March 31, 2026 17:57
@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Mar 31, 2026

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 31, 2026

@lpettyjo: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

Comment on lines +20 to +24
. Log in to the {product-title} web console.

. Do not change the default setting (none) for `localVolumeDeviceLink.spec.policy`.
+
This establishes link monitoring and generates an alert if the current and preferred paths do not match.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This procedure is odd. "Do nothing and this establishes link monitoring and generates an alert"

We should note that by default, LSO generates an alert when it detects that a PersistentVolume uses sub-optimal by-id symlink for a device. And then what to do with it, that part looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.22 size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants