[Pki-devel] [PATCH] 0062 Block startup until initial profile load completed

Fraser Tweedale ftweedal at redhat.com
Thu Jan 21 01:07:14 UTC 2016


On Wed, Jan 20, 2016 at 02:30:40PM -0600, Endi Sukma Dewata wrote:
> On 11/30/2015 11:57 PM, Fraser Tweedale wrote:
> >Hi all,
> >
> >The attached patch fixes https://fedorahosted.org/pki/ticket/1702
> >
> >Cheers,
> >Fraser
> 
> ACK. Just some comments:
> 
Thanks; pushed to master (5fae5826e5442d7266681d19f282dc7728062b89).
Will push to 10.2 branch shortly.

> 1. It would be nice to log the InterruptedException so we know if it
> happens.
> 
I added a log message.

> 2. Not a problem, but the code that reads the numSubordinates probably could
> be moved before the loop so it's guaranteed to be executed exactly once.
> 
It has to be done as part of the one search, to guarantee that no
profiles are added/removed between the search to determine the
number of profiles, and a subsequent search to load them.

Cheers,
Fraser




More information about the Pki-devel mailing list