[annotee] add SOURCE_MANIFEST annotation.

[annotee] add SOURCE_MANIFEST annotation.

This also changes the sha256 key to be bytes instead of string.

R=dnj@chromium.org, hinoka@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2994593004
Committed: https://github.com/luci/luci-go/commit/df8718467512453e74523cffd6679e59f05abfb3

[dnj, 2017-08-04 23:50:07]

lgtm w/ comment, take it or leave it.

https://codereview.chromium.org/2994593004/diff/1/logdog/client/annotee/annot...
File logdog/client/annotee/annotation/annotation_test.go (right):

https://codereview.chromium.org/2994593004/diff/1/logdog/client/annotee/annot...
logdog/client/annotee/annotation/annotation_test.go:282: {"source_manifest",
nil},
Idunno, might as well just slap this onto "coverage" IMO.

[Ryan Tseng, 2017-08-05 00:20:13]

hinoka@google.com changed reviewers:
+ hinoka@google.com

[Ryan Tseng, 2017-08-05 00:20:14]

lgtm

[iannucci, 2017-08-05 00:26:25]

The CQ bit was checked by iannucci@chromium.org

[iannucci, 2017-08-05 00:26:26]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@chromium.org,
hinoka@google.com
Link to the patchset: https://codereview.chromium.org/2994593004/#ps20001
(title: "Move to coverage test")

[commit-bot: I haz the power, 2017-08-05 00:26:35]

CQ is trying da patch.

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

[iannucci, 2017-08-05 00:27:38]

https://codereview.chromium.org/2994593004/diff/1/logdog/client/annotee/annot...
File logdog/client/annotee/annotation/annotation_test.go (right):

https://codereview.chromium.org/2994593004/diff/1/logdog/client/annotee/annot...
logdog/client/annotee/annotation/annotation_test.go:282: {"source_manifest",
nil},
On 2017/08/04 23:50:07, dnj wrote:
> Idunno, might as well just slap this onto "coverage" IMO.

doonnnneeeee

[iannucci, 2017-08-05 00:28:24]

The CQ bit was unchecked by iannucci@chromium.org

[iannucci, 2017-08-05 00:31:46]

The CQ bit was checked by iannucci@chromium.org

[iannucci, 2017-08-05 00:31:46]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@chromium.org,
hinoka@google.com
Link to the patchset: https://codereview.chromium.org/2994593004/#ps40001
(title: "rename to deployment_manifests")

[commit-bot: I haz the power, 2017-08-05 00:31:55]

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-08-05 00:40:47]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1501893106513270,
"parent_rev": "ddc861e154feb36e78c0b7b0784cacfab6414f70", "commit_rev":
"df8718467512453e74523cffd6679e59f05abfb3"}

[commit-bot: I haz the power, 2017-08-05 00:40:56]

Description was changed from

==========
[annotee] add SOURCE_MANIFEST annotation.

This also changes the sha256 key to be bytes instead of string.

R=dnj@chromium.org, hinoka@chromium.org
BUG=
==========

to

==========
[annotee] add SOURCE_MANIFEST annotation.

This also changes the sha256 key to be bytes instead of string.

R=dnj@chromium.org, hinoka@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2994593004
Committed:
https://github.com/luci/luci-go/commit/df8718467512453e74523cffd6679e59f05abfb3
==========

[commit-bot: I haz the power, 2017-08-05 00:40:57]

Committed patchset #3 (id:40001) as
https://github.com/luci/luci-go/commit/df8718467512453e74523cffd6679e59f05abfb3