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
nixos/drbd: fix syntax for etc config file and systemd service #109406
Conversation
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.
Do you use this module? I wonder how long this has been broken for... Any interest in writing a NixOS test?
I just started trying to use it for an HA SAN. There are some quirks to the pkg too. Version 8 is not compatible with Linux kernel 5.x; I had to set
It looks like the breaking change happened about one year ago: 1d61efb#diff-168f1268fc04de9bceed6749379d94d8c5d45c3aeb9780ae3337851cbd6b603f |
It looks like there are more things broken with this than I first thought. The systemd service is not configured correctly and calls the |
I don't know if would be able find some time dig into this. My experience with |
The second commit I just made fixes the systemd service. I might try to work on getting version 9 to compile (there are some nice new features), but I think this pull request works to fix version 8 for now. |
Thanks again for your work on this @jslight90. Unfortunately I still haven't found time to test this out... but really the module seems pretty broken so I don't think this PR has any possibility to make things worse 😆 From simple inspection of the change everything makes sense, so I'll go ahead and merge this if you're good with that... yes? |
Closing in favor of #119904. |
Motivation for this change
Fix typo that causes DRBD module to raise an error when enabled.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)