Skip to content

Temporarily disable OOM tests for ppc64le#36541

Closed
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:power-disable-oom-tests
Closed

Temporarily disable OOM tests for ppc64le#36541
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:power-disable-oom-tests

Conversation

@thaJeztah
Copy link
Member

Tests are failing due to configuration changes of the machines running CI.

see #36540

ping @vdemeester @dnephin @tophj-ibm @andrewhsu

Tests are failing due to configuration changes
of the machines running CI.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Copy link
Contributor

@seemethere seemethere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM when green on ppc64le

Copy link
Member

@dnephin dnephin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Either this or #36539

@andrewhsu
Copy link
Contributor

cc @psftw

@tophj-ibm
Copy link
Contributor

I'd prefer #36539 just because it also skips TestEventsOOMDisableTrue, which was also re-enabled (or you can add that here, doesn't matter to me)

@thaJeztah
Copy link
Member Author

That's fine, I can close this one (saw that TestEventsOOMDisableTrue seemed to still pass, so didn't disable it)

@thaJeztah thaJeztah closed this Mar 9, 2018
@thaJeztah thaJeztah deleted the power-disable-oom-tests branch April 6, 2018 18:27
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.

6 participants