Bug 223101 - [resultsdbpy] Save identifiers in commit table
Summary: [resultsdbpy] Save identifiers in commit table
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-11 16:22 PST by Jonathan Bedard
Modified: 2021-03-16 17:53 PDT (History)
2 users (show)

See Also:


Attachments
Patch (28.66 KB, patch)
2021-03-11 16:25 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (28.11 KB, patch)
2021-03-12 15:10 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (38.43 KB, patch)
2021-03-15 16:47 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (35.96 KB, patch)
2021-03-16 09:58 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (46.12 KB, patch)
2021-03-16 16:54 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2021-03-11 16:22:54 PST
Commits should be saved with identifiers, hashes and revisions.
Comment 1 Radar WebKit Bug Importer 2021-03-11 16:23:07 PST
<rdar://problem/75338338>
Comment 2 Jonathan Bedard 2021-03-11 16:25:17 PST
Created attachment 422989 [details]
Patch
Comment 3 Jonathan Bedard 2021-03-12 15:10:28 PST
Created attachment 423085 [details]
Patch
Comment 4 dewei_zhu 2021-03-12 15:48:36 PST
Comment on attachment 423085 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=423085&action=review

> Tools/Scripts/libraries/resultsdbpy/resultsdbpy/model/commit_context.py:390
> +                ('identifier', lambda commit: str(commit)),

This can be just `str`
Comment 5 Jonathan Bedard 2021-03-15 16:47:27 PDT
Created attachment 423262 [details]
Patch
Comment 6 dewei_zhu 2021-03-15 17:11:33 PDT
Comment on attachment 423262 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=423262&action=review

> Tools/Scripts/libraries/webkitscmpy/webkitscmpy/commit.py:156
> +        print('{} vs {}'.format(hash_from_id, revision_from_id))

Do we expect this print here?
Comment 7 Jonathan Bedard 2021-03-15 17:14:57 PDT
Comment on attachment 423262 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=423262&action=review

>> Tools/Scripts/libraries/webkitscmpy/webkitscmpy/commit.py:156
>> +        print('{} vs {}'.format(hash_from_id, revision_from_id))
> 
> Do we expect this print here?

No, I also broke this change out into an independent one: https://bugs.webkit.org/show_bug.cgi?id=223224 (although tests will fail without it, so the independent one needs to land first)
Comment 8 Jonathan Bedard 2021-03-16 09:58:18 PDT
Created attachment 423346 [details]
Patch
Comment 9 dewei_zhu 2021-03-16 10:43:20 PDT
r=me
Comment 10 Jonathan Bedard 2021-03-16 16:54:20 PDT
Created attachment 423414 [details]
Patch
Comment 11 EWS 2021-03-16 17:53:45 PDT
Committed r274542: <https://commits.webkit.org/r274542>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 423414 [details].