Thanks Mike for the prompt response :).
On 10/03/14 10:44 AM, "Mike Tutkowski" <mike.tutkow...@solidfire.com>
wrote:
>1d74daf has the fix.
>
>Thanks!
>
>
>On Mon, Mar 10, 2014 at 11:42 AM, Mike Tutkowski <
>mike.tutkow...@solidfire.com> wrote:
>
>> OK, I'm sorry about that - I missed a case in testing.
>>
>> Line 191 in CloudOrchestrator should be this:
>>
>> if (offering.isCustomizedIops() != null && offering.isCustomizedIops())
>>{
>>
>> isCustomizedIops() returns a Boolean - not a boolean.
>>
>> I'll check in this fix right now.
>>
>>
>> On Mon, Mar 10, 2014 at 11:32 AM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> Let me see if I can repro it.
>>>
>>> One sec
>>>
>>>
>>> On Mon, Mar 10, 2014 at 11:11 AM, Nitin Mehta
>>><nitin.me...@citrix.com>wrote:
>>>
>>>> I am facing the same issue as well.
>>>>
>>>> On 10/03/14 10:09 AM, "Koushik Das" <koushik....@citrix.com> wrote:
>>>>
>>>> >This is a new install using master. So this is not about db changes
>>>>but
>>>> >mostly a bug.
>>>> >
>>>> >On 10-Mar-2014, at 9:58 PM, Marcus <shadow...@gmail.com> wrote:
>>>> >
>>>> >> Mike has sent out several emails about the DB changes he made.
>>>> >>
>>>> >> On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das
>>>><koushik....@citrix.com>
>>>> >>wrote:
>>>> >>> I am seeing the following exception
>>>> >>>
>>>> >>> ERROR [c.c.a.ApiServer] (38419149@qtp-1917357200-11:ctx-116b8207
>>>> >>>ctx-9a54f764) unhandled exception executing api command:
>>>> >>>[Ljava.lang.String;@36d30b1d
>>>> >>> java.lang.NullPointerException
>>>> >>> at
>>>>
>>>>
>>>>>>>org.apache.cloudstack.engine.orchestration.CloudOrchestrator.createV
>>>>>>>irtu
>>>> >>>alMachine(CloudOrchestrator.java:191)
>>>> >>> at
>>>>
>>>>
>>>>>>>com.cloud.vm.UserVmManagerImpl$3.doInTransaction(UserVmManagerImpl.j
>>>>>>>ava:
>>>> >>>2908)
>>>> >>> at
>>>>
>>>>
>>>>>>>com.cloud.vm.UserVmManagerImpl$3.doInTransaction(UserVmManagerImpl.j
>>>>>>>ava:
>>>> >>>1)
>>>> >>> at
>>>>com.cloud.utils.db.Transaction.execute(Transaction.java:37)
>>>> >>> at
>>>>
>>>>
>>>>>>>com.cloud.vm.UserVmManagerImpl.commitUserVm(UserVmManagerImpl.java:2
>>>>>>>821)
>>>> >>> at
>>>>
>>>>
>>>>>>>com.cloud.vm.UserVmManagerImpl.createVirtualMachine(UserVmManagerImp
>>>>>>>l.ja
>>>> >>>va:2759)
>>>> >>> at
>>>>
>>>>
>>>>>>>com.cloud.vm.UserVmManagerImpl.createAdvancedVirtualMachine(UserVmMa
>>>>>>>nage
>>>> >>>rImpl.java:2465)
>>>> >>> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
>>>>Method)
>>>> >>> at
>>>>
>>>>
>>>>>>>sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl
>>>>>>>.jav
>>>> >>>a:57)
>>>> >>> at
>>>>
>>>>
>>>>>>>sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAcce
>>>>>>>ssor
>>>> >>>Impl.java:43)
>>>> >>> at java.lang.reflect.Method.invoke(Method.java:606)
>>>> >>>
>>>> >>>
>>>> >>> git blame shows the following:
>>>> >>>
>>>> >>> c427e8db
>>>>
>>>>
>>>>>>>engine/orchestration/src/org/apache/cloudstack/engine/orchestration/
>>>>>>>Clou
>>>> >>>dOrchestrator.java (Mike Tutkowski 2014-03-06 14:00:56 -0700
>>>> 191)
>>>> >>> if (offering.isCustomizedIops()) {
>>>> >>>
>>>> >>> And in the disk_offering table in the db the values for this
>>>>property
>>>> >>>is set to null for all entries.
>>>> >>>
>>>> >>> What is the correct way to use this property? Should the code be
>>>> fixed
>>>> >>>or the db entries need to be correctly updated?
>>>> >>>
>>>> >>> -Koushik
>>>> >>>
>>>> >
>>>>
>>>>
>>>
>>>
>>> --
>>> *Mike Tutkowski*
>>> *Senior CloudStack Developer, SolidFire Inc.*
>>> e: mike.tutkow...@solidfire.com
>>> o: 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
>> o: 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
>o: 303.746.7302
>Advancing the way the world uses the
>cloud<http://solidfire.com/solution/overview/?video=play>
>*(tm)*
