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

Odroid M1: add a network rule to rename default name #6757

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Conversation

igorpecovnik
Copy link
Member

@igorpecovnik igorpecovnik commented Jun 17, 2024

Description

Renaming network device to eth0

How Has This Been Tested?

  • Generated image

Checklist:

  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added size/small PR with less then 50 lines Hardware Hardware related like kernel, U-Boot, ... labels Jun 17, 2024
@igorpecovnik igorpecovnik added 08 Milestone: Third quarter release Needs review Seeking for review labels Jun 17, 2024
@ColorfulRhino
Copy link
Collaborator

What's the difference between "${destination}" and "${SDCARD}" here?

@igorpecovnik
Copy link
Member Author

What's the difference between "${destination}" and "${SDCARD}" here?

destination -> BSP package
SDCARD -> image only

@ColorfulRhino
Copy link
Collaborator

destination -> BSP package

Thanks! Which package is it in this case? Is this something a user could download/update to their existing image?

Do you think "${destination}" is better for this stuff in general? If so, we could change it on every board config, many use "${SDCARD}".

@igorpecovnik
Copy link
Member Author

Which package is it in this case?

armbian-bsp-cli-$BOARD-$BRANCH

Is this something a user could download/update to their existing image?

yes, that's the idea.

@ColorfulRhino
Copy link
Collaborator

I thought about this a bit. Is it a good idea to change the name of a network interface via update package? Imagine if someone had for example set up some rules (static IP, ...) for this interface. Renaming via update might break those rules unless the old name remains as an alternate name and whatever network managing software they use recognizes this as the same interface.

So, maybe adding the new interface name "only" for new images wouldn't be that bad? I'm not sure, this is just what came to my mind, I haven't run any test.

@igorpecovnik
Copy link
Member Author

So, maybe adding the new interface name "only" for new images wouldn't be that bad?

Yes, that you are right. Lets keep this for new images.

@ColorfulRhino
Copy link
Collaborator

Alright 👍
I don't have this board so I can't test that the interface is properly renamed. But from the code it looks okay.

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge and removed Needs review Seeking for review labels Jun 25, 2024
@igorpecovnik
Copy link
Member Author

I can't test

Tested.

@igorpecovnik igorpecovnik merged commit 2a2e609 into main Jun 25, 2024
6 checks passed
@igorpecovnik igorpecovnik deleted the AR-2374 branch June 25, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
08 Milestone: Third quarter release Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants