RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-12 Thread Suresh Sadhu
Congrats Vijay!!

> 
> On Saturday 10 August 2013 10:30 AM, Prasanna Santhanam wrote:
> > The Project Management Committee (PMC) for Apache CloudStack has 
> > asked Vijay Bhamidipati to become a committer and we are pleased to 
> > announce that they have accepted.
> >
> > Being a committer allows many contributors to contribute more 
> > autonomously. For developers, it makes it easier to submit changes 
> > and eliminates the need to have contributions reviewed via the patch 
> > submission process. Whether contributions are development-related or 
> > otherwise, it is a recognition of a contributor's participation in 
> > the project and commitment to the project and the Apache Way.
> >
> > Please join me in congratulating Vijay!
> >
  


Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

2013-08-12 Thread Dave Cahill

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

Review request for cloudstack.


Repository: cloudstack-git


Description
---

Previously database password was specified as a no-argument option, however the 
rest of the script expected an argument.


Diffs
-

  scripts/storage/secondary/cloud-install-sys-tmplt 
7237ffa840878e3ce31e71964430fba38f93efb2 

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


Testing
---

Tested installing System VM template using the script and specifying db 
username / pass before change (didn't work) and after (worked).


Thanks,

Dave Cahill



Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

2013-08-12 Thread ASF Subversion and Git Services

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


Commit 4c193a1b6950932698270828a831fcd994c7ae4a in branch refs/heads/4.2 from 
Rajesh Battala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4c193a1 ]

CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting 
the volume if the source and destination pool are same


- ASF Subversion and Git Services


On Aug. 12, 2013, 6:48 a.m., Rajesh Battala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13469/
> ---
> 
> (Updated Aug. 12, 2013, 6:48 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.
> 
> 
> Bugs: CLOUDSTACK-3946
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> while migrating volume, if the storage and destination pool are same, we are 
> still do overhead of copying the volume to same pool and deleting the other 
> src copy.
> 
> fixed the issue by restricting volumemigration if the source and destination 
> pool are same.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13469/diff/
> 
> 
> Testing
> ---
> 
> 1. Successfully migrated the volume from Storagepool A to pool B.
> 2. migration is not allowed if the source and destination are same.
> 3. tried the volume with storage pool scope (ZONE and Cluster)
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>



[ACS42] Listtemplates API respone without any Zone configuration

2013-08-12 Thread Sailaja Mada
Hi,

I have the setup with latest code.  When I tried to view the list if templates 
without configuring zone , it is listing all the system vm templates and 
default cent OS templates .

I have not observed this behavior before. Is it introduced intentionally or is 
It a issue with API response ?

Please confirm.


2013-08-12 13:18:53,916 INFO  [cloud.api.ApiServer] (catalina-exec-3:null) 
(userId=2 accountId=2 sessionId=2C596448B0E7D3291B7FA76334A085A7) 10.144.7.12 
-- GET 
command=listTemplates&response=json&sessionkey=wMUCQaSUAhWg4JmNNXrmhPGIOzc%3D&listAll=true&page=1&pagesize=20&templatefilter=all&_=1376293976571
 200 { "listtemplatesresponse" : { "count":8 ,"template" : [  
{"id":"4b58341c-031a-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
(LXC)","displaytext":"SystemVM Template 
(LXC)","ispublic":false,"isready":false,"passwordenabled":false,"format":"QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
 GNU/Linux 5.0 
(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"LXC","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"2755de1f9ef2ce4d6f2bee2efbb4da92","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
 {"id":"ec8e64b6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
(XenServer)","displaytext":"SystemVM Template 
(XenServer)","ispublic":false,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ed81a824-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
 GNU/Linux 
7(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"XenServer","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"74b92f031cc5c2089ee89efb81344dcf","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
 {"id":"ec98be84-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
(HyperV)","displaytext":"SystemVM Template 
(HyperV)","ispublic":false,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
 GNU/Linux 5.0 
(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"Hyperv","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"fb1b6e032a160d86f2c28feb5add6d83","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
 {"id":"ec931e0c-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
(KVM)","displaytext":"SystemVM Template 
(KVM)","ispublic":false,"isready":false,"passwordenabled":false,"format":"QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
 GNU/Linux 5.0 
(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"KVM","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"6cea42b2633841648040becb588bd8f0","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
 {"id":"ec95eaf6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
(vSphere)","displaytext":"SystemVM Template 
(vSphere)","ispublic":false,"isready":false,"passwordenabled":false,"format":"OVA","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
 GNU/Linux 5.0 
(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"VMware","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"8fde62b1089e5844a9cd3b9b953f9596","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":true},
 {"id":"ec947a86-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.6(64-bit) no GUI 
(XenServer)","displaytext":"CentOS 5.6(64-bit) no GUI 
(XenServer)","ispublic":true,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11e3-b7b2-5641688b41e2","ostypename":"CentOS
 5.3 
(64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"XenServer","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":true,"checksum":"905cec879afd9c9d22ecc8036131a180","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":true},
 {"id":"ec95326e-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.3(64-bit) no GUI 
(vSphere)","displaytext":"CentOS 5.3(64-bit) no GUI 
(vSphere)","ispublic":true,"isready":false,"passwordenabled":false,"format":"OVA","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11e3-b7b2-5641688b41e2","ostypename":"CentOS
 5.3 
(64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"VMware","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":true,"checksum":"f6f881b7f2292948d8494db837fe0f47","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
 {"id":"ec93cb72-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.5(64-bit) no GUI 
(KVM)","displaytext":"CentOS 5.5(64-bit) no GUI

Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

2013-08-12 Thread ASF Subversion and Git Services

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


Commit be9dbcc2c5edb995c48558ea5e1a87c8d3012965 in branch refs/heads/master 
from Rajesh Battala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=be9dbcc ]

CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting 
the volume if the source and destination pool are same


- ASF Subversion and Git Services


On Aug. 12, 2013, 6:48 a.m., Rajesh Battala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13469/
> ---
> 
> (Updated Aug. 12, 2013, 6:48 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.
> 
> 
> Bugs: CLOUDSTACK-3946
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> while migrating volume, if the storage and destination pool are same, we are 
> still do overhead of copying the volume to same pool and deleting the other 
> src copy.
> 
> fixed the issue by restricting volumemigration if the source and destination 
> pool are same.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13469/diff/
> 
> 
> Testing
> ---
> 
> 1. Successfully migrated the volume from Storagepool A to pool B.
> 2. migration is not allowed if the source and destination are same.
> 3. tried the volume with storage pool scope (ZONE and Cluster)
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>



Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

2013-08-12 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On Aug. 12, 2013, 6:48 a.m., Rajesh Battala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13469/
> ---
> 
> (Updated Aug. 12, 2013, 6:48 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.
> 
> 
> Bugs: CLOUDSTACK-3946
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> while migrating volume, if the storage and destination pool are same, we are 
> still do overhead of copying the volume to same pool and deleting the other 
> src copy.
> 
> fixed the issue by restricting volumemigration if the source and destination 
> pool are same.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13469/diff/
> 
> 
> Testing
> ---
> 
> 1. Successfully migrated the volume from Storagepool A to pool B.
> 2. migration is not allowed if the source and destination are same.
> 3. tried the volume with storage pool scope (ZONE and Cluster)
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>



Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-12 Thread Ian Duffy
Congrats Vijay!


Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

2013-08-12 Thread Rajesh Battala


> On Aug. 12, 2013, 8:25 a.m., Devdeep Singh wrote:
> > Ship It!

Thanks for reviewing and pushing the changes.


- Rajesh


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


On Aug. 12, 2013, 6:48 a.m., Rajesh Battala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13469/
> ---
> 
> (Updated Aug. 12, 2013, 6:48 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.
> 
> 
> Bugs: CLOUDSTACK-3946
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> while migrating volume, if the storage and destination pool are same, we are 
> still do overhead of copying the volume to same pool and deleting the other 
> src copy.
> 
> fixed the issue by restricting volumemigration if the source and destination 
> pool are same.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13469/diff/
> 
> 
> Testing
> ---
> 
> 1. Successfully migrated the volume from Storagepool A to pool B.
> 2. migration is not allowed if the source and destination are same.
> 3. tried the volume with storage pool scope (ZONE and Cluster)
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>



Re: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

2013-08-12 Thread Hugo Trippaers

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

Ship it!


Ship It!

- Hugo Trippaers


On Aug. 12, 2013, 7:14 a.m., Dave Cahill wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13491/
> ---
> 
> (Updated Aug. 12, 2013, 7:14 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Previously database password was specified as a no-argument option, however 
> the rest of the script expected an argument.
> 
> 
> Diffs
> -
> 
>   scripts/storage/secondary/cloud-install-sys-tmplt 
> 7237ffa840878e3ce31e71964430fba38f93efb2 
> 
> Diff: https://reviews.apache.org/r/13491/diff/
> 
> 
> Testing
> ---
> 
> Tested installing System VM template using the script and specifying db 
> username / pass before change (didn't work) and after (worked).
> 
> 
> Thanks,
> 
> Dave Cahill
> 
>



Review Request 13493: CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.

2013-08-12 Thread Sanjay Tripathi

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

Review request for cloudstack and Devdeep Singh.


Bugs: CLOUDSTACK-2909


Repository: cloudstack-git


Description
---

CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.


Diffs
-

  services/console-proxy/server/js/ajaxkeys.js be3a5c9 

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


Testing
---

Verified the keys on IE9, Chrome and Firefox.


Thanks,

Sanjay Tripathi



Re: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

2013-08-12 Thread Hugo Trippaers


> On Aug. 12, 2013, 8:59 a.m., Hugo Trippaers wrote:
> > Ship It!

master 062eab8df44ba76dad28800599d87ef131bb0bec
4.2 9d053d04ecd08ca3ae7cbf6d6530cf96c3007687


- Hugo


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


On Aug. 12, 2013, 7:14 a.m., Dave Cahill wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13491/
> ---
> 
> (Updated Aug. 12, 2013, 7:14 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Previously database password was specified as a no-argument option, however 
> the rest of the script expected an argument.
> 
> 
> Diffs
> -
> 
>   scripts/storage/secondary/cloud-install-sys-tmplt 
> 7237ffa840878e3ce31e71964430fba38f93efb2 
> 
> Diff: https://reviews.apache.org/r/13491/diff/
> 
> 
> Testing
> ---
> 
> Tested installing System VM template using the script and specifying db 
> username / pass before change (didn't work) and after (worked).
> 
> 
> Thanks,
> 
> Dave Cahill
> 
>



Re: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

2013-08-12 Thread Dave Cahill
Thanks! I'm going for the smallest patch of the year prize (1 character). :)

On Mon, Aug 12, 2013 at 6:03 PM, Hugo Trippaers <
htrippa...@schubergphilis.com> wrote:

>This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13491/
>
> On August 12th, 2013, 8:59 a.m. UTC, *Hugo Trippaers* wrote:
>
> Ship It!
>
>  master 062eab8df44ba76dad28800599d87ef131bb0bec
> 4.2 9d053d04ecd08ca3ae7cbf6d6530cf96c3007687
>
>
> - Hugo
>
> On August 12th, 2013, 7:14 a.m. UTC, Dave Cahill wrote:
>   Review request for cloudstack.
> By Dave Cahill.
>
> *Updated Aug. 12, 2013, 7:14 a.m.*
>  *Repository: * cloudstack-git
> Description
>
> Previously database password was specified as a no-argument option, however 
> the rest of the script expected an argument.
>
>   Testing
>
> Tested installing System VM template using the script and specifying db 
> username / pass before change (didn't work) and after (worked).
>
>   Diffs
>
>- scripts/storage/secondary/cloud-install-sys-tmplt
>(7237ffa840878e3ce31e71964430fba38f93efb2)
>
> View Diff 
>


Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-12 Thread Wei ZHOU
Congrats Vijay!


2013/8/12 Ian Duffy 

> Congrats Vijay!
>


RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-12 Thread Hugo Trippaers
Congrats!

:D

> -Original Message-
> From: Wei ZHOU [mailto:ustcweiz...@gmail.com]
> Sent: maandag 12 augustus 2013 11:12
> To: dev@cloudstack.apache.org
> Subject: Re: [ANNOUNCE] New Committer: Vijay Bhamidipati
> 
> Congrats Vijay!
> 
> 
> 2013/8/12 Ian Duffy 
> 
> > Congrats Vijay!
> >


RE: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

2013-08-12 Thread Hugo Trippaers
:D  This is going to be hard to beat unless we start counting the changes in 
bits when indivdual characters change.


Cheers,

Hugo

From: Dave Cahill [mailto:dcah...@midokura.com]
Sent: maandag 12 augustus 2013 11:05
To: Hugo Trippaers
Cc: cloudstack
Subject: Re: Review Request 13491: Fix specifying db passwd in 
cloud-install-sys-tmplt

Thanks! I'm going for the smallest patch of the year prize (1 character). :)
On Mon, Aug 12, 2013 at 6:03 PM, Hugo Trippaers 
mailto:htrippa...@schubergphilis.com>> wrote:
This is an automatically generated e-mail. To reply, visit: 
https://reviews.apache.org/r/13491/



On August 12th, 2013, 8:59 a.m. UTC, Hugo Trippaers wrote:

Ship It!

master 062eab8df44ba76dad28800599d87ef131bb0bec

4.2 9d053d04ecd08ca3ae7cbf6d6530cf96c3007687


- Hugo


On August 12th, 2013, 7:14 a.m. UTC, Dave Cahill wrote:
Review request for cloudstack.
By Dave Cahill.

Updated Aug. 12, 2013, 7:14 a.m.
Repository: cloudstack-git
Description

Previously database password was specified as a no-argument option, however the 
rest of the script expected an argument.


Testing

Tested installing System VM template using the script and specifying db 
username / pass before change (didn't work) and after (worked).


Diffs

  *   scripts/storage/secondary/cloud-install-sys-tmplt 
(7237ffa840878e3ce31e71964430fba38f93efb2)

View Diff




Change history lost for upgrade sql script schema-412to420.sql

2013-08-12 Thread Koushik Das
I noticed that the upgrade sql script (earlier used to be schema-410to420.sql) 
has been renamed and in the process all the change history is lost. Can we 
ensure as a process to do a proper git file rename instead of creating a new 
file so that the changes history is not lost? In this case is there a way to 
retrieve the history?

-Koushik


Re: [ACS42] Listtemplates API respone without any Zone configuration

2013-08-12 Thread Harikrishna Patnala
We mark all the system and default templates as downloaded and during template 
sync we remove the template entries that are not there.
I guess this we have introduced recently, this may cause listing all the 
templates.

Thanks
Harikrishna
On 12-Aug-2013, at 1:46 PM, Sailaja Mada 
 wrote:

> Hi,
> 
> I have the setup with latest code.  When I tried to view the list if 
> templates without configuring zone , it is listing all the system vm 
> templates and default cent OS templates .
> 
> I have not observed this behavior before. Is it introduced intentionally or 
> is It a issue with API response ?
> 
> Please confirm.
> 
> 
> 2013-08-12 13:18:53,916 INFO  [cloud.api.ApiServer] (catalina-exec-3:null) 
> (userId=2 accountId=2 sessionId=2C596448B0E7D3291B7FA76334A085A7) 10.144.7.12 
> -- GET 
> command=listTemplates&response=json&sessionkey=wMUCQaSUAhWg4JmNNXrmhPGIOzc%3D&listAll=true&page=1&pagesize=20&templatefilter=all&_=1376293976571
>  200 { "listtemplatesresponse" : { "count":8 ,"template" : [  
> {"id":"4b58341c-031a-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
> (LXC)","displaytext":"SystemVM Template 
> (LXC)","ispublic":false,"isready":false,"passwordenabled":false,"format":"QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
>  GNU/Linux 5.0 
> (32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"LXC","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"2755de1f9ef2ce4d6f2bee2efbb4da92","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
>  {"id":"ec8e64b6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
> (XenServer)","displaytext":"SystemVM Template 
> (XenServer)","ispublic":false,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ed81a824-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
>  GNU/Linux 
> 7(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"XenServer","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"74b92f031cc5c2089ee89efb81344dcf","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
>  {"id":"ec98be84-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
> (HyperV)","displaytext":"SystemVM Template 
> (HyperV)","ispublic":false,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
>  GNU/Linux 5.0 
> (32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"Hyperv","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"fb1b6e032a160d86f2c28feb5add6d83","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
>  {"id":"ec931e0c-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
> (KVM)","displaytext":"SystemVM Template 
> (KVM)","ispublic":false,"isready":false,"passwordenabled":false,"format":"QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
>  GNU/Linux 5.0 
> (32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"KVM","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"6cea42b2633841648040becb588bd8f0","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":false},
>  {"id":"ec95eaf6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
> (vSphere)","displaytext":"SystemVM Template 
> (vSphere)","ispublic":false,"isready":false,"passwordenabled":false,"format":"OVA","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3-b7b2-5641688b41e2","ostypename":"Debian
>  GNU/Linux 5.0 
> (32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"VMware","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":false,"checksum":"8fde62b1089e5844a9cd3b9b953f9596","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":true},
>  {"id":"ec947a86-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.6(64-bit) no 
> GUI (XenServer)","displaytext":"CentOS 5.6(64-bit) no GUI 
> (XenServer)","ispublic":true,"isready":false,"passwordenabled":false,"format":"VHD","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11e3-b7b2-5641688b41e2","ostypename":"CentOS
>  5.3 
> (64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"XenServer","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextractable":true,"checksum":"905cec879afd9c9d22ecc8036131a180","tags":[],"sshkeyenabled":false,"isdynamicallyscalable":true},
>  {"id":"ec95326e-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.3(64-bit) no 
> GUI (vSphere)","displaytext":"CentOS 5.3(64-bit) no GUI 
> (vSphere)","ispublic":true,"isready":false,"passwordenabled":false,"format":"OVA","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11e3-b7b2-5641688b41e2","ostypename":"CentOS
>  5.3 
> (64-b

Re: Review Request 13318: Update documentation for LDAP authentication

2013-08-12 Thread Abhinandan Prateek

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

Ship it!


Ship It!

- Abhinandan Prateek


On Aug. 6, 2013, 8:45 a.m., Ian Duffy wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13318/
> ---
> 
> (Updated Aug. 6, 2013, 8:45 a.m.)
> 
> 
> Review request for cloudstack and Abhinandan Prateek.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Update documentation
> 
> 
> Diffs
> -
> 
>   docs/en-US/LDAP-for-user-authentication.xml 61f204c 
>   docs/en-US/example-activedirectory-configuration.xml 98ab5da 
>   docs/en-US/example-openldap-configuration.xml 2b28032 
>   docs/en-US/images/add-ldap-configuration-ad.png 
> 7d0aa0f44f994fe7a6539527ef42fc68b2d333f6 
>   docs/en-US/images/add-ldap-configuration-openldap.png 
> c801275dbc964f1d0a4637aaf59e9b7e860c6624 
> 
> Diff: https://reviews.apache.org/r/13318/diff/
> 
> 
> Testing
> ---
> 
> Build with publican.
> 
> 
> Thanks,
> 
> Ian Duffy
> 
>



Re: Review Request 13472: make apache server listen on all the ipaliases to provide access to meta-data.

2013-08-12 Thread bharat kumar

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

(Updated Aug. 12, 2013, 10:24 a.m.)


Review request for cloudstack, Chiradeep Vittal, Murali Reddy, and Sheng Yang.


Bugs: Cloudstack-4231


Repository: cloudstack-git


Description
---

https://issues.apache.org/jira/browse/CLOUDSTACK-4231
make apache server listen on all the ipaliases to provide access to meta-data.

Earlier we were using the name data-server to access the meta-data in case of 
multiple subnets. 
This requires updating the guest templates. 

Now we make apache server listen on all the ipaliases. So no need to update the 
template.
we create separate file for each ipAlias 


Diffs (updated)
-

  patches/systemvm/debian/config/etc/init.d/cloud-early-config a552d44 
  patches/systemvm/debian/config/root/createIpAlias.sh 5498195 
  patches/systemvm/debian/config/root/deleteIpAlias.sh fa228fb 

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


Testing
---

Tested on 4.2
deployed vms in different subnets.
used the script cloud-set-guest-sshkey.in to fetch the public keys form the 
router.
was able to get the keys without using the name data-server.

deleted the subnet and checked if the apache config was removed.

ran a failure case, all the config files generated during this operation were 
moved to failure_directory.


Thanks,

bharat kumar



RE: Change history lost for upgrade sql script schema-412to420.sql

2013-08-12 Thread Devdeep Singh
See if this helps 
https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=history;f=setup/db/db/schema-410to420.sql;h=60335318c7b792a401cab36b8b456e652d47f2a6;hb=e61a38087d158da51f7b945d16e4749a83512ab5

Regards,
Devdeep

> -Original Message-
> From: Koushik Das [mailto:koushik@citrix.com]
> Sent: Monday, August 12, 2013 2:50 PM
> To: cloudstack-...@incubator.apache.org
> Subject: Change history lost for upgrade sql script schema-412to420.sql
> 
> I noticed that the upgrade sql script (earlier used to be schema-410to420.sql)
> has been renamed and in the process all the change history is lost. Can we
> ensure as a process to do a proper git file rename instead of creating a new
> file so that the changes history is not lost? In this case is there a way to
> retrieve the history?
> 
> -Koushik


Re: Review Request 13472: make apache server listen on all the ipaliases to provide access to meta-data.

2013-08-12 Thread bharat kumar

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

(Updated Aug. 12, 2013, 10:42 a.m.)


Review request for cloudstack, Chiradeep Vittal, Jayapal Reddy, Murali Reddy, 
and Sheng Yang.


Changes
---

Added jayapal


Bugs: Cloudstack-4231


Repository: cloudstack-git


Description
---

https://issues.apache.org/jira/browse/CLOUDSTACK-4231
make apache server listen on all the ipaliases to provide access to meta-data.

Earlier we were using the name data-server to access the meta-data in case of 
multiple subnets. 
This requires updating the guest templates. 

Now we make apache server listen on all the ipaliases. So no need to update the 
template.
we create separate file for each ipAlias 


Diffs
-

  patches/systemvm/debian/config/etc/init.d/cloud-early-config a552d44 
  patches/systemvm/debian/config/root/createIpAlias.sh 5498195 
  patches/systemvm/debian/config/root/deleteIpAlias.sh fa228fb 

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


Testing
---

Tested on 4.2
deployed vms in different subnets.
used the script cloud-set-guest-sshkey.in to fetch the public keys form the 
router.
was able to get the keys without using the name data-server.

deleted the subnet and checked if the apache config was removed.

ran a failure case, all the config files generated during this operation were 
moved to failure_directory.


Thanks,

bharat kumar



RE: Change history lost for upgrade sql script schema-412to420.sql

2013-08-12 Thread Koushik Das
This is fine but is it possible to bring this history to the renamed file? Also 
one has to know that there existed a file 410to420.sql.

> -Original Message-
> From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
> Sent: Monday, August 12, 2013 4:07 PM
> To: dev@cloudstack.apache.org
> Subject: RE: Change history lost for upgrade sql script schema-412to420.sql
> 
> See if this helps https://git-wip-
> us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=history;f=setu
> p/db/db/schema-
> 410to420.sql;h=60335318c7b792a401cab36b8b456e652d47f2a6;hb=e61a38087
> d158da51f7b945d16e4749a83512ab5
> 
> Regards,
> Devdeep
> 
> > -Original Message-
> > From: Koushik Das [mailto:koushik@citrix.com]
> > Sent: Monday, August 12, 2013 2:50 PM
> > To: cloudstack-...@incubator.apache.org
> > Subject: Change history lost for upgrade sql script
> > schema-412to420.sql
> >
> > I noticed that the upgrade sql script (earlier used to be
> > schema-410to420.sql) has been renamed and in the process all the
> > change history is lost. Can we ensure as a process to do a proper git
> > file rename instead of creating a new file so that the changes history
> > is not lost? In this case is there a way to retrieve the history?
> >
> > -Koushik


Re: Review Request 12941: [GSoC] refactor gre controller

2013-08-12 Thread Hugo Trippaers

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

Ship it!


Ship It!

- Hugo Trippaers


On Aug. 1, 2013, 1:41 a.m., tuna wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12941/
> ---
> 
> (Updated Aug. 1, 2013, 1:41 a.m.)
> 
> 
> Review request for cloudstack, Sebastien Goasguen and Hugo Trippaers.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> I made an update to refactor gre controller:
> 
> + remove ovs_devices table, because we'll have an ODL plugin separately.
> + move command/answer to new package: com.cloud.agent.api
> + add Connectivity service checking
> + add new NetworkProvider: Ovs
> + add L3 services to Ovs Capabilities
> + add L3 services prototype code.
> 
> Next step:
> + L3 services implement with VirtualRouter
> + ODL plugin
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Network.java a06208b 
>   api/src/com/cloud/network/OvsProvider.java PRE-CREATION 
>   api/src/com/cloud/network/element/VirtualRouterElementService.java ea971b8 
>   api/src/org/apache/cloudstack/api/ResponseGenerator.java d8d07cb 
>   
> api/src/org/apache/cloudstack/api/command/admin/router/ConfigureOvsElementCmd.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/router/ListOvsElementsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/OvsProviderResponse.java 
> PRE-CREATION 
>   client/tomcatconf/applicationContext.xml.in 60f1e30 
>   client/tomcatconf/commands.properties.in 6ec4d92 
>   engine/schema/src/com/cloud/network/dao/OvsProviderDao.java PRE-CREATION 
>   engine/schema/src/com/cloud/network/dao/OvsProviderDaoImpl.java 
> PRE-CREATION 
>   engine/schema/src/com/cloud/network/element/OvsProviderVO.java PRE-CREATION 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
>  30b0521 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDeleteFlowCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyBridgeCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetupBridgeCommand.java
>  PRE-CREATION 
>   plugins/network-elements/ovs/src/com/cloud/agent/api/StartupOvsCommand.java 
> PRE-CREATION 
>   plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java 
> 0ea6b52 
>   
> plugins/network-elements/ovs/src/com/cloud/network/element/OvsElementService.java
>  b55fe6b 
>   
> plugins/network-elements/ovs/src/com/cloud/network/guru/OvsGuestNetworkGuru.java
>  bbdf110 
>   
> plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java
>  b1ecaac 
>   scripts/vm/hypervisor/xenserver/ovstunnel a4760b2 
>   server/src/com/cloud/api/ApiResponseHelper.java 090d9c2 
>   server/src/com/cloud/network/NetworkServiceImpl.java d4f9030 
>   server/src/com/cloud/network/element/VirtualRouterElement.java 1916678 
>   server/src/com/cloud/server/ManagementServerImpl.java 36b3879 
>   setup/db/create-schema.sql 143023a 
>   setup/db/db/schema-410to420.sql b48ae6a 
>   tools/apidoc/gen_toc.py 33a7e75 
>   ui/scripts/zoneWizard.js e2ad910 
> 
> Diff: https://reviews.apache.org/r/12941/diff/
> 
> 
> Testing
> ---
> 
> Test done with following cases:
> 
> + choose GRE method, VLAN range when deploy advanced zone
> + create new Network Offering and choose Ovs for VirtualNetworking service, 
> VirtualRouter for other services.
> + create a new Guest network using this new NetworkOffering
> + deploy VMs on this Guest network
> + stop, restart, destroy VMs
> + deploy, stop, restart, destroy VMs on two hosts.
> 
> 
> Thanks,
> 
> tuna
> 
>



Re: Review Request 12941: [GSoC] refactor gre controller

2013-08-12 Thread Hugo Trippaers


> On Aug. 12, 2013, 11:30 a.m., Hugo Trippaers wrote:
> > Ship It!

Targeted to the sdnextensions branch.

Commit id : 0d6125f36449b1fcbbc5d296496c4a01f2208fc4


- Hugo


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


On Aug. 1, 2013, 1:41 a.m., tuna wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12941/
> ---
> 
> (Updated Aug. 1, 2013, 1:41 a.m.)
> 
> 
> Review request for cloudstack, Sebastien Goasguen and Hugo Trippaers.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> I made an update to refactor gre controller:
> 
> + remove ovs_devices table, because we'll have an ODL plugin separately.
> + move command/answer to new package: com.cloud.agent.api
> + add Connectivity service checking
> + add new NetworkProvider: Ovs
> + add L3 services to Ovs Capabilities
> + add L3 services prototype code.
> 
> Next step:
> + L3 services implement with VirtualRouter
> + ODL plugin
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Network.java a06208b 
>   api/src/com/cloud/network/OvsProvider.java PRE-CREATION 
>   api/src/com/cloud/network/element/VirtualRouterElementService.java ea971b8 
>   api/src/org/apache/cloudstack/api/ResponseGenerator.java d8d07cb 
>   
> api/src/org/apache/cloudstack/api/command/admin/router/ConfigureOvsElementCmd.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/router/ListOvsElementsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/OvsProviderResponse.java 
> PRE-CREATION 
>   client/tomcatconf/applicationContext.xml.in 60f1e30 
>   client/tomcatconf/commands.properties.in 6ec4d92 
>   engine/schema/src/com/cloud/network/dao/OvsProviderDao.java PRE-CREATION 
>   engine/schema/src/com/cloud/network/dao/OvsProviderDaoImpl.java 
> PRE-CREATION 
>   engine/schema/src/com/cloud/network/element/OvsProviderVO.java PRE-CREATION 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
>  30b0521 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDeleteFlowCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyBridgeCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyTunnelCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowAnswer.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowCommand.java
>  PRE-CREATION 
>   
> plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetupBridgeCommand.java
>  PRE-CREATION 
>   plugins/network-elements/ovs/src/com/cloud/agent/api/StartupOvsCommand.java 
> PRE-CREATION 
>   plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java 
> 0ea6b52 
>   
> plugins/network-elements/ovs/src/com/cloud/network/element/OvsElementService.java
>  b55fe6b 
>   
> plugins/network-elements/ovs/src/com/cloud/network/guru/OvsGuestNetworkGuru.java
>  bbdf110 
>   
> plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java
>  b1ecaac 
>   scripts/vm/hypervisor/xenserver/ovstunnel a4760b2 
>   server/src/com/cloud/api/ApiResponseHelper.java 090d9c2 
>   server/src/com/cloud/network/NetworkServiceImpl.java d4f9030 
>   server/src/com/cloud/network/element/VirtualRouterElement.java 1916678 
>   server/src/com/cloud/server/ManagementServerImpl.java 36b3879 
>   setup/db/create-schema.sql 143023a 
>   setup/db/db/schema-410to420.sql b48ae6a 
>   tools/apidoc/gen_toc.py 33a7e75 
>   ui/scripts/zoneWizard.js e2ad910 
> 
> Diff: https://reviews.apache.org/r/12941/diff/
> 
> 
> Testing
> ---
> 
> Test done with following cases:
> 
> + choose GRE method, VLAN range when deploy advanced zone
> + create new Network Offering and choose Ovs for VirtualNetworking service, 
> VirtualRouter for other services.
> + create a new Guest network using this new NetworkOffering
> + deploy VMs on this Guest network
> + stop, restart, destroy VMs
> + deploy, stop, restart

Re: Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-08-12 Thread Hugo Trippaers

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

Ship it!


I'm also OK with this patch. Documentation is pending, but i'm fine with 
committing this to master. It needs a rebase to the latest state of master, but 
after that we should commit this.  OK Chiradeep?

- Hugo Trippaers


On July 23, 2013, 6:45 a.m., Toshiaki Hatano wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12623/
> ---
> 
> (Updated July 23, 2013, 6:45 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Chiradeep Vittal, Murali 
> Reddy, Hugo Trippaers, and Sheng Yang.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-2328
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor
> 
> Initial patch for VXLAN support.
> Fully functional, hopefully, for GuestNetwork - AdvancedZone.
> 
> Patch Note:
>  in cloudstack-server
> - Add isolation method VXLAN
> - Add VxlanGuestNetworkGuru as plugin for VXLAN isolation
> - Modify NetworkServiceImpl to handle extended vNet range for VXLAN isolation
> - Add VXLAN isolation option in zoneWizard UI
> 
>  in cloudstack-agent (kvm)
> - Add modifyvxlan.sh script that handle bridge/vxlan interface manipulation 
> script
> -- Usage is exactly same to modifyvlan.sh
> - BridgeVifDriver will call modifyvxlan.sh instead of modifyvlan.sh when 
> VXLAN is used for isolation
> 
> Database changes:
> - No change in database structure.
> - VXLAN isolation uses same tables that VLAN uses to store vNet allocation 
> status.
> 
> Known Issue:
> - Some resource still says 'VLAN' in log even if VXLAN is used
> - in UI, "Network - GuestNetworks" dosen't display VNI
> -- VLAN ID field displays "N/A"
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Networks.java c76c3d4 
>   api/src/com/cloud/network/PhysicalNetwork.java f6cb1a6 
>   client/pom.xml 32ab94a 
>   client/tomcatconf/componentContext.xml.in 1fbec61 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java
>  195cf40 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
>  571bcc8 
>   plugins/network-elements/vxlan/pom.xml PRE-CREATION 
>   
> plugins/network-elements/vxlan/src/com/cloud/network/guru/VxlanGuestNetworkGuru.java
>  PRE-CREATION 
>   
> plugins/network-elements/vxlan/test/com/cloud/network/guru/VxlanGuestNetworkGuruTest.java
>  PRE-CREATION 
>   plugins/pom.xml ff1e9c9 
>   scripts/vm/network/vnet/modifyvlan.sh 8ed3905 
>   scripts/vm/network/vnet/modifyvxlan.sh PRE-CREATION 
>   server/src/com/cloud/network/NetworkManagerImpl.java 30c4242 
>   server/src/com/cloud/network/NetworkServiceImpl.java 23bed01 
>   ui/scripts/ui-custom/zoneWizard.js bba6c04 
> 
> Diff: https://reviews.apache.org/r/12623/diff/
> 
> 
> Testing
> ---
> 
> #) Test set up
> - Components
>   - 1x management server
>   - 1x nfs storage
>   - 3x Linux KVM host
>   -- CentOS 6.4 based
>   -- Replace kernel by version 3.8.13, VXLAN kernel module built as loadable 
> module
>   -- Replace iproute2 by version iproute2-ss130430
>   -- BridgeVifDriver (Default)
> 1. create advanced zone from zone wizard without security group option
> 2. select hypervisor: KVM
> 3. assign Guest network to separated physical network, isolated by VXLAN
>specify bridge name (traffic label) for Guest network, this bridge should 
> have IPv4 address (global/private both are OK).
> 4. assign Guest vNet range 1-2
> 5. other parameter are normal
> 6. add 2 more hosts into same zone/pod/cluster after zone wizard is finished
> 
> #) Test case 1: start/stop VR
> 1. Create network offering, same configuration as 
> DefaultIsolatedNetworkOfferingWithSourceNatService but persistent
> 2. Create network with network offering which is created in step 0
> 3. Confirm VR is started and bridge/vxlan device created on host
> 4. Delete network which is created in step 1
> 5. Confirm VR is deleted and bridge/vxlan device deleted on host
> 
> #) Test case 2: start/stop an instance (VR is on same host)
> 1. Add an instance from UI, create network during wizard.
> 2. Confirm VM and VR are on the same host
> 3. Confirm it's pingable from VM to VR
> 4. Confirm it's pingable from VM to public network (after opening Egress rule)
> 5. Destroy instance
> 6. Confirm bridge/vxlan device is still on the host
> 7. Delete network after the VM is expunged
> 8. Confirm VR are deleted and bridge/vxlan device deleted on the host
> 
> #) Test case 3: start/stop an instance (VR is on different host)
> 1. Add an instance from UI, create network during wizard.
> 2. Confirm VM and VR are on the different

Review Request 13495: CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a domain, shows other zones dedicated as well

2013-08-12 Thread Saksham Srivastava

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

Review request for cloudstack and Devdeep Singh.


Bugs: CLOUDSTACK-4189


Repository: cloudstack-git


Description
---

Listing out dedicated zones with zone id is returning all dedicated zones.
Issue: search criteria was always passing null for the zoneid.
Fixed the issue by setting zoneId inplace of datacenter id.


Diffs
-

  server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java 6c91ce7 

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


Testing
---

list dedicatedzone with zoneid now returns correct dedicated zone.


Thanks,

Saksham Srivastava



Re: Master build failure

2013-08-12 Thread Hugo Trippaers
Prassana,

What is broken? Do you have some details (just got back from some time off, so 
i might have missed a few mails)

By the way on the subject on gerrit, we had this discussion before. If anyone 
wants to refresh the arguments from last time: 
http://markmail.org/message/inerurmjtc6v57ba

Cheers,

Hugo


On Aug 10, 2013, at 7:15 AM, Prasanna Santhanam  wrote:

> The jenkins integration is broken atm. If someone has time to look
> into it please send a pull request with the fix here:
> https://github.com/CloudStack-extras/reviewboard-tools/issues/2
> 
> On Fri, Aug 09, 2013 at 10:35:01AM +0900, Dave Cahill wrote:
>> Hi,
>> 
>> Alex - sounds good, let's raise the Gerrit topic in a new thread after 4.2.
>> 
>> Daan - you're right that if Jenkins builds, that should prove the code
>> compiles cleanly.
>> Have we actually been gating commits on the catofong report though? I
>> looked at the
>> Jenkins build behind it [1], and it hasn't succeeded in 4 days, so I'm
>> guessing we're
>> not gating based on it.
>> 
>> Aside from the Jenkins integration, one of the reasons I wanted Gerrit is
>> to put a step
>> between writing code and it hitting master. Ideally, it would be nice to
>> enforce that a
>> positive code review from someone other than the author is present before
>> pushing.
>> 
>> Thanks,
>> Dave.
>> 
>> [1]
>> http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/
>> 
>> On Fri, Aug 9, 2013 at 4:16 AM, Daan Hoogland wrote:
>> 
>>> Dave, wouldn't jenkins do a build like that? Meanng the positive cato
>>> report is a must to allow commits
>>> 
>>> On Thu, Aug 8, 2013 at 4:54 PM, Alex Huang  wrote:
 Yup.  That was the reason why it worked on my machine but doesn't work
>>> on jenkins.  It had a copy of the jar in the local maven cache so it
>>> continued to build even though it's missing.  Thanks for reverting.  Will
>>> fix.
 
 I still don't understand why we have problems with Gerrit.  It would
>>> have prevented exactly this type of problems.  I know a lot of people are
>>> busy on 4.2.  Will bring it up post 4.2 again.
 
 --Alex
 
 
 
 From: Dave Cahill [mailto:dcah...@midokura.com]
 Sent: Thursday, August 8, 2013 3:45 AM
 To: dev@cloudstack.apache.org; Dave Cahill; Alex Huang
 Subject: Re: Master build failure
 
 Thanks Prasanna.
 
 One suggestion I have is that if anyone makes pom file changes (or any
>>> large changes at
 all, to be honest), that they nuke their local Maven cache (rm -rf
>>> ~/.m2/repository) and do
 a "clean install" before submitting a patch. This way, you don't end up
>>> thinking your work
 builds when it just happens to build due to leftover, cached jars.
 
 Alex, do you have any ideas on build stability? There have been 2 build
>>> breakages in the
 past few days (ref: "compile error on 4.2" thread), and looking at git
>>> log I don't see much
 sign of review on the changes. We discussed Gerrit a while back, but I
>>> think the conclusion
 was that it would slow us down?
 
 Thanks,
 Dave.
 
 On Thu, Aug 8, 2013 at 6:13 PM, Prasanna Santhanam >> > wrote:
 Reverted the range 30c33415..f6a2c817bc. Master builds now. Others
 who've checked in after these will still retain their commits but
 please build before push. It's clear we are only worried about 4.2 now
 but we are breaking master for those who are beginning to work ahead
 for 4.3
 
 On Thu, Aug 08, 2013 at 02:15:58PM +0530, Prasanna Santhanam wrote:
> Dave's been attempting to fix this and it looks to be intertwined in a
> few commits. Large swaths of code has moved around. I'll revert to
> last known state from Jenkins if we can't figure out a simple fix so
> Dave can proceed.
> 
> On Thu, Aug 08, 2013 at 04:11:36PM +0900, Dave Cahill wrote:
>> Hi,
>> 
>> As Prasanna noted in IRC, the cloudstack-master-maven Jenkins build is
>> failing with a similar error:
>> 
>> https://builds.apache.org/job/cloudstack-master-maven/2645/console
>> 
>> Failed to execute goal on project cloud-engine-components-api: Could
>> not resolve dependencies for project
>> org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
>> Could not find artifact
>> org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
>> Nexus (http://repository.apache.org/snapshots)
>> cause : Could not resolve dependencies for project
>> org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
>> Could not find artifact
>> org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
>> Nexus (http://repository.apache.org/snapshots)
>> 
>> 
>> Thanks,
>> 
>> Dave.
>> 
>> 
>> On Thu, Aug 8, 2013 at 3:53 PM, Dave Cahill >> > wrote:
>> 
>>> Hi,
>>> 
>>> I'm 

running BVT locally

2013-08-12 Thread Daan Hoogland
H, (Prasanna?)

Can I run the integration tests at Schuberg Philis' and what do I need
to do so? If there is document on how to set up these (and smoke
tests) can I get a link?

You may know; I want to get some work on vpc gateways on sdn accepted.

regards,


more review feedback for 12849?

2013-08-12 Thread Daan Hoogland
Is there more to say about
https://reviews.apache.org/r/12849/
?

Is it commit worthy? or if not what is missing?


Re: Review Request 13493: CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.

2013-08-12 Thread ASF Subversion and Git Services

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


Commit 86d2e8bdae1e7edef013a9cf01dbf2dee7528531 in branch refs/heads/4.2 from 
Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=86d2e8b ]

CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.


- ASF Subversion and Git Services


On Aug. 12, 2013, 9:02 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13493/
> ---
> 
> (Updated Aug. 12, 2013, 9:02 a.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js be3a5c9 
> 
> Diff: https://reviews.apache.org/r/13493/diff/
> 
> 
> Testing
> ---
> 
> Verified the keys on IE9, Chrome and Firefox.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13495: CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a domain, shows other zones dedicated as well

2013-08-12 Thread ASF Subversion and Git Services

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


Commit 2eceb7dcbe767f0f8a6cc3fb15655fb10d330406 in branch refs/heads/4.2 from 
Saksham Srivastava
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2eceb7d ]

CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a 
domain, shows other zones dedicated as well


- ASF Subversion and Git Services


On Aug. 12, 2013, 12:28 p.m., Saksham Srivastava wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13495/
> ---
> 
> (Updated Aug. 12, 2013, 12:28 p.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4189
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Listing out dedicated zones with zone id is returning all dedicated zones.
> Issue: search criteria was always passing null for the zoneid.
> Fixed the issue by setting zoneId inplace of datacenter id.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java 6c91ce7 
> 
> Diff: https://reviews.apache.org/r/13495/diff/
> 
> 
> Testing
> ---
> 
> list dedicatedzone with zoneid now returns correct dedicated zone.
> 
> 
> Thanks,
> 
> Saksham Srivastava
> 
>



Re: Review Request 13493: CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.

2013-08-12 Thread ASF Subversion and Git Services

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


Commit ead3b848d5652b4b8b2619e617af2bb9f025fc26 in branch refs/heads/master 
from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=ead3b84 ]

CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.


- ASF Subversion and Git Services


On Aug. 12, 2013, 9:02 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13493/
> ---
> 
> (Updated Aug. 12, 2013, 9:02 a.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js be3a5c9 
> 
> Diff: https://reviews.apache.org/r/13493/diff/
> 
> 
> Testing
> ---
> 
> Verified the keys on IE9, Chrome and Firefox.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13495: CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a domain, shows other zones dedicated as well

2013-08-12 Thread ASF Subversion and Git Services

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


Commit 5510bca9e5223730dc469b443b50d26550533360 in branch refs/heads/master 
from Saksham Srivastava
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5510bca ]

CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a 
domain, shows other zones dedicated as well


- ASF Subversion and Git Services


On Aug. 12, 2013, 12:28 p.m., Saksham Srivastava wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13495/
> ---
> 
> (Updated Aug. 12, 2013, 12:28 p.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4189
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Listing out dedicated zones with zone id is returning all dedicated zones.
> Issue: search criteria was always passing null for the zoneid.
> Fixed the issue by setting zoneId inplace of datacenter id.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java 6c91ce7 
> 
> Diff: https://reviews.apache.org/r/13495/diff/
> 
> 
> Testing
> ---
> 
> list dedicatedzone with zoneid now returns correct dedicated zone.
> 
> 
> Thanks,
> 
> Saksham Srivastava
> 
>



Re: Review Request 13493: CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.

2013-08-12 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On Aug. 12, 2013, 9:02 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13493/
> ---
> 
> (Updated Aug. 12, 2013, 9:02 a.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js be3a5c9 
> 
> Diff: https://reviews.apache.org/r/13493/diff/
> 
> 
> Testing
> ---
> 
> Verified the keys on IE9, Chrome and Firefox.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13495: CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a domain, shows other zones dedicated as well

2013-08-12 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On Aug. 12, 2013, 12:28 p.m., Saksham Srivastava wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13495/
> ---
> 
> (Updated Aug. 12, 2013, 12:28 p.m.)
> 
> 
> Review request for cloudstack and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4189
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Listing out dedicated zones with zone id is returning all dedicated zones.
> Issue: search criteria was always passing null for the zoneid.
> Fixed the issue by setting zoneId inplace of datacenter id.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java 6c91ce7 
> 
> Diff: https://reviews.apache.org/r/13495/diff/
> 
> 
> Testing
> ---
> 
> list dedicatedzone with zoneid now returns correct dedicated zone.
> 
> 
> Thanks,
> 
> Saksham Srivastava
> 
>



Re: Review Request 13472: make apache server listen on all the ipaliases to provide access to meta-data.

2013-08-12 Thread Jayapal Reddy

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

Ship it!


Ship It!

- Jayapal Reddy


On Aug. 12, 2013, 10:42 a.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13472/
> ---
> 
> (Updated Aug. 12, 2013, 10:42 a.m.)
> 
> 
> Review request for cloudstack, Chiradeep Vittal, Jayapal Reddy, Murali Reddy, 
> and Sheng Yang.
> 
> 
> Bugs: Cloudstack-4231
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-4231
> make apache server listen on all the ipaliases to provide access to meta-data.
> 
> Earlier we were using the name data-server to access the meta-data in case of 
> multiple subnets. 
> This requires updating the guest templates. 
> 
> Now we make apache server listen on all the ipaliases. So no need to update 
> the template.
> we create separate file for each ipAlias 
> 
> 
> Diffs
> -
> 
>   patches/systemvm/debian/config/etc/init.d/cloud-early-config a552d44 
>   patches/systemvm/debian/config/root/createIpAlias.sh 5498195 
>   patches/systemvm/debian/config/root/deleteIpAlias.sh fa228fb 
> 
> Diff: https://reviews.apache.org/r/13472/diff/
> 
> 
> Testing
> ---
> 
> Tested on 4.2
> deployed vms in different subnets.
> used the script cloud-set-guest-sshkey.in to fetch the public keys form the 
> router.
> was able to get the keys without using the name data-server.
> 
> deleted the subnet and checked if the apache config was removed.
> 
> ran a failure case, all the config files generated during this operation were 
> moved to failure_directory.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>



Review Request 13496: changes for guest vm password script for parallel vm deployment

2013-08-12 Thread Jayapal Reddy

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

Review request for cloudstack, anthony xu, Abhinandan Prateek, Chiradeep 
Vittal, and Sheng Yang.


Bugs: CLOUDSTACK-4184


Repository: cloudstack-git


Description
---

For parallel vm deployment guest vm password script is fixed by retrying after 
random sleep on failure.

Please review the changes and provide your comments so that I can commit this.

For windows guest VM script, changes for parallel vm deployment is not added.


Diffs
-

  setup/bindir/cloud-set-guest-password.in 3215894 

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


Testing
---

Tested by deploying 30 vms. All vms set its password successfully.


Thanks,

Jayapal Reddy



Re: Review Request 13490: [DOC] add document - custom cpu model for kvm guest.

2013-08-12 Thread Wei Zhou

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

Ship it!


Ship It!

- Wei Zhou


On Aug. 12, 2013, 6:57 a.m., Jijun wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13490/
> ---
> 
> (Updated Aug. 12, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack and Wei Zhou.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> add document for custom cpu model for kvm guest.
> 
> 
> Diffs
> -
> 
>   docs/en-US/hypervisor-host-install-agent.xml 41b6719 
> 
> Diff: https://reviews.apache.org/r/13490/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jijun
> 
>



Re: Review Request 13490: [DOC] add document - custom cpu model for kvm guest.

2013-08-12 Thread Wei Zhou


> On Aug. 12, 2013, 1:37 p.m., Wei Zhou wrote:
> > Ship It!

committed: 8d53887cec514e1eeed899e4d6b6465b02aa39a3 in master, and 
df1756b6197f42abff6a01c55a1b1cbfc9d83eb5 in 4.2


- Wei


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


On Aug. 12, 2013, 6:57 a.m., Jijun wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13490/
> ---
> 
> (Updated Aug. 12, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack and Wei Zhou.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> add document for custom cpu model for kvm guest.
> 
> 
> Diffs
> -
> 
>   docs/en-US/hypervisor-host-install-agent.xml 41b6719 
> 
> Diff: https://reviews.apache.org/r/13490/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jijun
> 
>



Did we already started developing support VxLAN?

2013-08-12 Thread 曹建林

 Hi guys

 My company has a project to delivery support of VxLAN network , using 
CloudStack.
I know the version 4.x does not support vxlan. 
So I wonder what the plan to support vxlan network .

We have a team about 5 guys in charge of developing the feature,

So  if there is anybody in this thing . please send me email , thank you very 
much.


Thanks.


Re: Did we already started developing support VxLAN?

2013-08-12 Thread David Nalley
On Mon, Aug 12, 2013 at 9:44 AM, 曹建林  wrote:
>
>  Hi guys
>
>  My company has a project to delivery support of VxLAN network , using 
> CloudStack.
> I know the version 4.x does not support vxlan.
> So I wonder what the plan to support vxlan network .
>
> We have a team about 5 guys in charge of developing the feature,
>
> So  if there is anybody in this thing . please send me email , thank you very 
> much.
>

Toshiaki from Verio has been working on VXLAN in KVM.
You can see his patch here:
https://reviews.apache.org/r/12623/
https://issues.apache.org/jira/browse/CLOUDSTACK-2328

I'm sure he'd welcome help.

--David


Re: Master build failure

2013-08-12 Thread Prasanna Santhanam
Apart from false failures because of wrongly targetted branch I saw the
job fail the build on master for patches that were to be applied on
master. Haven't seen the job reporting in the last few days however. 

Have you started on the fix for the branch mapping? If not - I could
start taking a look sometime this week.

On Mon, Aug 12, 2013 at 02:30:10PM +0200, Hugo Trippaers wrote:
> Prassana,
> 
> What is broken? Do you have some details (just got back from some
> time off, so i might have missed a few mails)
> 
> By the way on the subject on gerrit, we had this discussion before.
> If anyone wants to refresh the arguments from last time:
> http://markmail.org/message/inerurmjtc6v57ba
> 
> Cheers,
> 
> Hugo
> 
> 
> On Aug 10, 2013, at 7:15 AM, Prasanna Santhanam  wrote:
> 
> > The jenkins integration is broken atm. If someone has time to look
> > into it please send a pull request with the fix here:
> > https://github.com/CloudStack-extras/reviewboard-tools/issues/2
> > 
> > On Fri, Aug 09, 2013 at 10:35:01AM +0900, Dave Cahill wrote:
> >> Hi,
> >> 
> >> Alex - sounds good, let's raise the Gerrit topic in a new thread after 4.2.
> >> 
> >> Daan - you're right that if Jenkins builds, that should prove the code
> >> compiles cleanly.
> >> Have we actually been gating commits on the catofong report though? I
> >> looked at the
> >> Jenkins build behind it [1], and it hasn't succeeded in 4 days, so I'm
> >> guessing we're
> >> not gating based on it.
> >> 
> >> Aside from the Jenkins integration, one of the reasons I wanted Gerrit is
> >> to put a step
> >> between writing code and it hitting master. Ideally, it would be nice to
> >> enforce that a
> >> positive code review from someone other than the author is present before
> >> pushing.
> >> 
> >> Thanks,
> >> Dave.
> >> 
> >> [1]
> >> http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/
> >> 
> >> On Fri, Aug 9, 2013 at 4:16 AM, Daan Hoogland 
> >> wrote:
> >> 
> >>> Dave, wouldn't jenkins do a build like that? Meanng the positive cato
> >>> report is a must to allow commits
> >>> 
> >>> On Thu, Aug 8, 2013 at 4:54 PM, Alex Huang  wrote:
>  Yup.  That was the reason why it worked on my machine but doesn't work
> >>> on jenkins.  It had a copy of the jar in the local maven cache so it
> >>> continued to build even though it's missing.  Thanks for reverting.  Will
> >>> fix.
>  
>  I still don't understand why we have problems with Gerrit.  It would
> >>> have prevented exactly this type of problems.  I know a lot of people are
> >>> busy on 4.2.  Will bring it up post 4.2 again.
>  
>  --Alex
>  
>  
>  
>  From: Dave Cahill [mailto:dcah...@midokura.com]
>  Sent: Thursday, August 8, 2013 3:45 AM
>  To: dev@cloudstack.apache.org; Dave Cahill; Alex Huang
>  Subject: Re: Master build failure
>  
>  Thanks Prasanna.
>  
>  One suggestion I have is that if anyone makes pom file changes (or any
> >>> large changes at
>  all, to be honest), that they nuke their local Maven cache (rm -rf
> >>> ~/.m2/repository) and do
>  a "clean install" before submitting a patch. This way, you don't end up
> >>> thinking your work
>  builds when it just happens to build due to leftover, cached jars.
>  
>  Alex, do you have any ideas on build stability? There have been 2 build
> >>> breakages in the
>  past few days (ref: "compile error on 4.2" thread), and looking at git
> >>> log I don't see much
>  sign of review on the changes. We discussed Gerrit a while back, but I
> >>> think the conclusion
>  was that it would slow us down?
>  
>  Thanks,
>  Dave.
>  
>  On Thu, Aug 8, 2013 at 6:13 PM, Prasanna Santhanam  >>> > wrote:
>  Reverted the range 30c33415..f6a2c817bc. Master builds now. Others
>  who've checked in after these will still retain their commits but
>  please build before push. It's clear we are only worried about 4.2 now
>  but we are breaking master for those who are beginning to work ahead
>  for 4.3
>  
>  On Thu, Aug 08, 2013 at 02:15:58PM +0530, Prasanna Santhanam wrote:
> > Dave's been attempting to fix this and it looks to be intertwined in a
> > few commits. Large swaths of code has moved around. I'll revert to
> > last known state from Jenkins if we can't figure out a simple fix so
> > Dave can proceed.
> > 
> > On Thu, Aug 08, 2013 at 04:11:36PM +0900, Dave Cahill wrote:
> >> Hi,
> >> 
> >> As Prasanna noted in IRC, the cloudstack-master-maven Jenkins build is
> >> failing with a similar error:
> >> 
> >> https://builds.apache.org/job/cloudstack-master-maven/2645/console
> >> 
> >> Failed to execute goal on project cloud-engine-components-api: Could
> >> not resolve dependencies for project
> >> org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
> >> Could not find artifact
>

Re: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Laszlo Hornyak

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


Hi Reviewers,

Any feedback on this patch? :)



- Laszlo Hornyak


On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11942/
> ---
> 
> (Updated July 24, 2013, 4:42 a.m.)
> 
> 
> Review request for cloudstack, Frank Zhang and John Burwell.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - possible resource leak closed
> - file content read uses now commons-lang FileUtils
> - Added unit tests
> 
> 
> Diffs
> -
> 
>   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35 
>   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/11942/diff/
> 
> 
> Testing
> ---
> 
> test included
> 
> 
> Thanks,
> 
> Laszlo Hornyak
> 
>



[Responsiveness report] users 2013w32

2013-08-12 Thread Daan Hoogland
only two unanswerred mails on users last week:

http://markmail.org/message/rxl5mwhj2xgmm24e Does CS support Xen XCP?
by Jason Pavao
http://markmail.org/message/pgchyajufif5ylwi Why can't I choose a
different network offering? by Nux!

regards,
Daan


Re: understanding cloudstack networking

2013-08-12 Thread Daan Hoogland
sounds great Ron,

I'm sure I am not the guy you need for this but I'll keep an eye on
it. The 'someone who actually knows what it s supposed to say' is
bound to be around on this list or dev.

regards,
Daan


On Mon, Aug 12, 2013 at 4:55 PM, Ron Wheeler
 wrote:
> I have been reading and correcting the posted 4.2 documentation changes that
> I understand or where the English errors are very clear.
>
> I have filed JIRAs and some have been fixed.
>
> I would be willing to participate in a workshop to walk through the
> installation with someone who actually knows what it s supposed to say.
>
> Ron
>
>
>
> On 12/08/2013 10:41 AM, Daan Hoogland wrote:
>>
>> you are right Ron, but even those companies/people can only spend
>> their time once. So please submit you improvements whenever you can.
>>
>> regards,
>> Daan
>>
>> On Mon, Aug 12, 2013 at 4:32 PM, Ron Wheeler
>>  wrote:
>>>
>>> On 12/08/2013 10:08 AM, Travis Graham wrote:

 One of the most confusing things I've ran into, past the fact the
 documentation is wrong about 80% of the time, is the mix of CentOS and
 Ubuntu instructions.

 I think splitting things out into their own OS specific install guides
 would reduce a lot of confusion.
>>>
>>> Yes.
>>>
 I was browsing the 4.2 docs in the repo this weekend and I'm still not
 seeing swath of the incorrect info being updated. Maybe things that
 haven't
 been rolled into the 4.2 branch yet.
>>>
>>> I hope that this gets done. It is the biggest problem that CloudStack has
>>> in
>>> getting traction.
>>>
>>> You only get one chance to make a first impression and the impression at
>>> the
>>> moment is that the system does not work and is not ready for prime time
>>> except for organizations that have a development group ready to read the
>>> code and fix the docs for their installation.
>>>
>>> Ron
>>>
>>>
 Travis

 On Aug 12, 2013, at 9:59 AM, Ron Wheeler
 
 wrote:

> The documentation is wrong which is a big problem.
>
> It is also confusing with extraneous stuff stuck in the middle and
> missing introductory information to explain where the instructions are
> leading.
>
> There seems to be a big effort to get 4.2 out with accurate docs and I
> hope more clarifying text and drawings.
>
> It appears that there is a lot of effort going into external Wiki
> documentation to make up for the state of the manuals.
>
> Ron
>
>
> On 12/08/2013 4:10 AM, Mark van der Meulen wrote:
>>
>> Hi,
>>
>> I am having a little trouble understanding how the cloudstack
>> networking
>> model works, I have read the documentation and enquired on IRC(without
>> response) and still don't really get it. I suspect if I was able to
>> setup
>> CloudStack and play with it I would understand, however given that I
>> have to
>> go through a complex networking setup to get the Zone/Pod/Cluster/Host
>> even
>> setup to start with, I haven't been able to get far enough in to start
>> playing.
>>
>> Based on what I have read, I think I would like to setup a Public
>> Cloud,
>> essentially some hypervisors on a private network(lets say
>> 10.1.254.0/24)
>> and storage on another network(let's say 10.1.253.0/24) and then all
>> the
>> VM's given public IP's(let's say 200.10.10.0/24). I don't understand
>> how to
>> do that, or even what the difference is between a Guest network and
>> Public
>> network(do they have to be separate?)
>>
>> I'm used to just building VM's in vSphere and the reason I would like
>> to
>> move to CloudStack is for the automation and ability to give not so
>> technical people access to creating VM's. On vSphere this would be
>> easy,
>> iSCSI and Management on the same 10G NIC with different VLAN tags, and
>> then
>> guest network on another NIC. Replicating this into Cloudstack with
>> KVM
>> doesn't seem possible? Can I use VLAN tagging?
>>
>> Other questions I have are around the multitude of DNS
>> servers(internal,
>> external, etc) that the CloudStack Management server asks me for when
>> I set
>> up the Pod/Cluster/Host as well as internal and external networks -
>> then how
>> do I assign and make sure all configuration is okay across
>> hypervisors?
>>
>> If someone could point me towards a good guide I would really
>> appreciate
>> it.
>>
>> Mark
>
>
> --
>
>
>
> --
> Ron Wheeler
> President
> Artifact Software Inc
> email: rwhee...@artifact-software.com
> skype: ronaldmwheeler
> phone: 866-970-2435, ext 102
>


Re: Password requested when starting CS MS

2013-08-12 Thread Daan Hoogland
does this only happen on a new installation/clean db? Sounds like
admin account not enabled yet.

Daan

On Tue, Aug 6, 2013 at 7:21 PM, Mike Tutkowski
 wrote:
> -- Forwarded message --
> From: Mike Tutkowski 
> Date: Tue, Aug 6, 2013 at 11:19 AM
> Subject: Password requested when starting CS MS
> To: "dev@cloudstack.apache.org" 
>
>
> Hi,
>
> This happens to me every now and then, but - with the most recent 4.2 code
> - I seem to be seeing it again.
>
> When I start up the CS MS, I get a keystore exception and have to hit
>  several times on a Password prompt to get the CS MS to continue
> starting up.
>
> Is this acceptable or am I missing out on some functionality with this
> happening?
>
> Does anyone know why I'm prompted for a password? Even if I enter in one
> (the root password), it still prompts me again for my password. Maybe
> that's not the password it's looking for?
>
> Thanks!
>
> WARN  [cloud.server.ConfigurationServerImpl] (Timer-2:) Would use fail-safe
> keystore to continue.
> java.io.IOException: Fail to generate certificate!: timeout
>  at
> com.cloud.server.ConfigurationServerImpl.generateDefaultKeystore(ConfigurationServerImpl.java:496)
> at
> com.cloud.server.ConfigurationServerImpl.updateSSLKeystore(ConfigurationServerImpl.java:517)
>  at
> com.cloud.server.ConfigurationServerImpl.persistDefaultValues(ConfigurationServerImpl.java:289)
> at
> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
>  at
> com.cloud.server.ConfigurationServerImpl.configure(ConfigurationServerImpl.java:157)
> at
> com.cloud.utils.component.ComponentContext.initComponentsLifeCycle(ComponentContext.java:111)
>  at com.cloud.servlet.CloudStartupServlet$1.run(CloudStartupServlet.java:54)
> at java.util.TimerThread.mainLoop(Timer.java:512)
>  at java.util.TimerThread.run(Timer.java:462)
> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Processing
> updateKeyPairs
> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Keypairs already in
> database, updating local copy
> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Going to update
> systemvm iso with generated keypairs if needed
> Password:
> Password:
> Password:
> Password:
> Password:
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *™*
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *™*


Re: Question about GUI refresh logic

2013-08-12 Thread Daan Hoogland
Mike,

expunging is not a reply to an api call, destroying is. I have been
wondering about this as well but I think I just came up with the
answer (cs gurus:}?)

regards,

On Wed, Aug 7, 2013 at 3:00 AM, Mike Tutkowski
 wrote:
> Hi,
>
> I'm curious when I should expect the GUI to refresh and when I should be
> expected to manually do so.
>
> For example, after I destroy a VM, the GUI refreshes to tell me the VM's in
> the destroyed state.
>
> However, to see when the VM is actually expunged, I have to refresh
> manually by clicking on the Instance tab.
>
> Is that expected behavior?
>
> Thanks!
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *™*


[Responsiveness report] dev 2013w32

2013-08-12 Thread Daan Hoogland
http://markmail.org/message/jrgonqu5lmgscgnl Deleting affinity group
while it's still in use by Gaurav Aradhye
http://markmail.org/message/3brunuewgccrfmfy Upgrading form 4.1.0 to
4.1.1 success; some issues with console proxy by France

regards,


Re: Question about GUI refresh logic

2013-08-12 Thread Ahmad Emneina
I would imagine that pane to refresh every so often, reflecting against the
view. meaning after a vm is expunged, it should disappear from that view.


On Mon, Aug 12, 2013 at 8:21 AM, Daan Hoogland wrote:

> Mike,
>
> expunging is not a reply to an api call, destroying is. I have been
> wondering about this as well but I think I just came up with the
> answer (cs gurus:}?)
>
> regards,
>
> On Wed, Aug 7, 2013 at 3:00 AM, Mike Tutkowski
>  wrote:
> > Hi,
> >
> > I'm curious when I should expect the GUI to refresh and when I should be
> > expected to manually do so.
> >
> > For example, after I destroy a VM, the GUI refreshes to tell me the VM's
> in
> > the destroyed state.
> >
> > However, to see when the VM is actually expunged, I have to refresh
> > manually by clicking on the Instance tab.
> >
> > Is that expected behavior?
> >
> > Thanks!
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the
> > cloud
> > *™*
>


13317

2013-08-12 Thread Daan Hoogland
Can committer review https://reviews.apache.org/r/13317/, please?

regards,
Daan


Re: running BVT locally

2013-08-12 Thread Prasanna Santhanam
Daan,

This is possible. All the tests are under test/integration.

test/integration/smoke -> bvt
test/integration/component -> feature specific tests

I'll assume you have cloudstack setup manually and just want to fire
all the tests against this deployment. This could be your laptop from
where you have your management server running. Just open up the integration
port 8096. All tests run on 8080 but the 8096 is required for setting up
api/secret keys

Marvin [1] will drive the tests and you will need to have the latest
updated marvin libraries before you begin the tests. This ensures that
all the APIs and their corresponding libraries in marvin are synced.
Much like cloudmonkey's sync feature marvin can do a sync as well. The docs
explain this in detail.

A few caveats:
0. You need to define a sample configuration file in json. If you simply want
marvin to drive tests for you then you can use the example in
setup/dev/local.cfg and modify it to your purpose.

1.Tests depend on the network model deployed. If a test runs only
against an 'advanced' zone then it will be tagged such:

For eg: 

@attr(tags = ["advanced"])
def test_01_network_off_without_conserve_mode(self):
"""Test Network offering with Conserve mode off and VR - All services
"""
When you want to run only tests that run against your defined network model
just use the attrib selector as follows:
$ nosetests --with-marvin --marvin-config= -a tags="advanced" 
test/integration/smoke --load

2. Many tests also assume that shared storage is available. This is something
that we are working to fix.

3. Some tests require multiple hosts in the deployment eg: migration. But they
should automatically skip themselves if the environment is unsuitable.

Most of the details of running tests is explained in the documentation. Let me
know if anything is missing and I'll fix that up.
[1] https://cwiki.apache.org/confluence/x/QQzMAQ

On Mon, Aug 12, 2013 at 02:35:07PM +0200, Daan Hoogland wrote:
> H, (Prasanna?)
> 
> Can I run the integration tests at Schuberg Philis' and what do I need
> to do so? If there is document on how to set up these (and smoke
> tests) can I get a link?
> 
> You may know; I want to get some work on vpc gateways on sdn accepted.
> 
> regards,

-- 
Prasanna.,


Powered by BigRock.com



Re: 13317

2013-08-12 Thread Joe Brockmeier
Did you attach a patch to this? I see nothing. 

On Mon, Aug 12, 2013, at 10:43 AM, Daan Hoogland wrote:
> Can committer review https://reviews.apache.org/r/13317/, please?
> 
> regards,
> Daan


Best,

jzb
-- 
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


Re: Password requested when starting CS MS

2013-08-12 Thread SuichII, Christopher
I got around this by running the CS server as root. Would that be an option for 
you, Mike?

On Aug 12, 2013, at 11:19 AM, Daan Hoogland  wrote:

> does this only happen on a new installation/clean db? Sounds like
> admin account not enabled yet.
> 
> Daan
> 
> On Tue, Aug 6, 2013 at 7:21 PM, Mike Tutkowski
>  wrote:
>> -- Forwarded message --
>> From: Mike Tutkowski 
>> Date: Tue, Aug 6, 2013 at 11:19 AM
>> Subject: Password requested when starting CS MS
>> To: "dev@cloudstack.apache.org" 
>> 
>> 
>> Hi,
>> 
>> This happens to me every now and then, but - with the most recent 4.2 code
>> - I seem to be seeing it again.
>> 
>> When I start up the CS MS, I get a keystore exception and have to hit
>>  several times on a Password prompt to get the CS MS to continue
>> starting up.
>> 
>> Is this acceptable or am I missing out on some functionality with this
>> happening?
>> 
>> Does anyone know why I'm prompted for a password? Even if I enter in one
>> (the root password), it still prompts me again for my password. Maybe
>> that's not the password it's looking for?
>> 
>> Thanks!
>> 
>> WARN  [cloud.server.ConfigurationServerImpl] (Timer-2:) Would use fail-safe
>> keystore to continue.
>> java.io.IOException: Fail to generate certificate!: timeout
>> at
>> com.cloud.server.ConfigurationServerImpl.generateDefaultKeystore(ConfigurationServerImpl.java:496)
>> at
>> com.cloud.server.ConfigurationServerImpl.updateSSLKeystore(ConfigurationServerImpl.java:517)
>> at
>> com.cloud.server.ConfigurationServerImpl.persistDefaultValues(ConfigurationServerImpl.java:289)
>> at
>> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
>> at
>> com.cloud.server.ConfigurationServerImpl.configure(ConfigurationServerImpl.java:157)
>> at
>> com.cloud.utils.component.ComponentContext.initComponentsLifeCycle(ComponentContext.java:111)
>> at com.cloud.servlet.CloudStartupServlet$1.run(CloudStartupServlet.java:54)
>> at java.util.TimerThread.mainLoop(Timer.java:512)
>> at java.util.TimerThread.run(Timer.java:462)
>> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Processing
>> updateKeyPairs
>> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Keypairs already in
>> database, updating local copy
>> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Going to update
>> systemvm iso with generated keypairs if needed
>> Password:
>> Password:
>> Password:
>> Password:
>> Password:
>> 
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the
>> cloud
>> *™*
>> 
>> 
>> 
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the
>> cloud
>> *™*



Re: 13317

2013-08-12 Thread Daan Hoogland
yes,

it contains

_upgradeMap.put("4.1.1", new DbUpgrade[] { new
Upgrade410to420(), new Upgrade420to430() });

to add to the DatabaseUpgradeChecker

Daan

On Mon, Aug 12, 2013 at 5:52 PM, Joe Brockmeier  wrote:
> Did you attach a patch to this? I see nothing.
>
> On Mon, Aug 12, 2013, at 10:43 AM, Daan Hoogland wrote:
>> Can committer review https://reviews.apache.org/r/13317/, please?
>>
>> regards,
>> Daan
>
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/


Re: running BVT locally

2013-08-12 Thread Daan Hoogland
A bit of a stupid question, I guess

On Mon, Aug 12, 2013 at 5:43 PM, Prasanna Santhanam  wrote:
...
> I'll assume you have cloudstack setup manually and just want to fire
> all the tests against this deployment. This could be your laptop from
> where you have your management server running. Just open up the integration
> port 8096. All tests run on 8080 but the 8096 is required for setting up
> api/secret keys
What does just open up 8096 mean, given I am running in jetty in
eclipse? Should I edit some cfg or add params to the run?

...
> Most of the details of running tests is explained in the documentation. Let me
> know if anything is missing and I'll fix that up.
without saying

thanks,


Re: running BVT locally

2013-08-12 Thread Prasanna Santhanam
On Mon, Aug 12, 2013 at 06:20:58PM +0200, Daan Hoogland wrote:
> A bit of a stupid question, I guess
> 
> On Mon, Aug 12, 2013 at 5:43 PM, Prasanna Santhanam  wrote:
> ...
> > I'll assume you have cloudstack setup manually and just want to fire
> > all the tests against this deployment. This could be your laptop from
> > where you have your management server running. Just open up the integration
> > port 8096. All tests run on 8080 but the 8096 is required for setting up
> > api/secret keys
> What does just open up 8096 mean, given I am running in jetty in
> eclipse? Should I edit some cfg or add params to the run?
> 
If it's the dev environment, then you're good to go. port 8096 is the
port where system user interacts. It is closed by default for binary
installs but open on the dev environment.

> ...
> > Most of the details of running tests is explained in the documentation. Let 
> > me
> > know if anything is missing and I'll fix that up.
> without saying
> 
> thanks,

-- 
Prasanna.,


Powered by BigRock.com



Re: [ACS42] Listtemplates API respone without any Zone configuration

2013-08-12 Thread Min Chen
This seems introduced by storage refactoring where we have gotten rid of
previous direct sql to use db view to search for templates. Please file a
bug for this.

Thanks
-min

On 8/12/13 1:16 AM, "Sailaja Mada"  wrote:

>Hi,
>
>I have the setup with latest code.  When I tried to view the list if
>templates without configuring zone , it is listing all the system vm
>templates and default cent OS templates .
>
>I have not observed this behavior before. Is it introduced intentionally
>or is It a issue with API response ?
>
>Please confirm.
>
>
>2013-08-12 13:18:53,916 INFO  [cloud.api.ApiServer]
>(catalina-exec-3:null) (userId=2 accountId=2
>sessionId=2C596448B0E7D3291B7FA76334A085A7) 10.144.7.12 -- GET
>command=listTemplates&response=json&sessionkey=wMUCQaSUAhWg4JmNNXrmhPGIOzc
>%3D&listAll=true&page=1&pagesize=20&templatefilter=all&_=1376293976571
>200 { "listtemplatesresponse" : { "count":8 ,"template" : [
>{"id":"4b58341c-031a-11e3-b7b2-5641688b41e2","name":"SystemVM Template
>(LXC)","displaytext":"SystemVM Template
>(LXC)","ispublic":false,"isready":false,"passwordenabled":false,"format":"
>QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3
>-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"LXC","d
>omain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextract
>able":false,"checksum":"2755de1f9ef2ce4d6f2bee2efbb4da92","tags":[],"sshke
>yenabled":false,"isdynamicallyscalable":false},
>{"id":"ec8e64b6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template
>(XenServer)","displaytext":"SystemVM Template
>(XenServer)","ispublic":false,"isready":false,"passwordenabled":false,"for
>mat":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ed81a824-0318-
>11e3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux
>7(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"XenSer
>ver","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","is
>extractable":false,"checksum":"74b92f031cc5c2089ee89efb81344dcf","tags":[]
>,"sshkeyenabled":false,"isdynamicallyscalable":false},
>{"id":"ec98be84-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template
>(HyperV)","displaytext":"SystemVM Template
>(HyperV)","ispublic":false,"isready":false,"passwordenabled":false,"format
>":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e
>3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"Hyperv"
>,"domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextr
>actable":false,"checksum":"fb1b6e032a160d86f2c28feb5add6d83","tags":[],"ss
>hkeyenabled":false,"isdynamicallyscalable":false},
>{"id":"ec931e0c-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template
>(KVM)","displaytext":"SystemVM Template
>(KVM)","ispublic":false,"isready":false,"passwordenabled":false,"format":"
>QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11e3
>-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"KVM","d
>omain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextract
>able":false,"checksum":"6cea42b2633841648040becb588bd8f0","tags":[],"sshke
>yenabled":false,"isdynamicallyscalable":false},
>{"id":"ec95eaf6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template
>(vSphere)","displaytext":"SystemVM Template
>(vSphere)","ispublic":false,"isready":false,"passwordenabled":false,"forma
>t":"OVA","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-11
>e3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"VMware"
>,"domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextr
>actable":false,"checksum":"8fde62b1089e5844a9cd3b9b953f9596","tags":[],"ss
>hkeyenabled":false,"isdynamicallyscalable":true},
>{"id":"ec947a86-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.6(64-bit)
>no GUI (XenServer)","displaytext":"CentOS 5.6(64-bit) no GUI
>(XenServer)","ispublic":true,"isready":false,"passwordenabled":false,"form
>at":"VHD","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11
>e3-b7b2-5641688b41e2","ostypename":"CentOS 5.3
>(64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"XenSer
>ver","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","is
>extractable":true,"checksum":"905cec879afd9c9d22ecc8036131a180","tags":[],
>"sshkeyenabled":false,"isdynamicallyscalable":true},
>{"id":"ec95326e-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.3(64-bit)
>no GUI (vSphere)","displaytext":"CentOS 5.3(64-bit) no GUI
>(vSphere)","ispublic":true,"isready":false,"passwordenabled":false,"format
>":"OVA","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318-11e3
>-b7b2-5641688b41e2","ostypename":"CentOS 5.3
>(64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"VMware
>","domain":"ROOT","domainid":"ec8ad800

Re: Password requested when starting CS MS

2013-08-12 Thread Mike Tutkowski
Yeah, it seems to only happen when I have a fresh install.

I also notice - in this situation - that I get a password prompt in my
console when trying to upload an ISO for the first time.

Chris - I once wondered about running the CS MS as root, but someone warned
me not to because it might cover up issues that would otherwise be exposed
if I weren't running as root. What do you think about that?

Thanks, guys!


On Mon, Aug 12, 2013 at 9:54 AM, SuichII, Christopher <
chris.su...@netapp.com> wrote:

> I got around this by running the CS server as root. Would that be an
> option for you, Mike?
>
> On Aug 12, 2013, at 11:19 AM, Daan Hoogland 
> wrote:
>
> > does this only happen on a new installation/clean db? Sounds like
> > admin account not enabled yet.
> >
> > Daan
> >
> > On Tue, Aug 6, 2013 at 7:21 PM, Mike Tutkowski
> >  wrote:
> >> -- Forwarded message --
> >> From: Mike Tutkowski 
> >> Date: Tue, Aug 6, 2013 at 11:19 AM
> >> Subject: Password requested when starting CS MS
> >> To: "dev@cloudstack.apache.org" 
> >>
> >>
> >> Hi,
> >>
> >> This happens to me every now and then, but - with the most recent 4.2
> code
> >> - I seem to be seeing it again.
> >>
> >> When I start up the CS MS, I get a keystore exception and have to hit
> >>  several times on a Password prompt to get the CS MS to continue
> >> starting up.
> >>
> >> Is this acceptable or am I missing out on some functionality with this
> >> happening?
> >>
> >> Does anyone know why I'm prompted for a password? Even if I enter in one
> >> (the root password), it still prompts me again for my password. Maybe
> >> that's not the password it's looking for?
> >>
> >> Thanks!
> >>
> >> WARN  [cloud.server.ConfigurationServerImpl] (Timer-2:) Would use
> fail-safe
> >> keystore to continue.
> >> java.io.IOException: Fail to generate certificate!: timeout
> >> at
> >>
> com.cloud.server.ConfigurationServerImpl.generateDefaultKeystore(ConfigurationServerImpl.java:496)
> >> at
> >>
> com.cloud.server.ConfigurationServerImpl.updateSSLKeystore(ConfigurationServerImpl.java:517)
> >> at
> >>
> com.cloud.server.ConfigurationServerImpl.persistDefaultValues(ConfigurationServerImpl.java:289)
> >> at
> >>
> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
> >> at
> >>
> com.cloud.server.ConfigurationServerImpl.configure(ConfigurationServerImpl.java:157)
> >> at
> >>
> com.cloud.utils.component.ComponentContext.initComponentsLifeCycle(ComponentContext.java:111)
> >> at
> com.cloud.servlet.CloudStartupServlet$1.run(CloudStartupServlet.java:54)
> >> at java.util.TimerThread.mainLoop(Timer.java:512)
> >> at java.util.TimerThread.run(Timer.java:462)
> >> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Processing
> >> updateKeyPairs
> >> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Keypairs
> already in
> >> database, updating local copy
> >> INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Going to update
> >> systemvm iso with generated keypairs if needed
> >> Password:
> >> Password:
> >> Password:
> >> Password:
> >> Password:
> >>
> >> --
> >> *Mike Tutkowski*
> >> *Senior CloudStack Developer, SolidFire Inc.*
> >> e: mike.tutkow...@solidfire.com
> >> o: 303.746.7302
> >> Advancing the way the world uses the
> >> cloud
> >> *™*
> >>
> >>
> >>
> >> --
> >> *Mike Tutkowski*
> >> *Senior CloudStack Developer, SolidFire Inc.*
> >> e: mike.tutkow...@solidfire.com
> >> o: 303.746.7302
> >> Advancing the way the world uses the
> >> cloud
> >> *™*
>
>


-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: understanding cloudstack networking

2013-08-12 Thread Kelcey Jamison Damage
I'm just going to chime in on this. What we need to put together is a primer. 
Our docs work fairly well and are CloudStack centric, which I see as most 
peoples issue. On the topic of networking, building a cloud requires a very 
strong understanding of core Linux networking, which is technically outside the 
responsibilities of our project. This does not mean however that community 
members can not create primer guides, and many have... There is mostly a 
difficulty in new users finding these resources.

On the topic of 4.2, we all knew some big changes were coming, but it did feel 
like a couple strange housekeeping items snuck in under the radar that affected 
how certain components work. An example of this would be the new network 
interface handling from a user perspective.

I am more then happy to assist people with install and setup issues on the 
networking side of things, as collecting data on what parts of the process are 
most difficult would be quite valuable to us.

Thanks

Kelcey Damage
kdam...@apache.org

- Original Message -
From: "Daan Hoogland" 
To: rwhee...@artifact-software.com, us...@cloudstack.apache.org
Cc: "Travis Graham" , "dev" 
Sent: Monday, August 12, 2013 8:02:22 AM
Subject: Re: understanding cloudstack networking

sounds great Ron,

I'm sure I am not the guy you need for this but I'll keep an eye on
it. The 'someone who actually knows what it s supposed to say' is
bound to be around on this list or dev.

regards,
Daan


On Mon, Aug 12, 2013 at 4:55 PM, Ron Wheeler
 wrote:
> I have been reading and correcting the posted 4.2 documentation changes that
> I understand or where the English errors are very clear.
>
> I have filed JIRAs and some have been fixed.
>
> I would be willing to participate in a workshop to walk through the
> installation with someone who actually knows what it s supposed to say.
>
> Ron
>
>
>
> On 12/08/2013 10:41 AM, Daan Hoogland wrote:
>>
>> you are right Ron, but even those companies/people can only spend
>> their time once. So please submit you improvements whenever you can.
>>
>> regards,
>> Daan
>>
>> On Mon, Aug 12, 2013 at 4:32 PM, Ron Wheeler
>>  wrote:
>>>
>>> On 12/08/2013 10:08 AM, Travis Graham wrote:

 One of the most confusing things I've ran into, past the fact the
 documentation is wrong about 80% of the time, is the mix of CentOS and
 Ubuntu instructions.

 I think splitting things out into their own OS specific install guides
 would reduce a lot of confusion.
>>>
>>> Yes.
>>>
 I was browsing the 4.2 docs in the repo this weekend and I'm still not
 seeing swath of the incorrect info being updated. Maybe things that
 haven't
 been rolled into the 4.2 branch yet.
>>>
>>> I hope that this gets done. It is the biggest problem that CloudStack has
>>> in
>>> getting traction.
>>>
>>> You only get one chance to make a first impression and the impression at
>>> the
>>> moment is that the system does not work and is not ready for prime time
>>> except for organizations that have a development group ready to read the
>>> code and fix the docs for their installation.
>>>
>>> Ron
>>>
>>>
 Travis

 On Aug 12, 2013, at 9:59 AM, Ron Wheeler
 
 wrote:

> The documentation is wrong which is a big problem.
>
> It is also confusing with extraneous stuff stuck in the middle and
> missing introductory information to explain where the instructions are
> leading.
>
> There seems to be a big effort to get 4.2 out with accurate docs and I
> hope more clarifying text and drawings.
>
> It appears that there is a lot of effort going into external Wiki
> documentation to make up for the state of the manuals.
>
> Ron
>
>
> On 12/08/2013 4:10 AM, Mark van der Meulen wrote:
>>
>> Hi,
>>
>> I am having a little trouble understanding how the cloudstack
>> networking
>> model works, I have read the documentation and enquired on IRC(without
>> response) and still don't really get it. I suspect if I was able to
>> setup
>> CloudStack and play with it I would understand, however given that I
>> have to
>> go through a complex networking setup to get the Zone/Pod/Cluster/Host
>> even
>> setup to start with, I haven't been able to get far enough in to start
>> playing.
>>
>> Based on what I have read, I think I would like to setup a Public
>> Cloud,
>> essentially some hypervisors on a private network(lets say
>> 10.1.254.0/24)
>> and storage on another network(let's say 10.1.253.0/24) and then all
>> the
>> VM's given public IP's(let's say 200.10.10.0/24). I don't understand
>> how to
>> do that, or even what the difference is between a Guest network and
>> Public
>> network(do they have to be separate?)
>>
>> I'm used to just building VM's in vSphere and the reason I would like
>> to
>> move to CloudStack is for the aut

Re: running BVT locally

2013-08-12 Thread Daan Hoogland
Prasanna,

I checked out the marvin and tst setuyp pages.

I have marvin

$ python
Python 2.7.3 (default, Dec 18 2012, 13:50:09)
[GCC 4.5.3] on cygwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import marvin;
>>> marvin

>>>

but still

$ nosetests --with-marvin --marvin-config=setup/dev/local.cfg -a
tags="advanced" test/integration/smoke --load
Usage: nosetests [options]

nosetests: error: no such option: --with-marvin

which seems correct because

dhoogland@SBPLT101 /cygdrive/c/Users/dhoogland/cloudstack/cloudstack
$ nosetests -p
Plugin capture
Plugin failuredetail
Plugin xunit
Plugin deprecated
Plugin skip
Plugin multiprocess
Plugin logcapture
Plugin coverage
Plugin attributeselector
Plugin doctest
Plugin profile
Plugin id
Plugin allmodules
Plugin collect-only
Plugin isolation
Plugin pdb

So I try to build marvin but then I get

[INFO] BUILD FAILURE
[INFO] 
[INFO] Total time: 1.938s
[INFO] Finished at: Mon Aug 12 18:38:35 CEST 2013
[INFO] Final Memory: 19M/181M
[INFO] 
[ERROR] Failed to execute goal
org.codehaus.mojo:exec-maven-plugin:1.2.1:exec (generate-sources) on
project cloud-marvin: Command execution failed. Cannot run program
"python" (in directory
"C:\Users\dhoogland\cloudstack\cloudstack\tools\marvin\marvin"):
CreateProcess error=2, The system cannot find the file specified ->
[Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with
the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions,
please read the following articles:
[ERROR] [Help 1]
http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

even though

$ python
Python 2.7.3 (default, Dec 18 2012, 13:50:09)
[GCC 4.5.3] on cygwin
Type "help", "copyright", "credits" or "license" for more information.
>>>

Is this a windows thing? or is there some specific path coded
somewhere for python?

regards,

On Mon, Aug 12, 2013 at 6:25 PM, Prasanna Santhanam  wrote:
> On Mon, Aug 12, 2013 at 06:20:58PM +0200, Daan Hoogland wrote:
>> A bit of a stupid question, I guess
>>
>> On Mon, Aug 12, 2013 at 5:43 PM, Prasanna Santhanam  wrote:
>> ...
>> > I'll assume you have cloudstack setup manually and just want to fire
>> > all the tests against this deployment. This could be your laptop from
>> > where you have your management server running. Just open up the integration
>> > port 8096. All tests run on 8080 but the 8096 is required for setting up
>> > api/secret keys
>> What does just open up 8096 mean, given I am running in jetty in
>> eclipse? Should I edit some cfg or add params to the run?
>>
> If it's the dev environment, then you're good to go. port 8096 is the
> port where system user interacts. It is closed by default for binary
> installs but open on the dev environment.
>
>> ...
>> > Most of the details of running tests is explained in the documentation. 
>> > Let me
>> > know if anything is missing and I'll fix that up.
>> without saying
>>
>> thanks,
>
> --
> Prasanna.,
>
> 
> Powered by BigRock.com
>


Re: understanding cloudstack networking

2013-08-12 Thread Kelcey Jamison Damage
One last note. As far as stability is concerned... 4.1.x is the recommended 
build, and many of us(myself included) are running our production on 4.0.2 
which is rock solid for all standard features.

Thanks

- Original Message -
From: "Kelcey Jamison Damage" 
To: dev@cloudstack.apache.org
Sent: Monday, August 12, 2013 9:49:42 AM
Subject: Re: understanding cloudstack networking

I'm just going to chime in on this. What we need to put together is a primer. 
Our docs work fairly well and are CloudStack centric, which I see as most 
peoples issue. On the topic of networking, building a cloud requires a very 
strong understanding of core Linux networking, which is technically outside the 
responsibilities of our project. This does not mean however that community 
members can not create primer guides, and many have... There is mostly a 
difficulty in new users finding these resources.

On the topic of 4.2, we all knew some big changes were coming, but it did feel 
like a couple strange housekeeping items snuck in under the radar that affected 
how certain components work. An example of this would be the new network 
interface handling from a user perspective.

I am more then happy to assist people with install and setup issues on the 
networking side of things, as collecting data on what parts of the process are 
most difficult would be quite valuable to us.

Thanks

Kelcey Damage
kdam...@apache.org

- Original Message -
From: "Daan Hoogland" 
To: rwhee...@artifact-software.com, us...@cloudstack.apache.org
Cc: "Travis Graham" , "dev" 
Sent: Monday, August 12, 2013 8:02:22 AM
Subject: Re: understanding cloudstack networking

sounds great Ron,

I'm sure I am not the guy you need for this but I'll keep an eye on
it. The 'someone who actually knows what it s supposed to say' is
bound to be around on this list or dev.

regards,
Daan


On Mon, Aug 12, 2013 at 4:55 PM, Ron Wheeler
 wrote:
> I have been reading and correcting the posted 4.2 documentation changes that
> I understand or where the English errors are very clear.
>
> I have filed JIRAs and some have been fixed.
>
> I would be willing to participate in a workshop to walk through the
> installation with someone who actually knows what it s supposed to say.
>
> Ron
>
>
>
> On 12/08/2013 10:41 AM, Daan Hoogland wrote:
>>
>> you are right Ron, but even those companies/people can only spend
>> their time once. So please submit you improvements whenever you can.
>>
>> regards,
>> Daan
>>
>> On Mon, Aug 12, 2013 at 4:32 PM, Ron Wheeler
>>  wrote:
>>>
>>> On 12/08/2013 10:08 AM, Travis Graham wrote:

 One of the most confusing things I've ran into, past the fact the
 documentation is wrong about 80% of the time, is the mix of CentOS and
 Ubuntu instructions.

 I think splitting things out into their own OS specific install guides
 would reduce a lot of confusion.
>>>
>>> Yes.
>>>
 I was browsing the 4.2 docs in the repo this weekend and I'm still not
 seeing swath of the incorrect info being updated. Maybe things that
 haven't
 been rolled into the 4.2 branch yet.
>>>
>>> I hope that this gets done. It is the biggest problem that CloudStack has
>>> in
>>> getting traction.
>>>
>>> You only get one chance to make a first impression and the impression at
>>> the
>>> moment is that the system does not work and is not ready for prime time
>>> except for organizations that have a development group ready to read the
>>> code and fix the docs for their installation.
>>>
>>> Ron
>>>
>>>
 Travis

 On Aug 12, 2013, at 9:59 AM, Ron Wheeler
 
 wrote:

> The documentation is wrong which is a big problem.
>
> It is also confusing with extraneous stuff stuck in the middle and
> missing introductory information to explain where the instructions are
> leading.
>
> There seems to be a big effort to get 4.2 out with accurate docs and I
> hope more clarifying text and drawings.
>
> It appears that there is a lot of effort going into external Wiki
> documentation to make up for the state of the manuals.
>
> Ron
>
>
> On 12/08/2013 4:10 AM, Mark van der Meulen wrote:
>>
>> Hi,
>>
>> I am having a little trouble understanding how the cloudstack
>> networking
>> model works, I have read the documentation and enquired on IRC(without
>> response) and still don't really get it. I suspect if I was able to
>> setup
>> CloudStack and play with it I would understand, however given that I
>> have to
>> go through a complex networking setup to get the Zone/Pod/Cluster/Host
>> even
>> setup to start with, I haven't been able to get far enough in to start
>> playing.
>>
>> Based on what I have read, I think I would like to setup a Public
>> Cloud,
>> essentially some hypervisors on a private network(lets say
>> 10.1.254.0/24)
>> and storage on another network(let's say 10.1.25

Review Request 13499: [GSoC] Add staticNat support to GRE controller

2013-08-12 Thread tuna

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

Review request for cloudstack, Sebastien Goasguen and Hugo Trippaers.


Repository: cloudstack-git


Description
---

I make a patch for supporting staticNat service to gre controller. The scenario 
is:
+ use GRE isolation method
+ create a network offering with staticNat,Connectivity (Ovs) and 
sourceNat,...(VirtualRouter)
+ create a guest network using the above network offering
+ deploy VM, enable staticNat


Diffs
-

  
plugins/network-elements/ovs/src/com/cloud/api/response/OvsDeviceResponse.java 
c0901b2 
  
plugins/network-elements/ovs/src/com/cloud/network/commands/AddOvsDeviceCmd.java
 1abc324 
  
plugins/network-elements/ovs/src/com/cloud/network/commands/DeleteOvsDeviceCmd.java
 87eedfb 
  
plugins/network-elements/ovs/src/com/cloud/network/commands/ListOvsDevicesCmd.java
 2adb33a 
  plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java 
3824669 
  plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsApi.java b533312 
  plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsApiException.java 
20603e0 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsCreateGreTunnelAnswer.java
 5f0f8c1 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsCreateGreTunnelCommand.java
 e2cd2d8 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsCreateTunnelAnswer.java
 fc2eb8a 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsCreateTunnelCommand.java
 1ececa0 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsDeleteFlowCommand.java
 2a6d5d7 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsDestroyBridgeCommand.java
 8be5586 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsDestroyTunnelCommand.java
 4594d99 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsFetchInterfaceAnswer.java
 1ee6606 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsFetchInterfaceCommand.java
 c27daf0 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsSetTagAndFlowAnswer.java
 ba16839 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsSetTagAndFlowCommand.java
 17121a0 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsSetupBridgeCommand.java
 29cce15 
  plugins/network-elements/ovs/src/com/cloud/network/ovs/StartupOvsCommand.java 
b85331e 
  plugins/network-elements/ovs/src/com/cloud/network/ovs/dao/OvsDeviceDao.java 
794e45e 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/dao/OvsDeviceDaoImpl.java
 11a4d48 
  plugins/network-elements/ovs/src/com/cloud/network/ovs/dao/OvsDeviceVO.java 
cab63f6 
  plugins/network-elements/ovs/src/com/cloud/network/resource/OvsResource.java 
a94e4f8 
  server/src/com/cloud/network/element/VirtualRouterElement.java ecf6473 

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


Testing
---

Test done with above scenario. 
Deploy VMs.
enable/disable staticNat.


Thanks,

tuna



Re: Password requested when starting CS MS

2013-08-12 Thread Pedro Roque Marques
Mike,
If you create a cloudstack-management package and run 
"cloudstack-setup-management" before  starting the management server you can 
run the MS as user cloud without experiencing these issues...
It sets up the correct privileges.

  Pedro.

On Aug 12, 2013, at 9:47 AM, Mike Tutkowski wrote:

> Yeah, it seems to only happen when I have a fresh install.
> 
> I also notice - in this situation - that I get a password prompt in my
> console when trying to upload an ISO for the first time.
> 
> Chris - I once wondered about running the CS MS as root, but someone warned
> me not to because it might cover up issues that would otherwise be exposed
> if I weren't running as root. What do you think about that?
> 
> Thanks, guys!
> 
> 
> On Mon, Aug 12, 2013 at 9:54 AM, SuichII, Christopher <
> chris.su...@netapp.com> wrote:
> 
>> I got around this by running the CS server as root. Would that be an
>> option for you, Mike?
>> 
>> On Aug 12, 2013, at 11:19 AM, Daan Hoogland 
>> wrote:
>> 
>>> does this only happen on a new installation/clean db? Sounds like
>>> admin account not enabled yet.
>>> 
>>> Daan
>>> 
>>> On Tue, Aug 6, 2013 at 7:21 PM, Mike Tutkowski
>>>  wrote:
 -- Forwarded message --
 From: Mike Tutkowski 
 Date: Tue, Aug 6, 2013 at 11:19 AM
 Subject: Password requested when starting CS MS
 To: "dev@cloudstack.apache.org" 
 
 
 Hi,
 
 This happens to me every now and then, but - with the most recent 4.2
>> code
 - I seem to be seeing it again.
 
 When I start up the CS MS, I get a keystore exception and have to hit
  several times on a Password prompt to get the CS MS to continue
 starting up.
 
 Is this acceptable or am I missing out on some functionality with this
 happening?
 
 Does anyone know why I'm prompted for a password? Even if I enter in one
 (the root password), it still prompts me again for my password. Maybe
 that's not the password it's looking for?
 
 Thanks!
 
 WARN  [cloud.server.ConfigurationServerImpl] (Timer-2:) Would use
>> fail-safe
 keystore to continue.
 java.io.IOException: Fail to generate certificate!: timeout
 at
 
>> com.cloud.server.ConfigurationServerImpl.generateDefaultKeystore(ConfigurationServerImpl.java:496)
 at
 
>> com.cloud.server.ConfigurationServerImpl.updateSSLKeystore(ConfigurationServerImpl.java:517)
 at
 
>> com.cloud.server.ConfigurationServerImpl.persistDefaultValues(ConfigurationServerImpl.java:289)
 at
 
>> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
 at
 
>> com.cloud.server.ConfigurationServerImpl.configure(ConfigurationServerImpl.java:157)
 at
 
>> com.cloud.utils.component.ComponentContext.initComponentsLifeCycle(ComponentContext.java:111)
 at
>> com.cloud.servlet.CloudStartupServlet$1.run(CloudStartupServlet.java:54)
 at java.util.TimerThread.mainLoop(Timer.java:512)
 at java.util.TimerThread.run(Timer.java:462)
 INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Processing
 updateKeyPairs
 INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Keypairs
>> already in
 database, updating local copy
 INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Going to update
 systemvm iso with generated keypairs if needed
 Password:
 Password:
 Password:
 Password:
 Password:
 
 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloud
 *™*
 
 
 
 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloud
 *™*
>> 
>> 
> 
> 
> -- 
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *™*



Re: Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-08-12 Thread Chiradeep Vittal


> On Aug. 12, 2013, 11:38 a.m., Hugo Trippaers wrote:
> > I'm also OK with this patch. Documentation is pending, but i'm fine with 
> > committing this to master. It needs a rebase to the latest state of master, 
> > but after that we should commit this.  OK Chiradeep?

OK.


- Chiradeep


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


On July 23, 2013, 6:45 a.m., Toshiaki Hatano wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12623/
> ---
> 
> (Updated July 23, 2013, 6:45 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Chiradeep Vittal, Murali 
> Reddy, Hugo Trippaers, and Sheng Yang.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-2328
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor
> 
> Initial patch for VXLAN support.
> Fully functional, hopefully, for GuestNetwork - AdvancedZone.
> 
> Patch Note:
>  in cloudstack-server
> - Add isolation method VXLAN
> - Add VxlanGuestNetworkGuru as plugin for VXLAN isolation
> - Modify NetworkServiceImpl to handle extended vNet range for VXLAN isolation
> - Add VXLAN isolation option in zoneWizard UI
> 
>  in cloudstack-agent (kvm)
> - Add modifyvxlan.sh script that handle bridge/vxlan interface manipulation 
> script
> -- Usage is exactly same to modifyvlan.sh
> - BridgeVifDriver will call modifyvxlan.sh instead of modifyvlan.sh when 
> VXLAN is used for isolation
> 
> Database changes:
> - No change in database structure.
> - VXLAN isolation uses same tables that VLAN uses to store vNet allocation 
> status.
> 
> Known Issue:
> - Some resource still says 'VLAN' in log even if VXLAN is used
> - in UI, "Network - GuestNetworks" dosen't display VNI
> -- VLAN ID field displays "N/A"
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Networks.java c76c3d4 
>   api/src/com/cloud/network/PhysicalNetwork.java f6cb1a6 
>   client/pom.xml 32ab94a 
>   client/tomcatconf/componentContext.xml.in 1fbec61 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java
>  195cf40 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
>  571bcc8 
>   plugins/network-elements/vxlan/pom.xml PRE-CREATION 
>   
> plugins/network-elements/vxlan/src/com/cloud/network/guru/VxlanGuestNetworkGuru.java
>  PRE-CREATION 
>   
> plugins/network-elements/vxlan/test/com/cloud/network/guru/VxlanGuestNetworkGuruTest.java
>  PRE-CREATION 
>   plugins/pom.xml ff1e9c9 
>   scripts/vm/network/vnet/modifyvlan.sh 8ed3905 
>   scripts/vm/network/vnet/modifyvxlan.sh PRE-CREATION 
>   server/src/com/cloud/network/NetworkManagerImpl.java 30c4242 
>   server/src/com/cloud/network/NetworkServiceImpl.java 23bed01 
>   ui/scripts/ui-custom/zoneWizard.js bba6c04 
> 
> Diff: https://reviews.apache.org/r/12623/diff/
> 
> 
> Testing
> ---
> 
> #) Test set up
> - Components
>   - 1x management server
>   - 1x nfs storage
>   - 3x Linux KVM host
>   -- CentOS 6.4 based
>   -- Replace kernel by version 3.8.13, VXLAN kernel module built as loadable 
> module
>   -- Replace iproute2 by version iproute2-ss130430
>   -- BridgeVifDriver (Default)
> 1. create advanced zone from zone wizard without security group option
> 2. select hypervisor: KVM
> 3. assign Guest network to separated physical network, isolated by VXLAN
>specify bridge name (traffic label) for Guest network, this bridge should 
> have IPv4 address (global/private both are OK).
> 4. assign Guest vNet range 1-2
> 5. other parameter are normal
> 6. add 2 more hosts into same zone/pod/cluster after zone wizard is finished
> 
> #) Test case 1: start/stop VR
> 1. Create network offering, same configuration as 
> DefaultIsolatedNetworkOfferingWithSourceNatService but persistent
> 2. Create network with network offering which is created in step 0
> 3. Confirm VR is started and bridge/vxlan device created on host
> 4. Delete network which is created in step 1
> 5. Confirm VR is deleted and bridge/vxlan device deleted on host
> 
> #) Test case 2: start/stop an instance (VR is on same host)
> 1. Add an instance from UI, create network during wizard.
> 2. Confirm VM and VR are on the same host
> 3. Confirm it's pingable from VM to VR
> 4. Confirm it's pingable from VM to public network (after opening Egress rule)
> 5. Destroy instance
> 6. Confirm bridge/vxlan device is still on the host
> 7. Delete network after the VM is expunged
> 8. Confirm VR are deleted and bridge/vxlan device deleted on the host
> 
> #) Test case 3: start/stop an instance (VR is on different host)
> 1. Add an instance from UI, create network 

[RESULTS] (Was: Re: [VOTE] Update by-laws: non-technical decisions and other minor changes)

2013-08-12 Thread Noah Slater
3 +1 votes, and the vote passes. I will update the by-laws now.

Thanks!


On 6 August 2013 20:06, Musayev, Ilya  wrote:

> +1
>
> > -Original Message-
> > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
> > Sent: Tuesday, August 06, 2013 1:58 PM
> > To: dev@cloudstack.apache.org
> > Subject: Re: [VOTE] Update by-laws: non-technical decisions and other
> minor
> > changes
> >
> > +1
> >
> > On 8/5/13 2:43 PM, "Noah Slater"  wrote:
> >
> > >Hi dev,
> > >
> > >I have some more by-law changes to propose. This is essentially round 2
> > >for these changes. I incorporated feedback from the last thread.
> > >
> > >Per the by-laws, we're using a lazy majority for this vote. Please cast
> > >your vote now. I will tally the results in 72 hours.
> > >
> > >Here's my changelog:
> > >
> > >* Removed some spurious   entities
> > >
> > >* Added "A technical decision is any decision that involves changes to
> > >the source code that we distribute in our official releases." to §
> > >3.4.1 (Technical Decisions).
> > >
> > >* Added "discussion-lead" before "consensus gathering" in this section.
> > >
> > >* With the improved definition, I have tightened up the wording so that
> > >technical decisions must be made on @dev.
> > >
> > >* Added § 3.4.2, Non-Technical Decisions. Non-technical decisions are
> > >defined as in the inverse of technical decisions. They can be made on
> > >whatever list is appropriate. Formal voting will use a lazy 2/3
> majority.
> > >Votes cannot be vetoed.
> > >
> > >* Changed § 3.4.3. (Release Plan) to limit decisions to @dev.
> > >
> > >* Changed § 3.4.4. (Product Release) to limit decisions to @dev.
> > >
> > >* Changed § 3.4.5. (Adoption of New Codebase) to limit decisions to
> @dev.
> > >
> > >* Changed § 3.4.10. (Modifying Bylaws) to limit decisions to @dev.
> > >
> > >* Added an Oxford comma to the final paras of § 3.4.1. and § 3.4.2.
> > >
> > >* Section renumbering to accommodate § 3.4.2.
> > >
> > >And here's the patch:
> > >
> > >Index: bylaws.mdtext
> > >=
> > ==
> > >--- bylaws.mdtext (revision 1510739)
> > >+++ bylaws.mdtext (working copy)
> > >@@ -198,41 +198,64 @@
> > >
> > > 3.4.1. Technical Decisions
> > >
> > >+A technical decision is any decision that involves changes to the
> > >+source
> > >code
> > >+that we distribute in our official releases.
> > >+
> > > Technical decisions should normally be made by the entire community
> > >using -consensus gathering, and not through formal voting.
> > >+discussion-lead consensus gathering, and not through formal voting.
> > >
> > >-Technical decisions must be made on a project development mailing list.
> > >+Technical decisions must be made on the project development mailing
> list.
> > >
> > > During the consensus gathering process, technical decisions may be
> > >vetoed by  any Committer with a valid reason.
> > >
> > > If a formal vote is started for a technical decision, the vote will be
> > >held as -a lazy consensus of active committers.
> > >+a lazy consensus of active committers.
> > >
> > >-Any user, contributor, committer or PMC member can initiate a
> > >technical
> > >+Any user, contributor, committer, or PMC member can initiate a
> > >+technical
> > > decision making process.
> > >
> > >-3.4.2. Release Plan
> > >+3.4.2. Non-Technical Decisions
> > >
> > >+A non-technical decisions is any decision that does not involve
> > >+changes
> > >to
> > >the
> > >+source code that we distribute in our official releases.
> > >+
> > >+Non-technical decisions should normally be made by the entire
> > >+community
> > >using
> > >+discussion-lead consensus-building, and not through formal voting.
> > >+
> > >+Non-technical decisions can be made on whichever project mailing list
> > >+is
> > >most
> > >+appropriate.
> > >+
> > >+Non-technical decisions cannot be vetoed, but if there is strong
> > >opposition
> > >+a formal vote can be used to resolve the dispute.
> > >+
> > >+If a formal vote is started for a non-technical decision, the vote
> > >+will
> > >be
> > >held
> > >+as a lazy 2/3 majority of active committers.
> > >+
> > >+Any user, contributor, committer, or PMC member can initiate a
> > >non-technical
> > >+decision making process.
> > >+
> > >+3.4.3. Release Plan
> > >+
> > > Defines the timetable and work items for a release. The plan also
> > >nominates a  Release Manager.
> > >
> > > A lazy majority of active committers is required for approval.
> > >
> > >-Any active committer or PMC member may call a vote. The vote must
> > >occur on a
> > >+Any active committer or PMC member may call a vote. The vote must
> > >+occur
> > >on
> > >the
> > > project development mailing list.
> > >
> > >-3.4.3. Product Release
> > >+3.4.4. Product Release
> > >
> > > When a release of one of the project's products is ready, a vote is
> > >required to  accept the release as an official release of the project.
> > >
> > > Lazy Majority of active PMC members 

Re: [RESULTS] (Was: Re: [VOTE] Update by-laws: non-technical decisions and other minor changes)

2013-08-12 Thread Noah Slater
Ignore this. Unfortunately, we only have 2 +1 votes from the PMC. I am
going to try to get a third. Sorry for the confusion.


On 12 August 2013 18:55, Noah Slater  wrote:

> 3 +1 votes, and the vote passes. I will update the by-laws now.
>
> Thanks!
>
>
> On 6 August 2013 20:06, Musayev, Ilya  wrote:
>
>> +1
>>
>> > -Original Message-
>> > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
>> > Sent: Tuesday, August 06, 2013 1:58 PM
>> > To: dev@cloudstack.apache.org
>> > Subject: Re: [VOTE] Update by-laws: non-technical decisions and other
>> minor
>> > changes
>> >
>> > +1
>> >
>> > On 8/5/13 2:43 PM, "Noah Slater"  wrote:
>> >
>> > >Hi dev,
>> > >
>> > >I have some more by-law changes to propose. This is essentially round 2
>> > >for these changes. I incorporated feedback from the last thread.
>> > >
>> > >Per the by-laws, we're using a lazy majority for this vote. Please cast
>> > >your vote now. I will tally the results in 72 hours.
>> > >
>> > >Here's my changelog:
>> > >
>> > >* Removed some spurious   entities
>> > >
>> > >* Added "A technical decision is any decision that involves changes to
>> > >the source code that we distribute in our official releases." to §
>> > >3.4.1 (Technical Decisions).
>> > >
>> > >* Added "discussion-lead" before "consensus gathering" in this section.
>> > >
>> > >* With the improved definition, I have tightened up the wording so that
>> > >technical decisions must be made on @dev.
>> > >
>> > >* Added § 3.4.2, Non-Technical Decisions. Non-technical decisions are
>> > >defined as in the inverse of technical decisions. They can be made on
>> > >whatever list is appropriate. Formal voting will use a lazy 2/3
>> majority.
>> > >Votes cannot be vetoed.
>> > >
>> > >* Changed § 3.4.3. (Release Plan) to limit decisions to @dev.
>> > >
>> > >* Changed § 3.4.4. (Product Release) to limit decisions to @dev.
>> > >
>> > >* Changed § 3.4.5. (Adoption of New Codebase) to limit decisions to
>> @dev.
>> > >
>> > >* Changed § 3.4.10. (Modifying Bylaws) to limit decisions to @dev.
>> > >
>> > >* Added an Oxford comma to the final paras of § 3.4.1. and § 3.4.2.
>> > >
>> > >* Section renumbering to accommodate § 3.4.2.
>> > >
>> > >And here's the patch:
>> > >
>> > >Index: bylaws.mdtext
>> > >=
>> > ==
>> > >--- bylaws.mdtext (revision 1510739)
>> > >+++ bylaws.mdtext (working copy)
>> > >@@ -198,41 +198,64 @@
>> > >
>> > > 3.4.1. Technical Decisions
>> > >
>> > >+A technical decision is any decision that involves changes to the
>> > >+source
>> > >code
>> > >+that we distribute in our official releases.
>> > >+
>> > > Technical decisions should normally be made by the entire community
>> > >using -consensus gathering, and not through formal voting.
>> > >+discussion-lead consensus gathering, and not through formal voting.
>> > >
>> > >-Technical decisions must be made on a project development mailing
>> list.
>> > >+Technical decisions must be made on the project development mailing
>> list.
>> > >
>> > > During the consensus gathering process, technical decisions may be
>> > >vetoed by  any Committer with a valid reason.
>> > >
>> > > If a formal vote is started for a technical decision, the vote will be
>> > >held as -a lazy consensus of active committers.
>> > >+a lazy consensus of active committers.
>> > >
>> > >-Any user, contributor, committer or PMC member can initiate a
>> > >technical
>> > >+Any user, contributor, committer, or PMC member can initiate a
>> > >+technical
>> > > decision making process.
>> > >
>> > >-3.4.2. Release Plan
>> > >+3.4.2. Non-Technical Decisions
>> > >
>> > >+A non-technical decisions is any decision that does not involve
>> > >+changes
>> > >to
>> > >the
>> > >+source code that we distribute in our official releases.
>> > >+
>> > >+Non-technical decisions should normally be made by the entire
>> > >+community
>> > >using
>> > >+discussion-lead consensus-building, and not through formal voting.
>> > >+
>> > >+Non-technical decisions can be made on whichever project mailing list
>> > >+is
>> > >most
>> > >+appropriate.
>> > >+
>> > >+Non-technical decisions cannot be vetoed, but if there is strong
>> > >opposition
>> > >+a formal vote can be used to resolve the dispute.
>> > >+
>> > >+If a formal vote is started for a non-technical decision, the vote
>> > >+will
>> > >be
>> > >held
>> > >+as a lazy 2/3 majority of active committers.
>> > >+
>> > >+Any user, contributor, committer, or PMC member can initiate a
>> > >non-technical
>> > >+decision making process.
>> > >+
>> > >+3.4.3. Release Plan
>> > >+
>> > > Defines the timetable and work items for a release. The plan also
>> > >nominates a  Release Manager.
>> > >
>> > > A lazy majority of active committers is required for approval.
>> > >
>> > >-Any active committer or PMC member may call a vote. The vote must
>> > >occur on a
>> > >+Any active committer or PMC member may call a vote. The vote must
>> > >+occur
>> >

Re: [VOTE] Update by-laws: non-technical decisions and other minor changes

2013-08-12 Thread Noah Slater
Bumping the thread for a 3rd +1 vote from the PMC. Our by-laws stipulate we
need 3 +1 PMC votes.

Thanks.


On 5 August 2013 22:43, Noah Slater  wrote:

> Hi dev,
>
> I have some more by-law changes to propose. This is essentially round 2
> for these changes. I incorporated feedback from the last thread.
>
> Per the by-laws, we're using a lazy majority for this vote. Please cast
> your vote now. I will tally the results in 72 hours.
>
> Here's my changelog:
>
> * Removed some spurious   entities
>
> * Added "A technical decision is any decision that involves changes to the
> source code
> that we distribute in our official releases." to § 3.4.1 (Technical
> Decisions).
>
> * Added "discussion-lead" before "consensus gathering" in this section.
>
> * With the improved definition, I have tightened up the wording so that
> technical decisions must be made on @dev.
>
> * Added § 3.4.2, Non-Technical Decisions. Non-technical decisions are
> defined as in the inverse of technical decisions. They can be made on
> whatever list is appropriate. Formal voting will use a lazy 2/3 majority.
> Votes cannot be vetoed.
>
> * Changed § 3.4.3. (Release Plan) to limit decisions to @dev.
>
> * Changed § 3.4.4. (Product Release) to limit decisions to @dev.
>
> * Changed § 3.4.5. (Adoption of New Codebase) to limit decisions to @dev.
>
> * Changed § 3.4.10. (Modifying Bylaws) to limit decisions to @dev.
>
> * Added an Oxford comma to the final paras of § 3.4.1. and § 3.4.2.
>
> * Section renumbering to accommodate § 3.4.2.
>
> And here's the patch:
>
> Index: bylaws.mdtext
> ===
> --- bylaws.mdtext (revision 1510739)
> +++ bylaws.mdtext (working copy)
> @@ -198,41 +198,64 @@
>
>  3.4.1. Technical Decisions
>
> +A technical decision is any decision that involves changes to the source
> code
> +that we distribute in our official releases.
> +
>  Technical decisions should normally be made by the entire community using
> -consensus gathering, and not through formal voting.
> +discussion-lead consensus gathering, and not through formal voting.
>
> -Technical decisions must be made on a project development mailing list.
> +Technical decisions must be made on the project development mailing list.
>
>  During the consensus gathering process, technical decisions may be vetoed
> by
>  any Committer with a valid reason.
>
>  If a formal vote is started for a technical decision, the vote will be
> held as
> -a lazy consensus of active committers.
> +a lazy consensus of active committers.
>
> -Any user, contributor, committer or PMC member can initiate a technical
> +Any user, contributor, committer, or PMC member can initiate a technical
>  decision making process.
>
> -3.4.2. Release Plan
> +3.4.2. Non-Technical Decisions
>
> +A non-technical decisions is any decision that does not involve changes
> to the
> +source code that we distribute in our official releases.
> +
> +Non-technical decisions should normally be made by the entire community
> using
> +discussion-lead consensus-building, and not through formal voting.
> +
> +Non-technical decisions can be made on whichever project mailing list is
> most
> +appropriate.
> +
> +Non-technical decisions cannot be vetoed, but if there is strong
> opposition
> +a formal vote can be used to resolve the dispute.
> +
> +If a formal vote is started for a non-technical decision, the vote will
> be held
> +as a lazy 2/3 majority of active committers.
> +
> +Any user, contributor, committer, or PMC member can initiate a
> non-technical
> +decision making process.
> +
> +3.4.3. Release Plan
> +
>  Defines the timetable and work items for a release. The plan also
> nominates a
>  Release Manager.
>
>  A lazy majority of active committers is required for approval.
>
> -Any active committer or PMC member may call a vote. The vote must occur
> on a
> +Any active committer or PMC member may call a vote. The vote must occur
> on the
>  project development mailing list.
>
> -3.4.3. Product Release
> +3.4.4. Product Release
>
>  When a release of one of the project's products is ready, a vote is
> required to
>  accept the release as an official release of the project.
>
>  Lazy Majority of active PMC members is required for approval.
>
> -Any active committer or PMC member may call a vote. The vote must occur
> on a
> +Any active committer or PMC member may call a vote. The vote must occur
> on the
>  project development mailing list.
>
> -3.4.4. Adoption of New Codebase
> +3.4.5. Adoption of New Codebase
>
>  When the codebase for an existing, released product is to be replaced
> with an
>  alternative codebase. If such a vote fails to gain approval, the existing
> code
> @@ -242,10 +265,10 @@
>
>  Lazy 2/3 majority of active PMC members.
>
> -Any active committer or PMC member may call a vote. The vote must occur
> on a
> +Any active committer or PMC member may call a vote. The vote must occur
> on the
>  project development mailing list.
>

Re: 13317

2013-08-12 Thread Daan Hoogland
I get The following:

>From badb31ba4f5b1ed833a4509bd65a02d2977e41dc Mon Sep 17 00:00:00 2001
From: Daan Hoogland 
Date: Tue, 6 Aug 2013 10:26:58 +0200
Subject: [PATCH] db upgrade 4.1.1 -> master

---
 engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java
b/engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java
index eb987ea..a9cdeb6 100755
--- a/engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java
+++ b/engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java
@@ -175,6 +175,8 @@ public class DatabaseUpgradeChecker implements
SystemIntegrityChecker {

 _upgradeMap.put("4.1.0", new DbUpgrade[] { new
Upgrade410to420(), new Upgrade420to430() });

+_upgradeMap.put("4.1.1", new DbUpgrade[] { new
Upgrade410to420(), new Upgrade420to430() });
+
 _upgradeMap.put("4.2.0", new DbUpgrade[] { new Upgrade420to430() });

 //CP Upgrades
-- 
1.8.0.msysgit.0


On Mon, Aug 12, 2013 at 6:06 PM, Daan Hoogland  wrote:
> yes,
>
> it contains
>
> _upgradeMap.put("4.1.1", new DbUpgrade[] { new
> Upgrade410to420(), new Upgrade420to430() });
>
> to add to the DatabaseUpgradeChecker
>
> Daan
>
> On Mon, Aug 12, 2013 at 5:52 PM, Joe Brockmeier  wrote:
>> Did you attach a patch to this? I see nothing.
>>
>> On Mon, Aug 12, 2013, at 10:43 AM, Daan Hoogland wrote:
>>> Can committer review https://reviews.apache.org/r/13317/, please?
>>>
>>> regards,
>>> Daan
>>
>>
>> Best,
>>
>> jzb
>> --
>> Joe Brockmeier
>> j...@zonker.net
>> Twitter: @jzb
>> http://www.dissociatedpress.net/


Re: Password requested when starting CS MS

2013-08-12 Thread Mike Tutkowski
Ah, thanks, Pedro


On Mon, Aug 12, 2013 at 11:31 AM, Pedro Roque Marques <
pedro.r.marq...@gmail.com> wrote:

> Mike,
> If you create a cloudstack-management package and run
> "cloudstack-setup-management" before  starting the management server you
> can run the MS as user cloud without experiencing these issues...
> It sets up the correct privileges.
>
>   Pedro.
>
> On Aug 12, 2013, at 9:47 AM, Mike Tutkowski wrote:
>
> > Yeah, it seems to only happen when I have a fresh install.
> >
> > I also notice - in this situation - that I get a password prompt in my
> > console when trying to upload an ISO for the first time.
> >
> > Chris - I once wondered about running the CS MS as root, but someone
> warned
> > me not to because it might cover up issues that would otherwise be
> exposed
> > if I weren't running as root. What do you think about that?
> >
> > Thanks, guys!
> >
> >
> > On Mon, Aug 12, 2013 at 9:54 AM, SuichII, Christopher <
> > chris.su...@netapp.com> wrote:
> >
> >> I got around this by running the CS server as root. Would that be an
> >> option for you, Mike?
> >>
> >> On Aug 12, 2013, at 11:19 AM, Daan Hoogland 
> >> wrote:
> >>
> >>> does this only happen on a new installation/clean db? Sounds like
> >>> admin account not enabled yet.
> >>>
> >>> Daan
> >>>
> >>> On Tue, Aug 6, 2013 at 7:21 PM, Mike Tutkowski
> >>>  wrote:
>  -- Forwarded message --
>  From: Mike Tutkowski 
>  Date: Tue, Aug 6, 2013 at 11:19 AM
>  Subject: Password requested when starting CS MS
>  To: "dev@cloudstack.apache.org" 
> 
> 
>  Hi,
> 
>  This happens to me every now and then, but - with the most recent 4.2
> >> code
>  - I seem to be seeing it again.
> 
>  When I start up the CS MS, I get a keystore exception and have to hit
>   several times on a Password prompt to get the CS MS to
> continue
>  starting up.
> 
>  Is this acceptable or am I missing out on some functionality with this
>  happening?
> 
>  Does anyone know why I'm prompted for a password? Even if I enter in
> one
>  (the root password), it still prompts me again for my password. Maybe
>  that's not the password it's looking for?
> 
>  Thanks!
> 
>  WARN  [cloud.server.ConfigurationServerImpl] (Timer-2:) Would use
> >> fail-safe
>  keystore to continue.
>  java.io.IOException: Fail to generate certificate!: timeout
>  at
> 
> >>
> com.cloud.server.ConfigurationServerImpl.generateDefaultKeystore(ConfigurationServerImpl.java:496)
>  at
> 
> >>
> com.cloud.server.ConfigurationServerImpl.updateSSLKeystore(ConfigurationServerImpl.java:517)
>  at
> 
> >>
> com.cloud.server.ConfigurationServerImpl.persistDefaultValues(ConfigurationServerImpl.java:289)
>  at
> 
> >>
> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
>  at
> 
> >>
> com.cloud.server.ConfigurationServerImpl.configure(ConfigurationServerImpl.java:157)
>  at
> 
> >>
> com.cloud.utils.component.ComponentContext.initComponentsLifeCycle(ComponentContext.java:111)
>  at
> >> com.cloud.servlet.CloudStartupServlet$1.run(CloudStartupServlet.java:54)
>  at java.util.TimerThread.mainLoop(Timer.java:512)
>  at java.util.TimerThread.run(Timer.java:462)
>  INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Processing
>  updateKeyPairs
>  INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Keypairs
> >> already in
>  database, updating local copy
>  INFO  [cloud.server.ConfigurationServerImpl] (Timer-2:) Going to
> update
>  systemvm iso with generated keypairs if needed
>  Password:
>  Password:
>  Password:
>  Password:
>  Password:
> 
>  --
>  *Mike Tutkowski*
>  *Senior CloudStack Developer, SolidFire Inc.*
>  e: mike.tutkow...@solidfire.com
>  o: 303.746.7302
>  Advancing the way the world uses the
>  cloud
>  *™*
> 
> 
> 
>  --
>  *Mike Tutkowski*
>  *Senior CloudStack Developer, SolidFire Inc.*
>  e: mike.tutkow...@solidfire.com
>  o: 303.746.7302
>  Advancing the way the world uses the
>  cloud
>  *™*
> >>
> >>
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the
> > cloud
> > *™*
>
>


-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-12 Thread Vijayendra Bhamidipati
Thanks Rohit, Bharat, Sanjay, Abhinav, Hari, Issac, Radhika, Animesh, Likitha, 
Saksham, Anshul, Krishnan, Sadhu, Ian, Wei, Hugo! ^_^

Regards,
Vijay

-Original Message-
From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com] 
Sent: Monday, August 12, 2013 2:13 AM
To: dev@cloudstack.apache.org
Subject: RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

Congrats!

:D

> -Original Message-
> From: Wei ZHOU [mailto:ustcweiz...@gmail.com]
> Sent: maandag 12 augustus 2013 11:12
> To: dev@cloudstack.apache.org
> Subject: Re: [ANNOUNCE] New Committer: Vijay Bhamidipati
> 
> Congrats Vijay!
> 
> 
> 2013/8/12 Ian Duffy 
> 
> > Congrats Vijay!
> >


Build failure in 4.2 with Midokura Midonet

2013-08-12 Thread Mike Tutkowski
Has anyone else been seeing this?

[INFO] Apache CloudStack Plugin - Midokura Midonet ... FAILURE [1.864s]

It doesn't happen all the time, but I've seen it three times in a row now.
I assume it's a test failure, but I haven't looked into it much.

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: Build failure in 4.2 with Midokura Midonet

2013-08-12 Thread Mike Tutkowski
Often times if I do a clean, then install, it fixes the problem.


On Mon, Aug 12, 2013 at 2:34 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Has anyone else been seeing this?
>
> [INFO] Apache CloudStack Plugin - Midokura Midonet ... FAILURE [1.864s]
>
> It doesn't happen all the time, but I've seen it three times in a row now.
> I assume it's a test failure, but I haven't looked into it much.
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the 
> cloud
> *™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Template Folder Path (KVM/Libvirt)

2013-08-12 Thread Marty Sweet
Hi all,

I've hit a problem that I have been chewing on for the past few hours. A
few months ago I changed the local storage path naming convention (examples
below) and updated the corresponding records in the cloudstack database
manually. This works great until old templates are used. I understand I
could use symlinks to resolve this issue but I would prefer to have a sane
dataset and a cleaner setup :)

--> My Physical Volumes <--
/mount/msa0-enc0-vm0 (OLD /mount/enc0-vm)
/mount/msa0-enc0-vm1
/mount/msa0-enc1-vm0 (OLD /mount/enc1-vm)
/mount/msa0-enc1-vm1
/mount/msa0-enc2-vm0

In the database there is no mention of /mount/enc0-vm or /mount/enc1-vm,
from doing a global search on all content (and by dumping the database
and grepping the output). However, libvirt still manages
to receive commands to add these templates.

virStorageBackendFileSystemRefresh:889 : internal error cannot probe
backing volume info: /mount/enc1-vm/a0932ff2-e97c-446a-85f2-42ad0fb2cab7
virStorageBackendProbeTarget:118 : internal error cannot probe backing
volume format: /mount/enc0-vm/8dc8d179-ac43-4475-812a-3bcc3e5d7b43

These UUID Paths do correspond to templates within the '*
template_spool_ref' *table which then *presumably *are linked to
`storage_pool` to pick out the folder 'path'? although this should give the
correct, new mappings. I can rule out a storage pool issue within libvirt
as I resolved that issue when I changed the naming convention and have
rechecked it.

Unlike VM `volumes`, templates don't seem to possess a folder 'path', the
only possible field I can assume is `vm_template.checksum`, which is
obviously questionable, but would explain why a plain text search for the
old volumes is returning no results.

So my question is: How does CS compose template paths and from what parts
within the cloudstack database? If these are encrypted (as I am assuming
the folder path is somewhere, what is the best way to change these values).

Thanks in advance,
Marty Sweet (Rapid2214)


Re: Template Folder Path (KVM/Libvirt)

2013-08-12 Thread Ahmad Emneina
just dawned on me, you might have to forget the storage pools (if no vm's
are running) on the hosts. Then reconnect the kvm agents (by restarting the
service on the hosts or from cloudstack-> force reconnect command). they
should get programmed correctly!


On Mon, Aug 12, 2013 at 2:16 PM, Marty Sweet  wrote:

> Hi all,
>
> I've hit a problem that I have been chewing on for the past few hours. A
> few months ago I changed the local storage path naming convention (examples
> below) and updated the corresponding records in the cloudstack database
> manually. This works great until old templates are used. I understand I
> could use symlinks to resolve this issue but I would prefer to have a sane
> dataset and a cleaner setup :)
>
> --> My Physical Volumes <--
> /mount/msa0-enc0-vm0 (OLD /mount/enc0-vm)
> /mount/msa0-enc0-vm1
> /mount/msa0-enc1-vm0 (OLD /mount/enc1-vm)
> /mount/msa0-enc1-vm1
> /mount/msa0-enc2-vm0
>
> In the database there is no mention of /mount/enc0-vm or /mount/enc1-vm,
> from doing a global search on all content (and by dumping the database
> and grepping the output). However, libvirt still manages
> to receive commands to add these templates.
>
> virStorageBackendFileSystemRefresh:889 : internal error cannot probe
> backing volume info: /mount/enc1-vm/a0932ff2-e97c-446a-85f2-42ad0fb2cab7
> virStorageBackendProbeTarget:118 : internal error cannot probe backing
> volume format: /mount/enc0-vm/8dc8d179-ac43-4475-812a-3bcc3e5d7b43
>
> These UUID Paths do correspond to templates within the '*
> template_spool_ref' *table which then *presumably *are linked to
> `storage_pool` to pick out the folder 'path'? although this should give the
> correct, new mappings. I can rule out a storage pool issue within libvirt
> as I resolved that issue when I changed the naming convention and have
> rechecked it.
>
> Unlike VM `volumes`, templates don't seem to possess a folder 'path', the
> only possible field I can assume is `vm_template.checksum`, which is
> obviously questionable, but would explain why a plain text search for the
> old volumes is returning no results.
>
> So my question is: How does CS compose template paths and from what parts
> within the cloudstack database? If these are encrypted (as I am assuming
> the folder path is somewhere, what is the best way to change these values).
>
> Thanks in advance,
> Marty Sweet (Rapid2214)
>


Re: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang

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

Ship it!


This is very straightforward patch. Java filesystem api is known as unnecessary 
complex, we should take advantage of Apache common-io as much as possible.
shipped in 4.2 and master

- Frank Zhang


On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11942/
> ---
> 
> (Updated July 24, 2013, 4:42 a.m.)
> 
> 
> Review request for cloudstack, Frank Zhang and John Burwell.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - possible resource leak closed
> - file content read uses now commons-lang FileUtils
> - Added unit tests
> 
> 
> Diffs
> -
> 
>   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35 
>   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/11942/diff/
> 
> 
> Testing
> ---
> 
> test included
> 
> 
> Thanks,
> 
> Laszlo Hornyak
> 
>



Re: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread John Burwell
Laszlo,

I was only concerned about the lack of asserts in the original test case.  That 
has been updated, so it looks good to me now.

Thanks,
-John

On Aug 12, 2013, at 5:43 PM, "Frank Zhang"  wrote:

> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11942/#review25036
> ---
> 
> Ship it!
> 
> 
> This is very straightforward patch. Java filesystem api is known as 
> unnecessary complex, we should take advantage of Apache common-io as much as 
> possible.
> shipped in 4.2 and master
> 
> - Frank Zhang
> 
> 
> On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
>> 
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/11942/
>> ---
>> 
>> (Updated July 24, 2013, 4:42 a.m.)
>> 
>> 
>> Review request for cloudstack, Frank Zhang and John Burwell.
>> 
>> 
>> Repository: cloudstack-git
>> 
>> 
>> Description
>> ---
>> 
>> - possible resource leak closed
>> - file content read uses now commons-lang FileUtils
>> - Added unit tests
>> 
>> 
>> Diffs
>> -
>> 
>>  utils/src/com/cloud/utils/ProcessUtil.java c9fdf35 
>>  utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION 
>> 
>> Diff: https://reviews.apache.org/r/11942/diff/
>> 
>> 
>> Testing
>> ---
>> 
>> test included
>> 
>> 
>> Thanks,
>> 
>> Laszlo Hornyak
>> 
>> 
> 



signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: IPv6 plan - questions

2013-08-12 Thread Marcus Sorensen
has there been any further discussion that I might have missed around
ipv6 in VPC?

On Thu, Mar 7, 2013 at 12:09 PM, Sheng Yang  wrote:
> Hi Dave,
>
> I am glad it fits your need. That's our target. :)
>
> --Sheng
>
> On Thu, Mar 7, 2013 at 2:14 AM, Dave Cahill  wrote:
>> Hi Sheng,
>>
>> Thanks for the quick reply, that helps a lot.
>>
>> My main purpose was to figure out how these changes affect virtual
>> networking and pluggability. Having read through the IPv6 code today,
>> it looks like it will work very nicely with virtual networks.
>>
>> For example, when VMs are assigned an IPv6 address, the IPv6 address
>> is stored in the NicProfile object. So, taking DHCP as an example, if
>> the MidoNet plugin implements the DHCPServiceProvider interface, it
>> will receive the NicProfile as one of the parameters of addDhcpEntry.
>> If we want to implement IPv6, we can then take the IPv6 address from
>> the NicProfile, and just use it as needed.
>>
>> Thanks again for taking the time to respond, and for the detailed FS.
>>
>> Dave.
>>
>> On Thu, Mar 7, 2013 at 4:57 AM, Sheng Yang  wrote:
>>
>>> On Wed, Mar 6, 2013 at 1:36 AM, Dave Cahill  wrote:
>>> > Hi,
>>>
>>> Hi Dave,
>>> >
>>> > I've been catching up on IPv6 plans by reading the functional specs
>>> > and Jira tickets - it's great to have so much material to refer to.
>>> >
>>> > I still have a few questions though, and I'm hoping someone involved
>>> > with the feature can enlighten me.
>>> >
>>> > *[Support for Providers other than Virtual Router]*
>>> > In [3], the spec says "No external device support in plan."
>>> > What does this mean exactly?
>>>
>>> Because CloudStack also supports using external devices as network
>>> controller e.g. Juniper SRX as firewall and NetScaler as load
>>> balancer. The words here said is just we don't support these devices
>>> when using IPv6.
>>> >
>>> > For example, if using Providers other than the Virtual Router, does
>>> > the UI still allow setting IPv6 addresses?
>>> >
>>> > If so, do we attempt to pass IPv6 addresses to the Providers no
>>> > matter what, or do we check whether the Provider has IPv6 support?
>>>
>>> Yes, we checked it when you try to create a IPv6 network(currently
>>> only support advance shared network).
>>>
>>> >
>>> > *[Networking Modes]*
>>> > Advanced Shared mode and Basic mode are mentioned in the Jira
>>> > ticket [1] - "Isolated Network" is mentioned briefly in [2], but I
>>> > wanted to check if the Advanced Isolated and VPC modes are on the
>>> > roadmap?
>>>
>>> There is no "basic isolated" network, so "Isolated" network is what
>>> we're talking about. We haven't got plan for VPC yet.
>>>
>>> And one correction: we didn't support "basic" mode for phase 1. We
>>> support only "advance shared network" in phase 1. The supported cases
>>> are described in FS. Jira ticket only provided a rough idea at the
>>> time.
>>> >
>>> > *[IP Address Management / IPAM]*
>>> > From [1], re: handing out IPv6 addresses: "One way could be that the
>>> > network admin creates a static route for a /48 towards a Virtual
>>> > Router and then the VR can hand out /64s to Instances."
>>> >
>>> > With IPv4, IPAM is handled by the CloudStack management server, and
>>> > the VR is told which IP address to give to the VM over DHCP. Would
>>> > this change with IPv6? "The VR can hand out /64s to instances" sounds
>>> > like the VR is handling IPAM to some extent.
>>>
>>> Well, it's not how it works now. Please refer to the FS. The current
>>> implementation works like before. VR get a /64 then handle out IPv6
>>> addresses to VM.
>>> >
>>> > From [3], "Router advertisement should be sent by public gateway in
>>> > the network." - to double-check, does this mean the router outside the
>>> > CloudStack network should send RAs, but the VR won't send RAs?
>>>
>>> Yes. Because in phase 1, we support only "advance shared network", in
>>> which case, VR is NOT the gateway. So we assume the gateway router
>>> outside CloudStack should send out RA to the VMs.
>>>
>>> But in the phase 2, VR would acting as gateway, then it would send out RAs.
>>>
>>> --Sheng
>>> >
>>> > Thanks,
>>> > Dave.
>>> >
>>> > [1] IPv6 Support main Jira ticket
>>> > https://issues.apache.org/jira/browse/CLOUDSTACK-452
>>> >
>>> > [2] IPv6 Support in CloudStack FS
>>> > https://cwiki.apache.org/CLOUDSTACK/ipv6-support-in-cloudstack.html
>>> >
>>> > [3] IPv6 Support FS
>>> > https://cwiki.apache.org/CLOUDSTACK/ipv6-support.html
>>>


RE: [jira] [Commented] (CLOUDSTACK-4149) [upgrade][2.2.13 -> 2.2.14 -> 4.2][KVM] When we try to upgrade the KVM agent from 2.2.14 to 4.2 using the "U" option in install.sh script, management server al

2013-08-12 Thread Animesh Chaturvedi
Copying Hugo

> -Original Message-
> From: frank zhang (JIRA) [mailto:j...@apache.org]
> Sent: Monday, August 12, 2013 3:08 PM
> To: cloudstack-iss...@incubator.apache.org
> Subject: [jira] [Commented] (CLOUDSTACK-4149) [upgrade][2.2.13 ->
> 2.2.14 -> 4.2][KVM] When we try to upgrade the KVM agent from 2.2.14 to
> 4.2 using the "U" option in install.sh script, management server also
> gets installed!
> 
> 
> [ https://issues.apache.org/jira/browse/CLOUDSTACK-
> 4149?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel&focusedCommentId=13737421#comment-13737421 ]
> 
> frank zhang commented on CLOUDSTACK-4149:
> -
> 
> The issue is caused by rpm reorganization in ACS. ACS uses cloudstack-
> management package to make cloud-agent as obsolete, this causes
> cloudstack-management to be unnecessarily installed during agent
> upgrade
> 
> 
> > [upgrade][2.2.13 -> 2.2.14 -> 4.2][KVM] When we try to upgrade the
> KVM agent from 2.2.14 to 4.2 using the "U" option in install.sh script,
> management server also gets installed!
> > -
> -
> > -
> -
> > -
> >
> > Key: CLOUDSTACK-4149
> > URL:
> https://issues.apache.org/jira/browse/CLOUDSTACK-4149
> > Project: CloudStack
> >  Issue Type: Bug
> >  Security Level: Public(Anyone can view this level - this is the
> default.)
> >  Components: Packaging, Upgrade
> >Affects Versions: 4.2.0
> > Environment: upgrade from 2.2.13 (rhel 6.1 build) -> 2.2.14
> > (rhel 6.1 build) -> 4.2 (rhel 6.2 build) MS : CentOS 6.1 KVM : CentOS
> > 6.1
> >Reporter: Abhinav Roy
> >Assignee: frank zhang
> >Priority: Blocker
> > Fix For: 4.2.0
> >
> >
> > 1. Deploy a CS 2.2.13 advanced zone setup.
> > 2. Do some operations and upgrade to 2.2.14 3. Now when we try to
> > upgrade it from 2.2.14 to 4.2 using the "U" option in install.sh
> script, it installs the management server also.
> > [root@centos61-band28 CloudPlatform-4.2-4.2-104-rhel6.2]#
> ./install.sh
> > Setting up the temporary repository...
> > Cleaning Yum cache...
> > Loaded plugins: fastestmirror
> > Cleaning repos: cloud-temp rhel
> > 2 metadata files removed
> > Welcome to the CloudPlatform Installer.  What would you like to do?
> > NOTE:   For installing KVM agent, please setup
> EPEL yum repo first;
> > For installing CloudPlatform on RHEL6.x, please setup
> distribution yum repo either from ISO or from your registeration
> account.
> > M) Install the Management Server
> > A) Install the Agent
> > B) Install BareMetal Agent
> > S) Install the Usage Monitor
> > D) Install the database server (from distribution's repo)
> > U) Upgrade the CloudPlatform packages installed on this computer
> > R) Stop any running CloudPlatform services and remove the
> CloudPlatform packages from this computer
> > L) Install the MySQL 5.1.58 (only for CentOS5.x, Rhel6.x
> naturally has higher version MySql)
> > Q) Quit
> >  > u
> > Updating the CloudPlatform and its dependencies...
> > Loaded plugins: fastestmirror
> > Loading mirror speeds from cached hostfile
> > cloud-temp
> | 1.3 kB 00:00 ...
> > rhel
> | 4.0 kB 00:00 ...
> > Setting up Update Process
> > Resolving Dependencies
> > --> Running transaction check
> > ---> Package cloud-agent.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-agent-libs.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-core.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-daemonize.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-deps.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-python.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloud-utils.x86_64 0:2.2.14-1.el6 will be obsoleted
> > ---> Package cloudstack-agent.x86_64 0:4.2.0-SNAPSHOT.el6 will be
> > ---> obsoleting
> > --> Processing Dependency: jakarta-commons-daemon for package:
> > --> cloudstack-agent-4.2.0-SNAPSHOT.el6.x86_64
> > --> Processing Dependency: ipset for package:
> > --> cloudstack-agent-4.2.0-SNAPSHOT.el6.x86_64
> > ---> Package cloudstack-common.x86_64 0:4.2.0-SNAPSHOT.el6 will be
> > ---> obsoleting Package cloudstack-management.x86_64
> > ---> 0:4.2.0-SNAPSHOT.el6 will be obsoleting
> > --> Processing Dependency: cloudstack-awsapi = 4.2.0 for package:
> > --> cloudstack-management-4.2.0-SNAPSHOT.el6.x86_64
> > --> Processing Dependency: ws-commons-util for package:
> > --> cloudstack-management-4.2.0-SNAPSHOT.el6.x86_64
> > --> Processing Dependency: tomcat6 for package:
> > --> cloudstack-management-4.2.0-SNAPSHOT.el6.x86_64
> > --> Processing Dependency: mysql-connector-java for package:
> > --> clou

make cloud-daemonize obsolete using cloudstack-agent instead of cloudstack-management

2013-08-12 Thread Frank Zhang
Hi Hugo:
I have an issue needing to double confirm with you. 
https://issues.apache.org/jira/browse/CLOUDSTACK-4149
In short, we used to make old cloud-daemonize obsolete using 
cloudstack-management that makes it unnecessarily installed during upgrading 
KVM agent(cloud-agent) on host.
I make a simple fix that moves "Obsoletes: cloud-daemonize < 4.1.0" to 
cloudstack-agent. Just want to double check with you there is no side-effect.
Now upgrading old cloud-agent will only install cloudstack-agent and 
cloudstack-common, I hope these two packages have no dependency on 
cloudstack-management 


Re: Review Request 13459: cloudstack-3759: deployVM with state set to be stopped, attach volume to it. The volume should not be created.

2013-08-12 Thread Frank Zhang

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


please update your test result

- Frank Zhang


On Aug. 10, 2013, 12:50 a.m., Fang Wang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13459/
> ---
> 
> (Updated Aug. 10, 2013, 12:50 a.m.)
> 
> 
> Review request for cloudstack and Frank Zhang.
> 
> 
> Bugs: cloudstack-3759
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> When deploying a new VM, with its state set to be stopped. At that time, when 
> attaching volume to it, we need to delay the volume creation. Because in DB, 
> the host id and pod id, fields for the VM are empty. 
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13459/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Fang Wang
> 
>



Re: Build failure in 4.2 with Midokura Midonet

2013-08-12 Thread Dave Cahill
Hi Mike,

If you get a failure like this, Maven should also print out more details on
the cause -
could you paste those?

4.2 was building fine for me yesterday, but I'll take a look and see if
something has changed since.

Thanks for the heads up,
Dave.

2013年8月13日火曜日 Mike Tutkowski mike.tutkow...@solidfire.com:

> Often times if I do a clean, then install, it fixes the problem.
>
>
> On Mon, Aug 12, 2013 at 2:34 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com > wrote:
>
> > Has anyone else been seeing this?
> >
> > [INFO] Apache CloudStack Plugin - Midokura Midonet ... FAILURE
> [1.864s]
> >
> > It doesn't happen all the time, but I've seen it three times in a row
> now.
> > I assume it's a test failure, but I haven't looked into it much.
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com 
> > o: 303.746.7302
> > Advancing the way the world uses the cloud<
> http://solidfire.com/solution/overview/?video=play>
> > *(tm)*
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com 
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *(tm)*
>


RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Animesh Chaturvedi
Is this really needed for 4.2? 4.2 is in limited updates and only 
blocker/critical bug fixes and doc and test updates are allowed. All commits to 
4.2 should have associated BUG id.

> -Original Message-
> From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf Of Frank
> Zhang
> Sent: Monday, August 12, 2013 2:43 PM
> To: John Burwell; Frank Zhang
> Cc: Laszlo Hornyak; cloudstack
> Subject: Re: Review Request 11942: ProcessUtil cleanup
> 
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11942/#review25036
> ---
> 
> Ship it!
> 
> 
> This is very straightforward patch. Java filesystem api is known as
> unnecessary complex, we should take advantage of Apache common-io as
> much as possible.
> shipped in 4.2 and master
> 
> - Frank Zhang
> 
> 
> On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> >
> > ---
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/11942/
> > ---
> >
> > (Updated July 24, 2013, 4:42 a.m.)
> >
> >
> > Review request for cloudstack, Frank Zhang and John Burwell.
> >
> >
> > Repository: cloudstack-git
> >
> >
> > Description
> > ---
> >
> > - possible resource leak closed
> > - file content read uses now commons-lang FileUtils
> > - Added unit tests
> >
> >
> > Diffs
> > -
> >
> >   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35
> >   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION
> >
> > Diff: https://reviews.apache.org/r/11942/diff/
> >
> >
> > Testing
> > ---
> >
> > test included
> >
> >
> > Thanks,
> >
> > Laszlo Hornyak
> >
> >



RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang
Emm. Actually it's not a blocker or even critical for 4.2. 
Though I thought it's not harm, if needs I can revert it from 4.2 but still 
keep it in master.

> -Original Message-
> From: Animesh Chaturvedi
> Sent: Monday, August 12, 2013 5:37 PM
> To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
> Cc: Laszlo Hornyak
> Subject: RE: Review Request 11942: ProcessUtil cleanup
> 
> Is this really needed for 4.2? 4.2 is in limited updates and only 
> blocker/critical
> bug fixes and doc and test updates are allowed. All commits to 4.2 should have
> associated BUG id.
> 
> > -Original Message-
> > From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf Of
> > Frank Zhang
> > Sent: Monday, August 12, 2013 2:43 PM
> > To: John Burwell; Frank Zhang
> > Cc: Laszlo Hornyak; cloudstack
> > Subject: Re: Review Request 11942: ProcessUtil cleanup
> >
> >
> > ---
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/11942/#review25036
> > ---
> >
> > Ship it!
> >
> >
> > This is very straightforward patch. Java filesystem api is known as
> > unnecessary complex, we should take advantage of Apache common-io as
> > much as possible.
> > shipped in 4.2 and master
> >
> > - Frank Zhang
> >
> >
> > On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> > >
> > > ---
> > > This is an automatically generated e-mail. To reply, visit:
> > > https://reviews.apache.org/r/11942/
> > > ---
> > >
> > > (Updated July 24, 2013, 4:42 a.m.)
> > >
> > >
> > > Review request for cloudstack, Frank Zhang and John Burwell.
> > >
> > >
> > > Repository: cloudstack-git
> > >
> > >
> > > Description
> > > ---
> > >
> > > - possible resource leak closed
> > > - file content read uses now commons-lang FileUtils
> > > - Added unit tests
> > >
> > >
> > > Diffs
> > > -
> > >
> > >   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35
> > >   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION
> > >
> > > Diff: https://reviews.apache.org/r/11942/diff/
> > >
> > >
> > > Testing
> > > ---
> > >
> > > test included
> > >
> > >
> > > Thanks,
> > >
> > > Laszlo Hornyak
> > >
> > >



RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Animesh Chaturvedi
We should keep it for master

> -Original Message-
> From: Frank Zhang
> Sent: Monday, August 12, 2013 5:40 PM
> To: Animesh Chaturvedi; dev@cloudstack.apache.org; John Burwell
> Cc: Laszlo Hornyak
> Subject: RE: Review Request 11942: ProcessUtil cleanup
> 
> Emm. Actually it's not a blocker or even critical for 4.2.
> Though I thought it's not harm, if needs I can revert it from 4.2 but
> still keep it in master.
> 
> > -Original Message-
> > From: Animesh Chaturvedi
> > Sent: Monday, August 12, 2013 5:37 PM
> > To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
> > Cc: Laszlo Hornyak
> > Subject: RE: Review Request 11942: ProcessUtil cleanup
> >
> > Is this really needed for 4.2? 4.2 is in limited updates and only
> > blocker/critical bug fixes and doc and test updates are allowed. All
> > commits to 4.2 should have associated BUG id.
> >
> > > -Original Message-
> > > From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf Of
> > > Frank Zhang
> > > Sent: Monday, August 12, 2013 2:43 PM
> > > To: John Burwell; Frank Zhang
> > > Cc: Laszlo Hornyak; cloudstack
> > > Subject: Re: Review Request 11942: ProcessUtil cleanup
> > >
> > >
> > > ---
> > > This is an automatically generated e-mail. To reply, visit:
> > > https://reviews.apache.org/r/11942/#review25036
> > > ---
> > >
> > > Ship it!
> > >
> > >
> > > This is very straightforward patch. Java filesystem api is known as
> > > unnecessary complex, we should take advantage of Apache common-io as
> > > much as possible.
> > > shipped in 4.2 and master
> > >
> > > - Frank Zhang
> > >
> > >
> > > On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> > > >
> > > > ---
> > > > This is an automatically generated e-mail. To reply, visit:
> > > > https://reviews.apache.org/r/11942/
> > > > ---
> > > >
> > > > (Updated July 24, 2013, 4:42 a.m.)
> > > >
> > > >
> > > > Review request for cloudstack, Frank Zhang and John Burwell.
> > > >
> > > >
> > > > Repository: cloudstack-git
> > > >
> > > >
> > > > Description
> > > > ---
> > > >
> > > > - possible resource leak closed
> > > > - file content read uses now commons-lang FileUtils
> > > > - Added unit tests
> > > >
> > > >
> > > > Diffs
> > > > -
> > > >
> > > >   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35
> > > >   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION
> > > >
> > > > Diff: https://reviews.apache.org/r/11942/diff/
> > > >
> > > >
> > > > Testing
> > > > ---
> > > >
> > > > test included
> > > >
> > > >
> > > > Thanks,
> > > >
> > > > Laszlo Hornyak
> > > >
> > > >



RE: Did we already started developing support VxLAN?

2013-08-12 Thread Toshiaki Hatano
Hello Mr. 曹, 

I'm currently working on VXLAN specifically to KVM with Linux Bridge (not OVS) 
and Multicast mode.
I think some network plugin also supports (or will support) VXLAN as tunneling 
protocol.

For what environment (KVM, Xen, VMware (Nexus 1kV maybe?), etc.) and which type 
of VXLAN (Multicast mode/Unicast mode)  are you planning to develop? 

Sincerely,
--
Toshiaki

-Original Message-
From: David Nalley [mailto:da...@gnsa.us]
Sent: 2013/08/12 (月) 10:40
To: dev@cloudstack.apache.org
Subject: Re: Did we already started developing support VxLAN?
 
On Mon, Aug 12, 2013 at 9:44 AM, 曹建林  wrote:
>
>  Hi guys
>
>  My company has a project to delivery support of VxLAN network , using 
> CloudStack.
> I know the version 4.x does not support vxlan.
> So I wonder what the plan to support vxlan network .
>
> We have a team about 5 guys in charge of developing the feature,
>
> So  if there is anybody in this thing . please send me email , thank you very 
> much.
>

Toshiaki from Verio has been working on VXLAN in KVM.
You can see his patch here:
https://reviews.apache.org/r/12623/
https://issues.apache.org/jira/browse/CLOUDSTACK-2328

I'm sure he'd welcome help.

--David



This email message is intended for the use of the person to whom it has been 
sent, and may contain information that is confidential or legally protected. If 
you are not the intended recipient or have received this message in error, you 
are not authorized to copy, distribute, or otherwise use this message or its 
attachments. Please notify the sender immediately by return e-mail and 
permanently delete this message and any attachments. Verio Inc. makes no 
warranty that this email is error or virus free.  Thank you.


RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Animesh Chaturvedi
ProcessUtilTest is missing Apache License Header and failing rat-test

> -Original Message-
> From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
> Sent: Monday, August 12, 2013 5:40 PM
> To: Frank Zhang; dev@cloudstack.apache.org; John Burwell
> Cc: Laszlo Hornyak
> Subject: RE: Review Request 11942: ProcessUtil cleanup
> 
> We should keep it for master
> 
> > -Original Message-
> > From: Frank Zhang
> > Sent: Monday, August 12, 2013 5:40 PM
> > To: Animesh Chaturvedi; dev@cloudstack.apache.org; John Burwell
> > Cc: Laszlo Hornyak
> > Subject: RE: Review Request 11942: ProcessUtil cleanup
> >
> > Emm. Actually it's not a blocker or even critical for 4.2.
> > Though I thought it's not harm, if needs I can revert it from 4.2 but
> > still keep it in master.
> >
> > > -Original Message-
> > > From: Animesh Chaturvedi
> > > Sent: Monday, August 12, 2013 5:37 PM
> > > To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
> > > Cc: Laszlo Hornyak
> > > Subject: RE: Review Request 11942: ProcessUtil cleanup
> > >
> > > Is this really needed for 4.2? 4.2 is in limited updates and only
> > > blocker/critical bug fixes and doc and test updates are allowed. All
> > > commits to 4.2 should have associated BUG id.
> > >
> > > > -Original Message-
> > > > From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf Of
> > > > Frank Zhang
> > > > Sent: Monday, August 12, 2013 2:43 PM
> > > > To: John Burwell; Frank Zhang
> > > > Cc: Laszlo Hornyak; cloudstack
> > > > Subject: Re: Review Request 11942: ProcessUtil cleanup
> > > >
> > > >
> > > > ---
> > > > This is an automatically generated e-mail. To reply, visit:
> > > > https://reviews.apache.org/r/11942/#review25036
> > > > ---
> > > >
> > > > Ship it!
> > > >
> > > >
> > > > This is very straightforward patch. Java filesystem api is known
> > > > as unnecessary complex, we should take advantage of Apache
> > > > common-io as much as possible.
> > > > shipped in 4.2 and master
> > > >
> > > > - Frank Zhang
> > > >
> > > >
> > > > On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> > > > >
> > > > > ---
> > > > > This is an automatically generated e-mail. To reply, visit:
> > > > > https://reviews.apache.org/r/11942/
> > > > > ---
> > > > >
> > > > > (Updated July 24, 2013, 4:42 a.m.)
> > > > >
> > > > >
> > > > > Review request for cloudstack, Frank Zhang and John Burwell.
> > > > >
> > > > >
> > > > > Repository: cloudstack-git
> > > > >
> > > > >
> > > > > Description
> > > > > ---
> > > > >
> > > > > - possible resource leak closed
> > > > > - file content read uses now commons-lang FileUtils
> > > > > - Added unit tests
> > > > >
> > > > >
> > > > > Diffs
> > > > > -
> > > > >
> > > > >   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35
> > > > >   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION
> > > > >
> > > > > Diff: https://reviews.apache.org/r/11942/diff/
> > > > >
> > > > >
> > > > > Testing
> > > > > ---
> > > > >
> > > > > test included
> > > > >
> > > > >
> > > > > Thanks,
> > > > >
> > > > > Laszlo Hornyak
> > > > >
> > > > >



RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang
I reverted it from 4.2. now is adding the Apache header to master

> -Original Message-
> From: Animesh Chaturvedi
> Sent: Monday, August 12, 2013 5:56 PM
> To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
> Cc: Laszlo Hornyak
> Subject: RE: Review Request 11942: ProcessUtil cleanup
> 
> ProcessUtilTest is missing Apache License Header and failing rat-test
> 
> > -Original Message-
> > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
> > Sent: Monday, August 12, 2013 5:40 PM
> > To: Frank Zhang; dev@cloudstack.apache.org; John Burwell
> > Cc: Laszlo Hornyak
> > Subject: RE: Review Request 11942: ProcessUtil cleanup
> >
> > We should keep it for master
> >
> > > -Original Message-
> > > From: Frank Zhang
> > > Sent: Monday, August 12, 2013 5:40 PM
> > > To: Animesh Chaturvedi; dev@cloudstack.apache.org; John Burwell
> > > Cc: Laszlo Hornyak
> > > Subject: RE: Review Request 11942: ProcessUtil cleanup
> > >
> > > Emm. Actually it's not a blocker or even critical for 4.2.
> > > Though I thought it's not harm, if needs I can revert it from 4.2
> > > but still keep it in master.
> > >
> > > > -Original Message-
> > > > From: Animesh Chaturvedi
> > > > Sent: Monday, August 12, 2013 5:37 PM
> > > > To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
> > > > Cc: Laszlo Hornyak
> > > > Subject: RE: Review Request 11942: ProcessUtil cleanup
> > > >
> > > > Is this really needed for 4.2? 4.2 is in limited updates and only
> > > > blocker/critical bug fixes and doc and test updates are allowed.
> > > > All commits to 4.2 should have associated BUG id.
> > > >
> > > > > -Original Message-
> > > > > From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf
> > > > > Of Frank Zhang
> > > > > Sent: Monday, August 12, 2013 2:43 PM
> > > > > To: John Burwell; Frank Zhang
> > > > > Cc: Laszlo Hornyak; cloudstack
> > > > > Subject: Re: Review Request 11942: ProcessUtil cleanup
> > > > >
> > > > >
> > > > > ---
> > > > > This is an automatically generated e-mail. To reply, visit:
> > > > > https://reviews.apache.org/r/11942/#review25036
> > > > > ---
> > > > >
> > > > > Ship it!
> > > > >
> > > > >
> > > > > This is very straightforward patch. Java filesystem api is known
> > > > > as unnecessary complex, we should take advantage of Apache
> > > > > common-io as much as possible.
> > > > > shipped in 4.2 and master
> > > > >
> > > > > - Frank Zhang
> > > > >
> > > > >
> > > > > On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote:
> > > > > >
> > > > > > ---
> > > > > > This is an automatically generated e-mail. To reply, visit:
> > > > > > https://reviews.apache.org/r/11942/
> > > > > > ---
> > > > > >
> > > > > > (Updated July 24, 2013, 4:42 a.m.)
> > > > > >
> > > > > >
> > > > > > Review request for cloudstack, Frank Zhang and John Burwell.
> > > > > >
> > > > > >
> > > > > > Repository: cloudstack-git
> > > > > >
> > > > > >
> > > > > > Description
> > > > > > ---
> > > > > >
> > > > > > - possible resource leak closed
> > > > > > - file content read uses now commons-lang FileUtils
> > > > > > - Added unit tests
> > > > > >
> > > > > >
> > > > > > Diffs
> > > > > > -
> > > > > >
> > > > > >   utils/src/com/cloud/utils/ProcessUtil.java c9fdf35
> > > > > >   utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION
> > > > > >
> > > > > > Diff: https://reviews.apache.org/r/11942/diff/
> > > > > >
> > > > > >
> > > > > > Testing
> > > > > > ---
> > > > > >
> > > > > > test included
> > > > > >
> > > > > >
> > > > > > Thanks,
> > > > > >
> > > > > > Laszlo Hornyak
> > > > > >
> > > > > >



Re: Build failure in 4.2 with Midokura Midonet

2013-08-12 Thread Dave Cahill
Hi Mike,

I'm not seeing any failures with the latest 4.2.

It sounds like the failure you were seeing was intermittent, so if it crops
up again, just paste in some more logs and I'll take a look.

Thanks,
Dave.


On Tue, Aug 13, 2013 at 8:15 AM, Dave Cahill  wrote:

> Hi Mike,
>
> If you get a failure like this, Maven should also print out more details
> on the cause -
> could you paste those?
>
> 4.2 was building fine for me yesterday, but I'll take a look and see if
> something has changed since.
>
> Thanks for the heads up,
> Dave.
>
> 2013年8月13日火曜日 Mike Tutkowski mike.tutkow...@solidfire.com:
>
> Often times if I do a clean, then install, it fixes the problem.
>>
>>
>> On Mon, Aug 12, 2013 at 2:34 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>> > Has anyone else been seeing this?
>> >
>> > [INFO] Apache CloudStack Plugin - Midokura Midonet ... FAILURE
>> [1.864s]
>> >
>> > It doesn't happen all the time, but I've seen it three times in a row
>> now.
>> > I assume it's a test failure, but I haven't looked into it much.
>> >
>> > --
>> > *Mike Tutkowski*
>> > *Senior CloudStack Developer, SolidFire Inc.*
>> > e: mike.tutkow...@solidfire.com
>> > o: 303.746.7302
>> > Advancing the way the world uses the cloud<
>> http://solidfire.com/solution/overview/?video=play>
>> > *(tm)*
>> >
>>
>>
>>
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the
>> cloud
>> *(tm)*
>>
>


Re: Review Request 13496: changes for guest vm password script for parallel vm deployment

2013-08-12 Thread Chiradeep Vittal

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


#1. if you add fork to the TCP_LISTEN option of SOCAT, then it will fork a 
process for each connection, allowing more parallelism
#2. There is a bug in serve_password.sh (see below)
#3. You can also add 'su=nobody' to the TCP4_LISTEN option to increase the 
security of the procedure (after all we are blindly accepting strings from 
potentially untrusted vm)

diff --git a/patches/systemvm/debian/config/opt/cloud/bin/passwd_server_ip 
b/patches/systemvm/debian/config/opt/cloud/bin/passwd_server_ip
index 8d62dff..4622860 100755
--- a/patches/systemvm/debian/config/opt/cloud/bin/passwd_server_ip
+++ b/patches/systemvm/debian/config/opt/cloud/bin/passwd_server_ip
@@ -20,7 +20,7 @@
 addr=$1;
 while [ "$ENABLED" == "1" ]
 do
-   socat -lf /var/log/cloud.log TCP4-LISTEN:8080,reuseaddr,crnl,bind=$addr 
SYSTEM:"/opt/cloud/bin/serve_password.sh \"\$SOCAT_PEERADDR\""
+   socat -lf /var/log/cloud.log 
TCP4-LISTEN:8080,reuseaddr,su=nobody,fork,crnl,bind=$addr 
SYSTEM:"/opt/cloud/bin/serve_password.sh \"\$SOCAT_PEERADDR\""
 
rc=$?
if [ $rc -ne 0 ]
diff --git a/patches/systemvm/debian/config/opt/cloud/bin/serve_password.sh 
b/patches/systemvm/debian/config/opt/cloud/bin/serve_password.sh
index b829b54..a3a2732 100755
--- a/patches/systemvm/debian/config/opt/cloud/bin/serve_password.sh
+++ b/patches/systemvm/debian/config/opt/cloud/bin/serve_password.sh
@@ -62,7 +62,7 @@ do
break
fi
 
-   request=$(echo $input | grep "DomU_Request:" | cut -d: -f2 | sed 's/^[ 
\t]*//')
+   request=$(echo "$input" | grep "DomU_Request:" | cut -d: -f2 | sed 
's/^[ \t]*//')
 
if [ "$request" != "" ]
then

- Chiradeep Vittal


On Aug. 12, 2013, 1:24 p.m., Jayapal Reddy wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13496/
> ---
> 
> (Updated Aug. 12, 2013, 1:24 p.m.)
> 
> 
> Review request for cloudstack, anthony xu, Abhinandan Prateek, Chiradeep 
> Vittal, and Sheng Yang.
> 
> 
> Bugs: CLOUDSTACK-4184
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> For parallel vm deployment guest vm password script is fixed by retrying 
> after random sleep on failure.
> 
> Please review the changes and provide your comments so that I can commit this.
> 
> For windows guest VM script, changes for parallel vm deployment is not added.
> 
> 
> Diffs
> -
> 
>   setup/bindir/cloud-set-guest-password.in 3215894 
> 
> Diff: https://reviews.apache.org/r/13496/diff/
> 
> 
> Testing
> ---
> 
> Tested by deploying 30 vms. All vms set its password successfully.
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>



Re: more review feedback for 12849?

2013-08-12 Thread Chiradeep Vittal
I kind of lost the plot (perhaps because I was missing for a few weeks). I
didn't get why we need the change (or perhaps it is reverting an older
change?)

On 8/12/13 5:47 AM, "Daan Hoogland"  wrote:

>Is there more to say about
>https://reviews.apache.org/r/12849/
>?
>
>Is it commit worthy? or if not what is missing?



[ACS 42] Closing resolved defects

2013-08-12 Thread Sudha Ponnaganti
Hi All,

There are still 200+ critical and blocker defects pending to be validated. Even 
if a small %age of these gets rejected, that would cause quality issues for 
release. Pl make sure to close these ASAP. First column shows Total defects and 
second column shows Blocker/Critical. You can prioritize blocker and critical 
first

Thanks
/sudha

Reporter  Total  Blocker/Critical
Abhinandan Prateek  2  2
Abhinav Roy   5  5
Ahmad Emneina   5  5
Alena Prokharchyk  21   5
angeline shen1  1
Anthony Xu4  1
Bharat Kumar8  6
Brian Angus1  1
Brian Spindler1  1
Chandan Purushothama   11   8
Chip Childers  1  1
Chiradeep Vittal   1  0
Daan Hoogland 2  1
Dave Brosius  1  0
Dave Cahill  1  0
David Nalley   1  0
David Noland 2  0
david vz1  1
Devdeep Singh 3  3
Diogo Monteiro1  0
edison su 1  1
Fang Wang  3  3
Francois Gaudreault   4  2
frank zhang1  0
Girish Shilamkar2  1
Harikrishna Patnala 7  5
Hiroaki Kawai 2  1
Hongtu Zang  2  0
Hugo Trippaers 1  0
Ian Duffy 1  0
ilya musayev  3  2
Isaac Chiang   4  0
Jarosław Dzbęk 1  0
Jayapal Reddy   8  5
Jeronimo 1  0
Jessica Tomechak3  0
Jessica Wang  2  0
John Burwell  2  2
Kelven Yang   2  2
Kiran Koneti   2  2
Kishan Kavala 8  2
Koushik Das4  2
Likitha Shetty 5  3
manasaveloori  2  1
Mice Xia   1  0
Michael Dickey  1  1
Milamber2  0
Minying Bao   2  2
Murali Reddy 9  3
Naoki Sakamoto   1  1
Nick Wales  2  2
Nicolas Lamirault  1  1
Nik Martin   1  0
Nitin Mehta28   6
Parth Jagirdar7  0
Paul Angus  3  1
Pavan Kumar Bandarupally  2  2
Prachi Damle  14   5
Pranav Saxena  1  0
Prasanna Santhanam 1  1
prashant kumar mishra 2  2
Radhika Nair   2  1
Rajesh Battala   4  4
Ram Ganesh  1  0
Rayees Namathponnan19   6
Rohit Yadav2  1
roxanne chang  1  0
sadhu suresh 3  2
Sailaja Mada   18   16
Saksham Srivastava 9  4
Sangeetha Hariharan  35   18
Sanjay Tripathi  7  3
Sanjeev N   16   16
Sateesh Chodapuneedi 21   15
sebastien goasguen1  0
Shane Witbeck  1  0
Sharad Yadav 1  1
Sheng Yang 12   5
shweta agarwal5  4
Simon Weller 1  1
Sonny Chhen 1  0
Sowmya Krishnan10   4
Srikanteswararao Talluri12   12
Toshiaki Hatano2  0
Venkata Siva Vijayendra Bhamidipati  5  1
venkata swamybabu budumuru   6  6
Wido den Hollander4  0
Grand Total421 222


Quick ISO-related question

2013-08-12 Thread Mike Tutkowski
Hi,

When you spin up a VM based on an ISO, you have to specify the hypervisor.

Just curious, but why does the user pick this?

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


RE: HyperV support in 4.2

2013-08-12 Thread Sudha Ponnaganti

Alena/Murali,

Can one of you review and cleanup Hyper-V specific entries??

Thanks
/sudha

-Original Message-
From: Donal Lafferty 
Sent: Sunday, August 11, 2013 3:40 PM
To: Animesh Chaturvedi; dev@cloudstack.apache.org; Chip Childers
Cc: Sudha Ponnaganti; Murali Reddy
Subject: RE: HyperV support in 4.2
Importance: High

Hi Alena,

I had a closer look at this.

As before, Hyper-V is not available on 4.2.  It should be in place for 4.3. 

However, I noticed that the patch applied in June 27th that introduces 
"HypervisorType.Hyperv" to the list of configured hypervisors. See 
https://reviews.apache.org/r/12106/, especially 
server/src/com/cloud/configuration/Config.java@205

Removing this item should solve the problem of Hyper-V showing up where the 
hypervisor list is used to populate a dropdown box.  

I don't have committer status, so I'm not able to take care of this directly.  
:(

As before, for any other the settings, you could talk to Murali Reddy.  
However, he added them quite some time ago.  A look through the config may be 
the best approach.

DL


> -Original Message-
> From: Animesh Chaturvedi
> Sent: 09 August 2013 22:18
> To: dev@cloudstack.apache.org; Chip Childers
> Cc: Donal Lafferty; Sudha Ponnaganti
> Subject: RE: HyperV support in 4.2
> 
> Donal if it is not for 4.2 then can you move this ticket
> https://issues.apache.org/jira/browse/CLOUDSTACK-999 out of 4.2. 
> Copying Sudha as well
> 
> > -Original Message-
> > From: Animesh Chaturvedi
> > Sent: Friday, August 09, 2013 2:16 PM
> > To: dev@cloudstack.apache.org; Chip Childers
> > Cc: Donal Lafferty
> > Subject: RE: HyperV support in 4.2
> >
> >
> >
> > I see Donal's patch is pending in review board 
> > https://reviews.apache.org/r/12519/ but the branch mentioned is master.
> > Donal can you confirm if this is for 4.2 as well.
> >
> > Alena here is another thread that has more context
> > http://markmail.org/thread/asg62idkvrnn6ba7
> >
> >
> > > -Original Message-
> > > From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
> > > Sent: Friday, August 09, 2013 2:02 PM
> > > To: Chip Childers; dev@cloudstack.apache.org
> > > Subject: HyperV support in 4.2
> > >
> > > Hi Chip,
> > >
> > > Are we planning to officially support HyperV in 4.2, or is it 
> > > pushed to the next release? I'm seeing HyperV listed in the values 
> > > for "hypervisor.list" Global config (the list of hypervisors CS
> > > supports)
> > > - meaning that the code is in - but the bug below claims that 
> > > there
> > is
> > > no HyperV in 4.2:
> > >
> > > https://issues.apache.org/jira/browse/CLOUDSTACK-3893
> > >
> > > Thank you,
> > > Alena.


Re: Quick ISO-related question

2013-08-12 Thread Marcus Sorensen
One reason is that unlike the hypervisor specific templates, there's no
such information predetermined. Consider a cloudstack zone with both xen
and KVM clusters. User needs to choose which hypervisor to deploy in. Its
also important since the resulting install from iso will be in the chosen
hypervisors format.
On Aug 12, 2013 8:29 PM, "Mike Tutkowski" 
wrote:

> Hi,
>
> When you spin up a VM based on an ISO, you have to specify the hypervisor.
>
> Just curious, but why does the user pick this?
>
> Thanks!
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud
> *™*
>


RE: HyperV support in 4.2

2013-08-12 Thread Rajesh Battala
Sudha, 

Devdeep had removed the HyperV config  yesterday.

Thanks
Rajesh Battala

-Original Message-
From: Sudha Ponnaganti [mailto:sudha.ponnaga...@citrix.com] 
Sent: Tuesday, August 13, 2013 8:13 AM
To: Donal Lafferty; Animesh Chaturvedi; dev@cloudstack.apache.org; Chip Childers
Cc: Murali Reddy
Subject: RE: HyperV support in 4.2


Alena/Murali,

Can one of you review and cleanup Hyper-V specific entries??

Thanks
/sudha

-Original Message-
From: Donal Lafferty
Sent: Sunday, August 11, 2013 3:40 PM
To: Animesh Chaturvedi; dev@cloudstack.apache.org; Chip Childers
Cc: Sudha Ponnaganti; Murali Reddy
Subject: RE: HyperV support in 4.2
Importance: High

Hi Alena,

I had a closer look at this.

As before, Hyper-V is not available on 4.2.  It should be in place for 4.3. 

However, I noticed that the patch applied in June 27th that introduces 
"HypervisorType.Hyperv" to the list of configured hypervisors. See 
https://reviews.apache.org/r/12106/, especially 
server/src/com/cloud/configuration/Config.java@205

Removing this item should solve the problem of Hyper-V showing up where the 
hypervisor list is used to populate a dropdown box.  

I don't have committer status, so I'm not able to take care of this directly.  
:(

As before, for any other the settings, you could talk to Murali Reddy.  
However, he added them quite some time ago.  A look through the config may be 
the best approach.

DL


> -Original Message-
> From: Animesh Chaturvedi
> Sent: 09 August 2013 22:18
> To: dev@cloudstack.apache.org; Chip Childers
> Cc: Donal Lafferty; Sudha Ponnaganti
> Subject: RE: HyperV support in 4.2
> 
> Donal if it is not for 4.2 then can you move this ticket
> https://issues.apache.org/jira/browse/CLOUDSTACK-999 out of 4.2. 
> Copying Sudha as well
> 
> > -Original Message-
> > From: Animesh Chaturvedi
> > Sent: Friday, August 09, 2013 2:16 PM
> > To: dev@cloudstack.apache.org; Chip Childers
> > Cc: Donal Lafferty
> > Subject: RE: HyperV support in 4.2
> >
> >
> >
> > I see Donal's patch is pending in review board 
> > https://reviews.apache.org/r/12519/ but the branch mentioned is master.
> > Donal can you confirm if this is for 4.2 as well.
> >
> > Alena here is another thread that has more context
> > http://markmail.org/thread/asg62idkvrnn6ba7
> >
> >
> > > -Original Message-
> > > From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
> > > Sent: Friday, August 09, 2013 2:02 PM
> > > To: Chip Childers; dev@cloudstack.apache.org
> > > Subject: HyperV support in 4.2
> > >
> > > Hi Chip,
> > >
> > > Are we planning to officially support HyperV in 4.2, or is it 
> > > pushed to the next release? I'm seeing HyperV listed in the values 
> > > for "hypervisor.list" Global config (the list of hypervisors CS
> > > supports)
> > > - meaning that the code is in - but the bug below claims that 
> > > there
> > is
> > > no HyperV in 4.2:
> > >
> > > https://issues.apache.org/jira/browse/CLOUDSTACK-3893
> > >
> > > Thank you,
> > > Alena.


Re: Quick ISO-related question

2013-08-12 Thread Mike Tutkowski
Yeah, I guess I thought the end user wasn't generally concerned about
hypervisors...that CloudStack typically handles that, but maybe CloudStack
is more responsible for handling which cluster of the same hypervisor type
to place a new VM on.


On Mon, Aug 12, 2013 at 8:58 PM, Marcus Sorensen wrote:

> One reason is that unlike the hypervisor specific templates, there's no
> such information predetermined. Consider a cloudstack zone with both xen
> and KVM clusters. User needs to choose which hypervisor to deploy in. Its
> also important since the resulting install from iso will be in the chosen
> hypervisors format.
> On Aug 12, 2013 8:29 PM, "Mike Tutkowski" 
> wrote:
>
> > Hi,
> >
> > When you spin up a VM based on an ISO, you have to specify the
> hypervisor.
> >
> > Just curious, but why does the user pick this?
> >
> > Thanks!
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the
> > cloud
> > *™*
> >
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: git commit: updated refs/heads/master to d1b4a8d

2013-08-12 Thread David Nalley
So this fixes the specific problem referred to, but it also leaves
some crud around.
Historically (2.2.x and 3.0.x) -daemonize was required by
-console-proxy -usage, and -agent. At a minimum this likely means that
you'll need to add a similar obsoletes to -usage, or folks with -usage
installed will at least have crud left around, if not installation
failures.

--David

On Mon, Aug 12, 2013 at 5:42 PM,   wrote:
> Updated Branches:
>   refs/heads/master f081092b8 -> d1b4a8d98
>
>
> CLOUDSTACK-4149
> [upgrade][2.2.13 -> 2.2.14 -> 4.2][KVM] When we try to upgrade the KVM agent 
> from 2.2.14 to 4.2 using the "U" option in install.sh script, management 
> server also gets installed!
>
>
> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
> Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/d1b4a8d9
> Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/d1b4a8d9
> Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/d1b4a8d9
>
> Branch: refs/heads/master
> Commit: d1b4a8d98ca9b8fc52f9a99481aecee3fded0862
> Parents: f081092
> Author: frank 
> Authored: Mon Aug 12 14:49:15 2013 -0700
> Committer: frank 
> Committed: Mon Aug 12 14:52:28 2013 -0700
>
> --
>  packaging/centos63/cloud.spec | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> --
>
>
> http://git-wip-us.apache.org/repos/asf/cloudstack/blob/d1b4a8d9/packaging/centos63/cloud.spec
> --
> diff --git a/packaging/centos63/cloud.spec b/packaging/centos63/cloud.spec
> index 0586c61..c2102bf 100644
> --- a/packaging/centos63/cloud.spec
> +++ b/packaging/centos63/cloud.spec
> @@ -104,7 +104,6 @@ Obsoletes: cloud-deps < 4.1.0
>  Obsoletes: cloud-python < 4.1.0
>  Obsoletes: cloud-setup < 4.1.0
>  Obsoletes: cloud-cli < 4.1.0
> -Obsoletes: cloud-daemonize < 4.1.0
>  Group:   System Environment/Libraries
>  %description common
>  The Apache CloudStack files shared between agent and management server
> @@ -130,6 +129,7 @@ Provides: cloud-agent
>  Obsoletes: cloud-agent < 4.1.0
>  Obsoletes: cloud-agent-libs < 4.1.0
>  Obsoletes: cloud-test < 4.1.0
> +Obsoletes: cloud-daemonize < 4.1.0
>  Group: System Environment/Libraries
>  %description agent
>  The CloudStack agent for KVM hypervisors
>


RE: [ACS42] Listtemplates API respone without any Zone configuration

2013-08-12 Thread Sailaja Mada
Thanks Min.  Jira ticket @ 
https://issues.apache.org/jira/browse/CLOUDSTACK-4277 

Regards,
Sailaja.M

-Original Message-
From: Min Chen [mailto:min.c...@citrix.com] 
Sent: Monday, August 12, 2013 10:16 PM
To: dev@cloudstack.apache.org
Subject: Re: [ACS42] Listtemplates API respone without any Zone configuration

This seems introduced by storage refactoring where we have gotten rid of 
previous direct sql to use db view to search for templates. Please file a bug 
for this.

Thanks
-min

On 8/12/13 1:16 AM, "Sailaja Mada"  wrote:

>Hi,
>
>I have the setup with latest code.  When I tried to view the list if 
>templates without configuring zone , it is listing all the system vm 
>templates and default cent OS templates .
>
>I have not observed this behavior before. Is it introduced 
>intentionally or is It a issue with API response ?
>
>Please confirm.
>
>
>2013-08-12 13:18:53,916 INFO  [cloud.api.ApiServer]
>(catalina-exec-3:null) (userId=2 accountId=2
>sessionId=2C596448B0E7D3291B7FA76334A085A7) 10.144.7.12 -- GET 
>command=listTemplates&response=json&sessionkey=wMUCQaSUAhWg4JmNNXrmhPGI
>Ozc
>%3D&listAll=true&page=1&pagesize=20&templatefilter=all&_=1376293976571
>200 { "listtemplatesresponse" : { "count":8 ,"template" : [ 
>{"id":"4b58341c-031a-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
>(LXC)","displaytext":"SystemVM Template 
>(LXC)","ispublic":false,"isready":false,"passwordenabled":false,"format":"
>QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-1
>1e3 -b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0 
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"LXC"
>,"d 
>omain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextr
>act 
>able":false,"checksum":"2755de1f9ef2ce4d6f2bee2efbb4da92","tags":[],"ss
>hke yenabled":false,"isdynamicallyscalable":false},
>{"id":"ec8e64b6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
>(XenServer)","displaytext":"SystemVM Template 
>(XenServer)","ispublic":false,"isready":false,"passwordenabled":false,"
>for
>mat":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ed81a824-03
>18- 11e3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 
>7(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"Xen
>Ser 
>ver","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2",
>"is 
>extractable":false,"checksum":"74b92f031cc5c2089ee89efb81344dcf","tags"
>:[] ,"sshkeyenabled":false,"isdynamicallyscalable":false},
>{"id":"ec98be84-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
>(HyperV)","displaytext":"SystemVM Template 
>(HyperV)","ispublic":false,"isready":false,"passwordenabled":false,"for
>mat 
>":"VHD","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-
>11e 3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0 
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"Hyperv"
>,"domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","ise
>xtr 
>actable":false,"checksum":"fb1b6e032a160d86f2c28feb5add6d83","tags":[],
>"ss hkeyenabled":false,"isdynamicallyscalable":false},
>{"id":"ec931e0c-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
>(KVM)","displaytext":"SystemVM Template 
>(KVM)","ispublic":false,"isready":false,"passwordenabled":false,"format":"
>QCOW2","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318-1
>1e3 -b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0 
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"KVM"
>,"d 
>omain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","isextr
>act 
>able":false,"checksum":"6cea42b2633841648040becb588bd8f0","tags":[],"ss
>hke yenabled":false,"isdynamicallyscalable":false},
>{"id":"ec95eaf6-0318-11e3-b7b2-5641688b41e2","name":"SystemVM Template 
>(vSphere)","displaytext":"SystemVM Template 
>(vSphere)","ispublic":false,"isready":false,"passwordenabled":false,"fo
>rma
>t":"OVA","isfeatured":false,"crossZones":true,"ostypeid":"ecbdeaba-0318
>-11 e3-b7b2-5641688b41e2","ostypename":"Debian GNU/Linux 5.0 
>(32-bit)","account":"system","templatetype":"SYSTEM","hypervisor":"VMware"
>,"domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2","ise
>xtr 
>actable":false,"checksum":"8fde62b1089e5844a9cd3b9b953f9596","tags":[],
>"ss hkeyenabled":false,"isdynamicallyscalable":true},
>{"id":"ec947a86-0318-11e3-b7b2-5641688b41e2","name":"CentOS 5.6(64-bit) 
>no GUI (XenServer)","displaytext":"CentOS 5.6(64-bit) no GUI 
>(XenServer)","ispublic":true,"isready":false,"passwordenabled":false,"f
>orm
>at":"VHD","isfeatured":true,"crossZones":true,"ostypeid":"ecbc066e-0318
>-11 e3-b7b2-5641688b41e2","ostypename":"CentOS 5.3 
>(64-bit)","account":"system","templatetype":"BUILTIN","hypervisor":"Xen
>Ser 
>ver","domain":"ROOT","domainid":"ec8ad800-0318-11e3-b7b2-5641688b41e2",
>"is 
>extractable":true,"checksum":"905cec879afd9c9d22ecc8036131a180","tags":
>[], "sshkeyenabled":false,"isdynamicallyscalable":true},
>{"id":"ec95326e-0318-11e3-b7b2-5641688b41e2

Review Request 13523: 7 New tests for affinity groups

2013-08-12 Thread Ashutosh Kelkar

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

Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam.


Repository: cloudstack-git


Description
---

New tests for affinity groups


Diffs
-

  test/integration/component/test_affinity_groups.py 39baf3e 

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


Testing
---


Thanks,

Ashutosh Kelkar



Question: Has VolumeManager and VolumeManagerImpl been refactored/removed in master

2013-08-12 Thread Mike Tutkowski
Hi,

I was curious to learn more about VolumeManager and VolumeManagerImpl in
master.

They appear to be gone and I was interested in learning how this code has
been refactored.

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


  1   2   >