-
Notifications
You must be signed in to change notification settings - Fork 100
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
✨ Set rpi images with lvm by default #1353
Conversation
✅ Deploy Preview for kairos-io canceled.
|
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #1353 +/- ##
==========================================
+ Coverage 22.79% 25.92% +3.13%
==========================================
Files 22 17 -5
Lines 1610 1427 -183
==========================================
+ Hits 367 370 +3
+ Misses 1179 997 -182
+ Partials 64 60 -4
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
umm good point, rpi images are not installed, they are just dumped into the SD as they are raw images. Of course you are not gonna be able to install LOL Almost got me there for a second @mauromorales ;) RAW images are just installed systems ready to flash to a card. installation means nothing in there, only upgrade and Im guessing reset...upgrade I tested and it went correctly but reset + upgrade recovery could be tricky. |
😅 sorry for the brain fart hehe Maybe we should make the agent aware of this? Like don't offer to install if not possible, or give a message instead of an error? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
fails on leap+alpine :o |
how come it works well on Tumbleweed but not leap 🧐 |
looks like the issue is that one of the parts is in use and it cant delete it:
I dint get why, that part is not used....only loop3p1 is mounted?? |
for the life of me, I cant get this to happen locally, so Im guessing its a timing issue? |
c31bbd1
to
a2d49d5
Compare
Signed-off-by: Itxaka <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its green 🟢 yeiii ... great work @Itxaka 💪
welp, I wanted to try them manually, now that I got my rpi up and running with usb+serial :D |
What this PR does / why we need it:
Enables LVM by default on RPI images, which means that the recovery partition will be an lvm partition
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #1271