Page MenuHomePhabricator

Prevent artifact key collision when builds are restarted
ClosedPublic

Authored by hach-que on Aug 22 2014, 4:21 AM.
Tags
None
Referenced Files
F13186067: D10336.diff
Sat, May 11, 3:24 AM
Unknown Object (File)
Tue, May 7, 6:10 AM
Unknown Object (File)
Fri, May 3, 3:48 AM
Unknown Object (File)
Mon, Apr 29, 3:52 PM
Unknown Object (File)
Sat, Apr 27, 11:23 PM
Unknown Object (File)
Wed, Apr 24, 11:48 PM
Unknown Object (File)
Fri, Apr 19, 2:08 AM
Unknown Object (File)
Fri, Apr 19, 2:08 AM
Subscribers

Details

Summary

Ref T1049. Because we no longer destroy artifacts when builds are restarted, we need the build generation number to be part of the artifact key, otherwise we get collisions when restarting builds that contain build steps that emit artifacts.

Test Plan

Ran it with a build plan of "Lease Host" and "Run Command", no longer got an artifact key crash.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

hach-que retitled this revision from to Prevent artifact key collision when builds are restarted.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Fix up artifact retrieval as well

epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 27 2014, 7:14 PM
hach-que updated this revision to Diff 24966.

Closed by commit rPa26c6147f537 (authored by @hach-que).