Page MenuHomePhabricator

Prevent an error when the Drydock lease artifact does not exist
ClosedPublic

Authored by hach-que on May 30 2015, 12:38 PM.
Tags
None
Referenced Files
F14351996: D13074.id.diff
Thu, Dec 19, 12:12 PM
Unknown Object (File)
Mon, Dec 9, 1:29 PM
Unknown Object (File)
Sun, Dec 8, 2:32 PM
Unknown Object (File)
Wed, Nov 27, 1:17 AM
Unknown Object (File)
Sat, Nov 23, 11:25 PM
Unknown Object (File)
Sat, Nov 23, 1:35 AM
Unknown Object (File)
Nov 19 2024, 2:53 AM
Unknown Object (File)
Nov 14 2024, 10:52 PM

Details

Summary

If a host artifact exists in a Harbormaster build, where the Drydock lease no longer exists, then an error will be raised because of the attempt to access an undefined index. This changes the code to use idx() so that it correctly returns null instead.

Test Plan

Tested in production.

Diff Detail

Repository
rP Phabricator
Branch
harbormaster-artifact-error
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6370
Build 6392: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que retitled this revision from to Prevent an error when the Drydock lease artifact does not exist.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 31 2015, 2:29 PM
This revision was automatically updated to reflect the committed changes.