|
|
Chromium Code Reviews|
Created:
6 years, 8 months ago by iannucci Modified:
6 years, 8 months ago CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master Visibility:
Public. |
DescriptionAbort `git cl push` if pushing into a local branch.
R=agable@chromium.org, szager@chromium.org
BUG=338102
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=263202
Patch Set 1 #Patch Set 2 : better message #Patch Set 3 : fix one test and speculatively fix another #Patch Set 4 : restrict to non git-svn repos #Messages
Total messages: 13 (0 generated)
(tiny review) PTAL for wording.
lgtm
The CQ bit was checked by iannucci@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/iannucci@chromium.org/224863006/20001
The CQ bit was unchecked by commit-bot@chromium.org
Presubmit check for 224863006-20001 failed and returned exit status 1.
Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh
** Presubmit ERRORS **
tests/git_cl_test.py (0.49s) failed
.ERROR:root:@10 Expected: (['git', 'rev-list', '--merges', '--grep=^SVN changes
up to revision [0-9]*$', 'refs/remotes/origin/master^!'],) Actual: (['git',
'config', 'branch.working.merge'],)
FERROR:root:@10 Expected: (['git', 'rev-list', '--merges', '--grep=^SVN changes
up to revision [0-9]*$', 'refs/remotes/origin/master^!'],) Actual: (['git',
'config', 'branch.working.merge'],)
F...ERROR: Gerrit commit-msg hook not available.
...........
======================================================================
FAIL: test_dcommit (__main__.TestGitCl)
----------------------------------------------------------------------
Traceback (most recent call last):
File "tests/git_cl_test.py", line 517, in test_dcommit
git_cl.main(['dcommit'])
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 2552, in main
return dispatcher.execute(OptionParser(), argv)
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/subcommand.py",
line 245, in execute
return command(parser, args[1:])
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 2000, in CMDdcommit
return SendUpstream(parser, args, 'dcommit')
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 1783, in SendUpstream
args = [cl.GetUpstreamBranch()]
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 560, in GetUpstreamBranch
remote, upstream_branch = self.FetchUpstreamTuple(self.GetBranch())
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 521, in FetchUpstreamTuple
error_ok=True).strip()
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 91, in RunGit
return RunCommand(['git'] + args, **kwargs)
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 79, in RunCommand
return subprocess2.check_output(args, shell=False, **kwargs)
File "tests/git_cl_test.py", line 111, in _mocked_call
self.fail(msg)
AssertionError: @10 Expected: (['git', 'rev-list', '--merges', '--grep=^SVN
changes up to revision [0-9]*$', 'refs/remotes/origin/master^!'],) Actual:
(['git', 'config', 'branch.working.merge'],)
======================================================================
FAIL: test_dcommit_bypass_hooks (__main__.TestGitCl)
----------------------------------------------------------------------
Traceback (most recent call last):
File "tests/git_cl_test.py", line 524, in test_dcommit_bypass_hooks
git_cl.main(['dcommit', '--bypass-hooks'])
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 2552, in main
return dispatcher.execute(OptionParser(), argv)
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/subcommand.py",
line 245, in execute
return command(parser, args[1:])
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 2000, in CMDdcommit
return SendUpstream(parser, args, 'dcommit')
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 1783, in SendUpstream
args = [cl.GetUpstreamBranch()]
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 560, in GetUpstreamBranch
remote, upstream_branch = self.FetchUpstreamTuple(self.GetBranch())
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 521, in FetchUpstreamTuple
error_ok=True).strip()
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 91, in RunGit
return RunCommand(['git'] + args, **kwargs)
File "/b/infra_internal/commit_queue/workdir/tools/depot_tools/git_cl.py",
line 79, in RunCommand
return subprocess2.check_output(args, shell=False, **kwargs)
File "tests/git_cl_test.py", line 111, in _mocked_call
self.fail(msg)
AssertionError: @10 Expected: (['git', 'rev-list', '--merges', '--grep=^SVN
changes up to revision [0-9]*$', 'refs/remotes/origin/master^!'],) Actual:
(['git', 'config', 'branch.working.merge'],)
----------------------------------------------------------------------
Ran 17 tests in 0.053s
FAILED (failures=2)
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Note: Saving similarity of 70% in git config.
Using 70% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
Using 50% similarity for rename/copy detection. Override with --similarity.
submit-from-new-dir.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/submit-from-new-dir.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: upload succeeds
TESTING: git-cl dcommits ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'new' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/remotes/origin/trunk'
FAILURE: git-cl dcommits ok
submodule-merge-test.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/submodule-merge-test.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test remote git-svn-submodule repo...
Switched to branch 'master'
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/heads/master'
FAILURE: dcommitted code
post-dcommit-hook-test.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/post-dcommit-hook-test.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/heads/master'
FAILURE: dcommitted code
basic.sh failed
Command /b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/basic.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: git-cl upload wants a server
TESTING: git-cl status has no issue
TESTING: upload succeeds (needs a server running on localhost)
TESTING: git-cl status now knows the issue
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:--
0
100 77 0 0 100 77 0 1832 --:--:-- --:--:-- --:--:-- 1878
TESTING: git-cl dcommits ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/remotes/origin/trunk'
FAILURE: git-cl dcommits ok
Presubmit checks took 93.4s to calculate.
The CQ bit was checked by iannucci@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/iannucci@chromium.org/224863006/40001
The CQ bit was unchecked by commit-bot@chromium.org
Presubmit check for 224863006-40001 failed and returned exit status 1.
Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh
** Presubmit ERRORS **
submit-from-new-dir.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/submit-from-new-dir.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
fatal: Cannot update paths and switch to branch 'new' at the same time.
Did you intend to checkout 'origin/master' which can not be resolved as commit?
submodule-merge-test.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/submodule-merge-test.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test remote git-svn-submodule repo...
Switched to branch 'master'
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/heads/master'
FAILURE: dcommitted code
post-dcommit-hook-test.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/post-dcommit-hook-test.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/heads/master'
FAILURE: dcommitted code
basic.sh failed
Command /b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/basic.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: git-cl upload wants a server
TESTING: git-cl status has no issue
TESTING: upload succeeds (needs a server running on localhost)
TESTING: git-cl status now knows the issue
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:--
0
100 77 0 0 100 77 0 1857 --:--:-- --:--:-- --:--:-- 1925
TESTING: git-cl dcommits ok
Using 50% similarity for rename/copy detection. Override with --similarity.
Attempting to push branch 'work' into another local branch!
Either reparent this branch on top of origin/master:
git reparent-branch --root
OR run `git rebase-update` if you think the parent branch is already
committed.
Current parent: 'refs/remotes/origin/trunk'
FAILURE: git-cl dcommits ok
Presubmit checks took 79.7s to calculate.
The CQ bit was checked by iannucci@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/iannucci@chromium.org/224863006/60001
Message was sent while issue was closed.
Change committed as 263202 |
