Skip to content

Remove stress/cpu_stress_ng_test from SRU (Bugfix)#1452

Merged
Hook25 merged 1 commit intomainfrom
remove-cpu-stress-ng
Sep 4, 2024
Merged

Remove stress/cpu_stress_ng_test from SRU (Bugfix)#1452
Hook25 merged 1 commit intomainfrom
remove-cpu-stress-ng

Conversation

@jocave
Copy link
Copy Markdown
Contributor

@jocave jocave commented Sep 4, 2024

Description

Resolved issues

Documentation

Tests

WARNING: This modifies com.canonical.certification::sru-server

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.87%. Comparing base (924882a) to head (923a683).
Report is 115 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1452   +/-   ##
=======================================
  Coverage   45.87%   45.87%           
=======================================
  Files         367      367           
  Lines       39156    39156           
  Branches     6622     6622           
=======================================
  Hits        17961    17961           
  Misses      20504    20504           
  Partials      691      691           
Flag Coverage Δ
provider-sru 97.97% <ø> (ø)

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.

nadzyah
nadzyah previously approved these changes Sep 4, 2024
@nadzyah nadzyah changed the title Remove stress/cpu_stress_ng_test from SRU Remove stress/cpu_stress_ng_test from SRU (Bugfix) Sep 4, 2024
@nadzyah
Copy link
Copy Markdown
Member

nadzyah commented Sep 4, 2024

Looks like you need to fix the tests as well

@nadzyah nadzyah dismissed their stale review September 4, 2024 10:54

Tests need to be fixed

@Hook25 Hook25 merged commit 0881101 into main Sep 4, 2024
@Hook25 Hook25 deleted the remove-cpu-stress-ng branch September 4, 2024 11:09
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.

3 participants