From: Christian Brauner Date: Fri, 27 Oct 2017 19:15:48 +0000 (+0200) Subject: tests: add tests for console_log() X-Git-Tag: lxc-3.0.0.beta1~195^2 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a52c1c68cfeeaaf27768608b7179d53d8efb8122;p=thirdparty%2Flxc.git tests: add tests for console_log() Signed-off-by: Christian Brauner --- diff --git a/.gitignore b/.gitignore index 3930b11fa..892489cc9 100644 --- a/.gitignore +++ b/.gitignore @@ -78,6 +78,7 @@ src/tests/lxc-test-cgpath src/tests/lxc-test-clonetest src/tests/lxc-test-concurrent src/tests/lxc-test-console +src/tests/lxc-test-console-log src/tests/lxc-test-containertests src/tests/lxc-test-createtest src/tests/lxc-test-destroytest diff --git a/src/tests/Makefile.am b/src/tests/Makefile.am index 1fd58b2cb..c9a2dd275 100644 --- a/src/tests/Makefile.am +++ b/src/tests/Makefile.am @@ -15,6 +15,7 @@ lxc_test_lxcpath_SOURCES = lxcpath.c lxc_test_cgpath_SOURCES = cgpath.c lxc_test_clonetest_SOURCES = clonetest.c lxc_test_console_SOURCES = console.c +lxc_test_console_log_SOURCES = console_log.c lxctest.h lxc_test_snapshot_SOURCES = snapshot.c lxc_test_concurrent_SOURCES = concurrent.c lxc_test_may_control_SOURCES = may_control.c @@ -52,7 +53,7 @@ endif bin_PROGRAMS = lxc-test-containertests lxc-test-locktests lxc-test-startone \ lxc-test-destroytest lxc-test-saveconfig lxc-test-createtest \ lxc-test-shutdowntest lxc-test-get_item lxc-test-getkeys lxc-test-lxcpath \ - lxc-test-cgpath lxc-test-clonetest lxc-test-console \ + lxc-test-cgpath lxc-test-clonetest lxc-test-console lxc-test-console-log \ lxc-test-snapshot lxc-test-concurrent lxc-test-may-control \ lxc-test-reboot lxc-test-list lxc-test-attach lxc-test-device-add-remove \ lxc-test-apparmor lxc-test-utils lxc-test-parse-config-file \ @@ -85,6 +86,7 @@ EXTRA_DIST = \ concurrent.c \ config_jump_table.c \ console.c \ + console_log.c \ containertests.c \ createtest.c \ destroytest.c \ diff --git a/src/tests/console_log.c b/src/tests/console_log.c new file mode 100644 index 000000000..83a722967 --- /dev/null +++ b/src/tests/console_log.c @@ -0,0 +1,217 @@ +/* liblxcapi + * + * Copyright © 2017 Christian Brauner . + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2, as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#define __STDC_FORMAT_MACROS +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include + +#include "lxctest.h" +#include "utils.h" + +int main(int argc, char *argv[]) +{ + int logfd, ret; + char buf[4096 + 1]; + ssize_t bytes; + struct stat st; + struct lxc_container *c; + struct lxc_console_log log; + bool do_unlink = false; + int fret = EXIT_FAILURE; + + c = lxc_container_new("console-log", NULL); + if (!c) { + lxc_error("%s", "Failed to create container \"console-log\""); + exit(fret); + } + + if (c->is_defined(c)) { + lxc_error("%s\n", "Container \"console-log\" is defined"); + goto on_error_put; + } + + /* Set console ringbuffer size. */ + if (!c->set_config_item(c, "lxc.console.logsize", "4096")) { + lxc_error("%s\n", "Failed to set config item \"lxc.console.logsize\""); + goto on_error_put; + } + + /* Set on-disk logfile. */ + if (!c->set_config_item(c, "lxc.console.logfile", "/tmp/console-log.log")) { + lxc_error("%s\n", "Failed to set config item \"lxc.console.logfile\""); + goto on_error_put; + } + + if (!c->createl(c, "busybox", NULL, NULL, 0, NULL)) { + lxc_error("%s\n", "Failed to create busybox container \"console-log\""); + goto on_error_put; + } + + if (!c->is_defined(c)) { + lxc_error("%s\n", "Container \"console-log\" is not defined"); + goto on_error_put; + } + + c->clear_config(c); + + if (!c->load_config(c, NULL)) { + lxc_error("%s\n", "Failed to load config for container \"console-log\""); + goto on_error_stop; + } + + if (!c->want_daemonize(c, true)) { + lxc_error("%s\n", "Failed to mark container \"console-log\" daemonized"); + goto on_error_stop; + } + + if (!c->startl(c, 0, NULL)) { + lxc_error("%s\n", "Failed to start container \"console-log\" daemonized"); + goto on_error_stop; + } + + /* Leave some time for the container to write something to the log. */ + sleep(2); + + /* Retrieve the contents of the ringbuffer. */ + log.clear = false; + log.read_max = &(uint64_t){0}; + log.read = true; + log.write_logfile = false; + + ret = c->console_log(c, &log); + if (ret < 0) { + lxc_error("%s - Failed to retrieve console log \n", strerror(-ret)); + goto on_error_stop; + } else { + lxc_debug("Retrieved %" PRIu64 + " bytes from console log. Contents are \"%s\"\n", + *log.read_max, log.data); + } + + /* Leave another two seconds to ensure boot is finished. */ + sleep(2); + + /* Clear the console ringbuffer. */ + log.read_max = &(uint64_t){0}; + log.read = false; + log.write_logfile = false; + log.clear = true; + ret = c->console_log(c, &log); + if (ret < 0) { + if (ret != -ENODATA) { + lxc_error("%s - Failed to retrieve console log\n", strerror(-ret)); + goto on_error_stop; + } + } + + if (!c->stop(c)) { + lxc_error("%s\n", "Failed to stop container \"console-log\""); + goto on_error_stop; + } + + if (!c->startl(c, 0, NULL)) { + lxc_error("%s\n", "Failed to start container \"console-log\" daemonized"); + goto on_error_destroy; + } + do_unlink = true; + + /* Leave some time for the container to write something to the log. */ + sleep(2); + + log.read_max = &(uint64_t){0}; + log.read = true; + log.write_logfile = true; + log.clear = false; + ret = c->console_log(c, &log); + if (ret < 0) { + lxc_error("%s - Failed to retrieve console log \n", strerror(-ret)); + goto on_error_stop; + } else { + lxc_debug("Retrieved %" PRIu64 + " bytes from console log. Contents are \"%s\"\n", + *log.read_max, log.data); + } + + logfd = open("/tmp/console-log.log", O_RDONLY); + if (logfd < 0) { + lxc_error("%s - Failed to open console log file " + "\"/tmp/console-log.log\"\n", strerror(errno)); + goto on_error_stop; + } + + bytes = lxc_read_nointr(logfd, buf, 4096 + 1); + close(logfd); + if (bytes < 0 || ((uint64_t)bytes != *log.read_max)) { + lxc_error("%s - Failed to read console log file " + "\"/tmp/console-log.log\"\n", strerror(errno)); + goto on_error_stop; + } + + ret = stat("/tmp/console-log.log", &st); + if (ret < 0) { + lxc_error("%s - Failed to stat on-disk logfile\n", strerror(errno)); + goto on_error_stop; + } + + if ((uint64_t)st.st_size != *log.read_max) { + lxc_error("On-disk logfile size and used ringbuffer size do " + "not match: %" PRIu64 " != %" PRIu64 "\n", + (uint64_t)st.st_size, *log.read_max); + goto on_error_stop; + } + + if (memcmp(log.data, buf, *log.read_max)) { + lxc_error("%s - Contents of in-memory ringbuffer and on-disk " + "logfile do not match\n", strerror(errno)); + goto on_error_stop; + } else { + lxc_debug("Retrieved %" PRIu64 " bytes from console log and " + "console log file. Contents are: \"%s\" - \"%s\"\n", + *log.read_max, log.data, buf); + } + + fret = 0; + +on_error_stop: + if (c->is_running(c) && !c->stop(c)) + lxc_error("%s\n", "Failed to stop container \"console-log\""); + +on_error_destroy: + if (!c->destroy(c)) + lxc_error("%s\n", "Failed to destroy container \"console-log\""); + +on_error_put: + lxc_container_put(c); + if (do_unlink) { + ret = unlink("/tmp/console-log.log"); + if (ret < 0) + lxc_error("%s - Failed to remove container log file\n", strerror(errno)); + } + exit(fret); +}