1
42%
main: 42%

Ran 19 Nov 2020 11:00AM UTC

Files 13

Run time 2s

Badge

Embed ▾

Committed 19 Nov 2020 10:58AM UTC coverage: 39.255% (-1.3%) from 40.507%

Job # LSR_ANSIBLES='ansible==2.7.* ansible==2.8.* ansible==2.9.*' LSR_MSCENARIOS='default'

Build Type

push

travis-ci-com

Committed by

cathay4t

Commit Message

nm provider: Refactor the down action of network connection

When deactivating a profile in libNM, we should:

 * Check `NM.ActionConnection` existence
 * Check `NM.ActionConnection.props.state` not DEACTIVATED
 * Use signal `state-changed` of `NM.ActionConnection`.
 * Only invoke `NM.Client.deactivate_connection_async()` if not
   in DEACTIVATING state.
 * Ignore `NM.ManagerError.CONNECTIONNOTACTIVE` error.

This patch also introduced a new class `NetworkManagerProvider`
in `module_utils/network_lsr/nm`:

 * Independent from Ansible but need to use absolute import due to
   limitation of ansible 2.8.
 * Provide sync function wrapping async calls of libNM.
 * Use stable logging method of python.
 * Only load this module when provider is nm.

This patch also changed how logging is handling in
`Cmd_nm.run_action_down()` as initial step on isolate ansible log
mechanism from provider module.

By moving provider codes to `module_utils` folder, we can eventually
simplify the bloated `library/network_connections.py`.

Signed-off-by: Gris Ge <fge@redhat.com>

Run Details

948 of 2415 relevant lines covered (39.25%)

0.39 hits per line

linux-system-roles / network / 1244 / 1
42%
main: 42%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1244.1 (LSR_ANSIBLES='ansible==2.7.* ansible==2.8.* ansible==2.9.*' LSR_MSCENARIOS='default')

linux-system-roles / network / 1244 / 1 42% main: 42%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1244.1 (LSR_ANSIBLES='ansible==2.7.* ansible==2.8.* ansible==2.9.*' LSR_MSCENARIOS='default')

linux-system-roles / network / 1244 / 1
42%
main: 42%

README BADGES
x