Skip to content

Add more godoc to GuestFormat#2822

Merged
pierreprinetti merged 1 commit intogophercloud:masterfrom
EmilienM:guestformat
Oct 24, 2023
Merged

Add more godoc to GuestFormat#2822
pierreprinetti merged 1 commit intogophercloud:masterfrom
EmilienM:guestformat

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

I recently found out that when not providing the GuestFormat to a block
device mapping, the device will be formated to vfat anyway.

https://opendev.org/openstack/nova/src/commit/d0b459423dd81644e8d9382b6c87fabaa4f03ad4/nova/privsep/fs.py#L257

I think it's valuable that we document that so the users know what to
expect.

I recently found out that when not providing the GuestFormat to a block
device mapping, the device will be formated to `vfat` anyway.

https://opendev.org/openstack/nova/src/commit/d0b459423dd81644e8d9382b6c87fabaa4f03ad4/nova/privsep/fs.py#L257

I think it's valuable that we document that so the users know what to
expect.
@github-actions github-actions bot added the semver:patch No API change label Oct 24, 2023
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 77.962%. remained the same when pulling 902ed6e on EmilienM:guestformat into 32dbd09 on gophercloud:master.

Copy link
Copy Markdown
Member

@pierreprinetti pierreprinetti left a comment

Choose a reason for hiding this comment

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

Thank you: that is extremely valuable input for the user.

@pierreprinetti pierreprinetti merged commit cc89262 into gophercloud:master Oct 24, 2023
@pierreprinetti pierreprinetti added the backport-v1 This PR will be backported to v1 label Oct 24, 2023
@EmilienM EmilienM deleted the guestformat branch October 24, 2023 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v1 This PR will be backported to v1 semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants