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

Stop using legacy network interface names (eth0) #573

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Jul 29, 2024

It's no longer working for GCE and Azure appliances, they mark the connection as unmanaged and won't allow it to come up. Also, our scripts no longer depend on the interface name in EL9.

net.ifnames=0 reverts to legacy adapter names (eth0)
biosdevname does the same for Dell systems and requires a package we don't have installed
network-scripts are no longer used by NetworkManager

It's no longer working for GCE and Azure appliances, they mark the
connection as unmanaged and won't allow it to come up.  Also, our scripts
no longer depend on the interface name in EL9.

net.ifnames=0 reverts to legacy adapter names (eth0)
biosdevname does the same for Dell systems and requires a package we don't have installed
network-scripts are no longer used by NetworkManager
@miq-bot
Copy link
Member

miq-bot commented Jul 29, 2024

Checked commit bdunne@f9c43db with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@Fryguy
Copy link
Member

Fryguy commented Jul 29, 2024

Thanks for figuring this out @bdunne

cc @agrare thought you might be interested.

@Fryguy
Copy link
Member

Fryguy commented Jul 29, 2024

LGTM, but would like a second set of eyes. @agrare ?

@bdunne
Copy link
Member Author

bdunne commented Jul 29, 2024

I tested removing these kernel args from both Azure and GCP appliances locally and the interfaces came up.

@bdunne
Copy link
Member Author

bdunne commented Jul 29, 2024

May also be related to ManageIQ/manageiq#22973

@agrare
Copy link
Member

agrare commented Jul 29, 2024

Love it, and great that "doing the right thing" fixes these issues also. We do still assume eth0 in a few places that we'll need to update:

manageiq/app/models/miq_server/environment_management.rb:          eth0 = LinuxAdmin::NetworkInterface.new("eth0")
manageiq-appliance_console/bin/appliance_console:  eth0 = LinuxAdmin::NetworkInterface.new(NETWORK_INTERFACE)

@agrare
Copy link
Member

agrare commented Aug 13, 2024

There is one more place in appliance_console with eth0, database replication:

lib/manageiq/appliance_console/database_replication_primary.rb:      self.primary_host      = LinuxAdmin::NetworkInterface.new(NETWORK_INTERFACE).address
lib/manageiq/appliance_console/database_replication_standby.rb:      self.standby_host      = LinuxAdmin::NetworkInterface.new(NETWORK_INTERFACE).address
lib/manageiq/appliance_console/database_replication.rb:    NETWORK_INTERFACE = 'eth0'.freeze

Okay I've updated ManageIQ/manageiq-appliance_console#263 to take care of the database replication issue.

@bdunne
Copy link
Member Author

bdunne commented Aug 14, 2024

Ok, I think all related PRs are merged now.

@agrare agrare merged commit 7ef221c into ManageIQ:master Aug 14, 2024
2 checks passed
@agrare agrare assigned agrare and unassigned Fryguy Aug 14, 2024
@bdunne bdunne deleted the stop_using_eth0 branch August 14, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants