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

Include ZFS in installer #4362

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

andrewd-zededa
Copy link
Contributor

Needed for ZFS vault, HV=kubevirt

Needed for ZFS vault, HV=kubevirt

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
@eriknordmark eriknordmark added the stable Should be backported to stable release(s) label Oct 16, 2024
@eriknordmark
Copy link
Contributor

I assume this needs to be backported to 13.4-stable, so I added the stable label.

@andrewd-zededa
Copy link
Contributor Author

Ok, once merging I'll submit the cherry-pick back port.

@OhmSpectator
Copy link
Member

@eriknordmark, should we really wait for all the Eden tests here? Eden tests do not touch the functionality changed here, as far as I understand.

@eriknordmark
Copy link
Contributor

The Eden tests do go through the onboarding process and it's useful to sanity check that. I thought there were tests running the installer image as well, but clearly not with zfs since then we wouldn't have had this regression.

@OhmSpectator
Copy link
Member

OhmSpectator commented Oct 16, 2024

there were tests running the installer image as well

wow, I didn't know... I have at least to find what are the test cases that we run))

@OhmSpectator
Copy link
Member

All green! Merging!
@andrewd-zededa, waiting for backporting PR(s)

@OhmSpectator OhmSpectator merged commit 472daee into lf-edge:master Oct 16, 2024
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stable Should be backported to stable release(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants