Re: [PATCH 2/3] Ensure logging is initialized early in libvirt_lxc

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

 



On 05/01/2012 06:10 AM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"<berrange@xxxxxxxxxx>

The virLogSetFromEnv call was done too late in startup to
catch many log messages (eg from security driver initialization).
To assist debugging also explicitly log the security details
at startup
---
  src/lxc/lxc_controller.c |   12 +++++++++---
  1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/lxc/lxc_controller.c b/src/lxc/lxc_controller.c
index 1e3ec30..26b3115 100644
--- a/src/lxc/lxc_controller.c
+++ b/src/lxc/lxc_controller.c
@@ -1668,6 +1668,9 @@ int main(int argc, char *argv[])
          exit(EXIT_FAILURE);
      }

+    /* Initialize logging */
+    virLogSetFromEnv();
+
      while (1) {
          int c;

@@ -1784,6 +1787,12 @@ int main(int argc, char *argv[])
                                       0)) == NULL)
          goto cleanup;

+    VIR_DEBUG("Security model %s type %s label %s imagelabel %s",
+              NULLSTR(def->seclabel.model),
+              virDomainSeclabelTypeToString(def->seclabel.type),
+              NULLSTR(def->seclabel.label),
+              NULLSTR(def->seclabel.imagelabel));
+
      if (def->nnets != nveths) {
          fprintf(stderr, "%s: expecting %d veths, but got %d\n",
                  argv[0], def->nnets, nveths);
@@ -1828,9 +1837,6 @@ int main(int argc, char *argv[])
          }
      }

-    /* Initialize logging */
-    virLogSetFromEnv();
-
      /* Accept initial client which is the libvirtd daemon */
      if ((client = accept(monitor, NULL, 0))<  0) {
          virReportSystemError(errno, "%s",

ACK

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]