search

home-assistant / architecture

Repo to discuss Home Assistant architecture
315 stars 99 forks source link

Adjust required version of NetworkManager #431

Closed cerebrate closed 3 years ago

cerebrate commented 4 years ago

Changes the required version of NetworkManager to match that available in Debian 10 (buster), since 1.18.0 is not. Since supervisor reports no errors running with 1.14.6, this should be a harmless change, and eliminates confusion in diagnosing such errors, as seen here - https://community.home-assistant.io/t/networkmanager-1-18-0-on-debian-buster/223968 - and in issue #427 .

frenck commented 4 years ago

This is not entirely true/correct. Originally, the Network manager was bumped to support WireGuard (networkmanager 1.16+). So, this might have future consequences. I do realize this feature of the network manager is currently not used though).

cerebrate commented 4 years ago

Hm. I do see the problem there, but at the moment, the seeming need for an impossible configuration (at least without mixing distro versions or compiling your own from source) seems like a bigger issue? At least until/unless a later NetworkManager makes it into buster-backports.

frenck commented 4 years ago

I think it is fine to merge like this.

@pvizeli ?

ludeeus commented 3 years ago

Version 1.14.6 does not work with our current implementation