Skip to content

redhat: Use NetworkManager to set DHCP hostnames on recent RHEL distros#3151

Closed
ani-sinha wants to merge 1 commit into
Azure:masterfrom
ani-sinha:rhel-set-dhcp-hostnames
Closed

redhat: Use NetworkManager to set DHCP hostnames on recent RHEL distros#3151
ani-sinha wants to merge 1 commit into
Azure:masterfrom
ani-sinha:rhel-set-dhcp-hostnames

Conversation

@ani-sinha
Copy link
Copy Markdown

Recent versions of RHEL 8 and RHEL 9 started using NetworkManager to configure network intefaces instead of sysconfig files. Configurations are no longer stored in /etc/sysconfig/ifcfg-{interface}.

Fix this for setting DHCP hostnames in those RHEL versions.

Recent versions of RHEL 8 and RHEL 9 started using NetworkManager to configure
network intefaces instead of sysconfig files. Configurations are no longer
stored in /etc/sysconfig/ifcfg-{interface}.

Fix this for setting DHCP hostnames in those RHEL versions.

Signed-off-by: Ani Sinha <anisinha@redhat.com>
@maddieford
Copy link
Copy Markdown
Contributor

@ani-sinha Thank you for the PR. Hostname publishing on Rhel 8/9 is a high priority item for us. I'll work on stress testing this solution over the next few weeks.

@ani-sinha
Copy link
Copy Markdown
Author

ani-sinha commented Jun 21, 2024

@maddieford actually we have been using this patch downstream for a long time on RHEL 9. I just wanted you to know push this upstream so that we can drop this downstream only patch.

@vittyvk
Copy link
Copy Markdown
Contributor

vittyvk commented Oct 18, 2024

I somehow missed this PR and sent my own version: #3243
the difference between the two is that I do not believe "nmcli device modify" is sufficient and thus I do "nmcli connection modify" instead. This way, the change is supposed to be persistent.

@ani-sinha
Copy link
Copy Markdown
Author

I am closing this in favor of #3243

@ani-sinha ani-sinha closed this Dec 11, 2024
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