agama-project / agama / 15491140774
73%

DEFAULT BRANCH: master
Ran
Jobs 3
Files 534
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 62.689%. Remained the same
15491140774

push

github

web-flow 
Network persistent connections handling (#2402)

## Problem

With the current approach to network configuration in **SLE 16**, it is
possible that you end up with a system with no network at all.

Why? 
- We started copying all connections to the target system, but that
meant volatile connections were copied too and maybe the defaults are
not wanted in the target system so now only the persistent connections
are copied (the ones in **/etc/NetworkManager/system-connections**).
- Some products install NetworkManager-config-server package which
disables the auto-default DHCP probing on ethernet devices (see
https://github.com/agama-project/agama/pull/2279)
  - 

Agama should:

Warn the user.

Allow the user to get out of that situation at any time.

- https://bugzilla.suse.com/show_bug.cgi?id=1241969
-
https://trello.com/c/P6oGGmjH/4745-ui-handle-persistent-non-persistent-network-connections

## Solution
- The connections generated by nm-initrd-generator are moved instead of
copied to ensure them are seeing as permanent and not as in-memory only
by NM avoiding duplications and only the ones generated by
nm-initrd-generator will be copied (**TO BE FIXED:** ibft, nbft and
other kind of connections should be probably omitted).
- If no connection is going to be copied the user will be warned but by
now only in the networking section (there is no check of
**NetworkManager-config-server** package yet for showing the warn).
- The user can select which interfaces will be used for installation
only and switch between made them persistent or not.
- We have modified the way the connection is updated and we are now
using the Update2 and AddConnection2 methods using the
**NMSettingsAddConnection2Flags** and **NMSettingsUpdate2Flags** (see
https://networkmanager.dev/docs/api/latest/nm-dbus-types.html#NMSettingsConnectionFlags)


## Current Network State

In general,

**If an explicit network configuration through the kernel cmdline is
given then:**
1. Agama disables t... (continued)

92 of 132 branches covered (69.7%)

Branch coverage included in aggregate %.

7 of 72 new or added lines in 9 files covered. (9.72%)

498 existing lines in 7 files now uncovered.

11130 of 17769 relevant lines covered (62.64%)

41.71 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
0.0
 0.0% rust/agama-network/src/settings.rs
3
0.0
 0.0% rust/agama-server/src/network/web.rs
5
0.0
 0.0% rust/agama-network/src/nm/adapter.rs
14
0.0
 0.0% rust/agama-network/src/model.rs
15
0.0
 0.0% rust/agama-network/src/types.rs
27
0.0
 0.0% rust/agama-network/src/nm/client.rs

Uncovered Existing Lines

Lines Coverage ∆ File
3
0.0
 0.0% rust/agama-network/src/settings.rs
8
0.0
 0.0% rust/agama-server/src/network/web.rs
16
85.92
 0.0% service/lib/agama/manager.rb
21
0.0
 0.0% rust/agama-network/src/nm/adapter.rs
32
0.0
 0.0% rust/agama-network/src/types.rs
73
0.0
 0.0% rust/agama-network/src/nm/client.rs
345
0.0
 0.0% rust/agama-network/src/model.rs
Subprojects
ID Flag name Job ID Ran Files Coverage
3 web 15491140774.3 36
82.3
 GitHub Action Run
3 service 15386206965.3 380
89.17
 GitHub Action Run
1 rust 15484131453.1 118
1.32
 GitHub Action Run
Source Files on build 15491140774