Merge ~kissiel/plainbox-provider-snappy:fix-jobs-for-newer-stress-ng into plainbox-provider-snappy:master

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Maciej Kisielewski
Approved revision: 05a82a823575b7af8d1706169cf9e5e58e62922c
Merged at revision: 03f12c5209f69548c5b3feb73e709a15db59c77c
Proposed branch: ~kissiel/plainbox-provider-snappy:fix-jobs-for-newer-stress-ng
Merge into: plainbox-provider-snappy:master
Diff against target: 10 lines (+1/-1)
1 file modified
units/stress/stress-ng.pxu (+1/-1)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Review via email: mp+354686@code.launchpad.net

Description of the change

fix stress-ng invocation on newer stress-ng versions

At one point stress-ng changed the way it processess its CLI options making --class and then --sequential to blacklist stressors run.

This patch circumvents that problem.

To post a comment you must log in.
Revision history for this message
Jonathan Cave (jocave) wrote :

Strange, but looks ok if required to fix it

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/stress/stress-ng.pxu b/units/stress/stress-ng.pxu
2index eb3794b..9cbe615 100644
3--- a/units/stress/stress-ng.pxu
4+++ b/units/stress/stress-ng.pxu
5@@ -28,4 +28,4 @@ _description:
6 plugin: shell
7 estimated_duration: 1200.0
8 command:
9- stress-ng --class {stress-ng-class} --sequential 0
10+ stress-ng --sequential 0 --class {stress-ng-class}

Subscribers

People subscribed via source and target branches