Skip to content

Commit

Permalink
Prevent NM from automatic (DHCP/SLAAC) configuration on ethernet devices
Browse files Browse the repository at this point in the history
When edpm_network_config role uses os-net-config to perform
host network configuration, duplicate DHCP requests from NM are causing
issue. Eg.
https://issues.redhat.com/browse/OSPRH-9142

This fix prevents NM to configure DHCP by default
  • Loading branch information
vcandapp committed Dec 9, 2024
1 parent c7cc7ed commit fbda264
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 4 deletions.
2 changes: 2 additions & 0 deletions roles/edpm_network_config/molecule/default/converge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,5 +33,7 @@
next_hop: 192.168.180.1
edpm_network_config_manage_service: false
edpm_network_config_hide_sensitive_logs: false
edpm_bootstrap_network_service: 'NetworkManager'
edpm_network_config_tool: 'os-net-config'
roles:
- role: "osp.edpm.edpm_network_config"
1 change: 1 addition & 0 deletions roles/edpm_network_config/molecule/nmstate/converge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,5 +33,6 @@
prefix-length: 24
dhcp: false
enabled: true
edpm_bootstrap_network_service: 'NetworkManager'
roles:
- role: osp.edpm.edpm_network_config
27 changes: 23 additions & 4 deletions roles/edpm_network_config/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,26 @@
- name: Load system-roles.network tasks [nmstate]
ansible.builtin.include_role:
name: "{{ lookup('ansible.builtin.env', 'EDPM_SYSTEMROLES', default='fedora.linux_system_roles') + '.network' }}"
- name: Load edpm_network_config tasks [os-net-config]
ansible.builtin.include_tasks:
file: network_config.yml
when: edpm_network_config_tool == 'os-net-config'

- name: Disable auto-configuration of all interfaces by NetworkManager
when: ( edpm_bootstrap_network_service == 'NetworkManager' ) and
( edpm_network_config_tool == 'os-net-config' )
become: true
block:
- name: Set 'no-auto-default' in /etc/NetworkManager/NetworkManager.conf
community.general.ini_file:
path: /etc/NetworkManager/NetworkManager.conf
state: present
mode: "0644"
no_extra_spaces: true
section: main
option: no-auto-default
value: "*"
backup: true
- name: Restart NetworkManager
ansible.builtin.systemd:
name: NetworkManager
state: restarted
- name: Load edpm_network_config tasks for os-net-config tool
ansible.builtin.include_tasks:
file: network_config.yml

0 comments on commit fbda264

Please sign in to comment.