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

fix tear down and add readme file #2768

Merged
merged 2 commits into from
Apr 20, 2024

Conversation

disgoel
Copy link
Contributor

@disgoel disgoel commented Mar 6, 2024

Commit 1
The test result shows test passed with warning when test did not ran. Fix the tear down by adding a check to ensure removal of files only occurs if they exist.

Before fix:
avocado run perf_stress_ng.py -m perf_stress_ng.py.data/perf_stress_ng.yaml
Fetching asset from perf_stress_ng.py:Stressng.test
JOB ID : 2abafaa0af699b4a3918c7838dff061c9e774aaf
JOB LOG : /home/OpTest/avocado-fvt-wrapper/results/job-2024-03-06T04.10-2abafaa/job.log
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: STARTED
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: WARN: Test passed but there were warnings during execution. Check the log for details. (1.31 s)
RESULTS : PASS 0 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 1 | INTERRUPT 0 | CANCEL 0
JOB HTML : /home/OpTest/avocado-fvt-wrapper/results/job-2024-03-06T04.10-2abafaa/results.html
JOB TIME : 26.02 s

After fix:
avocado run perf_stress_ng.py -m perf_stress_ng.py.data/perf_stress_ng.yaml
Fetching asset from perf_stress_ng.py:Stressng.test
JOB ID : 4d8aed2110dd9338cb85d0d77649e9029cc8df14
JOB LOG : /home/OpTest/avocado-fvt-wrapper/results/job-2024-03-06T04.23-4d8aed2/job.log
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: STARTED
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: CANCEL: stress-ng is needed for the test to be run (0.74 s)
RESULTS : PASS 0 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
JOB HTML : /home/OpTest/avocado-fvt-wrapper/results/job-2024-03-06T04.23-4d8aed2/results.html
JOB TIME : 23.57 s
debug1.log

avocado run perf_stress_ng.py -m perf_stress_ng.py.data/perf_stress_ng.yaml
Fetching asset from perf_stress_ng.py:Stressng.test
JOB ID : 5e80e55d96efad7e1a655b4ed3baf3eb6d0c3ba0
JOB LOG : /home/avocado-fvt-wrapper/results/job-2024-03-08T08.31-5e80e55/job.log
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: STARTED
(1/1) perf_stress_ng.py:Stressng.test;run-coverage-run_type-df37: FAIL: CPU load 10 50 80 combination's failed with percentage difference {'50': 4.599999999999994, '80': 8.599999999999994} (201.27 s)
RESULTS : PASS 0 | ERROR 0 | FAIL 1 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML : /home/avocado-fvt-wrapper/results/job-2024-03-08T08.31-5e80e55/results.html
JOB TIME : 214.81 s
debug2.log

Commit 2
This commit introduces a README.txt file. The README provides a description of the test, outlines the steps followed during the test execution and lists the prerequisites necessary for running the test successfully.

The test result shows test passed with warning when test did not ran.
Fix the tear down by adding a check to ensure removal of files
only occurs if they exist.

Signed-off-by: Disha Goel <[email protected]>
@disgoel disgoel marked this pull request as ready for review March 8, 2024 14:42
@disgoel
Copy link
Contributor Author

disgoel commented Apr 18, 2024

PR to add psutil module in pre and post script
https://github.ibm.com/ltctest/avocado-fvt-wrapper/pull/585

This commit introduces a README.txt file. The README provides a
description of the test, outlines the steps followed during the
test execution and lists the prerequisites necessary for running
the test successfully.

Signed-off-by: Disha Goel <[email protected]>
@PraveenPenguin PraveenPenguin merged commit 4f1072a into avocado-framework-tests:master Apr 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants