Skip to content

ISSUE-368: Support disk by-path symlinks in deployment ISO#641

Merged
danielerez merged 1 commit into
openshift:masterfrom
danielerez:fix-disk-by-path-symlinks
Jan 1, 2026
Merged

ISSUE-368: Support disk by-path symlinks in deployment ISO#641
danielerez merged 1 commit into
openshift:masterfrom
danielerez:fix-disk-by-path-symlinks

Conversation

@danielerez
Copy link
Copy Markdown
Contributor

Fixes the deployment ISO failing to clone the appliance disk image when using /dev/disk/by-path/* paths as the target device. The virt-resize command was failing with 'No such file or directory' because symlinks were not being properly resolved.

Changes:

  • Wait for udev to settle before accessing disk paths
  • Wait up to 60 seconds for the target device to become available
  • Resolve symlinks to their actual device paths before passing to virt-resize
  • Add proper error handling when target device is not found
  • Add --pull=never to podman run to prevent TTY prompt errors
  • Tag loaded image to configured name to fix 'image not found' errors

Fixes: #368

@openshift-ci-robot
Copy link
Copy Markdown

@danielerez: No Jira issue with key ISSUE-368 exists in the tracker at https://issues.redhat.com/.
Once a valid jira issue is referenced in the title of this pull request, request a refresh with /jira refresh.

Details

In response to this:

Fixes the deployment ISO failing to clone the appliance disk image when using /dev/disk/by-path/* paths as the target device. The virt-resize command was failing with 'No such file or directory' because symlinks were not being properly resolved.

Changes:

  • Wait for udev to settle before accessing disk paths
  • Wait up to 60 seconds for the target device to become available
  • Resolve symlinks to their actual device paths before passing to virt-resize
  • Add proper error handling when target device is not found
  • Add --pull=never to podman run to prevent TTY prompt errors
  • Tag loaded image to configured name to fix 'image not found' errors

Fixes: #368

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot requested review from jhernand and oourfali January 1, 2026 14:36
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jan 1, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danielerez

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 1, 2026
Fixes the deployment ISO failing to clone the appliance disk image when
using /dev/disk/by-path/* paths as the target device. The virt-resize
command was failing with 'No such file or directory' because symlinks
were not being properly resolved.

Changes:
- Wait for udev to settle before accessing disk paths
- Wait up to 60 seconds for the target device to become available
- Resolve symlinks to their actual device paths before passing to virt-resize
- Add proper error handling when target device is not found
- Add --pull=never to podman run to prevent TTY prompt errors
- Tag loaded image to configured name to fix 'image not found' errors

Fixes: openshift#368
@danielerez danielerez force-pushed the fix-disk-by-path-symlinks branch from 234f62f to f7871d1 Compare January 1, 2026 16:23
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jan 1, 2026

@danielerez: 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.

@danielerez danielerez merged commit 1892eeb into openshift:master Jan 1, 2026
7 of 8 checks passed
@danielerez
Copy link
Copy Markdown
Contributor Author

/cherrypick release-4.21

@openshift-cherrypick-robot
Copy link
Copy Markdown

@danielerez: new pull request created: #710

Details

In response to this:

/cherrypick release-4.21

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.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Support disk by-path

3 participants