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

Fix adding /boot/zipl partition on s390x #4321

Merged
merged 1 commit into from Feb 1, 2018

Conversation

rwx788
Copy link
Member

@rwx788 rwx788 commented Feb 1, 2018

Previously it worked with /boot due to suggested mounts, so correct
mount point was used in the end. It doesn't happen with storage-ng.

Previously it worked with /boot due to suggested mounts, so correct
mount point was used in the end. It doesn't happen with storage-ng.

See [poo#28507](https://progress.opensuse.org/issues/28507).
@rwx788
Copy link
Member Author

rwx788 commented Feb 1, 2018

@Soulofdestiny, please, take a look.

@Soulofdestiny
Copy link
Contributor

uhm in general, looks good - but your verification runs are actually not verifying that it can really boot :)

@rwx788
Copy link
Member Author

rwx788 commented Feb 1, 2018

I know, because it fails before. One of the runs is old behavior, so if it doesn't boot it's a bug. With separate boot, it should work also, as partitioning is according to the specs. ATM it's wrong partitioning.

@Soulofdestiny
Copy link
Contributor

then let's conduct this experiment? or would it be another case of "merging too early"? ;)

@Soulofdestiny Soulofdestiny merged commit 9f665fb into os-autoinst:master Feb 1, 2018
@okurz
Copy link
Member

okurz commented Feb 3, 2018

then let's conduct this experiment? or would it be another case of "merging too early"? ;)

no, because it's limited to s390x and only the special partitioning module :)

@rwx788 rwx788 deleted the 28507_lvm_full_encrypt branch May 9, 2018 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants