The following pull request was submitted through Github.
It can be accessed and reviewed at: https://github.com/lxc/lxd/pull/1730

This e-mail was sent by the LXC bot, direct replies will not reach the author
unless they happen to be subscribed to this list.

=== Description (from pull-request) ===
This pull requests fixes inaccurate error message of snapshotting with existing snapshot name (issue #1725) by using message format "Snapshot '<container>/<snapshot>' already exists".

Signed-off-by: Janne Savikko <[email protected]>
From 42e86b7fff339b7a57c7f5d918e28d22c50fe286 Mon Sep 17 00:00:00 2001
From: Janne Savikko <[email protected]>
Date: Thu, 10 Mar 2016 00:31:18 +0200
Subject: [PATCH] Fix error message when snapshot with duplicate name

Signed-off-by: Janne Savikko <[email protected]>
---
 lxd/container.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lxd/container.go b/lxd/container.go
index 6a14fd9..ef34401 100644
--- a/lxd/container.go
+++ b/lxd/container.go
@@ -594,7 +594,7 @@ func containerCreateInternal(d *Daemon, args containerArgs) 
(container, error) {
 
        path := containerPath(args.Name, args.Ctype == cTypeSnapshot)
        if shared.PathExists(path) {
-               return nil, fmt.Errorf("The container already exists")
+               return nil, fmt.Errorf("Snapshot '%s' already exists", 
args.Name)
        }
 
        // Wipe any existing log for this container name
_______________________________________________
lxc-devel mailing list
[email protected]
http://lists.linuxcontainers.org/listinfo/lxc-devel

Reply via email to