linux-system-roles / network

An ansible role to configure networking
https://linux-system-roles.github.io/network/
BSD 3-Clause "New" or "Revised" License
238 stars 106 forks source link

test: Clean up mock wifi at the end of each wireless test #670

Closed liangwen12year closed 10 months ago

liangwen12year commented 10 months ago

Enhancement:

Reason:

Result:

Issue Tracker Tickets (Jira or BZ if any):

liangwen12year commented 10 months ago

[citest]

liangwen12year commented 10 months ago

I tested it locally, even if running qemu testing using --make-batch, I can only reproduce the tests_wireless_wpa3_sae_nm.yml (CentOS-Stream-8/ansible-2.9 ) failure. And the other test failures are caused by baseos ci testing environment, notably, there are two categories:

  1. control node lost the ssh connection to the managed host

    Data could not be sent to remote host "10.31.43.169". Make sure this host can be reached over ssh: 
    to retry, use: --limit @/tmp/tests_bond_cloned_mac_initscripts.retry
  2. DNS problem

    
    TASK [Install iproute] *********************************************************
    task path: /WORKDIR/git-ci_fixslgi2sy4/tests/playbooks/tasks/manage_test_interface.yml:16
    Tuesday 23 January 2024  19:48:22 +0000 (0:00:00.018)       0:00:03.363 ******* 
    fatal: [sut]: FAILED! => {
    "changed": false,
    "rc": 1,
    "results": []
    }

MSG:

Failed to download metadata for repo 'epel': Cannot prepare internal mirrorlist: Curl error (6): Couldn't resolve host name for https://mirrors.fedoraproject.org/metalink?repo=epel-8&arch=x86_64&infra=stock&content=centos [Could not resolve host: mirrors.fedoraproject.org] to retry, use: --limit @/tmp/tests_route_device_initscripts.retry

liangwen12year commented 10 months ago

[citest]

codecov[bot] commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4f11374) 45.00% compared to head (a66c022) 45.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #670 +/- ## ======================================= Coverage 45.00% 45.00% ======================================= Files 12 12 Lines 3100 3100 Branches 433 433 ======================================= Hits 1395 1395 Misses 1704 1704 Partials 1 1 ``` | [Flag](https://app.codecov.io/gh/linux-system-roles/network/pull/670/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles) | Coverage Δ | | |---|---|---| | [sanity](https://app.codecov.io/gh/linux-system-roles/network/pull/670/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles) | `20.50% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

richm commented 10 months ago

[citest bad]