Page MenuHomePhabricator

Don't try to publish build results to bare diffs
ClosedPublic

Authored by epriestley on Feb 12 2019, 11:29 PM.
Tags
None
Referenced Files
F15523054: D20156.diff
Sun, Apr 20, 11:12 PM
F15496198: D20156.id48142.diff
Sun, Apr 13, 6:18 AM
F15494308: D20156.id.diff
Sat, Apr 12, 11:12 PM
F15490391: D20156.diff
Fri, Apr 11, 4:53 PM
F15473477: D20156.diff
Sat, Apr 5, 10:16 PM
F15469583: D20156.id48133.diff
Fri, Apr 4, 2:54 PM
F15424418: D20156.id48142.diff
Mar 22 2025, 10:16 PM
F15340665: D20156.diff
Mar 9 2025, 4:04 PM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/conduit-call-is-generating-phd-log-error-message/2380/. If you run builds against a diff which is not attached to a revision (this is unusual) we still try to publish to the associated revision. This won't work since there is no associated revision.

Since bare diffs don't really have a timeline, just publish nowhere for now.

Test Plan
  • Created a diff.
  • Did not attach it to a revision.
  • Created a build plan with "make http request + wait for response".
  • Manually ran the build plan against the bare diff.
  • Used bin/phd debug task to run the build and hit a "revision not attached" exception during publishing.
  • Applied patch.
  • Ran bin/phd debug task, got clean (no-op) publish.
  • Sent build a failure message with "harbormaster.sendmessage", got a failed build.

This isn't a real workflow, but shouldn't fail.

Diff Detail

Repository
rP Phabricator
Branch
harbor1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21977
Build 30014: Run Core Tests
Build 30013: arc lint + arc unit