Skip to content

Conversation

@silug
Copy link
Contributor

@silug silug commented Jul 8, 2025

Pull Request (PR) description

Implement a NetworkManager provider using nmstate

Also adds beaker tests.

This Pull Request (PR) fixes the following issues

@bastelfreak
Copy link
Member

@rwaffen recently started a network manager module: https://github.com/voxpupuli/puppet-networkmanager

Each new release typically also includes the latest modulesync defaults.
These should not affect the functionality of the module.

## [Unreleased]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the changelog for vox pupuli modules is generated based on closed issues and PRs, on each release. Please don't edit it manually, it will be overwritten.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, that's Copilot cruft. I'll clean it up before I submit this for real.

@silug
Copy link
Contributor Author

silug commented Jul 8, 2025

@rwaffen recently started a network manager module: https://github.com/voxpupuli/puppet-networkmanager

@bastelfreak I'm not sure what to do with this information. 😁

I still like the idea of having a theoretically generic module for handling network configuration that can work correctly with multiple back-end providers. That's why I've been using this module for ages.

@pedmon
Copy link

pedmon commented Jul 29, 2025

Just +1 ing this as we are starting to integrate Rocky 9 and need this to be updated so that we can properly define routes.

@bwitt
Copy link
Contributor

bwitt commented Aug 8, 2025

This looks very useful; I would love to see it get merged!

@bwitt
Copy link
Contributor

bwitt commented Aug 8, 2025

maybe I can help test on Ubuntu? that's where I'd like to use this

@silug
Copy link
Contributor Author

silug commented Aug 11, 2025

@bwitt @pedmon This is still very much a work in progress, but you are welcome to give it a try (on a test system please!) and let me know how it goes. I'll get back to working on this soon.

@bwitt
Copy link
Contributor

bwitt commented Aug 13, 2025

thanks but I'm hitting #77 when trying to use this module at all

@bwitt
Copy link
Contributor

bwitt commented Aug 17, 2025

ok running puppet generate types fixes #77 but I have a config in hiera, and it does not seem to be doing anything. I'm trying to configure a DHCP and a static connection, both using the same wired config

network_config:
  'Wired connection 1':
    ensure: present
    provider: nm
    family: inet
    method: static
    ipaddress: 'xxxx'
    netmask: '255.255.255.0'
    gateway: 'xxxxx'
    onboot: true
  'Wired connection 2':
    ensure: present
    provider: nm
    family: inet
    method: dhcp
    onboot: false
    hotplug: true


On Debian/Ubuntu:
```bash
apt-get install nmstate
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on what Ubuntu or Debian versions does this work?

@bwitt
Copy link
Contributor

bwitt commented Aug 20, 2025

solved my other problem, but now it appears I cannot apt install nmstate on Ubuntu 22.04 so I am blocked by that

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.

RHEL switch to networkmanager instead of network scripts Add support for RHEL9

4 participants