Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Ed Bartosh <ed.bartosh@linux.intel.com>
To: Maciej Borzecki <maciej.borzecki@rndity.com>
Cc: Maciek Borzecki <maciek.borzecki@gmail.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH v4 0/5] wic: selftest: enhancements for non-x86 MACHINE support & fixed-size tests
Date: Wed, 22 Mar 2017 14:10:58 +0200	[thread overview]
Message-ID: <20170322121058.GA29465@linux.intel.com> (raw)
In-Reply-To: <cover.1490095649.git.maciej.borzecki@rndity.com>

Hi Maciej,

The patchset looks great! Thank you!


On Tue, Mar 21, 2017 at 12:35:00PM +0100, Maciej Borzecki wrote:
> v4 of a series with enhancements and new test cases for wic selftest.
> The enhancements allow for running wic selftest for non-x86 MACHINE,
> such as qemuarm (the whole series was verified on qemux86-64 and
> qemuarm).
> 
> The first patch adds a machine independent kickstart file - wictestdisk.
> Most of test cases build a disk image using one of the kickstart files
> shipped in Poky (mostly directdisk. These are usually x86 oriented due
> to use of syslinux and cannot be used when running with non-x86
> compatible MACHINE. On the other hand, the image built during tests does
> not need to be bootable (with exception of TCs that verify if the image
> is indeed bootable).
> 
> Patch 2 introduces wictestdisk in test cases where its use is possible.
> 
> Patch 3 adds only_for_arch() convenience decorator and applies it to test
> cases where x86 specific image must be used. In the end, only 13 TCs are
> skipped on qemuarm.
> 
> Changes from previous version:
> - renamed onlyForArch() to only_for_arch(), changed input arguments to
>   take image name and list of compatible architectures
> - pylint fixes
> 
> Patch 4 removes some assumptions about kernel image type.
> 
> Patch 5 adds tests for --fixed-size partition flags.
> 
> Changes from previous version:
> - pylint fixes
> 
> Maciej Borzecki (5):
>   selftest: wictestdisk: machine agnostic WKS for use with selftest
>   selftest: wic: replace directdisk with wictestdisk where possible
>   wic: selftest: avoid COMPATIBLE_HOST issues
>   wic: selftest: do not assume bzImage kernel image
>   wic: selftest: add tests for --fixed-size partition flags
> 
>  meta-selftest/wic/wictestdisk.wks |   7 ++
>  meta/lib/oeqa/selftest/wic.py     | 171 ++++++++++++++++++++++++++++++--------
>  2 files changed, 144 insertions(+), 34 deletions(-)
>  create mode 100644 meta-selftest/wic/wictestdisk.wks
> 
> -- 
> 2.9.3
> 

-- 
--
Regards,
Ed


      parent reply	other threads:[~2017-03-22 12:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-16 12:44 [PATCH v3 0/5] wic: selftest: enhancements for non-x86 MACHINE support & fixed-size tests Maciej Borzecki
2017-03-16 12:44 ` [PATCH v3 1/5] selftest: wictestdisk: machine agnostic WKS for use with selftest Maciej Borzecki
2017-03-16 12:44 ` [PATCH v3 2/5] selftest: wic: replace directdisk with wictestdisk where possible Maciej Borzecki
2017-03-16 12:44 ` [PATCH v3 3/5] wic: selftest: avoid COMPATIBLE_HOST issues Maciej Borzecki
2017-03-16 12:44 ` [PATCH v3 4/5] wic: selftest: do not assume bzImage kernel image Maciej Borzecki
2017-03-16 12:44 ` [PATCH v3 5/5] wic: selftest: add tests for --fixed-size partition flags Maciej Borzecki
2017-03-17 14:40 ` [PATCH v3 0/5] wic: selftest: enhancements for non-x86 MACHINE support & fixed-size tests Ed Bartosh
2017-03-21 11:35   ` [PATCH v4 " Maciej Borzecki
2017-03-21 11:35     ` [PATCH v4 1/5] selftest: wictestdisk: machine agnostic WKS for use with selftest Maciej Borzecki
2017-03-21 11:35     ` [PATCH v4 2/5] selftest: wic: replace directdisk with wictestdisk where possible Maciej Borzecki
2017-03-21 11:35     ` [PATCH v4 3/5] wic: selftest: avoid COMPATIBLE_HOST issues Maciej Borzecki
2017-03-21 11:35     ` [PATCH v4 4/5] wic: selftest: do not assume bzImage kernel image Maciej Borzecki
2017-03-21 11:35     ` [PATCH v4 5/5] wic: selftest: add tests for --fixed-size partition flags Maciej Borzecki
2017-03-22 12:10     ` Ed Bartosh [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170322121058.GA29465@linux.intel.com \
    --to=ed.bartosh@linux.intel.com \
    --cc=maciej.borzecki@rndity.com \
    --cc=maciek.borzecki@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox