[libvirt] [PATCH 3/6] lxc: driver: Improve logging when launching emulator
Daniel P. Berrange
berrange at redhat.com
Tue May 10 09:47:06 UTC 2011
On Fri, May 06, 2011 at 01:26:08PM -0400, Cole Robinson wrote:
> Log the full command line and a timestamp like we do for QEMU
>
> Signed-off-by: Cole Robinson <crobinso at redhat.com>
> ---
> src/lxc/lxc_driver.c | 23 +++++++++++++++++++++++
> 1 files changed, 23 insertions(+), 0 deletions(-)
>
> diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c
> index 2537f5d..7aed910 100644
> --- a/src/lxc/lxc_driver.c
> +++ b/src/lxc/lxc_driver.c
> @@ -56,6 +56,8 @@
>
> #define VIR_FROM_THIS VIR_FROM_LXC
>
> +#define START_POSTFIX ": starting up\n"
> +
> #define LXC_NB_MEM_PARAM 3
>
> typedef struct _lxcDomainObjPrivate lxcDomainObjPrivate;
> @@ -1254,6 +1256,9 @@ static int lxcControllerStart(lxc_driver_t *driver,
> char *filterstr;
> char *outputstr;
> virCommandPtr cmd;
> + off_t pos = -1;
> + char ebuf[1024];
> + char *timestamp;
>
> cmd = virCommandNew(vm->def->emulator);
>
> @@ -1315,6 +1320,24 @@ static int lxcControllerStart(lxc_driver_t *driver,
> goto cleanup;
> }
>
> + /* Log timestamp */
> + if ((timestamp = virTimestamp()) == NULL) {
> + virReportOOMError();
> + goto cleanup;
> + }
> + if (safewrite(logfile, timestamp, strlen(timestamp)) < 0 ||
> + safewrite(logfile, START_POSTFIX, strlen(START_POSTFIX)) < 0) {
> + VIR_WARN("Unable to write timestamp to logfile: %s",
> + virStrerror(errno, ebuf, sizeof ebuf));
> + }
> + VIR_FREE(timestamp);
> +
> + /* Log generated command line */
> + virCommandWriteArgLog(cmd, logfile);
> + if ((pos = lseek(logfile, 0, SEEK_END)) < 0)
> + VIR_WARN("Unable to seek to end of logfile: %s",
> + virStrerror(errno, ebuf, sizeof ebuf));
> +
> virCommandPreserveFD(cmd, appPty);
> virCommandSetOutputFD(cmd, &logfile);
> virCommandSetErrorFD(cmd, &logfile);
ACK,
No equivalent in shutdown though ?
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list