Skip to content

Conversation

@jrester
Copy link

@jrester jrester commented Jan 14, 2026

NetworkManager supports the persistent-keepalive parameter for wireguard peers. This MR adds this parameter for wireguard peers, so that one can read and modify the persistent-keepalive parameter.

@jrester jrester force-pushed the add-persistent-keepalive-wireguard-peers branch from 64879f6 to 34873e6 Compare January 14, 2026 09:33
@igo95862
Copy link
Collaborator

Hello @jrester

Did you manually add that field or did you use generator in tools/generate-settings-dataclasses-jinja.py? There is an automated script that will generate all settings dataclasses from NetworkManager metadata.

@jrester
Copy link
Author

jrester commented Jan 20, 2026

I generated new settings modules with the generator script, however it did not add the field to WireguardPeers. From my understanding, the generator script only modifies the settings modules but not the datatypes.py module which contains WireguardPeers. So I added the field manually instead.

Should the generator also populate datatypes.py or does it only generate settings modules?

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.

2 participants