Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

[Marcus Sorensen <sh...@gmail.com>]

Anything special required to do this POST? Just read this thread when
searching for "userdata 2k" and noticed that POST is supposed to be
supported, but if I run the following it doesn't work:

 curl --request POST 'http://localhost:8096/client/api' --data
'command=listVirtualMachines'


On Wed, Apr 24, 2013 at 5:30 PM, Min Chen <mi...@citrix.com> wrote:
>
>
>> On April 24, 2013, 11:29 p.m., Min Chen wrote:
>> > Ship It!
>
> Prasanna has fixed the marvin test and pushed to master.
> These are the rebased commits on master:
> 4e9e7937ed5e56cbd7f678b2ed9863a56af16635 Fix Vijay's test to use the marvin integratin libararies
> b0caae6b33347f36721f28dca1fa12e7b18f1cd1 CLOUDSTACK-1086: DeployVirtualMachine userdata enhancements
>
>
> - Min
>
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10294/#review19680
> -----------------------------------------------------------
>
>
> On April 23, 2013, 9:45 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
>>
>> -----------------------------------------------------------
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/10294/
>> -----------------------------------------------------------
>>
>> (Updated April 23, 2013, 9:45 p.m.)
>>
>>
>> Review request for cloudstack, Chip Childers, Hugo Trippaers, Kelven Yang, and Min Chen.
>>
>>
>> Description
>> -------
>>
>> Please refer to https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeployVirtualMachine+userdata+enhancements for a background on the requirements driving this patch.
>>
>> This patch hasn't been extensively tested yet, and I will update this request with more info. I am uploading a first diff for initial review/comments.
>>
>>
>> This addresses bug CLOUDSTACK-1086.
>>
>>
>> Diffs
>> -----
>>
>>   api/src/com/cloud/vm/UserVmService.java aa21136
>>   api/src/org/apache/cloudstack/api/BaseCmd.java 42c0680
>>   api/src/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java 70c0159
>>   api/src/org/apache/cloudstack/api/command/user/vm/UpdateVMCmd.java ff8fff1
>>   core/src/com/cloud/vm/UserVmVO.java a16eaf9
>>   server/src/com/cloud/api/ApiDispatcher.java 925d90a
>>   server/src/com/cloud/api/ApiServer.java d842819
>>   server/src/com/cloud/api/ApiServerService.java 12d8b52
>>   server/src/com/cloud/api/ApiServlet.java 03bfb5f
>>   server/src/com/cloud/vm/UserVmManagerImpl.java 1843f60
>>   server/test/com/cloud/vm/MockUserVmManagerImpl.java 0d0a8f4
>>   server/test/com/cloud/vm/dao/UserVmDaoImplTest.java 0936180
>>   server/test/com/cloud/vm/dao/UserVmDaoTestConfiguration.java PRE-CREATION
>>   server/test/resources/UserVMDaoTestContext.xml PRE-CREATION
>>   setup/db/db/schema-410to420.sql 10cdbba
>>   test/integration/component/test_deploy_vm_with_userdata.py PRE-CREATION
>>
>> Diff: https://reviews.apache.org/r/10294/diff/
>>
>>
>> Testing
>> -------
>>
>> Manual testing done with scripts generating large/small userdata during creation of VMs, with both GET and POST requests confirmed to work correctly, on both integration port and the 8080 port.
>>
>> Basic Marvin test to create a VM with user data has been written. Since marvin doesn't yet support POSTable APIs, as soon as that is made available for create/update VM operations, marvin tests will be written for the same. Requesting that this be noted as an AI for the future.
>>
>>
>> Thanks,
>>
>> Venkata Siva Vijayendra Bhamidipati
>>
>>
>