Commit Graph

30859 Commits

Author SHA1 Message Date
Chris Sosa
69cdce233e Fix tree closer. Deal with none case for PFQ.
Change-Id: Ia7883c196c688236051a4637561c9a01be5358cb

BUG=TreeCloser
TEST=Ran with simple prefix for x86-generic and no keys

Review URL: http://codereview.chromium.org/6480085
2011-02-15 12:51:06 -08:00
Chris Sosa
8dad50d97f Update test harness to take in optional public and private keys to sign payloads.
In addition some refactoring to make this cleaner and easier.

Change-Id: I1607700d065c71aff2b2833b10acbd3ebace68ce

BUG=chromium-os:8212
TEST=Ran with Simple ... running now with full

Review URL: http://codereview.chromium.org/6482017
2011-02-14 15:29:32 -08:00
David James
1ed7fe6b70 Update links to point at new location of parallel_emerge.
BUG=chromium-os:4230
TEST=grep for references to old parallel_emerge location

Change-Id: I3238e719e01df57648d3d91c0dfd7c37f7a934ee

Review URL: http://codereview.chromium.org/6246058
2011-02-14 14:49:25 -08:00
Chris Sosa
c04833cb5b Add tarball option and print out what gsutil would do to make it easier to debug.
Change-Id: I041053a017ee0aabbbeb30b4753bf9594e8d1d83

BUG=chromium-os:8364
TEST=Ran it manually here.

Review URL: http://codereview.chromium.org/6508011
2011-02-14 11:31:14 -08:00
David Rochberg
a821a8c856 Revert "enter_chroot: Only mount SSH auth socket when we need to."
This reverts commit 11753bf13442ce1da1488abb372940353f8844ff.

BUG=12058

Change-Id: I5175ea81dbf20cc3a4c290c750a7cc5c9841e3fa

TEST=

Review URL: http://codereview.chromium.org/6520012
2011-02-14 12:13:37 -05:00
David Rochberg
abaf1ccf15 enter_chroot: Only mount SSH auth socket when we need to.
Also refactors some cut-and-paste code

BUG=11944
TEST=enter_chroot.sh with and without another enter_chroot running.  Note that there are 0 or 1 mounts of the /tmp/ssh-.... directory.  Check that /proc,/sys,/dev,$SSH_AUTH_SOCK,/dev/pts,/home/${USER}/trunk,/home/${USER}/.subversion are all mounted

Change-Id: I9dada6f7f98d263345af29a5734c1c70709f6a1e

Review URL: http://codereview.chromium.org/6498001
2011-02-14 11:09:26 -05:00
Chris Sosa
89e9244910 Print out full path to archive url.
Change-Id: I6df0244d786794107c7d166c882c3585837c48a5

BUG=chromium-os:8364
TEST=Visually

Review URL: http://codereview.chromium.org/6512004
2011-02-11 17:42:55 -08:00
Doug Anderson
ae573e7a6c Update chroot for chromite if needed.
Change-Id: I6593336a52b7298bd358f3e55531b2e6a9e6dff1

BUG=chromium-os:11883
TEST=Tried deleting the symlink and deleting the line from bashrc and things got updated; ran again w/ no problems.

Review URL: http://codereview.chromium.org/6480005
2011-02-11 12:09:02 -08:00
Zdenek Behan
23bc6ca892 build_image: use string comparison for comparing strings
BUG=n0ne
TEST=./build_image and see it fail and ask me a question, instead of death

Change-Id: I27f7ac71ce74ff711e8b152ee202121d518045cd

Review URL: http://codereview.chromium.org/6417001
2011-02-10 02:04:58 +01:00
Darin Petkov
3cb5eb045c Remove no-op oobe_completed touch
The original intent of this code was to prevent mounting
of the OEM partition. However, chromeos_startup mounts
/mnt/stateful_partition/home to /home and this file gets
hidden. Given that we might mount the OEM partition always
soon, removing this code.

BUG=chromium-os:11661
TEST=build_packages, build_image, ran suite_Smoke

Change-Id: I7ef4dd5c78484664cbdeb4ffaff26e2fcf8d2507

Review URL: http://codereview.chromium.org/6458007
2011-02-09 16:56:13 -08:00
Don Garrett
6620630c34 Switch ctest to array/appends for cros_au_test_harness call.
I was getting the error about passing along an empty string argument, and this will fix that.

Change-Id: Iffd05fe570f3a6092bbeee05abf8a1884decd644

BUG=N0ne
TEST=remote testing in progress (no VM testing)

Review URL: http://codereview.chromium.org/6465014
2011-02-09 13:51:43 -08:00
Simon Glass
142ca06512 Factored out the code to copy an image and modify it for test
into mod_image_for_test.sh rather than slightly different versions
of the same in image_to_usb.sh and image_to_vm.sh

Added a function to get a test image into common.sh

Added --inplace option to mod_image_for_test, which is the default,
and preserves the original behaviour. But using --noinplace it will
now do the copy for you.

Found that chromiumos_image.bin appears throughout the scripts, so added it and the test variant to common.sh

BUG=chromiumos-10126
TEST=run mod_image_for_test.sh with and without --noinplace
run image_to_usb.sh and image_to_vm.sh with both options
test on Seaboard that correct image is provided

Really we should have automated testing for these scripts

Change-Id: I5cfa91792c7fded35e7f4ca8f8f27c6b270817fb

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=4fc5227

Review URL: http://codereview.chromium.org/5271010
2011-02-09 13:39:43 -08:00
J. Richard Barnette
90d2613692 Need to run ldconfig after adding chromeos-test packages
Change-Id: I1118a840e8ecc0ae0620003ea79ee0fd4174a96b

BUG=chromium-os:11811
TEST=Boot the system, and run 'xset'.  Also, run the BVT suite
TEST=Test mod_image_for_test.sh --factory_install to see that there are no error messages

Review URL: http://codereview.chromium.org/6451009
2011-02-08 16:19:56 -08:00
Zdenek Behan
aabded5941 make_factory_package: allow calling from outside the chroot
As documented by the developer workflow, this script need to be
called from outside the chroot. This adds a temporary reference to
the new location of chromeos-common.sh, unbreaking the factory
workflow, and giving us time to come up with a better solution.

TEST=run inside the chroot and outside the chroot
BUG=chromium-os:11769

Change-Id: Icb90211111629ed037bc40682a94200d949050e0

Review URL: http://codereview.chromium.org/6462007
2011-02-09 01:16:04 +01:00
David James
ac65e1e20a Remove chromite from crosutils.git. It's been moved to chromite.git.
TEST=chromite works in new location
BUG=chromium-os:11507

Change-Id: I8e5dee287dd0120f1cd01966953b79cf53a14790

Review URL: http://codereview.chromium.org/6371018
2011-02-08 15:42:01 -08:00
David James
17d44e3659 Verify that SCRIPTS_DIR exists in build_image.
This hack is necessary for old chroots to ensure that we source a good
common.sh when the one inside the chroot is out of date. We'll be able
to get rid of this after everybody has updated their chroot to have a working
version of common.sh and/or we have deprecated running scripts from the
source directory.

Also fix logic for when common.sh is missing to exit properly.

BUG=chromium-os:11598
TEST=Verify that build_image still works when the old common.sh is present in
     the chroot.

Change-Id: I2e8c2fd5ad06446e669ad665ee2e70f4364958fe

Review URL: http://codereview.chromium.org/6286084
2011-02-08 14:48:51 -08:00
Don Garrett
a0e7ea185a Switch from v_info logging to debug
Change-Id: I9a3177bf295462d40a6e7045a2ffb793bbfe3166

BUG=chromium-os:11672
TEST=Manual

Review URL: http://codereview.chromium.org/6246148
2011-02-07 18:39:59 -08:00
Mandeep Singh Baines
fa42b03ffe autotest: use installed version of cros_workon
BUG=11507
TEST=Ran run_remote_test.sh and verified that the following is printed:

INFO   : Detected cros_workon autotests. Building and running your autotests from /home/msb/trunk/src/third_party/autotest/files. To use emerged autotest, pass --use_emerged.

Change-Id: Ifc4305088e4edd059fad250d074347bc81a451cf

Review URL: http://codereview.chromium.org/6246141
2011-02-07 11:08:03 -08:00
Chris Sosa
99768e8e12 Robustify starting of ssh connection to vm updater.
Change-Id: Ia5e0042250da22cebadfd48d143f21597bcada5d

BUG=chromium-os:11716
TEST=Ran vm test and vm update scripts.

Review URL: http://codereview.chromium.org/6413019
2011-02-07 09:02:22 -08:00
Mandeep Singh Baines
93da99be03 cros_workon: fix comments/messages to be consistent with new location
cros_workon has move to /usr/bin/

BUG=11507
TEST=none

Change-Id: I7301f659900713c432104ccceb3e658218e0ecac

Review URL: http://codereview.chromium.org/6334137
2011-02-05 11:42:19 -08:00
Don Garrett
ad3f059401 Make enter_chroot not be chatty unless you use --verbose
Change-Id: I352ae16a248483da35575b5d74889b3d2e6683cc

BUG=chromium-os:11672
TEST=Ran manually.

Review URL: http://codereview.chromium.org/6250170
2011-02-04 14:59:56 -08:00
Zdenek Behan
ca9b334e8e Revert "cros_generate_update_payload: kill old style update image generation"
This reverts commit fad7fecc4d.

BUG=n0ne
TEST=

Review URL: http://codereview.chromium.org/6246073

Change-Id: I7652b57145931b9d6ed85d82f648eb6f5dc35ae2
2011-02-04 23:44:15 +01:00
Dale Curtis
d569a90bbd Fix recent change which prevented mount_gpt_image from running outside chroot.
Change-Id: Id58de45b72522077179024ee0310c70d2af1a497

BUG=chromium-os:4230
TEST=Ran mount_gpt_image.sh inside/outside chroot.

Review URL: http://codereview.chromium.org/6312112
2011-02-02 20:30:14 -08:00
Zdenek Behan
fad7fecc4d cros_generate_update_payload: kill old style update image generation
This disables an unwanted dependency.

Change-Id: I02752fa38cf5f64af366f1831a351b1b903563d0

BUG=n0ne
TEST=

Review URL: http://codereview.chromium.org/6349060
2011-02-03 01:44:58 +01:00
Doug Anderson
3d7fa3a0e9 Don't use sudo -v, which was having issues (hanging bots).
Change-Id: I5f76f0f27b5789d88740b36fa6ce696ef7e68e8a

BUG=chromium-os:11602
TEST=Tried steps in bug 11602 (but hacked a sudo -K to the start of teardown_env so I didn't need to wait 15 mins).

Review URL: http://codereview.chromium.org/6286069
2011-02-02 16:10:34 -08:00
Anton Staaf
fd135cd9d0 Move cros_overlay_list to src/platform/dev.
This is step 3/3, after cros_overlay_list is added to dev, and
after the ebuilds are updated to install from dev and not
crosutils, then this can go in.

Change-Id: I8ae26938fb14ccf6737acd9ca99b58aaf65a1aca

BUG=chromium-os:11507
TEST=None

Review URL: http://codereview.chromium.org/6392034
2011-02-02 15:49:23 -08:00
Doug Anderson
a8d9cc18b0 Avoid problems where PW prompt in enter_chroot.sh would block other terminals.
Change-Id: Ibd32a4be12e88d183dd0acbcb156582ad946c92f

BUG=chromium-os:11602
TEST=Added a 'sudo -K' as the first line of teardown_env to simulate sudo timeout, then performed steps in bug report (without 15 minute wait).

Review URL: http://codereview.chromium.org/6250116
2011-02-02 15:47:00 -08:00
Chris Sosa
79088f496e Remove buildbot code that has been added to chromite.
Change-Id: I7eb27ac81704de8689a65203a440303a52a01dc2

BUG=chromium-os:11171
TEST=we'll see if things explode

Review URL: http://codereview.chromium.org/6286040
2011-02-02 15:39:58 -08:00
David James
2b2f19fc2e Update archive_build.sh to use new location of prebuilt.py.
BUG=chromium-os:11171
TEST=none

Change-Id: I1ae54be5fd0e1606ca2690399c465f667e4be922

Review URL: http://codereview.chromium.org/6286051
2011-02-02 10:35:42 -08:00
Greg Spencer
c24465670d This fixes a build break in mod_image_for_test.sh
BUG=chromium-os:4230
TEST=Ran mod_image_for_test.sh locally.

Review URL: http://codereview.chromium.org/6368045

Change-Id: If6ae676fb41b366760b0c753677172090a9a4943
2011-02-01 22:44:41 -08:00
Greg Spencer
798d75f3be This starts to fix the scripts so that they load from /usr/lib/crosutils
from within the chroot.

It also fixes a number of style issues.

It changes the meaning of cros_workon "list-all" to list all available
packages, and adds "list-live" to list all live packages.

It changes things that load chromeos-common.sh from the installer to
load it from /usr/lib/installer.

BUG=chromium-os:4230
TEST=synced, rebuilt chroot, made packages, made images, built chrome
from source, and wrote an image to a USB stick.

Review URL: http://codereview.chromium.org/6240018

Change-Id: I90c34420af1a64020402bafef8e9e77f56837c02
2011-02-01 22:04:49 -08:00
Chris Sosa
24da49e6ee Revert remote_access refactor.
Reverts http://codereview.chromium.org/6322002/

Change-Id: I3cf60747b4d25bf6c8ad8f290fc46cf360b8c074

BUG=chromium-os:11568
TEST=Ran it

Review URL: http://codereview.chromium.org/6349036
2011-02-01 17:06:12 -08:00
Chris Sosa
4c5396d872 Respect debug flag setting when building unit tests.
Change-Id: Id942eefeea4bfb51b076d8d567c398557a522937

BUG=7976
TEST=Ran with both options

Review URL: http://codereview.chromium.org/6312073
2011-02-01 14:17:19 -08:00
Chris Sosa
558940dddb Make git config optional to allow multiple calls to enter_chroot at the same time.
It's very rarely needed anyway but enter_chroot does everything under the sun.

Avoids:

error: could not lock config file ~/.gitconfig

Change-Id: Ide72b14fd434b182a88d2fc636559b3515905509

BUG=chromium-os:11523
TEST=Ran it ... 10x

Review URL: http://codereview.chromium.org/6312069
2011-02-01 13:07:55 -08:00
Chris Sosa
19ef17732c Do not pregenerate update payloads in parallel.
TBR dgarret

Change-Id: I71ef4b02108f43717e4bc404290952df99112654

BUG=n0ne
TEST=NA

Review URL: http://codereview.chromium.org/6349021
2011-02-01 09:57:29 -08:00
Mandeep Singh Baines
0fdf2a7aeb cros_workon: use portageq instead of sourcing make.conf
Fixes the following issue:

/home/msb/trunk/src/overlays/overlay-x86-generic/make.conf: line 7: prebuilt.conf: No such file or directory

BUG=11513
TEST=Verified that the error goes away.

Change-Id: I5b1dfab55394ba40c02e2a1a2ee7b03a5c4a7941

Review URL: http://codereview.chromium.org/6409031
2011-01-31 16:59:37 -08:00
Mandeep Singh Baines
f9ea0036fa cros_workon: collapse cros_workon_common.sh
cros_workon is the only user of cros_workon_common.sh

This is pre-requisite to moving cros_workon into devutils.git

BUG=11507
TEST=Ran ./cros_workon --board x86-generic --all list

Change-Id: I1eab551ab33646360e507328932c151a0d36f50a

Review URL: http://codereview.chromium.org/6347052
2011-01-31 16:13:30 -08:00
Chris Sosa
39ed0d0ebe Clear cache for runs of ctest if --cache not set.
Change-Id: I013c838c83dd23cbfe04f77f87df74778d8e874e

BUG=chromium-os:10723
TEST=Ran it

Review URL: http://codereview.chromium.org/6389004
2011-01-31 12:12:41 -08:00
Chris Sosa
b8a84b2cb7 Change uprev to False for Chrome pfq.
Change-Id: I8c635cf228c9bf1c77d97656d6b38de5289ff9e6

BUG=
TEST=

Review URL: http://codereview.chromium.org/6347027
2011-01-28 22:48:46 -08:00
Chris Sosa
25272888e8 add cwd
TBR dgarret

Change-Id: I8d7f86d716f1da76f4c85259f401c3ccc9a031ff

BUG=
TEST=

Review URL: http://codereview.chromium.org/6392010
2011-01-28 20:53:08 -08:00
Chris Sosa
d7beed79b6 Rebuild chroot for full builders.
TBR dgarrett

Change-Id: Ibed5341626d498adeeb259cac54ebb05ca1b4421

BUG=
TEST=

Review URL: http://codereview.chromium.org/6407001
2011-01-28 18:43:46 -08:00
Chris Sosa
b62c4860f8 Fix issue where full builders end prematurely.
TBR dgarrett

Change-Id: I08b7c19b8010003857f9791ca45ec9cadb972d47

BUG=N0ne
TEST=Ran

Review URL: http://codereview.chromium.org/6352027
2011-01-28 18:30:02 -08:00
Chris Sosa
af47e0e488 Make push and clean more robust.
Change-Id: Iea9f8947b8ad651abe92cfdd8ea575603d1e2bb1

BUG=chromium-os:10723
TEST=Ran it ... saw nothing glaring

Review URL: http://codereview.chromium.org/6250037
2011-01-28 17:31:57 -08:00
Don Garrett
7b8075ac6d Added arm-tegra2-seaboard-full, should have added in previous change.
Change-Id: Iecb40121b4bc8ee652fe72c4a8c07e3d231293bf

BUG=chromium-os:11229
TEST=Passed unittests (shows correct syntax)

Review URL: http://codereview.chromium.org/6379015
2011-01-28 14:52:17 -08:00
Chris Sosa
436209ec5b Have the ability for the PFQ to both rev Chrome and other packages.
This issue seems bigger than it first appears because we had put extra logic to "Delete" the branch if we didn't rev anything.  This CL basically takes out all the Deletes and relies and cros_mark_as_stable clean to clean up any old dirs correctly.  To do this, I had to fix a bug in clean so that we actually delete the stable branch as part of it (if it exists).

EOM

Change-Id: Ic1020451dc1d492d4b6e2a233d1d49cf8c1c16b3

BUG=chromium-os:11329
TEST=Ran x86-generic-pfq w/ dev options + --chrome_rev=latest_release ...built chrome,
and revved packages without error.  Doing more testing still.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=a8ac0ec

Review URL: http://codereview.chromium.org/6291013
2011-01-28 14:27:38 -08:00
Don Garrett
e7b9a4f558 Add new configs to cbuildbot_conf to match master.cfg
Change-Id: I4a28ac0f8e1a90313a9373e213e8a30003d1f8ad

BUG=chromium-os:11229
TEST=Unittests

Review URL: http://codereview.chromium.org/6268020
2011-01-28 14:14:26 -08:00
Vincent Palatin
3eb3543e9a Revert "Have the ability for the PFQ to both rev Chrome and ..."
This reverts the commit a8ac0ec8c3.

TBR the x86 PFQ buildbot fails

BUG=chromium-os:11447
TEST=
TBR=sosa

Change-Id: I019de669db13b753365e1208acc9923c4bb5c993

Review URL: http://codereview.chromium.org/6339018
2011-01-28 15:55:47 -05:00
Chris Sosa
a8ac0ec8c3 Have the ability for the PFQ to both rev Chrome and other packages.
This issue seems bigger than it first appears because we had put extra logic to "Delete" the branch if we didn't rev anything.  This CL basically takes out all the Deletes and relies and cros_mark_as_stable clean to clean up any old dirs correctly.  To do this, I had to fix a bug in clean so that we actually delete the stable branch as part of it (if it exists).

EOM

Change-Id: Ic1020451dc1d492d4b6e2a233d1d49cf8c1c16b3

BUG=chromium-os:11329
TEST=Ran x86-generic-pfq w/ dev options + --chrome_rev=latest_release ...built chrome,
and revved packages without error.  Doing more testing still.

Review URL: http://codereview.chromium.org/6291013
2011-01-28 11:17:57 -08:00
Ruchi Chaturvedi
571d1c95f9 Added HWQual utility script - call_script.py
Change-Id: I07722c5eac6836051dedf015593d4717bf852853

BUG=chromium-os:10197
TEST= Run HWQual scripts as -

python call_autoserv.py --ip=<remote machine IP> --test=<server or client HWQual testname in suite_HWQual or site_tests>

Review URL: http://codereview.chromium.org/6187006
2011-01-27 16:19:55 -08:00
Don Garrett
38a66574b6 Add more cbuild options, revise configs
BUG=chromium-os:11229
TEST=Local, unittests

Review URL: http://codereview.chromium.org/6245013

Change-Id: I103d693a2e6ce033c1f495f9f48450156ad72e94
2011-01-27 14:04:15 -08:00