board/MynaPlayer odyssey: fix IPv6 for the wired connection. #12

Merged xogium merged 1 commit into LuminaSensum:stable-dev-board from LuminaSensum:NM-ipv6-fixups on 6 Sep 2020
@xogium xogium commented on 1 Sep 2020

This fix IPv6 by making SLLAC addresses predictable again -- disables stable-privacy which makes entirely unpredictable addresses.

Should UUID be removed? Would NetworkManager automatically generate a new one?

I don't believe uuid should be removed. The NetworkManager folks seemed to imply that we should copy the generated nmconnection file as it is in our buildroot overlay.

As for the permissions, this has now been fixed, and this should now work as expected. NetworkManager requires mode 0600 on the nmconnection files.

It sounds good to me as long as the UUID doesn't map to machine or network-specific state or something like that. If the UUID is just used for making configurations themselves unique that seems fine.

I think this is to make the connection unique. That said I don't know how they are generated. But the folks seemed to say that we could just pre-deploy that file even on multiple devices, so I figure they know what they are talking about... But maybe it is better to check the doc, though I have no idea where we could find doc about uuid generation.

If they said it's okay and it's for connection UUIDs, that makes sense and I'm good with this merging.

@xogium xogium referenced the pull request on 6 Sep 2020

board/MynaPlayer odyssey: fix IPv6 for the wired connection.

@xogium xogium merged commit 242ed6a into stable-dev-board from NM-ipv6-fixups on 6 Sep 2020
@xogium xogium closed this pull request on 6 Sep 2020
@xogium xogium deleted the NM-ipv6-fixups branch on 6 Sep 2020
Labels

Priority
default
Milestone
No milestone
Assignee
No one assigned
2 participants
@xogium @Jookia