Bug 209125 - old commit-queue sometimes fail to commit because of files being out of date
Summary: old commit-queue sometimes fail to commit because of files being out of date
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-03-15 10:37 PDT by Aakash Jain
Modified: 2020-03-21 04:36 PDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2020-03-15 10:37:54 PDT
old commit-queue sometimes fail to commit because of files being out of date. e.g.: https://bugs.webkit.org/show_bug.cgi?id=209088#c10

ERROR from SVN:
Item is out of date: File '/trunk/LayoutTests/ChangeLog' is out of date
W: 23fe8aff4c70caece1e47234baecb63c93028b07 and refs/remotes/origin/master differ, using rebase:
:040000 040000 13393f41d126e8f5e02c69124c1de79643a1a8df faa930175bf1f627b95b2085415cfd51e1758370 M	LayoutTests
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Comment 1 Aakash Jain 2020-03-15 10:50:13 PDT
This is probably because during the time patch was applied, and attempted to committed, there was another commit which modified one of the file being modified by the patch.

The chances of this happening seems higher with large patches.


From the logs on :
2020-03-13 23:34:21,893 - Running: webkit-patch --status-host=webkit-queues.webkit.org --bot-id=webkit-cq-03 land-attachment --force-clean --non-interactive --parent-command=commit-queue 393570 --port=mac
2020-03-14 00:10:44,032 - Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 393570, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

There was a commit https://trac.webkit.org/changeset/258462/webkit at 11:43:46 PM which also modified LayoutTests/ChangeLog.
Comment 2 Aakash Jain 2020-03-21 04:35:41 PDT
commit-queue has been re-implemented from scratch. old commit-queue has been decommissioned.
Comment 3 Radar WebKit Bug Importer 2020-03-21 04:36:15 PDT
<rdar://problem/60722829>