Skip to content

Minor fixes to SCSI mount operation#1798

Merged
ambarve merged 1 commit intomicrosoft:mainfrom
ambarve:HEAD
Jun 2, 2023
Merged

Minor fixes to SCSI mount operation#1798
ambarve merged 1 commit intomicrosoft:mainfrom
ambarve:HEAD

Conversation

@ambarve
Copy link
Copy Markdown
Contributor

@ambarve ambarve commented Jun 2, 2023

During a recent refactor SCSI mount operation code removed a retry logic that is needed when examining the filesystem type on a SCSI device. Retry is needed because sometimes attempting to open a SCSI devices immediately after attaching it results in ENXIO/ENOENT errors. This adds the retry logic back.

Names of some images used in the tests had changed, this commit updates those names too.

@ambarve ambarve requested a review from a team as a code owner June 2, 2023 15:17
Comment thread ext4/tar2ext4/tar2ext4.go Outdated
During a recent refactor SCSI mount operation code removed a retry logic that is needed when examining the
filesystem type on a SCSI device. Retry is needed because sometimes attempting to open a SCSI devices
immediately after attaching it results in ENXIO/ENOENT errors. This adds the retry logic back.

Names of some images used in the tests had changed, this commit updates those names too.

Signed-off-by: Amit Barve <[email protected]>
@ambarve ambarve merged commit 8d4a20c into microsoft:main Jun 2, 2023
anmaxvl pushed a commit that referenced this pull request Oct 20, 2023
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
During a recent refactor SCSI mount operation code removed a retry logic that is needed when examining the
filesystem type on a SCSI device. Retry is needed because sometimes attempting to open a SCSI devices
immediately after attaching it results in ENXIO/ENOENT errors. This adds the retry logic back.

Names of some images used in the tests had changed, this commit updates those names too.

Signed-off-by: Amit Barve <[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.

3 participants