Page MenuHomePhabricator

Merge "phage" from "experimental"
ClosedPublic

Authored by epriestley on Feb 12 2020, 11:45 PM.
Tags
None
Referenced Files
F15436384: D20982.diff
Tue, Mar 25, 1:51 PM
F15400983: D20982.diff
Mon, Mar 17, 3:04 PM
F15397417: D20982.id49999.diff
Sun, Mar 16, 8:00 PM
F15312067: D20982.diff
Thu, Mar 6, 1:37 PM
Unknown Object (File)
Feb 20 2025, 12:03 PM
Unknown Object (File)
Feb 13 2025, 3:49 AM
Unknown Object (File)
Feb 9 2025, 12:42 PM
Unknown Object (File)
Feb 9 2025, 8:39 AM
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.