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
Unknown Object (File)
Wed, Sep 11, 4:08 AM
Unknown Object (File)
Sat, Aug 31, 5:38 AM
Unknown Object (File)
Wed, Aug 21, 8:05 AM
Unknown Object (File)
Sat, Aug 17, 9:22 AM
Unknown Object (File)
Fri, Aug 16, 10:18 PM
Unknown Object (File)
Thu, Aug 15, 10:48 PM
Unknown Object (File)
Aug 11 2024, 3:03 AM
Unknown Object (File)
Aug 8 2024, 10:37 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.