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
lvm2: 2.02.177 -> 2.03.01 #51756
lvm2: 2.02.177 -> 2.03.01 #51756
Conversation
@GrahamcOfBorg test installer.lvm |
@GrahamcOfBorg biuld nixosTests.installer.lvm |
And this is the first bump across what they called a set of large refactorings on the mailing list, so probably we shouldn't hurry with backporting? |
What do you mean by backporting? |
I meant |
OK, if there is any need for backporting, there is also lvm2-2.02.183. |
I have some doubts even about 2.02.183 |
Not directly pertaining to the changes in this PR but just FYI the LVM installer test is likely broken on master anyway for unrelated reasons - #51444 (comment) |
IIRC there were also other problems with the installer tests, such as randomly timed out tests. |
* switch to sources to git * add libaio to buildInputs
@pbogdan Well, there was also that |
I just did a rebase. This should eliminate the issue with |
Now the test times out on x86_64-linux. |
Locally it failed for GRUB reasons without this PR. #51790 |
@GrahamcOfBorg build nixosTests.installer.lvm |
With the XFS patch the lvm installer test succeeds again (locally, with ofborg it times out). |
xfsprogs-4.16 introduced an new on disk format feature that is not recognized properly by grub. This patch allows grub to access XFS filesystem created with xfsprogs >= 4.16. This upstream patch can be removed for grub-2.03.
Test works locally here too 👍 any reason not to merge? at the very worse we'll break an already broke trunk :) |
If there are no further comments it can be merged. |
Hmm, seems like
The lvm2 commit removing it. osquery breaks due to the removal:
cc latest |
The lvm update also breaks |
Even if it would be fixed on Until this is fixed upstream and I do the upgrade for |
Reinstating 2.02.183 seems to be a natural plan… |
So far it looks like |
👍 do you want to file a patch for this? (feel free to ping me then, I can test and merge this for you then :)) |
As discussed in NixOS#51756, recently packaged versions of `lvm2` miss the `lvm2app.h` header which breaks the osquery build. Please note that this simply fixes the build and is not an upgrade. The CMake patches are fairly diverged in constrast to the current upstream packaging which requires a lot more effort I can't provide ATM. cc @markuskowa @hedning
Motivation for this change
Version bump. Switched the source to the git repository. Should now be fit for the auto updater.
CC @7c6f434c
Things done
libaio
tobuildInputs
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)