test: fix test failure on ubuntu 24.04: "top: unknown option 'n'"

weizhouapache commented 2 weeks ago

Description

This PR fixes the smoke test failure test_service_offering.py on Ubuntu 24.04

======================================================================
ERROR: Test CPU Cap on KVM
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/marvin/lib/decoratorGenerators.py", line 30, in test_wrapper
    return test(self, *args, **kwargs)
  File "/marvin/tests/smoke/test_service_offerings.py", line 1049, in test_01_service_offering_cpu_limit_use
    host_cpu_usage_before = round(float(host_cpu_usage_before_str))
ValueError: could not convert string to float: "top: unknown option 'n'"
-------------------- >> begin captured stdout << ---------------------
=== TestName: test_01_service_offering_cpu_limit_use | Status : EXCEPTION ===

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 14.95%. Comparing base (00fe25a) to head (53c0d9a). Report is 2 commits behind head on 4.19.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## 4.19 #9262 +/- ## ============================================ - Coverage 14.95% 14.95% -0.01% + Complexity 11017 11013 -4 ============================================ Files 5378 5378 Lines 469935 469935 Branches 59493 58185 -1308 ============================================ - Hits 70301 70289 -12 - Misses 391847 391860 +13 + Partials 7787 7786 -1 ``` | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/9262/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/9262/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `4.29% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/9262/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `15.66% <ø> (-0.01%)` | :arrow_down: | 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=apache#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.