How about set hypervisorType to Any? Haven't take a look at the master change yet.
From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Thursday, June 13, 2013 1:41 PM To: dev@cloudstack.apache.org Cc: Edison Su Subject: Re: [MERGE] disk_io_throttling to MASTER (Second Round) Actually, I am noticing some new behavior around picking a storage pool for zone-wide storage. The current implementation that I've brought down from master no longer finds storage for me because my plug-in is zone wide and not associated with a hypervisor. Edison? On Thu, Jun 13, 2013 at 1:13 PM, Mike Tutkowski <mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>> wrote: Hi Edison, I notice after I updated from master that Hypervisor Type is now a required parameter for creating a storage pool. Should I just use HypervisorType.Any? Thanks! On Thu, Jun 13, 2013 at 12:21 PM, John Burwell <jburw...@basho.com<mailto:jburw...@basho.com>> wrote: Wei, I published my review. I didn't see any code to validating the rate values (e.g. values greater than 0, values less than an maximum value). Did I miss it? I also noticed that 0 is being used when no value has been specified. I recommend using the Long type rather primitive long in order to use null to represent unspecified values rather than a magic value. Thanks, -John On Jun 13, 2013, at 11:34 AM, Wei ZHOU <ustcweiz...@gmail.com<mailto:ustcweiz...@gmail.com>> wrote: > John, > > Please review the code on https://reviews.apache.org/r/11782 > The storage provisioned IOPS does not affect hypervisor throttled I/O, I > think. > Mike may change UI and java code for storage provisioned IOPS after the > merge. > > -Wei > > > 2013/6/13 John Burwell <jburw...@basho.com<mailto:jburw...@basho.com>> > >> Wei, >> >> There are open questions on the thread regarding mutual exclusion of >> hypervisor throttled I/O and storage provisioned IOPS. We need to >> understand how and where it will be implemented in both the UI and >> service layers. Also, can you resend the Review Board review? My >> email search skills have failed to find it. >> >> Thanks, >> -John >> >> -- Mike Tutkowski Senior CloudStack Developer, SolidFire Inc. e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com> o: 303.746.7302<tel:303.746.7302> Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play>(tm) -- Mike Tutkowski Senior CloudStack Developer, SolidFire Inc. e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com> o: 303.746.7302 Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play>(tm)
