Re: [PATCH 0/3] fix and test wic exclude-path

[Henning Schild <henning.schild@siemens.com>]

Am Tue, 1 Sep 2020 21:53:14 +0200
schrieb Baurzhan Ismagulov <ibr@radix50.net>:

> Hello Henning,
> 
> On Fri, Jun 12, 2020 at 08:03:35PM +0200, Henning Schild wrote:
> > > 1. I'd like to understand what exactly is failing when the fix is
> > > not applied.  
> > 
> > You have the code for that. Just revert the fix and you will see.  
> 
> Fair enough -- just thought it would speed up things.

Using the exclude will cause our wic to raise an exception

> 
> > > 2. The test case is failing with the fix applied.  
> > 
> > Just apply the changes and you will see ;).  
> 
> I meant that your patches started to fail in CI, see e.g.
> http://ci.isar-build.org:8080/job/isar_ibr_devel_2_fast/12/consoleFull.
> The build succeeds, but start_vm fails due to init dying.
> 
> The problem seems to be that Debian doesn't support operating without
> /usr anymore. We can fix it by saving a backup of the image before
> generating one without /usr -- I'll send the updated patch.

Oh i see. The idea was to just test wic and whether it would still crap
out with its exception. There was no focus on whether that image was in
fact bootable. And i am not surprised it is not ...
 
> I'd be ready to apply this with the modification above. I see that
> the wic series also touches this part but doesn't include the CI
> patch. What would be the preferred way?
> 
> 1. Apply this, rework the wic series, OR
> 
> 2. Include the CI patch in the wic series and apply that.

3. apply the wic series and do not even care about the test

The test was just there to motivate the backport, if we bump that would
be much better.

Henning 
> 
> With kind regards,
> Baurzhan.
>