Return-Path: X-Original-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A97AFE597 for ; Tue, 26 Feb 2013 05:10:03 +0000 (UTC) Received: (qmail 84052 invoked by uid 500); 26 Feb 2013 05:10:03 -0000 Delivered-To: apmail-incubator-cloudstack-dev-archive@incubator.apache.org Received: (qmail 83844 invoked by uid 500); 26 Feb 2013 05:10:02 -0000 Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cloudstack-dev@incubator.apache.org Delivered-To: mailing list cloudstack-dev@incubator.apache.org Received: (qmail 83790 invoked by uid 99); 26 Feb 2013 05:10:01 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 26 Feb 2013 05:10:01 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 26D0E1C7A44; Tue, 26 Feb 2013 05:09:53 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============5269890093779613220==" MIME-Version: 1.0 Subject: Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size. From: "Nitin Mehta" To: "Sateesh Chodapuneedi" , "mice xia" , "Min Chen" , "Devdeep Singh" Cc: "Nitin Mehta" , "cloudstack" , "Sanjay Tripathi" Date: Tue, 26 Feb 2013 05:09:52 -0000 Message-ID: <20130226050952.1280.26840@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Nitin Mehta" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/9558/ X-Sender: "Nitin Mehta" References: <20130225161513.1280.52518@reviews.apache.org> In-Reply-To: <20130225161513.1280.52518@reviews.apache.org> Reply-To: "Nitin Mehta" --===============5269890093779613220== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9558/#review17072 ----------------------------------------------------------- Ship it! Ship It! - Nitin Mehta On Feb. 25, 2013, 4:15 p.m., Sanjay Tripathi wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9558/ > ----------------------------------------------------------- > = > (Updated Feb. 25, 2013, 4:15 p.m.) > = > = > Review request for cloudstack, Devdeep Singh, Sateesh Chodapuneedi, mice = xia, and Min Chen. > = > = > Description > ------- > = > When user uploads a volume using uploadVolume API command, only volume_ho= st_ref table is getting updated with corresponding size of volume and "volu= mes" table, the size of volume still remains zero even ager the volume gets= completely downloaded. > = > = > This addresses bug CLOUDSTACK-1374. > = > = > Diffs > ----- > = > server/src/com/cloud/storage/download/DownloadListener.java d0b1868 = > server/src/com/cloud/storage/download/DownloadMonitorImpl.java 1fd1996 = > = > Diff: https://reviews.apache.org/r/9558/diff/ > = > = > Testing > ------- > = > Verified the fix on my local CloudStack setup. > = > = > Thanks, > = > Sanjay Tripathi > = > --===============5269890093779613220==--