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

✨ Set rpi images with lvm by default #1353

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented Apr 25, 2023

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

@Itxaka Itxaka requested a review from a team April 25, 2023 10:31
@netlify
Copy link

netlify bot commented Apr 25, 2023

Deploy Preview for kairos-io canceled.

Name Link
🔨 Latest commit 45f7f86
🔍 Latest deploy log https://app.netlify.com/sites/kairos-io/deploys/6447ac1dd9975d0008e6812a

@codecov-commenter
Copy link

Codecov Report

Merging #1353 (45f7f86) into master (7bb9af9) will increase coverage by 3.13%.
The diff coverage is 51.05%.

📣 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     
Impacted Files Coverage Δ
internal/agent/config.go 0.00% <ø> (ø)
internal/agent/interactive_install.go 0.00% <ø> (ø)
internal/agent/notify.go 0.00% <0.00%> (ø)
internal/agent/recovery.go 0.00% <ø> (ø)
internal/agent/reset.go 0.00% <0.00%> (ø)
internal/agent/upgrade.go 0.00% <0.00%> (ø)
pkg/config/config.go 0.00% <0.00%> (-51.36%) ⬇️
internal/agent/install.go 5.10% <21.21%> (+5.10%) ⬆️
pkg/config/collector/options.go 64.51% <37.50%> (ø)
pkg/config/collector/collector.go 66.55% <66.55%> (ø)
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mauromorales
Copy link
Member

mauromorales commented Apr 25, 2023

Seems to boot correctly

image

but when trying to install I get the following error

image

Does my way to try to install here make any sense? Probably not and I need to build the image with the config I want already in

@Itxaka
Copy link
Member Author

Itxaka commented Apr 25, 2023

Seems to boot correctly but when trying to install I get the following error

image

nice!

Can you try with a normal image as well? Maybe its still broken :D
v2.0.3 should be ready soon

@Itxaka
Copy link
Member Author

Itxaka commented Apr 25, 2023

Does my way to try to install here make any sense? Probably not and I need to build the image with the config I want already in

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.

@mauromorales
Copy link
Member

😅 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?

mauromorales
mauromorales previously approved these changes Apr 25, 2023
Copy link
Member

@mauromorales mauromorales left a comment

Choose a reason for hiding this comment

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

LGTM

@Itxaka
Copy link
Member Author

Itxaka commented Apr 26, 2023

fails on leap+alpine :o

@mauromorales
Copy link
Member

how come it works well on Tumbleweed but not leap 🧐

@Itxaka
Copy link
Member Author

Itxaka commented Apr 27, 2023

looks like the issue is that one of the parts is in use and it cant delete it:

           +arm-image *failed* | + kpartx -dv /dev/loop3
          +arm-image *failed* | del devmap : loop3p1
          +arm-image *failed* | del devmap : loop3p2
          +arm-image *failed* | loop3p3 is in use. Not removing

I dint get why, that part is not used....only loop3p1 is mounted??

@Itxaka
Copy link
Member Author

Itxaka commented Apr 27, 2023

for the life of me, I cant get this to happen locally, so Im guessing its a timing issue?

@Itxaka Itxaka force-pushed the enable-lvm-rpi branch 4 times, most recently from c31bbd1 to a2d49d5 Compare April 27, 2023 14:07
Copy link
Member

@mauromorales mauromorales left a 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 💪

@mauromorales mauromorales merged commit 0bfdfbd into kairos-io:master Apr 27, 2023
@Itxaka
Copy link
Member Author

Itxaka commented Apr 27, 2023

its green green_circle yeiii ... great work @Itxaka muscle

welp, I wanted to try them manually, now that I got my rpi up and running with usb+serial :D

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.

🤖 Make default ARM images (RPI64) LVM-based
3 participants