Page MenuHomePhabricator

Merge "phage" from "experimental"
ClosedPublic

Authored by epriestley on Feb 12 2020, 11:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 1:24 AM
Unknown Object (File)
Sun, Apr 7, 4:32 AM
Unknown Object (File)
Sun, Apr 7, 4:19 AM
Unknown Object (File)
Sun, Apr 7, 1:29 AM
Unknown Object (File)
Thu, Apr 4, 8:03 PM
Unknown Object (File)
Mon, Apr 1, 6:38 AM
Unknown Object (File)
Jan 30 2024, 9:31 PM
Unknown Object (File)
Jan 30 2024, 8:38 PM
Subscribers
None

Details

Summary

Ref T13395. Currently, "phage" is required for various cluster operations. Bring the working code out of "experimental". This isn't the final form; "wilds" has a fancier version.

Test Plan

Ran phage workflows against the cluster.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
SeverityLocationCodeMessage
Advicescripts/phage.php:13XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 23823
Build 32768: Run Core Tests
Build 32767: arc lint + arc unit

Unit TestsFailed

TimeTest
12 msPhutilDeferredLogTestCase::testManyWriters
Assertion failed, expected values to be equal (at PhutilDeferredLogTestCase.php:116). Expected vs Actual Output Diff --- Old Value
22 msPhutilFileLockTestCase::testInProcessHolding
EXCEPTION (Exception): Unable to hold lock in external process! #0 /core/data/drydock/workingcopy-88/repo/arcanist/src/filesystem/__tests__/PhutilFileLockTestCase.php(51): PhutilFileLockTestCase->holdLock(Object(TempFile)) #1 [internal function]: PhutilFileLockTestCase->testInProcessHolding()
30 msPhutilFileLockTestCase::testInProcessLocking
Assertion failed, expected 'true' (at PhutilFileLockTestCase.php:41). ACTUAL VALUE
22 msPhutilFileLockTestCase::testLockHolding
EXCEPTION (Exception): Unable to hold lock in external process! #0 /core/data/drydock/workingcopy-88/repo/arcanist/src/filesystem/__tests__/PhutilFileLockTestCase.php(20): PhutilFileLockTestCase->holdLock(Object(TempFile)) #1 [internal function]: PhutilFileLockTestCase->testLockHolding()
15 msPhutilFileLockTestCase::testLockTesting
Assertion failed, expected 'true' (at PhutilFileLockTestCase.php:10). ACTUAL VALUE
View Full Test Results (6 Failed · 470 Passed · 36 Skipped)

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 12 2020, 11:45 PM
Harbormaster failed remote builds in B23823: Diff 49997!
This revision was not accepted when it landed; it landed in state Changes Planned.Feb 12 2020, 11:53 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.