Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

actions: fix fstab generated in case of FAT{12|16|32}. #501

Merged
merged 1 commit into from
May 21, 2024

Conversation

azayet01
Copy link
Contributor

Recent changes introduced a tuned formatting of the partitions created with FAT{12|16|32}. The issue is that fstab is generated using the fs property which won't match the system expectations.

If the fs property is "fat", "fat12", "fat16", "fat32" or "msdos", then fstab must use "vfat".

Recent changes introduced a tuned formatting of the partitions created
with FAT{12|16|32}. The issue is that fstab is generated using the fs
property which won't match the system expectations.

If the fs property is "fat", "fat12", "fat16", "fat32" or "msdos", then
fstab must use "vfat".

Signed-off-by: Aymen Zayet <[email protected]>
@obbardc
Copy link
Member

obbardc commented May 21, 2024

Thank you for this fix!

@obbardc obbardc added this pull request to the merge queue May 21, 2024
Merged via the queue into go-debos:main with commit 227cf70 May 21, 2024
31 checks passed
@azayet01
Copy link
Contributor Author

@obbardc ,
You are welcome ! thanks for reviewing and reacting fast ! Very much appreciated. There are the two critical stuff that I have. But for the remaining contribution , it's not critical so I will be pushing some proposals but not critical to have.

Have a great day !

@obbardc
Copy link
Member

obbardc commented May 21, 2024

@azayet01 Please feel free to open issues/PRs about anything you wish to discuss :-). Thank you again for your contributions!

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.

fstab generated with image-partition action is wrong when fatxx is specified.
2 participants