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
F18615908: D13074.diff
Sun, Sep 14, 3:21 PM
F18600688: D13074.diff
Sat, Sep 13, 10:19 AM
F18588591: D13074.diff
Fri, Sep 12, 12:50 AM
F18295513: D13074.id31540.diff
Sat, Aug 23, 2:55 PM
F18284060: D13074.id31561.diff
Sat, Aug 23, 2:30 AM
F18209936: D13074.id31561.diff
Mon, Aug 18, 11:41 PM
F17689830: D13074.id.diff
Jul 14 2025, 2:05 PM
F17675191: D13074.diff
Jul 13 2025, 8:24 AM

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.