New issue
Advanced search Search tips

Issue 888400 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 24
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 0
Type: Bug



Sign in to add a comment

Several ios bots failing to sync EarlGrey

Project Member Reported by jbudorick@chromium.org, Sep 24

Issue description

Includes:

ios-device
ios-device-xcode-clang
ios-simulator-full-configs
ios-simulator-xcode-clang


Log:

Error: Command 'git checkout --quiet 451b6497352d0731e9827a338f32024e564078c6' returned non-zero exit status 128 in /b/s/w/ir/cache/builder/src/ios/third_party/earl_grey/src
fatal: reference is not a tree: 451b6497352d0731e9827a338f32024e564078c6
Warnings:
Conflicting directory /b/s/w/ir/cache/builder/src/ios/third_party/earl_grey/src moved to /b/s/w/ir/cache/builder/_bad_scm/src/ios/third_party/earl_grey/src0pjJmA.
===Failed in 0.2 mins of /b/s/w/ir/cache/vpython/09242d/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py sync --verbose --reset --force --ignore_locks --output-json /b/s/w/ir/tmp/t/tmph0rqSX.json --nohooks --noprehooks --delete_unversioned_trees --revision src@unmanaged ===
Traceback (most recent call last):
  File "/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", line 1203, in <module>
    sys.exit(main())
  File "/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", line 1187, in main
    checkout(options, git_slns, specs, revisions, step_text)
  File "/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", line 1097, in checkout
    gclient_output = ensure_checkout(**checkout_parameters)
  File "/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", line 873, in ensure_checkout
    apply_patch_on_gclient)
  File "/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", line 385, in gclient_sync
    raise GclientSyncFailed(e.message, e.code, e.output)
__main__.GclientSyncFailed: /b/s/w/ir/cache/vpython/09242d/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py sync --verbose --reset --force --ignore_locks --output-json /b/s/w/ir/tmp/t/tmph0rqSX.json --nohooks --noprehooks --delete_unversioned_trees --revision src@unmanaged failed with code 1 in /b/s/w/ir/cache/builder.
step returned non-zero exit code: 1


Paging -> P0
 
Issue 888224 has been merged into this issue.
Issue 888293 has been merged into this issue.
Issue 888384 has been merged into this issue.
Issue 888224 has been merged into this issue.
Issue 888293 has been merged into this issue.
Issue 888384 has been merged into this issue.
Status: Started (was: Untriaged)
ios-simulator still has 451b6497352d0731e9827a338f32024e564078c6 in its git cache. `git log --graph --oneline --all` shows http://shortn/_4oLYDKlKs5
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 24

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/94d5eeb6cff7a9262c1c491076b7f7bf5105ced0

commit 94d5eeb6cff7a9262c1c491076b7f7bf5105ced0
Author: John Budorick <jbudorick@chromium.org>
Date: Mon Sep 24 13:58:47 2018

ios: fix earl grey DEPS hash.

TBR=justincohen@chromium.org

No-Try: true
Bug:  888400 
Change-Id: I7a6095dda1c10009b5649338439b09b0c9433d5f
Reviewed-on: https://chromium-review.googlesource.com/1240177
Commit-Queue: John Budorick <jbudorick@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593525}
[modify] https://crrev.com/94d5eeb6cff7a9262c1c491076b7f7bf5105ced0/DEPS

Issue 888403 has been merged into this issue.
Issue 888499 has been merged into this issue.
Labels: Merge-Request-70
Project Member

Comment 11 by sheriffbot@chromium.org, Oct 12

Labels: -Merge-Request-70 Merge-Review-70 Hotlist-Merge-Review
This bug requires manual review: Less than 0 days to go before AppStore submit on M70
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
The only effect of the proposed cherry-pick is to change the EG DEPS entry from a commit that somehow got deleted to an equivalent commit that still exists.
Project Member

Comment 13 by bugdroid1@chromium.org, Oct 12

Labels: merge-merged-3538
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/83ad81a1ad7113b223d10ab10ec46bececfe0652

commit 83ad81a1ad7113b223d10ab10ec46bececfe0652
Author: John Budorick <jbudorick@chromium.org>
Date: Fri Oct 12 17:05:53 2018

3538: ios: fix earl grey DEPS hash.

Bug:  888400 
Change-Id: I7a6095dda1c10009b5649338439b09b0c9433d5f
Reviewed-on: https://chromium-review.googlesource.com/1240177
Commit-Queue: John Budorick <jbudorick@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#593525}(cherry picked from commit 94d5eeb6cff7a9262c1c491076b7f7bf5105ced0)
Reviewed-on: https://chromium-review.googlesource.com/c/1278538
Reviewed-by: Michael Moss <mmoss@chromium.org>
Reviewed-by: Kariah Davis <kariahda@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#982}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
[modify] https://crrev.com/83ad81a1ad7113b223d10ab10ec46bececfe0652/DEPS

Labels: CommitLog-Audit-Violation Merge-Without-Approval M-70
Here's a summary of the rules that were executed: 
 - OnlyMergeApprovedChange: Rule Failed -- Revision 83ad81a1ad7113b223d10ab10ec46bececfe0652 was merged to refs/branch-heads/3538 branch with no merge approval from a TPM! 
Please explain why this change was merged to the branch!
 - AcknowledgeMerge: Notification Required -- 
Labels: Merge-Merged-70-3538
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/83ad81a1ad7113b223d10ab10ec46bececfe0652

Commit: 83ad81a1ad7113b223d10ab10ec46bececfe0652
Author: jbudorick@chromium.org
Commiter: jbudorick@chromium.org
Date: 2018-10-12 17:05:53 +0000 UTC

3538: ios: fix earl grey DEPS hash.

Bug:  888400 
Change-Id: I7a6095dda1c10009b5649338439b09b0c9433d5f
Reviewed-on: https://chromium-review.googlesource.com/1240177
Commit-Queue: John Budorick <jbudorick@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#593525}(cherry picked from commit 94d5eeb6cff7a9262c1c491076b7f7bf5105ced0)
Reviewed-on: https://chromium-review.googlesource.com/c/1278538
Reviewed-by: Michael Moss <mmoss@chromium.org>
Reviewed-by: Kariah Davis <kariahda@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#982}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
Approval was given on the cl: crrev.com/c/1278538
Labels: -CommitLog-Audit-Violation -Merge-Without-Approval
Approval was given on the CL.
Labels: -Hotlist-Merge-Review -Merge-Review-70

Sign in to add a comment