Page MenuHomePhabricator
Files F15413626

D18837.id45193.diff
No OneTemporary
Actions

D18837.id45193.diff
View Options

diff --git a/src/__phutil_library_map__.php b/src/__phutil_library_map__.php
--- a/src/__phutil_library_map__.php
+++ b/src/__phutil_library_map__.php
@@ -42,6 +42,7 @@
'ArcanistBraceFormattingXHPASTLinterRuleTestCase' => 'lint/linter/xhpast/rules/__tests__/ArcanistBraceFormattingXHPASTLinterRuleTestCase.php',
'ArcanistBranchWorkflow' => 'workflow/ArcanistBranchWorkflow.php',
'ArcanistBrowseWorkflow' => 'workflow/ArcanistBrowseWorkflow.php',
+ 'ArcanistBuildRef' => 'ref/ArcanistBuildRef.php',
'ArcanistBundle' => 'parser/ArcanistBundle.php',
'ArcanistBundleTestCase' => 'parser/__tests__/ArcanistBundleTestCase.php',
'ArcanistCSSLintLinter' => 'lint/linter/ArcanistCSSLintLinter.php',
@@ -459,6 +460,7 @@
'ArcanistBraceFormattingXHPASTLinterRuleTestCase' => 'ArcanistXHPASTLinterRuleTestCase',
'ArcanistBranchWorkflow' => 'ArcanistFeatureWorkflow',
'ArcanistBrowseWorkflow' => 'ArcanistWorkflow',
+ 'ArcanistBuildRef' => 'Phobject',
'ArcanistBundle' => 'Phobject',
'ArcanistBundleTestCase' => 'PhutilTestCase',
'ArcanistCSSLintLinter' => 'ArcanistExternalLinter',
diff --git a/src/ref/ArcanistBuildRef.php b/src/ref/ArcanistBuildRef.php
new file mode 100644
--- /dev/null
+++ b/src/ref/ArcanistBuildRef.php
@@ -0,0 +1,107 @@
+<?php
+
+final class ArcanistBuildRef
+ extends Phobject {
+
+ private $parameters;
+ private $legacy;
+
+ public static function newFromConduit(array $data, $legacy) {
+ $ref = new self();
+ $ref->parameters = $data;
+
+ // This flag indicates that the data is from "harbormaster.querybuilds",
+ // an older API call, rather than "harbormaster.build.search", the modern
+ // API call.
+ $ref->legacy = $legacy;
+ return $ref;
+ }
+
+ private function getStatusMap() {
+ if (isset($this->parameters['buildStatus'])) {
+ $status = $this->parameters['buildStatus'];
+
+ // We may either have an array or a scalar here. The array comes from
+ // "harbormaster.build.search", or from "harbormaster.querybuilds" if
+ // the server is newer than August 2016. The scalar comes from older
+ // versions of that method. See PHI261.
+ if (is_array($status)) {
+ $map = $status;
+ } else {
+ $map = array(
+ 'value' => $status,
+ );
+ }
+ } else {
+ $map = array(
+ 'value' => 'unknown',
+ );
+ }
+
+ // If we don't have a name, try to fill one in.
+ if (!isset($map['name'])) {
+ $name_map = array(
+ 'inactive' => pht('Inactive'),
+ 'pending' => pht('Pending'),
+ 'building' => pht('Building'),
+ 'passed' => pht('Passed'),
+ 'failed' => pht('Failed'),
+ 'aborted' => pht('Aborted'),
+ 'error' => pht('Error'),
+ 'paused' => pht('Paused'),
+ 'deadlocked' => pht('Deadlocked'),
+ 'unknown' => pht('Unknown'),
+ );
+
+ $map['name'] = idx($name_map, $map['value'], $map['value']);
+ }
+
+ // If we don't have an ANSI color code, try to fill one in.
+ if (!isset($map['color.ansi'])) {
+ $color_map = array(
+ 'failed' => 'red',
+ 'passed' => 'green',
+ );
+
+ $map['color.ansi'] = idx($color_map, $map['value'], 'yellow');
+ }
+
+ return $map;
+ }
+
+ public function getStatus() {
+ $map = $this->getStatusMap();
+ return $map['value'];
+ }
+
+ public function getStatusName() {
+ $map = $this->getStatusMap();
+ return $map['name'];
+ }
+
+ public function getStatusANSIColor() {
+ $map = $this->getStatusMap();
+ return $map['color.ansi'];
+ }
+
+ public function getObjectName() {
+ return pht('Build %d', $this->getID());
+ }
+
+ public function getStatusSortVector() {
+ $status = $this->getStatus();
+
+ // For now, just sort passed builds first.
+ if ($this->getStatus() == 'passed') {
+ $status_class = 1;
+ } else {
+ $status_class = 2;
+ }
+
+ return id(new PhutilSortVector())
+ ->addInt($status_class)
+ ->addString($status);
+ }
+
+
+}
diff --git a/src/workflow/ArcanistLandWorkflow.php b/src/workflow/ArcanistLandWorkflow.php
--- a/src/workflow/ArcanistLandWorkflow.php
+++ b/src/workflow/ArcanistLandWorkflow.php
@@ -1404,13 +1404,12 @@
switch ($buildable['buildableStatus']) {
case 'building':
$message = pht(
- 'Harbormaster is still building the active diff for this revision:');
+ 'Harbormaster is still building the active diff for this revision.');
$prompt = pht('Land revision anyway, despite ongoing build?');
break;
case 'failed':
$message = pht(
- 'Harbormaster failed to build the active diff for this revision. '.
- 'Build failures:');
+ 'Harbormaster failed to build the active diff for this revision.');
$prompt = pht('Land revision anyway, despite build failures?');
break;
default:
@@ -1418,28 +1417,25 @@
return;
}
- $builds = $this->getConduit()->callMethodSynchronous(
- 'harbormaster.querybuilds',
+ $builds = $this->queryBuilds(
array(
'buildablePHIDs' => array($buildable['phid']),
));
$console->writeOut($message."\n\n");
- foreach ($builds['data'] as $build) {
- switch ($build['buildStatus']) {
- case 'failed':
- $color = 'red';
- break;
- default:
- $color = 'yellow';
- break;
- }
- $console->writeOut(
- " **<bg:".$color."> %s </bg>** %s: %s\n",
- phutil_utf8_strtoupper($build['buildStatusName']),
- pht('Build %d', $build['id']),
- $build['name']);
+ $builds = msort($builds, 'getStatusSortVector');
+ foreach ($builds as $build) {
+ $ansi_color = $build->getStatusANSIColor();
+ $status_name = $build->getStatusName();
+ $object_name = $build->getObjectName();
+ $build_name = $build->getName();
+
+ echo tsprintf(
+ " **<bg:".$ansi_color."> %s </bg>** %s: %s\n",
+ $status_name,
+ $object_name,
+ $build_name);
}
$console->writeOut(
@@ -1478,4 +1474,34 @@
$mark_workflow->run();
}
+ private function queryBuilds(array $constraints) {
+ $conduit = $this->getConduit();
+
+ try {
+ $raw_result = $conduit->callMethodSynchronous(
+ 'harbormaster.build.search',
+ array(
+ 'constraints' => $constraints,
+ ));
+
+ $is_legacy = false;
+ } catch (Exception $ex) {
+ // If the server doesn't have "harbormaster.build.search" yet (Aug 2016),
+ // try the older "harbormaster.querybuilds" instead.
+ $raw_result = $conduit->callMethodSynchronous(
+ 'harbormaster.querybuilds',
+ $constraints);
+
+ $is_legacy = true;
+ }
+
+ $refs = array();
+ foreach ($raw_result['data'] as $raw_data) {
+ $refs[] = ArcanistBuildRef::newFromConduit($raw_data, $is_legacy);
+ }
+
+ return $refs;
+ }
+
+
}

File Metadata

Mime Type
text/plain
Expires
Thu, Mar 20, 8:06 PM (2 d, 2 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7714408
Default Alt Text
D18837.id45193.diff (6 KB)

Event Timeline

Phabricator · Built by Phacility