start: set loglevel correctly

We want the loglevel that the user specified when starting the container because it overrides the default one set in the config. Closes #2003. Reported-by: Felix Abecassis <fabecassis@nvidia.com> Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

start: set loglevel correctly
b6ebd1f2 · Christian Brauner · 07b6e2e6 · b6ebd1f2
Unverified Commit b6ebd1f2 authored Dec 06, 2017 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

start.c src/lxc/start.c +4 -2

No files found.
--- a/src/lxc/start.c
+++ b/src/lxc/start.c
@@ -595,6 +595,7 @@ on_error:

 int lxc_init(const char *name, struct lxc_handler *handler)
 {
+	const char *loglevel;
 	struct lxc_conf *conf = handler->conf;

 	lsm_init();
@@ -635,8 +636,9 @@ int lxc_init(const char *name, struct lxc_handler *handler)
 	if (setenv("LXC_CGNS_AWARE", "1", 1))
 		SYSERROR("Failed to set environment variable LXC_CGNS_AWARE=1.");

-	if (setenv("LXC_LOG_LEVEL", lxc_log_priority_to_string(handler->conf->loglevel), 1))
-		SYSERROR("Failed to set environment variable LXC_CGNS_AWARE=1.");
+	loglevel = lxc_log_priority_to_string(lxc_log_get_level());
+	if (setenv("LXC_LOG_LEVEL", loglevel, 1))
+		SYSERROR("Failed to set environment variable LXC_LOG_LEVEL=%s", loglevel);
 	/* End of environment variable setup for hooks. */

 	TRACE("set environment variables");