All of lore.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: [PATCHv2 4/4] meta-selftest: wic: Add test for --use-uuid / --fsuuid
Date: Tue, 23 Jan 2018 16:05:16 -0600	[thread overview]
Message-ID: <c03d28a1-8495-5885-2a2f-43061ff077c4@gmail.com> (raw)
In-Reply-To: <20171214031512.GK372@bill-the-cat>

On 12/13/17 9:15 PM, Tom Rini wrote:
> On Wed, Dec 13, 2017 at 09:45:13PM -0500, Tom Rini wrote:
>> On Wed, Dec 13, 2017 at 02:40:34PM +0000, Burton, Ross wrote:
>>
>>> I finally made it break locally, with MACHINE=qemux86 and using sysvinit
>>> instead of systemd.  Basically, a pristine poky local.conf.  I've not dug
>>> into it beyond that, or why it passes with systemd.
>>
>> I found it, and, erm, ugh?  Here's what appears to be a systemd vs
>> sysvinit difference.  With systemd, if you have an fstab entry and the
>> mount point doesn't exist, it will get made.  So an entry of /uuid (a
>> very non-HFS and just made up directory) the mount happens.  With
>> sysvinit, no, the mount fails.  That's why the test fails.
>>
>> My gut reaction is that no, it's not wic's job to ensure that the
>> underlying filesystems have all mount points and I should just re-do the
>> test to use /mnt instead of /uuid, for the UUID test mount point.
> 
> And in fact, systemd just can't be used for oe-selftest and have
> everything expect to pass today.  The existing wic.Wic.test_qemu will
> fail on /dev/root vs /dev/sda2.  So I'm going to post a v4 that puts the
> UUID one into /media, and I've run the tests and they pass atm.  Thanks!
> 

/media is usually automounted when removable media is inserted. you
might want to use something else.



  reply	other threads:[~2018-01-23 22:05 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-17 16:08 [PATCHv2 0/4] wic: Further enhance UUID / fstab support Tom Rini
2017-11-17 16:08 ` [PATCHv2 1/4] wic: kparser.py: Check for SquashFS and use-uuid Tom Rini
2017-11-17 16:08 ` [PATCHv2 2/4] wic: partition.py: Update comments slightly Tom Rini
2017-11-17 16:08 ` [PATCHv2 3/4] wic: Introduce --fsuuid and have --use-uuid make use of UUID too Tom Rini
2017-11-17 16:08 ` [PATCHv2 4/4] meta-selftest: wic: Add test for --use-uuid / --fsuuid Tom Rini
2017-11-24 15:28   ` Burton, Ross
2017-11-24 15:36     ` Tom Rini
2017-11-28 15:55       ` Tom Rini
2017-12-13 14:40         ` Burton, Ross
2017-12-14  1:22           ` Tom Rini
2017-12-14  2:45           ` Tom Rini
2017-12-14  3:15             ` Tom Rini
2018-01-23 22:05               ` Khem Raj [this message]
2017-11-22  8:39 ` [PATCHv2 0/4] wic: Further enhance UUID / fstab support Ed Bartosh

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=c03d28a1-8495-5885-2a2f-43061ff077c4@gmail.com \
    --to=raj.khem@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.