New issue
Advanced search Search tips

chromite manifest tests utilize ToT repo code

Project Member Reported by vapier@chromium.org, Sep 15 2016

Issue description

branch builders use ToT repo code and ToT repo from depot_tools.  this is OK.

the problem is that inside of the chroot, repo comes from a pinned version of depot_tools, and that version might not be usable with ToT repo.  an example:
- new key is added to ToT repo
- new key is used to sign tag in ToT repo
- old copy of repo in depot_tools doesn't know/trust the key
- signature checking fails

for now we'll disable the tests on branches (since it's unlikely this code will break), and figure out a better solution with ToT

these tests are impacted:
TestManifestCheckout.testGetManifestsBranch
TestManifestCheckout.testGitMatchBranchName
TestManifestCheckout.testManifestInheritance

example failure output:
======================================================================
ERROR: [chromite.lib.cros_build_lib_unittest] TestManifestCheckout.testManifestInheritance
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/host/source/chromite/lib/cros_test_lib.py", line 262, in _stacked_setUp
    target(obj)
  File "/mnt/host/source/chromite/lib/cros_build_lib_unittest.py", line 1254, in setUp
    repo.Initialize()
  File "/mnt/host/source/chromite/cbuildbot/repository.py", line 331, in Initialize
    cros_build_lib.RunCommand(init_cmd, cwd=self.directory, input='\n\ny\n')
  File "/mnt/host/source/chromite/lib/cros_build_lib.py", line 624, in RunCommand
    raise RunCommandError(msg, cmd_result)
RunCommandError: return code: 1; command: repo init --repo-url file:///mnt/host/source/.repo/repo/.git --manifest-url /tmp/chromite.testiHBlez/manifests.git --manifest-name default.xml --repo-branch default
Get file:///mnt/host/source/.repo/repo/.git
From file:///mnt/host/source/.repo/repo/
 * [new branch]                default    -> origin/default
...lots of tags...
 * [new tag]                   v1.9.6     -> v1.9.6

object 8c3607d42e4d0b3f3c5225db5de0596ff1cad283
type commit
tag v1.12.17-cr1
tagger Mike Frysinger <vapier@chromium.org> 1473898719 -0400

Chromium-specific release

gpg: keyblock resource `/home/vapier/.repoconfig/gnupg/pubring.gpg': file open error
gpg: Signature made Wed 14 Sep 2016 08:18:39 PM EDT using RSA key ID 16B500A8
gpg: Can't check signature: public key not found
error: could not verify the tag 'v1.12.17-cr1'
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R53-8530.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/89965827fef2925b05a8fc037c57432ebedacd95

commit 89965827fef2925b05a8fc037c57432ebedacd95
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386196
Reviewed-by: Ketaki Deshpande <ketakid@chromium.org>
Commit-Queue: Ketaki Deshpande <ketakid@chromium.org>
Tested-by: Ketaki Deshpande <ketakid@chromium.org>

[modify] https://crrev.com/89965827fef2925b05a8fc037c57432ebedacd95/lib/cros_build_lib_unittest.py

Project Member

Comment 2 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R50-7978.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/50b6449349db8833f5df6eeb1fb53b2d861cdfec

commit 50b6449349db8833f5df6eeb1fb53b2d861cdfec
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386199
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/50b6449349db8833f5df6eeb1fb53b2d861cdfec/lib/cros_build_lib_unittest.py

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/50b6449349db8833f5df6eeb1fb53b2d861cdfec

commit 50b6449349db8833f5df6eeb1fb53b2d861cdfec
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386199
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/50b6449349db8833f5df6eeb1fb53b2d861cdfec/lib/cros_build_lib_unittest.py

Project Member

Comment 4 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R51-8172.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/4ef6b0301e8bc20b000fd2e6aac9d5da0c08f5ed

commit 4ef6b0301e8bc20b000fd2e6aac9d5da0c08f5ed
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386198
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/4ef6b0301e8bc20b000fd2e6aac9d5da0c08f5ed/lib/cros_build_lib_unittest.py

Project Member

Comment 5 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R52-8350.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/de7ee6b4cefcf914003f11b9d28018c8ffd1a409

commit de7ee6b4cefcf914003f11b9d28018c8ffd1a409
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386197
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/de7ee6b4cefcf914003f11b9d28018c8ffd1a409/lib/cros_build_lib_unittest.py

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R54-8743.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/fdbe35390977f5c52d06a93d1da56c71ead10d7c

commit fdbe35390977f5c52d06a93d1da56c71ead10d7c
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/385997
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/fdbe35390977f5c52d06a93d1da56c71ead10d7c/lib/cros_build_lib_unittest.py

Project Member

Comment 7 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-release-R49-7834.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/239e232dd335c9968d48bac0e729f756139e35d7

commit 239e232dd335c9968d48bac0e729f756139e35d7
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386200
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/239e232dd335c9968d48bac0e729f756139e35d7/lib/cros_build_lib_unittest.py

Project Member

Comment 8 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-gale-8281.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/a1f08e2c03bc1b375a99f530f04bfc8b9ac244b3

commit a1f08e2c03bc1b375a99f530f04bfc8b9ac244b3
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386201
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/a1f08e2c03bc1b375a99f530f04bfc8b9ac244b3/lib/cros_build_lib_unittest.py

Project Member

Comment 9 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-gru-8785.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e6f9782cb94e0b786a19537544d3707bd831fc16

commit e6f9782cb94e0b786a19537544d3707bd831fc16
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386202
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/e6f9782cb94e0b786a19537544d3707bd831fc16/lib/cros_build_lib_unittest.py

Project Member

Comment 10 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-lucid-8173.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/10e14d923ac5d2c1b92d14244eef7a0e645ffb1d

commit 10e14d923ac5d2c1b92d14244eef7a0e645ffb1d
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386203
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/10e14d923ac5d2c1b92d14244eef7a0e645ffb1d/lib/cros_build_lib_unittest.py

Project Member

Comment 11 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-oak-8438.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e15aebece2e2937e1b5d326f0fe05fe27114dea9

commit e15aebece2e2937e1b5d326f0fe05fe27114dea9
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386204
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/e15aebece2e2937e1b5d326f0fe05fe27114dea9/lib/cros_build_lib_unittest.py

Project Member

Comment 12 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-celes-7287.92.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/d7f71cd6d7a7899e3cc88b876f3f1c3c5c5ce24a

commit d7f71cd6d7a7899e3cc88b876f3f1c3c5c5ce24a
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386205
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/d7f71cd6d7a7899e3cc88b876f3f1c3c5c5ce24a/lib/cros_build_lib_unittest.py

Project Member

Comment 13 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-cyan-7287.57.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/10487f0905369502d94c2d97f584f3c94b996879

commit 10487f0905369502d94c2d97f584f3c94b996879
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386206
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/10487f0905369502d94c2d97f584f3c94b996879/lib/cros_build_lib_unittest.py

Project Member

Comment 14 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-edgar-7287.167.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/125acc778f1fffd908553a38f3be29ed1cbfdd29

commit 125acc778f1fffd908553a38f3be29ed1cbfdd29
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386207
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/125acc778f1fffd908553a38f3be29ed1cbfdd29/lib/cros_build_lib_unittest.py

Project Member

Comment 15 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-glados-7820.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/55b278ac3dfe1911f0f802d3f880ae5c0ab496a6

commit 55b278ac3dfe1911f0f802d3f880ae5c0ab496a6
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386208
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/55b278ac3dfe1911f0f802d3f880ae5c0ab496a6/lib/cros_build_lib_unittest.py

Project Member

Comment 16 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-reks-7287.133.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/4a96d8f4a32fdbfb851f82b42a6fe10881a45b80

commit 4a96d8f4a32fdbfb851f82b42a6fe10881a45b80
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386209
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/4a96d8f4a32fdbfb851f82b42a6fe10881a45b80/lib/cros_build_lib_unittest.py

Project Member

Comment 17 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-smaug-7132.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/cbbdfdb02437253f1e441b2eb8fb0e275a1bebcb

commit cbbdfdb02437253f1e441b2eb8fb0e275a1bebcb
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386210
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/cbbdfdb02437253f1e441b2eb8fb0e275a1bebcb/lib/cros_build_lib_unittest.py

Project Member

Comment 18 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-smaug-7900.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/31126a8be92223a7ef06608e1ea51910c5e9680a

commit 31126a8be92223a7ef06608e1ea51910c5e9680a
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386211
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/31126a8be92223a7ef06608e1ea51910c5e9680a/lib/cros_build_lib_unittest.py

Project Member

Comment 19 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-strago-7287.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/2d64fc7750122711fa8a94335ef479c26d10535d

commit 2d64fc7750122711fa8a94335ef479c26d10535d
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386212
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/2d64fc7750122711fa8a94335ef479c26d10535d/lib/cros_build_lib_unittest.py

Project Member

Comment 20 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-terra-7287.154.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/0a58b253ab0035fbd66d41bbdc230d5e540eac89

commit 0a58b253ab0035fbd66d41bbdc230d5e540eac89
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386213
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/0a58b253ab0035fbd66d41bbdc230d5e540eac89/lib/cros_build_lib_unittest.py

Project Member

Comment 21 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-firmware-ultima-7287.131.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/4ea958ac9c97dfde9b4823fcf55098c5bb4cf93c

commit 4ea958ac9c97dfde9b4823fcf55098c5bb4cf93c
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386214
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/4ea958ac9c97dfde9b4823fcf55098c5bb4cf93c/lib/cros_build_lib_unittest.py

Project Member

Comment 22 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-arkham-7077.113.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/f2c1f1d5739acfcabf4c6df563f30b86c0678b5e

commit f2c1f1d5739acfcabf4c6df563f30b86c0678b5e
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386215
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/f2c1f1d5739acfcabf4c6df563f30b86c0678b5e/lib/cros_build_lib_unittest.py

Project Member

Comment 23 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-gale-8743.19.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/b6478d0ab062c78124fe6175026d21c608edbc21

commit b6478d0ab062c78124fe6175026d21c608edbc21
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386216
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/b6478d0ab062c78124fe6175026d21c608edbc21/lib/cros_build_lib_unittest.py

Project Member

Comment 24 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-glados-7657.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/add37d9cf092acd2fe23cefd91c734567ec38310

commit add37d9cf092acd2fe23cefd91c734567ec38310
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386217
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/add37d9cf092acd2fe23cefd91c734567ec38310/lib/cros_build_lib_unittest.py

Project Member

Comment 25 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-glados-7828.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/eb8be461664ad16a19c52d302cbd2e8f1aafb7ab

commit eb8be461664ad16a19c52d302cbd2e8f1aafb7ab
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386218
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/eb8be461664ad16a19c52d302cbd2e8f1aafb7ab/lib/cros_build_lib_unittest.py

Project Member

Comment 26 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-gru-8557.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e7e19eef3ede9e168dc867ee9f43c3d0c986a937

commit e7e19eef3ede9e168dc867ee9f43c3d0c986a937
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386219
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/e7e19eef3ede9e168dc867ee9f43c3d0c986a937/lib/cros_build_lib_unittest.py

Project Member

Comment 27 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-gru-8652.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/cc6820e69ff52a6c006a6494e30173a296ca05cd

commit cc6820e69ff52a6c006a6494e30173a296ca05cd
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386220
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/cc6820e69ff52a6c006a6494e30173a296ca05cd/lib/cros_build_lib_unittest.py

Project Member

Comment 28 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-oak-8141.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/fc4ae072e72abb9d0d88eebbf59c7dbc7ca644bc

commit fc4ae072e72abb9d0d88eebbf59c7dbc7ca644bc
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386221
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/fc4ae072e72abb9d0d88eebbf59c7dbc7ca644bc/lib/cros_build_lib_unittest.py

Project Member

Comment 29 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-oak-8182.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/86f4b187017e119475c8816a7d0dd820a82e8c58

commit 86f4b187017e119475c8816a7d0dd820a82e8c58
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386222
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/86f4b187017e119475c8816a7d0dd820a82e8c58/lib/cros_build_lib_unittest.py

Project Member

Comment 30 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-smaug-7265.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/c41f28d7ba3ea922d08ea025950ec62d9b8924ca

commit c41f28d7ba3ea922d08ea025950ec62d9b8924ca
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386223
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/c41f28d7ba3ea922d08ea025950ec62d9b8924ca/lib/cros_build_lib_unittest.py

Project Member

Comment 31 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-strago-7458.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/755eb37b761344bb40b0d3c779afd0bc0c82daab

commit 755eb37b761344bb40b0d3c779afd0bc0c82daab
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386224
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/755eb37b761344bb40b0d3c779afd0bc0c82daab/lib/cros_build_lib_unittest.py

Project Member

Comment 32 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-veyron-6591.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/f797830166480c91e2495aca4281cf53ea854e25

commit f797830166480c91e2495aca4281cf53ea854e25
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386225
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/f797830166480c91e2495aca4281cf53ea854e25/lib/cros_build_lib_unittest.py

Project Member

Comment 33 by bugdroid1@chromium.org, Sep 15 2016

Labels: merge-merged-factory-veyron-7505.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/6d53d9b476e1f79a93774ede742896ba57df87d8

commit 6d53d9b476e1f79a93774ede742896ba57df87d8
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Sep 15 21:17:56 2016

cros_build_lib: disable repo manifest tests

These rely on ToT repo code which can break things.  Disable tests in
branches since it's unlikely this code will break/change.  We'll find
a better solution in ToT.

BUG=chromium:647414
TEST=`./lib/cros_build_lib_unittest` no longer fails in branch w/ToT repo

Change-Id: I8133bff80cc64bd6c5c7bf43df6f32c058eb7987
Reviewed-on: https://chromium-review.googlesource.com/386226
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/6d53d9b476e1f79a93774ede742896ba57df87d8/lib/cros_build_lib_unittest.py

Components: -Build Infra>Client>ChromeOS>Build
Labels: -Pri-1 Pri-3
Owner: ----
Status: Available (was: Started)

Sign in to add a comment