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

gadget.yaml: add ubuntu-save #55

Merged
merged 2 commits into from
Nov 23, 2020

Conversation

anonymouse64
Copy link
Contributor

@anonymouse64 anonymouse64 commented Nov 20, 2020

Fixes #54 for arm64

Note that this needs snapd 2.48 to build and install successfully.

Signed-off-by: Ian Johnson <[email protected]>
@waveform80
Copy link
Member

Looks like a sane change to me. One thing I would note is that there's now four partitions in the image and, if they're all created as primary (I don't know if this is the case) then that potentially means that users can't create any more partitions (for whatever purposes).

It may be worth noting that the Pi's boot firmware gained the ability to handle GPT partition tables a while ago (primarily to support those booting off USB, but I understand it works on SD cards too). If the 4 partitions in the image are all primary (i.e. if there's no extended partition which users could create logical partitions within) then it might be worth switching the image to use GPT partitioning instead.

Anyway, overall it's a +1 from me.

@anonymouse64
Copy link
Contributor Author

that potentially means that users can't create any more partitions (for whatever purposes).

Well if users want additional partitions they will have to modify the gadget.yaml anyways and so they can probably weigh the benefits of keeping ubuntu-save or adding their own partition, but in light of this I will add a comment to the gadget.yaml explaining that ubuntu-save is optional for unencrypted devices and that with MBR there can no longer be any partitions.

It may be worth noting that the Pi's boot firmware gained the ability to handle GPT partition tables a while ago

This is great news actually, when we last looked at this, the firmware didn't support this, so if GPT is now supported across all the pi's we care about, then I think we should look into doing this. Would changing to GPT affect our u-boot support at all? I am fairly confident u-boot supports GPT, but the pi is special in numerous ways so I'm not sure if how we use it means it will be supported properly.

But not for this PR obviously since we want this before 1.0 😄

Thanks to @waveform for pointing this out.

Signed-off-by: Ian Johnson <[email protected]>
@xnox xnox merged commit 15657cb into canonical:20-arm64 Nov 23, 2020
@xnox
Copy link
Contributor

xnox commented Nov 23, 2020

Re GPT partitions table opened #58

btw are our classic presintalled images not gpt already?

@anonymouse64 anonymouse64 deleted the feature/uc20-ubuntu-save-arm64 branch November 23, 2020 17:41
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.

add ubuntu-save to gadget.yaml
3 participants