Use NetworkManager #241

zehortigoza · 2017-08-01T23:00:47Z

No description provided.

avinash-palleti · 2017-08-02T06:30:10Z

@zehortigoza I tried with this PR and it is working.
I am able to camp to live network.

I am getting below message in journalctl, but i think it is just warning.
"Couldn't load power state: 'Failed to parse +CFUN? response '1,0' "

vosst · 2017-08-02T06:42:25Z

recipes-connectivity/networkmanager/networkmanager_%.bbappend

+PACKAGECONFIG += "ppp"
+
+DEPENDS += "nss"
+PACKAGECONFIG += "systemd"


Missing newline

vosst · 2017-08-02T06:42:35Z

recipes-core/images/intel-aero-image-modem.bb

+
+IMAGE_INSTALL += "modemmanager"
+IMAGE_INSTALL += "networkmanager"
+IMAGE_INSTALL += "ppp"


Missing newline

vosst · 2017-08-02T13:38:35Z

We tested the latest changes, too, and we can connect the drone to the network. Great work, thanks a lot 👍

zehortigoza · 2017-08-02T20:13:46Z

Now it is setting the DNS automatically.

lucasdemarchi · 2017-08-02T20:18:33Z

You need to configure NetworkManager to ignore the usb-ethernet interface, otherwise it will fight with systemd-networkd.

lucasdemarchi · 2017-08-02T20:21:13Z

see https://wiki.archlinux.org/index.php/NetworkManager#Ignore_specific_devices

anselmolsm · 2017-08-02T20:30:57Z

conf/local.conf.sample

@@ -243,9 +243,6 @@ RDEPENDS_packagegroup-core-full-cmdline-sys-services_remove=" nfs-utils at tcp-w
 PACKAGE_EXCLUDE+=" ed"
 RDEPENDS_packagegroup-core-full-cmdline-utils_remove = " ed"

-PACKAGE_EXCLUDE+=" wireless-tools"
-RDEPENDS_packagegroup-base-wifi_remove = " wireless-tools"
-


These PACKAGE_EXCLUDE, RDEPENDS_packagegroup-xyz should be moved to the images bb files. RDEPENDS_packagegroup_xyz lines remain the same, PACKAGE_EXCLUDE lines becomes IMAGE_INSTALL_remove.

zehortigoza · 2017-08-02T23:45:01Z

You need to configure NetworkManager to ignore the usb-ethernet interface, otherwise it will fight with systemd-networkd.

I will leave this as it is because it is working and when I try to set the usb-ethernet as unmanageable by NM it loses the ip address.

lucasdemarchi · 2017-08-03T06:46:17Z

conf/conf-notes.txt

@@ -1,5 +1,7 @@
 Supported targets:
    intel-aero-image
+    intel-aero-image-test
+    intel-aero-image-modem


Let's not create another image...

lucasdemarchi · 2017-08-03T06:47:45Z

recipes-connectivity/networkmanager/networkmanager/firstboot-networkmanager-setup

+
+# Let NetworkManager start dnsmasq with the right configuration file
+systemctl disable dnsmasq
+systemctl stop dnsmasq


This should be a bbappend on dnsmasq recipe to not enable it.

lucasdemarchi · 2017-08-03T06:48:50Z

recipes-connectivity/networkmanager/networkmanager/firstboot-networkmanager-setup

+WIFI_AP_DEFAULTPASSWORD="1234567890"
+
+# This is the APN name for AT&T HSPA+ user should change this with:
+# nmcli con modify Modem gsm.apn <name>


So I guess this should be read from a configuration file

lucasdemarchi · 2017-08-03T06:52:03Z

recipes-connectivity/networkmanager/networkmanager/firstboot-networkmanager-setup

+nmcli con add type gsm ifname '*' con-name Modem apn $MODEM_APN_NAME autoconnect yes
+nmcli con up Modem
+
+# USB over ethernet


I don't think we should touch this interface. It's the one "debug port" we have and I'd prefer to leave it with systemd-networkd.

It is the solution that I found to fix the resolv.conf bug, systemd created a symbolic link in /etc/resolv.conf and it do not let NM to se the modem dns.
Although I did not had time to flash the image with this last patch.

avinash-palleti · 2017-08-03T08:09:21Z

I verified latest commit, AP mode is working fine. Able to connect to Aero hotspot.
Not able to switch to client mode. I followed the steps from below wiki,
https://github.com/intel-aero/meta-intel-aero/wiki/Functionality

Scripts that wiki is pointing to are not available in current image, got those scripts from below commit,
1e17012

Scripts i took autostart-supplicant.sh, autostart-hostapd.sh and p2p_supplicant.conf
These scripts require dhclient to be available in the system which is not present in current image. I installed same using rpm.

It is showing the type as manager after running "autostart-supplicant start" but when i run "wpa_cli status" it is giving below error,

"Failed to connect to non-global ctrl_ifname: (nil) error: No such file or directory"

I found below difference in networkmanager recipe when compared to one in v1.3.1 image,

EXTRA_OECONF = --disable-polkit
--enable-more-warnings
--disable-gtk-doc
--disable-vala
--without-docs
--with-gnu-ld
--with-session-tracking=none
--with-dhclient=${base_sbindir}/dhclient
--with-dhcpcd=no
--with-resolvconf=no
--with-iptables=${sbindir}/iptables
--enable-ifupdown
--with-modem-manager-1
--disable-ifcfg-rh
--disable-ifcfg-suse
--disable-ifnet
--without-netconfig
--without-resolvconf
--enable-introspection=no \

Am i missing anything ?

avinash-palleti · 2017-08-03T08:36:09Z

usb net is working i can able to ssh from Ubuntu to Aero through usb interface.

But i observed by default ip address is not getting assigned to usb interface, i ran below command on Aero to get ip address assigned to usb interface,

ifconfig usb0 192.168.7.2 up

Then again i ran below command on Ubuntu,

sudo ifconfig enp0s20u12 192.168.7.1 up

After this i could able to do ssh.

lucasdemarchi · 2017-08-03T20:10:44Z

recipes-connectivity/networkmanager/networkmanager/firstboot-networkmanager-setup

@@ -28,6 +28,11 @@ done
 systemctl disable dnsmasq
 systemctl stop dnsmasq

+rm /etc/resolv.conf


this should be a change to the installed tmpfiles.d

change the systemd recipe to do that?

maybe for next release we could bring NM 1.6 that plays better with systemd-resolved so we wouldn't need resolveconf

Is not better keep all the workarrounds in here than spread it into .bbappend of systemd and dnsmasq? This way if in future we want to go back to connman(after someone adds support to our modem in ofono) it would be just remove the nm and mm from image and add comman and ofono.

zehortigoza · 2017-08-03T20:19:13Z

@avinash-palleti we will provide new instructions of how change to wifi client.

This would do it:

# Turn Wifi hostspot down and disable the auto connect
nmcli c down Wifi-hostspot
nmcli c modify Wifi-hostspot connection.autoconnect no

# To scan Wifi networks
nmcli dev wifi
# Connect to Wifi network(by default it will auto connect)
nmcli d wifi connect '<SSID>' password '<password>'

This is necessary to build network manager

ModemManager have support to Intel modems, ofono still don't have. So replacing Connman with NetworkManager in image.

Same setup as we had with connman + modem connection.

zehortigoza · 2017-08-04T00:56:59Z

@avinash-palleti Thanks for the test and review, we decided to keep usb0 over systemd so now it should be working again

But i observed by default ip address is not getting assigned to usb interface, i ran below command on Aero to get ip address assigned to usb interface,

anselmolsm requested review from lucasdemarchi and avinash-palleti August 1, 2017 23:03

vosst reviewed Aug 2, 2017

View reviewed changes

zehortigoza force-pushed the modem branch 2 times, most recently from 4cb25a1 to 240b86b Compare August 2, 2017 20:12

anselmolsm reviewed Aug 2, 2017

View reviewed changes

zehortigoza force-pushed the modem branch 2 times, most recently from d7575fe to be094f9 Compare August 2, 2017 23:08

zehortigoza force-pushed the modem branch 4 times, most recently from a334b7b to 218d652 Compare August 3, 2017 00:54

anselmolsm added this to the v1.5 milestone Aug 3, 2017

lucasdemarchi suggested changes Aug 3, 2017

View reviewed changes

lucasdemarchi reviewed Aug 3, 2017

View reviewed changes

zehortigoza force-pushed the modem branch 5 times, most recently from 6a1d1d7 to fc71b43 Compare August 3, 2017 23:09

zehortigoza force-pushed the modem branch from fc71b43 to cc37d96 Compare August 3, 2017 23:10

zehortigoza added 4 commits August 3, 2017 16:21

conf: Allow wireless-tools back into image

54c70c5

This is necessary to build network manager

intel-aero-image: Install NetworkManager and ModemManager

eea5e48

ModemManager have support to Intel modems, ofono still don't have. So replacing Connman with NetworkManager in image.

networkmanager: Configure networks

40d7760

Same setup as we had with connman + modem connection.

modemmanager: Add runtime dependency that Intel modems have

1b492b9

zehortigoza force-pushed the modem branch from cc37d96 to 1b492b9 Compare August 3, 2017 23:23

zehortigoza changed the title ~~WIP: Enable modem~~ Enable modem Aug 3, 2017

lucasdemarchi mentioned this pull request Aug 4, 2017

Release updates #252

Merged

lucasdemarchi changed the title ~~Enable modem~~ Use NetworkManger Aug 4, 2017

lucasdemarchi changed the title ~~Use NetworkManger~~ Use NetworkManager Aug 4, 2017

lucasdemarchi approved these changes Aug 4, 2017

View reviewed changes

lucasdemarchi merged commit 62b679c into intel-aero:master Aug 4, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use NetworkManager #241

Use NetworkManager #241

zehortigoza commented Aug 1, 2017 •

edited

Loading

avinash-palleti commented Aug 2, 2017 •

edited

Loading

vosst Aug 2, 2017

vosst Aug 2, 2017

vosst commented Aug 2, 2017

zehortigoza commented Aug 2, 2017

lucasdemarchi commented Aug 2, 2017

lucasdemarchi commented Aug 2, 2017

anselmolsm Aug 2, 2017

zehortigoza commented Aug 2, 2017

lucasdemarchi Aug 3, 2017

lucasdemarchi Aug 3, 2017

lucasdemarchi Aug 3, 2017

lucasdemarchi Aug 3, 2017

zehortigoza Aug 3, 2017

avinash-palleti commented Aug 3, 2017

avinash-palleti commented Aug 3, 2017

lucasdemarchi Aug 3, 2017

zehortigoza Aug 3, 2017

lucasdemarchi Aug 3, 2017

lucasdemarchi Aug 3, 2017

zehortigoza Aug 3, 2017

zehortigoza commented Aug 3, 2017 •

edited

Loading

zehortigoza commented Aug 4, 2017

Use NetworkManager #241

Use NetworkManager #241

Conversation

zehortigoza commented Aug 1, 2017 • edited Loading

avinash-palleti commented Aug 2, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

vosst commented Aug 2, 2017

zehortigoza commented Aug 2, 2017

lucasdemarchi commented Aug 2, 2017

lucasdemarchi commented Aug 2, 2017

Choose a reason for hiding this comment

zehortigoza commented Aug 2, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

avinash-palleti commented Aug 3, 2017

avinash-palleti commented Aug 3, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

zehortigoza commented Aug 3, 2017 • edited Loading

zehortigoza commented Aug 4, 2017

zehortigoza commented Aug 1, 2017 •

edited

Loading

avinash-palleti commented Aug 2, 2017 •

edited

Loading

zehortigoza commented Aug 3, 2017 •

edited

Loading