You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by ir...@apache.org on 2020/05/25 17:32:52 UTC
[ignite-teamcity-bot] 02/02: Checkbox 'Delete all files in checkout
directory before each snapshot dependency build' added (#161)
This is an automated email from the ASF dual-hosted git repository.
irakov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git
commit 6d9f9a5d78293de5a9377786f3227c5bb101c4f9
Author: sergeyuttsel <ut...@gmail.com>
AuthorDate: Mon May 25 20:32:22 2020 +0300
Checkbox 'Delete all files in checkout directory before each snapshot dependency build' added (#161)
Signed-off-by: Ivan Rakov <iv...@gmail.com>
---
.../ci/tcbot/visa/TcBotTriggerAndSignOffService.java | 3 ++-
.../org/apache/ignite/ci/web/rest/TriggerBuilds.java | 5 +++--
ignite-tc-helper-web/src/main/webapp/js/testfails-2.2.js | 16 +++++++++++-----
3 files changed, 16 insertions(+), 8 deletions(-)
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
index c97fd66..8a05a79 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
@@ -265,6 +265,7 @@ public class TcBotTriggerAndSignOffService {
@Nullable String ticketId,
@Nullable String prNum,
@Nullable String baseBranchForTc,
+ @Nonnull Boolean cleanSources,
@Nullable ITcBotUserCreds prov) {
String jiraRes = "";
@@ -295,7 +296,7 @@ public class TcBotTriggerAndSignOffService {
Set<Integer> buildidsToSync = new HashSet<>();
for (int i = 0; i < suiteIds.length; i++) {
- T2<Build, Set<Integer>> objects = teamcity.triggerBuild(suiteIds[i], branchForTc, false, top != null && top, new HashMap<>(),
+ T2<Build, Set<Integer>> objects = teamcity.triggerBuild(suiteIds[i], branchForTc, cleanSources, top != null && top, new HashMap<>(),
false, "");
buildidsToSync.addAll(objects.get2());
builds[i] = objects.get1();
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/rest/TriggerBuilds.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/rest/TriggerBuilds.java
index 75a8d69..ed93be6 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/rest/TriggerBuilds.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/rest/TriggerBuilds.java
@@ -83,7 +83,8 @@ public class TriggerBuilds {
@Nullable @QueryParam("observe") Boolean observe,
@Nullable @QueryParam("ticketId") String ticketId,
@Nullable @QueryParam("prNum") String prNum,
- @Nullable @QueryParam("baseBranchForTc") String baseBranchForTc
+ @Nullable @QueryParam("baseBranchForTc") String baseBranchForTc,
+ @Nonnull @QueryParam("cleanSources") Boolean cleanSources
) {
ITcBotUserCreds prov = ITcBotUserCreds.get(req);
Injector injector = CtxListener.getInjector(ctx);
@@ -95,7 +96,7 @@ public class TriggerBuilds {
String jiraRes = injector
.getInstance(TcBotTriggerAndSignOffService.class)
- .triggerBuildsAndObserve(srvCodeOrAlias, branchForTc, parentSuiteId, suiteIdList, top, observe, ticketId, prNum, baseBranchForTc, prov);
+ .triggerBuildsAndObserve(srvCodeOrAlias, branchForTc, parentSuiteId, suiteIdList, top, observe, ticketId, prNum, baseBranchForTc, cleanSources, prov);
return new TriggerResult("Tests started." + (!jiraRes.isEmpty() ? "<br>" + jiraRes : ""));
}
diff --git a/ignite-tc-helper-web/src/main/webapp/js/testfails-2.2.js b/ignite-tc-helper-web/src/main/webapp/js/testfails-2.2.js
index 0424aec..9475f30 100644
--- a/ignite-tc-helper-web/src/main/webapp/js/testfails-2.2.js
+++ b/ignite-tc-helper-web/src/main/webapp/js/testfails-2.2.js
@@ -256,7 +256,7 @@ function showChainCurrentStatusData(chain, settings) {
"\"" + chain.branchName + "\", " +
"\"" + parentSuitId + "\", " +
"\"\", " + // ticket id
- "\"" + baseBranchForTc + "\")'>Comment JIRA</button> ";
+ "\"" + baseBranchForTc + "\")'>Comment JIRA</button><br>";
var blockersList = "";
@@ -273,6 +273,8 @@ function showChainCurrentStatusData(chain, settings) {
}
}
+ res += "<label for='cleanSources'><input id='cleanSources' type='checkbox'>Delete all files in checkout directory before each snapshot dependency build</label><br>"
+
res += "<button onclick='triggerBuilds(" +
"\"" + srvCodeForTriggering + "\", " +
"\"" + parentSuitId + "\", " +
@@ -282,7 +284,8 @@ function showChainCurrentStatusData(chain, settings) {
"false, " + //observe
"null, " + // ticketId
"\"" + + chain.prNum + "\", " +
- "\"" + baseBranchForTc + "\" " +
+ "\"" + baseBranchForTc + "\", " +
+ "document.getElementById(\"cleanSources\").checked" +
")'> " +
"Re-run possible blockers</button><br>";
@@ -295,7 +298,9 @@ function showChainCurrentStatusData(chain, settings) {
"false, " + //observe
"null, " + // ticketId
"\"" + chain.prNum + "\", " + //prNum
- "\"" + baseBranchForTc + "\")'> " +
+ "\"" + baseBranchForTc + "\", " +
+ "document.getElementById(\"cleanSources\").checked" +
+ ")'> " +
"Re-run possible blockers (top queue)</button><br>";
}
@@ -402,7 +407,7 @@ function filterPossibleBlocker(suite) {
return null;
}
-function triggerBuilds(tcServerCode, parentSuiteId, suiteIdList, branchName, top, observe, ticketId, prNum, baseBranchForTc) {
+function triggerBuilds(tcServerCode, parentSuiteId, suiteIdList, branchName, top, observe, ticketId, prNum, baseBranchForTc, cleanSources) {
var queueAtTop = isDefinedAndFilled(top) && top;
var observeJira = isDefinedAndFilled(observe) && observe;
var suiteIdsNotExists = !isDefinedAndFilled(suiteIdList) || suiteIdList.length === 0;
@@ -467,7 +472,8 @@ function triggerBuilds(tcServerCode, parentSuiteId, suiteIdList, branchName, top
"observe": observeJira,
"ticketId": ticketId,
"prNum": prNum,
- "baseBranchForTc": baseBranchForTc
+ "baseBranchForTc": baseBranchForTc,
+ "cleanSources": cleanSources
},
success: successDialog,
error: showErrInLoadStatus