Re: [bitbake-devel] [PATCH v5 2/5] prserv: move code from __init__ to bitbake-prserv

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Fri, 2024-05-03 at 07:14 -0700, Khem Raj via lists.openembedded.org
wrote:
> 
> 
> On Fri, May 3, 2024 at 1:04 AM Michael Opdenacker
> <michael.opdenacker@bootlin.com> wrote:
> > Hi Khem
> > 
> > On 5/2/24 at 18:57, Khem Raj wrote:
> > > Hi Michael
> > > 
> > > I am seeing a problem with this series, specifically following
> > > patchset is causing a regression
> > > 
> > > * ea1d7b8a36e bitbake: prserv: add bitbake selftests
> > > * 5c66d345300 bitbake: prserv: import simplification
> > > * c23f2c1680f bitbake: prserv: store_value() improvements
> > > * 395b321af29 bitbake: prserv: avoid possible race condition in
> > > database code
> > > * 8022dc1029d bitbake: prserv: enable database sharing
> > > * b81bfdfbed8 bitbake: prserv: add "upstream" server support
> > > * 9539ce3b14a bitbake: prserv: move code from __init__ to
> > > bitbake-prserv
> > > * 4ec1553710b bitbake: prserv: declare "max_package_pr" client
> > > hook
> > > 
> > > 
> > > error seen during packagedata task of base-files
> > > 
> > > WARNING: base-files-3.0.14-r0 do_packagedata: Error talking to
> > > server:
> > > Connection closed
> > > ERROR: Error from client ('127.0.0.1', 59846): 'PRTable' object
> > > has no
> > > attribute 'sync'
> > > Traceback (most recent call last):
> > 
> > 
> > Thanks for the review. Actually, did you test with the V6 update I
> > sent?
> > 
> 
> 
> I think it was version in master-next that Richard has staged
> 

It was against v6 so I'm not able to merge these until that is fixed.

Cheers,

Richard