incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nitin Mehta <Nitin.Me...@citrix.com>
Subject Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size.
Date Mon, 25 Feb 2013 14:49:51 GMT
True, but you should check that the size is not already set. If it is set then you shouldn't
update the DB

From: Sanjay Tripathi <sanjay.tripathi@citrix.com<mailto:sanjay.tripathi@citrix.com>>
Reply-To: Sanjay Tripathi <sanjay.tripathi@citrix.com<mailto:sanjay.tripathi@citrix.com>>
To: Sateesh Chodapuneedi <sateesh.chodapuneedi@citrix.com<mailto:sateesh.chodapuneedi@citrix.com>>,
"mice_xia@tcloudcomputing.com<mailto:mice_xia@tcloudcomputing.com>" <mice_xia@tcloudcomputing.com<mailto:mice_xia@tcloudcomputing.com>>,
Min Chen <min.chen@citrix.com<mailto:min.chen@citrix.com>>, Devdeep Singh <devdeep.singh@citrix.com<mailto:devdeep.singh@citrix.com>>
Cc: Nitin Mehta <nitin.mehta@citrix.com<mailto:nitin.mehta@citrix.com>>, "cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>"
<cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>>,
Sanjay Tripathi <sanjay.tripathi@citrix.com<mailto:sanjay.tripathi@citrix.com>>
Subject: Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume
size.

This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9558/


On February 25th, 2013, 12:37 p.m., Nitin Mehta wrote:

server/src/com/cloud/storage/download/DownloadMonitorImpl.java<https://reviews.apache.org/r/9558/diff/1/?file=261150#file261150line742>
(Diff revision 1)

{'text': 'public void handleVolumeSync(HostVO ssHost) {', 'line': 693, 'expand_offset': 18}


        742

                    // Set volume size in volumes table


What if the size is already set ? why do we need to set it again ?

This is for the final sync of volumes (same way we update the volume_host_ref table) in cases
if the volume is not completely downloaded and MS is in shutdown state(this may be one case)
etc.


- Sanjay


On February 25th, 2013, 12:31 p.m., Sanjay Tripathi wrote:

Review request for cloudstack, Devdeep Singh, Sateesh Chodapuneedi, mice xia, and Min Chen.
By Sanjay Tripathi.

Updated Feb. 25, 2013, 12:31 p.m.

Description

When user uploads a volume using uploadVolume API command, only volume_host_ref table is getting
updated with corresponding size of volume and "volumes" table, the size of volume still remains
zero even ager the volume gets completely downloaded.


Testing

Verified the fix on my local CloudStack setup.


Bugs: CLOUDSTACK-1374
Diffs

 *   server/src/com/cloud/storage/download/DownloadListener.java (d0b1868)
 *   server/src/com/cloud/storage/download/DownloadMonitorImpl.java (e12bc32)

View Diff<https://reviews.apache.org/r/9558/diff/>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message