include Package proto into Result proto

include Package proto into Result proto

Kitchen runs recipes using recipe engine and needs to know revisions of
recipe dependencies that were used for a run.

Export entire contents of recipes.cfg in the new result proto.

R=iannucci@chromium.org, martiniss@chromium.org
BUG=695485
Review-Url: https://codereview.chromium.org/2802693003
Committed: https://github.com/luci/recipes-py/commit/c7d91ef9861b04e41c6a2381382efd58d111fe19

[nodir, 2017-04-05 18:35:46]

PTAL

[iannucci, 2017-04-05 19:33:53]

lgtm

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/loader.py
File recipe_engine/loader.py (right):

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/loader.py...
recipe_engine/loader.py:178: return self.universe.config_file
let's not add another undocumented property. just use universe.config_file for
now

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/result.proto
File recipe_engine/result.proto (right):

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/result.pr...
recipe_engine/result.proto:18: // Description of the recipe's package.
"Initial recipe repo's package spec (recipes.cfg)"

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/run.py
File recipe_engine/run.py (right):

https://codereview.chromium.org/2802693003/diff/20001/recipe_engine/run.py#ne...
recipe_engine/run.py:323: except recipe_api.StepFailure as f:
So actually; InfraFailures will get caught by this, currently.

[nodir, 2017-05-11 22:19:23]

The CQ bit was checked by nodir@chromium.org

[nodir, 2017-05-11 22:19:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2802693003/#ps40001
(title: "rebased and addressed comments")

[commit-bot: I haz the power, 2017-05-11 22:19:25]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-11 22:27:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494541163730820,
"parent_rev": "6eaacf24833ebd2565177157d368da33780fced9", "commit_rev":
"c7d91ef9861b04e41c6a2381382efd58d111fe19"}

[commit-bot: I haz the power, 2017-05-11 22:28:00]

Description was changed from

==========
include Package proto into Result proto

Kitchen runs recipes using recipe engine and needs to know revisions of
recipe dependencies that were used for a run.

Export entire contents of recipes.cfg in the new result proto.

R=iannucci@chromium.org, martiniss@chromium.org
BUG=695485
==========

to

==========
include Package proto into Result proto

Kitchen runs recipes using recipe engine and needs to know revisions of
recipe dependencies that were used for a run.

Export entire contents of recipes.cfg in the new result proto.

R=iannucci@chromium.org, martiniss@chromium.org
BUG=695485

Review-Url: https://codereview.chromium.org/2802693003
Committed:
https://github.com/luci/recipes-py/commit/c7d91ef9861b04e41c6a2381382efd58d11...
==========

[commit-bot: I haz the power, 2017-05-11 22:28:01]

Committed patchset #3 (id:40001) as
https://github.com/luci/recipes-py/commit/c7d91ef9861b04e41c6a2381382efd58d11...

[iannucci, 2017-05-12 00:43:35]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2882563002/ by iannucci@chromium.org.

The reason for reverting is: broke the world:
https://build.chromium.org/p/chromium.infra/builders/infra-continuous-trusty-....