e6fda6f upgrade: avoid stopping certmonger when fixing requests

Authored and Committed by ftweedal 3 years ago
    upgrade: avoid stopping certmonger when fixing requests
    
    During upgrade, if discrepancies are detected in Certmonger tracking
    request configuration we remove and re-create tracking requests.
    The default behaviour of the CAInstance and KRAInstance
    stop_tracking_certificates() method is to stop certmonger after the
    requests have been removed.  This behaviour results in an
    unnecessary restart of certmonger and has also been observed to
    cause problems.  For example, subsequent certmonger operations have
    to start the certmonger process and can fail because certmonger is
    not yet properly initialised (manifesting as D-Bus errors).
    
    Suppress the unnecessary restart(s) of certmonger during tracking
    request update.
    
    Related: https://pagure.io/freeipa/issue/8186
    Reviewed-By: Alexander Bokovoy <abbra@users.noreply.github.com>