You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by he...@apache.org on 2019/12/06 18:48:10 UTC
[geode-benchmarks] branch GEODE-7554 updated (9057a37 -> 7800b80)
This is an automated email from the ASF dual-hosted git repository.
heybales pushed a change to branch GEODE-7554
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git.
from 9057a37 write each test only once
new 224a023 only exit 1 from analyze task if not in CI
new 429aaeb Revert "only exit 1 from analyze task if not in CI"
new 7800b80 fail test without running for testing
The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java | 2 ++
1 file changed, 2 insertions(+)
[geode-benchmarks] 02/03: Revert "only exit 1 from analyze task if
not in CI"
Posted by he...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
heybales pushed a commit to branch GEODE-7554
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git
commit 429aaeb8706360476e2ec1068824018c849b8675
Author: Robert Houghton <rh...@pivotal.io>
AuthorDate: Thu Dec 5 16:29:20 2019 -0800
Revert "only exit 1 from analyze task if not in CI"
This reverts commit 224a02313c4da629ed51f4772d0343bf572cdf70.
Signed-off-by: Helena A. Bales <hb...@pivotal.io>
---
.../src/main/java/org/apache/geode/perftest/analysis/Analyzer.java | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java b/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
index ec23704..f4cca01 100644
--- a/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
+++ b/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
@@ -107,9 +107,7 @@ public class Analyzer {
if (errorMessage.length() > 0) {
System.out.println(errorMessage);
- if(!isCI) {
- System.exit(1);
- }
+ System.exit(1);
}
}
[geode-benchmarks] 01/03: only exit 1 from analyze task if not in CI
Posted by he...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
heybales pushed a commit to branch GEODE-7554
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git
commit 224a02313c4da629ed51f4772d0343bf572cdf70
Author: Helena A. Bales <hb...@pivotal.io>
AuthorDate: Thu Dec 5 16:24:46 2019 -0800
only exit 1 from analyze task if not in CI
Signed-off-by: Robert Houghton <rh...@pivotal.io>
---
.../src/main/java/org/apache/geode/perftest/analysis/Analyzer.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java b/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
index f4cca01..ec23704 100644
--- a/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
+++ b/harness/src/main/java/org/apache/geode/perftest/analysis/Analyzer.java
@@ -107,7 +107,9 @@ public class Analyzer {
if (errorMessage.length() > 0) {
System.out.println(errorMessage);
- System.exit(1);
+ if(!isCI) {
+ System.exit(1);
+ }
}
}
[geode-benchmarks] 03/03: fail test without running for testing
Posted by he...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
heybales pushed a commit to branch GEODE-7554
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git
commit 7800b80053b135f2d72652291c1166dfdc3164c9
Author: Helena A. Bales <hb...@pivotal.io>
AuthorDate: Fri Dec 6 10:46:47 2019 -0800
fail test without running for testing
Signed-off-by: Robert Houghton <rh...@pivotal.io>
---
.../src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java | 2 ++
1 file changed, 2 insertions(+)
diff --git a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
index 8d4e612..8609bf1 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
@@ -18,6 +18,7 @@
package org.apache.geode.benchmark.tests;
+import static java.lang.System.exit;
import static org.apache.geode.benchmark.topology.ClientServerTopology.Roles.CLIENT;
import org.junit.jupiter.api.Test;
@@ -42,6 +43,7 @@ public class NoopBenchmark implements PerformanceTest {
@Override
public TestConfig configure() {
+ exit(1);
TestConfig config = GeodeBenchmark.createConfig();
ClientServerTopology.configure(config);
config.workload(new NoopTask(), CLIENT);