[GitHub] cloudstack pull request: CLOUDSTACK-7688, CLOUDSTACK-7747: restric...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7688, CLOUDSTACK-7747: restricted various operations for VM w...

...ith VM snapshots which breaks VM snapshots.

 Now they are informed that they cannot perform the operation.
 To perform operation they have to remove VM snapshots of VM.

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

$ git pull https://github.com/anshul1886/cloudstack-1 vmsnapshotfixes

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

https://github.com/apache/cloudstack/pull/26.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 #26


commit a4d8215ebba6009fc4b17ef7b61ff51238f0f404
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-10-09T05:07:21Z

CLOUDSTACK-7688, CLOUDSTACK-7747: restricted various operations for VM with 
VM snapshots which breaks VM snapshots.
 Now they are informed that they cannot perform the operation.
 To perform operation they have to remove VM snapshots of VM.




---
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 pull request: CLOUDSTACK-7767: fixed events are not gen...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7767: fixed events are not generated for snapshot creation



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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-7767

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

https://github.com/apache/cloudstack/pull/27.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 #27


commit d8b46b81b946672ab6836f13ea0389cee63321d8
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-10-28T06:19:30Z

CLOUDSTACK-7767: fixed events are not generated for snapshot creation




---
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 pull request: CLOUDSTACK-7541: Added restriction to not...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7541: Added restriction to not allow custom disk offering with 
disksize

Added restriction to not allow custom disk offering with disksize

UI doesn't allow but with API we were able to create custom disk offering 
with disk size
which was causing this issue

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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-7541

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

https://github.com/apache/cloudstack/pull/28.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 #28


commit faf037f42a662660838f6ff129b11b8e52421a7e
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-10-22T04:29:42Z

CLOUDSTACK-7541: Added restriction to not allow custom disk offering with 
disksize
UI doesn't allow but with API we were able to create custom disk offering 
with disk size
which was causing this issue




---
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 pull request: CLOUDSTACK-7758: Fixed although api calls...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7758: Fixed although api calls are failing, event tab shows them 
as successful

Fixed although api calls are failing, event tab shows them as successful

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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-7758

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

https://github.com/apache/cloudstack/pull/29.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 #29


commit a4f7f682785fc76288d3544e9f163a12ca032282
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-10-21T09:27:16Z

CLOUDSTACK-7758: Fixed although api calls are failing, event tab shows them 
as successful




---
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 pull request: CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed d...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed deployment planner stuck in 
infinite loop

CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed deployment planner stuck in 
infinite loop.

If we create VM with shared service offering and attach disk with local 
disk offering,
and one of storage pool is full(cannot be allocated) and other is not full 
then
we are not putting the cluster in avoid list which is causing this infinite 
loop.

Fixed by putting the cluster in avoid list even if one of the storage pool 
is full(cannot be allocated)

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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-7703

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

https://github.com/apache/cloudstack/pull/30.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 #30


commit 4705933a6459c1b25e71da891a6f736c0a736836
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-10-20T10:34:25Z

CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed deployment planner stuck in 
infinite loop.
If we create VM with shared service offering and attach disk with local 
disk offering,
and one of storage pool is full(cannot be allocated) and other is not full 
then
we are not putting the cluster in avoid list which is causing this infinite 
loop.

Fixed by putting the cluster in avoid list even if one of the storage pool 
is full(cannot be allocated)




---
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 pull request: CLOUDSTACK-7620: Added SNMP MIB file for ...

2014-10-30 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-7620: Added SNMP MIB file for snmp-alerts plugin



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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-7620

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

https://github.com/apache/cloudstack/pull/31.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 #31


commit 96e7081b64c452171af4d7f3b6e62fd2018c
Author: Anshul Gangwar anshul.gang...@citrix.com
Date:   2014-09-24T07:20:16Z

CLOUDSTACK-7620: Added SNMP MIB file for snmp-alerts plugin




---
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.
---


Review Request 27377: CLOUDSTACK-7408: sshClient.py - removing call to paramiko function load_host_keys(). This is used to load host keys from local file which is undesirable while running test cases

2014-10-30 Thread Gaurav Aradhye

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

Review request for cloudstack and SrikanteswaraRao Talluri.


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


Repository: cloudstack-git


Description
---

This also reverts the commit 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=e921ec6ec79c50096d58264d60c15091969ff888


Description:
The function load_host_keys is typically used to load the host keys from a 
local file (If user has any such file, than the default system host key file). 
If this function is called with a local host keys file, then keys from this 
file will be matched while connecting to any host. If the key is not present, 
and if AutoAddPolicy is present, then the key will be saved back to the local 
host key file.

While running test cases, we don't have any local host key file which we need 
to provide, nor we want to save any new keys to local host key file.

In the code, we already have the AutoAddPolicy defined for missing keys. Hence 
in our case, we are not loading any host keys file. Hence whenever we conect to 
a host, then the host key will be saved in memory (not file) due to 
AutoAddPolicy and the connection will be successful provided that the password 
or the private key file is correctly provided for the host. Hence it is not 
necessary to call load_host_keys function.


Diffs
-

  tools/marvin/marvin/lib/base.py 77faeeb 
  tools/marvin/marvin/lib/utils.py b58b59d 
  tools/marvin/marvin/sshClient.py f027890 

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


Testing
---

Yes. Ran the test_reset_ssh_keypair test suite and all the tests passed.

Log:
Test Reset SSH keys for VM  already having SSH key ... === TestName: 
test_01_reset_ssh_keys | Status : SUCCESS ===
ok
Reset SSH keys for VM  created from password enabled template and ... === 
TestName: test_02_reset_ssh_key_password_enabled_template | Status : SUCCESS ===
ok
Reset SSH key for VM  having no SSH key ... === TestName: 
test_03_reset_ssh_with_no_key | Status : SUCCESS ===
ok
Reset SSH keys for VM  created from password enabled template and ... === 
TestName: test_04_reset_key_passwd_enabled_no_key | Status : SUCCESS ===
ok
Reset SSH keys for VM  already having SSH key when VM is in running ... === 
TestName: test_05_reset_key_in_running_state | Status : SUCCESS ===
ok
Reset SSH keys for VM  created from password enabled template and ... === 
TestName: test_06_reset_key_passwd_enabled_vm_running | Status : SUCCESS ===
ok
Verify API resetSSHKeyForVirtualMachine with incorrect parameters ... === 
TestName: test_07_reset_keypair_invalid_params | Status : SUCCESS ===
ok
Verify API resetSSHKeyForVirtualMachine for non admin non root ... === 
TestName: test_01_reset_keypair_normal_user | Status : SUCCESS ===
ok
Verify API resetSSHKeyForVirtualMachine for domain admin non root ... === 
TestName: test_02_reset_keypair_domain_admin | Status : SUCCESS ===
ok
Verify API resetSSHKeyForVirtualMachine for domain admin root ... === TestName: 
test_03_reset_keypair_root_admin | Status : SUCCESS ===
ok


Thanks,

Gaurav Aradhye



Re: Review Request 27278: CLOUDSTACK-7805: test_vpc_vms_deployment.py - Removing invalid steps from test case

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


a4174d3e91fd4b090de2f9d1688a862fe46f99e5 4.5

- SrikanteswaraRao Talluri


On Oct. 28, 2014, 9:12 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27278/
 ---
 
 (Updated Oct. 28, 2014, 9:12 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7805
 https://issues.apache.org/jira/browse/CLOUDSTACK-7805
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 The test case fails because there are two ip addresses acquired in a network. 
 The step in the test case matches the first element in the list with the 2nd 
 ip addresses. It can't be guaranteed to succeed because the elements can be 
 listed in any order.
 
 Basically, the steps are invalid and even unnecessary. Removing these steps 
 from test case.
 
 
 Diffs
 -
 
   test/integration/component/test_vpc_vms_deployment.py 54586d0 
 
 Diff: https://reviews.apache.org/r/27278/diff/
 
 
 Testing
 ---
 
 N/A
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 27377: CLOUDSTACK-7408: sshClient.py - removing call to paramiko function load_host_keys(). This is used to load host keys from local file which is undesirable while running test ca

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


e87da186f3b094e76a174713d6933f83d1c00854 4.5

- SrikanteswaraRao Talluri


On Oct. 30, 2014, 9:08 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27377/
 ---
 
 (Updated Oct. 30, 2014, 9:08 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7408
 https://issues.apache.org/jira/browse/CLOUDSTACK-7408
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 This also reverts the commit 
 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=e921ec6ec79c50096d58264d60c15091969ff888
 
 
 Description:
 The function load_host_keys is typically used to load the host keys from a 
 local file (If user has any such file, than the default system host key 
 file). If this function is called with a local host keys file, then keys from 
 this file will be matched while connecting to any host. If the key is not 
 present, and if AutoAddPolicy is present, then the key will be saved back to 
 the local host key file.
 
 While running test cases, we don't have any local host key file which we need 
 to provide, nor we want to save any new keys to local host key file.
 
 In the code, we already have the AutoAddPolicy defined for missing keys. 
 Hence in our case, we are not loading any host keys file. Hence whenever we 
 conect to a host, then the host key will be saved in memory (not file) due to 
 AutoAddPolicy and the connection will be successful provided that the 
 password or the private key file is correctly provided for the host. Hence it 
 is not necessary to call load_host_keys function.
 
 
 Diffs
 -
 
   tools/marvin/marvin/lib/base.py 77faeeb 
   tools/marvin/marvin/lib/utils.py b58b59d 
   tools/marvin/marvin/sshClient.py f027890 
 
 Diff: https://reviews.apache.org/r/27377/diff/
 
 
 Testing
 ---
 
 Yes. Ran the test_reset_ssh_keypair test suite and all the tests passed.
 
 Log:
 Test Reset SSH keys for VM  already having SSH key ... === TestName: 
 test_01_reset_ssh_keys | Status : SUCCESS ===
 ok
 Reset SSH keys for VM  created from password enabled template and ... === 
 TestName: test_02_reset_ssh_key_password_enabled_template | Status : SUCCESS 
 ===
 ok
 Reset SSH key for VM  having no SSH key ... === TestName: 
 test_03_reset_ssh_with_no_key | Status : SUCCESS ===
 ok
 Reset SSH keys for VM  created from password enabled template and ... === 
 TestName: test_04_reset_key_passwd_enabled_no_key | Status : SUCCESS ===
 ok
 Reset SSH keys for VM  already having SSH key when VM is in running ... === 
 TestName: test_05_reset_key_in_running_state | Status : SUCCESS ===
 ok
 Reset SSH keys for VM  created from password enabled template and ... === 
 TestName: test_06_reset_key_passwd_enabled_vm_running | Status : SUCCESS ===
 ok
 Verify API resetSSHKeyForVirtualMachine with incorrect parameters ... === 
 TestName: test_07_reset_keypair_invalid_params | Status : SUCCESS ===
 ok
 Verify API resetSSHKeyForVirtualMachine for non admin non root ... === 
 TestName: test_01_reset_keypair_normal_user | Status : SUCCESS ===
 ok
 Verify API resetSSHKeyForVirtualMachine for domain admin non root ... === 
 TestName: test_02_reset_keypair_domain_admin | Status : SUCCESS ===
 ok
 Verify API resetSSHKeyForVirtualMachine for domain admin root ... === 
 TestName: test_03_reset_keypair_root_admin | Status : SUCCESS ===
 ok
 
 
 Thanks,
 
 Gaurav Aradhye
 




[GitHub] cloudstack pull request: CLOUDSTACK-7688, CLOUDSTACK-7747: restric...

2014-10-30 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/26#issuecomment-61065599
  
Tests passed, merging this on master. Thanks for using PR.


---
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.
---


Jenkins build is back to normal : build-4.5 #60

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/build-4.5/60/changes



[GitHub] cloudstack pull request: CLOUDSTACK-7767: fixed events are not gen...

2014-10-30 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/27#issuecomment-61066104
  
Thanks, merging since CI build passed.


---
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 pull request: CLOUDSTACK-7767: fixed events are not gen...

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

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


---
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 26640: CLOUDSTACK-7698: Do not create nat rule/acquire public IP by default while deploying VM if it is not necessary

2014-10-30 Thread SrikanteswaraRao Talluri

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


Applying: CLOUDSTACK-7698: Do not create nat rule/acquire public IP by default 
while deploying VM if it is not necessary
error: patch failed: test/integration/smoke/test_nic.py:19
error: test/integration/smoke/test_nic.py: patch does not apply
Patch failed at 0001 CLOUDSTACK-7698: Do not create nat rule/acquire public IP 
by default while deploying VM if it is not necessary
The copy of the patch that failed is found in:
   /Users/talluri/asf/cloudstack/.git/rebase-apply/patch
When you have resolved this problem, run git am --continue.
If you prefer to skip this patch, run git am --skip instead.
To restore the original branch and stop patching, run git am --abort.

- SrikanteswaraRao Talluri


On None, Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26640/
 ---
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7698
 https://issues.apache.org/jira/browse/CLOUDSTACK-7698
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 When parameter mode is passed while deploying VM, following code is 
 executed to make VM accessible via SSH.
 
 In Advanced zone,
 Public IP is acquired in network, Firewall is opened, NAT rule is created
 
 In Basic zone,
 New security group is created is default group is not present, else VM is 
 added to default security group which has ingress rule enabled.
 
 In many test cases, mode is passed even when Making Vm accessible via SSH 
 is not necessary. Hence removing this parameter in such cases will prevent 
 test cases from acquiring resources which it doesn't need.
 
 
 Diffs
 -
 
   test/integration/component/test_add_remove_network.py bc9be6f 
   test/integration/component/test_escalations_isos.py 4e818a5 
   test/integration/component/test_explicit_dedication.py 71c4141 
   test/integration/component/test_implicit_planner.py b538a3d 
   test/integration/component/test_lb_secondary_ip.py 23ff1ee 
   test/integration/component/test_multiple_ip_ranges.py 32c39f7 
   test/integration/component/test_portable_ip.py cf0cb3b 
   test/integration/component/test_recurring_snapshots.py 58dd769 
   test/integration/component/test_snapshots_improvement.py 6638ddd 
   test/integration/component/test_stopped_vm.py 1a928cc 
   test/integration/component/test_storage_motion.py acbf92d 
   test/integration/component/test_tags.py c5a8ced 
   test/integration/component/test_templates.py 769848d 
   test/integration/component/test_usage.py 81cce48 
   test/integration/smoke/test_nic.py 488ed0a 
   test/integration/smoke/test_reset_vm_on_reboot.py 668c77f 
   test/integration/smoke/test_scale_vm.py 3a2983e 
   test/integration/smoke/test_templates.py 4153ca3 
 
 Diff: https://reviews.apache.org/r/26640/diff/
 
 
 Testing
 ---
 
 Have tested for syntax and import errors.
 Have made changes only in those test cases which don't use the default public 
 IP for SSH which would otherwise get associated with the VM when mode was 
 passed. (Hence only the test cases which unnecessarily passing the mode 
 parameter)
 
 
 Thanks,
 
 Gaurav Aradhye
 




[GitHub] cloudstack pull request: CLOUDSTACK-7688, CLOUDSTACK-7747: restric...

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

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


---
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 pull request: debian: Fix changelog to use 4.4.1 releas...

2014-10-30 Thread bhaisaab
Github user bhaisaab closed the pull request at:

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


---
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 26442: CLOUDSTACK-7685: Fixed copy template method call in test_escalations_template.py. Removed unbound method of the same name as bound method from base l ibrary and changed metho

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


a1e7feead91b22350f96114add4bf8d75a74b8b1 4.5

- SrikanteswaraRao Talluri


On Oct. 20, 2014, 5:39 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26442/
 ---
 
 (Updated Oct. 20, 2014, 5:39 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7685
 https://issues.apache.org/jira/browse/CLOUDSTACK-7685
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Error details:
 unbound method copy() must be called with Template instance as first argument 
 (got CloudStackAPIClient instance instead)
 
 Reason:
 There are bound and unbound methods copy in Template class with the same 
 name. Apparently when bound and unbound methods with same name are defined, 
 call to unbound method fails.
 
 Solution:
 Keep only bound method.
 
 Please note that I have ensured no test case makes call to unbound method 
 (Template.copy).
 
 
 Diffs
 -
 
   test/integration/component/test_escalations_templates.py 78028bc 
   tools/marvin/marvin/lib/base.py 77faeeb 
 
 Diff: https://reviews.apache.org/r/26442/diff/
 
 
 Testing
 ---
 
 Tested with python command/pyflakes.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 27273: CLOUDSTACK-7801: test_volumes.py - Fixed import and pep8 issues

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


b1ab888bf0ae1c5f63e515de28be482d18bb884c 4.5

- SrikanteswaraRao Talluri


On Oct. 28, 2014, 7:04 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27273/
 ---
 
 (Updated Oct. 28, 2014, 7:04 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: cloudstack-7801
 https://issues.apache.org/jira/browse/cloudstack-7801
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 test_volumes.py has been introduced with import errors during commit 
 5c2094eec1408494586e48d38d9076b31ca3e5f5
 
 Fixed the import errors and also all the pep8 issues.
 
 
 Diffs
 -
 
   test/integration/component/test_volumes.py 36a910b 
 
 Diff: https://reviews.apache.org/r/27273/diff/
 
 
 Testing
 ---
 
 Yes.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 27276: CLOUDSTACK-7802: test_vpc_network.py - Increasing polling period for network state check

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


af510eb656eac08eb7f299ad970d49d9f3d96eb3 4.5

- SrikanteswaraRao Talluri


On Oct. 28, 2014, 8:19 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27276/
 ---
 
 (Updated Oct. 28, 2014, 8:19 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7802
 https://issues.apache.org/jira/browse/CLOUDSTACK-7802
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Test case fails with Network state should change to Allocated, it is 
 Implemented after all the VMs in the network are stopped and network state 
 is checked.
 
 We are already waiting for 600 seconds, but still it is taking much longer 
 for network state to change.
 
 Changes:
 Increased the retries from 10 to 20.
 
 
 Diffs
 -
 
   test/integration/component/test_vpc_network.py 096315c 
 
 Diff: https://reviews.apache.org/r/27276/diff/
 
 
 Testing
 ---
 
 Yes. Tested on KVM.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 26438: CLOUDSTACK-7682: Fixed usage event issue. Used api client of account for which we are checking the usage

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


b9fed951ef6498c828f07ab120a242fcf15e14f6 4.5

- SrikanteswaraRao Talluri


On Oct. 17, 2014, 9:51 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26438/
 ---
 
 (Updated Oct. 17, 2014, 9:51 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7682
 https://issues.apache.org/jira/browse/CLOUDSTACK-7682
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 The events TEMPLATE.CREATE and TEMPLATE.DELETE does not exist in usage event 
 table against that particular account because the template is not created 
 using the api client of the account and hence the template belongs to the 
 root admin account.
 
 Changes:
 Use the api client of the account against which we are checking the usage 
 while creating and deleting the template.
 
 Have not addressed the pep8 issues already present in the file because that 
 will mask the changes to fix the issue. I will be happy to add a separate 
 patch for it if required. Considering that many files need pep8 fixes, a 
 separate task for this should be created and addressed.
 
 
 Diffs
 -
 
   test/integration/component/test_usage.py 81cce48 
 
 Diff: https://reviews.apache.org/r/26438/diff/
 
 
 Testing
 ---
 
 Tested for syntax and import errors.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 26492: CLOUDSTACK-7691: Fixed 'Exception not raised' issue in test_project_limits.py

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


e2ba1b614c7792856314cfa363f2a4e9cd667065 4.5

- SrikanteswaraRao Talluri


On Oct. 17, 2014, 9:50 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26492/
 ---
 
 (Updated Oct. 17, 2014, 9:50 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7691
 https://issues.apache.org/jira/browse/CLOUDSTACK-7691
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Reason of failure:
 The test case is failing while checking the template limits of the project 
 because the template is not registered with the api client of the project 
 owner account and it is registered with api client of the root admin account.
 
 
 Changes:
 1. Registered template with correct account
 2. Removed import * and included specific imports
 
 
 Diffs
 -
 
   test/integration/component/test_project_limits.py 5f33ad8 
 
 Diff: https://reviews.apache.org/r/26492/diff/
 
 
 Testing
 ---
 
 Tested for syntax and import errors.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 26439: CLOUDSTACK-7684: Fixed Exception not raised issue in secondary storage limits test suite test_ss_max_limits.py

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


dc4b04fbe87048849d51e9e4dbcce566de7f9cf0 4.5

- SrikanteswaraRao Talluri


On Oct. 17, 2014, 9:51 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26439/
 ---
 
 (Updated Oct. 17, 2014, 9:51 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7684
 https://issues.apache.org/jira/browse/CLOUDSTACK-7684
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Test case failed because api client of project owner account is not used 
 (Instead of root admin api client).
 
 
 Diffs
 -
 
   test/integration/component/test_ss_max_limits.py ba886e8 
 
 Diff: https://reviews.apache.org/r/26439/diff/
 
 
 Testing
 ---
 
 Tested for syntax and import errors.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 26868: CLOUDSTACK-7728: Adding steps to existing test case to identify product issue

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


aa1e26081208b5d6c758a6efbbf3b62ab2d6720d 4.5

- SrikanteswaraRao Talluri


On Oct. 17, 2014, 9:19 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26868/
 ---
 
 (Updated Oct. 17, 2014, 9:19 a.m.)
 
 
 Review request for cloudstack, Jayapal Reddy and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7728
 https://issues.apache.org/jira/browse/CLOUDSTACK-7728
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Adding steps to verify a recently fixed product issue.
 
 Steps already present:
 1. Create isolated network with egress policy allow
 2. Deploy VM in network
 3. Check egress traffic from VM, it should be allowed
 
 Steps added:
 Reboot router and then verify egress traffic. Egress traffic should still be 
 allowed.
 
 
 Diffs
 -
 
   test/integration/component/test_egress_fw_rules.py 12bade4 
 
 Diff: https://reviews.apache.org/r/26868/diff/
 
 
 Testing
 ---
 
 Not tested.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Re: Review Request 26493: CLOUDSTACK-7692: Fixed template register issue in test_project_usage.py

2014-10-30 Thread SrikanteswaraRao Talluri

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

Ship it!


4f297efef5627f04348a6b9ac550bef2364cb661 4.5

- SrikanteswaraRao Talluri


On Oct. 17, 2014, 9:49 a.m., Gaurav Aradhye wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26493/
 ---
 
 (Updated Oct. 17, 2014, 9:49 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-7692
 https://issues.apache.org/jira/browse/CLOUDSTACK-7692
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Template needs to be registered with api client of project owner account.
 
 
 Diffs
 -
 
   test/integration/component/test_project_usage.py 3262e40 
 
 Diff: https://reviews.apache.org/r/26493/diff/
 
 
 Testing
 ---
 
 Tested for syntax and import errors.
 
 
 Thanks,
 
 Gaurav Aradhye
 




Snapshotting broken on Centos 6.5

2014-10-30 Thread Andrija Panic
Hi guys,

snapshoting is broken on CentOS 6.5 - because RedHat removed the -s flag
within qemu-img tool in the 6.5 binaries - and this -s flag is still
being used in latest ACS 4.4.1.

Is this going to be fixed any time soon, or what are the plan's for this ?

-- 

Andrija Panić


Re: [PROPOSAL] Move to github PR only during moratorium on commit

2014-10-30 Thread Srikanteswararao Talluri
Hey Folks,

After all these discussions, What is the direction to commit the code to
master and 4.5? I have some commits on 4.5 which I would like to have them
on master too. How to go about it?

Thanks,
~Talluri

On 23/10/14 11:47 am, Animesh Chaturvedi animesh.chaturv...@citrix.com
wrote:



 -Original Message-
 From: sebgoa [mailto:run...@gmail.com]
 Sent: Saturday, October 18, 2014 2:00 AM
 To: dev@cloudstack.apache.org
 Subject: [PROPOSAL] Move to github PR only during moratorium on
 commit
 
 After [1] I would like to officially bring up the following proposal.
 
 [Proposal]
 
 All commits come through github PR, *even* for committers. We declare a
 moratorium period (agreed suspension of activity) during which direct
 commit to master is forbidden.
 Only the master RM is allowed to merge PR in master (we define a master
 RM). If direct commit to master is done, master RM reverts without
 warning. Same for 4.5 and 4.4. branches.
 
 
 This is drastic and I am sure some folks will not like it, but here is
my
 justification for such a measure:
 
 [Reasons]:
 
 Our commit and release processes have so far been based on the idea that
 development happens on master and that a release branch is cut from
 master (unstable development branch). Then a different set of community
 members harden the release branch, QA and bring it to GA level. During
 that time development keeps on going in master.
 
 This is an OK process if we have the luxury of having a QA team and can
 cope with split personality of being developers and release managers.
 
 My point of view is that as a community we cannot afford such a split
brain
 organization and our experience overt the last year proves my point
 (delayed release date, broken builds, features merged without
warning...)
 
 We can avoid this by cutting a release branch from a stable one (from
the
 start), then as you (Daan) have mentioned several times, fix bugs in the
 release branch and merge them back in the stable source of the release
(be
 it master).


[Animesh] Sebastian you have brought up a  good point dependency on QA
team from Citrix is an issue for the project. This was raised in the past
as well and Alex's proposal [1] few months back using CI was in my
opinion is the optimal solution. Why? Because CloudStack is a huge
project and one single person cannot have the full knowledge to safely
review all the code and certainly cannot scale, which CI and automation
can address

Keeping master stable is something no one would argue against and my
point would match the original proposal from Alex. May be we can  have a
staging branch for master and then merging the commit only after they
have passed CI into master. The proposal got derailed and delayed because
as called out at that time community does not want to work with a process
that has a dependency on infrastructure that is not controlled by
community. David and I are working to get the hardware from Citrix into
ACS infra. 

The approach for fixing issues in release branch first and master later
is not practical as we may miss out commits not made into master and
future release regressing without the fixes. Also as the release goes
into hardening cycle there will be a number of fixes which will not be
allowed in release branch but need to be fixes for future, they should
all go in master. Master is the catch all default branch and in my
opinion should get fixes first.

[1] http://markmail.org/thread/xoyjw2sduenlytwm
 
 Feature development need to be done outside master, period. Not only for
 non-committers but also for committers. And merge request need to be
 called. This will help review and avoid surprises.
 
[Animesh] Completely Agreed
 New git workflow were proposed and shutdown, mostly calling for better
CI
 to solve quality issues. CI will not solve our quality issues alone. We
need to
 better police ourselves.

[Animesh] I have already expressed my opinion in favor of CI

 To avoid long discussions, I propose this simple but drastic measure. We
 move all our commits to github PR until 4.5 is out, this stands for
 committers and non-committers, direct commits (especially to master)
 would be reverted immediately.
 
 
 Our development and release process is broken, we cannot continue like
 this, let's fix it.
[Animesh] I  followed up the release process as established by Chip for
4.2 and 4.3 release for which I was the RM and frankly I did not feel it
that way other than the pain of multiple RCs. Folks may not like it but I
am entitled to my opinion and I do feel part of the issues for 4.4 were
self- inflicted because of pre mature code freeze and too early cherry
picking. 
 
 [1] http://markmail.org/thread/xeliefp3oleq3g54
 
 -sebastien



[USAGE][MASTER] java.sql.SQLException: No suitable driver found for jdbc:mysql

2014-10-30 Thread Ilia Shakitko
Hello,

I know this was a known issue and fixed in 4.4 and master (as Sebastian 
mentioned), but it appears again in master.

I am running mgt with jetty-run. Works OK.
I am running usage then (mvn -pl usage -Drun -Dpid=$$). Getting an exception:

com.cloud.utils.exception.CloudRuntimeException: DB Exception on: null
at 
com.cloud.utils.db.GenericDaoBase.searchIncludingRemoved(GenericDaoBase.java:425)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$searchIncludingRemoved$36(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.searchIncludingRemoved(generated)
at 
com.cloud.utils.db.GenericDaoBase.searchIncludingRemoved(GenericDaoBase.java:361)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$searchIncludingRemoved$35(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.searchIncludingRemoved(generated)
at 
com.cloud.utils.db.GenericDaoBase.listIncludingRemovedBy(GenericDaoBase.java:931)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$listIncludingRemovedBy$60(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.listIncludingRemovedBy(generated)
at 
com.cloud.utils.db.GenericDaoBase.listIncludingRemovedBy(GenericDaoBase.java:936)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$listIncludingRemovedBy$62(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.listIncludingRemovedBy(generated)
   at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl.getConfiguration(ConfigurationDaoImpl.java:82)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$getConfiguration$7(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.getConfiguration(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl.getConfiguration(ConfigurationDaoImpl.java:110)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$getConfiguration$8(generated)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
at 
org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.getConfiguration(generated)
at 

[VOTE] Release Apache CloudStack CloudMonkey 5.3.0

2014-10-30 Thread Rohit Yadav
Hi All,

I've created a 5.3.0 release of CloudMonkey, with the following artifacts up 
for a vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=shortlog;h=refs/heads/X.X
Commit: 715630aebdcf18fbb2094893569a0c20fe7b65ab

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

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

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

The CHANGES file notes changes since 5.2.0, major changes in this version:
- Parameter completion works much better, enabled by default
- Current server profile displayed on prompt
- Filtering support for default display output
- New command line arg: -d for display (json, table or default)
- Non-interactive commands from command line are outputted without colors
- CloudMonkey returns non-zero exit code when run on shell and a error is return
  from managment server, the error message is written to stderr

Since the next 72 hour window intersects with the weekend, the vote will be 
open till Tuesday (120 hours). Please test this release and give your vote. 
Thanks.

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. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

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

IaaS Cloud Design  Buildhttp://shapeblue.com/iaas-cloud-design-and-build//
CSForge – rapid IaaS deployment frameworkhttp://shapeblue.com/csforge/
CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
CloudStack Infrastructure 
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/
CloudStack Bootcamp Training Courseshttp://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.


Jenkins build is still unstable: simulator-singlerun #515

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes



Re: Snapshotting broken on Centos 6.5

2014-10-30 Thread Nux!
Hi,

I haven't heard anything about any fixes, but I don't think it's worth spending 
too much time over it, there is a simple workaround.

http://www.nux.ro/archive/2014/01/Taking_KVM_volume_snapshots_with_Cloudstack_4_2_on_CentOS_6_5.html

CentOS 7 will not be affected by it whenever ACS will support it.

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: Andrija Panic andrija.pa...@gmail.com
 To: us...@cloudstack.apache.org, dev@cloudstack.apache.org
 Sent: Thursday, 30 October, 2014 10:14:11
 Subject: Snapshotting broken on Centos 6.5

 Hi guys,
 
 snapshoting is broken on CentOS 6.5 - because RedHat removed the -s flag
 within qemu-img tool in the 6.5 binaries - and this -s flag is still
 being used in latest ACS 4.4.1.
 
 Is this going to be fixed any time soon, or what are the plan's for this ?
 
 --
 
 Andrija Panić


Re: Snapshotting broken on Centos 6.5

2014-10-30 Thread Andrija Panic
Thanks Nux.

On 30 October 2014 12:21, Nux! n...@li.nux.ro wrote:

 Hi,

 I haven't heard anything about any fixes, but I don't think it's worth
 spending too much time over it, there is a simple workaround.


 http://www.nux.ro/archive/2014/01/Taking_KVM_volume_snapshots_with_Cloudstack_4_2_on_CentOS_6_5.html

 CentOS 7 will not be affected by it whenever ACS will support it.

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
  From: Andrija Panic andrija.pa...@gmail.com
  To: us...@cloudstack.apache.org, dev@cloudstack.apache.org
  Sent: Thursday, 30 October, 2014 10:14:11
  Subject: Snapshotting broken on Centos 6.5

  Hi guys,
 
  snapshoting is broken on CentOS 6.5 - because RedHat removed the -s
 flag
  within qemu-img tool in the 6.5 binaries - and this -s flag is still
  being used in latest ACS 4.4.1.
 
  Is this going to be fixed any time soon, or what are the plan's for this
 ?
 
  --
 
  Andrija Panić




-- 

Andrija Panić
--
  http://admintweets.com
--


[GitHub] cloudstack pull request: CLOUDSTACK-7632:Automation for volume lif...

2014-10-30 Thread srikanteswar
GitHub user srikanteswar opened a pull request:

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

CLOUDSTACK-7632:Automation for volume life cycle testPath

Signed-off-by: SrikanteswaraRao Talluri tall...@apache.org

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

$ git pull https://github.com/srikanteswar/cloudstack 4.5

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

https://github.com/apache/cloudstack/pull/32.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 #32


commit fb82a8bd046089e2342cd4ae56aa9ff5a0df9bef
Author: Prashant Kumar Mishra prashantkumar.mis...@citrix.comm
Date:   2014-10-09T06:12:56Z

CLOUDSTACK-7632:Automation for volume life cycle testPath

Signed-off-by: SrikanteswaraRao Talluri tall...@apache.org




---
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 pull request: CLOUDSTACK-7541: Added restriction to not...

2014-10-30 Thread anshul1886
Github user anshul1886 commented on the pull request:

https://github.com/apache/cloudstack/pull/28#issuecomment-61080108
  
There seems to be some setup issue. This commit has made changes into 
custom disk offering. And that test is passing fine. Also error logs also shows 
some setup issue.


usr/local/lib/python2.7/dist-packages/nose-1.3.4-py2.7.egg/nose/plugins/manager.py:395:
 RuntimeWarning: Unable to load plugin marvinPlugin = 
marvin.marvinPlugin:MarvinPlugin: mysql-connector-python=1.1.6
  RuntimeWarning)
Usage: nosetests [options]

nosetests: error: no such option: --with-marvin
/home/travis/build.sh: line 108: 14848 Terminated  
travis_jigger $! $timeout $cmd

travis_time:end:0da67387:start=1414660406188033558,finish=1414660407047833113,duration=859799555

The command for suite in $TESTS; do travis_wait 30 nosetests 
--with-xunit --xunit-file=integration-test-results/$suite.xml --with-marvin 
--marvin-config=setup/dev/advanced.cfg test/integration/smoke/$suite.py -s -a 
tags=advanced,required_hardware=false --zone=Sandbox-simulator 
--hypervisor=simulator ; done exited with 2.
travis_time:start:05291c61



---
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.
---


Jenkins build is still unstable: simulator-4.5-singlerun #3

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-4.5-singlerun/changes



Review Request 27379: CLOUDSTACK-7118: Fixing test_escalations_instances.py, Removing dependency of test cases on each other

2014-10-30 Thread Gaurav Aradhye

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

Review request for cloudstack and SrikanteswaraRao Talluri.


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


Repository: cloudstack-git


Description
---

Changes:

1. Removing dependency of test cases on each other, creating new account for 
each test case.
2. Fix pep8 issues
3. Fixing imports


Diffs
-

  test/integration/component/test_escalations_instances.py 1aaa688 

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


Testing
---

Yes. Tested with KVM advanced setup. Only one test case failed because it was 
run on wrong configuration.

Log:
@Desc: Test Attach ISO to VM and Detach ISO from VM. ... SKIP: This feature is 
not supported on existing hypervisor. Hence, skipping the test
@Desc: Test VM Snapshots pagination. ... SKIP: This feature is not supported on 
existing hypervisor. Hence, skipping the test
@Desc: Test Revert VM to Snapshot functionality. ... SKIP: This feature is not 
supported on existing hypervisor. Hence, skipping the test
@Desc: Test to verify pagination of Volumes for a VM ... === TestName: 
test_16_list_vm_volumes_pagination | Status : SUCCESS ===
ok
@Desc: Test to verify change service for Running VM ... SKIP: ScaleVM is not 
supported on KVM. Hence, skipping the test
@Desc: Test to verify change service for Stopped VM ... === TestName: 
test_18_stopped_vm_change_service | Status : SUCCESS ===
ok
@Desc: Test to verify creation and reset of SSH Key for VM ... === TestName: 
test_19_create_reset_vm_sshkey | Status : SUCCESS ===
ok
@Desc: Test to verify Update VM details ... === TestName: 
test_20_update_vm_displayname_group | Status : SUCCESS ===
ok
@Desc: Test to verify Restore VM ... === TestName: test_21_restore_vm | Status 
: SUCCESS ===
ok
@Desc: Test to verify deploy VM with multiple networks ... === TestName: 
test_22_deploy_vm_multiple_networks | Status : SUCCESS ===
ok
@Desc: Test to verify deploy VM with multiple Security Groups ... === TestName: 
test_23_deploy_vm_multiple_securitygroups | Status : EXCEPTION ===
ERROR
@Desc: Test to verify deploy VM with static ip address assignment ... === 
TestName: test_24_deploy_vm_with_static_ip_ES1662 | Status : SUCCESS ===
ok
@Desc: Test to verify dnsmasq dhcp conflict issue due to /ect/hosts not getting 
udpated ... === TestName: test_25_ip_reallocation_ES1377 | Status : SUCCESS ===
ok
@Desc: Test List Instances pagination ... === TestName: 
test_01_list_instances_pagination | Status : SUCCESS ===
ok
@Desc: Test List Running VM's ... === TestName: test_02_list_Running_vm | 
Status : SUCCESS ===
ok
@Desc: Test List Stopped VM's ... === TestName: test_03_list_Stopped_vm | 
Status : SUCCESS ===
ok
@Desc: Test List Destroyed VM's ... === TestName: test_04_list_Destroyed_vm | 
Status : SUCCESS ===
ok
@Desc: Test List VM by Id ... === TestName: test_05_list_vm_by_id | Status : 
SUCCESS ===
ok
@Desc: Test List VM's by Name ... === TestName: test_06_list_vm_by_name | 
Status : SUCCESS ===
ok
@Desc: Test List VM's by Name and State ... === TestName: 
test_07_list_vm_by_name_state | Status : SUCCESS ===
ok
@Desc: Test List VM by Zone. ... === TestName: test_08_list_vm_by_zone | Status 
: SUCCESS ===
ok
@Desc: Test List VM by Zone. ... === TestName: test_09_list_vm_by_zone_name | 
Status : SUCCESS ===
ok
@Desc: Test List VM by Zone. ... === TestName: 
test_10_list_vm_by_zone_name_state | Status : SUCCESS ===
ok
@Desc: Test to verify registering and reset of SSH Key for VM ... === TestName: 
test_11_register_reset_vm_sshkey | Status : SUCCESS ===
ok
@Desc: Test to verify change service for Running VM ... === TestName: 
test_12_running_vm_change_service | Status : SUCCESS ===
ok
@Desc: Test to verify Nics for a VM ... === TestName: test_13_vm_nics | Status 
: SUCCESS ===
ok


Thanks,

Gaurav Aradhye



Re: [USAGE][MASTER] java.sql.SQLException: No suitable driver found for jdbc:mysql

2014-10-30 Thread Rajani Karuturi
The only way I could run in a dev environment was to run
com.cloud.usage.UsageServer as a plain java process and explicitly set
mysql connector in the classpath

for example in intellij, add mysql connector to the classpath by editing
the project structure and adding mysql connector jar to the dependencies of
cloud-usage module and right click run as java process.


~Rajani

On Thu, Oct 30, 2014 at 4:01 PM, Ilia Shakitko i.shaki...@tech.leaseweb.com
 wrote:

 Hello,

 I know this was a known issue and fixed in 4.4 and master (as Sebastian
 mentioned), but it appears again in master.

 I am running mgt with jetty-run. Works OK.
 I am running usage then (mvn -pl usage -Drun -Dpid=$$). Getting an
 exception:

 com.cloud.utils.exception.CloudRuntimeException: DB Exception on: null
 at
 com.cloud.utils.db.GenericDaoBase.searchIncludingRemoved(GenericDaoBase.java:425)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$searchIncludingRemoved$36(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
 at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
 at
 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.searchIncludingRemoved(generated)
 at
 com.cloud.utils.db.GenericDaoBase.searchIncludingRemoved(GenericDaoBase.java:361)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$searchIncludingRemoved$35(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
 at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
 at
 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.searchIncludingRemoved(generated)
 at
 com.cloud.utils.db.GenericDaoBase.listIncludingRemovedBy(GenericDaoBase.java:931)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$listIncludingRemovedBy$60(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
 at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
 at
 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.listIncludingRemovedBy(generated)
 at
 com.cloud.utils.db.GenericDaoBase.listIncludingRemovedBy(GenericDaoBase.java:936)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$listIncludingRemovedBy$62(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
 at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
 at
 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.listIncludingRemovedBy(generated)
at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl.getConfiguration(ConfigurationDaoImpl.java:82)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$getConfiguration$7(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2_FastClassByCloudStack_ce5b857a.invoke(generated)
 at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:228)
 at
 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:122)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.getConfiguration(generated)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl.getConfiguration(ConfigurationDaoImpl.java:110)
 at
 org.apache.cloudstack.framework.config.dao.ConfigurationDaoImpl_EnhancerByCloudStack_6c3a6ca2.CGLIB$getConfiguration$8(generated)
 at
 

[GitHub] cloudstack pull request: CLOUDSTACK-7767: fixed events are not gen...

2014-10-30 Thread karuturi
Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/27#issuecomment-61086586
  
shouldnt this go in 4.5 as well?


---
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.
---


Jenkins build is still unstable: simulator-singlerun #516

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes



build 4.4.1 from source

2014-10-30 Thread Pierre-Luc Dion
I got this error when building 4.4.1 from git tag and tar file:

Results :

Failed tests:
runUploadSslCertBadkeyAlgo(org.apache.cloudstack.network.lb.CertServiceTest)

runUploadSslCertBadkeyPair(org.apache.cloudstack.network.lb.CertServiceTest)

Tests in error:

runUploadSslCertSelfSignedNoPassword(org.apache.cloudstack.network.lb.CertServiceTest):
Error parsing certificate data Certificate expired or not valid

runUploadSslCertSelfSignedWithPassword(org.apache.cloudstack.network.lb.CertServiceTest):
Error parsing certificate data Certificate expired or not valid


I'm not sure where it come from, but because of that, 'mvn install' does
not continue.
Also,  the source tar file have  4.4.1-SNAPSHOT as version.

[INFO]

[INFO] Building Apache CloudStack Server 4.4.1-SNAPSHOT
[INFO]


Should be 4.4.1 only as when building from the git tag?

Thanks,

Pierre-Luc


Re: cloudmonkey 5.3.0 autocompletion

2014-10-30 Thread Rohit Yadav
Hi,

Thanks, glad you find it useful.

 On 30-Oct-2014, at 6:43 pm, Nux! n...@li.nux.ro wrote:

 Second, should I expect cloudmonkey to also be able to fill in stuff like
 list templates templatefilter=tabtab ?

At present, it only autocompletes API arguments which are of UUID type. This 
can be seen by doing the api -h command: list templates -h;

CloudMonkey is able to autocomplete all arguments which are of UUID type, if 
you’re seeing any issue it’s probably a bug; so please share if you find any of 
those.

In future versions we can try to address autocompletion for strings, ints, 
maps, types, filters etc. Also, there are many inconsistencies; for example in 
some APIs the “account” argument is string and in some it’s UUID; so it can 
autocomplete for ones which are UUIDs but not for others (which are string, 
ints etc).

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

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

IaaS Cloud Design  Buildhttp://shapeblue.com/iaas-cloud-design-and-build//
CSForge – rapid IaaS deployment frameworkhttp://shapeblue.com/csforge/
CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
CloudStack Infrastructure 
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/
CloudStack Bootcamp Training Courseshttp://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.


RE: build 4.4.1 from source

2014-10-30 Thread Ilia Shakitko
Looks like it's a bug and it is there for 4.4 branch (for 4.4.2 as well). You 
can remove this test (CertServiceTest.java) in case you need to test something 
specific.


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, Netherlands


-Original Message-
From: Pierre-Luc Dion [mailto:pd...@cloudops.com]
Sent: donderdag, 30 oktober, 2014 14:23
To: dev@cloudstack.apache.org
Subject: build 4.4.1 from source

I got this error when building 4.4.1 from git tag and tar file:

Results :

Failed tests:
runUploadSslCertBadkeyAlgo(org.apache.cloudstack.network.lb.CertServiceTest)

runUploadSslCertBadkeyPair(org.apache.cloudstack.network.lb.CertServiceTest)

Tests in error:

runUploadSslCertSelfSignedNoPassword(org.apache.cloudstack.network.lb.CertServiceTest):
Error parsing certificate data Certificate expired or not valid

runUploadSslCertSelfSignedWithPassword(org.apache.cloudstack.network.lb.CertServiceTest):
Error parsing certificate data Certificate expired or not valid


I'm not sure where it come from, but because of that, 'mvn install' does not 
continue.
Also,  the source tar file have  4.4.1-SNAPSHOT as version.

[INFO]

[INFO] Building Apache CloudStack Server 4.4.1-SNAPSHOT [INFO]


Should be 4.4.1 only as when building from the git tag?

Thanks,

Pierre-Luc


Re: cloudmonkey 5.3.0 autocompletion

2014-10-30 Thread Nux!
Thanks, that makes sense.

The fact account is string instead of UUID is one of my pet peeves with ACS. 
it's not consistent with the rest of the stuff and caused some devs I work for 
to scratch their heads.
Hope it gets fixed in the future versions.

Regards,
Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: Rohit Yadav rohit.ya...@shapeblue.com
 To: dev@cloudstack.apache.org
 Sent: Thursday, 30 October, 2014 13:47:05
 Subject: Re: cloudmonkey 5.3.0 autocompletion

 Hi,
 
 Thanks, glad you find it useful.
 
 On 30-Oct-2014, at 6:43 pm, Nux! n...@li.nux.ro wrote:

 Second, should I expect cloudmonkey to also be able to fill in stuff like
 list templates templatefilter=tabtab ?
 
 At present, it only autocompletes API arguments which are of UUID type. This 
 can
 be seen by doing the api -h command: list templates -h;
 
 CloudMonkey is able to autocomplete all arguments which are of UUID type, if
 you’re seeing any issue it’s probably a bug; so please share if you find any 
 of
 those.
 
 In future versions we can try to address autocompletion for strings, ints, 
 maps,
 types, filters etc. Also, there are many inconsistencies; for example in some
 APIs the “account” argument is string and in some it’s UUID; so it can
 autocomplete for ones which are UUIDs but not for others (which are string,
 ints etc).
 
 Regards,
 Rohit Yadav
 Software Architect, ShapeBlue
 M. +91 88 262 30892 | rohit.ya...@shapeblue.com
 Blog: bhaisaab.org | Twitter: @_bhaisaab
 
 Find out more about ShapeBlue and our range of CloudStack related services
 
 IaaS Cloud Design  Buildhttp://shapeblue.com/iaas-cloud-design-and-build//
 CSForge – rapid IaaS deployment frameworkhttp://shapeblue.com/csforge/
 CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
 CloudStack Infrastructure
 Supporthttp://shapeblue.com/cloudstack-infrastructure-support/
 CloudStack Bootcamp Training 
 Courseshttp://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.


RE: build 4.4.1 from source

2014-10-30 Thread Will Stevens
The test certs expired for the Netscaler SSL termination functionality that
Syed wrote which is causing this problem.

I saw in a different thread that Syed was planning to update those certs,
but he probably only did it on master if he has done it already. This would
have to be done on the 4.4 branch as well to solve this. Or you can remove
that test as ilia suggested.

Will
On Oct 30, 2014 9:52 AM, Ilia Shakitko i.shaki...@tech.leaseweb.com
wrote:

 Looks like it's a bug and it is there for 4.4 branch (for 4.4.2 as well).
 You can remove this test (CertServiceTest.java) in case you need to test
 something specific.


 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, Netherlands


 -Original Message-
 From: Pierre-Luc Dion [mailto:pd...@cloudops.com]
 Sent: donderdag, 30 oktober, 2014 14:23
 To: dev@cloudstack.apache.org
 Subject: build 4.4.1 from source

 I got this error when building 4.4.1 from git tag and tar file:

 Results :

 Failed tests:

 runUploadSslCertBadkeyAlgo(org.apache.cloudstack.network.lb.CertServiceTest)


 runUploadSslCertBadkeyPair(org.apache.cloudstack.network.lb.CertServiceTest)

 Tests in error:


 runUploadSslCertSelfSignedNoPassword(org.apache.cloudstack.network.lb.CertServiceTest):
 Error parsing certificate data Certificate expired or not valid


 runUploadSslCertSelfSignedWithPassword(org.apache.cloudstack.network.lb.CertServiceTest):
 Error parsing certificate data Certificate expired or not valid


 I'm not sure where it come from, but because of that, 'mvn install' does
 not continue.
 Also,  the source tar file have  4.4.1-SNAPSHOT as version.

 [INFO]
 
 [INFO] Building Apache CloudStack Server 4.4.1-SNAPSHOT [INFO]
 

 Should be 4.4.1 only as when building from the git tag?

 Thanks,

 Pierre-Luc



Jenkins build is still unstable: simulator-4.5-singlerun #4

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-4.5-singlerun/changes



Re: [PROPOSAL] Move to github PR only during moratorium on commit

2014-10-30 Thread David Nalley
Commit to 4.5 and master please.

--David

On Thu, Oct 30, 2014 at 6:21 AM, Srikanteswararao Talluri
srikanteswararao.tall...@citrix.com wrote:
 Hey Folks,

 After all these discussions, What is the direction to commit the code to
 master and 4.5? I have some commits on 4.5 which I would like to have them
 on master too. How to go about it?

 Thanks,
 ~Talluri

 On 23/10/14 11:47 am, Animesh Chaturvedi animesh.chaturv...@citrix.com
 wrote:



 -Original Message-
 From: sebgoa [mailto:run...@gmail.com]
 Sent: Saturday, October 18, 2014 2:00 AM
 To: dev@cloudstack.apache.org
 Subject: [PROPOSAL] Move to github PR only during moratorium on
 commit

 After [1] I would like to officially bring up the following proposal.

 [Proposal]
 
 All commits come through github PR, *even* for committers. We declare a
 moratorium period (agreed suspension of activity) during which direct
 commit to master is forbidden.
 Only the master RM is allowed to merge PR in master (we define a master
 RM). If direct commit to master is done, master RM reverts without
 warning. Same for 4.5 and 4.4. branches.
 

 This is drastic and I am sure some folks will not like it, but here is
my
 justification for such a measure:

 [Reasons]:
 
 Our commit and release processes have so far been based on the idea that
 development happens on master and that a release branch is cut from
 master (unstable development branch). Then a different set of community
 members harden the release branch, QA and bring it to GA level. During
 that time development keeps on going in master.

 This is an OK process if we have the luxury of having a QA team and can
 cope with split personality of being developers and release managers.

 My point of view is that as a community we cannot afford such a split
brain
 organization and our experience overt the last year proves my point
 (delayed release date, broken builds, features merged without
warning...)

 We can avoid this by cutting a release branch from a stable one (from
the
 start), then as you (Daan) have mentioned several times, fix bugs in the
 release branch and merge them back in the stable source of the release
(be
 it master).


[Animesh] Sebastian you have brought up a  good point dependency on QA
team from Citrix is an issue for the project. This was raised in the past
as well and Alex's proposal [1] few months back using CI was in my
opinion is the optimal solution. Why? Because CloudStack is a huge
project and one single person cannot have the full knowledge to safely
review all the code and certainly cannot scale, which CI and automation
can address

Keeping master stable is something no one would argue against and my
point would match the original proposal from Alex. May be we can  have a
staging branch for master and then merging the commit only after they
have passed CI into master. The proposal got derailed and delayed because
as called out at that time community does not want to work with a process
that has a dependency on infrastructure that is not controlled by
community. David and I are working to get the hardware from Citrix into
ACS infra.

The approach for fixing issues in release branch first and master later
is not practical as we may miss out commits not made into master and
future release regressing without the fixes. Also as the release goes
into hardening cycle there will be a number of fixes which will not be
allowed in release branch but need to be fixes for future, they should
all go in master. Master is the catch all default branch and in my
opinion should get fixes first.

[1] http://markmail.org/thread/xoyjw2sduenlytwm

 Feature development need to be done outside master, period. Not only for
 non-committers but also for committers. And merge request need to be
 called. This will help review and avoid surprises.

[Animesh] Completely Agreed
 New git workflow were proposed and shutdown, mostly calling for better
CI
 to solve quality issues. CI will not solve our quality issues alone. We
need to
 better police ourselves.

[Animesh] I have already expressed my opinion in favor of CI

 To avoid long discussions, I propose this simple but drastic measure. We
 move all our commits to github PR until 4.5 is out, this stands for
 committers and non-committers, direct commits (especially to master)
 would be reverted immediately.
 

 Our development and release process is broken, we cannot continue like
 this, let's fix it.
[Animesh] I  followed up the release process as established by Chip for
4.2 and 4.3 release for which I was the RM and frankly I did not feel it
that way other than the pain of multiple RCs. Folks may not like it but I
am entitled to my opinion and I do feel part of the issues for 4.4 were
self- inflicted because of pre mature code freeze and too early cherry
picking.

 [1] http://markmail.org/thread/xeliefp3oleq3g54

 -sebastien



Build failed in Jenkins: build-4.5 #64

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/build-4.5/64/changes

Changes:

[htrippaers] CLOUDSTACK-7646 Move the nuage test out of smoke as it depends on 
noredist components

--
[...truncated 3705 lines...]

---
 T E S T S
---
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-user-authenticator-saml2 ---
[INFO] Surefire report directory: 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/user-authenticators/saml2/target/surefire-reports

---
 T E S T S
---
Running src.com.cloud.server.auth.test.AuthenticatorTest
Running org.apache.cloudstack.api.command.SAML2LogoutAPIAuthenticatorCmdTest
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-snmp-alerts ---
[INFO] Surefire report directory: 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/alert-handlers/snmp-alerts/target/surefire-reports

---
 T E S T S
---
Running org.apache.cloudstack.alert.snmp.SnmpTrapAppenderTest
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-syslog-alerts ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/alert-handlers/syslog-alerts/test/resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-syslog-alerts ---
[INFO] Compiling 1 source file to 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/alert-handlers/syslog-alerts/target/test-classes
log4j:WARN No appenders could be found for logger 
(com.cloud.server.auth.SHA256SaltedUserAuthenticator).
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: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.673 sec

Results :

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

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - Network VXLAN 4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-network-vxlan ---
[INFO] Deleting 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/network-elements/vxlan/target
 (includes = [**/*], excludes = [])
[INFO] Deleting 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/network-elements/vxlan
 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-network-vxlan ---
log4j:ERROR  Invalid format for the IP Addresses or Ports parameter 
log4j:ERROR  size of ip addresses , communities, and ports list doesn't match, 
setting all to null
log4j:ERROR  size of ip addresses , communities, and ports list doesn't match, 
setting all to null
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-network-vxlan ---
log4j:ERROR  Invalid format for the IP Addresses or Ports parameter 
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.36 sec
Running org.apache.cloudstack.alert.snmp.SnmpEnhancedPatternLayoutTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.243 sec

Results :

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

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - GloboDNS 4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-network-globodns ---
[INFO] Deleting 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/network-elements/globodns/target
 (includes = [**/*], excludes = [])
[INFO] Deleting 
http://jenkins.buildacloud.org/job/build-4.5/ws/plugins/network-elements/globodns
 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-network-globodns ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-network-vxlan ---
[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 

RE: UI: where has Acquire new IP button disappeared?

2014-10-30 Thread Gabor Apati-Nagy
Could someone cherry-pick this to 4.4? That would fix CLOUDSTACK-7782.

Cheers,
Gabor

-Original Message-
From: Brian Federle 
Sent: 27 October 2014 17:58
To: Gabor Apati-Nagy; dev@cloudstack.apache.org
Subject: RE: UI: where has Acquire new IP button disappeared?

Gabor, yes, that shouldn't be a problem to cherry-pick it into 4.4.

-Brian

From: Gabor Apati-Nagy
Sent: Monday, October 27, 2014 9:26 AM
To: dev@cloudstack.apache.org
Cc: Brian Federle
Subject: RE: UI: where has Acquire new IP button disappeared?

It seems that this fix for Critical bug CLOUDSTACK-6590 (commit 
d9fcb877309ce176d5458e63bfd1a174344b5fce) is only in master, but not in 4.4.

In order to fix this in 4.4 as well, I would suggest it to be cherry-picked to 
4.4.
Brian, is this feasible?

Thanks,
Gabor

-Original Message-
From: Nux! [mailto:n...@li.nux.ro]
Sent: 27 October 2014 10:45
To: dev@cloudstack.apache.org
Subject: Re: UI: where has Acquire new IP button disappeared?

Thanks for that, the new javascript file fixed the issue.

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: Jayapal Reddy Uradi jayapalreddy.ur...@citrix.com
 To: dev@cloudstack.apache.org dev@cloudstack.apache.org
 Sent: Monday, 27 October, 2014 10:24:14
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Please check similar bug fix
 https://issues.apache.org/jira/browse/CLOUDSTACK-6590

 -Jayapal

 On 27-Oct-2014, at 3:40 PM, Gabor Apati-Nagy 
 gabor.apati-n...@citrix.com
 wrote:

 I mean gridview is the table that the page displays. This has a row 
 with the IP Address, that should display the button, but I see it is not 
 there...
 Let me look into this later.

 Thanks
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 16:42
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Hello,

 This might sound stupid, but what is this gridview you are talking of?
 This is what I am seeing http://imgur.com/9fl6ERD

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Gabor Apati-Nagy gabor.apati-n...@citrix.com
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 14:26:40
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Hi Lucian,

 The View secondary IP button should be displayed next to the IP 
 Address in the gridview. Clicking on that this page should display 
 the Acquire new secondary IP button. I have tested this in master 
 and we seem to have the same code in 4.4.
 Could you double check this? It could be confusing that the first 
 button is not in the heading, but is located in the grid.

 Cheers,
 Gabor


 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 09:44
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Opened https://issues.apache.org/jira/browse/CLOUDSTACK-7782 for this.

 Anyone knows who we can bug for a fix?

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Nux! n...@li.nux.ro
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 00:32:55
 Subject: UI: where has Acquire new IP button disappeared?

 Hi,

 Only now I notice that the option in the NIC tab of a VM
 (Basic/Adv+SG) no longer has the Acquire secondary button, nor 
 does it list the Secondary IPs.
 I can set and see the extra IPs in cloudmonkey though if I run:

 add iptonic nicid=XXX
 list nics nicid=XXX virtualmachineid=XXX


 This is not OK ... alas I notice it too late. Promise to test more next 
 release.

 Any way I can get that back? I have people using the UI who rely on 
 this feature.

 Lucian

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
  www.nux.ro


Jenkins build is back to stable : simulator-singlerun #517

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/517/changes



Re: UI: where has Acquire new IP button disappeared?

2014-10-30 Thread David Nalley
Nux:

If you cherry-pick this to 4.4 will you also make sure it makes it into 4.5?

--David

On Thu, Oct 30, 2014 at 12:47 PM, Gabor Apati-Nagy
gabor.apati-n...@citrix.com wrote:
 Could someone cherry-pick this to 4.4? That would fix CLOUDSTACK-7782.

 Cheers,
 Gabor

 -Original Message-
 From: Brian Federle
 Sent: 27 October 2014 17:58
 To: Gabor Apati-Nagy; dev@cloudstack.apache.org
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Gabor, yes, that shouldn't be a problem to cherry-pick it into 4.4.

 -Brian
 
 From: Gabor Apati-Nagy
 Sent: Monday, October 27, 2014 9:26 AM
 To: dev@cloudstack.apache.org
 Cc: Brian Federle
 Subject: RE: UI: where has Acquire new IP button disappeared?

 It seems that this fix for Critical bug CLOUDSTACK-6590 (commit 
 d9fcb877309ce176d5458e63bfd1a174344b5fce) is only in master, but not in 4.4.

 In order to fix this in 4.4 as well, I would suggest it to be cherry-picked 
 to 4.4.
 Brian, is this feasible?

 Thanks,
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 27 October 2014 10:45
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Thanks for that, the new javascript file fixed the issue.

 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Jayapal Reddy Uradi jayapalreddy.ur...@citrix.com
 To: dev@cloudstack.apache.org dev@cloudstack.apache.org
 Sent: Monday, 27 October, 2014 10:24:14
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Please check similar bug fix
 https://issues.apache.org/jira/browse/CLOUDSTACK-6590

 -Jayapal

 On 27-Oct-2014, at 3:40 PM, Gabor Apati-Nagy
 gabor.apati-n...@citrix.com
 wrote:

 I mean gridview is the table that the page displays. This has a row
 with the IP Address, that should display the button, but I see it is not 
 there...
 Let me look into this later.

 Thanks
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 16:42
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Hello,

 This might sound stupid, but what is this gridview you are talking of?
 This is what I am seeing http://imgur.com/9fl6ERD

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Gabor Apati-Nagy gabor.apati-n...@citrix.com
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 14:26:40
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Hi Lucian,

 The View secondary IP button should be displayed next to the IP
 Address in the gridview. Clicking on that this page should display
 the Acquire new secondary IP button. I have tested this in master
 and we seem to have the same code in 4.4.
 Could you double check this? It could be confusing that the first
 button is not in the heading, but is located in the grid.

 Cheers,
 Gabor


 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 09:44
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Opened https://issues.apache.org/jira/browse/CLOUDSTACK-7782 for this.

 Anyone knows who we can bug for a fix?

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Nux! n...@li.nux.ro
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 00:32:55
 Subject: UI: where has Acquire new IP button disappeared?

 Hi,

 Only now I notice that the option in the NIC tab of a VM
 (Basic/Adv+SG) no longer has the Acquire secondary button, nor
 does it list the Secondary IPs.
 I can set and see the extra IPs in cloudmonkey though if I run:

 add iptonic nicid=XXX
 list nics nicid=XXX virtualmachineid=XXX


 This is not OK ... alas I notice it too late. Promise to test more next 
 release.

 Any way I can get that back? I have people using the UI who rely on
 this feature.

 Lucian

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
  www.nux.ro


Re: [VOTE] Release Apache CloudStack CloudMonkey 5.3.0

2014-10-30 Thread Pierre-Luc Dion
+1 (binding)

Quickly test against 4.2.1 and 4.4.1.  autocompletion seams to work quite
well.
thanks !


On Thu, Oct 30, 2014 at 6:55 AM, Rohit Yadav rohit.ya...@shapeblue.com
wrote:

 Hi All,

 I've created a 5.3.0 release of CloudMonkey, with the following artifacts
 up for a vote:

 Git Branch and Commit SH:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=shortlog;h=refs/heads/X.X
 Commit: 715630aebdcf18fbb2094893569a0c20fe7b65ab

 List of changes:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=blob_plain;f=CHANGES;hb=X.X

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

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

 The CHANGES file notes changes since 5.2.0, major changes in this version:
 - Parameter completion works much better, enabled by default
 - Current server profile displayed on prompt
 - Filtering support for default display output
 - New command line arg: -d for display (json, table or default)
 - Non-interactive commands from command line are outputted without colors
 - CloudMonkey returns non-zero exit code when run on shell and a error is
 return
   from managment server, the error message is written to stderr

 Since the next 72 hour window intersects with the weekend, the vote will
 be open till Tuesday (120 hours). Please test this release and give your
 vote. Thanks.

 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. +91 88 262 30892 | 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 frameworkhttp://shapeblue.com/csforge/
 CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
 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.



Re: UI: where has Acquire new IP button disappeared?

2014-10-30 Thread Nux!
I ended up patching/replacing the file manually. I'd rather leave the git stuff 
to someone who actually knows how to do it.

What worked for me is this file: 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: David Nalley da...@gnsa.us
 To: dev@cloudstack.apache.org
 Sent: Thursday, 30 October, 2014 17:52:59
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Nux:
 
 If you cherry-pick this to 4.4 will you also make sure it makes it into 4.5?
 
 --David
 
 On Thu, Oct 30, 2014 at 12:47 PM, Gabor Apati-Nagy
 gabor.apati-n...@citrix.com wrote:
 Could someone cherry-pick this to 4.4? That would fix CLOUDSTACK-7782.

 Cheers,
 Gabor

 -Original Message-
 From: Brian Federle
 Sent: 27 October 2014 17:58
 To: Gabor Apati-Nagy; dev@cloudstack.apache.org
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Gabor, yes, that shouldn't be a problem to cherry-pick it into 4.4.

 -Brian
 
 From: Gabor Apati-Nagy
 Sent: Monday, October 27, 2014 9:26 AM
 To: dev@cloudstack.apache.org
 Cc: Brian Federle
 Subject: RE: UI: where has Acquire new IP button disappeared?

 It seems that this fix for Critical bug CLOUDSTACK-6590 (commit
 d9fcb877309ce176d5458e63bfd1a174344b5fce) is only in master, but not in 4.4.

 In order to fix this in 4.4 as well, I would suggest it to be cherry-picked 
 to
 4.4.
 Brian, is this feasible?

 Thanks,
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 27 October 2014 10:45
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Thanks for that, the new javascript file fixed the issue.

 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Jayapal Reddy Uradi jayapalreddy.ur...@citrix.com
 To: dev@cloudstack.apache.org dev@cloudstack.apache.org
 Sent: Monday, 27 October, 2014 10:24:14
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Please check similar bug fix
 https://issues.apache.org/jira/browse/CLOUDSTACK-6590

 -Jayapal

 On 27-Oct-2014, at 3:40 PM, Gabor Apati-Nagy
 gabor.apati-n...@citrix.com
 wrote:

 I mean gridview is the table that the page displays. This has a row
 with the IP Address, that should display the button, but I see it is not
 there...
 Let me look into this later.

 Thanks
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 16:42
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Hello,

 This might sound stupid, but what is this gridview you are talking of?
 This is what I am seeing http://imgur.com/9fl6ERD

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Gabor Apati-Nagy gabor.apati-n...@citrix.com
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 14:26:40
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Hi Lucian,

 The View secondary IP button should be displayed next to the IP
 Address in the gridview. Clicking on that this page should display
 the Acquire new secondary IP button. I have tested this in master
 and we seem to have the same code in 4.4.
 Could you double check this? It could be confusing that the first
 button is not in the heading, but is located in the grid.

 Cheers,
 Gabor


 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 09:44
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Opened https://issues.apache.org/jira/browse/CLOUDSTACK-7782 for this.

 Anyone knows who we can bug for a fix?

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Nux! n...@li.nux.ro
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 00:32:55
 Subject: UI: where has Acquire new IP button disappeared?

 Hi,

 Only now I notice that the option in the NIC tab of a VM
 (Basic/Adv+SG) no longer has the Acquire secondary button, nor
 does it list the Secondary IPs.
 I can set and see the extra IPs in cloudmonkey though if I run:

 add iptonic nicid=XXX
 list nics nicid=XXX virtualmachineid=XXX


 This is not OK ... alas I notice it too late. Promise to test more next 
 release.

 Any way I can get that back? I have people using the UI who rely on
 this feature.

 Lucian

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
   www.nux.ro


Re: [VOTE] Release Apache CloudStack CloudMonkey 5.3.0

2014-10-30 Thread Nux!
+1 for me as well, quick tests were all successful.

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: Pierre-Luc Dion pdion...@apache.org
 To: dev@cloudstack.apache.org
 Sent: Thursday, 30 October, 2014 18:06:29
 Subject: Re: [VOTE] Release Apache CloudStack CloudMonkey 5.3.0

 +1 (binding)
 
 Quickly test against 4.2.1 and 4.4.1.  autocompletion seams to work quite
 well.
 thanks !
 
 
 On Thu, Oct 30, 2014 at 6:55 AM, Rohit Yadav rohit.ya...@shapeblue.com
 wrote:
 
 Hi All,

 I've created a 5.3.0 release of CloudMonkey, with the following artifacts
 up for a vote:

 Git Branch and Commit SH:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=shortlog;h=refs/heads/X.X
 Commit: 715630aebdcf18fbb2094893569a0c20fe7b65ab

 List of changes:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=blob_plain;f=CHANGES;hb=X.X

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

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

 The CHANGES file notes changes since 5.2.0, major changes in this version:
 - Parameter completion works much better, enabled by default
 - Current server profile displayed on prompt
 - Filtering support for default display output
 - New command line arg: -d for display (json, table or default)
 - Non-interactive commands from command line are outputted without colors
 - CloudMonkey returns non-zero exit code when run on shell and a error is
 return
   from managment server, the error message is written to stderr

 Since the next 72 hour window intersects with the weekend, the vote will
 be open till Tuesday (120 hours). Please test this release and give your
 vote. Thanks.

 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. +91 88 262 30892 | 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 frameworkhttp://shapeblue.com/csforge/
 CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
 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.


Jenkins build is back to normal : build-4.5 #65

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/build-4.5/65/changes



Guest IP Reservation

2014-10-30 Thread Logan Barfield
Just trying to clear something up before I submit a bug report:

When using IP reservation in an isolated network it looks like the virtual
router is getting the wrong netmask.

For example:
- Network CIDR: 10.1.1.0/24
- Guest CIDR: 10.1.1.0/25
- Reserved IP Range: 10.1.1.127-10.1.1.254
- Virtual Router IP: 10.1.1.1

With this configuration, the Virtual Router gets get following
netmask/routing:

eth0  Link encap:Ethernet  HWaddr 02:00:55:eb:00:03
  inet addr:10.1.1.1  Bcast:10.1.1.127  Mask:255.255.255.128

Destination Gateway Genmask Flags Metric RefUse
Iface
0.0.0.0 162.223.12.129  0.0.0.0 UG0  00 eth2
10.1.1.00.0.0.0 255.255.255.128 U 0  00 eth0
162.223.12.128  0.0.0.0 255.255.255.128 U 0  00 eth2
169.254.0.0 0.0.0.0 255.255.0.0 U 0  00 eth1

This means that any server or VM configured in the reserved IP range cannot
ping the VR or use it for routing.  They also will not be able to contact
VMs deployed by CloudStack because there is no routing available, and the
CloudStack VMs inherit the /25 netmask from the VR.

To resolve this I think the following changes would be required:

Right now it seems that the /etc/init.d/postinit script configures the VR
interfaces using the details in '/var/cache/cloud/cmdline':

# cat /var/cache/cloud/cmdline
template=domP name=r-236-VM eth2ip=162.223.12.140 eth2mask=255.255.255.128
gateway=162.223.12.129 eth0ip=10.1.1.1 eth0mask=255.255.255.128 domain=
cs2dv.tqcloud.net cidrsize=25 dhcprange=10.1.1.1 eth1ip=169.254.2.4
eth1mask=255.255.0.0 type=router disable_rp_filter=true dns1=8.8.8.8
dns2=8.8.4.4 useextdns=true

I believe whatever is generating the data in '/var/cache/cloud/cmdline'
should be changed.  It should pull the 'eth0mask' from the 'Network CIDR'
instead of the 'Guest CIDR'.  This will allow for routing and communication
between CloudStack VMs, and hosts on the reserved portion of the network.

The remaining issue is ensuring the VR doesn't issue IPs from the reserved
range.  I don't think this is a problem anyway since CloudStack seems to
manually set up the static DHCP reservations (with /etc/dhcphosts.txt), but
the following change could still be made:

- Instead of using the VR IP in the dhcp-range (ex:
dhcp-range=10.1.1.1,static), it could be set as the inverse of the reserved
network (ex: dhcp-range=10.1.1.2,10.1.1.126,255.255.255.0,infinite).  I
believe the dhcp-range is also pulled from '/var/cache/cloud/cmdline'.


Am I misunderstanding how this feature is supposed to work, or should I go
ahead and create a bug report for this?


Thank You,

Logan Barfield
Tranquil Hosting


Re: [VOTE] Release Apache CloudStack CloudMonkey 5.3.0

2014-10-30 Thread Erik Weber
On Thu, Oct 30, 2014 at 11:55 AM, Rohit Yadav rohit.ya...@shapeblue.com
wrote:

 Hi All,

 I've created a 5.3.0 release of CloudMonkey, with the following artifacts
 up for a vote:

 Git Branch and Commit SH:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=shortlog;h=refs/heads/X.X
 Commit: 715630aebdcf18fbb2094893569a0c20fe7b65ab

 List of changes:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=blob_plain;f=CHANGES;hb=X.X

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



+1

- Successfully upgraded from earlier pip install
- Worked with previous profile
- Added new profile
- param completion tested ok after enabling it (sweet feature by the way!)
- tested various calls, list zones/serviceofferings/templates, deploy
virtualmachine

-- 
Erik


Jenkins build is back to stable : simulator-4.5-singlerun #5

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-4.5-singlerun/5/changes



RE: UI: where has Acquire new IP button disappeared?

2014-10-30 Thread Brian Federle
I just cherry-picked and verified that commit in 4.4.

-Brian

-Original Message-
From: Gabor Apati-Nagy 
Sent: Thursday, October 30, 2014 9:48 AM
To: Brian Federle; dev@cloudstack.apache.org
Subject: RE: UI: where has Acquire new IP button disappeared?

Could someone cherry-pick this to 4.4? That would fix CLOUDSTACK-7782.

Cheers,
Gabor

-Original Message-
From: Brian Federle
Sent: 27 October 2014 17:58
To: Gabor Apati-Nagy; dev@cloudstack.apache.org
Subject: RE: UI: where has Acquire new IP button disappeared?

Gabor, yes, that shouldn't be a problem to cherry-pick it into 4.4.

-Brian

From: Gabor Apati-Nagy
Sent: Monday, October 27, 2014 9:26 AM
To: dev@cloudstack.apache.org
Cc: Brian Federle
Subject: RE: UI: where has Acquire new IP button disappeared?

It seems that this fix for Critical bug CLOUDSTACK-6590 (commit 
d9fcb877309ce176d5458e63bfd1a174344b5fce) is only in master, but not in 4.4.

In order to fix this in 4.4 as well, I would suggest it to be cherry-picked to 
4.4.
Brian, is this feasible?

Thanks,
Gabor

-Original Message-
From: Nux! [mailto:n...@li.nux.ro]
Sent: 27 October 2014 10:45
To: dev@cloudstack.apache.org
Subject: Re: UI: where has Acquire new IP button disappeared?

Thanks for that, the new javascript file fixed the issue.

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
 From: Jayapal Reddy Uradi jayapalreddy.ur...@citrix.com
 To: dev@cloudstack.apache.org dev@cloudstack.apache.org
 Sent: Monday, 27 October, 2014 10:24:14
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Please check similar bug fix
 https://issues.apache.org/jira/browse/CLOUDSTACK-6590

 -Jayapal

 On 27-Oct-2014, at 3:40 PM, Gabor Apati-Nagy 
 gabor.apati-n...@citrix.com
 wrote:

 I mean gridview is the table that the page displays. This has a row 
 with the IP Address, that should display the button, but I see it is not 
 there...
 Let me look into this later.

 Thanks
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 16:42
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Hello,

 This might sound stupid, but what is this gridview you are talking of?
 This is what I am seeing http://imgur.com/9fl6ERD

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Gabor Apati-Nagy gabor.apati-n...@citrix.com
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 14:26:40
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Hi Lucian,

 The View secondary IP button should be displayed next to the IP 
 Address in the gridview. Clicking on that this page should display 
 the Acquire new secondary IP button. I have tested this in master 
 and we seem to have the same code in 4.4.
 Could you double check this? It could be confusing that the first 
 button is not in the heading, but is located in the grid.

 Cheers,
 Gabor


 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 09:44
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Opened https://issues.apache.org/jira/browse/CLOUDSTACK-7782 for this.

 Anyone knows who we can bug for a fix?

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Nux! n...@li.nux.ro
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 00:32:55
 Subject: UI: where has Acquire new IP button disappeared?

 Hi,

 Only now I notice that the option in the NIC tab of a VM
 (Basic/Adv+SG) no longer has the Acquire secondary button, nor 
 does it list the Secondary IPs.
 I can set and see the extra IPs in cloudmonkey though if I run:

 add iptonic nicid=XXX
 list nics nicid=XXX virtualmachineid=XXX


 This is not OK ... alas I notice it too late. Promise to test more next 
 release.

 Any way I can get that back? I have people using the UI who rely on 
 this feature.

 Lucian

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
  www.nux.ro


Review Request 27393: Fix for Resize RBD Root Disk on Deploy

2014-10-30 Thread Logan B

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

Review request for cloudstack and Wido den Hollander.


Repository: cloudstack-git


Description
---

Currently if you specify a root disk size when deploying an instance using Ceph 
RBD as the Primary Storage backend the disk will not be resized.

This patch adds a check after deploying an instance that checks the specified 
root disk volume size against the size of the template it was deployed from.  
If the specified volume size is larger than the template it will resize the 
root disk volume to the specified size.

If the specified volume size is equal to or smaller than the template size then 
no resizing occurs, and the instance will be deployed with a root disk size 
equal to the size of the template.


Diffs
-

  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
 af7a0e0 

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


Testing
---

Test: Created new instance from 5GB template with 20GB root disk size specified.
Result: Instance created successfully with 20GB volume size.

Test: Created new instance from 5GB template with 5GB root disk size specified.
Result: Instance created successfully with 5GB volume size.

Test: Created new instance from 5GB template with 1GB root disk size specified.
Result: Instance created successfully with 5GB volume size.

Test: Created new instance from 5GB template with no root disk size specified.
Result: Instance created successfully with 5GB volume size.

Patch was deployed to our production infrastructure on July 25th, 2014.  It's 
been running stable since that time.


Thanks,

Logan B



Build failed in Jenkins: cloudstack-4.4-maven-build #481

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/481/changes

Changes:

[brian.federle] CLOUDSTACK-6590: Fix view all link for multi-item detail view

--
[...truncated 1306 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 30 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-server 
---
[INFO] Compiling 360 source files to 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/classes
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-server ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 29 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-server ---
[INFO] Compiling 82 source files to 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/test-classes
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ cloud-server ---
[INFO] Surefire report directory: 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/surefire-reports

---
 T E S T S
---
Running com.cloud.event.EventControlsUnitTest
log4j:WARN No appenders could be found for logger 
(com.cloud.event.EventControlsUnitTest).
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: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.65 sec
Running com.cloud.keystore.KeystoreTest
org.apache.cloudstack.api.response.UserVmResponse/null/{id:3,securitygroup:[],nic:[],tags:[],affinitygroup:[]}
org.apache.cloudstack.api.response.AlertResponse/null/{id:100,description:Hello}
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.16 sec
Running com.cloud.alert.AlertControlsUnitTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.027 sec
Running com.cloud.capacity.CapacityManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.136 sec
Running com.cloud.resourcelimit.ResourceLimitManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 sec
Running com.cloud.network.firewall.FirewallManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.001 sec
Running com.cloud.network.UpdatePhysicalNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.218 sec
Running com.cloud.network.CreatePrivateNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.125 sec
Running com.cloud.network.NetworkModelTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.097 sec
Running com.cloud.network.security.SecurityGroupManagerImplTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.066 sec
Running com.cloud.network.security.SecurityGroupQueueTest
Total jobs dequeued = 10, num queued=1004 queue current size=994
Num Vms= 50 Queue size = 50
Num Vms= 2 Queue size = 2 time=1845 ms
Num Vms= 5000 Queue size = 5000 time=1127 ms
Num Vms= 1 Queue size = 1 time=0 ms
Num Vms= 100 Queue size = 100 time=351 ms
Num Vms= 1 Queue size = 1 time=267 ms
Total jobs dequeued = 10, num queued=1010 queue current size=1000
Total jobs dequeued = 1, num queued=1001 queue current size=1000
Total jobs dequeued = 10, num queued=1000 queue current size=990
Total jobs dequeued = 10, num queued=10 queue current size=0
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.789 sec
Running com.cloud.network.vpc.VpcManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.025 sec
Running com.cloud.network.DedicateGuestVlanRangesTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.048 sec
Running com.cloud.network.lb.AssignLoadBalancerTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.092 sec
Running com.cloud.network.dao.NetworkDaoTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec
Running com.cloud.vm.DeploymentPlanningManagerImplTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.691 sec
Running com.cloud.vm.snapshot.VMSnapshotManagerTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.154 sec
Running com.cloud.vm.UserVmManagerTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.214 sec
Running com.cloud.configuration.ValidateIpRangeTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.019 sec
Running com.cloud.configuration.ConfigurationManagerTest
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.061 sec
Running com.cloud.server.ConfigurationServerImplTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.016 

Build failed in Jenkins: cloudstack-4.4-maven-build #482

2014-10-30 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/482/

--
[...truncated 1326 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 30 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-server 
---
[INFO] Compiling 360 source files to 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/classes
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-server ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 29 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-server ---
[INFO] Compiling 82 source files to 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/test-classes
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ cloud-server ---
[INFO] Surefire report directory: 
http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/ws/server/target/surefire-reports

---
 T E S T S
---
Running com.cloud.event.EventControlsUnitTest
log4j:WARN No appenders could be found for logger 
(com.cloud.event.EventControlsUnitTest).
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: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.58 sec
Running com.cloud.keystore.KeystoreTest
org.apache.cloudstack.api.response.UserVmResponse/null/{id:3,securitygroup:[],nic:[],tags:[],affinitygroup:[]}
org.apache.cloudstack.api.response.AlertResponse/null/{id:100,description:Hello}
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.166 sec
Running com.cloud.alert.AlertControlsUnitTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.039 sec
Running com.cloud.capacity.CapacityManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.124 sec
Running com.cloud.resourcelimit.ResourceLimitManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.cloud.network.firewall.FirewallManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.001 sec
Running com.cloud.network.UpdatePhysicalNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.198 sec
Running com.cloud.network.CreatePrivateNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.107 sec
Running com.cloud.network.NetworkModelTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.09 sec
Running com.cloud.network.security.SecurityGroupManagerImplTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.965 sec
Running com.cloud.network.security.SecurityGroupQueueTest
Total jobs dequeued = 10, num queued=1001 queue current size=991
Num Vms= 50 Queue size = 50
Num Vms= 2 Queue size = 2 time=1305 ms
Num Vms= 5000 Queue size = 5000 time=793 ms
Num Vms= 1 Queue size = 1 time=0 ms
Num Vms= 100 Queue size = 100 time=218 ms
Num Vms= 1 Queue size = 1 time=185 ms
Total jobs dequeued = 10, num queued=1003 queue current size=993
Total jobs dequeued = 1, num queued=1001 queue current size=1000
Total jobs dequeued = 10, num queued=1000 queue current size=990
Total jobs dequeued = 10, num queued=10 queue current size=0
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.668 sec
Running com.cloud.network.vpc.VpcManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.023 sec
Running com.cloud.network.DedicateGuestVlanRangesTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.04 sec
Running com.cloud.network.lb.AssignLoadBalancerTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.093 sec
Running com.cloud.network.dao.NetworkDaoTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec
Running com.cloud.vm.DeploymentPlanningManagerImplTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.763 sec
Running com.cloud.vm.snapshot.VMSnapshotManagerTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.155 sec
Running com.cloud.vm.UserVmManagerTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.191 sec
Running com.cloud.configuration.ValidateIpRangeTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.015 sec
Running com.cloud.configuration.ConfigurationManagerTest
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.052 sec
Running com.cloud.server.ConfigurationServerImplTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.016 sec
Running com.cloud.template.TemplateManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, 

Re: Review Request 25772: new nfs storage adapter for kvm hypervisor plugin to support managed storage.[CLOUDSTACK-7576].

2014-10-30 Thread Mike Tutkowski

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

Ship it!


Committed under SHA 09a62b8389c8bf67f4bdd5e938d8b95e99d78e99

- Mike Tutkowski


On Oct. 5, 2014, 10:22 p.m., punith s wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25772/
 ---
 
 (Updated Oct. 5, 2014, 10:22 p.m.)
 
 
 Review request for cloudstack, Mike Tutkowski and Marcus Sorensen.
 
 
 Bugs: CLOUDSTACK-7576
 https://issues.apache.org/jira/browse/CLOUDSTACK-7576
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 this adapter provides one to one mapping between the SAN volume to a VM's 
 disk, so that it can guarantee the QoS for the performance sensitive 
 applications residing on the disk.
 this adapter is based on nfs protocol.
 
 features being supported:
 * attach volume
 * detach volume
 
 
 Diffs
 -
 
   api/src/com/cloud/agent/api/to/DiskTO.java afab856 
   api/src/com/cloud/storage/Storage.java 4d267eb 
   
 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java
  f44bb03 
   
 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java
  1a756cb 
   
 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/ManagedNfsStorageAdaptor.java
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/25772/diff/
 
 
 Testing
 ---
 
 system testing done
 
 
 File Attachments
 
 
 architecture
   
 https://reviews.apache.org/media/uploaded/files/2014/09/18/86449aef-17f3-49fb-b087-be74fade231a__ACS_cloudbyte_integration_with_KVM_for_guaranteeing_QoS_at_VM_level.__.jpg
 
 
 Thanks,
 
 punith s
 




Re: [VOTE] Release Apache CloudStack CloudMonkey 5.3.0

2014-10-30 Thread Rohit Yadav
Thanks everyone for voting so far.

I found some issues in my testing which I’ve fixed, so I need to call off this 
voting round. I’ll start another round today.

The new changes are;
- Double url encoding removed and requests bugfixes
- Autocompletion fixes, autocompletions for filter options and boolean api 
arguments
- CloudMonkey becomes unicode friendly (this would need some testing)

Regards.

 On 31-Oct-2014, at 12:56 am, Erik Weber terbol...@gmail.com wrote:

 On Thu, Oct 30, 2014 at 11:55 AM, Rohit Yadav rohit.ya...@shapeblue.com
 wrote:

 Hi All,

 I've created a 5.3.0 release of CloudMonkey, with the following artifacts
 up for a vote:

 Git Branch and Commit SH:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=shortlog;h=refs/heads/X.X
 Commit: 715630aebdcf18fbb2094893569a0c20fe7b65ab

 List of changes:

 https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git;a=blob_plain;f=CHANGES;hb=X.X

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



 +1

 - Successfully upgraded from earlier pip install
 - Worked with previous profile
 - Added new profile
 - param completion tested ok after enabling it (sweet feature by the way!)
 - tested various calls, list zones/serviceofferings/templates, deploy
 virtualmachine

 --
 Erik

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



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

IaaS Cloud Design  Buildhttp://shapeblue.com/iaas-cloud-design-and-build//
CSForge – rapid IaaS deployment frameworkhttp://shapeblue.com/csforge/
CloudStack Consultinghttp://shapeblue.com/cloudstack-consultancy/
CloudStack Infrastructure 
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/
CloudStack Bootcamp Training Courseshttp://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.


Re: UI: where has Acquire new IP button disappeared?

2014-10-30 Thread David Nalley
Thanks Brian

--David

On Thu, Oct 30, 2014 at 4:16 PM, Brian Federle brian.fede...@citrix.com wrote:
 I just cherry-picked and verified that commit in 4.4.

 -Brian

 -Original Message-
 From: Gabor Apati-Nagy
 Sent: Thursday, October 30, 2014 9:48 AM
 To: Brian Federle; dev@cloudstack.apache.org
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Could someone cherry-pick this to 4.4? That would fix CLOUDSTACK-7782.

 Cheers,
 Gabor

 -Original Message-
 From: Brian Federle
 Sent: 27 October 2014 17:58
 To: Gabor Apati-Nagy; dev@cloudstack.apache.org
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Gabor, yes, that shouldn't be a problem to cherry-pick it into 4.4.

 -Brian
 
 From: Gabor Apati-Nagy
 Sent: Monday, October 27, 2014 9:26 AM
 To: dev@cloudstack.apache.org
 Cc: Brian Federle
 Subject: RE: UI: where has Acquire new IP button disappeared?

 It seems that this fix for Critical bug CLOUDSTACK-6590 (commit 
 d9fcb877309ce176d5458e63bfd1a174344b5fce) is only in master, but not in 4.4.

 In order to fix this in 4.4 as well, I would suggest it to be cherry-picked 
 to 4.4.
 Brian, is this feasible?

 Thanks,
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 27 October 2014 10:45
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Thanks for that, the new javascript file fixed the issue.

 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=ui/scripts/ui/widgets/detailView.js;h=9c73911f3dcaecd8327f76c84043c115fdb184d9;hb=d9fcb87

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Jayapal Reddy Uradi jayapalreddy.ur...@citrix.com
 To: dev@cloudstack.apache.org dev@cloudstack.apache.org
 Sent: Monday, 27 October, 2014 10:24:14
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Please check similar bug fix
 https://issues.apache.org/jira/browse/CLOUDSTACK-6590

 -Jayapal

 On 27-Oct-2014, at 3:40 PM, Gabor Apati-Nagy
 gabor.apati-n...@citrix.com
 wrote:

 I mean gridview is the table that the page displays. This has a row
 with the IP Address, that should display the button, but I see it is not 
 there...
 Let me look into this later.

 Thanks
 Gabor

 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 16:42
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Hello,

 This might sound stupid, but what is this gridview you are talking of?
 This is what I am seeing http://imgur.com/9fl6ERD

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Gabor Apati-Nagy gabor.apati-n...@citrix.com
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 14:26:40
 Subject: RE: UI: where has Acquire new IP button disappeared?

 Hi Lucian,

 The View secondary IP button should be displayed next to the IP
 Address in the gridview. Clicking on that this page should display
 the Acquire new secondary IP button. I have tested this in master
 and we seem to have the same code in 4.4.
 Could you double check this? It could be confusing that the first
 button is not in the heading, but is located in the grid.

 Cheers,
 Gabor


 -Original Message-
 From: Nux! [mailto:n...@li.nux.ro]
 Sent: 24 October 2014 09:44
 To: dev@cloudstack.apache.org
 Subject: Re: UI: where has Acquire new IP button disappeared?

 Opened https://issues.apache.org/jira/browse/CLOUDSTACK-7782 for this.

 Anyone knows who we can bug for a fix?

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
 www.nux.ro

 - Original Message -
 From: Nux! n...@li.nux.ro
 To: dev@cloudstack.apache.org
 Sent: Friday, 24 October, 2014 00:32:55
 Subject: UI: where has Acquire new IP button disappeared?

 Hi,

 Only now I notice that the option in the NIC tab of a VM
 (Basic/Adv+SG) no longer has the Acquire secondary button, nor
 does it list the Secondary IPs.
 I can set and see the extra IPs in cloudmonkey though if I run:

 add iptonic nicid=XXX
 list nics nicid=XXX virtualmachineid=XXX


 This is not OK ... alas I notice it too late. Promise to test more next 
 release.

 Any way I can get that back? I have people using the UI who rely on
 this feature.

 Lucian

 --
 Sent from the Delta quadrant using Borg technology!

 Nux!
  www.nux.ro


[ACS442] merge request

2014-10-30 Thread Pierre-Luc Dion
Hi Daan,

could it be possible to merge hotfix/4.4-7822  and cherry-pick:
b04cc750652642e8e2593128c7c0f5de3ef6b0d7
into 4.4?

thanks,

PL


Re: Review Request 27017: CLOUDSTACK-6282: Added newly automated tests and also modified some existing tests to remove dependency

2014-10-30 Thread Vinay Varma

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

(Updated Oct. 31, 2014, 4:37 a.m.)


Review request for cloudstack, Alex Brett and Santhosh Edukulla.


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


Repository: cloudstack-git


Description
---

CLOUDSTACK-6282: Added newly automated tests and also modified some existing 
tests to remove dependency


Diffs
-

  test/integration/component/test_escalations_instances.py 1aaa688 
  test/integration/component/test_escalations_snapshots.py 4b6b7f5 
  test/integration/component/test_escalations_templates.py 78028bc 
  test/integration/component/test_escalations_volumes.py 7290325 
  test/integration/component/test_escalations_vpncustomergateways.py b09930a 
  tools/marvin/marvin/cloudstackTestClient.py ce7ffc9 
  tools/marvin/marvin/lib/base.py 77faeeb 

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


Testing
---

Attached are the results files for each of the file modified and the results 
shows everything is fine


File Attachments


Instancesresults.txt
  
https://reviews.apache.org/media/uploaded/files/2014/10/22/6ea95260-f31e-4fdd-a9f3-f30bac872df5__Instancesresults.txt
Snapshotsresults.txt
  
https://reviews.apache.org/media/uploaded/files/2014/10/22/a91e862c-dc2e-403e-85e4-6479eefcd9d1__Snapshotsresults.txt
Templatesresults.txt
  
https://reviews.apache.org/media/uploaded/files/2014/10/22/545fd06e-4975-4330-8390-3723d944ec2b__Templatesresults.txt
Voumesresults.txt
  
https://reviews.apache.org/media/uploaded/files/2014/10/22/9932b16c-684f-41ce-b6f9-192fe887c2b8__Voumesresults.txt
VPNCustomerGatewaysresults.txt
  
https://reviews.apache.org/media/uploaded/files/2014/10/22/58c5f08e-cac7-4873-a922-8c874a9a8e3a__VPNCustomerGatewaysresults.txt


Thanks,

Vinay Varma



Re: [PROPOSAL] Move to github PR only during moratorium on commit

2014-10-30 Thread Rajani Karuturi
Can we atleast follow the merge part of it? ie) commit to 4.5 and then
merge 4.5 to master?

merging wont be easy unless everybody agrees and does merge for their
commits.

~Rajani

On Thu, Oct 30, 2014 at 8:44 PM, David Nalley da...@gnsa.us wrote:

 Commit to 4.5 and master please.

 --David

 On Thu, Oct 30, 2014 at 6:21 AM, Srikanteswararao Talluri
 srikanteswararao.tall...@citrix.com wrote:
  Hey Folks,
 
  After all these discussions, What is the direction to commit the code to
  master and 4.5? I have some commits on 4.5 which I would like to have
 them
  on master too. How to go about it?
 
  Thanks,
  ~Talluri
 
  On 23/10/14 11:47 am, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com
  wrote:
 
 
 
  -Original Message-
  From: sebgoa [mailto:run...@gmail.com]
  Sent: Saturday, October 18, 2014 2:00 AM
  To: dev@cloudstack.apache.org
  Subject: [PROPOSAL] Move to github PR only during moratorium on
  commit
 
  After [1] I would like to officially bring up the following proposal.
 
  [Proposal]
  
  All commits come through github PR, *even* for committers. We declare a
  moratorium period (agreed suspension of activity) during which direct
  commit to master is forbidden.
  Only the master RM is allowed to merge PR in master (we define a master
  RM). If direct commit to master is done, master RM reverts without
  warning. Same for 4.5 and 4.4. branches.
  
 
  This is drastic and I am sure some folks will not like it, but here is
 my
  justification for such a measure:
 
  [Reasons]:
  
  Our commit and release processes have so far been based on the idea
 that
  development happens on master and that a release branch is cut from
  master (unstable development branch). Then a different set of community
  members harden the release branch, QA and bring it to GA level. During
  that time development keeps on going in master.
 
  This is an OK process if we have the luxury of having a QA team and can
  cope with split personality of being developers and release managers.
 
  My point of view is that as a community we cannot afford such a split
 brain
  organization and our experience overt the last year proves my point
  (delayed release date, broken builds, features merged without
 warning...)
 
  We can avoid this by cutting a release branch from a stable one (from
 the
  start), then as you (Daan) have mentioned several times, fix bugs in
 the
  release branch and merge them back in the stable source of the release
 (be
  it master).
 
 
 [Animesh] Sebastian you have brought up a  good point dependency on QA
 team from Citrix is an issue for the project. This was raised in the past
 as well and Alex's proposal [1] few months back using CI was in my
 opinion is the optimal solution. Why? Because CloudStack is a huge
 project and one single person cannot have the full knowledge to safely
 review all the code and certainly cannot scale, which CI and automation
 can address
 
 Keeping master stable is something no one would argue against and my
 point would match the original proposal from Alex. May be we can  have a
 staging branch for master and then merging the commit only after they
 have passed CI into master. The proposal got derailed and delayed because
 as called out at that time community does not want to work with a process
 that has a dependency on infrastructure that is not controlled by
 community. David and I are working to get the hardware from Citrix into
 ACS infra.
 
 The approach for fixing issues in release branch first and master later
 is not practical as we may miss out commits not made into master and
 future release regressing without the fixes. Also as the release goes
 into hardening cycle there will be a number of fixes which will not be
 allowed in release branch but need to be fixes for future, they should
 all go in master. Master is the catch all default branch and in my
 opinion should get fixes first.
 
 [1] http://markmail.org/thread/xoyjw2sduenlytwm
 
  Feature development need to be done outside master, period. Not only
 for
  non-committers but also for committers. And merge request need to be
  called. This will help review and avoid surprises.
 
 [Animesh] Completely Agreed
  New git workflow were proposed and shutdown, mostly calling for better
 CI
  to solve quality issues. CI will not solve our quality issues alone. We
 need to
  better police ourselves.
 
 [Animesh] I have already expressed my opinion in favor of CI
 
  To avoid long discussions, I propose this simple but drastic measure.
 We
  move all our commits to github PR until 4.5 is out, this stands for
  committers and non-committers, direct commits (especially to master)
  would be reverted immediately.
  
 
  Our development and release process is broken, we cannot continue like
  this, let's fix it.
 [Animesh] I  followed up the release process as established by Chip for
 4.2 and 4.3 release for which I was the RM and frankly I did not feel it
 that way other than the 

[GitHub] cloudstack pull request: CLOUDSTACK-7767: fixed events are not gen...

2014-10-30 Thread anshul1886
Github user anshul1886 commented on the pull request:

https://github.com/apache/cloudstack/pull/27#issuecomment-61217217
  
Yes, this should go to 4.5 as well. Somebody can cherry-pick this to 4.5.


---
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.
---


The SSVM cannot run succesfully in the development environment

2014-10-30 Thread 徐书敏
Hi

I have done the dev-environment building.when I configure the host,the 
Error :Monitor DownloaderListener says there is a error in the connect process 
for 1 due to failed to update state.before it there is a warn there is no 
seondnary storage vm for downloading template to image store .
In the secondnary storage host ,there are a  template file of vhd and a 
properties file of template.


Thanks,
XQ