commit 8f80c35bf9a264f17fc08a415bd09ab1121f4e38
author Marri Devender Rao <devenrao@in.ibm.com> Mon May 13 00:53:01 2019 -0500
committer MARRI DEVENDER RAO <devenrao@in.ibm.com> Mon Jun 03 13:37:56 2019 +0000
tree d0fc7e40e763057d4dbce7892abe5f0c950c1ffc
parent 15647204a62e504a83d46e37449237374a14cad1

Pass flag to skip restart of units for certificate object

During bootup Certificate objects are created by
loading the existing certificates in the system.

At present system is restarting/reloading units
after a certificate object is created, but the
units to restart/reload might not be up yet
causing failure.

Reloading of services is required only when a new
certificate is installed/replaced onto the system.

Modified to not to reload the specified units for
the certificate objects created for existing
certificates in the system.

Change-Id: I211a8386de1a5aa0a42d11cb89945bafa6792ba4
Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>

test/certs_manager_test.cpp

diff --git a/test/certs_manager_test.cpp b/test/certs_manager_test.cpp
index 7b318d7..9fd7110 100644
--- a/test/certs_manager_test.cpp
+++ b/test/certs_manager_test.cpp
@@ -114,7 +114,7 @@
     UnitsToRestart verifyUnit(unit);
     auto objPath = std::string(OBJPATH) + '/' + type + '/' + endpoint;
     Certificate certificate(bus, objPath, type, unit, installPath,
-                            certificateFile);
+                            certificateFile, false);
     EXPECT_TRUE(fs::exists(verifyPath));
 }
 
@@ -130,7 +130,7 @@
     UnitsToRestart verifyUnit(unit);
     auto objPath = std::string(OBJPATH) + '/' + type + '/' + endpoint;
     Certificate certificate(bus, objPath, type, unit, installPath,
-                            certificateFile);
+                            certificateFile, false);
     EXPECT_TRUE(fs::exists(verifyPath));
 }
 
@@ -146,7 +146,7 @@
     UnitsToRestart verifyUnit(unit);
     auto objPath = std::string(OBJPATH) + '/' + type + '/' + endpoint;
     Certificate certificate(bus, objPath, type, unit, installPath,
-                            certificateFile);
+                            certificateFile, false);
     EXPECT_TRUE(fs::exists(verifyPath));
 }
 
@@ -162,7 +162,7 @@
     UnitsToRestart verifyUnit(unit);
     auto objPath = std::string(OBJPATH) + '/' + type + '/' + endpoint;
     Certificate certificate(bus, objPath, type, unit, installPath,
-                            certificateFile);
+                            certificateFile, false);
     EXPECT_TRUE(fs::exists(verifyPath));
     EXPECT_TRUE(compareFiles(verifyPath, certificateFile));
 }
@@ -184,7 +184,7 @@
             try
             {
                 Certificate certificate(bus, objPath, type, unit, installPath,
-                                        uploadFile);
+                                        uploadFile, false);
             }
             catch (const InternalFailure& e)
             {
@@ -215,7 +215,7 @@
             try
             {
                 Certificate certificate(bus, objPath, type, unit, installPath,
-                                        emptyFile);
+                                        emptyFile, false);
             }
             catch (const InvalidCertificate& e)
             {
@@ -251,7 +251,7 @@
             try
             {
                 Certificate certificate(bus, objPath, type, unit, installPath,
-                                        certificateFile);
+                                        certificateFile, false);
             }
             catch (const InvalidCertificate& e)
             {
@@ -361,7 +361,7 @@
             try
             {
                 Certificate certificate(bus, objPath, type, unit, installPath,
-                                        certificateFile);
+                                        certificateFile, false);
             }
             catch (const InvalidCertificate& e)
             {
@@ -389,7 +389,7 @@
             try
             {
                 Certificate certificate(bus, objPath, type, unit, installPath,
-                                        keyFile);
+                                        keyFile, false);
             }
             catch (const InvalidCertificate& e)
             {