Re: Reminder: please review: PR 50307 - Revise CleanAllRUV restart process

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 04/03/2019 03:24 PM, Mark Reynolds wrote:

On 4/3/19 9:01 AM, Ludwig Krispenz wrote:
Hi,
added some small comments, but it is overall ok.
I'll review them shortly :-)

Just thinking about it, wouldn't it be an option that the task entry survives until it is completed and written to the dse.ldif at shutdown, so at startup tasks could be resumed from the task entry not from params in the repl entry ?

I like the idea, but I don't know if the server will. I ran into "timing" issues just trying to add the task during plugin startup. I'm worried that when we process dse.ldif it might be too soon to start a task.
but i think the event queue is only started after plugins did start, so registering an event should be ok
Either way, it's a good idea and it can remove my hacky code and allow us to cleanup some of the cleanAllRUV code too, so I'm going to try it! I'll add a "nsTaskPersist" attribute to the task, etc...

Thanks,
Mark


Ludwig

On 04/03/2019 02:27 PM, Mark Reynolds wrote:
Friendly reminder :-)


https://pagure.io/389-ds-base/pull-request/50307
_______________________________________________
_______________________________________________
389-devel mailing list -- 389-devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to 389-devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/389-devel@xxxxxxxxxxxxxxxxxxxxxxx


--
Red Hat GmbH, http://www.de.redhat.com/, Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Michael Cunningham, Michael O'Neill, Eric Shander
_______________________________________________
389-devel mailing list -- 389-devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to 389-devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/389-devel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Directory Announce]     [Fedora Users]     [Older Fedora Users Mail]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Review]     [Fedora Art]     [Fedora Music]     [Fedora Packaging]     [CentOS]     [Fedora SELinux]     [Big List of Linux Books]     [KDE Users]     [Fedora Art]     [Fedora Docs]

  Powered by Linux