Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

liangwen12year
Copy link
Collaborator

Enhancement:

Reason:

Result:

Issue Tracker Tickets (Jira or BZ if any):

@liangwen12year
Copy link
Collaborator Author

[citest]

@liangwen12year liangwen12year marked this pull request as draft January 23, 2024 18:57
@liangwen12year
Copy link
Collaborator Author

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
  1. 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

Baseos CI runs different wireless tests in a sequence, the mock wifi
needs to be cleaned up properly at the end of each wireless test,
otherwise, it will cause the failure during the setup of mock wifi
for the next wireless test.

Signed-off-by: Wen Liang <[email protected]>
@liangwen12year
Copy link
Collaborator Author

[citest]

Copy link

codecov bot commented Jan 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files
@@           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 Coverage Δ
sanity 20.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@liangwen12year liangwen12year marked this pull request as ready for review January 26, 2024 20:41
@liangwen12year liangwen12year requested a review from richm January 26, 2024 20:43
@liangwen12year liangwen12year changed the title test: WIP - investigate baseos CI failures test: Clean up mock wifi at the end of each wireless test Jan 26, 2024
@richm
Copy link
Contributor

richm commented Jan 27, 2024

[citest bad]

@richm richm merged commit 9b76608 into linux-system-roles:main Jan 29, 2024
33 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants