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

[Arista] rename management interface in initrd #9856

Merged
merged 1 commit into from
Mar 21, 2022

Conversation

Staphylo
Copy link
Collaborator

Why I did it

On some products the pci enumeration adds randomness into which nic gets
initialized first.
Because SONiC doesn't use deterministic interface naming but instead old
style interface naming, this leads to eth0 not always being the
management port.
To make sure eth0 is always the management port (SONiC expectation)
rename the interfaces in the initramfs for Arista products.

How I did it

Rename interfaces in the initramfs based on kernel cmdline parameters

How to verify it

eth0 is the management port

Description for the changelog

Rename management interface to always be eth0 on Arista devices

@Staphylo Staphylo requested a review from lguohan as a code owner January 26, 2022 10:39
@Staphylo Staphylo force-pushed the master-mgmt-intf-name branch from 8221603 to 350c72c Compare January 28, 2022 10:26
@Staphylo
Copy link
Collaborator Author

Staphylo commented Feb 1, 2022

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@Staphylo Staphylo force-pushed the master-mgmt-intf-name branch from 350c72c to 64aaa55 Compare February 3, 2022 16:49
@Staphylo Staphylo force-pushed the master-mgmt-intf-name branch from 64aaa55 to 7c8b252 Compare February 14, 2022 14:21
On some products the pci enumeration adds randomness into which nic gets
initialized first.
Because SONiC doesn't use deterministic interface naming but instead old
style interface naming, this leads to eth0 not always being the
management port.
To make sure eth0 is always the management port (SONiC expectation)
rename the interfaces in the initramfs for Arista products.
@Staphylo Staphylo force-pushed the master-mgmt-intf-name branch from 7c8b252 to 5faa65f Compare February 28, 2022 13:44
@Staphylo
Copy link
Collaborator Author

@prgeor

@prgeor prgeor merged commit e4b507f into sonic-net:master Mar 21, 2022
Staphylo added a commit to Staphylo/sonic-buildimage that referenced this pull request Apr 20, 2022
On some products the pci enumeration adds randomness into which nic gets
initialized first.
Because SONiC doesn't use deterministic interface naming but instead old
style interface naming, this leads to eth0 not always being the
management port.
To make sure eth0 is always the management port (SONiC expectation)
rename the interfaces in the initramfs for Arista products.
lguohan pushed a commit that referenced this pull request Apr 20, 2022
On some products the pci enumeration adds randomness into which nic gets
initialized first.
Because SONiC doesn't use deterministic interface naming but instead old
style interface naming, this leads to eth0 not always being the
management port.
To make sure eth0 is always the management port (SONiC expectation)
rename the interfaces in the initramfs for Arista products.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants