New issue
Advanced search Search tips

Issue 645342 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

initramfs: add basic testing framework

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

Issue description

the platform/initramfs/ code atm is not easily testable ... you have to set up a device and do a full build in order to find out if your one line change to the init code was junk :).

i posted a CL to improve things a bit:
  https://chromium-review.googlesource.com/266443

just going to outline the goals for the test logic so we can have an end in sight.
(0) we can assume we'll have an existing sysroot for a board -- i.e. someone already ran `./build_packages` for us
(1) build the initramfs for each target
(2) build the kernel with the right settings locally for each target and merge in the initramfs
(3) create a simple raw disk image with the right partition layout
(3a) no need to fully populate the contents of the disk image -- TBD what exactly will be dumped in here
(4) boot the system in qemu, either directly given the kernel, or indirectly given the disk image
(5) all testing from there on out is manual review
(5b) automation to check the results would be nice if feasible, but can punt till later
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/initramfs/+/c69914d450be055badfaab6c925c942c9878bb95

commit c69914d450be055badfaab6c925c942c9878bb95
Author: Mike Frysinger <vapier@chromium.org>
Date: Mon Apr 20 08:45:11 2015

basic framework for testing in qemu

BUG= chromium:645342 
TEST=ran local tests

Change-Id: Ie7a588e79e89ac95f05389401e7efe7e0e8f2e02
Reviewed-on: https://chromium-review.googlesource.com/266443
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>

[add] https://crrev.com/c69914d450be055badfaab6c925c942c9878bb95/test/qemu
[add] https://crrev.com/c69914d450be055badfaab6c925c942c9878bb95/test/README.md
[add] https://crrev.com/c69914d450be055badfaab6c925c942c9878bb95/test/test.sh

Comment 2 by vapier@chromium.org, Oct 17 2016

Status: Fixed (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 22 2016

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

commit f6ed8bfc34655be36a9574bc93c7e77141495e25
Author: Mike Frysinger <vapier@chromium.org>
Date: Mon Apr 20 08:45:11 2015

basic framework for testing in qemu

BUG= chromium:645342 
TEST=ran local tests

Reviewed-on: https://chromium-review.googlesource.com/266443
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>

(cherry picked from commit c69914d450be055badfaab6c925c942c9878bb95)

Change-Id: I068bbe7aac8ac0796ff0964471d51fdd09fb45dc
Reviewed-on: https://chromium-review.googlesource.com/412539
Reviewed-by: Chun-Tsen Kuo <chuntsen@chromium.org>
Commit-Queue: Chun-Tsen Kuo <chuntsen@chromium.org>
Tested-by: Chun-Tsen Kuo <chuntsen@chromium.org>

[add] https://crrev.com/f6ed8bfc34655be36a9574bc93c7e77141495e25/test/qemu
[add] https://crrev.com/f6ed8bfc34655be36a9574bc93c7e77141495e25/test/README.md
[add] https://crrev.com/f6ed8bfc34655be36a9574bc93c7e77141495e25/test/test.sh

Comment 4 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 5 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 6 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 7 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 8 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 9 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)
Status: Fixed (was: Archived)

Sign in to add a comment