|
|
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 |