[lxc-devel] [lxd/master] actually surface the last used update error

[tych0 on Github]

A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 361 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20161115/801df185/attachment.bin>
-------------- next part --------------
From f7b55179e6799507a4a92c926767cfd6db461fdd Mon Sep 17 00:00:00 2001
From: Tycho Andersen <tycho.andersen at canonical.com>
Date: Tue, 15 Nov 2016 11:26:11 -0700
Subject: [PATCH] actually surface the last used update error

Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>
---
 lxd/container_lxc.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lxd/container_lxc.go b/lxd/container_lxc.go
index fec0f2e..d178fdb 100644
--- a/lxd/container_lxc.go
+++ b/lxd/container_lxc.go
@@ -1470,7 +1470,7 @@ func (c *containerLXC) startCommon() (string, error) {
 	// Update time container was last started
 	err = dbContainerLastUsedUpdate(c.daemon.db, c.id, time.Now().UTC())
 	if err != nil {
-		fmt.Printf("Error updating last used: %v", err)
+		return "", fmt.Errorf("Error updating last used: %v", err)
 	}
 
 	return configPath, nil