You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by dp...@apache.org on 2018/11/14 11:47:13 UTC
[ignite-teamcity-bot] branch ignite-10243 updated (06a61f7 ->
e02db4f)
This is an automated email from the ASF dual-hosted git repository.
dpavlov pushed a change to branch ignite-10243
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git.
from 06a61f7 IGNITE-10243 Chain analysis entry points are get now from preloaded builds supporting cancelled
add fc0905c IGNITE-10215 Suite critical failures registration added - Fixes #70.
new 10d13c5 Merge branch 'master' into ignite-10243
new e02db4f IGNITE-10243 Cancelled is now shown as problem
The 2 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:
.../org/apache/ignite/ci/IgnitePersistentTeamcity.java | 10 ++++++++++
.../org/apache/ignite/ci/IgniteTeamcityConnection.java | 7 +++++++
.../src/main/java/org/apache/ignite/ci/TcHelper.java | 2 +-
.../org/apache/ignite/ci/analysis/MultBuildRunCtx.java | 15 ++++++++++++++-
.../org/apache/ignite/ci/analysis/SingleBuildRunCtx.java | 4 ++++
.../ignite/ci/teamcity/ignited/BuildRefCompacted.java | 6 +++++-
.../teamcity/ignited/fatbuild/ProactiveFatBuildSync.java | 2 +-
.../org/apache/ignite/ci/teamcity/pure/ITeamcityConn.java | 4 ++++
.../ci/web/model/current/ChainAtServerCurrentStatus.java | 3 ++-
.../ignite/ci/web/model/current/SuiteCurrentStatus.java | 9 ++-------
10 files changed, 50 insertions(+), 12 deletions(-)
[ignite-teamcity-bot] 02/02: IGNITE-10243 Cancelled is now shown as
problem
Posted by dp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
dpavlov pushed a commit to branch ignite-10243
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git
commit e02db4f70af3028d34f3d943165a75aa45b43c1e
Author: Dmitriy Pavlov <dp...@apache.org>
AuthorDate: Wed Nov 14 14:47:12 2018 +0300
IGNITE-10243 Cancelled is now shown as problem
---
.../src/main/java/org/apache/ignite/ci/TcHelper.java | 2 +-
.../org/apache/ignite/ci/analysis/MultBuildRunCtx.java | 15 ++++++++++++++-
.../org/apache/ignite/ci/analysis/SingleBuildRunCtx.java | 4 ++++
.../ignite/ci/teamcity/ignited/BuildRefCompacted.java | 6 +++++-
.../ci/web/model/current/ChainAtServerCurrentStatus.java | 3 ++-
.../ignite/ci/web/model/current/SuiteCurrentStatus.java | 9 ++-------
6 files changed, 28 insertions(+), 11 deletions(-)
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/TcHelper.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/TcHelper.java
index 44facf2..12a4f03 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/TcHelper.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/TcHelper.java
@@ -330,7 +330,7 @@ public class TcHelper implements ITcHelper, IJiraIntegration {
}
if (failType != null)
- fails.computeIfAbsent(failType, k->new ArrayList<>()).add(suite);
+ fails.computeIfAbsent(failType, k -> new ArrayList<>()).add(suite);
}
return fails;
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/MultBuildRunCtx.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/MultBuildRunCtx.java
index 56613c2..50679a2 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/MultBuildRunCtx.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/MultBuildRunCtx.java
@@ -95,6 +95,10 @@ public class MultBuildRunCtx implements ISuiteResults {
.anyMatch(p -> !p.isFailedTests(compactor) && !p.isSnapshotDepProblem(compactor));
}
+ public boolean onlyCancelledBuilds() {
+ return buildsStream().allMatch(bCtx -> !bCtx.isComposite() && bCtx.isCancelled());
+ }
+
@NotNull
private Stream<ProblemCompacted> allProblemsInAllBuilds() {
return buildsStream().flatMap(SingleBuildRunCtx::getProblemsStream);
@@ -152,6 +156,15 @@ public class MultBuildRunCtx implements ISuiteResults {
public String getResult() {
StringBuilder res = new StringBuilder();
+ long cancelledCnt = buildsStream().filter(bCtx -> !bCtx.isComposite() && bCtx.isCancelled()).count();
+
+ if (cancelledCnt > 0) {
+ res.append("CANCELLED");
+
+ if (cancelledCnt > 1)
+ res.append(" ").append(cancelledCnt);
+ }
+
addKnownProblemCnt(res, "TIMEOUT", getExecutionTimeoutCount());
addKnownProblemCnt(res, "JVM CRASH", getJvmCrashProblemCount());
addKnownProblemCnt(res, "Out Of Memory Error", getOomeProblemCount());
@@ -257,7 +270,7 @@ public class MultBuildRunCtx implements ISuiteResults {
}
boolean isFailed() {
- return failedTests() != 0 || hasAnyBuildProblemExceptTestOrSnapshot();
+ return failedTests() != 0 || hasAnyBuildProblemExceptTestOrSnapshot() || onlyCancelledBuilds();
}
public String branchName() {
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/SingleBuildRunCtx.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/SingleBuildRunCtx.java
index 5560929..942d890 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/SingleBuildRunCtx.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/analysis/SingleBuildRunCtx.java
@@ -201,4 +201,8 @@ public class SingleBuildRunCtx implements ISuiteResults {
this.changes.clear();
this.changes.addAll(changes);
}
+
+ public boolean isCancelled() {
+ return buildCompacted.isCancelled(compactor);
+ }
}
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/teamcity/ignited/BuildRefCompacted.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/teamcity/ignited/BuildRefCompacted.java
index a0f6d36..61f984a 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/teamcity/ignited/BuildRefCompacted.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/teamcity/ignited/BuildRefCompacted.java
@@ -155,7 +155,11 @@ public class BuildRefCompacted {
}
public boolean isNotCancelled(IStringCompactor compactor) {
- return !hasUnknownStatus(compactor);
+ return !isCancelled(compactor);
+ }
+
+ public boolean isCancelled(IStringCompactor compactor) {
+ return hasUnknownStatus(compactor);
}
private boolean hasUnknownStatus(IStringCompactor compactor) {
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/ChainAtServerCurrentStatus.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/ChainAtServerCurrentStatus.java
index 8aacb86..cae9d95 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/ChainAtServerCurrentStatus.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/ChainAtServerCurrentStatus.java
@@ -97,10 +97,11 @@ public class ChainAtServerCurrentStatus {
stream.forEach(
suite -> {
final SuiteCurrentStatus suiteCurStatus = new SuiteCurrentStatus();
+
suiteCurStatus.initFromContext(teamcity, suite, tcAnalytics, baseBranchTc);
failedTests += suiteCurStatus.failedTests;
- if (suite.hasAnyBuildProblemExceptTestOrSnapshot())
+ if (suite.hasAnyBuildProblemExceptTestOrSnapshot() || suite.onlyCancelledBuilds())
failedToFinish++;
this.suites.add(suiteCurStatus);
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/SuiteCurrentStatus.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/SuiteCurrentStatus.java
index 0784e96..9e64711 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/SuiteCurrentStatus.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/SuiteCurrentStatus.java
@@ -183,13 +183,8 @@ import static org.apache.ignite.ci.util.UrlUtil.escape;
}
);
- Stream<Map.Entry<String, Long>> stream = suite.getTopLogConsumers();
-
- stream.forEach(
- (entry) -> {
- TestFailure failure = createOccurForLogConsumer(entry);
- logConsumers.add(failure);
- }
+ suite.getTopLogConsumers().forEach(
+ (entry) -> logConsumers.add(createOccurForLogConsumer(entry))
);
suite.getBuildsWithThreadDump().forEach(buildId -> {
[ignite-teamcity-bot] 01/02: Merge branch 'master' into ignite-10243
Posted by dp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
dpavlov pushed a commit to branch ignite-10243
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git
commit 10d13c5a4b686ef1b7982b05a9ef8f3a172c15a9
Merge: 06a61f7 fc0905c
Author: Dmitriy Pavlov <dp...@apache.org>
AuthorDate: Wed Nov 14 13:00:02 2018 +0300
Merge branch 'master' into ignite-10243
.../java/org/apache/ignite/ci/IgnitePersistentTeamcity.java | 10 ++++++++++
.../java/org/apache/ignite/ci/IgniteTeamcityConnection.java | 7 +++++++
.../ci/teamcity/ignited/fatbuild/ProactiveFatBuildSync.java | 2 +-
.../java/org/apache/ignite/ci/teamcity/pure/ITeamcityConn.java | 4 ++++
4 files changed, 22 insertions(+), 1 deletion(-)