WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-api

[Xen-API] [PATCH] Don't copy sm-config when cloning/copying VDIs

To: xen-api@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-API] [PATCH] Don't copy sm-config when cloning/copying VDIs
From: Jon Ludlam <jonathan.ludlam@xxxxxxxxxxxxx>
Date: Fri, 24 Sep 2010 17:33:03 +0100
Delivery-date: Fri, 24 Sep 2010 09:36:01 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-api-request@lists.xensource.com?subject=help>
List-id: Discussion of API issues surrounding Xen <xen-api.lists.xensource.com>
List-post: <mailto:xen-api@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-api-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mercurial-patchbomb/1.4.3
# HG changeset patch
# User Jon Ludlam <Jonathan.Ludlam@xxxxxxxxxxxxx>
# Date 1285345948 -3600
# Node ID b5f83ca52e4bfd898972edb1086e850d37a0a7aa
# Parent  c9096a7ecb48a10462c280b8b672e8bf04bbd3e1
The storage backends use sm-config for various bits of per-vdi info. We 
shouldn't copy that when cloning/copying a VDI.

Signed-off-by: Jon Ludlam <Jonathan.Ludlam@xxxxxxxxxxxxx>

diff -r c9096a7ecb48 -r b5f83ca52e4b ocaml/xapi/xapi_vdi.ml
--- a/ocaml/xapi/xapi_vdi.ml
+++ b/ocaml/xapi/xapi_vdi.ml
@@ -447,7 +447,7 @@
       ~read_only:a.Db_actions.vDI_read_only
       ~other_config:a.Db_actions.vDI_other_config
       ~xenstore_data:a.Db_actions.vDI_xenstore_data
-      ~sm_config:a.Db_actions.vDI_sm_config ~tags:[]
+      ~sm_config:[] ~tags:[]
     in
     (try
        (* Remove the vdi_clone from the SR's current operations, this prevents 
the whole
@@ -488,7 +488,7 @@
           ~read_only:src.API.vDI_read_only
           ~other_config:src.API.vDI_other_config
           ~xenstore_data:src.API.vDI_xenstore_data
-          ~sm_config:src.API.vDI_sm_config ~tags:[] in
+          ~sm_config:[] ~tags:[] in
     if src.API.vDI_on_boot = `reset then begin
                try Client.VDI.set_on_boot ~rpc ~session_id ~self:result 
~value:(`reset) with _ -> ()
        end;

Attachment: xen-api2.hg.patch
Description: Text Data

_______________________________________________
xen-api mailing list
xen-api@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/mailman/listinfo/xen-api
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-API] [PATCH] Don't copy sm-config when cloning/copying VDIs, Jon Ludlam <=