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

cjdns declarative configuration #3241

Merged
merged 2 commits into from
Sep 1, 2014
Merged

cjdns declarative configuration #3241

merged 2 commits into from
Sep 1, 2014

Conversation

ehmry
Copy link
Contributor

@ehmry ehmry commented Jul 11, 2014

At present the cjdns service reads a user generated config from /etc.
This PR generates the node secret key and stores it in /etc, and embeds the rest of the configuration in the systemd unit. The secret is not updated until it goes missing from the file.

In this way the same configuration can be copied between machines without addresses clashing.

@manpages
Copy link

👍
I have reviewed the logic, nothing shady in makekey.c, the keys get regenerated only if those are missing from the filesystem.
This should be merged if it works on latest nixos (didn't test it).

@ehmry ehmry force-pushed the cjdns branch 2 times, most recently from 1450fcd to 2564161 Compare August 29, 2014 13:37
@ehmry
Copy link
Contributor Author

ehmry commented Aug 30, 2014

I need to update the cjdns revision for the package, but I have to work out an issue with the service module.

@ehmry ehmry force-pushed the cjdns branch 2 times, most recently from bde0917 to 05bba2a Compare August 30, 2014 02:53
systemd service wants network-interfaces.target rather than network.target
assertion on config.networking.enableIPv6
@ehmry
Copy link
Contributor Author

ehmry commented Aug 31, 2014

I had the systemd unit want network.target when it should have wanted network-interfaces.target (like OpenVPN), because of that CJDNS would not connect to peers reliably and that is now fixed.

I've also updated the package, which was almost 6 months out of date.

7c6f434c added a commit that referenced this pull request Sep 1, 2014
cjdns declarative configuration
@7c6f434c 7c6f434c merged commit a6dfb4d into NixOS:master Sep 1, 2014
@ehmry ehmry deleted the cjdns branch September 2, 2014 13:00
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.

3 participants