<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 03-11-2014 12:36, Aline Manera wrote:<br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite">
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <p> </p>
        <h3>Update a domain snapshot</h3>
      </blockquote>
      <br>
      We can postpone it.<br>
      I think the most important operations are: create, delete and
      revert a snapshot. When we have those 3 operations done we can
      work on update.<br>
    </blockquote>
    <br>
    OK.<br>
    <br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite">
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <h3>CHANGE: List the current snapshot in a domain<br>
        </h3>
      </blockquote>
      <br>
      I'd say to use the API /vms/&lt;name&gt;/snaphosts to provide this
      information.<br>
      You can do it by a parameter or using a specific API for it: GET
      /vms/&lt;name&gt;/snaphosts/current<br>
    </blockquote>
    <br>
    OK. We can use the new API you proposed above or we can add a field
    "current" when listing a snapshot, and then we can find out which
    one is the current one.<br>
    <br>
    I'll stick with your proposal, if there's no problem with it.<br>
    <br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite">
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <ol>
          <li>When creating a snapshot in a running domain, should we
            try to minimize the domain's downtime? There's a libvirt
            flag for that but it increases the memory dump file size.<br>
          </li>
        </ol>
      </blockquote>
      <br>
      As Kimchi can not deal well with paused domains yet, I suggest to
      use the libvirt flag to avoid this state.<br>
      But of course, we can revert it when we properly support paused
      state.<br>
      <br>
    </blockquote>
    <br>
    OK. We could investigate later if the increase in file size is
    actually that big. Otherwise, we can continue creating a live
    snapshot even when we support suspend/resume. That's better from the
    user's perspective IMO.<br>
    <br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite">
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <ol>
          <li> <br>
          </li>
          <li>When deleting a snapshot, should we keep their children or
            should we only delete that snapshot? Keep in mind that
            snapshots follow a tree-like structure. This feature is
            already implemented by libvirt, it makes no difference for
            us in the implementation (it's just a different flag), but
            we need to think if the user would expect that the
            snapshot's children will be deleted as well when deleting
            one snapshot.<br>
          </li>
        </ol>
      </blockquote>
      <br>
      I'd say to only remove the snapshot itself. But we could provide a
      check box to user select or not to remove all its children<br>
      In that case we need to accept a parameter on DELETE request<br>
      <br>
      DELETE /vms/&lt;name&gt;/snapshots/&lt;snap&gt;  {mode: node|all}<br>
      <br>
    </blockquote>
    <br>
    OK, using a parameter seems nice.<br>
    <br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite">
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <ol>
          <li> <br>
          </li>
          <li>The properties returned by the action "Look up one domain
            snapshot" are just the ones I thought it might be useful for
            now. A snapshot has more properties than that but I'm not
            sure if we should just dump everything.<br>
          </li>
        </ol>
      </blockquote>
      <br>
      No. We should provide and expose only the information needed by
      user otherwise they do not make sense.<br>
    </blockquote>
    <br>
    OK.<br>
    <br>
    <blockquote cite="mid:545792F1.5000907@linux.vnet.ibm.com"
      type="cite"> <br>
      <blockquote cite="mid:5456D4BA.2030506@linux.vnet.ibm.com"
        type="cite">
        <ol>
          <li> <br>
          </li>
          <li>The property updated by the action "Update a domain
            snapshot" (i.e. "name") is the only one I thought it might
            be relevant. Other properties can be updated as well if we
            want to.<br>
          </li>
        </ol>
      </blockquote>
      <br>
      I only can think on name as well.<br>
    </blockquote>
    <br>
    Well, we're postponing the update method for now anyway :-)<br>
  </body>
</html>