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

["vijaikumar....@gmail.com" <vijaikumar.kanagarajan@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1888 bytes --]

Hi Baurzhan,

On Wednesday, September 2, 2020 at 1:23:23 AM UTC+5:30 i...@radix50.net 
wrote:

> 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. 
>
>
> > > 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. 
>

My suggestion would be to exclude one of /srv /media instead of /usr. Since 
those
might not be critical for the system to boot.
 

>
>
> 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 
>

We could apply this and I can rework the wic series based on this.
 

>
> 2. Include the CI patch in the wic series and apply that. 
>

Right now we are testing only --exclude-path which this series adds support 
for.
For a comprehensive wic test, I was thinking to have a custom wks file 
which would
have most of the options exercised and  we could use some tools to validate 
whether the
final image is as expected. It is something in my pipeline. Will try to 
work on it and send a patch
as early as possible.

Thanks,
Vijai Kumar K


>
> With kind regards, 
> Baurzhan. 
>

[-- Attachment #1.2: Type: text/html, Size: 3118 bytes --]