Skip to content

find_devs/openbsd: accept ISO on disk#1132

Merged
TheRealFalcon merged 1 commit into
canonical:mainfrom
goneri:find_devs-openbsd-accept-ISO-on-disk_23674
Dec 9, 2021
Merged

find_devs/openbsd: accept ISO on disk#1132
TheRealFalcon merged 1 commit into
canonical:mainfrom
goneri:find_devs-openbsd-accept-ISO-on-disk_23674

Conversation

@goneri
Copy link
Copy Markdown
Contributor

@goneri goneri commented Dec 3, 2021

When the metadata is an ISO image and is exposed through a disk,
the device is called /dev/sd?a internally. For instance /dev/sd1a.

It can then be mounted with mount_cd9660 /dev/sd1a /mnt.

For comparison, metadata in the FAT32 format are exposed as /dev/sd?i because this time there is a partition table.

With this change, cloud-init ttries to mount /dev/sd?a in addition to /dev/sd?i.

Closes: ContainerCraft/kmi#12

@TheRealFalcon
Copy link
Copy Markdown
Contributor

@goneri , the change looks good to me, but a few unit tests are failing. Can you fix the failing unit tests?

@goneri goneri force-pushed the find_devs-openbsd-accept-ISO-on-disk_23674 branch from bdc9783 to 768b3a0 Compare December 7, 2021 14:21
When the metadata is an ISO image and is exposed through a disk,
the device is called `/dev/sd?a` internally. For instance `/dev/sd1a`.

It can then be mounted with `mount_cd9660 /dev/sd1a /mnt`.

Metadata in the FAT32 format are exposed as `/dev/sd?i`.

With this change, we try to mount `/dev/sd?a` in addition to `/dev/sd?i`.

Closes: ContainerCraft/kmi#12
@goneri goneri force-pushed the find_devs-openbsd-accept-ISO-on-disk_23674 branch from 768b3a0 to 7e3089e Compare December 9, 2021 16:44
Copy link
Copy Markdown
Contributor

@TheRealFalcon TheRealFalcon left a comment

Choose a reason for hiding this comment

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

Thanks!

@TheRealFalcon TheRealFalcon merged commit 2473959 into canonical:main Dec 9, 2021
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.

openbsd cloud-init fails to apply (could not find data source)

2 participants