Skip to content

Conversation

Sunnatillo
Copy link
Member

@Sunnatillo Sunnatillo commented Aug 13, 2025

This PR makes changes to interface names after enabling predictable interface names in node images.
We have build centos node images with kernel parameter "net.ifnames=0", as a result we had interface names eth0, eth1 on centos nodes.
On project infra PR we are changing this parameters "net.ifnames=1". Now interface names will be predicted. Stable names based on hardware information. enp1s0, eno1.

@metal3-io-bot
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@metal3-io-bot metal3-io-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 13, 2025
@metal3-io-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign rozzii for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 13, 2025
@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch from feec983 to 116ac5a Compare August 13, 2025 10:51
@metal3-io-bot metal3-io-bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 13, 2025
@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch from 116ac5a to 335a12a Compare August 13, 2025 10:52
@Sunnatillo
Copy link
Member Author

/test metal3-centos-e2e-integration-test-main

@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch 2 times, most recently from 45b67e9 to f5b3b1f Compare August 13, 2025 11:42
@metal3-io-bot metal3-io-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 13, 2025
@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch 2 times, most recently from 19a6fd7 to 71733a2 Compare August 14, 2025 11:58
@metal3-io-bot metal3-io-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 14, 2025
@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch from 71733a2 to 3934126 Compare August 14, 2025 12:01
@Sunnatillo Sunnatillo changed the title WIP: Change interface name after disabling predicted names Change interface name after enabling predicted interface names Aug 18, 2025
@Sunnatillo Sunnatillo marked this pull request as ready for review August 18, 2025 07:37
@metal3-io-bot metal3-io-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 18, 2025
@Sunnatillo
Copy link
Member Author

/hold
It needs to merge at the same time with
metal3-io/project-infra#1053

@metal3-io-bot metal3-io-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 18, 2025
@tuminoid
Copy link
Member

Can you document in the PR description what is the value of this change?

Signed-off-by: Sunnatillo <sunnat.samadov@est.tech>
@Sunnatillo Sunnatillo force-pushed the Sunnatillo/fix-interface-name-vrrp branch from 3934126 to e34124a Compare August 19, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants