New issue
Advanced search Search tips

Issue 741791 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 731905



Sign in to add a comment

autotest_lib.site_utils.autoupdate.full_release_test_test autotest-server unit test is failing.

Project Member Reported by haddowk@chromium.org, Jul 12 2017

Issue description

autotest-server-0.0.1-r3464: Running autotest_lib.site_utils.autoupdate.full_release_test_test
autotest-server-0.0.1-r3464: DEBUG:root:Running '/usr/bin/git --git-dir=/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver/.git --work-tree=/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver count-objects'
autotest-server-0.0.1-r3464: INFO:root:Cloning git repo https://chromium.googlesource.com/chromiumos/platform/dev-util
autotest-server-0.0.1-r3464: DEBUG:root:Running '/usr/bin/git --git-dir=/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver/.git clone https://chromium.googlesource.com/chromiumos/platform/dev-util /build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver '
autotest-server-0.0.1-r3464: ERROR:root:Cloning into '/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver'...
autotest-server-0.0.1-r3464: fatal: unable to access 'https://chromium.googlesource.com/chromiumos/platform/dev-util/': Couldn't resolve host 'chromium.googlesource.com'
autotest-server-0.0.1-r3464: 
autotest-server-0.0.1-r3464: Traceback (most recent call last):
autotest-server-0.0.1-r3464:   File "./utils/unittest_suite.py", line 287, in <module>
autotest-server-0.0.1-r3464:     main()
autotest-server-0.0.1-r3464:   File "./utils/unittest_suite.py", line 274, in main
autotest-server-0.0.1-r3464:     errors = find_and_run_tests(absolute_start, options)
autotest-server-0.0.1-r3464:   File "./utils/unittest_suite.py", line 257, in find_and_run_tests
autotest-server-0.0.1-r3464:     pe.run_until_completion()
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/utils/parallel.py", line 67, in run_until_completion
autotest-server-0.0.1-r3464:     self._run(self.ready_to_run.pop())
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/utils/parallel.py", line 53, in _run
autotest-server-0.0.1-r3464:     function()
autotest-server-0.0.1-r3464:   File "./utils/unittest_suite.py", line 247, in <lambda>
autotest-server-0.0.1-r3464:     run_module = lambda module=module_names: run_test(module, options)
autotest-server-0.0.1-r3464:   File "./utils/unittest_suite.py", line 144, in run_test
autotest-server-0.0.1-r3464:     '.'.join(mod_names[:-1]))
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/client/setup_modules.py", line 70, in import_module
autotest-server-0.0.1-r3464:     from_module = __import__(from_where, globals(), locals(), [module])
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site_utils/autoupdate/full_release_test_test.py", line 16, in <module>
autotest-server-0.0.1-r3464:     from autotest_lib.site_utils.autoupdate import full_release_test
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site_utils/autoupdate/full_release_test.py", line 27, in <module>
autotest-server-0.0.1-r3464:     from autotest_lib.site_utils.autoupdate import test_image
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site_utils/autoupdate/test_image.py", line 16, in <module>
autotest-server-0.0.1-r3464:     external_packages.DevServerRepo())
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site_utils/autoupdate/import_common.py", line 28, in download_and_import
autotest-server-0.0.1-r3464:     [package_class], install_dir)
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/utils/build_externals.py", line 170, in build_and_install_packages
autotest-server-0.0.1-r3464:     result = package.build_and_install(install_dir)
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/utils/external_packages.py", line 1232, in build_and_install
autotest-server-0.0.1-r3464:     git_repo.reinit_repo_at(self.PROD_BRANCH)
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/client/common_lib/revision_control.py", line 274, in reinit_repo_at
autotest-server-0.0.1-r3464:     self.clone()
autotest-server-0.0.1-r3464:   File "/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/client/common_lib/revision_control.py", line 180, in clone
autotest-server-0.0.1-r3464:     raise GitCloneError('Failed to clone git url', rv)
autotest-server-0.0.1-r3464: autotest_lib.client.common_lib.revision_control.GitCloneError: ('Failed to clone git url', * Command: 
autotest-server-0.0.1-r3464:     /usr/bin/git --git-dir=/build/guado_moblab/tmp/portage/chromeos-base
autotest-server-0.0.1-r3464:     /autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-
autotest-server-0.0.1-r3464:     packages/devserver/.git clone
autotest-server-0.0.1-r3464:     https://chromium.googlesource.com/chromiumos/platform/dev-util
autotest-server-0.0.1-r3464:     /build/guado_moblab/tmp/portage/chromeos-base/autotest-
autotest-server-0.0.1-r3464:     server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver
autotest-server-0.0.1-r3464: Exit status: 128
autotest-server-0.0.1-r3464: Duration: 0.221374034882
autotest-server-0.0.1-r3464: 
autotest-server-0.0.1-r3464: stderr:
autotest-server-0.0.1-r3464: Cloning into '/build/guado_moblab/tmp/portage/chromeos-base/autotest-server-0.0.1-r3464/work/autotest-server-0.0.1/site-packages/devserver'...
autotest-server-0.0.1-r3464: fatal: unable to access 'https://chromium.googlesource.com/chromiumos/platform/dev-util/': Couldn't resolve host 'chromium.googlesource.com')
 

Comment 1 by derat@chromium.org, Jul 12 2017

Cc: vapier@chromium.org
This seems like it could be related to the network access restriction recently imposed on ebuilds and discussed at https://groups.google.com/a/chromium.org/d/topic/chromium-os-dev/puD1c42lLSQ/discussion. That thread mentions problems with autotest.

Comment 2 by vapier@chromium.org, Jul 12 2017

Blocking: 731905
yes, this is almost certainly related.  i'll post a CL to re-enable, but autotest-server is def misbehaving here and should not be cloning git repos directly.
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/43084fa48affbc5ebb1016f2e0284e139b8ae70e

commit 43084fa48affbc5ebb1016f2e0284e139b8ae70e
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Jul 13 13:51:36 2017

autotest-server: allow network access for unittests

We'll want to revert this once someone figures out how to fix the code.

BUG= chromium:741791 
TEST=precq passes

Change-Id: Iec04191b9cfe24a14b8fa702a66ab0505ac33cd9
Reviewed-on: https://chromium-review.googlesource.com/568803
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/43084fa48affbc5ebb1016f2e0284e139b8ae70e/chromeos-base/autotest-server/autotest-server-9999.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, Oct 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/57ce8e3e625576f264f46b78c00989ca52624bd6

commit 57ce8e3e625576f264f46b78c00989ca52624bd6
Author: Mike Frysinger <vapier@chromium.org>
Date: Fri Oct 13 21:44:51 2017

autotest: enable network restrictions

The statsd module no longer exists which means we don't need network.

BUG= chromium:741791 
TEST=precq passes

Change-Id: I4f963f9531d451578b848e2098e0f12414320d71
Reviewed-on: https://chromium-review.googlesource.com/602630
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[modify] https://crrev.com/57ce8e3e625576f264f46b78c00989ca52624bd6/chromeos-base/autotest/autotest-9999.ebuild

Project Member

Comment 5 by bugdroid1@chromium.org, Oct 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/ab6f61e54e57a30e9f47898fda0e2d91a87969d7

commit ab6f61e54e57a30e9f47898fda0e2d91a87969d7
Author: Mike Frysinger <vapier@chromium.org>
Date: Sat Oct 14 07:52:24 2017

Revert "autotest-server: allow network access for unittests"

This reverts commit 43084fa48affbc5ebb1016f2e0284e139b8ae70e.

The source no longer needs network, so lock it down again.

BUG= chromium:741791 
TEST=precq passes

Change-Id: If97dd3da086ca9bca34d4b12554e9f0ffc154bbc
Reviewed-on: https://chromium-review.googlesource.com/602647
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/ab6f61e54e57a30e9f47898fda0e2d91a87969d7/chromeos-base/autotest-server/autotest-server-9999.ebuild

Components: Infra>Client>ChromeOS
Status: Fixed (was: Untriaged)

Sign in to add a comment