geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jared Stewart <jstew...@pivotal.io>
Subject Re: Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest
Date Wed, 07 Jun 2017 23:13:58 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59893/
-----------------------------------------------------------

(Updated June 7, 2017, 11:13 p.m.)


Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg.


Changes
-------

Upon further exception, MemberCommandsDUnitTest doesn't even read the "useHTTP" system property
being set by the CommandOverHttpDUnitTest Suite.  As such, we can simply remove it from the
suite.


Repository: geode


Description
-------

GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest


Diffs (updated)
-----

  geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/MemberCommandsDUnitTest.java
5bbfc5b 
  geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/CommandOverHttpDUnitTest.java
2a6a03b 


Diff: https://reviews.apache.org/r/59893/diff/2/

Changes: https://reviews.apache.org/r/59893/diff/1-2/


Testing
-------

Precheckin is green except for unrelated flaky DUnits.


Thanks,

Jared Stewart


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message