Review Request 25471: CLOUDSTACK-7516: Fixed resource permission issue in test_snapshots.py, account was using template registered with other account

2014-09-09 Thread Gaurav Aradhye

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

Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-7516
https://issues.apache.org/jira/browse/CLOUDSTACK-7516


Repository: cloudstack-git


Description
---

Template was registered with admin api client, hence there was conflict in 
permissions available to the user account.
The test case failed while deploying template with the custom template.

Error:
Execute cmd: deployvirtualmachine failed, due to: errorCode: 531, 
errorText:Acct[51d00171-895e-4893-90c8-6630b98f852a-test-TestCreateVMSnapshotTemplate-BJ9XFN]
 does not have permission to operate with resource 
Acct[e7b7973c-3512-11e4-9ac6-1a6f7bb0d0a8-admin]

Changes:
Used api client of the user account for all the operations so that there is no 
conflict in permissions.


Diffs
-

  test/integration/component/test_snapshots.py 8a5c7ff 

Diff: https://reviews.apache.org/r/25471/diff/


Testing
---

Yes.

Ran two test cases separately.

Log:

[1]
Create Template from snapshot ... === TestName: test_07_template_from_snapshot 
| Status : SUCCESS ===
ok

--
Ran 1 test in 637.534s

OK


[2]
Test create VM, Snapshot and Template ... === TestName: 
test_01_createVM_snapshotTemplate | Status : SUCCESS ===
ok

--
Ran 1 test in 614.318s

OK


Thanks,

Gaurav Aradhye



Jenkins build is still unstable: simulator-singlerun #322

2014-09-09 Thread jenkins
See 



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko

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

(Updated Sept. 9, 2014, 7:58 a.m.)


Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng Yang.


Changes
---

Fixes


Bugs: CLOUDSTACK-7159
https://issues.apache.org/jira/browse/CLOUDSTACK-7159


Repository: cloudstack-git


Description
---

Working with Usage server and usage records very often I need to get only 
records for that particular usage ID. For example when filtering out 
network_bytes_received/sent with big amount of data it's not very fast to 
process hundreds of objects looking for the only one you need.
It would be useful to have an ability to filter out usage records only for 
specific resource ID.

This parch brings that to the API.


Diffs (updated)
-

  api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
  api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
21a7e4a 
  server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 

Diff: https://reviews.apache.org/r/25435/diff/


Testing
---

Tested cases:
- usageid is specified w/o "type": an exception is thrown (correct)
- provided usageid is not exists: an empty response is returned (since no 
records were found, correct)
- no usageid specified: work as is
- an existing usageid specified (with type, for example type=4 or type=5): only 
records for that usage type is returned


Thanks,

Ilia Shakitko



Jenkins build is still unstable: simulator-singlerun #323

2014-09-09 Thread jenkins
See 



[GitHub] cloudstack-docs-admin pull request: accounts: document SAML authen...

2014-09-09 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack-docs-admin/pull/19


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack-docs-admin pull request: Update templates.rst

2014-09-09 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack-docs-admin/pull/18


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav

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


Thanks for updating it.


api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java


Let's keep the coding convention for annotation consistent with others, 
keep 'em in same line.

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Coding+conventions


By using variable type to CommandType.STRING, we run the risk of blowing up 
inner layers if they are UUIDs. If there are other places where STRING is used 
instead of UUID, they must be fixed too. By using the commandtype UUID, the 
apidocs will get this information right and also clients using listApis such as 
cloudmonkey's sync and it sfuzzy parameter completer. You simply need to change 
the type to CommandType.UUID, the variable is still a String. You may read in 
detail why it should be UUID, 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring

- Rohit Yadav


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring

I tried. It throws an exception because it tries to map it with an Entity. But 
the entityType is not provided. I just tried to change CommandType.STRING => 
CommandType.UUID and it didn't work


- Ilia


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work

yes, in the parameter annotation can you try adding entityType = 
UsageTypeResponse.class and see if this works?


- Rohit


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Confluence edit permissions

2014-09-09 Thread Sanjay Tripathi
Can someone please grant me permissions to edit pages on Confluence?



My username is: sanjay.tripathi



--Sanjay



Re: Confluence edit permissions

2014-09-09 Thread Daan Hoogland
On Tue, Sep 9, 2014 at 11:37 AM, Sanjay Tripathi  wrote:

> sanjay.tripathi


​your username is sanjay.tripa...@citrix.com.
Do you nee another one?​



-- 
Daan


[ACS44][DISCUSS] 4.4.1 blockers

2014-09-09 Thread Daan Hoogland
H,

There are two blockers open for 4.4:
1. CLOUDSTACK7184: xen heartbeat is not observing interval and uses
interval as timeout. I overlooked that one and will pick it up.
2. CLOUDSTACK-7340: in kvm, using security groups, internet can not be
reached. (unassigned)

Can we discuss if these two are really blockers and if maybe other tickets
are blockers for 4.4.1?

​thanks,​

-- 
Daan


Re: [ACS44][DISCUSS] 4.4.1 blockers

2014-09-09 Thread Nux!
Daan,

I'll try to do some testing for the security groups issue. Which branch should 
be used? 4.4 master?

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


- Original Message -
> From: "Daan Hoogland" 
> To: "dev" 
> Sent: Tuesday, 9 September, 2014 11:18:44 AM
> Subject: [ACS44][DISCUSS] 4.4.1 blockers
> 
> H,
> 
> There are two blockers open for 4.4:
> 1. CLOUDSTACK7184: xen heartbeat is not observing interval and uses
> interval as timeout. I overlooked that one and will pick it up.
> 2. CLOUDSTACK-7340: in kvm, using security groups, internet can not be
> reached. (unassigned)
> 
> Can we discuss if these two are really blockers and if maybe other tickets
> are blockers for 4.4.1?
> 
> ​thanks,​
> 
> --
> Daan
> 


[CANCEL][VOTE] Release Apache CloudStack 4.3.1, round #2

2014-09-09 Thread sebgoa
This vote is cancelled to include a fix for 6624

On Sep 8, 2014, at 1:38 PM, Rajani Karuturi  wrote:

> +1 based on the minimal testing I did to launch a VM and the LDAP
> integration.
> 
> ~Rajani
> 
> On Fri, Sep 5, 2014 at 9:21 PM, sebgoa  wrote:
> 
>> Hi All,
>> 
>> I've created a 4.3.1 release, with the following artifacts up for a vote:
>> 
>> Git Branch and Commit SH:
>> 
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
>> Commit: 15092167296dd7d4d70e69b8ed3b3390ed40fc0e
>> 
>> List of changes:
>> 
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3
>> 
>> Source release (checksums and signatures are available at the same
>> location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/
>> 
>> PGP release keys (signed using 8F89EB84):
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>> 
>> Vote will be open for 72 hours.
>> 
>> For sanity in tallying the vote, can PMC members please be sure to
>> indicate "(binding)" with their vote?
>> 
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)



Re: [ACS431] Help testing 4.3.1 vote candidate

2014-09-09 Thread sebgoa

On Sep 8, 2014, at 10:35 PM, Sebastien Goasguen  wrote:

> This was mentioned off-list as a blocker:
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-6624
> 
> It was fixed by Rohit right away.
> 
> I am fine canceling the current vote (since there has not been many votes or 
> clear testers).
> 
> I will wait till tomorrow morning Europe time, if no-one objects to the 
> cancellation, I will cancel and re-launch a vote which will contain the 6624 
> fix.
> 
> 

No objections, so I will cancel the vote for 4.3.1 and start a third vote, this 
will include a fix for 6624

> -sebastien
> 
> On Sep 5, 2014, at 11:52 AM, sebgoa  wrote:
> 
>> I cancelled the first round and started a second round.
>> 
>> It brings in a debian/changelog change for building debs
>> plus a tiny bug fix for 7404 that arrived after the first round started:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=c1bf7eb3bd4dad384225d411e21859cce470
>> 
>> 
>> 
>> 
>> On Sep 5, 2014, at 2:37 PM, sebgoa  wrote:
>> 
>>> 
>>> On Sep 5, 2014, at 1:38 PM, Harikrishna Patnala 
>>>  wrote:
>>> 
 Yes the available 4.3 system vm templates on jenkins are post heart bleed.
 
 -Harikrishna
 
>>> 
>>> Any chance you can check why the 4.3 systemvm jobs are failing on 
>>> jenkins.buildacloud.org ?
>>> 
 
 On 05-Sep-2014, at 12:59 pm, Sebastien Goasguen 
 mailto:run...@gmail.com>> wrote:
 
 
 On Sep 4, 2014, at 5:25 PM, Rohit Yadav 
 mailto:rohit.ya...@shapeblue.com>> wrote:
 
 FYI, the deb/rpm repo I shared hosts noredist/nonoss builds.
 
 On 04-Sep-2014, at 11:14 pm, ilya musayev 
 mailto:ilya.mailing.li...@gmail.com>> wrote:
 Ack, will be trying later tonight.
 
 Thanks Ilya. Please also test if you are able to download volumes on 
 4.3.0/ESX as well.
 
 SystemVMs can be downloaded from: http://jenkins.buildacloud.org/view/4.3
 They are failing for some issue now, you may use 4.3.0 or the last 
 successful builds from the above link.
 
 @Sebastien, @Hugo: can we do something about the failing systemvm builds?
 
 
 we need to check if the ones on jenkins are even the right ones, since 4.3 
 was released before heart bleed…
 
 I don't recall the exact dates and don't really pay attention to system 
 discussions :(
 
 Regards,
 Rohit Yadav
 Software Architect, ShapeBlue
 M. +41 779015219 | 
 rohit.ya...@shapeblue.com
 Blog: bhaisaab.org | Twitter: @_bhaisaab
 
 Find out more about ShapeBlue and our range of CloudStack related services
 
 IaaS Cloud Design & 
 Build
 CSForge – rapid IaaS deployment framework
 CloudStack Consulting
 CloudStack Infrastructure 
 Support
 CloudStack Bootcamp Training 
 Courses
 
 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based 
 upon its contents, nor copy or show it to anyone. Please contact the 
 sender if you believe you have received this email in error. Shape Blue 
 Ltd is a company incorporated in England & Wales. ShapeBlue Services India 
 LLP is a company incorporated in India and is operated under license from 
 Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a company 
 incorporated in Brasil and is operated under license from Shape Blue Ltd. 
 ShapeBlue SA Pty Ltd is a company registered by The Republic of South 
 Africa and is traded under license from Shape Blue Ltd. ShapeBlue is a 
 registered trademark.
 
>>> 
>> 
> 



[VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread sebgoa
Hi All,

I've created a 4.3.1 release, with the following artifacts up for a vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
Commit: 33911723f851fe9a816c400bb4739539b8087cd3

List of changes:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3

Source release (checksums and signatures are available at the same location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/

PGP release keys (signed using 8F89EB84):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for 72 hours.

For sanity in tallying the vote, can PMC members please be sure to indicate 
"(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)

[TRAVIS] expunge vm test still failing

2014-09-09 Thread sebgoa
FYI: Keeping an eye on Travis it seems that test_09_expunge_vm is still failing:

https://travis-ci.org/apache/cloudstack/jobs/34786178

-sebastien

Review Request 25474: CLOUDSTACK-7519: Using bound/unbound methods instead of directly calling API methods from test case

2014-09-09 Thread Gaurav Aradhye

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

Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-7519
https://issues.apache.org/jira/browse/CLOUDSTACK-7519


Repository: cloudstack-git


Description
---

As suggested in RR: https://reviews.apache.org/r/25433/

test_templates.py file has many instances where direct APIs are called from the 
test case. Instead we have the methods in base library that can be used.


Diffs
-

  test/integration/smoke/test_templates.py b7fe49d 
  tools/marvin/marvin/lib/base.py 2a763ce 

Diff: https://reviews.apache.org/r/25474/diff/


Testing
---

Yes.

Tested on KVM setup.

Log:
Test create public & private template ... === TestName: test_01_create_template 
| Status : SUCCESS ===
ok
Test Edit template ... === TestName: test_02_edit_template | Status : SUCCESS 
===
ok
Test delete template ... === TestName: test_03_delete_template | Status : 
SUCCESS ===
ok
Test for extract template ... === TestName: test_04_extract_template | Status : 
SUCCESS ===
ok
Update & Test for template permissions ... === TestName: 
test_05_template_permissions | Status : SUCCESS ===
ok
Test for copy template from one zone to another ... === TestName: 
test_06_copy_template | Status : SUCCESS ===
ok
Test only public templates are visible to normal user ... === TestName: 
test_07_list_public_templates | Status : SUCCESS ===
ok
Test System templates are not visible to normal user ... === TestName: 
test_08_list_system_templates | Status : SUCCESS ===
ok

--
Ran 8 tests in 1007.401s

OK


Thanks,

Gaurav Aradhye



Build failed in Jenkins: cloudstack-4.3-maven-build #584

2014-09-09 Thread jenkins
See 

Changes:

[runseb] Update CHANGES file with 6224

--
[...truncated 2431 lines...]
215/419 KB   
219/419 KB   
223/419 KB   
227/419 KB   
231/419 KB   
235/419 KB   
239/419 KB   
243/419 KB   
247/419 KB   
249/419 KB   
252/419 KB   
256/419 KB   
260/419 KB   
264/419 KB   
268/419 KB   
272/419 KB   
276/419 KB   
280/419 KB   
284/419 KB   
288/419 KB   
292/419 KB   
296/419 KB   
300/419 KB   
304/419 KB   
308/419 KB   
312/419 KB   
316/419 KB   
320/419 KB   
324/419 KB   
328/419 KB   
332/419 KB   
336/419 KB   
340/419 KB   
344/419 KB   
348/419 KB   
351/419 KB   
355/419 KB   
359/419 KB   
363/419 KB   
367/419 KB   
371/419 KB   
375/419 KB   
379/419 KB   
383/419 KB   
387/419 KB   
391/419 KB   
395/419 KB   
399/419 KB   
403/419 KB   
407/419 KB   
411/419 KB   
415/419 KB   
419/419 KB   
419/419 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/httpcomponents/httpclient/4.2.2/httpclient-4.2.2.jar
 (419 KB at 4498.0 KB/sec)
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-hypervisor-xen ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 22 source files to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[108,33]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[171,39]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[308,35]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[177,34]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[236,34]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,com.xensource.xenapi.VDI,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[347,43]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[465,27]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[581,37]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,,,com.xensource.xenapi.VDI)
location: class com.xensource.xenapi.VDI
[ERROR] 


Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?

I can, but did you try that way? entityType should be the XxxxResponce.class 
where it will search for an UUID<->ID transformation. Logically speaking 
UsageTypeResponse shouldn't be there, what will it use for? The parameter is 
UsageId and we are forcing CS to consider it as UsageType...


- Ilia


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...

I did not check if UsageTypeResponse had an ID field for doing the 
transformation; You're right, so you may add a new response class or modify the 
usage type response class so that UUID<-> ID transformation will happen fine. 
We should avoid shortcuts if possible.


- Rohit


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...
> 
> Rohit Yadav wrote:
> I did not check if UsageTypeResponse had an ID field for doing the 
> transformation; You're right, so you may add a new response class or modify 
> the usage type response class so that UUID<-> ID transformation will happen 
> fine. We should avoid shortcuts if possible.

If possible. It will be quite expensive and complex to build a map in new 
response class. Usage ID can be related to many of resources, and usage type 
won't be available in that new created Response.class, so how do you suggest 
map the UUID to one of the PROPER entities in the Response.class? In the 
UsageService implementation I can understand, I can pick the usageType (from 
the command) and operate it.


- Ilia


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...
> 
> Rohit Yadav wrote:
> I did not check if UsageTypeResponse had an ID field for doing the 
> transformation; You're right, so you may add a new response class or modify 
> the usage type response class so that UUID<-> ID transformation will happen 
> fine. We should avoid shortcuts if possible.
> 
> Ilia Shakitko wrote:
> If possible. It will be quite expensive and complex to build a map in new 
> response class. Usage ID can be related to many of resources, and usage type 
> won't be available in that new created Response.class, so how do you suggest 
> map the UUID to one of the PROPER entities in the Response.class? In the 
> UsageService implementation I can understand, I can pick the usageType (from 
> the command) and operate it.

Alright, how about you fix the syntax on @Parameter and I'll take it further 
and fix the ID<->UUID thing afterwards.


- Rohit


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


On Sept. 9, 2014, 7:58 a.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 7:58 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Rohit Yadav
+1

Tested to work with Xen and KVM with basic networking and SG.

I’ve created new debs/rpms, you may use this repository for testing this vote 
candidate:
http://packages.bhaisaab.org/cloudstack/main

On 09-Sep-2014, at 2:12 pm, sebgoa  wrote:
> Hi All,
>
> I've created a 4.3.1 release, with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
> Commit: 33911723f851fe9a816c400bb4739539b8087cd3
>
> List of changes:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3
>
> Source release (checksums and signatures are available at the same location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/
>
> PGP release keys (signed using 8F89EB84):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to indicate 
> "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread sebgoa

On Sep 9, 2014, at 3:35 PM, Rohit Yadav  wrote:

> +1
> 
> Tested to work with Xen and KVM with basic networking and SG.
> 
> I’ve created new debs/rpms, you may use this repository for testing this vote 
> candidate:
> http://packages.bhaisaab.org/cloudstack/main

kind of, all testing and voting needs to be based on the source release. but 
that's handy thanks.

> 
> On 09-Sep-2014, at 2:12 pm, sebgoa  wrote:
>> Hi All,
>> 
>> I've created a 4.3.1 release, with the following artifacts up for a vote:
>> 
>> Git Branch and Commit SH:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
>> Commit: 33911723f851fe9a816c400bb4739539b8087cd3
>> 
>> List of changes:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3
>> 
>> Source release (checksums and signatures are available at the same location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/
>> 
>> PGP release keys (signed using 8F89EB84):
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>> 
>> Vote will be open for 72 hours.
>> 
>> For sanity in tallying the vote, can PMC members please be sure to indicate 
>> "(binding)" with their vote?
>> 
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko

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

(Updated Sept. 9, 2014, 1:44 p.m.)


Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng Yang.


Bugs: CLOUDSTACK-7159
https://issues.apache.org/jira/browse/CLOUDSTACK-7159


Repository: cloudstack-git


Description
---

Working with Usage server and usage records very often I need to get only 
records for that particular usage ID. For example when filtering out 
network_bytes_received/sent with big amount of data it's not very fast to 
process hundreds of objects looking for the only one you need.
It would be useful to have an ability to filter out usage records only for 
specific resource ID.

This parch brings that to the API.


Diffs (updated)
-

  api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
  api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
21a7e4a 
  server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 

Diff: https://reviews.apache.org/r/25435/diff/


Testing
---

Tested cases:
- usageid is specified w/o "type": an exception is thrown (correct)
- provided usageid is not exists: an empty response is returned (since no 
records were found, correct)
- no usageid specified: work as is
- an existing usageid specified (with type, for example type=4 or type=5): only 
records for that usage type is returned


Thanks,

Ilia Shakitko



Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

2014-09-09 Thread Mihaela Stoica

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

Review request for cloudstack, Brian Federle and Jessica Wang.


Repository: cloudstack-git


Description
---

- Preserve the advanced search parameters, so that when the advanced search box 
is shown again,
it is populated with the values selected/entered previously, unless they have 
navigated away from the search results page,
or applied any additional filters/search parameters.


Diffs
-

  ui/scripts/ui/widgets/listView.js 22182c1 

Diff: https://reviews.apache.org/r/25476/diff/


Testing
---


Thanks,

Mihaela Stoica



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...
> 
> Rohit Yadav wrote:
> I did not check if UsageTypeResponse had an ID field for doing the 
> transformation; You're right, so you may add a new response class or modify 
> the usage type response class so that UUID<-> ID transformation will happen 
> fine. We should avoid shortcuts if possible.
> 
> Ilia Shakitko wrote:
> If possible. It will be quite expensive and complex to build a map in new 
> response class. Usage ID can be related to many of resources, and usage type 
> won't be available in that new created Response.class, so how do you suggest 
> map the UUID to one of the PROPER entities in the Response.class? In the 
> UsageService implementation I can understand, I can pick the usageType (from 
> the command) and operate it.
> 
> Rohit Yadav wrote:
> Alright, how about you fix the syntax on @Parameter and I'll take it 
> further and fix the ID<->UUID thing afterwards.
> 
> Ilia Shakitko wrote:
> Collaboration is the everything we have! Could you include me in review 
> or just notify in the JIRA ticket then? I'd like to participate in further 
> refactoring. Maybe I'll do that for the other places where we have UUID 
> accepted in comand as string.

Yes. At present I understand changing it UUID would involve changes at some 
places and since it's is not consistent everywhere so this should be done as 
part of refactoring so but at the same time I want to include your patch on 
master soon. Why don't you open a JIRA issue to fix the UUID stuff here and 
make it consistent everywhere else. Tag me to that ticket as well so I can 
collaborate with you.


- Rohit


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


On Sept. 9, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: [ACS44][DISCUSS] 4.4.1 blockers

2014-09-09 Thread Daan Hoogland
head of 4.4 (4.4.1-SNAPSHOT)


On Tue, Sep 9, 2014 at 1:27 PM, Nux!  wrote:

> Daan,
>
> I'll try to do some testing for the security groups issue. Which branch
> should be used? 4.4 master?
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
>
> - Original Message -
> > From: "Daan Hoogland" 
> > To: "dev" 
> > Sent: Tuesday, 9 September, 2014 11:18:44 AM
> > Subject: [ACS44][DISCUSS] 4.4.1 blockers
> >
> > H,
> >
> > There are two blockers open for 4.4:
> > 1. CLOUDSTACK7184: xen heartbeat is not observing interval and uses
> > interval as timeout. I overlooked that one and will pick it up.
> > 2. CLOUDSTACK-7340: in kvm, using security groups, internet can not be
> > reached. (unassigned)
> >
> > Can we discuss if these two are really blockers and if maybe other
> tickets
> > are blockers for 4.4.1?
> >
> > ​thanks,​
> >
> > --
> > Daan
> >
>



-- 
Daan


Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...
> 
> Rohit Yadav wrote:
> I did not check if UsageTypeResponse had an ID field for doing the 
> transformation; You're right, so you may add a new response class or modify 
> the usage type response class so that UUID<-> ID transformation will happen 
> fine. We should avoid shortcuts if possible.
> 
> Ilia Shakitko wrote:
> If possible. It will be quite expensive and complex to build a map in new 
> response class. Usage ID can be related to many of resources, and usage type 
> won't be available in that new created Response.class, so how do you suggest 
> map the UUID to one of the PROPER entities in the Response.class? In the 
> UsageService implementation I can understand, I can pick the usageType (from 
> the command) and operate it.
> 
> Rohit Yadav wrote:
> Alright, how about you fix the syntax on @Parameter and I'll take it 
> further and fix the ID<->UUID thing afterwards.
> 
> Ilia Shakitko wrote:
> Collaboration is the everything we have! Could you include me in review 
> or just notify in the JIRA ticket then? I'd like to participate in further 
> refactoring. Maybe I'll do that for the other places where we have UUID 
> accepted in comand as string.
> 
> Rohit Yadav wrote:
> Yes. At present I understand changing it UUID would involve changes at 
> some places and since it's is not consistent everywhere so this should be 
> done as part of refactoring so but at the same time I want to include your 
> patch on master soon. Why don't you open a JIRA issue to fix the UUID stuff 
> here and make it consistent everywhere else. Tag me to that ticket as well so 
> I can collaborate with you.

I will, good idea!


- Ilia


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


On Sept. 9, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Sha

Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko


On Sept. 9, 2014, 8:55 a.m., Ilia Shakitko wrote:
> > By using variable type to CommandType.STRING, we run the risk of blowing up 
> > inner layers if they are UUIDs. If there are other places where STRING is 
> > used instead of UUID, they must be fixed too. By using the commandtype 
> > UUID, the apidocs will get this information right and also clients using 
> > listApis such as cloudmonkey's sync and it sfuzzy parameter completer. You 
> > simply need to change the type to CommandType.UUID, the variable is still a 
> > String. You may read in detail why it should be UUID, 
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring
> 
> Ilia Shakitko wrote:
> I tried. It throws an exception because it tries to map it with an 
> Entity. But the entityType is not provided. I just tried to change 
> CommandType.STRING => CommandType.UUID and it didn't work
> 
> Rohit Yadav wrote:
> yes, in the parameter annotation can you try adding entityType = 
> UsageTypeResponse.class and see if this works?
> 
> Ilia Shakitko wrote:
> I can, but did you try that way? entityType should be the 
> XxxxResponce.class where it will search for an UUID<->ID transformation. 
> Logically speaking UsageTypeResponse shouldn't be there, what will it use 
> for? The parameter is UsageId and we are forcing CS to consider it as 
> UsageType...
> 
> Rohit Yadav wrote:
> I did not check if UsageTypeResponse had an ID field for doing the 
> transformation; You're right, so you may add a new response class or modify 
> the usage type response class so that UUID<-> ID transformation will happen 
> fine. We should avoid shortcuts if possible.
> 
> Ilia Shakitko wrote:
> If possible. It will be quite expensive and complex to build a map in new 
> response class. Usage ID can be related to many of resources, and usage type 
> won't be available in that new created Response.class, so how do you suggest 
> map the UUID to one of the PROPER entities in the Response.class? In the 
> UsageService implementation I can understand, I can pick the usageType (from 
> the command) and operate it.
> 
> Rohit Yadav wrote:
> Alright, how about you fix the syntax on @Parameter and I'll take it 
> further and fix the ID<->UUID thing afterwards.

Collaboration is the everything we have! Could you include me in review or just 
notify in the JIRA ticket then? I'd like to participate in further refactoring. 
Maybe I'll do that for the other places where we have UUID accepted in comand 
as string.


- Ilia


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


On Sept. 9, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: S3/Swift Problem around Virtual Size

2014-09-09 Thread Punith S
yes mike,

w.r.t to option 1:
it will be like creating a VM w.r.t ISO, where admin will have to specify
the os disk(ROOT) disk size.

for option 2:
i have figured out the issue, post downloading the template to S3,
cloudstack will again download the template from S3 to staging nfs store.
here we need to access the file and process it with VHD processor in order
to calculate the virtualsize but we are skipping this process,
hence the virtual size is not being calculated while using the S3 or swift.

the templates already present in the staging nfs storage cannot be applied
to this process.

for option 3:
it's convenient to calculate the template virtual size while it is being
copied from s3 to staged nfs store instead of staged nfs to primary,
since admin might be using more than one primary stores.

i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.

thanks!

On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> By the way, for anyone new to this issue, this is what we're referring to
> here:
>
> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>
> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> Great :) Then a question might be, "Is it too late in the game to
>> interrogate the template to discover its virtual size if we're just about
>> to copy the template to primary storage?"
>>
>> If it's not, this might be the place to run the logic to figure out the
>> virtual size.
>>
>> Really, there are three big possibilities:
>>
>> 1) Just ask the end user to provide the virtual size (not commenting here
>> on what happens for already-uploaded templates)
>>
>> or
>>
>> 2) Figure out the virtual size when the template is copied from object
>> storage to secondary storage and update the DB with this info (not sure
>> what happens if the template has already been copied to (secondary-storage)
>> NFS because it was used before)
>>
>> or
>>
>> 3) Figure out the virtual size when the template is about to be copied
>> from secondary storage to primary storage
>>
>> On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
>> sanjeev.neelar...@citrix.com> wrote:
>>
>>> Mike,
>>>
>>> You are right. Template gets copied to (secondary-storage) NFS before
>>> being copied to primary storage
>>>
>>> -Sanjeev
>>>
>>> -Original Message-
>>> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>>> Sent: Tuesday, September 09, 2014 10:55 AM
>>> To: dev@cloudstack.apache.org
>>> Cc: Punith S; Francois Gaudreault
>>> Subject: Re: S3/Swift Problem around Virtual Size
>>>
>>> Hi Will,
>>>
>>> Thanks for the input!
>>>
>>> I like the idea of storing the virtual size as metadata in S3 or Swift
>>> although this could require that the end user provide this value when
>>> uploading the template.
>>>
>>> However, if we have the ability to determine the virtual size of the
>>> template after it gets downloaded to (secondary-storage) NFS and we're able
>>> to update the database with this info, then it would seem we would never
>>> need to ask the user for this value.
>>>
>>> Either way, the tricky part might be if the template in object storage
>>> has already been downloaded to (secondary-storage) NFS (say it was used
>>> before). In this case, we won't need to download it to (secondary-storage)
>>> NFS again (at least not in the same zone), so we won't have an easy
>>> opportunity to figure out the virtual size upon download from object
>>> storage.
>>>
>>> I wonder if it's too late in this process if we figured out the virtual
>>> size before the copied template (now on (secondary-storage) NFS) gets
>>> copied to primary storage. If we could do it at this point, then we
>>> wouldn't have to worry about fixing the "legacy" situation because it would
>>> just work out naturally. We would look in the DB to see if the virtual size
>>> for this template is known and, if not, we could figure out the virtual
>>> size before downloading from (secondary-storage) NFS to primary storage
>>> each time. (Although I'm thinking this would come too late in the process
>>> because we may have already asked the primary-storage plug-in to create the
>>> necessary volume.)
>>>
>>> By the way, I'm assuming that a template gets copied to
>>> (secondary-storage) NFS before being copied to primary storage. I'm not
>>> super familiar with how this process works.
>>>
>>> Talk to you later,
>>> Mike
>>>
>>> On Mon, Sep 8, 2014 at 10:59 PM, Will Stevens 
>>> wrote:
>>>
>>> > My two cents on the topic.
>>> >
>>> > Ideally we would save the size in the object store metadata and
>>> > retrieve it from the metadata if it is set.  If it is not set in the
>>> > object store metadata, then when it is fetched, we have to put it on
>>> > NFS and determine the size (then ideally save the metadata back to the
>>> > object store) and remove the NFS copy.
>>> >
>>> > This way the NFS copy approach is only ever done once and then the
>>> > data is populated (for backw

Looking for a contacts of Jessica (UI contribution)

2014-09-09 Thread Ilia Shakitko
How do I contact Jessica Wang?

I can see her commits in repo, but I can’t get an answer from her (used this 
email: jessicaw...@apache.org) or via mailing 
list for few weeks ☹

Kind regards,

Ilia Shakitko
Innovation Engineer
LeaseWeb Technologies B.V.

T: +31 20 316 0235

E: i.shaki...@tech.leaseweb.com
W: www.leaseweb.com

Luttenbergweg 8, 1101 EC Amsterdam, The Netherlands




Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav

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

Ship it!


Ship It!

- Rohit Yadav


On Sept. 9, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Rohit Yadav


> On Sept. 9, 2014, 1:55 p.m., Rohit Yadav wrote:
> > Ship It!

LGTM, we'll wait for usage server maintainer Kishan to comment by tomorrow, 
else I'll go ahead and merge it on master. Thanks for your contribution Ilia.


- Rohit


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


On Sept. 9, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7159
> https://issues.apache.org/jira/browse/CLOUDSTACK-7159
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>



Re: Looking for a contacts of Jessica (UI contribution)

2014-09-09 Thread Daan Hoogland
do you have a citrix address for her? That's her employer


On Tue, Sep 9, 2014 at 3:53 PM, Ilia Shakitko 
wrote:

> How do I contact Jessica Wang?
>
> I can see her commits in repo, but I can’t get an answer from her (used
> this email: jessicaw...@apache.org) or via
> mailing list for few weeks ☹
>
> Kind regards,
>
> Ilia Shakitko
> Innovation Engineer
> LeaseWeb Technologies B.V.
>
> T: +31 20 316 0235
>
> E: i.shaki...@tech.leaseweb.com
> W: www.leaseweb.com
>
> Luttenbergweg 8, 1101 EC Amsterdam, The Netherlands
>
>
>


-- 
Daan


Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

2014-09-09 Thread Ilia Shakitko

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

(Updated Sept. 9, 2014, 1:57 p.m.)


Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng Yang.


Bugs: CLOUDSTACK-7510
https://issues.apache.org/jira/browse/CLOUDSTACK-7510


Repository: cloudstack-git


Description
---

Working with Usage server and usage records very often I need to get only 
records for that particular usage ID. For example when filtering out 
network_bytes_received/sent with big amount of data it's not very fast to 
process hundreds of objects looking for the only one you need.
It would be useful to have an ability to filter out usage records only for 
specific resource ID.

This parch brings that to the API.


Diffs
-

  api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
  api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
21a7e4a 
  server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 

Diff: https://reviews.apache.org/r/25435/diff/


Testing
---

Tested cases:
- usageid is specified w/o "type": an exception is thrown (correct)
- provided usageid is not exists: an empty response is returned (since no 
records were found, correct)
- no usageid specified: work as is
- an existing usageid specified (with type, for example type=4 or type=5): only 
records for that usage type is returned


Thanks,

Ilia Shakitko



Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Rohit Yadav

On 09-Sep-2014, at 3:42 pm, sebgoa  wrote:
> kind of, all testing and voting needs to be based on the source release. but 
> that's handy thanks.

Just for reference, the deb/rpm repo I created was built out of commit: 
c3617f986788186d0a4f1cd98c6d65b48e9f894b which does not include Sebastien’s 
change to the CHANGES.md file only (which does not bring any code changes).

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

2014-09-09 Thread Rohit Yadav

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



ui/scripts/ui/widgets/listView.js


Should n't this be declared as var, also this is not used or referenced in 
the function definition.


- Rohit Yadav


On Sept. 9, 2014, 1:47 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25476/
> ---
> 
> (Updated Sept. 9, 2014, 1:47 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Preserve the advanced search parameters, so that when the advanced search 
> box is shown again,
> it is populated with the values selected/entered previously, unless they have 
> navigated away from the search results page,
> or applied any additional filters/search parameters.
> 
> 
> Diffs
> -
> 
>   ui/scripts/ui/widgets/listView.js 22182c1 
> 
> Diff: https://reviews.apache.org/r/25476/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>



RE: Looking for a contacts of Jessica (UI contribution)

2014-09-09 Thread Stephen Turner
It's jessica.w...@citrix.com.

-- 
Stephen Turner


-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] 
Sent: 09 September 2014 14:57
To: dev
Subject: Re: Looking for a contacts of Jessica (UI contribution)

do you have a citrix address for her? That's her employer


On Tue, Sep 9, 2014 at 3:53 PM, Ilia Shakitko 
wrote:

> How do I contact Jessica Wang?
>
> I can see her commits in repo, but I can’t get an answer from her 
> (used this email: 
> jessicaw...@apache.org) or via mailing 
> list for few weeks ☹
>
> Kind regards,
>
> Ilia Shakitko
> Innovation Engineer
> LeaseWeb Technologies B.V.
>
> T: +31 20 316 0235
>
> E: i.shaki...@tech.leaseweb.com
> W: www.leaseweb.com
>
> Luttenbergweg 8, 1101 EC Amsterdam, The Netherlands
>
>
>


--
Daan


Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Nux!
-1, source doesnt build the rpms for me

INFO] Total time: 3:45.324s
[INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
[INFO] Final Memory: 40M/243M
[INFO] 
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile (default-compile) 
on project cloud-plugin-hypervisor-xen: Compilation failure: Compilation 
failure:
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,,)
[ERROR] location: variable srcVdi of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
[ERROR] location: variable snapshotvdi of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,,)
[ERROR] location: variable snapshotVdi of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,,)
[ERROR] location: variable vol of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,,,VDI)
[ERROR] location: variable snapChain of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,,)
[ERROR] location: variable srcVdi of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[696,34]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,SR,,)
[ERROR] location: variable srcVdi of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[780,37]
 error: cannot find symbol
[ERROR] symbol:   method copyAsync2(Connection,,,VDI)
[ERROR] location: variable snapChain of type VDI
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[108,33]
 error: cannot find symbol
[ERROR] symbol:   method properFrom(Connection,Set,String,Double)
[ERROR] location: class Event
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[171,39]
 error: cannot find symbol
[ERROR] symbol:   method properFrom(Connection,Set,String,Double)
[ERROR] location: class Event
[ERROR] 
/root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[308,35]
 error: cannot find symbol
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please 
read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :cloud-plugin-hypervisor-xen
error: Bad exit status from /var/tmp/rpm-tmp.l9Lnmb (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.l9Lnmb (%build)


--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


- Original Message -
> From: "Rohit Yadav" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 9 September, 2014 2:35:42 PM
> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
> 
> +1
> 
> Tested to work with Xen and KVM with basic networking and SG.
> 
> I’ve created new debs/rpms, you may use this repository for testing this vote
> candidate:
> http://packages.bhaisaab.org/cloudstack/main
> 
> On 09-Sep-2014, at 2:12 pm, sebgoa  wrote:
> > Hi All,
> >
> > I've 

RE: Looking for a contacts of Jessica (UI contribution)

2014-09-09 Thread Ilia Shakitko
Stephen, Daan - thanks!


Kind regards,

Ilia Shakitko
Innovation Engineer
LeaseWeb Technologies B.V.

T: +31 20 316 0235
M:
E: i.shaki...@tech.leaseweb.com
W: http://www.leaseweb.com

Luttenbergweg 8, 1101 EC Amsterdam, The Netherlands


-Original Message-
From: Stephen Turner [mailto: stephen.tur...@citrix.com]
Sent: dinsdag, 09 september, 2014 16:06
To: dev@cloudstack.apache.org
Subject: RE: Looking for a contacts of Jessica (UI contribution)

It's jessica.w...@citrix.com.

--
Stephen Turner


-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: 09 September 2014 14:57
To: dev
Subject: Re: Looking for a contacts of Jessica (UI contribution)

do you have a citrix address for her? That's her employer


On Tue, Sep 9, 2014 at 3:53 PM, Ilia Shakitko 
wrote:

> How do I contact Jessica Wang?
>
> I can see her commits in repo, but I can’t get an answer from her
> (used this email:
> jessicaw...@apache.org) or via mailing
> list for few weeks ☹
>
> Kind regards,
>
> Ilia Shakitko
> Innovation Engineer
> LeaseWeb Technologies B.V.
>
> T: +31 20 316 0235
>
> E: i.shaki...@tech.leaseweb.com
> W: www.leaseweb.com
>
> Luttenbergweg 8, 1101 EC Amsterdam, The Netherlands
>
>
>


--
Daan


Build failed in Jenkins: cloudstack-4.3-maven-build #585

2014-09-09 Thread jenkins
See 

--
[...truncated 2035 lines...]
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-hypervisor-ovm ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - Open vSwitch 4.3.1
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-network-ovs ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-network-ovs ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-network-ovs ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-network-ovs ---
[INFO] Compiling 23 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-network-ovs ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-network-ovs ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-network-ovs ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - Hypervisor Xen 4.3.1
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-hypervisor-xen ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 22 source files to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[108,33]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[171,39]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Rohit Yadav
Hi Lucian, looks like you’re hitting mvn/cache issues. I see this issue on our 
jenkins.buildacloud.org as well.

Can you remove files from ~/.m2/repository/net/java/dev/vcc/thirdparty/xen-api 
and try building it again. You may even remove you whole ~/.m2 and try 
building. If you hit it again, can you share the full log with us for debuggin?

I’m able to build 4.3.1 from source. Here’s my build log: 
http://people.apache.org/~bhaisaab/4.3.1-rpmbuild.txt

I’ve put built rpms from above here: 
http://packages.bhaisaab.org/cloudstack/main/centos/


On 09-Sep-2014, at 4:12 pm, Nux!  wrote:

> -1, source doesnt build the rpms for me
>
> INFO] Total time: 3:45.324s
> [INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
> [INFO] Final Memory: 40M/243M
> [INFO] 
> 
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile 
> (default-compile) on project cloud-plugin-hypervisor-xen: Compilation 
> failure: Compilation failure:
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> [ERROR] location: variable srcVdi of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
> [ERROR] location: variable snapshotvdi of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> [ERROR] location: variable snapshotVdi of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> [ERROR] location: variable vol of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> [ERROR] location: variable snapChain of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> [ERROR] location: variable srcVdi of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[696,34]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> [ERROR] location: variable srcVdi of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[780,37]
>  error: cannot find symbol
> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> [ERROR] location: variable snapChain of type VDI
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[108,33]
>  error: cannot find symbol
> [ERROR] symbol:   method properFrom(Connection,Set,String,Double)
> [ERROR] location: class Event
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[171,39]
>  error: cannot find symbol
> [ERROR] symbol:   method properFrom(Connection,Set,String,Double)
> [ERROR] location: class Event
> [ERROR] 
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[308,35]
>  error: cannot find symbol
> [ERROR] -> [Help 1]
> [ERROR]
> [ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
> switch.
> [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> [ERROR]
> [ERROR] For more information about the errors and possible solutions, please 
> read the following articles:
> [ERROR] [Help 1] 
> http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
> [ERROR]
> [ERROR] After correcting the problems, you can resume the build with the 
> command

Re: Cloudstack/KVM/arch i686 case reopened

2014-09-09 Thread Laurent Steff
Hi Rohit,

Thanks for your answer !

I'm missing real use cases of 32 bit emulation, only knowing that some
researchers are working on low-level instructions on various CPU architectures.

And seeing CPU features (in our case "lm" but I've also heard of "sse*") 
that could not be implemented fools their low-level code.

For now we seem to have only a ugly workaround (updating Cloudstack database) 
and we're
trying to learn how to code in Cloudstack to propose patches.

We will certainly add a feature request too, enabling on UI/API
an "advanced choice for advanced users" for instances on libvirt/KVM clusters :
- cpu arch
- cpu models (AFAIK only available for now as guest.cpu.model
on Cloudstack KVM agent configuration)
- cpu features

Best Regards,

- Mail original -
> De: "Rohit Yadav" 
> À: dev@cloudstack.apache.org
> Envoyé: Lundi 8 Septembre 2014 18:21:34
> Objet: Re: Cloudstack/KVM/arch i686 case reopened
> 
> Hi Laurent,
> 
> You’re welcome to contribute any work that you’ve already done. Else, you
> could have any developers at INRIA or may sponsor developers to work on this
> issue and the ACS community will be happy to assist them during development,
> help with patches, reviews etc.
> 
> I’m curious why your researchers would need “real” 32 bit CPUs support and
> what the use cases are.
> One can always run 32bit VMs on 64bit CPU/KVM. Will be happy to help.
> 
> On 08-Sep-2014, at 5:13 pm, Laurent Steff  wrote:
> > Hello,
> >
> > As a lot of our researchers still need "real" 32bits CPU support
> > in KVM/Qemu with CloudStack, we reopened and old bug reported by
> > a colleague almost one year ago, without any activity.
> >
> > https://issues.apache.org/jira/browse/CLOUDSTACK-7511
> >
> > Tell us if we could help (testing, patch, ...)
> >
> > Thanks in advance.
> >
> > Regards,
> > --
> > Laurent Steff
> >
> > DSI/SESI
> > INRIA
> > http://www.inria.fr/
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> 
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure
> Support
> CloudStack Bootcamp Training
> Courses
> 
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a
> company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> 

-- 
Laurent Steff

DSI/SESI
INRIA
Tél.  : +33 1 39 63 50 81
Port. : +33 6 87 66 77 85
http://www.inria.fr/


Re: Cloudstack/KVM/arch i686 case reopened

2014-09-09 Thread Marcus
Yes, I'd be curious what 32 bit instructions don't work on an x86_64 arch.
 As you mentioned, all other available features (sse, nested vmx, hardware
aes) that Qemu can provide are available via setting cpu model, which is
supported. Worst case you have to tweak the cpu model definitions in the
libvirt configs.


On Tue, Sep 9, 2014 at 9:11 AM, Laurent Steff 
wrote:

> Hi Rohit,
>
> Thanks for your answer !
>
> I'm missing real use cases of 32 bit emulation, only knowing that some
> researchers are working on low-level instructions on various CPU
> architectures.
>
> And seeing CPU features (in our case "lm" but I've also heard of "sse*")
> that could not be implemented fools their low-level code.
>
> For now we seem to have only a ugly workaround (updating Cloudstack
> database) and we're
> trying to learn how to code in Cloudstack to propose patches.
>
> We will certainly add a feature request too, enabling on UI/API
> an "advanced choice for advanced users" for instances on libvirt/KVM
> clusters :
> - cpu arch
> - cpu models (AFAIK only available for now as guest.cpu.model
> on Cloudstack KVM agent configuration)
> - cpu features
>
> Best Regards,
>
> - Mail original -
> > De: "Rohit Yadav" 
> > À: dev@cloudstack.apache.org
> > Envoyé: Lundi 8 Septembre 2014 18:21:34
> > Objet: Re: Cloudstack/KVM/arch i686 case reopened
> >
> > Hi Laurent,
> >
> > You’re welcome to contribute any work that you’ve already done. Else, you
> > could have any developers at INRIA or may sponsor developers to work on
> this
> > issue and the ACS community will be happy to assist them during
> development,
> > help with patches, reviews etc.
> >
> > I’m curious why your researchers would need “real” 32 bit CPUs support
> and
> > what the use cases are.
> > One can always run 32bit VMs on 64bit CPU/KVM. Will be happy to help.
> >
> > On 08-Sep-2014, at 5:13 pm, Laurent Steff 
> wrote:
> > > Hello,
> > >
> > > As a lot of our researchers still need "real" 32bits CPU support
> > > in KVM/Qemu with CloudStack, we reopened and old bug reported by
> > > a colleague almost one year ago, without any activity.
> > >
> > > https://issues.apache.org/jira/browse/CLOUDSTACK-7511
> > >
> > > Tell us if we could help (testing, patch, ...)
> > >
> > > Thanks in advance.
> > >
> > > Regards,
> > > --
> > > Laurent Steff
> > >
> > > DSI/SESI
> > > INRIA
> > > http://www.inria.fr/
> >
> > Regards,
> > Rohit Yadav
> > Software Architect, ShapeBlue
> > M. +41 779015219 | rohit.ya...@shapeblue.com
> > Blog: bhaisaab.org | Twitter: @_bhaisaab
> >
> >
> >
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack Infrastructure
> > Support
> > CloudStack Bootcamp Training
> > Courses
> >
> > This email and any attachments to it may be confidential and are intended
> > solely for the use of the individual to whom it is addressed. Any views
> or
> > opinions expressed are solely those of the author and do not necessarily
> > represent those of Shape Blue Ltd or related companies. If you are not
> the
> > intended recipient of this email, you must neither take any action based
> > upon its contents, nor copy or show it to anyone. Please contact the
> sender
> > if you believe you have received this email in error. Shape Blue Ltd is a
> > company incorporated in England & Wales. ShapeBlue Services India LLP is
> a
> > company incorporated in India and is operated under license from Shape
> Blue
> > Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> Brasil
> > and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd
> is a
> > company registered by The Republic of South Africa and is traded under
> > license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> >
>
> --
> Laurent Steff
>
> DSI/SESI
> INRIA
> Tél.  : +33 1 39 63 50 81
> Port. : +33 6 87 66 77 85
> http://www.inria.fr/
>


[DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Rohit Yadav
Hi,

Can anyone explain how xs-tools (xen server tools, guest utilities) are 
installed on systemvms and on user instances?

I see that we’ve a script in 
tools/appliance/definitions/builtin/install-xs-tools.sh that is supposed to 
download and install xs-tool on vms but we are not using this for our 
systemvmtemplate or systemvm64template definition.

If we need it, should we run this script (which downloads and installs xs 
tools) on the systemvms?

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.


Build failed in Jenkins: cloudstack-4.3-maven-build #586

2014-09-09 Thread jenkins
See 

--
Started by user Rohit Yadav
[EnvInject] - Loading node environment variables.
Building remotely on cloudstack-buildslave-centos6-246 
(cloudstack-buildslave-centos6) in workspace 

Fetching changes from the remote Git repository
Fetching upstream changes from git://git.apache.org/cloudstack.git
Checking out Revision 33911723f851fe9a816c400bb4739539b8087cd3 (origin/4.3)
[cloudstack-4.3-maven-build] $ /bin/sh -xe /tmp/hudson1892639830231483867.sh
+ rm -fr /jenkins/.m2/repository/net/java/dev/vcc/thirdparty
+ exit
FATAL: Couldn’t find any executable in /opt/apache-maven-3.0.5
Build step 'Invoke top-level Maven targets' marked build as failure


Build failed in Jenkins: cloudstack-4.3-maven-build #587

2014-09-09 Thread jenkins
See 

--
[...truncated 2054 lines...]
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-hypervisor-ovm ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - Open vSwitch 4.3.1
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-network-ovs ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-network-ovs ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-network-ovs ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-network-ovs ---
[INFO] Compiling 23 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-network-ovs ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-network-ovs ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-network-ovs ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - Hypervisor Xen 4.3.1
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-hypervisor-xen ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 22 source files to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[108,33]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[171,39]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,

Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Pierre-Luc Dion
Hi Rohit,

I've tried to investigate on this one recently because right now the
xentools are not installed on systemvm templates (acs 4.2+).
I've tried to build systemvm template for xenserver which this script and
tuned version for XS6.2 but not success during the build process.  I did
not investigate further but I've got into error related to running out of
disk space during the systemvm creation.

This was not for the systemvm.iso but to generate .vhd.  I'm curious to
investigate and need this fixed but did not had enough time yet...

>From what I've saw, you need to add install-xs-tool.sh into the script
lists in : tools/appliance/definitions/systemvm64template/postinstall.sh

You might have to update install-xs-tools.sh since it would install an old
xs-tools version...

Regards,



*Pierre-Luc DION*
Architecte de Solution Cloud | Cloud Solutions Architect
t 855.652.5683

*CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_


On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav 
wrote:

> Hi,
>
> Can anyone explain how xs-tools (xen server tools, guest utilities) are
> installed on systemvms and on user instances?
>
> I see that we’ve a script in
> tools/appliance/definitions/builtin/install-xs-tools.sh that is supposed to
> download and install xs-tool on vms but we are not using this for our
> systemvmtemplate or systemvm64template definition.
>
> If we need it, should we run this script (which downloads and installs xs
> tools) on the systemvms?
>
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
>
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
>
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
> a company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>


Build failed in Jenkins: cloudstack-4.3-maven-build #588

2014-09-09 Thread jenkins
See 

--
[...truncated 16530 lines...]
214/419 KB   
218/419 KB   
222/419 KB   
226/419 KB   
230/419 KB   
234/419 KB   
238/419 KB   
242/419 KB   
246/419 KB   
250/419 KB   
254/419 KB   
258/419 KB   
262/419 KB   
266/419 KB   
270/419 KB   
274/419 KB   
278/419 KB   
282/419 KB   
286/419 KB   
290/419 KB   
294/419 KB   
298/419 KB   
302/419 KB   
306/419 KB   
310/419 KB   
314/419 KB   
318/419 KB   
322/419 KB   
326/419 KB   
330/419 KB   
334/419 KB   
338/419 KB   
342/419 KB   
346/419 KB   
350/419 KB   
354/419 KB   
358/419 KB   
362/419 KB   
366/419 KB   
370/419 KB   
374/419 KB   
378/419 KB   
382/419 KB   
386/419 KB   
390/419 KB   
394/419 KB   
398/419 KB   
402/419 KB   
406/419 KB   
410/419 KB   
414/419 KB   
418/419 KB   
419/419 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/httpcomponents/httpclient/4.2.2/httpclient-4.2.2.jar
 (419 KB at 12303.3 KB/sec)
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-hypervisor-xen ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 22 source files to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[108,33]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[171,39]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[308,35]
 cannot find symbol
symbol  : method 
properFrom(com.xensource.xenapi.Connection,java.util.Set,java.lang.String,java.lang.Double)
location: class com.xensource.xenapi.Event
[ERROR] 
:[177,34]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[236,34]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,com.xensource.xenapi.VDI,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[347,43]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[465,27]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,com.xensource.xenapi.SR,,)
location: class com.xensource.xenapi.VDI
[ERROR] 
:[581,37]
 cannot find symbol
symbol  : method 
copyAsync2(com.xensource.xenapi.Connection,,,com.xensource.xenapi.VDI)
location: class com.xensource.xenapi.VDI
[ERROR] 


Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Rohit Yadav
Hi,

Thanks for replying Pierre.

On 09-Sep-2014, at 5:37 pm, Pierre-Luc Dion  wrote:
> Hi Rohit,
>
> I've tried to investigate on this one recently because right now the
> xentools are not installed on systemvm templates (acs 4.2+).
> I've tried to build systemvm template for xenserver which this script and
> tuned version for XS6.2 but not success during the build process.  I did
> not investigate further but I've got into error related to running out of
> disk space during the systemvm creation.
>
> This was not for the systemvm.iso but to generate .vhd.  I'm curious to
> investigate and need this fixed but did not had enough time yet...
>
> From what I've saw, you need to add install-xs-tool.sh into the script
> lists in : tools/appliance/definitions/systemvm64template/postinstall.sh
>
> You might have to update install-xs-tools.sh since it would install an old
> xs-tools version…

So, do we need to install xs-tools on systemvm templatest, what does it bring 
with it and what is the use of having it?

If we want it which newer version of xs-tools iso should we pre-install on 
systemvms? And is there a publicly hosted location of a newer iso?

> Regards,
>
>
>
> *Pierre-Luc DION*
> Architecte de Solution Cloud | Cloud Solutions Architect
> t 855.652.5683
>
> *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> w cloudops.com *|* tw @CloudOps_
>
>
> On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav 
> wrote:
>
>> Hi,
>>
>> Can anyone explain how xs-tools (xen server tools, guest utilities) are
>> installed on systemvms and on user instances?
>>
>> I see that we’ve a script in
>> tools/appliance/definitions/builtin/install-xs-tools.sh that is supposed to
>> download and install xs-tool on vms but we are not using this for our
>> systemvmtemplate or systemvm64template definition.
>>
>> If we need it, should we run this script (which downloads and installs xs
>> tools) on the systemvms?
>>
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +41 779015219 | rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>>
>> Find out more about ShapeBlue and our range of CloudStack related services
>>
>> IaaS Cloud Design & Build<
>> http://shapeblue.com/iaas-cloud-design-and-build//>
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Infrastructure Support<
>> http://shapeblue.com/cloudstack-infrastructure-support/>
>> CloudStack Bootcamp Training Courses<
>> http://shapeblue.com/cloudstack-training/>
>>
>> This email and any attachments to it may be confidential and are intended
>> solely for the use of the individual to whom it is addressed. Any views or
>> opinions expressed are solely those of the author and do not necessarily
>> represent those of Shape Blue Ltd or related companies. If you are not the
>> intended recipient of this email, you must neither take any action based
>> upon its contents, nor copy or show it to anyone. Please contact the sender
>> if you believe you have received this email in error. Shape Blue Ltd is a
>> company incorporated in England & Wales. ShapeBlue Services India LLP is a
>> company incorporated in India and is operated under license from Shape Blue
>> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
>> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
>> a company registered by The Republic of South Africa and is traded under
>> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>>

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated 

Re: S3/Swift Problem around Virtual Size

2014-09-09 Thread Mike Tutkowski
Thanks for the info, Punith!

Does that mean you are fixing the issue via Option2?

On Tue, Sep 9, 2014 at 7:52 AM, Punith S  wrote:

> yes mike,
>
> w.r.t to option 1:
> it will be like creating a VM w.r.t ISO, where admin will have to specify
> the os disk(ROOT) disk size.
>
> for option 2:
> i have figured out the issue, post downloading the template to S3,
> cloudstack will again download the template from S3 to staging nfs store.
> here we need to access the file and process it with VHD processor in order
> to calculate the virtualsize but we are skipping this process,
> hence the virtual size is not being calculated while using the S3 or swift.
>
> the templates already present in the staging nfs storage cannot be applied
> to this process.
>
> for option 3:
> it's convenient to calculate the template virtual size while it is being
> copied from s3 to staged nfs store instead of staged nfs to primary,
> since admin might be using more than one primary stores.
>
> i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.
>
> thanks!
>
> On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> By the way, for anyone new to this issue, this is what we're referring to
>> here:
>>
>> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>>
>> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> Great :) Then a question might be, "Is it too late in the game to
>>> interrogate the template to discover its virtual size if we're just about
>>> to copy the template to primary storage?"
>>>
>>> If it's not, this might be the place to run the logic to figure out the
>>> virtual size.
>>>
>>> Really, there are three big possibilities:
>>>
>>> 1) Just ask the end user to provide the virtual size (not commenting
>>> here on what happens for already-uploaded templates)
>>>
>>> or
>>>
>>> 2) Figure out the virtual size when the template is copied from object
>>> storage to secondary storage and update the DB with this info (not sure
>>> what happens if the template has already been copied to (secondary-storage)
>>> NFS because it was used before)
>>>
>>> or
>>>
>>> 3) Figure out the virtual size when the template is about to be copied
>>> from secondary storage to primary storage
>>>
>>> On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
>>> sanjeev.neelar...@citrix.com> wrote:
>>>
 Mike,

 You are right. Template gets copied to (secondary-storage) NFS before
 being copied to primary storage

 -Sanjeev

 -Original Message-
 From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
 Sent: Tuesday, September 09, 2014 10:55 AM
 To: dev@cloudstack.apache.org
 Cc: Punith S; Francois Gaudreault
 Subject: Re: S3/Swift Problem around Virtual Size

 Hi Will,

 Thanks for the input!

 I like the idea of storing the virtual size as metadata in S3 or Swift
 although this could require that the end user provide this value when
 uploading the template.

 However, if we have the ability to determine the virtual size of the
 template after it gets downloaded to (secondary-storage) NFS and we're able
 to update the database with this info, then it would seem we would never
 need to ask the user for this value.

 Either way, the tricky part might be if the template in object storage
 has already been downloaded to (secondary-storage) NFS (say it was used
 before). In this case, we won't need to download it to (secondary-storage)
 NFS again (at least not in the same zone), so we won't have an easy
 opportunity to figure out the virtual size upon download from object
 storage.

 I wonder if it's too late in this process if we figured out the virtual
 size before the copied template (now on (secondary-storage) NFS) gets
 copied to primary storage. If we could do it at this point, then we
 wouldn't have to worry about fixing the "legacy" situation because it would
 just work out naturally. We would look in the DB to see if the virtual size
 for this template is known and, if not, we could figure out the virtual
 size before downloading from (secondary-storage) NFS to primary storage
 each time. (Although I'm thinking this would come too late in the process
 because we may have already asked the primary-storage plug-in to create the
 necessary volume.)

 By the way, I'm assuming that a template gets copied to
 (secondary-storage) NFS before being copied to primary storage. I'm not
 super familiar with how this process works.

 Talk to you later,
 Mike

 On Mon, Sep 8, 2014 at 10:59 PM, Will Stevens 
 wrote:

 > My two cents on the topic.
 >
 > Ideally we would save the size in the object store metadata and
 > retrieve it from the metadata if it is set.  If it is not set in the
 > object store metadata, then

Re: S3/Swift Problem around Virtual Size

2014-09-09 Thread Francois Gaudreault
That's great Punith :)

Thanks for handling this one. I am not too worried about the option, as
long as it fixes SF integration for 4.5 :)

FG

On Tue, Sep 9, 2014 at 11:55 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Thanks for the info, Punith!
>
> Does that mean you are fixing the issue via Option2?
>
> On Tue, Sep 9, 2014 at 7:52 AM, Punith S  wrote:
>
>> yes mike,
>>
>> w.r.t to option 1:
>> it will be like creating a VM w.r.t ISO, where admin will have to specify
>> the os disk(ROOT) disk size.
>>
>> for option 2:
>> i have figured out the issue, post downloading the template to S3,
>> cloudstack will again download the template from S3 to staging nfs store.
>> here we need to access the file and process it with VHD processor in
>> order to calculate the virtualsize but we are skipping this process,
>> hence the virtual size is not being calculated while using the S3 or
>> swift.
>>
>> the templates already present in the staging nfs storage cannot be
>> applied to this process.
>>
>> for option 3:
>> it's convenient to calculate the template virtual size while it is being
>> copied from s3 to staged nfs store instead of staged nfs to primary,
>> since admin might be using more than one primary stores.
>>
>> i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.
>>
>> thanks!
>>
>> On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> By the way, for anyone new to this issue, this is what we're referring
>>> to here:
>>>
>>> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>>>
>>> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
>>>
 Great :) Then a question might be, "Is it too late in the game to
 interrogate the template to discover its virtual size if we're just about
 to copy the template to primary storage?"

 If it's not, this might be the place to run the logic to figure out the
 virtual size.

 Really, there are three big possibilities:

 1) Just ask the end user to provide the virtual size (not commenting
 here on what happens for already-uploaded templates)

 or

 2) Figure out the virtual size when the template is copied from object
 storage to secondary storage and update the DB with this info (not sure
 what happens if the template has already been copied to (secondary-storage)
 NFS because it was used before)

 or

 3) Figure out the virtual size when the template is about to be copied
 from secondary storage to primary storage

 On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
 sanjeev.neelar...@citrix.com> wrote:

> Mike,
>
> You are right. Template gets copied to (secondary-storage) NFS before
> being copied to primary storage
>
> -Sanjeev
>
> -Original Message-
> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
> Sent: Tuesday, September 09, 2014 10:55 AM
> To: dev@cloudstack.apache.org
> Cc: Punith S; Francois Gaudreault
> Subject: Re: S3/Swift Problem around Virtual Size
>
> Hi Will,
>
> Thanks for the input!
>
> I like the idea of storing the virtual size as metadata in S3 or Swift
> although this could require that the end user provide this value when
> uploading the template.
>
> However, if we have the ability to determine the virtual size of the
> template after it gets downloaded to (secondary-storage) NFS and we're 
> able
> to update the database with this info, then it would seem we would never
> need to ask the user for this value.
>
> Either way, the tricky part might be if the template in object storage
> has already been downloaded to (secondary-storage) NFS (say it was used
> before). In this case, we won't need to download it to (secondary-storage)
> NFS again (at least not in the same zone), so we won't have an easy
> opportunity to figure out the virtual size upon download from object
> storage.
>
> I wonder if it's too late in this process if we figured out the
> virtual size before the copied template (now on (secondary-storage) NFS)
> gets copied to primary storage. If we could do it at this point, then we
> wouldn't have to worry about fixing the "legacy" situation because it 
> would
> just work out naturally. We would look in the DB to see if the virtual 
> size
> for this template is known and, if not, we could figure out the virtual
> size before downloading from (secondary-storage) NFS to primary storage
> each time. (Although I'm thinking this would come too late in the process
> because we may have already asked the primary-storage plug-in to create 
> the
> necessary volume.)
>
> By the way, I'm assuming that a template gets copied to
> (secondary-storage) NFS before being copied to primary storage. I'm not
>>

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Nux!
Hi Rohit,

Thanks for the advice, but removing that did not help. Here's the log:
http://tmp.nux.ro/431xenfailflog.txt

I see jenkins job #583 is also failing for the same reason.

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


- Original Message -
> From: "Rohit Yadav" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 9 September, 2014 4:00:56 PM
> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
> 
> Hi Lucian, looks like you’re hitting mvn/cache issues. I see this issue on
> our jenkins.buildacloud.org as well.
> 
> Can you remove files from
> ~/.m2/repository/net/java/dev/vcc/thirdparty/xen-api and try building it
> again. You may even remove you whole ~/.m2 and try building. If you hit it
> again, can you share the full log with us for debuggin?
> 
> I’m able to build 4.3.1 from source. Here’s my build log:
> http://people.apache.org/~bhaisaab/4.3.1-rpmbuild.txt
> 
> I’ve put built rpms from above here:
> http://packages.bhaisaab.org/cloudstack/main/centos/
> 
> 
> On 09-Sep-2014, at 4:12 pm, Nux!  wrote:
> 
> > -1, source doesnt build the rpms for me
> >
> > INFO] Total time: 3:45.324s
> > [INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
> > [INFO] Final Memory: 40M/243M
> > [INFO]
> > 
> > [ERROR] Failed to execute goal
> > org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile
> > (default-compile) on project cloud-plugin-hypervisor-xen: Compilation
> > failure: Compilation failure:
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> > [ERROR] location: variable srcVdi of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
> > [ERROR] location: variable snapshotvdi of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> > [ERROR] location: variable snapshotVdi of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> > [ERROR] location: variable vol of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> > [ERROR] location: variable snapChain of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> > [ERROR] location: variable srcVdi of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[696,34]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> > [ERROR] location: variable srcVdi of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[780,37]
> > error: cannot find symbol
> > [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> > [ERROR] location: variable snapChain of type VDI
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[108,33]
> > error: cannot find symbol
> > [ERROR] symbol:   method properFrom(Connection,Set,String,Double)
> > [ERROR] location: class Event
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[171,39]
> > error: cannot find symbol
> > [ERROR] symbol:   method properFrom(Connection,Set,String,Double)
> > [ERROR] location: class Event
> > [ERROR]
> > /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hy

Re: S3/Swift Problem around Virtual Size

2014-09-09 Thread Mike Tutkowski
Yeah, either solution will fix this issue for managed storage in general
(ex. SolidFire, CloudByte).

On Tue, Sep 9, 2014 at 9:58 AM, Francois Gaudreault <
fgaudrea...@cloudops.com> wrote:

> That's great Punith :)
>
> Thanks for handling this one. I am not too worried about the option, as
> long as it fixes SF integration for 4.5 :)
>
> FG
>
> On Tue, Sep 9, 2014 at 11:55 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> Thanks for the info, Punith!
>>
>> Does that mean you are fixing the issue via Option2?
>>
>> On Tue, Sep 9, 2014 at 7:52 AM, Punith S  wrote:
>>
>>> yes mike,
>>>
>>> w.r.t to option 1:
>>> it will be like creating a VM w.r.t ISO, where admin will have to
>>> specify the os disk(ROOT) disk size.
>>>
>>> for option 2:
>>> i have figured out the issue, post downloading the template to S3,
>>> cloudstack will again download the template from S3 to staging nfs store.
>>> here we need to access the file and process it with VHD processor in
>>> order to calculate the virtualsize but we are skipping this process,
>>> hence the virtual size is not being calculated while using the S3 or
>>> swift.
>>>
>>> the templates already present in the staging nfs storage cannot be
>>> applied to this process.
>>>
>>> for option 3:
>>> it's convenient to calculate the template virtual size while it is being
>>> copied from s3 to staged nfs store instead of staged nfs to primary,
>>> since admin might be using more than one primary stores.
>>>
>>> i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.
>>>
>>> thanks!
>>>
>>> On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
>>>
 By the way, for anyone new to this issue, this is what we're referring
 to here:

 https://issues.apache.org/jira/browse/CLOUDSTACK-7406

 On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
 mike.tutkow...@solidfire.com> wrote:

> Great :) Then a question might be, "Is it too late in the game to
> interrogate the template to discover its virtual size if we're just about
> to copy the template to primary storage?"
>
> If it's not, this might be the place to run the logic to figure out
> the virtual size.
>
> Really, there are three big possibilities:
>
> 1) Just ask the end user to provide the virtual size (not commenting
> here on what happens for already-uploaded templates)
>
> or
>
> 2) Figure out the virtual size when the template is copied from object
> storage to secondary storage and update the DB with this info (not sure
> what happens if the template has already been copied to 
> (secondary-storage)
> NFS because it was used before)
>
> or
>
> 3) Figure out the virtual size when the template is about to be copied
> from secondary storage to primary storage
>
> On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
> sanjeev.neelar...@citrix.com> wrote:
>
>> Mike,
>>
>> You are right. Template gets copied to (secondary-storage) NFS before
>> being copied to primary storage
>>
>> -Sanjeev
>>
>> -Original Message-
>> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>> Sent: Tuesday, September 09, 2014 10:55 AM
>> To: dev@cloudstack.apache.org
>> Cc: Punith S; Francois Gaudreault
>> Subject: Re: S3/Swift Problem around Virtual Size
>>
>> Hi Will,
>>
>> Thanks for the input!
>>
>> I like the idea of storing the virtual size as metadata in S3 or
>> Swift although this could require that the end user provide this value 
>> when
>> uploading the template.
>>
>> However, if we have the ability to determine the virtual size of the
>> template after it gets downloaded to (secondary-storage) NFS and we're 
>> able
>> to update the database with this info, then it would seem we would never
>> need to ask the user for this value.
>>
>> Either way, the tricky part might be if the template in object
>> storage has already been downloaded to (secondary-storage) NFS (say it 
>> was
>> used before). In this case, we won't need to download it to
>> (secondary-storage) NFS again (at least not in the same zone), so we 
>> won't
>> have an easy opportunity to figure out the virtual size upon download 
>> from
>> object storage.
>>
>> I wonder if it's too late in this process if we figured out the
>> virtual size before the copied template (now on (secondary-storage) NFS)
>> gets copied to primary storage. If we could do it at this point, then we
>> wouldn't have to worry about fixing the "legacy" situation because it 
>> would
>> just work out naturally. We would look in the DB to see if the virtual 
>> size
>> for this template is known and, if not, we could figure out the virtual
>> size before downloading from (secondary-storage) NFS to

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread sebgoa
I don't know if that's the issue, but I mentioned something to Mike T.

In  deps/XenServerJava/pom.xml

the release build script removed a -SNAPSHOT in the xapi version

so

 xapi

 5.6.100-1

used to be 5.6.100-1-SNAPSHOT

Sadly I have no clue if this is normal or not…



On Sep 9, 2014, at 6:01 PM, Nux!  wrote:

> Hi Rohit,
> 
> Thanks for the advice, but removing that did not help. Here's the log:
> http://tmp.nux.ro/431xenfailflog.txt
> 
> I see jenkins job #583 is also failing for the same reason.
> 
> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> 
> - Original Message -
>> From: "Rohit Yadav" 
>> To: dev@cloudstack.apache.org
>> Sent: Tuesday, 9 September, 2014 4:00:56 PM
>> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
>> 
>> Hi Lucian, looks like you’re hitting mvn/cache issues. I see this issue on
>> our jenkins.buildacloud.org as well.
>> 
>> Can you remove files from
>> ~/.m2/repository/net/java/dev/vcc/thirdparty/xen-api and try building it
>> again. You may even remove you whole ~/.m2 and try building. If you hit it
>> again, can you share the full log with us for debuggin?
>> 
>> I’m able to build 4.3.1 from source. Here’s my build log:
>> http://people.apache.org/~bhaisaab/4.3.1-rpmbuild.txt
>> 
>> I’ve put built rpms from above here:
>> http://packages.bhaisaab.org/cloudstack/main/centos/
>> 
>> 
>> On 09-Sep-2014, at 4:12 pm, Nux!  wrote:
>> 
>>> -1, source doesnt build the rpms for me
>>> 
>>> INFO] Total time: 3:45.324s
>>> [INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
>>> [INFO] Final Memory: 40M/243M
>>> [INFO]
>>> 
>>> [ERROR] Failed to execute goal
>>> org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile
>>> (default-compile) on project cloud-plugin-hypervisor-xen: Compilation
>>> failure: Compilation failure:
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
>>> [ERROR] location: variable srcVdi of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
>>> [ERROR] location: variable snapshotvdi of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
>>> [ERROR] location: variable snapshotVdi of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
>>> [ERROR] location: variable vol of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
>>> [ERROR] location: variable snapChain of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
>>> [ERROR] location: variable srcVdi of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[696,34]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
>>> [ERROR] location: variable srcVdi of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[780,37]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
>>> [ERROR] location: variable snapChain of type VDI
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/org/apache/cloudstack/hypervisor/xenserver/XenServerResourceNewBase.java:[108,33]
>>> error: cannot find symbol
>>> [ERROR] symbol:   method properFrom(Connection,Set,String,Double)
>>> [ERROR] location: class Event
>>> [ERROR]
>>> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hy

Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Todd Pigram
Same issue with CCP4.3.1. I have to do the install manually.

On Tuesday, September 9, 2014, Rohit Yadav 
wrote:

> Hi,
>
> Thanks for replying Pierre.
>
> On 09-Sep-2014, at 5:37 pm, Pierre-Luc Dion  > wrote:
> > Hi Rohit,
> >
> > I've tried to investigate on this one recently because right now the
> > xentools are not installed on systemvm templates (acs 4.2+).
> > I've tried to build systemvm template for xenserver which this script and
> > tuned version for XS6.2 but not success during the build process.  I did
> > not investigate further but I've got into error related to running out of
> > disk space during the systemvm creation.
> >
> > This was not for the systemvm.iso but to generate .vhd.  I'm curious to
> > investigate and need this fixed but did not had enough time yet...
> >
> > From what I've saw, you need to add install-xs-tool.sh into the script
> > lists in : tools/appliance/definitions/systemvm64template/postinstall.sh
> >
> > You might have to update install-xs-tools.sh since it would install an
> old
> > xs-tools version…
>
> So, do we need to install xs-tools on systemvm templatest, what does it
> bring with it and what is the use of having it?
>
> If we want it which newer version of xs-tools iso should we pre-install on
> systemvms? And is there a publicly hosted location of a newer iso?
>
> > Regards,
> >
> >
> >
> > *Pierre-Luc DION*
> > Architecte de Solution Cloud | Cloud Solutions Architect
> > t 855.652.5683
> >
> > *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
> > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> > w cloudops.com *|* tw @CloudOps_
> >
> >
> > On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav  >
> > wrote:
> >
> >> Hi,
> >>
> >> Can anyone explain how xs-tools (xen server tools, guest utilities) are
> >> installed on systemvms and on user instances?
> >>
> >> I see that we’ve a script in
> >> tools/appliance/definitions/builtin/install-xs-tools.sh that is
> supposed to
> >> download and install xs-tool on vms but we are not using this for our
> >> systemvmtemplate or systemvm64template definition.
> >>
> >> If we need it, should we run this script (which downloads and installs
> xs
> >> tools) on the systemvms?
> >>
> >> Regards,
> >> Rohit Yadav
> >> Software Architect, ShapeBlue
> >> M. +41 779015219 | rohit.ya...@shapeblue.com 
> >> Blog: bhaisaab.org | Twitter: @_bhaisaab
> >>
> >> Find out more about ShapeBlue and our range of CloudStack related
> services
> >>
> >> IaaS Cloud Design & Build<
> >> http://shapeblue.com/iaas-cloud-design-and-build//>
> >> CSForge – rapid IaaS deployment framework >
> >> CloudStack Consulting
> >> CloudStack Infrastructure Support<
> >> http://shapeblue.com/cloudstack-infrastructure-support/>
> >> CloudStack Bootcamp Training Courses<
> >> http://shapeblue.com/cloudstack-training/>
> >>
> >> This email and any attachments to it may be confidential and are
> intended
> >> solely for the use of the individual to whom it is addressed. Any views
> or
> >> opinions expressed are solely those of the author and do not necessarily
> >> represent those of Shape Blue Ltd or related companies. If you are not
> the
> >> intended recipient of this email, you must neither take any action based
> >> upon its contents, nor copy or show it to anyone. Please contact the
> sender
> >> if you believe you have received this email in error. Shape Blue Ltd is
> a
> >> company incorporated in England & Wales. ShapeBlue Services India LLP
> is a
> >> company incorporated in India and is operated under license from Shape
> Blue
> >> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> Brasil
> >> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd
> is
> >> a company registered by The Republic of South Africa and is traded under
> >> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> >>
>
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com 
> Blog: bhaisaab.org | Twitter: @_bhaisaab
>
>
>
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
>
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its

Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

2014-09-09 Thread Mihaela Stoica


> On Sept. 9, 2014, 3:02 p.m., Rohit Yadav wrote:
> > ui/scripts/ui/widgets/listView.js, line 1857
> > 
> >
> > Should n't this be declared as var, also this is not used or referenced 
> > in the function definition.

This is where the saved advanced search parameters are cleared when a basic 
search is run.
The variable is declared in the scope of makeListView() function (ln 1934) and 
is referenced in three places:
- populated after the advanced search is run (ln 1960)
- read when the advanced search box is invocked (ln 1949)
- cleared when a basic search is run (ln 1857)


- Mihaela


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


On Sept. 9, 2014, 2:47 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25476/
> ---
> 
> (Updated Sept. 9, 2014, 2:47 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Preserve the advanced search parameters, so that when the advanced search 
> box is shown again,
> it is populated with the values selected/entered previously, unless they have 
> navigated away from the search results page,
> or applied any additional filters/search parameters.
> 
> 
> Diffs
> -
> 
>   ui/scripts/ui/widgets/listView.js 22182c1 
> 
> Diff: https://reviews.apache.org/r/25476/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>



Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

2014-09-09 Thread Ian Duffy
I might be completely wrong here.

I believe the xs-tools give us the PV drivers.

Since we're using Debian 7 which comes with Linux Kernel Version 3 which
has PV support we do not require the drivers.


Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

2014-09-09 Thread Ian Duffy
Argh wrong thread, ignore.

On 9 September 2014 17:27, Ian Duffy  wrote:

> I might be completely wrong here.
>
> I believe the xs-tools give us the PV drivers.
>
> Since we're using Debian 7 which comes with Linux Kernel Version 3 which
> has PV support we do not require the drivers.
>


Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Ian Duffy
I might be completely wrong here.

I believe the xs-tools give us the PV drivers.

Since we're using Debian 7 which comes with Linux Kernel Version 3 which
has PV support we do not require the drivers.

On 9 September 2014 17:15, Todd Pigram  wrote:

> Same issue with CCP4.3.1. I have to do the install manually.
>
> On Tuesday, September 9, 2014, Rohit Yadav 
> wrote:
>
> > Hi,
> >
> > Thanks for replying Pierre.
> >
> > On 09-Sep-2014, at 5:37 pm, Pierre-Luc Dion  > > wrote:
> > > Hi Rohit,
> > >
> > > I've tried to investigate on this one recently because right now the
> > > xentools are not installed on systemvm templates (acs 4.2+).
> > > I've tried to build systemvm template for xenserver which this script
> and
> > > tuned version for XS6.2 but not success during the build process.  I
> did
> > > not investigate further but I've got into error related to running out
> of
> > > disk space during the systemvm creation.
> > >
> > > This was not for the systemvm.iso but to generate .vhd.  I'm curious to
> > > investigate and need this fixed but did not had enough time yet...
> > >
> > > From what I've saw, you need to add install-xs-tool.sh into the script
> > > lists in :
> tools/appliance/definitions/systemvm64template/postinstall.sh
> > >
> > > You might have to update install-xs-tools.sh since it would install an
> > old
> > > xs-tools version…
> >
> > So, do we need to install xs-tools on systemvm templatest, what does it
> > bring with it and what is the use of having it?
> >
> > If we want it which newer version of xs-tools iso should we pre-install
> on
> > systemvms? And is there a publicly hosted location of a newer iso?
> >
> > > Regards,
> > >
> > >
> > >
> > > *Pierre-Luc DION*
> > > Architecte de Solution Cloud | Cloud Solutions Architect
> > > t 855.652.5683
> > >
> > > *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
> > > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> > > w cloudops.com *|* tw @CloudOps_
> > >
> > >
> > > On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav <
> rohit.ya...@shapeblue.com
> > >
> > > wrote:
> > >
> > >> Hi,
> > >>
> > >> Can anyone explain how xs-tools (xen server tools, guest utilities)
> are
> > >> installed on systemvms and on user instances?
> > >>
> > >> I see that we’ve a script in
> > >> tools/appliance/definitions/builtin/install-xs-tools.sh that is
> > supposed to
> > >> download and install xs-tool on vms but we are not using this for our
> > >> systemvmtemplate or systemvm64template definition.
> > >>
> > >> If we need it, should we run this script (which downloads and installs
> > xs
> > >> tools) on the systemvms?
> > >>
> > >> Regards,
> > >> Rohit Yadav
> > >> Software Architect, ShapeBlue
> > >> M. +41 779015219 | rohit.ya...@shapeblue.com 
> > >> Blog: bhaisaab.org | Twitter: @_bhaisaab
> > >>
> > >> Find out more about ShapeBlue and our range of CloudStack related
> > services
> > >>
> > >> IaaS Cloud Design & Build<
> > >> http://shapeblue.com/iaas-cloud-design-and-build//>
> > >> CSForge – rapid IaaS deployment framework<
> http://shapeblue.com/csforge/
> > >
> > >> CloudStack Consulting
> > >> CloudStack Infrastructure Support<
> > >> http://shapeblue.com/cloudstack-infrastructure-support/>
> > >> CloudStack Bootcamp Training Courses<
> > >> http://shapeblue.com/cloudstack-training/>
> > >>
> > >> This email and any attachments to it may be confidential and are
> > intended
> > >> solely for the use of the individual to whom it is addressed. Any
> views
> > or
> > >> opinions expressed are solely those of the author and do not
> necessarily
> > >> represent those of Shape Blue Ltd or related companies. If you are not
> > the
> > >> intended recipient of this email, you must neither take any action
> based
> > >> upon its contents, nor copy or show it to anyone. Please contact the
> > sender
> > >> if you believe you have received this email in error. Shape Blue Ltd
> is
> > a
> > >> company incorporated in England & Wales. ShapeBlue Services India LLP
> > is a
> > >> company incorporated in India and is operated under license from Shape
> > Blue
> > >> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> > Brasil
> > >> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty
> Ltd
> > is
> > >> a company registered by The Republic of South Africa and is traded
> under
> > >> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> > >>
> >
> > Regards,
> > Rohit Yadav
> > Software Architect, ShapeBlue
> > M. +41 779015219 | rohit.ya...@shapeblue.com 
> > Blog: bhaisaab.org | Twitter: @_bhaisaab
> >
> >
> >
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> > http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Rohit Yadav
I think this should be fine. I see that 
http://repository.apache.org/snapshots/org/apache/cloudstack is taking time, in 
some cases timing out or giving 5xx (service unavailable) errors.

Lucian, can you please try this:
- rm -fr ~/.m2/repository/org/apache/cloudstack/xapi
- cd /deps/XenServerJava && mvn clean install -> this will 
install xapi 5.6.100-1.jar
- now build your source with latest 4.3 (I fixed this in root pom) or remove 
SNAPSHOT from root pom in the xapi version and then build

Commit 41c0eaa on 4.3 branch removed SNAPSHOT name from version of 
deps/XenServer so, on systems which have not built other branches incorrect 
xapi build will fail the build.

I hate this, but we should create a new voting candidate with this fix on 4.3.

Sebastien, sorry can you do this again. And, also if we should have xs-tools 
pre-installed on systemvm templates (from another thread). If we want it, we 
should fix it starting 4.3.1 systemvm templates.

On 09-Sep-2014, at 6:08 pm, sebgoa  wrote:
> I don't know if that's the issue, but I mentioned something to Mike T.
>
> In  deps/XenServerJava/pom.xml
>
> the release build script removed a -SNAPSHOT in the xapi version
>
> so
>
> xapi
>
> 5.6.100-1
>
> used to be 5.6.100-1-SNAPSHOT
>
> Sadly I have no clue if this is normal or not…
>
>
>
> On Sep 9, 2014, at 6:01 PM, Nux!  wrote:
>
>> Hi Rohit,
>>
>> Thanks for the advice, but removing that did not help. Here's the log:
>> http://tmp.nux.ro/431xenfailflog.txt
>>
>> I see jenkins job #583 is also failing for the same reason.
>>
>> Lucian
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
>>
>> - Original Message -
>>> From: "Rohit Yadav" 
>>> To: dev@cloudstack.apache.org
>>> Sent: Tuesday, 9 September, 2014 4:00:56 PM
>>> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
>>>
>>> Hi Lucian, looks like you’re hitting mvn/cache issues. I see this issue on
>>> our jenkins.buildacloud.org as well.
>>>
>>> Can you remove files from
>>> ~/.m2/repository/net/java/dev/vcc/thirdparty/xen-api and try building it
>>> again. You may even remove you whole ~/.m2 and try building. If you hit it
>>> again, can you share the full log with us for debuggin?
>>>
>>> I’m able to build 4.3.1 from source. Here’s my build log:
>>> http://people.apache.org/~bhaisaab/4.3.1-rpmbuild.txt
>>>
>>> I’ve put built rpms from above here:
>>> http://packages.bhaisaab.org/cloudstack/main/centos/
>>>
>>>
>>> On 09-Sep-2014, at 4:12 pm, Nux!  wrote:
>>>
 -1, source doesnt build the rpms for me

 INFO] Total time: 3:45.324s
 [INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
 [INFO] Final Memory: 40M/243M
 [INFO]
 
 [ERROR] Failed to execute goal
 org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile
 (default-compile) on project cloud-plugin-hypervisor-xen: Compilation
 failure: Compilation failure:
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,SR,,)
 [ERROR] location: variable srcVdi of type VDI
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
 [ERROR] location: variable snapshotvdi of type VDI
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,SR,,)
 [ERROR] location: variable snapshotVdi of type VDI
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,SR,,)
 [ERROR] location: variable vol of type VDI
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
 [ERROR] location: variable snapChain of type VDI
 [ERROR]
 /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
 error: cannot find symbol
 [ERROR] symbol:   method copyAsync2(Connection,SR,,)
 [ERROR] location: variable srcVdi of t

Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Rohit Yadav
Hi Todd,

Thanks for sharing, do you have the xs-tools iso link? We can fix it with ACS 
4.3.1, and have them preinstalled on systemvms.

On 09-Sep-2014, at 6:15 pm, Todd Pigram  wrote:

> Same issue with CCP4.3.1. I have to do the install manually.
>
> On Tuesday, September 9, 2014, Rohit Yadav 
> wrote:
>
>> Hi,
>>
>> Thanks for replying Pierre.
>>
>> On 09-Sep-2014, at 5:37 pm, Pierre-Luc Dion > > wrote:
>>> Hi Rohit,
>>>
>>> I've tried to investigate on this one recently because right now the
>>> xentools are not installed on systemvm templates (acs 4.2+).
>>> I've tried to build systemvm template for xenserver which this script and
>>> tuned version for XS6.2 but not success during the build process.  I did
>>> not investigate further but I've got into error related to running out of
>>> disk space during the systemvm creation.
>>>
>>> This was not for the systemvm.iso but to generate .vhd.  I'm curious to
>>> investigate and need this fixed but did not had enough time yet...
>>>
>>> From what I've saw, you need to add install-xs-tool.sh into the script
>>> lists in : tools/appliance/definitions/systemvm64template/postinstall.sh
>>>
>>> You might have to update install-xs-tools.sh since it would install an
>> old
>>> xs-tools version…
>>
>> So, do we need to install xs-tools on systemvm templatest, what does it
>> bring with it and what is the use of having it?
>>
>> If we want it which newer version of xs-tools iso should we pre-install on
>> systemvms? And is there a publicly hosted location of a newer iso?
>>
>>> Regards,
>>>
>>>
>>>
>>> *Pierre-Luc DION*
>>> Architecte de Solution Cloud | Cloud Solutions Architect
>>> t 855.652.5683
>>>
>>> *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
>>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
>>> w cloudops.com *|* tw @CloudOps_
>>>
>>>
>>> On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav > >
>>> wrote:
>>>
 Hi,

 Can anyone explain how xs-tools (xen server tools, guest utilities) are
 installed on systemvms and on user instances?

 I see that we’ve a script in
 tools/appliance/definitions/builtin/install-xs-tools.sh that is
>> supposed to
 download and install xs-tool on vms but we are not using this for our
 systemvmtemplate or systemvm64template definition.

 If we need it, should we run this script (which downloads and installs
>> xs
 tools) on the systemvms?

 Regards,
 Rohit Yadav
 Software Architect, ShapeBlue
 M. +41 779015219 | rohit.ya...@shapeblue.com 
 Blog: bhaisaab.org | Twitter: @_bhaisaab

 Find out more about ShapeBlue and our range of CloudStack related
>> services

 IaaS Cloud Design & Build<
 http://shapeblue.com/iaas-cloud-design-and-build//>
 CSForge – rapid IaaS deployment framework>>
 CloudStack Consulting
 CloudStack Infrastructure Support<
 http://shapeblue.com/cloudstack-infrastructure-support/>
 CloudStack Bootcamp Training Courses<
 http://shapeblue.com/cloudstack-training/>

 This email and any attachments to it may be confidential and are
>> intended
 solely for the use of the individual to whom it is addressed. Any views
>> or
 opinions expressed are solely those of the author and do not necessarily
 represent those of Shape Blue Ltd or related companies. If you are not
>> the
 intended recipient of this email, you must neither take any action based
 upon its contents, nor copy or show it to anyone. Please contact the
>> sender
 if you believe you have received this email in error. Shape Blue Ltd is
>> a
 company incorporated in England & Wales. ShapeBlue Services India LLP
>> is a
 company incorporated in India and is operated under license from Shape
>> Blue
 Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
>> Brasil
 and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd
>> is
 a company registered by The Republic of South Africa and is traded under
 license from Shape Blue Ltd. ShapeBlue is a registered trademark.

>>
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +41 779015219 | rohit.ya...@shapeblue.com 
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>>
>>
>>
>> Find out more about ShapeBlue and our range of CloudStack related services
>>
>> IaaS Cloud Design & Build<
>> http://shapeblue.com/iaas-cloud-design-and-build//>
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Infrastructure Support<
>> http://shapeblue.com/cloudstack-infrastructure-support/>
>> CloudStack Bootcamp Training Courses<
>> http://shapeblue.com/cloudstack-training/>
>>
>> This email and any attachments to it may be confidential and are intended
>> solely for the use of the individual to whom it is addressed. Any vi

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Mike Tutkowski
Yeah, I looked at that.

I, also, do not know if that is normal or not.

On Tue, Sep 9, 2014 at 10:08 AM, sebgoa  wrote:

> I don't know if that's the issue, but I mentioned something to Mike T.
>
> In  deps/XenServerJava/pom.xml
>
> the release build script removed a -SNAPSHOT in the xapi version
>
> so
>
>  xapi
>
>  5.6.100-1
>
> used to be 5.6.100-1-SNAPSHOT
>
> Sadly I have no clue if this is normal or not…
>
>
>
> On Sep 9, 2014, at 6:01 PM, Nux!  wrote:
>
> > Hi Rohit,
> >
> > Thanks for the advice, but removing that did not help. Here's the log:
> > http://tmp.nux.ro/431xenfailflog.txt
> >
> > I see jenkins job #583 is also failing for the same reason.
> >
> > Lucian
> >
> > --
> > Sent from the Delta quadrant using Borg technology!
> >
> > Nux!
> > www.nux.ro
> >
> >
> > - Original Message -
> >> From: "Rohit Yadav" 
> >> To: dev@cloudstack.apache.org
> >> Sent: Tuesday, 9 September, 2014 4:00:56 PM
> >> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
> >>
> >> Hi Lucian, looks like you’re hitting mvn/cache issues. I see this issue
> on
> >> our jenkins.buildacloud.org as well.
> >>
> >> Can you remove files from
> >> ~/.m2/repository/net/java/dev/vcc/thirdparty/xen-api and try building it
> >> again. You may even remove you whole ~/.m2 and try building. If you hit
> it
> >> again, can you share the full log with us for debuggin?
> >>
> >> I’m able to build 4.3.1 from source. Here’s my build log:
> >> http://people.apache.org/~bhaisaab/4.3.1-rpmbuild.txt
> >>
> >> I’ve put built rpms from above here:
> >> http://packages.bhaisaab.org/cloudstack/main/centos/
> >>
> >>
> >> On 09-Sep-2014, at 4:12 pm, Nux!  wrote:
> >>
> >>> -1, source doesnt build the rpms for me
> >>>
> >>> INFO] Total time: 3:45.324s
> >>> [INFO] Finished at: Tue Sep 09 14:52:58 BST 2014
> >>> [INFO] Final Memory: 40M/243M
> >>> [INFO]
> >>>
> 
> >>> [ERROR] Failed to execute goal
> >>> org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile
> >>> (default-compile) on project cloud-plugin-hypervisor-xen: Compilation
> >>> failure: Compilation failure:
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[177,34]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> >>> [ERROR] location: variable srcVdi of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[236,34]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,VDI,)
> >>> [ERROR] location: variable snapshotvdi of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[347,43]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> >>> [ERROR] location: variable snapshotVdi of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[465,27]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> >>> [ERROR] location: variable vol of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[581,37]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> >>> [ERROR] location: variable snapChain of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[644,34]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> >>> [ERROR] location: variable srcVdi of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[696,34]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,SR,,)
> >>> [ERROR] location: variable srcVdi of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3.1/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625StorageProcessor.java:[780,37]
> >>> error: cannot find symbol
> >>> [ERROR] symbol:   method copyAsync2(Connection,,,VDI)
> >>> [ERROR] location: variable snapChain of type VDI
> >>> [ERROR]
> >>>
> /root/apache-cloudstack-4.3.1-src/dist/rpmbuild/BUILD/cloudstack-4.3

Jenkins build is back to normal : cloudstack-4.3-maven-build #589

2014-09-09 Thread jenkins
See 



Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread Rohit Yadav

On 09-Sep-2014, at 6:33 pm, Mike Tutkowski  wrote:
> Yeah, I looked at that.
>
> I, also, do not know if that is normal or not.

On 4.3 and previous/old branches, we had xapi build out of deps/XenServer, I 
think starting 4.4 and master we don’t have xapi within our source (this was 
done by the xenserver plugin refactor I think by Tim).

So, you make sure whatever version you declare in root pom.xml is the same in 
deps/XenServer/pom.xml

I put a fix on 4.3 that makes sure we’ve using same version on both places.
The latest build on jenkins.b.o is working now:
http://jenkins.buildacloud.org/view/4.3/job/cloudstack-4.3-maven-build/589/console


Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.


Jenkins build is back to normal : cloudstack-4.3-maven-build #593

2014-09-09 Thread jenkins
See 



Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Pierre-Luc Dion
Hi Ian,  you are probably right about the PV driver of Debian 7, I can't
confirmed. But, the xs-tools also provide interesting ressources usage
metrics of VMs in XenServer. I think it is also required for xenmotion of
VMs for maintenance purposes.


*Pierre-Luc DION*
Architecte de Solution Cloud | Cloud Solutions Architect
t 855.652.5683

*CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_


On Tue, Sep 9, 2014 at 12:31 PM, Rohit Yadav 
wrote:

> Hi Todd,
>
> Thanks for sharing, do you have the xs-tools iso link? We can fix it with
> ACS 4.3.1, and have them preinstalled on systemvms.
>
> On 09-Sep-2014, at 6:15 pm, Todd Pigram  wrote:
>
> > Same issue with CCP4.3.1. I have to do the install manually.
> >
> > On Tuesday, September 9, 2014, Rohit Yadav 
> > wrote:
> >
> >> Hi,
> >>
> >> Thanks for replying Pierre.
> >>
> >> On 09-Sep-2014, at 5:37 pm, Pierre-Luc Dion  >> > wrote:
> >>> Hi Rohit,
> >>>
> >>> I've tried to investigate on this one recently because right now the
> >>> xentools are not installed on systemvm templates (acs 4.2+).
> >>> I've tried to build systemvm template for xenserver which this script
> and
> >>> tuned version for XS6.2 but not success during the build process.  I
> did
> >>> not investigate further but I've got into error related to running out
> of
> >>> disk space during the systemvm creation.
> >>>
> >>> This was not for the systemvm.iso but to generate .vhd.  I'm curious to
> >>> investigate and need this fixed but did not had enough time yet...
> >>>
> >>> From what I've saw, you need to add install-xs-tool.sh into the script
> >>> lists in :
> tools/appliance/definitions/systemvm64template/postinstall.sh
> >>>
> >>> You might have to update install-xs-tools.sh since it would install an
> >> old
> >>> xs-tools version…
> >>
> >> So, do we need to install xs-tools on systemvm templatest, what does it
> >> bring with it and what is the use of having it?
> >>
> >> If we want it which newer version of xs-tools iso should we pre-install
> on
> >> systemvms? And is there a publicly hosted location of a newer iso?
> >>
> >>> Regards,
> >>>
> >>>
> >>>
> >>> *Pierre-Luc DION*
> >>> Architecte de Solution Cloud | Cloud Solutions Architect
> >>> t 855.652.5683
> >>>
> >>> *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts
> >>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> >>> w cloudops.com *|* tw @CloudOps_
> >>>
> >>>
> >>> On Tue, Sep 9, 2014 at 11:22 AM, Rohit Yadav <
> rohit.ya...@shapeblue.com
> >> >
> >>> wrote:
> >>>
>  Hi,
> 
>  Can anyone explain how xs-tools (xen server tools, guest utilities)
> are
>  installed on systemvms and on user instances?
> 
>  I see that we’ve a script in
>  tools/appliance/definitions/builtin/install-xs-tools.sh that is
> >> supposed to
>  download and install xs-tool on vms but we are not using this for our
>  systemvmtemplate or systemvm64template definition.
> 
>  If we need it, should we run this script (which downloads and installs
> >> xs
>  tools) on the systemvms?
> 
>  Regards,
>  Rohit Yadav
>  Software Architect, ShapeBlue
>  M. +41 779015219 | rohit.ya...@shapeblue.com 
>  Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
>  Find out more about ShapeBlue and our range of CloudStack related
> >> services
> 
>  IaaS Cloud Design & Build<
>  http://shapeblue.com/iaas-cloud-design-and-build//>
>  CSForge – rapid IaaS deployment framework<
> http://shapeblue.com/csforge/
> >>>
>  CloudStack Consulting
>  CloudStack Infrastructure Support<
>  http://shapeblue.com/cloudstack-infrastructure-support/>
>  CloudStack Bootcamp Training Courses<
>  http://shapeblue.com/cloudstack-training/>
> 
>  This email and any attachments to it may be confidential and are
> >> intended
>  solely for the use of the individual to whom it is addressed. Any
> views
> >> or
>  opinions expressed are solely those of the author and do not
> necessarily
>  represent those of Shape Blue Ltd or related companies. If you are not
> >> the
>  intended recipient of this email, you must neither take any action
> based
>  upon its contents, nor copy or show it to anyone. Please contact the
> >> sender
>  if you believe you have received this email in error. Shape Blue Ltd
> is
> >> a
>  company incorporated in England & Wales. ShapeBlue Services India LLP
> >> is a
>  company incorporated in India and is operated under license from Shape
> >> Blue
>  Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> >> Brasil
>  and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty
> Ltd
> >> is
>  a company registered by The Republic of South Africa and is traded
> under
>  license from Shape Blue Ltd. ShapeBlue is a register

Re: Review Request 22707: Test suite contains test cases relating to access checks for listSnapshot() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain ad

2014-09-09 Thread Min Chen

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


Can we change to use consistent docstring (triple quotes style) for your method 
comments?

- Min Chen


On Sept. 9, 2014, 12:27 a.m., sangeetha hariharan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22707/
> ---
> 
> (Updated Sept. 9, 2014, 12:27 a.m.)
> 
> 
> Review request for cloudstack, edison su, Min Chen, Prachi Damle, and 
> Santhosh Edukulla.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This test suite contains test cases relating to access checks for 
> listSnapshot() with parameters - id,listall,isrecursive,account and domainid 
> executed as ROOT admin,domain admin and regular users.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_acl_listsnapshot.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/22707/diff/
> 
> 
> Testing
> ---
> 
> test_listSnapshot_as_domainadmin 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin | Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_cross_domainid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_cross_domainid | Status : SUCCESS 
> ===
> ok
> test_listSnapshot_as_domainadmin_cross_domainid_accountid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_cross_domainid_accountid | Status 
> : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid | Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid | Status : 
> SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_listall_false | Status : 
> SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listS

Jenkins build is still unstable: simulator-singlerun #324

2014-09-09 Thread jenkins
See 



Re: Review Request 22712: This Test suite has test cases relating to acess checks for deleteNetwork() for Admin, domain admin and regular users

2014-09-09 Thread Min Chen

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


Please change method comments to use one docstrings style instead of mixing 
them with single-line comments.

- Min Chen


On Sept. 8, 2014, 7:52 p.m., sangeetha hariharan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22712/
> ---
> 
> (Updated Sept. 8, 2014, 7:52 p.m.)
> 
> 
> Review request for cloudstack, Min Chen, Prachi Damle, and Santhosh Edukulla.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This Test suite has test cases relating to acess checks for deleteNetwork() 
> for Admin, domain admin and regular users
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_acl_isolatednetwork_delete.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/22712/diff/
> 
> 
> Testing
> ---
> 
> Test Suite was executed against a management server built from 4.4-forward 
> branch using a simulator set up:
> test_deleteNetwork_admin 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin | Status : SUCCESS ===
> ok
> test_deleteNetwork_admin_foruserinotherdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin_foruserinotherdomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_admin_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin_foruserinsamedomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_domaindmin 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin | Status : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_forcrossdomainuser 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_forcrossdomainuser | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_foruserinsamedomain | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_foruserinsubdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_foruserinsubdomain | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_user 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user | Status : SUCCESS ===
> ok
> test_deleteNetwork_user_foruserinotherdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user_foruserinotherdomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_user_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user_foruserinsamedomain | Status : 
> SUCCESS ===
> ok
> 
> --
> Ran 10 tests in 61.766s
> 
> OK
> 
> 
> Thanks,
> 
> sangeetha hariharan
> 
>



Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread sebgoa

On Sep 9, 2014, at 6:45 PM, Rohit Yadav  wrote:

> 
> On 09-Sep-2014, at 6:33 pm, Mike Tutkowski  
> wrote:
>> Yeah, I looked at that.
>> 
>> I, also, do not know if that is normal or not.
> 
> On 4.3 and previous/old branches, we had xapi build out of deps/XenServer, I 
> think starting 4.4 and master we don’t have xapi within our source (this was 
> done by the xenserver plugin refactor I think by Tim).
> 
> So, you make sure whatever version you declare in root pom.xml is the same in 
> deps/XenServer/pom.xml
> 

Ok, so I would like to get confirmation from other folks on whether this is the 
right way to do it or not.
And if it's the current version or not.

I am cc Abhi and animesh who worked on 4.3.0

> I put a fix on 4.3 that makes sure we’ve using same version on both places.
> The latest build on jenkins.b.o is working now:
> http://jenkins.buildacloud.org/view/4.3/job/cloudstack-4.3-maven-build/589/console
> 
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> 
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.



Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

2014-09-09 Thread David Nalley
repository.a.o has had a number of delays and an outage or two - see
if it builds now.

On Tue, Sep 9, 2014 at 3:13 PM, sebgoa  wrote:
>
> On Sep 9, 2014, at 6:45 PM, Rohit Yadav  wrote:
>
>>
>> On 09-Sep-2014, at 6:33 pm, Mike Tutkowski  
>> wrote:
>>> Yeah, I looked at that.
>>>
>>> I, also, do not know if that is normal or not.
>>
>> On 4.3 and previous/old branches, we had xapi build out of deps/XenServer, I 
>> think starting 4.4 and master we don’t have xapi within our source (this was 
>> done by the xenserver plugin refactor I think by Tim).
>>
>> So, you make sure whatever version you declare in root pom.xml is the same 
>> in deps/XenServer/pom.xml
>>
>
> Ok, so I would like to get confirmation from other folks on whether this is 
> the right way to do it or not.
> And if it's the current version or not.
>
> I am cc Abhi and animesh who worked on 4.3.0
>
>> I put a fix on 4.3 that makes sure we’ve using same version on both places.
>> The latest build on jenkins.b.o is working now:
>> http://jenkins.buildacloud.org/view/4.3/job/cloudstack-4.3-maven-build/589/console
>>
>>
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +41 779015219 | rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>>
>>
>>
>> Find out more about ShapeBlue and our range of CloudStack related services
>>
>> IaaS Cloud Design & Build
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Infrastructure 
>> Support
>> CloudStack Bootcamp Training 
>> Courses
>>
>> This email and any attachments to it may be confidential and are intended 
>> solely for the use of the individual to whom it is addressed. Any views or 
>> opinions expressed are solely those of the author and do not necessarily 
>> represent those of Shape Blue Ltd or related companies. If you are not the 
>> intended recipient of this email, you must neither take any action based 
>> upon its contents, nor copy or show it to anyone. Please contact the sender 
>> if you believe you have received this email in error. Shape Blue Ltd is a 
>> company incorporated in England & Wales. ShapeBlue Services India LLP is a 
>> company incorporated in India and is operated under license from Shape Blue 
>> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil 
>> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a 
>> company registered by The Republic of South Africa and is traded under 
>> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>


Jenkins build is still unstable: simulator-singlerun #325

2014-09-09 Thread jenkins
See 



Re: Review Request 22707: Test suite contains test cases relating to access checks for listSnapshot() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain ad

2014-09-09 Thread sangeetha hariharan

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

(Updated Sept. 9, 2014, 7:21 p.m.)


Review request for cloudstack, edison su, Min Chen, Prachi Damle, and Santhosh 
Edukulla.


Changes
---

Removed "#" from comments that are already enclosed within """ block.


Repository: cloudstack-git


Description
---

This test suite contains test cases relating to access checks for 
listSnapshot() with parameters - id,listall,isrecursive,account and domainid 
executed as ROOT admin,domain admin and regular users.


Diffs (updated)
-

  test/integration/component/test_acl_listsnapshot.py PRE-CREATION 

Diff: https://reviews.apache.org/r/22707/diff/


Testing
---

test_listSnapshot_as_domainadmin 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin | Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_cross_domainid 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_cross_domainid | Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_cross_domainid_accountid 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_cross_domainid_accountid | Status : 
SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid | Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_accountid | Status : 
SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_false | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: 
test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: 
test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_true | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: 
test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: 
test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_rec_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_false | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_accountid_rec_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_true | Status 
: SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_listall_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_listall_false | Status : 
SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_listall_false_rec_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_listall_false_rec_true | 
Status : SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_listall_true 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_listall_true | Status : 
SUCCESS ===
ok
test_listSnapshot_as_domainadmin_domainid_listall_true_rec_false 
(integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
TestName: test_listSnapshot_as_domainadmin_domainid_listall_true_rec_false | 
Status :

Re: Review Request 22712: This Test suite has test cases relating to acess checks for deleteNetwork() for Admin, domain admin and regular users

2014-09-09 Thread sangeetha hariharan

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

(Updated Sept. 9, 2014, 8:39 p.m.)


Review request for cloudstack, Min Chen, Prachi Damle, and Santhosh Edukulla.


Changes
---

Removed "#" from comments that are already encampassed with in """ block


Repository: cloudstack-git


Description
---

This Test suite has test cases relating to acess checks for deleteNetwork() for 
Admin, domain admin and regular users


Diffs (updated)
-

  test/integration/component/test_acl_isolatednetwork_delete.py PRE-CREATION 

Diff: https://reviews.apache.org/r/22712/diff/


Testing
---

Test Suite was executed against a management server built from 4.4-forward 
branch using a simulator set up:
test_deleteNetwork_admin 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_admin | Status : SUCCESS ===
ok
test_deleteNetwork_admin_foruserinotherdomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_admin_foruserinotherdomain | Status : 
SUCCESS ===
ok
test_deleteNetwork_admin_foruserinsamedomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_admin_foruserinsamedomain | Status : 
SUCCESS ===
ok
test_deleteNetwork_domaindmin 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_domaindmin | Status : SUCCESS ===
ok
test_deleteNetwork_domaindmin_forcrossdomainuser 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_domaindmin_forcrossdomainuser | Status : 
SUCCESS ===
ok
test_deleteNetwork_domaindmin_foruserinsamedomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_domaindmin_foruserinsamedomain | Status : 
SUCCESS ===
ok
test_deleteNetwork_domaindmin_foruserinsubdomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_domaindmin_foruserinsubdomain | Status : 
SUCCESS ===
ok
test_deleteNetwork_user 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_user | Status : SUCCESS ===
ok
test_deleteNetwork_user_foruserinotherdomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_user_foruserinotherdomain | Status : 
SUCCESS ===
ok
test_deleteNetwork_user_foruserinsamedomain 
(integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
 ... === TestName: test_deleteNetwork_user_foruserinsamedomain | Status : 
SUCCESS ===
ok

--
Ran 10 tests in 61.766s

OK


Thanks,

sangeetha hariharan



Re: [jira] [Commented] (CLOUDSTACK-7184) HA should wait for at least 'xen.heartbeat.interval' sec before starting HA on vm's when host is marked down

2014-09-09 Thread Daan Hoogland
Could some of you please review hotfix/4.4/CLOUDSTACK-7184

I could do with some advice on how to thoroughly test this functionality as
well.

thanks,
Daan


On Tue, Sep 9, 2014 at 10:22 PM, ASF subversion and git services (JIRA) <
j...@apache.org> wrote:

>
> [
> https://issues.apache.org/jira/browse/CLOUDSTACK-7184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14127488#comment-14127488
> ]
>
> ASF subversion and git services commented on CLOUDSTACK-7184:
> -
>
> Commit 7f97bf7c58a348e684f7138c93c287a3c86ca55b in cloudstack's branch
> refs/heads/hotfix/4.4/CLOUDSTACK-7184 from [~dahn]
> [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=7f97bf7 ]
>
> CLOUDSTACK-7184: xenheartbeat gets passed timeout and interval
>
> > HA should wait for at least 'xen.heartbeat.interval' sec before starting
> HA on vm's when host is marked down
> >
> 
> >
> > Key: CLOUDSTACK-7184
> > URL:
> https://issues.apache.org/jira/browse/CLOUDSTACK-7184
> > Project: CloudStack
> >  Issue Type: Bug
> >  Security Level: Public(Anyone can view this level - this is the
> default.)
> >  Components: Hypervisor Controller, Management Server, XenServer
> >Affects Versions: 4.3.0, 4.4.0, 4.5.0
> > Environment: CloudStack 4.3 with XenServer 6.2 hypervisors
> >Reporter: Remi Bergsma
> >Assignee: Daan Hoogland
> >Priority: Blocker
> >
> > Hypervisor got isolated for 30 seconds due to a network issue.
> CloudStack did discover this and marked the host as down, and immediately
> started HA. Just 18 seconds later the hypervisor returned and we ended up
> with 5 vm's that were running on two hypervisors at the same time.
> > This, of course, resulted in file system corruption and the loss of the
> vm's. One side of the story is why XenServer allowed this to happen (will
> not bother you with this one). The CloudStack side of the story: HA should
> only start after at least xen.heartbeat.interval seconds. If the host is
> down long enough, the Xen heartbeat script will fence the hypervisor and
> prevent corruption. If it is not down long enough, nothing should happen.
> > Logs (short):
> > 2014-07-25 05:03:28,596 WARN  [c.c.a.m.DirectAgentAttache]
> (DirectAgent-122:ctx-690badc5) Unable to get current status on 505(mccpvmXX)
> > .
> > 2014-07-25 05:03:31,920 ERROR [c.c.a.m.AgentManagerImpl]
> (AgentTaskPool-10:ctx-11b9af3e) Host is down: 505-mccpvmXX.  Starting HA on
> the VMs
> > .
> > 2014-07-25 05:03:49,655 DEBUG [c.c.h.Status] (ClusteredAgentManager
> Timer:ctx-0e00979c) Transition:[Resource state = Enabled, Agent event =
> AgentDisconnected, Host id = 505, name = mccpvmXX]
> > cs marks host down: 2014-07-25  05:03:31,920
> > cs marks host up: 2014-07-25  05:03:49,655
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.4#6332)
>



-- 
Daan


Re: Review Request 22712: This Test suite has test cases relating to acess checks for deleteNetwork() for Admin, domain admin and regular users

2014-09-09 Thread Min Chen

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

Ship it!


Ship It!

- Min Chen


On Sept. 9, 2014, 8:39 p.m., sangeetha hariharan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22712/
> ---
> 
> (Updated Sept. 9, 2014, 8:39 p.m.)
> 
> 
> Review request for cloudstack, Min Chen, Prachi Damle, and Santhosh Edukulla.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This Test suite has test cases relating to acess checks for deleteNetwork() 
> for Admin, domain admin and regular users
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_acl_isolatednetwork_delete.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/22712/diff/
> 
> 
> Testing
> ---
> 
> Test Suite was executed against a management server built from 4.4-forward 
> branch using a simulator set up:
> test_deleteNetwork_admin 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin | Status : SUCCESS ===
> ok
> test_deleteNetwork_admin_foruserinotherdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin_foruserinotherdomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_admin_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_admin_foruserinsamedomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_domaindmin 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin | Status : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_forcrossdomainuser 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_forcrossdomainuser | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_foruserinsamedomain | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_domaindmin_foruserinsubdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_domaindmin_foruserinsubdomain | Status 
> : SUCCESS ===
> ok
> test_deleteNetwork_user 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user | Status : SUCCESS ===
> ok
> test_deleteNetwork_user_foruserinotherdomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user_foruserinotherdomain | Status : 
> SUCCESS ===
> ok
> test_deleteNetwork_user_foruserinsamedomain 
> (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete)
>  ... === TestName: test_deleteNetwork_user_foruserinsamedomain | Status : 
> SUCCESS ===
> ok
> 
> --
> Ran 10 tests in 61.766s
> 
> OK
> 
> 
> Thanks,
> 
> sangeetha hariharan
> 
>



Re: Review Request 22707: Test suite contains test cases relating to access checks for listSnapshot() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain ad

2014-09-09 Thread Min Chen

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

Ship it!


Ship It!

- Min Chen


On Sept. 9, 2014, 7:21 p.m., sangeetha hariharan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22707/
> ---
> 
> (Updated Sept. 9, 2014, 7:21 p.m.)
> 
> 
> Review request for cloudstack, edison su, Min Chen, Prachi Damle, and 
> Santhosh Edukulla.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This test suite contains test cases relating to access checks for 
> listSnapshot() with parameters - id,listall,isrecursive,account and domainid 
> executed as ROOT admin,domain admin and regular users.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_acl_listsnapshot.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/22707/diff/
> 
> 
> Testing
> ---
> 
> test_listSnapshot_as_domainadmin 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin | Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_cross_domainid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_cross_domainid | Status : SUCCESS 
> ===
> ok
> test_listSnapshot_as_domainadmin_cross_domainid_accountid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_cross_domainid_accountid | Status 
> : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid | Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid | Status : 
> SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_false_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_listall_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: 
> test_listSnapshot_as_domainadmin_domainid_accountid_listall_true_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_accountid_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_accountid_rec_true | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_listall_false | Status : 
> SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_listall_false_rec_false | 
> Status : SUCCESS ===
> ok
> test_listSnapshot_as_domainadmin_domainid_listall_false_rec_true 
> (integration.component.test_acl_listsnapshot.TestSnapshotList) ... === 
> TestName: test_listSnapshot_as_domainadmin_domainid_listall_false_rec_true | 
> Status : SUCCE

RE: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

2014-09-09 Thread Alex Brett
On essentially any current Linux distro, xs-tools doesn't actually install any 
drivers (in the days of RHEL4 and the like it used to have custom kernels as 
the vendor ones had severe limitations), because as you say the kernel has the 
support built in (PVops).

What xs-tools gives in Linux is:
 - Reporting of the VM distro etc back to the toolstack
 - In guest memory usage reporting back to the toolstack
 - Installs some userspace xenstore tools

The XenServer toolstack does gate some VM operations on the presence of the 
tools, in particular migrate. This is somewhat historical (as the tools aren't 
doing any driver changes they don't actually affect whether the migrate would 
succeed or not, but when different kernels were required the presence of the 
agent gave an indication the correct kernel was in use, and it is shared code 
with Windows guests where you do need the drivers to migrate safely), and may 
be changed in future, but for now is a limitation.

In terms of versions, we should ideally always install one from the latest 
XenServer release, these should (though I'll need to do some tests to be 100% 
sure) work on older ones as well with no problems. Older ones will likely work 
properly on newer versions as well, though it will in some cases cause an out 
of date warning to appear in XenCenter, which could be worrying for users.

Alex


From: Pierre-Luc Dion [pd...@cloudops.com]
Sent: 09 September 2014 17:59
To: dev@cloudstack.apache.org
Subject: Re: [DISCUSS] How xs-tools gets installed for xen vms and systemvms

Hi Ian,  you are probably right about the PV driver of Debian 7, I can't
confirmed. But, the xs-tools also provide interesting ressources usage
metrics of VMs in XenServer. I think it is also required for xenmotion of
VMs for maintenance purposes.



Jenkins build is still unstable: simulator-singlerun #326

2014-09-09 Thread jenkins
See 



[GitHub] cloudstack pull request: Test cmd rebase

2014-09-09 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request:

https://github.com/apache/cloudstack/pull/18

Test cmd rebase

This is a new Pull Request in order to fix the conflicts we found in the 
old pull request #14 . I will put the most update version of the description 
here:

Pull request of changes in the "cloud-server" module

In the last 10 weeks we have worked in the cloud-server, focusing our time 
in the refactor of the [Vpc]VirtualNetworkApplianceManagerImpl. We had a mains 
goals increase of Maintainability, Extensibility, Readability and test 
coverage. That was just a first step towards the development, still in 
progress, of the Redundant Virtual Routers for VPC.

== What has been done so far:

• The VirtualNetworkApplianceManagerImpl class line numbers dropped from  
4440 to 2558
• The VpcVirtualNetworkApplianceImpl class line numbers dropped from 1484 
to 749
• We created 35 new classes in order to split the code/responsibility
• We added 97.8% unit test coverage for com.cloud.network.element/router 
and org.cloud.network.router.deployment packages
o   The most complex classes we changed are in those packages
o   About 1700 lines of unit tests
• We executed many Marvin tests that we got from ACS and made compliant 
with our domain:
o   test_01_create_account
o   test_01_add_vm_to_subdomain
o   test_DeleteDomain
o   test_forceDeleteDomain
o   test_updateAdminDetails
o   test_updateDomainAdminDetails
o   test_updateUserDetails
o   test_LoginApiDomain
o   test_LoginApiUuidResponse
o   test_privategw_acl
o   test_01_reset_vm_on_reboot
o   test_03_restart_network_cleanup
o   test_05_router_basic
o   test_06_router_advanced
o   test_07_stop_router
o   test_08_start_router
o   test_09_reboot_router
o   test_01_create_service_offering
o   test_02_edit_service_offering
o   test_03_delete_service_offering
o   test_01_start_stop_router_after_addition_of_one_guest_network
o   test_02_reboot_router_after_addition_of_one_guest_network
o   test_04_chg_srv_off_router_after_addition_of_one_guest_network
o   test_05_destroy_router_after_addition_of_one_guest_network
o   test_01_stop_start_router_after_creating_vpc
o   test_02_reboot_router_after_creating_vpc
o   test_04_change_service_offerring_vpc
o   test_05_destroy_router_after_creating_vpc
o   test_vpc_remote_access_vpn
o   test_vpc_site2site_vpn

We started the changes in the network area, trying to identify the 
differences in the 2 types of network we have. For that we created Basic and 
Advanced Network Topology classes. The network topology classes are responsible 
by invoking the Apply/Setup/Create/Save rules that were previously done by the 
[Vpc]VirtualNetworkAppliance. A topology instance is retrieved via a context 
object that is injected in the [Vpc]VirtualElement. The context object will 
return the most appropriate topology instance based on the Network Type, which 
is defined in the Data Centre. That was the first step towards the refactor.

From the topology class we reach the Rule Applier implementation that will 
be used to do all the rule setup preparation (i.e. invoke DAOs and prepare the 
command object). The RuleApplier interface was extracted from the 
VirtualNetworkApplianceManagerImpl, where it use to be an inner interface. For 
each anonymous implementation of the RuleApplier we created a concrete class. 
The rules are used as elements of a Visitor class, which will perform some 
extra logic, depending on the rule it's visiting, and call the send commands to 
router method. The latter has also been extracted from the 
VirtualNetworkApplianceManagerImpl and is now in a new helper class: 
NetworkHelperImpl.

The visitor has been used because we were aiming to split the 
responsibility and also because the way the RuleApplier was implemented before, 
it was clear that every command sent to the router was following a 2-steps 
approach: gather information to create the commands, apply some logic to send 
to the router. For those reason we implemented the visitor pattern. Since we 
already had the Basic/Advanced Network Topology classes, we created 2 concrete 
classes to visit the rules: Basic/Advanced Network Visitors. Both classes 
extend the abstract class NetworkTopologyVisitor, which defines all the visit 
methods per type of rule. By doing so, we can use the same rule and separate 
the logic based on the type of visitor that we have - Basic or Advanced.

Continuing on the refactor, we also added some helper classes for the 
"getSomething" related methods. Following this approach we ended up having the 
following classes:

• NetworkHelper (interface)
• N

Jenkins build is still unstable: simulator-singlerun #327

2014-09-09 Thread jenkins
See 



Marvin test cleanup

2014-09-09 Thread Alex Brett
Hello all,

At the moment we have a lot of Marvin tests that follow a pattern that looks 
roughly like this:

1. Setup some resources (e.g. accounts, service offerings, VMs etc)
2. Add the resources to a list in the testcase (often called self.cleanup)
3. Do the test(s)
4. Call cleanup_resources with the list of resources from 2

(obviously in some cases resources get created/allocated during the actual test 
rather than in setup, but it's a similar principle)

In theory this is fine, however there are a number of cases where resources are 
being created and then not added to the cleanup list, which results in things 
being 'left behind', potentially using up resources which may then affect 
future tests. For example I'm currently attempting to run various tests in 
parallel (to speed up execution), and I'm hitting some issues I believe to be 
caused by this.

The thought that occurs to me here is do we actually need the testcase to have 
to manually add resources to a list to cleanup, with the inherent risk of 
resources getting missed etc - could we not make this something the framework 
does for us (at least by default, with the option to override the behaviour if 
needed).

I've got some ideas as to how this could be done (one example that's a bit of a 
layer violation but might be acceptable would be to wrap/extend the apiClient 
to have a method that can be called on it from the various object create 
methods to add the resulting object for cleanup), but before I go ahead and 
start trying to prototype something I wanted to see if anybody had any reasons 
why this sort of automatic cleanup behaviour might be a bad idea or has 
investigated anything similar in the past?

Cheers,
Alex



Build failed in Jenkins: build-master-noredist #3493

2014-09-09 Thread jenkins
See 

Changes:

[anthony.xu] removed unused class

--
[...truncated 5058 lines...]
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-md5/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-md5-4.5.0-SNAPSHOT.jar
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-md5/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-md5-4.5.0-SNAPSHOT.pom
[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - User Authenticator Plain Text 
4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Building jar: 

[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-plaintext/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-plaintext-4.5.0-SNAPSHOT.jar
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-plaintext/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-plaintext-4.5.0-SNAPSHOT.pom
[INFO] 
[INFO] 

Jenkins build is still unstable: simulator-singlerun #328

2014-09-09 Thread jenkins
See 



Re: Marvin test cleanup

2014-09-09 Thread Prasanna Santhanam
On Wed, Sep 10, 2014 at 5:42 AM, Alex Brett  wrote:
> Hello all,
>
> At the moment we have a lot of Marvin tests that follow a pattern that looks 
> roughly like this:
>
> 1. Setup some resources (e.g. accounts, service offerings, VMs etc)
> 2. Add the resources to a list in the testcase (often called self.cleanup)
> 3. Do the test(s)
> 4. Call cleanup_resources with the list of resources from 2
>
> (obviously in some cases resources get created/allocated during the actual 
> test rather than in setup, but it's a similar principle)
>
> In theory this is fine, however there are a number of cases where resources 
> are being created and then not added to the cleanup list, which results in 
> things being 'left behind', potentially using up resources which may then 
> affect future tests. For example I'm currently attempting to run various 
> tests in parallel (to speed up execution), and I'm hitting some issues I 
> believe to be caused by this.
>
> The thought that occurs to me here is do we actually need the testcase to 
> have to manually add resources to a list to cleanup, with the inherent risk 
> of resources getting missed etc - could we not make this something the 
> framework does for us (at least by default, with the option to override the 
> behaviour if needed).
>
> I've got some ideas as to how this could be done (one example that's a bit of 
> a layer violation but might be acceptable would be to wrap/extend the 
> apiClient to have a method that can be called on it from the various object 
> create methods to add the resulting object for cleanup), but before I go 
> ahead and start trying to prototype something I wanted to see if anybody had 
> any reasons why this sort of automatic cleanup behaviour might be a bad idea 
> or has investigated anything similar in the past?
>

+1 - the cleanup of cloud resources is nasty especially when VMs and
billable entities are left behind. Many tests that follow fail as a
result of exhausted capacity. The nose framework sort of imposes the
setUp - test - tearDown model of test case authoring and perhaps led
us down that path of accumulating entities in cleanup lists. It would
be elegant if you can identify the corresponding Object.destroy method
and add the necessary finalizer automatically during create. Sort of
like the way unittest2 and py.test do it. The ACS API is quite
expressive and broken down easily into this model where you can figure
out the entity/resource you operate upon. Each entity has its API for
creation and a corresponding one for listing and destroy/deletion. If
that's too involved the overloaded apiClient works fine too.

> Cheers,
> Alex
>


Jenkins build is still unstable: simulator-singlerun #329

2014-09-09 Thread jenkins
See 



[GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634: added more de...

2014-09-09 Thread karuturi
GitHub user karuturi opened a pull request:

https://github.com/apache/cloudstack-docs-admin/pull/20

CLOUDSTACK-6634: added more details for ldap ssl



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/karuturi/cloudstack-docs-admin patch-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack-docs-admin/pull/20.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #20


commit 34ec5acc24ec4f665aa355414483e0bb10432178
Author: Rajani Karuturi 
Date:   2014-09-10T05:38:56Z

CLOUDSTACK-6634: added more details for ldap ssl




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[TRAVIS] - failing for pull request #18

2014-09-09 Thread Wilder Rodrigues
Hi guys,

Last night I sent a pull request, after rebasing and fixing some conflicts in 
VirtualNetworkApplianceManagerImpl class. Unfortunately, the 3 builds are not 
okay on Travis. For the first job, I got the following:


Still running (30 of 30): ./tools/travis/before_script.sh

Timeout (30 minutes) reached. Terminating "./tools/travis/before_script.sh"

The command "travis_wait 30 ./tools/travis/before_script.sh" failed and exited 
with 1 during .

The second job is a bit stranger than that because all the tests that I 
actually executed on our environment, and passed, just failed on that job:

test_privategw_acl
test_01_reset_vm_on_reboot
test_routers

Concerning the 3rd job, same story: tests that have passed are now failing:

test_service_offerings
test_vpc_vpn

Stil on the 3rd job I got problems with test_volumes and test_templates. The 
test_vm_life_cycle is still failing for everybody.

My test environment is the following:


· XenServer (host) running in our VMWare zone (which is managed by 
CloudStack).

o   MySQL

o   Secondary Storage

o   Local Primary Storage

· Management Server

o   Debian 7 VM (inside Virtual Box)

o   Pre/Post routing to allow my Debian to get connected to the host via its 
public IPs

o   SSH reverse tunnel to allow communication between the host and the 
management server (for the Agent)

· Tests

o   I create a separate project with a bunch of ACS Marving tests

§  Changed storage type in my tests from "shared" to "local"

§  Changed DataCenter configuration

· IP ranges

§  Changed VPCs CIDRs and Network Gatways

· To cope with my IP ranges

· All the tests below are passing:

o   test_01_create_account

o   test_01_add_vm_to_subdomain

o   test_DeleteDomain

o   test_forceDeleteDomain

o   test_updateAdminDetails

o   test_updateDomainAdminDetails

o   test_updateUserDetails

o   test_LoginApiDomain

o   test_LoginApiUuidResponse

o   test_privategw_acl

o   test_01_reset_vm_on_reboot

o   test_03_restart_network_cleanup

o   test_05_router_basic

o   test_06_router_advanced

o   test_07_stop_router

o   test_08_start_router

o   test_09_reboot_router

o   test_01_create_service_offering

o   test_02_edit_service_offering

o   test_03_delete_service_offering

o   test_01_start_stop_router_after_addition_of_one_guest_network

o   test_02_reboot_router_after_addition_of_one_guest_network

o   test_04_chg_srv_off_router_after_addition_of_one_guest_network

o   test_05_destroy_router_after_addition_of_one_guest_network

o   test_01_stop_start_router_after_creating_vpc

o   test_02_reboot_router_after_creating_vpc

o   test_04_change_service_offerring_vpc

o   test_05_destroy_router_after_creating_vpc

o   test_vpc_remote_access_vpn

o   test_vpc_site2site_vpn

Unfortunately, Travis does not give much help about what is going on, it only 
says "exception". Is there anyone out there that could give some help?

The pull request is: VPC and Virtual Network Managers refactor. Part of the VPC 
Redundant Router work #18

Thanks in advance.

Cheers,
Wilder


Re: Marvin test cleanup

2014-09-09 Thread Gaurav Aradhye
One thing that is important to consider here is "order in which we cleanup
resources". Many times, cleanup operation fails if the resource can't be
deleted unless few other resources are deleted (child/dependent).

Alex, if you think if this too can be handled efficiently with the
prototype you are likely to build, then it will be great! If not, please
consider this too.

Also, please let me know if I can help. I had started fixing the cleanup
problems in test cases (identifying the missing items, fixing the cleanup
orders), but due to other priorities I could not finish it. I will take it
up soon anyway to complete it. But this change in framework itself to
accommodate cleanup operation will be great!

Regards,
Gaurav

On Wed, Sep 10, 2014 at 9:13 AM, Prasanna Santhanam  wrote:

> On Wed, Sep 10, 2014 at 5:42 AM, Alex Brett  wrote:
> > Hello all,
> >
> > At the moment we have a lot of Marvin tests that follow a pattern that
> looks roughly like this:
> >
> > 1. Setup some resources (e.g. accounts, service offerings, VMs etc)
> > 2. Add the resources to a list in the testcase (often called
> self.cleanup)
> > 3. Do the test(s)
> > 4. Call cleanup_resources with the list of resources from 2
> >
> > (obviously in some cases resources get created/allocated during the
> actual test rather than in setup, but it's a similar principle)
> >
> > In theory this is fine, however there are a number of cases where
> resources are being created and then not added to the cleanup list, which
> results in things being 'left behind', potentially using up resources which
> may then affect future tests. For example I'm currently attempting to run
> various tests in parallel (to speed up execution), and I'm hitting some
> issues I believe to be caused by this.
> >
> > The thought that occurs to me here is do we actually need the testcase
> to have to manually add resources to a list to cleanup, with the inherent
> risk of resources getting missed etc - could we not make this something the
> framework does for us (at least by default, with the option to override the
> behaviour if needed).
> >
> > I've got some ideas as to how this could be done (one example that's a
> bit of a layer violation but might be acceptable would be to wrap/extend
> the apiClient to have a method that can be called on it from the various
> object create methods to add the resulting object for cleanup), but before
> I go ahead and start trying to prototype something I wanted to see if
> anybody had any reasons why this sort of automatic cleanup behaviour might
> be a bad idea or has investigated anything similar in the past?
> >
>
> +1 - the cleanup of cloud resources is nasty especially when VMs and
> billable entities are left behind. Many tests that follow fail as a
> result of exhausted capacity. The nose framework sort of imposes the
> setUp - test - tearDown model of test case authoring and perhaps led
> us down that path of accumulating entities in cleanup lists. It would
> be elegant if you can identify the corresponding Object.destroy method
> and add the necessary finalizer automatically during create. Sort of
> like the way unittest2 and py.test do it. The ACS API is quite
> expressive and broken down easily into this model where you can figure
> out the entity/resource you operate upon. Each entity has its API for
> creation and a corresponding one for listing and destroy/deletion. If
> that's too involved the overloaded apiClient works fine too.
>
> > Cheers,
> > Alex
> >
>


Re: Review Request 25292: CLOUDSTACK-6898: [Hyper-V] fixed rdp console freezing during reboot.

2014-09-09 Thread Devdeep Singh

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

Ship it!


Committed in 4c9507d21e82e9c55ac64e75165329565ec6a14f to master

- Devdeep Singh


On Sept. 3, 2014, 9:07 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25292/
> ---
> 
> (Updated Sept. 3, 2014, 9:07 a.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-6898
> https://issues.apache.org/jira/browse/CLOUDSTACK-6898
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> 
> 
> Console was freezing because we read data from socket in blocking mode.
> During reboot it was blocking infintely.
> To fix issue, now we are reading data in non-blocking mode.
> In non-blocking mode I set the timeout to 5 seconds.
> 
> 
> Diffs
> -
> 
>   services/console-proxy-rdp/rdpconsole/src/main/java/streamer/SyncLink.java 
> 94281d2 
>   
> services/console-proxy-rdp/rdpconsole/src/main/java/streamer/apr/AprSocketSource.java
>  0298349 
> 
> Diff: https://reviews.apache.org/r/25292/diff/
> 
> 
> Testing
> ---
> 
> tested console is working fine during reboot. Now it freezes 5-6 seconds 
> before becoming responsive again.
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 25427: fixed few coverity issues

2014-09-09 Thread Devdeep Singh

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

Ship it!


Committed in a400eecf6c868c05bacee21689238a1e0ce2 to master

- Devdeep Singh


On Sept. 8, 2014, 8:57 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25427/
> ---
> 
> (Updated Sept. 8, 2014, 8:57 a.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-7503
> https://issues.apache.org/jira/browse/CLOUDSTACK-7503
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> fixed few coverity issues. Not logging the proper string.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
>  76f8e75 
> 
> Diff: https://reviews.apache.org/r/25427/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 25426: CLOUDSTACK-7370: Fixed password visible in plain text if password passed is in the end in url

2014-09-09 Thread Devdeep Singh

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


Change looks ok but Utils unit test is failing. Kindly address the issue with 
failed unit tests too.

- Devdeep Singh


On Sept. 8, 2014, 8:22 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25426/
> ---
> 
> (Updated Sept. 8, 2014, 8:22 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-7370
> https://issues.apache.org/jira/browse/CLOUDSTACK-7370
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> updated the regular expression which is used to clean password
> 
> 
> Diffs
> -
> 
>   utils/src/com/cloud/utils/StringUtils.java 17b8fc0 
> 
> Diff: https://reviews.apache.org/r/25426/diff/
> 
> 
> Testing
> ---
> 
> tested by passing affected strings through regular expressions.
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Build failed in Jenkins: build-master #1577

2014-09-09 Thread jenkins
See 

Changes:

[Devdeep Singh] CLOUDSTACK-6898: [Hyper-V] fixed rdp console freezing during 
reboot.

[Devdeep Singh] CLOUDSTACK-7503: Fixed few coverity issues

--
[...truncated 3885 lines...]
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] Surefire report directory: 


---
 T E S T S
---
Running com.cloud.server.auth.MD5UserAuthenticatorTest
log4j:WARN No appenders could be found for logger 
(com.cloud.server.auth.MD5UserAuthenticator).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.574 sec

Results :

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - User Authenticator Plain Text 
4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - User Authenticator SAML2 
4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-user-authenticator-saml2 ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-user-authenticator-saml2 ---
[INFO] Starting audit...
Audit done.

[GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634: added more de...

2014-09-09 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack-docs-admin/pull/20


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---