date:20130329

On 30 March 2013 09:20, Justin Grudzien  wrote:
>
> My company is building a private cloud and we are moving to cloudstack. As
> we begun investigating the cloudmonkey CLI we found that the output was
> slightly hard to read. I have begun working on some optimizations that I
> think will benefit the community and I reached out to Rohit, who
> recommended that I join this list and present my ideas. Here is what I am
> proposing:
>
> 1. Add json output to cloudmonkey
> I have accomplished this by adding a config parameter called display, which
> can be set to json, tabularize, or default. I have removed the tabularize
> parameter.
>
> Justins-MacBook-Pro:cloudmonkey grudzien$ cloudmonkey list users
> account=grudzien
> {
>   "count": 1,
>   "user": [
> {
>   "account": "grudzien",
>   "accountid": "b799783d-e5bb-460a-be0e-3966bd69edda",
>   "accounttype": 1,
>   "apikey": "*nokey*",
>   "created": "2013-03-27T16:09:17-0500",
>   "domain": "ROOT",
>   "domainid": "7e61c32f-9873-4944-947a-dcc00d3bebdc",
>   "email": "grudz...@gmail.com",
>   "firstname": "Justin",
>   "id": "265930bc-62ef-41f8-849c-e58593ca4b1f",
>   "lastname": "Grudzien",
>   "secretkey": "*nokey*",
>   "state": "enabled",
>   "username": "grudzien"
> }
>   ]
> }
>
> 2. Add filtering as a standard parameter for all output types.
> The only thing that has filtering now is the tabular output and grep breaks
> the json.
>
> Justins-MacBook-Pro:cloudmonkey grudzien$ cloudmonkey list users
> account=grudzien filter=account,email,username,state
> {
>   "count": 1,
>   "user": [
> {
>   "account": "grudzien",
>   "email": "grudz...@gmail.com",
>   "state": "enabled",
>   "username": "grudzien"
> }
>   ]
> }
>

+1 - JSON output would make it very good to consume into other api
response processing. Right now cloudmonkey actually isn't aware of the
response object model. For certain API responses that return embedded
objects (eg: listNetworks [1]) it would be good to return the JSON
appropriately formatted as well.

[1] https://gist.github.com/vogxn/5255883


> 3. Add color to the json output
> I was thinking of colorizing the keys in the key/value pairs to increase
> readability.
>
> 4. Move the color option from the config file to the command line.
> There are two reasons for this. First, I want to be able to wrap a script
> around cloudmonkey and not have to worry about colorization that will
> impede me processing the output and second I think it would be more useful
> to use the highlighting on demand rather than having to back out of the
> shell to edit a config file.
>
> 5. Standardize messaging for the output types.
> Right now certain kinds of messaging is presented differently for an output
> type. For example, if I issue an api command that doesn't exist it displays
> a generic error message, regardless of the output type selected. Ideally,
> all output would be in the specified format.
>
> I have the first two working and am planning on implementing the others as
> I flesh them out. I will submit a patch when I feel it is ready. Any early
> feedback on whether these changes will be useful to others is appreciated.

Patches most welcome!




--
Prasanna.,

RE: Scalability Limitations

2013-03-29 Thread Edison Su

No limitations at the mgt server side.

Sent from my Windows Phone

From: Mike Tutkowski
Sent: ‎3/‎29/‎2013 9:56 AM
To: 
cloudstack-...@incubator.apache.org
Subject: Scalability Limitations

Hi,

Can someone answer these questions for me?

Is there a max number of clusters CS supports?  If so, what is that number?

Is there a max number of primary storages CS supports?  If so, what is that
number?

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
*™*

cloudmonkey printing enhancements proposal

2013-03-29 Thread Justin Grudzien

My company is building a private cloud and we are moving to cloudstack. As
we begun investigating the cloudmonkey CLI we found that the output was
slightly hard to read. I have begun working on some optimizations that I
think will benefit the community and I reached out to Rohit, who
recommended that I join this list and present my ideas. Here is what I am
proposing:

1. Add json output to cloudmonkey
I have accomplished this by adding a config parameter called display, which
can be set to json, tabularize, or default. I have removed the tabularize
parameter.

Justins-MacBook-Pro:cloudmonkey grudzien$ cloudmonkey list users
account=grudzien
{
  "count": 1,
  "user": [
{
  "account": "grudzien",
  "accountid": "b799783d-e5bb-460a-be0e-3966bd69edda",
  "accounttype": 1,
  "apikey": "*nokey*",
  "created": "2013-03-27T16:09:17-0500",
  "domain": "ROOT",
  "domainid": "7e61c32f-9873-4944-947a-dcc00d3bebdc",
  "email": "grudz...@gmail.com",
  "firstname": "Justin",
  "id": "265930bc-62ef-41f8-849c-e58593ca4b1f",
  "lastname": "Grudzien",
  "secretkey": "*nokey*",
  "state": "enabled",
  "username": "grudzien"
}
  ]
}

2. Add filtering as a standard parameter for all output types.
The only thing that has filtering now is the tabular output and grep breaks
the json.

Justins-MacBook-Pro:cloudmonkey grudzien$ cloudmonkey list users
account=grudzien filter=account,email,username,state
{
  "count": 1,
  "user": [
{
  "account": "grudzien",
  "email": "grudz...@gmail.com",
  "state": "enabled",
  "username": "grudzien"
}
  ]
}

3. Add color to the json output
I was thinking of colorizing the keys in the key/value pairs to increase
readability.

4. Move the color option from the config file to the command line.
There are two reasons for this. First, I want to be able to wrap a script
around cloudmonkey and not have to worry about colorization that will
impede me processing the output and second I think it would be more useful
to use the highlighting on demand rather than having to back out of the
shell to edit a config file.

5. Standardize messaging for the output types.
Right now certain kinds of messaging is presented differently for an output
type. For example, if I issue an api command that doesn't exist it displays
a generic error message, regardless of the output type selected. Ideally,
all output would be in the specified format.

I have the first two working and am planning on implementing the others as
I flesh them out. I will submit a patch when I feel it is ready. Any early
feedback on whether these changes will be useful to others is appreciated.

Justin

Apache BarCamp Today at PESIT Bangalore

2013-03-29 Thread Radhika Puthiyetath

Gentle Reminder..
-- Forwarded message --
From: Nick Burch mailto:n...@apache.org>>
Date: Wed, Mar 27, 2013 at 11:21 AM
Subject: Upcoming BarCamps
To: committ...@apache.org

Hi All

Firstly, a couple of upcoming BarCamps for your diary:

Saturday 30th March (this Saturday!) - Bangalore
 Our first event in India! A one day Apache BarCamp and Hackathon
 http://www.meetup.com/CloudStack-Bangalore-Group/events/104410272/

Saturday 19th May Boston
 Planning is underway, more volunteers needed to help!
 http://wiki.apache.org/concom-planning/BarCampBoston2013

There are also currently plans for another Washington DC event, and a New York 
event, both for the summer/autumn, more details expected in a few months.

We've also had interest in events in Lyon in France, Bucharest in Romania, 
Ljublana in Slovenia, Sydney in Australia, and LA. All of those need more 
volunteers before they can go ahead, so if you live/work in/near those places, 
pipe up on the small events list[1] so we can put you in touch with the 
potential organisers!

Finally, if nothing near you is listed above, if you can rope in some other 
volunteers to help with an event, we can provide instructions / advice / 
mentoring / funding to help you make an event happen :) Just get in touch on 
small-events-discuss@ [1] and we'll help you get started!

Nick

[1] http://mail-archives.apache.org/mod_mbox/www-small-events-discuss/

--
Genius is 99% Hardwork and 1% Luck

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-29 Thread Justin Grudzien

Understandable that SHA256 is good for today but SHA512 is good for tomorrow 
and has excellent support in all major languages, products, etc. Oracle is the 
only thing I can think of that doesn't support it out of the box.

I agree on balance but I can certainly spare the 32 bytes of storage. :)

Justin

On Mar 29, 2013, at 7:23 PM, John Kinsella  wrote:

> Never too late. :)
> 
> Looking at [1], it'll take about 2E64 days to search the SHA256 keyspace with 
> a brute forcer written for a GPU accelerator. Even with 10,000 of them, it'll 
> take way, way too long. Bitcoin coin uses SHA256, so I think it's safe for a 
> while. :)
> 
> Yes, SHA512 would be "more safe," but ya gotta ask what you're really getting 
> by going to the bigger key…security is about a balance. If you're using ACS 
> to run a cloud cracking SHA256 passwords, then maybe you should use SHA512 
> passwords in ACS. ;)
> 
> John
> http://www.insidepro.com/eng/egb.shtml
> 
> On Mar 29, 2013, at 4:36 PM, Justin Grudzien 
> mailto:grudz...@gmail.com>>
> wrote:
> 
> I apologize for jumping into this conversation late, but I am new to the 
> developer mailing list. Why would we choose SHA256+salt over SHA512+salt? 
> SHA512+salt's storage is insignificant when compared to SHA256 and the 
> chances of a birthday attack are significantly reduced. As a security 
> professional I would argue for the best possible hashing algorithm available, 
> especially when there is little to no cost.
> 
> Justin
> 
> On Mar 29, 2013, at 6:00 PM, "Min Chen" 
> mailto:min.c...@citrix.com>> wrote:
> 
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10039/#review18548
> ---
> 
> 
> 
> client/tomcatconf/componentContext.xml.in
> 
> 
>  This componentContext.xml.in and nonossComponentContext.xml.in should be put 
> into applicationContext.xml since they are the same for nonoss and oss 
> versions.
> 
> 
> - Min Chen
> 
> 
> On March 28, 2013, 8:26 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10039/
> ---
> 
> (Updated March 28, 2013, 8:26 p.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers, Kelven Yang, and Min Chen.
> 
> 
> Description
> ---
> 
> Changing default password encoding mechanism from MD5 to SHA256Salted.
> 
> 
> This addresses bug CS-1734.
> 
> 
> Diffs
> -
> 
> api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 
> 89673ea
> api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 
> fb29e1a
> api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java 
> 1f31662
> client/tomcatconf/componentContext.xml.in 016df0a
> client/tomcatconf/nonossComponentContext.xml.in 8f8dae5
> developer/developer-prefill.sql 6300d35
> plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
>  61eebe5
> plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java
>  026125e
> plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java
>  52e7cb3
> plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java
>  1b29f69
> server/src/com/cloud/server/ManagementServerImpl.java b689f93
> server/src/com/cloud/user/AccountManagerImpl.java b69f314
> 
> Diff: https://reviews.apache.org/r/10039/diff/
> 
> 
> Testing
> ---
> 
> Manual testing done for both oss and nonoss components. Both admin and users 
> added later are encoded according to the scheme configured, and authenticated 
> by the same scheme.
> 
> To change the order of the schemes, modify the following list properties in 
> client/tomcatconf/nonossComponentContext.xml.in or 
> client/tomcatconf/componentContext.xml.in as applicable, to the desired order:
> 
>  
>   
>  
>  
>  
>  
>  
>  
> 
>  
>  
>  
>   
>   
>  
>   
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
> 
> Stratosec - Secure Infrastructure as a Service
> o: 415.315.9385
> @johnlkinsella
>

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-29 Thread John Kinsella

Never too late. :)

Looking at [1], it'll take about 2E64 days to search the SHA256 keyspace with a 
brute forcer written for a GPU accelerator. Even with 10,000 of them, it'll 
take way, way too long. Bitcoin coin uses SHA256, so I think it's safe for a 
while. :)

Yes, SHA512 would be "more safe," but ya gotta ask what you're really getting 
by going to the bigger key…security is about a balance. If you're using ACS to 
run a cloud cracking SHA256 passwords, then maybe you should use SHA512 
passwords in ACS. ;)

John
http://www.insidepro.com/eng/egb.shtml

On Mar 29, 2013, at 4:36 PM, Justin Grudzien 
mailto:grudz...@gmail.com>>
 wrote:

I apologize for jumping into this conversation late, but I am new to the 
developer mailing list. Why would we choose SHA256+salt over SHA512+salt? 
SHA512+salt's storage is insignificant when compared to SHA256 and the chances 
of a birthday attack are significantly reduced. As a security professional I 
would argue for the best possible hashing algorithm available, especially when 
there is little to no cost.

Justin

On Mar 29, 2013, at 6:00 PM, "Min Chen" 
mailto:min.c...@citrix.com>> wrote:


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



client/tomcatconf/componentContext.xml.in


  This componentContext.xml.in and nonossComponentContext.xml.in should be put 
into applicationContext.xml since they are the same for nonoss and oss versions.


- Min Chen


On March 28, 2013, 8:26 p.m., Venkata Siva Vijayendra Bhamidipati wrote:

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

(Updated March 28, 2013, 8:26 p.m.)


Review request for cloudstack, Hugo Trippaers, Kelven Yang, and Min Chen.


Description
---

Changing default password encoding mechanism from MD5 to SHA256Salted.


This addresses bug CS-1734.


Diffs
-

api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 
89673ea
api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java fb29e1a
api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java 1f31662
client/tomcatconf/componentContext.xml.in 016df0a
client/tomcatconf/nonossComponentContext.xml.in 8f8dae5
developer/developer-prefill.sql 6300d35
plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
 61eebe5
plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java
 026125e
plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java
 52e7cb3
plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java
 1b29f69
server/src/com/cloud/server/ManagementServerImpl.java b689f93
server/src/com/cloud/user/AccountManagerImpl.java b69f314

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


Testing
---

Manual testing done for both oss and nonoss components. Both admin and users 
added later are encoded according to the scheme configured, and authenticated 
by the same scheme.

To change the order of the schemes, modify the following list properties in 
client/tomcatconf/nonossComponentContext.xml.in or 
client/tomcatconf/componentContext.xml.in as applicable, to the desired order:

  
   
  
  
  
  
  
  

  
  
  
   
   
  
   


Thanks,

Venkata Siva Vijayendra Bhamidipati


Stratosec - Secure Infrastructure as a Service
o: 415.315.9385
@johnlkinsella

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-29 Thread Justin Grudzien

I apologize for jumping into this conversation late, but I am new to the 
developer mailing list. Why would we choose SHA256+salt over SHA512+salt? 
SHA512+salt's storage is insignificant when compared to SHA256 and the chances 
of a birthday attack are significantly reduced. As a security professional I 
would argue for the best possible hashing algorithm available, especially when 
there is little to no cost.

Justin

On Mar 29, 2013, at 6:00 PM, "Min Chen"  wrote:

> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10039/#review18548
> ---
> 
> 
> 
> client/tomcatconf/componentContext.xml.in
> 
> 
>This componentContext.xml.in and nonossComponentContext.xml.in should be 
> put into applicationContext.xml since they are the same for nonoss and oss 
> versions.
> 
> 
> - Min Chen
> 
> 
> On March 28, 2013, 8:26 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
>> 
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/10039/
>> ---
>> 
>> (Updated March 28, 2013, 8:26 p.m.)
>> 
>> 
>> Review request for cloudstack, Hugo Trippaers, Kelven Yang, and Min Chen.
>> 
>> 
>> Description
>> ---
>> 
>> Changing default password encoding mechanism from MD5 to SHA256Salted.
>> 
>> 
>> This addresses bug CS-1734.
>> 
>> 
>> Diffs
>> -
>> 
>>  
>> api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java
>>  89673ea 
>>  api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 
>> fb29e1a 
>>  api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java 
>> 1f31662 
>>  client/tomcatconf/componentContext.xml.in 016df0a 
>>  client/tomcatconf/nonossComponentContext.xml.in 8f8dae5 
>>  developer/developer-prefill.sql 6300d35 
>>  
>> plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
>>  61eebe5 
>>  
>> plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java
>>  026125e 
>>  
>> plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java
>>  52e7cb3 
>>  
>> plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java
>>  1b29f69 
>>  server/src/com/cloud/server/ManagementServerImpl.java b689f93 
>>  server/src/com/cloud/user/AccountManagerImpl.java b69f314 
>> 
>> Diff: https://reviews.apache.org/r/10039/diff/
>> 
>> 
>> Testing
>> ---
>> 
>> Manual testing done for both oss and nonoss components. Both admin and users 
>> added later are encoded according to the scheme configured, and 
>> authenticated by the same scheme.
>> 
>> To change the order of the schemes, modify the following list properties in 
>> client/tomcatconf/nonossComponentContext.xml.in or 
>> client/tomcatconf/componentContext.xml.in as applicable, to the desired 
>> order:
>> 
>>
>> 
>>
>>
>>
>>
>>
>>
>> 
>>
>>
>>
>> 
>> 
>>
>> 
>> 
>> 
>> Thanks,
>> 
>> Venkata Siva Vijayendra Bhamidipati
>

[DISCUSS] git rebase vs git merge in your feature branch?

2013-03-29 Thread Edison Su

Hi all,
 I am trying to review some feature branches, when I see merge requests 
coming from mailing list, one thing that makes code review almost unrealistic 
is that, developers tend to use "git merge" to master branch whenever rebase is 
needed. I don't know other people really do review feature branch or not, if 
so, how to review a feature branch, with several "merge branch "master""  on 
the feature branch. I really don't find a better way to do that.
If, all we use "git rebase" to master branch, then the code review will be 
much easier, at least, what kind of commits you did on the feature branch can 
be easily identified. For example, I worked on storage_refactor branch for a 
few months, with a lot of changes, before sending out merge request, there are 
only less than 10 commits on the branch, reviewer can use "git diff 
since..HEAD" to get all the changes.
   Should we advocate "git rebase" in 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Git?

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-29 Thread Min Chen


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



client/tomcatconf/componentContext.xml.in


This componentContext.xml.in and nonossComponentContext.xml.in should be 
put into applicationContext.xml since they are the same for nonoss and oss 
versions.


- Min Chen


On March 28, 2013, 8:26 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10039/
> ---
> 
> (Updated March 28, 2013, 8:26 p.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers, Kelven Yang, and Min Chen.
> 
> 
> Description
> ---
> 
> Changing default password encoding mechanism from MD5 to SHA256Salted.
> 
> 
> This addresses bug CS-1734.
> 
> 
> Diffs
> -
> 
>   
> api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 
> 89673ea 
>   api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 
> fb29e1a 
>   api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java 
> 1f31662 
>   client/tomcatconf/componentContext.xml.in 016df0a 
>   client/tomcatconf/nonossComponentContext.xml.in 8f8dae5 
>   developer/developer-prefill.sql 6300d35 
>   
> plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
>  61eebe5 
>   
> plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java
>  026125e 
>   
> plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java
>  52e7cb3 
>   
> plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java
>  1b29f69 
>   server/src/com/cloud/server/ManagementServerImpl.java b689f93 
>   server/src/com/cloud/user/AccountManagerImpl.java b69f314 
> 
> Diff: https://reviews.apache.org/r/10039/diff/
> 
> 
> Testing
> ---
> 
> Manual testing done for both oss and nonoss components. Both admin and users 
> added later are encoded according to the scheme configured, and authenticated 
> by the same scheme.
> 
> To change the order of the schemes, modify the following list properties in 
> client/tomcatconf/nonossComponentContext.xml.in or 
> client/tomcatconf/componentContext.xml.in as applicable, to the desired order:
> 
> 
>  
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
>  
>  
> 
>  
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>

Re: [PROPOSAL][CLOUDSTACK-779] Egress firewall rules for SRX

LGTM.
Only thing I'd be careful of the length of the policy name.
Note that the networkid is long, so you could end up with a policy name
egress-trust-untrust-1234567890123. Perhaps e-t-u-



On 3/28/13 8:38 AM, "Jayapal Reddy Uradi" 
wrote:

>I would like to propose the egress firewall rules feature for the
>external firewall device SRX guest network.
>
>Currently egress firewall rules is supported in the VR isolated network.
>With this feature egress firewall rules
>is supported in the external device SRX guest networks.
>
>Jira Id:
>https://issues.apache.org/jira/browse/CLOUDSTACK-779
>
>FS:
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/Egress+firewall+rul
>es+feature+support+for+external+device+Juniper+SRX
>Please let me know your comments on the above FS.
>
>Thanks,
>Jayapal

Re: [ACS41][PATCH request]CLOUDSTACK-1827: Fix redundant router update mechanism

Sorry for missing it. Will test and push in about an hour.

On Mar 29, 2013, at 6:30 PM, Sheng Yang  wrote:

> Bump...
>
> --Sheng
>
> On Thu, Mar 28, 2013 at 3:55 PM, Sheng Yang  wrote:
>> commit e3f67d126d66c31e027c96b5524667884084053b
>> Author: Sheng Yang 
>> Date:   Thu Mar 28 15:02:45 2013 -0700
>>
>>CLOUDSTACK-1827: Fix redundant router update mechanism
>>
>>There is a bug that if the smaller number ID RvR stop running, that pair 
>> of
>>RvR's status won't be updated. This patch fixed it.
>>
>> --Sheng
>

Re: [PROPOSAL][CLOUDSTACK-1828] NAT on private gateway

Thanks for this. 
It seems a reasonable approach, except that it reflects a failure of
previous modeling of the private gateway feature.
So, today we want to provide source nat,
Tomorrow perhaps static NAT?
The day after PF and LB ?

The difference between the public network and the private gateway (or the
network behind the private gateway) is not much (if any).

On 3/27/13 10:42 PM, "Jayapal Reddy Uradi" 
wrote:

>I would like to propose feature NAT on private gateway.
>This feature is sub feature of nTier2.0 apps.
>
>Using this feature we can enable/disable the source NAT on the vpc
>private gateway .
>
>
>Jira ticket:
>
>https://issues.apache.org/jira/browse/CLOUDSTACK-1828
>
>
>FS :
>
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/NAT+on+private+gate
>way
>
>
>
>Please provide your comments on above FS.
>
>
>
>Thanks,
>
>Jayapal
>

Re: [ACS41][PATCH request]CLOUDSTACK-1827: Fix redundant router update mechanism

2013-03-29 Thread Sheng Yang

Bump...

--Sheng

On Thu, Mar 28, 2013 at 3:55 PM, Sheng Yang  wrote:
> commit e3f67d126d66c31e027c96b5524667884084053b
> Author: Sheng Yang 
> Date:   Thu Mar 28 15:02:45 2013 -0700
>
> CLOUDSTACK-1827: Fix redundant router update mechanism
>
> There is a bug that if the smaller number ID RvR stop running, that pair 
> of
> RvR's status won't be updated. This patch fixed it.
>
> --Sheng

Re: [PROPOSAL][CLOUDSTACK-768] ACL on private gateway

This depends on the changes proposed in the ACL deny rules feature. Can
you re-word taking into account the new proposed APIs?

On 3/28/13 12:46 AM, "Abhinandan Prateek" 
wrote:

>+1, thanks for proposing and working on this feature.
>
>On 28/03/13 11:32 AM, "Jayapal Reddy Uradi"
> wrote:
>
>>I would like to propose the new feature ACL on private gateway.
>>This feature is sub feature of the nTier 2.0 apps.
>>
>>Currently we do not have way to control the traffic on the private
>>gateway.
>>Using this feature we can configure the ingress/egress ACL on the private
>>gateway.
>>
>>
>>Jira Id:
>>https://issues.apache.org/jira/browse/CLOUDSTACK-768
>>
>>FS:
>>https://cwiki.apache.org/confluence/display/CLOUDSTACK/ACLs+on+private+GW
>>
>>Please provide your comments on the above FS.
>>
>>Thanks,
>>Jayapal
>>
>

Re: Question pertaining to the Support of ACL deny rules

I would think that an ACL container is associated with a VPC and not with
multiple VPCs.
You could create an ACL container that makes sense for VPC #1 but not for
VPC #2. If you update the container for VPC #1 you might unwittingly make
a dangerous change in VPC #2.

Also, isn't the term redundant? (Access Control List Container). Should
the original API be aliased to createNetworkAclItem or
createNetworkAclEntry?

I'd like to suggest the following API name changes
* updateNetworkACL -> updateNetworkACLItem

* removeNetworkACL -> deleteNetworkACLItem (not specified but mentioned)
* replaceNetworkACLContainer -> replaceNetworkACLAssociation
* createNetworkACL -> createNetworkACLItem
* createNetworkACLContainer -> createNetworkACLList (yes redundant, but
not introducing new terminology)

Other comments
* I don't see removeNetworkACL (deleteNetworkACLItem) being specified
* createNetwork - I like this idea of being able to specify at creation
time, but it should fail if the ACL service is not present
* createNetworkAclItem - adding new mandatory parameters breaks the old
API which cannot be done in 4.2 (needs 5.0)
* createNetworkAclList - needs VPC id
* deleteNetworkAclList - does this delete all the ACL items contained? Can
you delete the default one?
* listNetworkAclContainers - listAPIs usually have filters as parameters.
You are proposing two filters -- by ACLList Id and network id. I could
easily see filtering by list of network ids, by vpc id, those that contain
a particular ACLItem, etc. At the very least can we rewrite the API that
takes a filter as an input ? How do I know which ACLList is the default
one? 
* How do you list the ACLItems inside an ACLList? Can you filter? List
only ingress?
* vpc_id should be required in all APIs?
* call out the asynchronous APIs vs the synchronous APIs
* Scripts - do you propose deleting and re-creating the entire chain when
you update a rule? Or do you plan to surgically move around the rules as
the ordering changes?
* what are the contents of the default ACLList?
* firewall_rules - should we create a new table instead? The upgrade can
move the rules to the new table

On 3/28/13 3:49 AM, "Kishan Kavala"  wrote:

>Chandan,
>  User can assign any number as rule priority. But the number has to be
>unique within the container. Two ACLs in the same container cannot have
>same priority number.
>e.g.
>ACL1 - number 10
>ACL2 - number 40
>ACL3 - number 30
>
>In the above example, ACL1 will have the highest priority followed by
>ACL3 and ACL2.
>
>Priority number of the deleted rule can be re-used. Priority number can
>be modified using updateNetworkACL API.
>
>Same NetworkACLContainer can be assigned to multiple tiers (belonging to
>two different VPCs also) as long as they belong to same account.
>
>Regards,
>Kishan
>
>
>> -Original Message-
>> From: Chandan Purushothama
>> Sent: Thursday, 28 March 2013 5:19 AM
>> To: dev@cloudstack.apache.org; Kishan Kavala
>> Subject: RE: Question pertaining to the Support of ACL deny rules
>> 
>> Kishan,
>> 
>> Can NetworkACLContainer be used for two network tiers belonging to two
>> different VPCs?
>> 
>> Thank you,
>> Chandan.
>> 
>> -Original Message-
>> From: Chandan Purushothama [mailto:chandan.purushoth...@citrix.com]
>> Sent: Wednesday, March 27, 2013 4:37 PM
>> To: dev@cloudstack.apache.org; Kishan Kavala
>> Subject: RE: Question pertaining to the Support of ACL deny rules
>> 
>> Kishan,
>> 
>> I referred to your FS at
>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Support+ACL+de
>> ny+rules . May I know how does a user choose a rule priority number. Do
>>we
>> allow two rules with the same priority for any reason?  Can the user
>>re-use
>> the priority number of the deleted rule? Can the User shuffle the
>>priorities
>> between rules at any time? Can more information pertaining to the rule
>> number priority be mentioned in your functional specification,
>> 
>> Thank you,
>> Chandan.
>> 
>> -Original Message-
>> From: Chandan Purushothama [mailto:chandan.purushoth...@citrix.com]
>> Sent: Monday, March 04, 2013 10:00 AM
>> To: cloudstack-...@incubator.apache.org
>> Cc: Kishan Kavala
>> Subject: RE: Question pertaining to the Support of ACL deny rules
>> 
>> May I know how can I use this feature in CloudStack. I mean, What do I
>>need
>> to do on CloudStack to specify multiple ACL deny rules?
>> 
>> Thank you,
>> Chandan.
>> 
>> -Original Message-
>> From: Pranav Saxena [mailto:pranav.sax...@citrix.com]
>> Sent: Friday, March 01, 2013 9:35 PM
>> To: cloudstack-...@incubator.apache.org
>> Cc: Kishan Kavala
>> Subject: RE: Question pertaining to the Support of ACL deny rules
>> 
>> This feature is also under development and I believe , Kishan is yet to
>>check
>> in his code . After that we'll be adding the UI support for this and
>>then you 'll
>> be able to do it from the UI itself.
>> 
>> Thanks,
>> Pranav
>> 
>> -Original Message-
>> From: Chandan Purushothama [mailto:chandan

Re: Scalability Limitations

2013-03-29 Thread Manan Shah

Besides the items that Chiradeep mentioned, there would be limits on
Management Server Scalability. This can be easily overcome by deploying
multiple management servers front-ended by a LB. Since all of the
management servers are stateless, this should not cause any problems.

Regards,
Manan Shah




On 3/29/13 12:43 PM, "Mike Tutkowski"  wrote:

>Thanks, Chiradeep
>
>Are you aware of any documentation we have with regards to scalability?  A
>customer asked for such a doc today.
>
>I found this one:
>
>http://www.citrix.co.jp/event/pdf/5-scalability.pdf
>
>
>On Fri, Mar 29, 2013 at 1:39 PM, Chiradeep Vittal <
>chiradeep.vit...@citrix.com> wrote:
>
>> No hard-coded maximums. Probably limited by the performance of the
>> database most likely.
>> I know of deployments which have tens of thousands.
>>
>> On 3/29/13 9:56 AM, "Mike Tutkowski" 
>>wrote:
>>
>> >Hi,
>> >
>> >Can someone answer these questions for me?
>> >
>> >Is there a max number of clusters CS supports?  If so, what is that
>> >number?
>> >
>> >Is there a max number of primary storages CS supports?  If so, what is
>> >that
>> >number?
>> >
>> >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: Unable to start domR(VR) in devcloud using laster 4.1/master.

You should roll back your devcloud VM to the original and try again.

On DevCloud2, I notice that sometimes the xapi0 bridge loses its ip
address and hence loses the 169.254.0.0/16 route.
You can work around this by
ifconfig xapi0 (check for ipv4 address)
ip route show (check for 169.254 route)

If these are missing

ifconfig xapi0 169.254.169.254 netmask 255.255.0.0
ip route show


On 3/29/13 1:06 PM, "Marcus Sorensen"  wrote:

>Xen? KVM?
>
>Generally GetDomRVersionCmd failing is only a symptom, it usually
>means that cloudstack was never able to communicate with the system
>vm. You would need to look at whether the VM actually started on the
>host, and some of the text around when the VM was deployed to get any
>meaningful errors.  Sometimes this happens when there are bugs around
>updating the SSH keys, or if the filesystem is corrupt on the system
>vm.
>
>
>On Fri, Mar 29, 2013 at 1:40 PM, Rajesh Battala
> wrote:
>> Hi All,
>>
>> VR is not getting started failing the instance creation. Anywork around
>>for this issue?
>>
>> [network.router.VirtualNetworkApplianceManagerImpl]
>>(Job-Executor-1:job-8) Unable to get the template/scripts version of
>>router r-4-VM due to: getDomRVersionCmd failed
>> INFO  [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8) The
>>guru did not like the answers so stopping VM[DomainRouter|r-4-VM]
>> ERROR [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8)
>>Failed to start instance VM[DomainRouter|r-4-VM]
>> com.cloud.utils.exception.ExecutionException: Unable to start
>>VM[DomainRouter|r-4-VM] due to error in finalizeStart, not retrying
>> at 
>>com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManager
>>Impl.java:808)
>> at 
>>com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.ja
>>va:471)
>> at 
>>com.cloud.network.router.VirtualNetworkApplianceManagerImpl.start(Virtual
>>NetworkApplianceManagerImpl.java:2606)
>> at 
>>com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startVirtualR
>>outer(VirtualNetworkApplianceManagerImpl.java:1815)
>> at 
>>com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startRouters(
>>VirtualNetworkApplianceManagerImpl.java:1915)
>> at 
>>com.cloud.network.router.VirtualNetworkApplianceManagerImpl.deployVirtual
>>RouterInGuestNetwork(VirtualNetworkApplianceManagerImpl.java:1893)
>> at 
>>com.cloud.network.element.VirtualRouterElement.prepare(VirtualRouterEleme
>>nt.java:208)
>> at 
>>com.cloud.network.NetworkManagerImpl.prepareElement(NetworkManagerImpl.ja
>>va:1541)
>> at 
>>com.cloud.network.NetworkManagerImpl.prepareNic(NetworkManagerImpl.java:1
>>658)
>> at 
>>com.cloud.network.NetworkManagerImpl.prepare(NetworkManagerImpl.java:1599
>>)
>> at 
>>com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManager
>>Impl.java:746)
>> at 
>>com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.ja
>>va:471)
>> at 
>>org.apache.cloudstack.engine.cloud.entity.api.VMEntityManagerImpl.deployV
>>irtualMachine(VMEntityManagerImpl.java:212)
>> at 
>>org.apache.cloudstack.engine.cloud.entity.api.VirtualMachineEntityImpl.de
>>ploy(VirtualMachineEntityImpl.java:209)
>> at 
>>com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java
>>:3860)
>> at 
>>com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java
>>:3458)
>> at 
>>com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java
>>:3444)
>> at 
>>com.cloud.utils.component.ComponentInstantiationPostProcessor$Interceptor
>>Dispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
>> at 
>>org.apache.cloudstack.api.command.user.vm.DeployVMCmd.execute(DeployVMCmd
>>.java:379)
>>
>>
>> Thanks
>> Rajesh Battala

Re: Unable to start domR(VR) in devcloud using laster 4.1/master.

2013-03-29 Thread Marcus Sorensen

Xen? KVM?

Generally GetDomRVersionCmd failing is only a symptom, it usually
means that cloudstack was never able to communicate with the system
vm. You would need to look at whether the VM actually started on the
host, and some of the text around when the VM was deployed to get any
meaningful errors.  Sometimes this happens when there are bugs around
updating the SSH keys, or if the filesystem is corrupt on the system
vm.


On Fri, Mar 29, 2013 at 1:40 PM, Rajesh Battala
 wrote:
> Hi All,
>
> VR is not getting started failing the instance creation. Anywork around for 
> this issue?
>
> [network.router.VirtualNetworkApplianceManagerImpl] (Job-Executor-1:job-8) 
> Unable to get the template/scripts version of router r-4-VM due to: 
> getDomRVersionCmd failed
> INFO  [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8) The guru 
> did not like the answers so stopping VM[DomainRouter|r-4-VM]
> ERROR [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8) Failed to 
> start instance VM[DomainRouter|r-4-VM]
> com.cloud.utils.exception.ExecutionException: Unable to start 
> VM[DomainRouter|r-4-VM] due to error in finalizeStart, not retrying
> at 
> com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:808)
> at 
> com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:471)
> at 
> com.cloud.network.router.VirtualNetworkApplianceManagerImpl.start(VirtualNetworkApplianceManagerImpl.java:2606)
> at 
> com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startVirtualRouter(VirtualNetworkApplianceManagerImpl.java:1815)
> at 
> com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startRouters(VirtualNetworkApplianceManagerImpl.java:1915)
> at 
> com.cloud.network.router.VirtualNetworkApplianceManagerImpl.deployVirtualRouterInGuestNetwork(VirtualNetworkApplianceManagerImpl.java:1893)
> at 
> com.cloud.network.element.VirtualRouterElement.prepare(VirtualRouterElement.java:208)
> at 
> com.cloud.network.NetworkManagerImpl.prepareElement(NetworkManagerImpl.java:1541)
> at 
> com.cloud.network.NetworkManagerImpl.prepareNic(NetworkManagerImpl.java:1658)
> at 
> com.cloud.network.NetworkManagerImpl.prepare(NetworkManagerImpl.java:1599)
> at 
> com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:746)
> at 
> com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:471)
> at 
> org.apache.cloudstack.engine.cloud.entity.api.VMEntityManagerImpl.deployVirtualMachine(VMEntityManagerImpl.java:212)
> at 
> org.apache.cloudstack.engine.cloud.entity.api.VirtualMachineEntityImpl.deploy(VirtualMachineEntityImpl.java:209)
> at 
> com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3860)
> at 
> com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3458)
> at 
> com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3444)
> at 
> com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
> at 
> org.apache.cloudstack.api.command.user.vm.DeployVMCmd.execute(DeployVMCmd.java:379)
>
>
> Thanks
> Rajesh Battala

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.


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


I do think an explicit migration interface on NetworkElement is the right way 
to do it. This way, network elements can decide explicitly when and how to 
handle this state. 
Sprinkling
   if(!nic.getReservationId().equals(context.getReservationId())){
// migration operation
return; 
}
everywhere is error prone:
- Implementors of new NetworkElements are not aware of this indirectly 
expressed dependency
- This snippet of code (except for the comment) does not in any way indicate 
the operation.

- Chiradeep Vittal


On March 29, 2013, 1:49 a.m., Hiroaki Kawai wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9871/
> ---
> 
> (Updated March 29, 2013, 1:49 a.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
> 
> 
> Description
> ---
> 
> The location of the virtual machine is provided by DeployDestination, which 
> will be passed in NetworkGuru#reserve and NetworkElement#prepare. 
> 
> During the virtual machine migration, it actually changes DeployDestination 
> and it looks like that it will tell that event to network components as it 
> has NetworkManager#prepareNicForMigration. The problem is that althogh the 
> interface has that method, NetworkManagerImpl does not tell the 
> DeployDestination changes to network components. 
> 
> So IMHO, we need to add calls of NetworkGuru#reserve and 
> NetworkElement#prepare in NetworkManagerImpl#prepareNicForMigration . And 
> then, we also need to add calls NetworkGuru#release and 
> NetworkElement#release after the migration, otherwise the network resources 
> that plugin reserved will be kept even when the vm leaves off.
> 
> Created a first minimum patch to show the concept.
> 
> 
> This addresses bug CLOUDSTACK-1638.
> 
> 
> Diffs
> -
> 
>   docs/en-US/plugin-niciranvp-tables.xml 4f81655 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/NiciraNvpNicMappingVO.java
>  0779e69 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/element/NiciraNvpElement.java
>  1fcccdb 
>   server/src/com/cloud/network/NetworkManager.java 4124b19 
>   server/src/com/cloud/network/NetworkManagerImpl.java a98bdd4 
>   server/src/com/cloud/network/guru/ControlNetworkGuru.java 934cd70 
>   server/src/com/cloud/network/guru/DirectNetworkGuru.java ee824af 
>   server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java 354d7ed 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java 24d24f8 
>   server/src/com/cloud/network/guru/GuestNetworkGuru.java cebfb08 
>   server/src/com/cloud/network/guru/PodBasedNetworkGuru.java b513325 
>   server/src/com/cloud/network/guru/StorageNetworkGuru.java 879d0cd 
>   server/src/com/cloud/vm/VirtualMachineManagerImpl.java 9230f4a 
>   setup/db/create-schema.sql 5b6dc04 
> 
> Diff: https://reviews.apache.org/r/9871/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Hiroaki Kawai
> 
>

Re: VM not obtaing DHCP address when migrating to another host.

On 3/29/13 7:04 AM, "Sean Truman"  wrote:

>All,
>
>When I use the migrate to another host option for a running VM (Basic
>networking), the VM refuses to get a DHCP address?  Is this a known issue?
>or is something wrong with my hosts? is there something I can check? When
>I
>migrate it back to the original host it obtains an IP address.
>
>v/r
>Sean

Which hypervisor? DHCP is only triggered at VM boot time, so I don't see
what it has to do with migration (which can happen well after boot time).
Are you migrating during boot?

Re: Scalability Limitations

2013-03-29 Thread Mike Tutkowski

Thanks, Chiradeep

Are you aware of any documentation we have with regards to scalability?  A
customer asked for such a doc today.

I found this one:

http://www.citrix.co.jp/event/pdf/5-scalability.pdf


On Fri, Mar 29, 2013 at 1:39 PM, Chiradeep Vittal <
chiradeep.vit...@citrix.com> wrote:

> No hard-coded maximums. Probably limited by the performance of the
> database most likely.
> I know of deployments which have tens of thousands.
>
> On 3/29/13 9:56 AM, "Mike Tutkowski"  wrote:
>
> >Hi,
> >
> >Can someone answer these questions for me?
> >
> >Is there a max number of clusters CS supports?  If so, what is that
> >number?
> >
> >Is there a max number of primary storages CS supports?  If so, what is
> >that
> >number?
> >
> >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: How to create a new type of SystemVM ?



On 3/29/13 6:36 AM, "Serge A. Salamanka"  wrote:

>On 29.03.2013 16:27, Chip Childers wrote:
>> Can you explain a little more about what you are trying to accomplish?
>> What do you mean by "grid-service", and why does it need to be a "system
>> vm"?
>
>For guest VMs in a Zone I'm trying to provide a specific service (that
>doesn't have to be a Grid service, just anything at all that you want
>your guest VMs to be able to access on the default IP address).
>
>*I call a grid-service any service that provides access to Grid (HPC).

I assume you mean:
A) a VM that is spun up on (every/some) guest network creation
B) the VM lifecycle is controlled by CloudStack
C) the VM provides some service API that is called by CloudStack
D) the VM is not visible to the owner of the network

If it is just A), why not use the even framework and listen for network
creation events?

Unable to start domR(VR) in devcloud using laster 4.1/master.

2013-03-29 Thread Rajesh Battala

Hi All,

VR is not getting started failing the instance creation. Anywork around for 
this issue?

[network.router.VirtualNetworkApplianceManagerImpl] (Job-Executor-1:job-8) 
Unable to get the template/scripts version of router r-4-VM due to: 
getDomRVersionCmd failed
INFO  [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8) The guru did 
not like the answers so stopping VM[DomainRouter|r-4-VM]
ERROR [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-1:job-8) Failed to 
start instance VM[DomainRouter|r-4-VM]
com.cloud.utils.exception.ExecutionException: Unable to start 
VM[DomainRouter|r-4-VM] due to error in finalizeStart, not retrying
at 
com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:808)
at 
com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:471)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.start(VirtualNetworkApplianceManagerImpl.java:2606)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startVirtualRouter(VirtualNetworkApplianceManagerImpl.java:1815)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.startRouters(VirtualNetworkApplianceManagerImpl.java:1915)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.deployVirtualRouterInGuestNetwork(VirtualNetworkApplianceManagerImpl.java:1893)
at 
com.cloud.network.element.VirtualRouterElement.prepare(VirtualRouterElement.java:208)
at 
com.cloud.network.NetworkManagerImpl.prepareElement(NetworkManagerImpl.java:1541)
at 
com.cloud.network.NetworkManagerImpl.prepareNic(NetworkManagerImpl.java:1658)
at 
com.cloud.network.NetworkManagerImpl.prepare(NetworkManagerImpl.java:1599)
at 
com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:746)
at 
com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:471)
at 
org.apache.cloudstack.engine.cloud.entity.api.VMEntityManagerImpl.deployVirtualMachine(VMEntityManagerImpl.java:212)
at 
org.apache.cloudstack.engine.cloud.entity.api.VirtualMachineEntityImpl.deploy(VirtualMachineEntityImpl.java:209)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3860)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3458)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3444)
at 
com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
at 
org.apache.cloudstack.api.command.user.vm.DeployVMCmd.execute(DeployVMCmd.java:379)


Thanks
Rajesh Battala

Re: Scalability Limitations

No hard-coded maximums. Probably limited by the performance of the
database most likely.
I know of deployments which have tens of thousands.

On 3/29/13 9:56 AM, "Mike Tutkowski"  wrote:

>Hi,
>
>Can someone answer these questions for me?
>
>Is there a max number of clusters CS supports?  If so, what is that
>number?
>
>Is there a max number of primary storages CS supports?  If so, what is
>that
>number?
>
>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: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

2013-03-29 Thread Sudha Ponnaganti

Thanks Sheng for clarification. I checked Affected version and now filtered 
with fix version as well and see that it is targeted to 4.2.00 


-Original Message-
From: Sheng Yang [mailto:sh...@yasker.org] 
Sent: Friday, March 29, 2013 10:16 AM
To: 
Subject: Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

CLOUDSTACK-1827: patch is already in MASTER, patch request for 4.1 sent to 
mailing list.
CLOUDSTACK-1755: Fix version is 4.2, we didn't plan to fix it in 4.1 and it's 
not a release blocker as we agreed.

--Sheng

On Fri, Mar 29, 2013 at 9:33 AM, Sudha Ponnaganti  
wrote:
> Below are today's ( 3/29 ) defects.   Upgrade is in progress - ~ 60% done.
> Couple of defects are not picked up - sent mail separately if anyone is 
> interested to pick them up.
> Rohit mentioned that he would be offline so 1839 need to be picked up as 
> well. I will keep it in unassigned state so someone else can pick it up if 
> there are no objections.
> It would help to wrap up QA cycle if we can get resolutions.
>
>
> CLOUDSTACK-1746 Cloudstack Usage Server won't start Kelven Yang
> CLOUDSTACK-1836 License header failures for ja-JP .po translation files 
> sebastien goasguen
> CLOUDSTACK-1848 Cloudstack Packages are not got updated with scenario 4.0 to 
> 4.1 upgrade where MS is on Ubuntu 12.04Unassigned
> CLOUDSTACK-1805 com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen 
> after long time of inactivity resulting in not being able to log in to thr 
> management server.  Kelven Yang
> CLOUDSTACK-1846 KVM - storage pools can silently fail to be unregistered, 
> leading to failure to register later  Marcus Sorensen
> CLOUDSTACK-1845 KVM - storage migration often fails Marcus Sorensen
> CLOUDSTACK-1839 Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and 
> indexes for many tables compare to the fresh installed 4.1 DB.   Rohit Yadav
> CLOUDSTACK-1755 IPv6 - When Vm is part of Dual network , /etc/reslov.conf 
> does not have ipv6 router ipaddress and ipv6 external dns.Sheng Yang
> CLOUDSTACK-1827 Redundant router - When VR Master was stopped failover to VR 
> Backup did not occur   Sheng Yang
> CLOUDSTACK-1842 ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types 
> on the Upgraded Setup   Unassigned
>
> Thanks
> /sudha
>
> -Original Message-
> From: Sebastien Goasguen [mailto:run...@gmail.com]
> Sent: Friday, March 29, 2013 2:47 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!
>
>
> On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
>
>> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
>>
>> Blocker:
>> CLOUDSTACK-1746
>> Cloudstack Usage Server won't start
>> Kishan KavalaFrancois Gaudreault
>>
>> CLOUDSTACK-1836
>> License header failures for ja-JP .po translation files
>> sebastien goasguen   Chandan Purushothama
>
> I fixed that one yesterday night. I did not close the ticket because I was 
> waiting for the next build to run.
> Can you tell me if the RAT passed properly ?
>
>>
>>
>> Critical:
>> CLOUDSTACK-1846
>> KVM - storage pools can silently fail to be unregistered, leading to 
>> failure to register later Marcus Sorensen
>>
>> CLOUDSTACK-1845
>> KVM - storage migration often fails
>> Marcus Sorensen
>>
>> CLOUDSTACK-1842
>> ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types on the 
>> Upgraded Setup
>> Unassigned   Chandan Purushothama
>>
>> CLOUDSTACK-1841
>> ASF 4.0 to 4.1 Upgrade: Missing Few Global Configuration parameters 
>> on the Upgraded Setup
>> Unassigned   Chandan Purushothama
>>
>> CLOUDSTACK-1839
>> Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and indexes for 
>> many tables compare to the fresh installed 4.1 DB.
>> Rohit Yadav  Sangeetha Hariharan
>> CLOUDSTACK-1834
>> Events are not generated for registerUserKeys(), Enabling account and 
>> Editing account.
>> Murali Reddy Sangeetha Hariharan
>>
>> CLOUDSTACK-1827  (This *may* already have been fixed by Sheng) 
>> Redundant router - When VR Master was stopped failover to VR Backup 
>> did not occur
>> Sheng Yang   angeline shen
>>
>> CLOUDSTACK-1819
>> AWS Regions - Issues seen when trying to move a zone from 1 region to 
>> another.
>> Kishan KavalaSangeetha Hariharan
>>
>> CLOUDSTACK-1805
>> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after 
>> long time of inactivity resulting in not being able to log in to thr 
>> management server.
>> Kelven Yang  Sangeetha Hariharan
>>
>>
>> On Thu, Mar 28, 2013 at 05:05:22PM -0400, Chip Childers wrote:
>>> Hi all,
>>>
>>> Here's the list as of right now (and we are still pending a clear 
>>> statement from Sudha / others that upgrade testing has been completed).
>>>
>>> Shout if you want help, or if there is a pending patch that I missed!
>>>
>>> Blocker:
>>> CLOUDSTACK-1836
>>> License header failures for ja-JP .po translation files sebastien 
>>> goasguen
>>>
>>> Critical:
>>> CLOUDSTACK-1805
>>> com.mysql.jdbc.exceptions.jd

Re: Assigned IPs

2013-03-29 Thread Sheng Yang

On Fri, Mar 29, 2013 at 12:49 AM, Maurice Lawler  wrote:
> In the account setup in CS it states allotted IPs allowed. Is it possible to 
> automatically from a template made to pull two IPs from the pool.
>
> I would think simply make a ifcfg-eth0:1 with DHCP settings would accomplish 
> this.
>
> Am I wrong in my thinking?

Your VM need to be in two networks to get two IPs. Each network would
only grant you one IP for your user VM.

--Sheng

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-29 Thread Min Chen


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



api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java


If password is default hashed to SHA256SALT, then it should not be clear 
text password. This description is contradictory to me.



developer/developer-prefill.sql


Should this password be encoded using SHA if that is the default way? The 
pre-filled one is still MD5.


- Min Chen


On March 28, 2013, 8:26 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10039/
> ---
> 
> (Updated March 28, 2013, 8:26 p.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers, Kelven Yang, and Min Chen.
> 
> 
> Description
> ---
> 
> Changing default password encoding mechanism from MD5 to SHA256Salted.
> 
> 
> This addresses bug CS-1734.
> 
> 
> Diffs
> -
> 
>   
> api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 
> 89673ea 
>   api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 
> fb29e1a 
>   api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java 
> 1f31662 
>   client/tomcatconf/componentContext.xml.in 016df0a 
>   client/tomcatconf/nonossComponentContext.xml.in 8f8dae5 
>   developer/developer-prefill.sql 6300d35 
>   
> plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
>  61eebe5 
>   
> plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java
>  026125e 
>   
> plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java
>  52e7cb3 
>   
> plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java
>  1b29f69 
>   server/src/com/cloud/server/ManagementServerImpl.java b689f93 
>   server/src/com/cloud/user/AccountManagerImpl.java b69f314 
> 
> Diff: https://reviews.apache.org/r/10039/diff/
> 
> 
> Testing
> ---
> 
> Manual testing done for both oss and nonoss components. Both admin and users 
> added later are encoded according to the scheme configured, and authenticated 
> by the same scheme.
> 
> To change the order of the schemes, modify the following list properties in 
> client/tomcatconf/nonossComponentContext.xml.in or 
> client/tomcatconf/componentContext.xml.in as applicable, to the desired order:
> 
> 
>  
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
>  
>  
> 
>  
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

2013-03-29 Thread Sheng Yang

CLOUDSTACK-1827: patch is already in MASTER, patch request for 4.1
sent to mailing list.
CLOUDSTACK-1755: Fix version is 4.2, we didn't plan to fix it in 4.1
and it's not a release blocker as we agreed.

--Sheng

On Fri, Mar 29, 2013 at 9:33 AM, Sudha Ponnaganti
 wrote:
> Below are today's ( 3/29 ) defects.   Upgrade is in progress - ~ 60% done.
> Couple of defects are not picked up - sent mail separately if anyone is 
> interested to pick them up.
> Rohit mentioned that he would be offline so 1839 need to be picked up as 
> well. I will keep it in unassigned state so someone else can pick it up if 
> there are no objections.
> It would help to wrap up QA cycle if we can get resolutions.
>
>
> CLOUDSTACK-1746 Cloudstack Usage Server won't start Kelven Yang
> CLOUDSTACK-1836 License header failures for ja-JP .po translation files 
> sebastien goasguen
> CLOUDSTACK-1848 Cloudstack Packages are not got updated with scenario 4.0 to 
> 4.1 upgrade where MS is on Ubuntu 12.04Unassigned
> CLOUDSTACK-1805 com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen 
> after long time of inactivity resulting in not being able to log in to thr 
> management server.  Kelven Yang
> CLOUDSTACK-1846 KVM - storage pools can silently fail to be unregistered, 
> leading to failure to register later  Marcus Sorensen
> CLOUDSTACK-1845 KVM - storage migration often fails Marcus Sorensen
> CLOUDSTACK-1839 Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and 
> indexes for many tables compare to the fresh installed 4.1 DB.   Rohit Yadav
> CLOUDSTACK-1755 IPv6 - When Vm is part of Dual network , /etc/reslov.conf 
> does not have ipv6 router ipaddress and ipv6 external dns.Sheng Yang
> CLOUDSTACK-1827 Redundant router - When VR Master was stopped failover to VR 
> Backup did not occur   Sheng Yang
> CLOUDSTACK-1842 ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types 
> on the Upgraded Setup   Unassigned
>
> Thanks
> /sudha
>
> -Original Message-
> From: Sebastien Goasguen [mailto:run...@gmail.com]
> Sent: Friday, March 29, 2013 2:47 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!
>
>
> On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
>
>> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
>>
>> Blocker:
>> CLOUDSTACK-1746
>> Cloudstack Usage Server won't start
>> Kishan KavalaFrancois Gaudreault
>>
>> CLOUDSTACK-1836
>> License header failures for ja-JP .po translation files
>> sebastien goasguen   Chandan Purushothama
>
> I fixed that one yesterday night. I did not close the ticket because I was 
> waiting for the next build to run.
> Can you tell me if the RAT passed properly ?
>
>>
>>
>> Critical:
>> CLOUDSTACK-1846
>> KVM - storage pools can silently fail to be unregistered, leading to
>> failure to register later Marcus Sorensen
>>
>> CLOUDSTACK-1845
>> KVM - storage migration often fails
>> Marcus Sorensen
>>
>> CLOUDSTACK-1842
>> ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types on the
>> Upgraded Setup
>> Unassigned   Chandan Purushothama
>>
>> CLOUDSTACK-1841
>> ASF 4.0 to 4.1 Upgrade: Missing Few Global Configuration parameters on
>> the Upgraded Setup
>> Unassigned   Chandan Purushothama
>>
>> CLOUDSTACK-1839
>> Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and indexes for
>> many tables compare to the fresh installed 4.1 DB.
>> Rohit Yadav  Sangeetha Hariharan
>> CLOUDSTACK-1834
>> Events are not generated for registerUserKeys(), Enabling account and
>> Editing account.
>> Murali Reddy Sangeetha Hariharan
>>
>> CLOUDSTACK-1827  (This *may* already have been fixed by Sheng)
>> Redundant router - When VR Master was stopped failover to VR Backup
>> did not occur
>> Sheng Yang   angeline shen
>>
>> CLOUDSTACK-1819
>> AWS Regions - Issues seen when trying to move a zone from 1 region to 
>> another.
>> Kishan KavalaSangeetha Hariharan
>>
>> CLOUDSTACK-1805
>> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after
>> long time of inactivity resulting in not being able to log in to thr 
>> management server.
>> Kelven Yang  Sangeetha Hariharan
>>
>>
>> On Thu, Mar 28, 2013 at 05:05:22PM -0400, Chip Childers wrote:
>>> Hi all,
>>>
>>> Here's the list as of right now (and we are still pending a clear
>>> statement from Sudha / others that upgrade testing has been completed).
>>>
>>> Shout if you want help, or if there is a pending patch that I missed!
>>>
>>> Blocker:
>>> CLOUDSTACK-1836
>>> License header failures for ja-JP .po translation files sebastien
>>> goasguen
>>>
>>> Critical:
>>> CLOUDSTACK-1805
>>> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after long 
>>> time of inactivity resulting in not being able to log in to thr management 
>>> server.
>>> Kelven Yang
>>>
>>> CLOUDSTACK-1819
>>> AWS Regions - Issues seen when trying to move a zone from 1 region to 
>>> another.
>>> Kishan Kavala
>>>
>>> CLOUDSTACK-1834
>>> Events are not generated for

Scalability Limitations

2013-03-29 Thread Mike Tutkowski

Hi,

Can someone answer these questions for me?

Is there a max number of clusters CS supports?  If so, what is that number?

Is there a max number of primary storages CS supports?  If so, what is that
number?

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: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

2013-03-29 Thread Sudha Ponnaganti

Below are today's ( 3/29 ) defects.   Upgrade is in progress - ~ 60% done. 
Couple of defects are not picked up - sent mail separately if anyone is 
interested to pick them up. 
Rohit mentioned that he would be offline so 1839 need to be picked up as well. 
I will keep it in unassigned state so someone else can pick it up if there are 
no objections. 
It would help to wrap up QA cycle if we can get resolutions.  

CLOUDSTACK-1746 Cloudstack Usage Server won't start Kelven Yang
CLOUDSTACK-1836 License header failures for ja-JP .po translation files 
sebastien goasguen
CLOUDSTACK-1848 Cloudstack Packages are not got updated with scenario 4.0 to 
4.1 upgrade where MS is on Ubuntu 12.04Unassigned
CLOUDSTACK-1805 com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen 
after long time of inactivity resulting in not being able to log in to thr 
management server.  Kelven Yang
CLOUDSTACK-1846 KVM - storage pools can silently fail to be unregistered, 
leading to failure to register later  Marcus Sorensen
CLOUDSTACK-1845 KVM - storage migration often fails Marcus Sorensen
CLOUDSTACK-1839 Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and indexes 
for many tables compare to the fresh installed 4.1 DB.   Rohit Yadav
CLOUDSTACK-1755 IPv6 - When Vm is part of Dual network , /etc/reslov.conf does 
not have ipv6 router ipaddress and ipv6 external dns.Sheng Yang
CLOUDSTACK-1827 Redundant router - When VR Master was stopped failover to VR 
Backup did not occur   Sheng Yang
CLOUDSTACK-1842 ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types on 
the Upgraded Setup   Unassigned

Thanks
/sudha

-Original Message-
From: Sebastien Goasguen [mailto:run...@gmail.com] 
Sent: Friday, March 29, 2013 2:47 AM
To: dev@cloudstack.apache.org
Subject: Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:

> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
> 
> Blocker:
> CLOUDSTACK-1746
> Cloudstack Usage Server won't start
> Kishan KavalaFrancois Gaudreault 
> 
> CLOUDSTACK-1836
> License header failures for ja-JP .po translation files
> sebastien goasguen   Chandan Purushothama

I fixed that one yesterday night. I did not close the ticket because I was 
waiting for the next build to run.
Can you tell me if the RAT passed properly ?

> 
> 
> Critical:
> CLOUDSTACK-1846
> KVM - storage pools can silently fail to be unregistered, leading to 
> failure to register later Marcus Sorensen
> 
> CLOUDSTACK-1845
> KVM - storage migration often fails
> Marcus Sorensen
> 
> CLOUDSTACK-1842
> ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types on the 
> Upgraded Setup
> Unassigned   Chandan Purushothama
> 
> CLOUDSTACK-1841
> ASF 4.0 to 4.1 Upgrade: Missing Few Global Configuration parameters on 
> the Upgraded Setup
> Unassigned   Chandan Purushothama 
> 
> CLOUDSTACK-1839
> Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and indexes for 
> many tables compare to the fresh installed 4.1 DB.
> Rohit Yadav  Sangeetha Hariharan
> CLOUDSTACK-1834
> Events are not generated for registerUserKeys(), Enabling account and 
> Editing account.
> Murali Reddy Sangeetha Hariharan 
> 
> CLOUDSTACK-1827  (This *may* already have been fixed by Sheng) 
> Redundant router - When VR Master was stopped failover to VR Backup 
> did not occur
> Sheng Yang   angeline shen
> 
> CLOUDSTACK-1819
> AWS Regions - Issues seen when trying to move a zone from 1 region to another.
> Kishan KavalaSangeetha Hariharan
> 
> CLOUDSTACK-1805
> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after 
> long time of inactivity resulting in not being able to log in to thr 
> management server.
> Kelven Yang  Sangeetha Hariharan
> 
> 
> On Thu, Mar 28, 2013 at 05:05:22PM -0400, Chip Childers wrote:
>> Hi all,
>> 
>> Here's the list as of right now (and we are still pending a clear 
>> statement from Sudha / others that upgrade testing has been completed).
>> 
>> Shout if you want help, or if there is a pending patch that I missed!
>> 
>> Blocker:
>> CLOUDSTACK-1836
>> License header failures for ja-JP .po translation files sebastien 
>> goasguen
>> 
>> Critical:
>> CLOUDSTACK-1805
>> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after long time 
>> of inactivity resulting in not being able to log in to thr management 
>> server.  
>> Kelven Yang
>> 
>> CLOUDSTACK-1819
>> AWS Regions - Issues seen when trying to move a zone from 1 region to 
>> another.  
>> Kishan Kavala
>> 
>> CLOUDSTACK-1834
>> Events are not generated for registerUserKeys(), Enabling account and 
>> Editing account.  
>> Murali Reddy
>> 
>> -chip
> 
> --
> Prasanna.,

[ACS41][QA] Upgrade defects

2013-03-29 Thread Sudha Ponnaganti

Want to check if anyone is interested to pick these up today

https://issues.apache.org/jira/browse/CLOUDSTACK-1842
https://issues.apache.org/jira/browse/CLOUDSTACK-1848

thanks
/sudha

RE: [master branch issue ]Problem with SystemVMs and Primary Storage in latest Master

2013-03-29 Thread Pranav Saxena

I have made the UI changes to support  zone wide primary storage only for KVM 
as the hypervisor . For all other hypervisors , we 'll have "cluster wide 
primary storage" as the default option . The fix has been pushed to asf/master.

Thanks,
Pranav

-Original Message-
From: Nitin Mehta [mailto:nitin.me...@citrix.com] 
Sent: Friday, March 29, 2013 12:23 PM
To: dev@cloudstack.apache.org
Subject: Re: [master branch issue ]Problem with SystemVMs and Primary Storage 
in latest Master

Besides UI changes suggested below, I also think that we shouldn't allow adding 
Zone wide storage in XS/Vmware if the capability doesn't exist.
There should be an explicit error thrown rather than waiting for system vms not 
starting.

On 29/03/13 12:16 AM, "Edison Su"  wrote:

>
>
>> -Original Message-
>> From: Srikanteswararao Talluri
>>[mailto:srikanteswararao.tall...@citrix.com]
>> Sent: Thursday, March 28, 2013 5:02 AM
>> To: dev@cloudstack.apache.org
>> Subject: RE: [master branch issue ]Problem with SystemVMs and Primary  
>>Storage in latest Master
>> 
>> 
>> 
>> > -Original Message-
>> > From: Chip Childers [mailto:chip.child...@sungard.com]
>> > Sent: Thursday, March 28, 2013 5:24 PM
>> > To: 
>> > Subject: Re: [master branch issue ]Problem with SystemVMs and 
>> > Primary Storage in latest Master
>> >
>> > On Mar 28, 2013, at 6:00 AM, Pranav Saxena 
>> > 
>> > wrote:
>> >
>> > > Ok , Suresh was right in pointing that out that we need to select 
>> > > "cluster " right
>> > now as the scope for primary storage with xenserver as the 
>> > hypervisor
>>.
>> >
>> > Sounds like we should actually have the default behavior match the 
>> > previous versions, and make cluster the default.
>> 
>> +1
>
>+1, The UI should use "cluster" wide storage as the default, that's the
>default behavior of createstoragepool API: if no scope is specified, it 
>implies "cluster" scope.


+1 to this.

>
>> 
>> >
>> >
>> > > With "Zone-wide" , we see this error . Perhaps I can introduce a 
>> > > check in the UI
>> > for the scope depending upon the hypervisor type (xen/KVM) since we 
>> > don't support zone-wide primary storage for Xenserver as of now.
>> >
>> > +1
>> >
>> > >
>> > > My set up is working fine now.
>> > >
>> > > Thanks,
>> > > Pranav
>> > >
>> > > -Original Message-
>> > > From: Nitin Mehta [mailto:nitin.me...@citrix.com]
>> > > Sent: Thursday, March 28, 2013 3:17 PM
>> > > To: dev@cloudstack.apache.org
>> > > Cc: Abhinandan Prateek
>> > > Subject: Re: [master branch issue ]Problem with SystemVMs and 
>> > > Primary Storage in latest Master
>> > >
>> > > Yes, I saw the same issue yesterday evening.
>> > >
>> > > On 28/03/13 2:29 PM, "Pranav Saxena" 
>> wrote:
>> > >
>> > >> Hi ,
>> > >>
>> > >> There seems to be a problem with the latest master branch code 
>> > >> though I did not happen to see any exception being thrown.
>> > >>
>> > >> The system VM's don't come up and the primary storage added as 
>> > >> part of the zone creation wizard does not show up on the host in 
>> > >> the XenCenter UI .
>> > >>
>> > >> Anyone else happened to see this issue ?
>> > >>
>> > >> Thanks,
>> > >> Pranav
>> > >
>> > >

Review request for storage motion on xenserver

2013-03-29 Thread Devdeep Singh

I have put the feature proposed [1] and developed in the feature branch [2] up 
for review. Code for this feature conforms to what was proposed in FS [3]. The 
patch available at [4]. It includes marvin tests and unit tests for verifying 
the functionality. Please take a look at it and let me know your comments.

[1] http://markmail.org/message/numdk6pdab2hekdp
[2] https://github.com/devdeep/cloudstack/commits/sm3
[3] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enabling+Storage+XenMotion+for+XenServer
[4] https://reviews.apache.org/r/10196/

Regards,
Devdeep

Re: [ACS41][Patch Request]

On Fri, Mar 29, 2013 at 06:12:14PM +0530, Prasanna Santhanam wrote:
> On Fri, Mar 29, 2013 at 01:29:33PM +0530, Prasanna Santhanam wrote:
> > commit d13c18516a30d035e2251d187979e805670aab81
> > Author: Prasanna Santhanam 
> > Date:   Fri Mar 29 13:25:20 2013 +0530
> > 
> > CLOUDSTACK-1841: Missing `cloud`.`configuration` stmts in upgrade
> > 
> > Following global configurations introduced b/w 3.0.2 and 4.1 were
> > missed.
> > 
> > concurrent.snapshots.threshold.perhost | NULL
> > network.ipv6.search.retry.max | 1
> > traffic.sentinel.exclude.zones |
> > traffic.sentinel.include.zones | EXTERNAL
> > 
> > Signed-off-by: Prasanna Santhanam 
> 
> Chip - I've applied this to 4.1.
> c54d9c96359334e0a0a81143095c97f213900acc
> 
> -- 
> Prasanna.,
>

ack - thanks

Re: [ACS41][Patch Request]

On Fri, Mar 29, 2013 at 10:25:23AM +, Murali Reddy wrote:
> commit 48311363d600264cb3542992a615706cb92f90ad
> Author: Murali Reddy 
> Date:   Fri Mar 29 15:53:21 2013 +0530
> 
> CLOUDSTACK-1834: Events are not generated for registerUserKeys(),
> Enabling account and Editing account.

Pushed to 4.1:

commit 2e395b850138bed41e198c9b0aba0b9de245df76
Author: Murali Reddy 
Date:   Fri Mar 29 15:53:21 2013 +0530

CLOUDSTACK-1834: Events are not generated for registerUserKeys(),
Enabling account and Editing account.

Review Request: Storage motion changes for xenserver

2013-03-29 Thread Devdeep Singh

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

Review request for cloudstack, Abhinandan Prateek, edison su, Alex Huang, and
anthony xu.

Description
---

Storage motion for Xenserver. FS for the feature
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enabling+Storage+XenMotion+for+XenServer
Changes made.
1. Implemented Api listStoragePoolsForMigration as per the specification.
It lists out all the pools available to which a volume me be migrated
2. Updated migrateVolume api for allowing migrating volumes of running vms.
These changes are integrated into the latest storage refactoring changes.
3. Added the implementation for listHostsForMigration api. It lists the
hosts to which an instance can be migrated, including hosts from other clusters
to which an instance may be migrated with storage motion
4. Added the implementation for migrateVirtualMachineWithVolume api. It
migrates a VM to another host with its volumes. The host may be in the same
cluster or another cluster.
5. Updated the listHosts api for backward compatibility.
6. Resolved an issue where storage xenmotion of 2nd VM created from the
same template to a host was failing with duplicate_vm exception. Made changes
to remove the mac_seed key value pair from other_config when vms are created.
The mac_seed is auto generated by the api if it is not passed in the record.
7. Updated the db schema script.
8. Added the right permissions in commands.properties for the apis.
9. Marvin tests for testing storage motion. Following scenarios are tested.
9.1. A virtual machine is migrated to another host. Its volumes are also
migrated to another storage pool.
9.2. Just the volumes of a vm are migrated to another storage pool while
the vm continues to run on the same host.
10. Unit tests for testing migration of a vm with its volumes.

This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-659.

Diffs
-

api/src/com/cloud/agent/api/MigrateWithStorageAnswer.java PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageCommand.java PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageCompleteAnswer.java
PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageCompleteCommand.java
PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageReceiveAnswer.java PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageReceiveCommand.java
PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageSendAnswer.java PRE-CREATION
api/src/com/cloud/agent/api/MigrateWithStorageSendCommand.java PRE-CREATION
api/src/com/cloud/agent/api/storage/MigrateVolumeAnswer.java PRE-CREATION
api/src/com/cloud/agent/api/storage/MigrateVolumeCommand.java PRE-CREATION
api/src/com/cloud/hypervisor/HypervisorCapabilities.java aff81b0
api/src/com/cloud/server/ManagementService.java 1e6ca8d
api/src/com/cloud/vm/UserVmService.java 2c33d41
api/src/org/apache/cloudstack/api/ApiConstants.java c518830
api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java
29844c3

api/src/org/apache/cloudstack/api/command/admin/host/ListHostsForMigrationCmd.java
PRE-CREATION

api/src/org/apache/cloudstack/api/command/admin/storage/ListStoragePoolsForMigrationCmd.java
PRE-CREATION

api/src/org/apache/cloudstack/api/command/admin/vm/MigrateVirtualMachineWithVolumeCmd.java
PRE-CREATION
api/src/org/apache/cloudstack/api/command/user/volume/MigrateVolumeCmd.java
287241a
api/src/org/apache/cloudstack/api/response/HostResponse.java f5aa8f9
api/src/org/apache/cloudstack/api/response/StoragePoolResponse.java 66dde36
client/tomcatconf/commands.properties.in 163c2ce
core/src/com/cloud/hypervisor/HypervisorCapabilitiesVO.java b525a2d

engine/api/src/org/apache/cloudstack/engine/subsystem/api/storage/ObjectInDataStoreStateMachine.java
726ce08

engine/api/src/org/apache/cloudstack/engine/subsystem/api/storage/VolumeService.java
102c471

engine/storage/imagemotion/src/org/apache/cloudstack/storage/image/motion/DefaultImageMotionStrategy.java
c49a521

engine/storage/integration-test/test/org/apache/cloudstack/storage/test/MockStorageMotionStrategy.java
b619ee9

engine/storage/src/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java
3602bb1

engine/storage/src/org/apache/cloudstack/storage/motion/DataMotionService.java
db36f64

engine/storage/src/org/apache/cloudstack/storage/motion/DataMotionServiceImpl.java
343140f

engine/storage/src/org/apache/cloudstack/storage/motion/DataMotionStrategy.java
ba40c6d

engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeObject.java
ceadb25

engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
32e7d27

plugins/host-allocators/random/s

Re: Review Request: MidoNet Networking Plugin [2/2]

2013-03-29 Thread Dave Cahill

Sounds great, thanks Hugo!


On Fri, Mar 29, 2013 at 8:47 PM, Hugo Trippaers <
htrippa...@schubergphilis.com> wrote:

>  Hey Dave,
>
>  My "ship it" is waiting for a last compile and test run on my dev
> platform. I'll try to do that over the weekend.
>
>   Cheers,
>
>  Hugo
>
> Sent from my iPhone
>
> On 29 mrt. 2013, at 02:17, "Dave Cahill"  wrote:
>
>   Hi all,
>
>  I think all review comments have been addressed on this, but review
> progress seems to have stalled - anything I should be doing to keep things
> moving?
>
>  Thanks,
> Dave.
>
>
> On Wed, Mar 27, 2013 at 3:33 PM, Dave Cahill  wrote:
>
>>This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/9898/
>>Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
>> By Dave Cahill.
>>
>> *Updated March 27, 2013, 6:33 a.m.*
>> Changes
>>
>> Fixed Hiroaki's exception-related review comment.
>>
>>   Description
>>
>> Feature 
>> spec:https://cwiki.apache.org/confluence/display/CLOUDSTACK/Midokura+Networking+Plugin
>>
>> Jira ticket:https://issues.apache.org/jira/browse/CLOUDSTACK-996
>>
>> Notes:
>>
>> * Documentation will follow as a separate commit
>>
>> * One main difference from existing networking plugins is the lack of a 
>> Resource class; we didn't feel it was necessary in this case. As mentioned 
>> in Extending CloudStack Networking [1]:
>> "Just like managers, resources are not strictly necessary. In theory a 
>> Network Element could implement a client for the API of the new controller 
>> and therefore be completely self-contained."
>>
>> * We allow overriding Public traffic via the MidoNetPublicNetworkGuru. We 
>> checked this approach with the list [2] and received no comments, so we're 
>> going with it for now.
>>
>> [1] https://cwiki.apache.org/CLOUDSTACK/extending-cloudstack-networking.html
>> [2] http://markmail.org/message/k5qse63eyylszm3i
>>
>>   Testing
>>
>> Built and deployed, spun up Advanced Isolated network with two VMs, verified 
>> internal and external connectivity via MidoNet.
>>
>>   *Bugs: *CLOUDSTACK-996
>> Diffs (updated)
>>
>>- api/src/com/cloud/network/Network.java (c2ab655)
>>- api/src/com/cloud/network/Networks.java (e3d2158)
>>- api/src/com/cloud/network/PhysicalNetwork.java (343a2b1)
>>- api/src/org/apache/cloudstack/network/ExternalNetworkDeviceManager.java
>>(bc22804)
>>- client/pom.xml (7ad2eff)
>>- 
>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
>>(b622b6d)
>>- 
>> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
>>(c93aeeb)
>>- plugins/network-elements/midokura-midonet/pom.xml (7f2e2d3)
>>- plugins/network-elements/midonet/pom.xml (PRE-CREATION)
>>- 
>> plugins/network-elements/midonet/src/com/cloud/network/element/MidoNetElement.java
>>(48833b3)
>>- 
>> plugins/network-elements/midonet/src/com/cloud/network/element/SimpleFirewallRule.java
>>(PRE-CREATION)
>>- 
>> plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetGuestNetworkGuru.java
>>(ed0eb3c)
>>- 
>> plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetPublicNetworkGuru.java
>>(PRE-CREATION)
>>- 
>> plugins/network-elements/midonet/src/com/cloud/network/resource/MidoNetVifDriver.java
>>(PRE-CREATION)
>>- 
>> plugins/network-elements/midonet/test/com/cloud/network/element/MidoNetElementTest.java
>>(PRE-CREATION)
>>- plugins/pom.xml (39d9907)
>>- server/src/com/cloud/configuration/Config.java (9db7dbd)
>>- server/src/com/cloud/network/NetworkManagerImpl.java (b1236cc)
>>- ui/scripts/system.js (c0a5d14)
>>
>> View Diff 
>>
>
>

Re: [CI] Upgrading Jenkins

The repo was pointed wrong. Fixed now.


On 29 March 2013 19:52, Prasanna Santhanam wrote:

> Shouldn't be. But not sure, will take a look shortly.
>
>
> --
> Prasanna.,
>
> - Original Message -
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Friday, March 29, 2013 09:24 PM
> To: CloudStack Dev 
> Subject: Re: [CI] Upgrading Jenkins
>
> On Fri, Mar 29, 2013 at 05:07:23PM +0530, Prasanna Santhanam wrote:
> > On Fri, Mar 29, 2013 at 04:54:02PM +0530, Prasanna Santhanam wrote:
> > > Hi,
> > >
> > > I'm hitting couple of jenkins bugs [1][2] on my build jobs.
> > > Jenkins.cs.o will be upgraded.
> > >
> > > [1] https://issues.jenkins-ci.org/browse/JENKINS-13972
> > > [2] https://issues.jenkins-ci.org/browse/JENKINS-17337
> > >
> > This is done. Let me know if there's any problems.
> >
> > --
> > Prasanna.,
> >
>
> Is the master docs build failing due to this upgrade?
>
> http://jenkins.cloudstack.org/job/build-docs-admin-master/1223/console
>



-- 
Prasanna.,

Re: [CI] Upgrading Jenkins

Shouldn't be. But not sure, will take a look shortly.


--
Prasanna.,

- Original Message -
From: Chip Childers [mailto:chip.child...@sungard.com]
Sent: Friday, March 29, 2013 09:24 PM
To: CloudStack Dev 
Subject: Re: [CI] Upgrading Jenkins

On Fri, Mar 29, 2013 at 05:07:23PM +0530, Prasanna Santhanam wrote:
> On Fri, Mar 29, 2013 at 04:54:02PM +0530, Prasanna Santhanam wrote:
> > Hi,
> > 
> > I'm hitting couple of jenkins bugs [1][2] on my build jobs.
> > Jenkins.cs.o will be upgraded.
> > 
> > [1] https://issues.jenkins-ci.org/browse/JENKINS-13972
> > [2] https://issues.jenkins-ci.org/browse/JENKINS-17337
> > 
> This is done. Let me know if there's any problems.
> 
> -- 
> Prasanna.,
>

Is the master docs build failing due to this upgrade?

http://jenkins.cloudstack.org/job/build-docs-admin-master/1223/console

VM not obtaing DHCP address when migrating to another host.

2013-03-29 Thread Sean Truman

All,

When I use the migrate to another host option for a running VM (Basic
networking), the VM refuses to get a DHCP address?  Is this a known issue?
or is something wrong with my hosts? is there something I can check? When I
migrate it back to the original host it obtains an IP address.

v/r
Sean

Re: Review Request: Fix log4j configuration for KVM agent

On Fri, Mar 29, 2013 at 05:40:31AM +, Animesh Chaturvedi wrote:
> 
> 
> > On March 15, 2013, 1:59 p.m., Chip Childers wrote:
> > > also, what's the bug ID?
> 
> I updated the BUG ID. Chip looking at JIRA you have already committed it. 
> Kelven can you mark the patch as submitted?
> 
> 
> - Animesh

I just closed the review.

Re: How to create a new type of SystemVM ?

2013-03-29 Thread Serge A. Salamanka

On 29.03.2013 16:27, Chip Childers wrote:
> Can you explain a little more about what you are trying to accomplish?
> What do you mean by "grid-service", and why does it need to be a "system
> vm"?

For guest VMs in a Zone I'm trying to provide a specific service (that
doesn't have to be a Grid service, just anything at all that you want
your guest VMs to be able to access on the default IP address).

*I call a grid-service any service that provides access to Grid (HPC).

Re: How to create a new type of SystemVM ?

On Fri, Mar 29, 2013 at 04:05:02PM +0300, Serge A. Salamanka wrote:
> Dear all,
> 
> There is a need to create a new type of system VM that will provide a
> grid-service for a Zone.

Can you explain a little more about what you are trying to accomplish?
What do you mean by "grid-service", and why does it need to be a "system
vm"?

> 
> Could anyone give me some tips, please, where to start from?
> 
> I have already all the information on how to create new system VM but
> there is no information on the list and in the docs on how to deploy a
> new type of system VM. I have looked into the code but seem to be
> frustrated a bit.
> 
> #Serge
>

Re: Review Request: BigSwitch networking plugin update


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

Ship it!


Branch: refs/heads/master
Commit: ee3fd1843533528eb3872951730fee6e5802c348
Parents: 690c312
Author: Kanzhe Jiang 
Authored: Fri Mar 29 18:48:50 2013 +0530
Committer: Murali Reddy 
Committed: Fri Mar 29 18:54:48 2013 +0530

- Murali Reddy


On March 29, 2013, 6:38 a.m., Kanzhe Jiang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9789/
> ---
> 
> (Updated March 29, 2013, 6:38 a.m.)
> 
> 
> Review request for cloudstack, Murali Reddy and Chiradeep Vittal.
> 
> 
> Description
> ---
> 
> Update BigSwitch Plugin:
> . UI support for adding BigSwitch controller as a network service provider
> . Correct the message format between the plugin and the controller
> . Add Health check API
> 
> 
> This addresses bug 733.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties d167a5c 
>   client/tomcatconf/componentContext.xml.in 23566e2 
>   client/tomcatconf/nonossComponentContext.xml.in 20e0c32 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/DeleteBigSwitchVnsDeviceCmd.java
>  f26bdde 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/ListBigSwitchVnsDevicesCmd.java
>  002d750 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/api/response/BigSwitchVnsDeviceResponse.java
>  ede5adf 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/BigSwitchVnsDeviceVO.java
>  7e64833 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Attachment.java
>  9a13467 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/AttachmentData.java
>  PRE-CREATION 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/BigSwitchVnsApi.java
>  c409dce 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/ControlClusterStatus.java
>  9181afd 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Network.java
>  c89c693 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/NetworkData.java
>  PRE-CREATION 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Port.java
>  bf749d0 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/PortData.java
>  PRE-CREATION 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/element/BigSwitchVnsElement.java
>  95c9e18 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitchVnsGuestNetworkGuru.java
>  b96ea56 
>   
> plugins/network-elements/bigswitch-vns/src/com/cloud/network/resource/BigSwitchVnsResource.java
>  8d070b5 
>   
> plugins/network-elements/bigswitch-vns/test/com/cloud/network/bigswitch/BigSwitchApiTest.java
>  cab06ba 
>   
> plugins/network-elements/bigswitch-vns/test/com/cloud/network/resource/BigSwitchVnsResourceTest.java
>  514385f 
>   ui/dictionary.jsp 3aecaaa 
>   ui/scripts/system.js 4d529ae 
> 
> Diff: https://reviews.apache.org/r/9789/diff/
> 
> 
> Testing
> ---
> 
> Test to add a BigSwitch controller as a network service provider and view.
> Test to implementation of guest network on the controller
> Test to associate VM to its guest network
> 
> Test procedure:
> 1. Create an advanced zone with isolation = VNS
> 2. Add BigSwitch controller as the network service provider for the physical 
> network
> 3. Configure OVS on the host to point to the controller
> 4. Create two guest networks
> 5. Create two VM instances and make sure each VM belongs to the right network.
> 6. Confirm VMs in the same network can ping each other and the virtual router.
> 
> 
> Thanks,
> 
> Kanzhe Jiang
> 
>

Re: [CI] Upgrading Jenkins

On Fri, Mar 29, 2013 at 05:07:23PM +0530, Prasanna Santhanam wrote:
> On Fri, Mar 29, 2013 at 04:54:02PM +0530, Prasanna Santhanam wrote:
> > Hi,
> > 
> > I'm hitting couple of jenkins bugs [1][2] on my build jobs.
> > Jenkins.cs.o will be upgraded.
> > 
> > [1] https://issues.jenkins-ci.org/browse/JENKINS-13972
> > [2] https://issues.jenkins-ci.org/browse/JENKINS-17337
> > 
> This is done. Let me know if there's any problems.
> 
> -- 
> Prasanna.,
>

Is the master docs build failing due to this upgrade?

http://jenkins.cloudstack.org/job/build-docs-admin-master/1223/console

How to create a new type of SystemVM ?

2013-03-29 Thread Serge A. Salamanka

Dear all,

There is a need to create a new type of system VM that will provide a
grid-service for a Zone.

Could anyone give me some tips, please, where to start from?

I have already all the information on how to create new system VM but
there is no information on the list and in the docs on how to deploy a
new type of system VM. I have looked into the code but seem to be
frustrated a bit.

#Serge

Re: Review Request: BUG-ID: CLOUDSTACK-1689


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


Commit 5a66ec35ad08ff3313e72b54eba7e1754868eee4 in branch refs/heads/bvt from 
Prasanna Santhanam 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5a66ec3 ]

CLOUDSTACK-1689: KVM agent install.sh should install ipset command

Added ipset in cloud.spec as part of agent install.

Signed-off-by: Pradeep 
Signed-off-by: Prasanna Santhanam 


- ASF Subversion and Git Services


On March 29, 2013, 10:41 a.m., Pradeep Soundararajan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9946/
> ---
> 
> (Updated March 29, 2013, 10:41 a.m.)
> 
> 
> Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, 
> and Wido den Hollander.
> 
> 
> Description
> ---
> 
> BUG-ID: CLOUDSTACK-1689
> 
> Summary: KVM agent install.sh should install ipset command
> 
> Added ipset in cloud.spec as part of agent install.
> 
> Signed-off-by: Pradeep 
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec e10ae1e 
> 
> Diff: https://reviews.apache.org/r/9946/diff/
> 
> 
> Testing
> ---
> 
> Packaging ipset properly after this change. 
> 
> 
> Thanks,
> 
> Pradeep Soundararajan
> 
>

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts


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


Commit 95fce112157526b0727821e145ec53d0e8f3e824 in branch refs/heads/bvt from 
Hugo Trippaers 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=95fce11 ]

CLOUDSTACK-1156 Fix small typo

- ASF Subversion and Git Services


On March 28, 2013, 8:36 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9541/
> ---
> 
> (Updated March 28, 2013, 8:36 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, Nitin Mehta, Sateesh 
> Chodapuneedi, mice xia, and Min Chen.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts
> 
> Addition of two new resource types i.e. Primary and Secondary storage 
> space in the existing pool of
> resource types.
> Added methods to set the limits on these resources using 
> updateResourceLimit
> API command and to get a count using updateResourceCount. Also added 
> calls in the
> Templates, Volumes, Snapshots life cycle to check these limits and to 
> increment/decrement the new
> resource types
> 
> Resource Name  :: Resource type number
> Primary Storage   10
> Secondary Storage 11
> 
> Also added jUnit Tests for the same.
> 
> 
> This addresses bug CLOUDSTACK-1156.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/configuration/Resource.java 7614c8a 
>   api/src/com/cloud/storage/VolumeApiService.java 8517988 
>   
> api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceCountCmd.java
>  f6d3a98 
>   
> api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceLimitCmd.java
>  0039f62 
>   api/src/org/apache/cloudstack/api/command/user/volume/ResizeVolumeCmd.java 
> 955727a 
>   api/src/org/apache/cloudstack/api/response/AccountResponse.java 9a98a35 
>   api/src/org/apache/cloudstack/api/response/ResourceCountResponse.java 
> a7fbbf2 
>   api/src/org/apache/cloudstack/api/response/ResourceLimitResponse.java 
> b444e7a 
>   
> plugins/hypervisors/baremetal/src/com/cloud/baremetal/manager/BareMetalTemplateAdapter.java
>  33725f6 
>   server/src/com/cloud/alert/AlertManagerImpl.java a45482f 
>   server/src/com/cloud/api/ApiResponseHelper.java f33601f 
>   server/src/com/cloud/api/query/dao/AccountJoinDaoImpl.java 898bafc 
>   server/src/com/cloud/api/query/vo/AccountJoinVO.java cd7231c 
>   server/src/com/cloud/configuration/Config.java 1a57a64 
>   server/src/com/cloud/resourcelimit/ResourceLimitManagerImpl.java 23c0796 
>   server/src/com/cloud/storage/VolumeManager.java af3cbbf 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 737ed0a 
>   server/src/com/cloud/storage/dao/SnapshotDao.java 0e378a7 
>   server/src/com/cloud/storage/dao/SnapshotDaoImpl.java 825b6d5 
>   server/src/com/cloud/storage/dao/VolumeDao.java d7a2667 
>   server/src/com/cloud/storage/dao/VolumeDaoImpl.java 40ed875 
>   server/src/com/cloud/storage/download/DownloadListener.java d4c20cf 
>   server/src/com/cloud/storage/download/DownloadMonitorImpl.java 0bc89e3 
>   server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java bacca01 
>   server/src/com/cloud/template/HypervisorTemplateAdapter.java 1426421 
>   server/src/com/cloud/template/TemplateManagerImpl.java d843dbc 
>   server/src/com/cloud/vm/UserVmManagerImpl.java 82a69bd 
>   server/test/com/cloud/resourcelimit/ResourceLimitManagerImplTest.java 
> d311ad3 
>   server/test/com/cloud/vpc/MockResourceLimitManagerImpl.java b9fc861 
>   setup/db/db/schema-40to410.sql 60ad025 
>   setup/db/db/schema-410to420.sql 9eed0a6 
>   utils/src/com/cloud/utils/UriUtils.java a8b5ccb 
> 
> Diff: https://reviews.apache.org/r/9541/diff/
> 
> 
> Testing
> ---
> 
> Tested life cycle of templates, volumes, snapshots, vm on my local CloudStack 
> setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>

Re: [ACS41][Patch Request]

On Fri, Mar 29, 2013 at 01:29:33PM +0530, Prasanna Santhanam wrote:
> commit d13c18516a30d035e2251d187979e805670aab81
> Author: Prasanna Santhanam 
> Date:   Fri Mar 29 13:25:20 2013 +0530
> 
> CLOUDSTACK-1841: Missing `cloud`.`configuration` stmts in upgrade
> 
> Following global configurations introduced b/w 3.0.2 and 4.1 were
> missed.
> 
> concurrent.snapshots.threshold.perhost | NULL
> network.ipv6.search.retry.max | 1
> traffic.sentinel.exclude.zones |
> traffic.sentinel.include.zones | EXTERNAL
> 
> Signed-off-by: Prasanna Santhanam 

Chip - I've applied this to 4.1.
c54d9c96359334e0a0a81143095c97f213900acc

-- 
Prasanna.,

Review Request: Cloudstack-1739

2013-03-29 Thread bharat kumar


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

Review request for cloudstack and Abhinandan Prateek.


Description
---

VMs are getting 1 GB memory on KVM hypervisor ,when they are deployed with 
Service Offering having RAM >=1GB and overcommit ratio is set to some value>1


This addresses bug Cloudstack-1739.


Diffs
-

  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 f786f88 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
c93aeeb 

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


Testing
---

manual testing on master.


Thanks,

bharat kumar

Re: Maven POM file question (related to Eclipse)

2013-03-29 Thread Hugo Trippaers

I'm just ignoring those. I haven't seen any trouble during development. I do 
use a separate linux host to build packages to avoid problems.

Cheers,

Hugo

Sent from my iPhone

On 29 mrt. 2013, at 03:21, "Mike Tutkowski"  
wrote:

> Hi,
> 
> I've been building and running outside of Eclipse for the past few months,
> but decided to give it a try again today. Everything seems to build now,
> but I have three Maven issues (looks like m2e doesn't support certain Maven
> features we're using).
> 
> What do you suggest I do with the following three problems (all the same
> type of problem):
> 
> http://i.imgur.com/lsFI2Yh.png
> 
> 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: Review Request: MidoNet Networking Plugin [2/2]

2013-03-29 Thread Hugo Trippaers

Hey Dave,

My "ship it" is waiting for a last compile and test run on my dev platform. 
I'll try to do that over the weekend.

 Cheers,

Hugo

Sent from my iPhone

On 29 mrt. 2013, at 02:17, "Dave Cahill" 
mailto:dcah...@midokura.com>> wrote:

Hi all,

I think all review comments have been addressed on this, but review progress 
seems to have stalled - anything I should be doing to keep things moving?

Thanks,
Dave.


On Wed, Mar 27, 2013 at 3:33 PM, Dave Cahill 
mailto:dcah...@midokura.com>> wrote:
This is an automatically generated e-mail. To reply, visit: 
https://reviews.apache.org/r/9898/

Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
By Dave Cahill.

Updated March 27, 2013, 6:33 a.m.

Changes

Fixed Hiroaki's exception-related review comment.


Description

Feature spec:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Midokura+Networking+Plugin

Jira ticket:
https://issues.apache.org/jira/browse/CLOUDSTACK-996

Notes:

* Documentation will follow as a separate commit

* One main difference from existing networking plugins is the lack of a 
Resource class; we didn't feel it was necessary in this case. As mentioned in 
Extending CloudStack Networking [1]:
"Just like managers, resources are not strictly necessary. In theory a Network 
Element could implement a client for the API of the new controller and 
therefore be completely self-contained."

* We allow overriding Public traffic via the MidoNetPublicNetworkGuru. We 
checked this approach with the list [2] and received no comments, so we're 
going with it for now.

[1] https://cwiki.apache.org/CLOUDSTACK/extending-cloudstack-networking.html
[2] http://markmail.org/message/k5qse63eyylszm3i


Testing

Built and deployed, spun up Advanced Isolated network with two VMs, verified 
internal and external connectivity via MidoNet.


Bugs: CLOUDSTACK-996
Diffs (updated)

  *   api/src/com/cloud/network/Network.java (c2ab655)
  *   api/src/com/cloud/network/Networks.java (e3d2158)
  *   api/src/com/cloud/network/PhysicalNetwork.java (343a2b1)
  *   api/src/org/apache/cloudstack/network/ExternalNetworkDeviceManager.java 
(bc22804)
  *   client/pom.xml (7ad2eff)
  *   
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
 (b622b6d)
  *   
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
(c93aeeb)
  *   plugins/network-elements/midokura-midonet/pom.xml (7f2e2d3)
  *   plugins/network-elements/midonet/pom.xml (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/element/MidoNetElement.java
 (48833b3)
  *   
plugins/network-elements/midonet/src/com/cloud/network/element/SimpleFirewallRule.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetGuestNetworkGuru.java
 (ed0eb3c)
  *   
plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetPublicNetworkGuru.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/resource/MidoNetVifDriver.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/test/com/cloud/network/element/MidoNetElementTest.java
 (PRE-CREATION)
  *   plugins/pom.xml (39d9907)
  *   server/src/com/cloud/configuration/Config.java (9db7dbd)
  *   server/src/com/cloud/network/NetworkManagerImpl.java (b1236cc)
  *   ui/scripts/system.js (c0a5d14)

View Diff

Re: [CI] Upgrading Jenkins

On Fri, Mar 29, 2013 at 04:54:02PM +0530, Prasanna Santhanam wrote:
> Hi,
> 
> I'm hitting couple of jenkins bugs [1][2] on my build jobs.
> Jenkins.cs.o will be upgraded.
> 
> [1] https://issues.jenkins-ci.org/browse/JENKINS-13972
> [2] https://issues.jenkins-ci.org/browse/JENKINS-17337
> 
This is done. Let me know if there's any problems.

-- 
Prasanna.,

RE: 4.1 Patches on review board

Sure, thanks Prasanna.


-Original Message-
From: prasanna [mailto:srivatsav.prasa...@gmail.com] On Behalf Of Prasanna 
Santhanam
Sent: Friday, March 29, 2013 4:46 PM
To: dev@cloudstack.apache.org
Subject: Re: 4.1 Patches on review board

On Fri, Mar 29, 2013 at 10:43:12AM +, Pradeep Soundararajan wrote:
> 9946  Referenced bug is fixed Pradeep 
> Update the
> patch with commit details and mark the patch as submitted
> 
> This patch has not been applied. I have sent the request once again 
> with the latest diff to apply it.

Thanks Pradeep, Applied to master and 4.1. Be sure to remind the list when the 
patch hasn't recieved attention within a week.

--
Prasanna.,

[CI] Upgrading Jenkins

Hi,

I'm hitting couple of jenkins bugs [1][2] on my build jobs.
Jenkins.cs.o will be upgraded.

[1] https://issues.jenkins-ci.org/browse/JENKINS-13972
[2] https://issues.jenkins-ci.org/browse/JENKINS-17337

Thanks,

-- 
Prasanna.,

Re: 4.1 Patches on review board

On Fri, Mar 29, 2013 at 10:43:12AM +, Pradeep Soundararajan wrote:
> 9946  Referenced bug is fixed Pradeep 
> Update the
> patch with commit details and mark the patch as submitted
> 
> This patch has not been applied. I have sent the request once again
> with the latest diff to apply it.

Thanks Pradeep, Applied to master and 4.1. Be sure to remind the list
when the patch hasn't recieved attention within a week.

-- 
Prasanna.,

Re: Review Request: BUG-ID: CLOUDSTACK-1689

2013-03-29 Thread Hugo Trippaers


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

Ship it!


Ship It!

- Hugo Trippaers


On March 29, 2013, 10:41 a.m., Pradeep Soundararajan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9946/
> ---
> 
> (Updated March 29, 2013, 10:41 a.m.)
> 
> 
> Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, 
> and Wido den Hollander.
> 
> 
> Description
> ---
> 
> BUG-ID: CLOUDSTACK-1689
> 
> Summary: KVM agent install.sh should install ipset command
> 
> Added ipset in cloud.spec as part of agent install.
> 
> Signed-off-by: Pradeep 
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec e10ae1e 
> 
> Diff: https://reviews.apache.org/r/9946/diff/
> 
> 
> Testing
> ---
> 
> Packaging ipset properly after this change. 
> 
> 
> Thanks,
> 
> Pradeep Soundararajan
> 
>

Re: Review Request: BUG-ID: CLOUDSTACK-1689


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


Commit 5a66ec35ad08ff3313e72b54eba7e1754868eee4 in branch refs/heads/master 
from Prasanna Santhanam 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5a66ec3 ]

CLOUDSTACK-1689: KVM agent install.sh should install ipset command

Added ipset in cloud.spec as part of agent install.

Signed-off-by: Pradeep 
Signed-off-by: Prasanna Santhanam 


- ASF Subversion and Git Services


On March 29, 2013, 10:41 a.m., Pradeep Soundararajan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9946/
> ---
> 
> (Updated March 29, 2013, 10:41 a.m.)
> 
> 
> Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, 
> and Wido den Hollander.
> 
> 
> Description
> ---
> 
> BUG-ID: CLOUDSTACK-1689
> 
> Summary: KVM agent install.sh should install ipset command
> 
> Added ipset in cloud.spec as part of agent install.
> 
> Signed-off-by: Pradeep 
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec e10ae1e 
> 
> Diff: https://reviews.apache.org/r/9946/diff/
> 
> 
> Testing
> ---
> 
> Packaging ipset properly after this change. 
> 
> 
> Thanks,
> 
> Pradeep Soundararajan
> 
>

Re: Review Request: BUG-ID: CLOUDSTACK-1689


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


Commit fa47d30866f17201a74ec145c96c9bd79b022887 in branch refs/heads/4.1 from 
Prasanna Santhanam 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=fa47d30 ]

CLOUDSTACK-1689: KVM agent install.sh should install ipset command

Added ipset in cloud.spec as part of agent install.

Signed-off-by: Pradeep 
Signed-off-by: Prasanna Santhanam 


- ASF Subversion and Git Services


On March 29, 2013, 10:41 a.m., Pradeep Soundararajan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9946/
> ---
> 
> (Updated March 29, 2013, 10:41 a.m.)
> 
> 
> Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, 
> and Wido den Hollander.
> 
> 
> Description
> ---
> 
> BUG-ID: CLOUDSTACK-1689
> 
> Summary: KVM agent install.sh should install ipset command
> 
> Added ipset in cloud.spec as part of agent install.
> 
> Signed-off-by: Pradeep 
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec e10ae1e 
> 
> Diff: https://reviews.apache.org/r/9946/diff/
> 
> 
> Testing
> ---
> 
> Packaging ipset properly after this change. 
> 
> 
> Thanks,
> 
> Pradeep Soundararajan
> 
>

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!


On Mar 29, 2013, at 6:57 AM, Prasanna Santhanam  wrote:

> On Fri, Mar 29, 2013 at 06:30:50AM -0400, Sebastien Goasguen wrote:
>> 
>> On Mar 29, 2013, at 5:55 AM, prasanna  wrote:
>> 
>>> On 29 March 2013 15:16, Sebastien Goasguen  wrote:
 
 On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
 
> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
> 
> Blocker:
> CLOUDSTACK-1746
> Cloudstack Usage Server won't start
> Kishan KavalaFrancois Gaudreault
> 
> CLOUDSTACK-1836
> License header failures for ja-JP .po translation files
> sebastien goasguen   Chandan Purushothama
 
 I fixed that one yesterday night. I did not close the ticket because I was 
 waiting for the next build to run.
 Can you tell me if the RAT passed properly ?
 
>>> 
>>> Hey Sebastien, builds.a.o was down all day y'day. I see that RAT passes now
>>> https://builds.apache.org/view/A-F/view/Cloudstack/job/cloudstack-rat-41/205/
>> 
>> Ok so we are good now ?
>> 
>> I am going to be offline till Tuesday morning. So shout now if there is 
>> still an issue.
>> 
> 
> Assuming ja-JP files are handled by default RAT build (which I'm unsure of) I 
> think
> we're good. :)
> 

Yes they are, as regular files. 

> -- 
> Prasanna.,

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts


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


Commit 95fce112157526b0727821e145ec53d0e8f3e824 in branch refs/heads/master 
from Hugo Trippaers 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=95fce11 ]

CLOUDSTACK-1156 Fix small typo

- ASF Subversion and Git Services


On March 28, 2013, 8:36 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9541/
> ---
> 
> (Updated March 28, 2013, 8:36 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, Nitin Mehta, Sateesh 
> Chodapuneedi, mice xia, and Min Chen.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts
> 
> Addition of two new resource types i.e. Primary and Secondary storage 
> space in the existing pool of
> resource types.
> Added methods to set the limits on these resources using 
> updateResourceLimit
> API command and to get a count using updateResourceCount. Also added 
> calls in the
> Templates, Volumes, Snapshots life cycle to check these limits and to 
> increment/decrement the new
> resource types
> 
> Resource Name  :: Resource type number
> Primary Storage   10
> Secondary Storage 11
> 
> Also added jUnit Tests for the same.
> 
> 
> This addresses bug CLOUDSTACK-1156.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/configuration/Resource.java 7614c8a 
>   api/src/com/cloud/storage/VolumeApiService.java 8517988 
>   
> api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceCountCmd.java
>  f6d3a98 
>   
> api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceLimitCmd.java
>  0039f62 
>   api/src/org/apache/cloudstack/api/command/user/volume/ResizeVolumeCmd.java 
> 955727a 
>   api/src/org/apache/cloudstack/api/response/AccountResponse.java 9a98a35 
>   api/src/org/apache/cloudstack/api/response/ResourceCountResponse.java 
> a7fbbf2 
>   api/src/org/apache/cloudstack/api/response/ResourceLimitResponse.java 
> b444e7a 
>   
> plugins/hypervisors/baremetal/src/com/cloud/baremetal/manager/BareMetalTemplateAdapter.java
>  33725f6 
>   server/src/com/cloud/alert/AlertManagerImpl.java a45482f 
>   server/src/com/cloud/api/ApiResponseHelper.java f33601f 
>   server/src/com/cloud/api/query/dao/AccountJoinDaoImpl.java 898bafc 
>   server/src/com/cloud/api/query/vo/AccountJoinVO.java cd7231c 
>   server/src/com/cloud/configuration/Config.java 1a57a64 
>   server/src/com/cloud/resourcelimit/ResourceLimitManagerImpl.java 23c0796 
>   server/src/com/cloud/storage/VolumeManager.java af3cbbf 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 737ed0a 
>   server/src/com/cloud/storage/dao/SnapshotDao.java 0e378a7 
>   server/src/com/cloud/storage/dao/SnapshotDaoImpl.java 825b6d5 
>   server/src/com/cloud/storage/dao/VolumeDao.java d7a2667 
>   server/src/com/cloud/storage/dao/VolumeDaoImpl.java 40ed875 
>   server/src/com/cloud/storage/download/DownloadListener.java d4c20cf 
>   server/src/com/cloud/storage/download/DownloadMonitorImpl.java 0bc89e3 
>   server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java bacca01 
>   server/src/com/cloud/template/HypervisorTemplateAdapter.java 1426421 
>   server/src/com/cloud/template/TemplateManagerImpl.java d843dbc 
>   server/src/com/cloud/vm/UserVmManagerImpl.java 82a69bd 
>   server/test/com/cloud/resourcelimit/ResourceLimitManagerImplTest.java 
> d311ad3 
>   server/test/com/cloud/vpc/MockResourceLimitManagerImpl.java b9fc861 
>   setup/db/db/schema-40to410.sql 60ad025 
>   setup/db/db/schema-410to420.sql 9eed0a6 
>   utils/src/com/cloud/utils/UriUtils.java a8b5ccb 
> 
> Diff: https://reviews.apache.org/r/9541/diff/
> 
> 
> Testing
> ---
> 
> Tested life cycle of templates, volumes, snapshots, vm on my local CloudStack 
> setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>

Re: Review Request: Mulitiple ip address per nic support for advacned shared networks


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



server/src/com/cloud/network/NetworkServiceImpl.java


this check not required



server/src/com/cloud/network/NetworkServiceImpl.java


Same here.this check not required


- Murali Reddy


On March 21, 2013, 3:58 a.m., Jayapal Reddy wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10040/
> ---
> 
> (Updated March 21, 2013, 3:58 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Murali Reddy, and anthony 
> xu.
> 
> 
> Description
> ---
> 
> Changes added for supporting multiple ip address per nic for shared networks
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/network/NetworkServiceImpl.java 8303b0b 
> 
> Diff: https://reviews.apache.org/r/10040/diff/
> 
> 
> Testing
> ---
> 
> 1. Tested adding and removing the ip for nic using the APIs
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

On Fri, Mar 29, 2013 at 06:30:50AM -0400, Sebastien Goasguen wrote:
> 
> On Mar 29, 2013, at 5:55 AM, prasanna  wrote:
> 
> > On 29 March 2013 15:16, Sebastien Goasguen  wrote:
> >> 
> >> On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
> >> 
> >>> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
> >>> 
> >>> Blocker:
> >>> CLOUDSTACK-1746
> >>> Cloudstack Usage Server won't start
> >>> Kishan KavalaFrancois Gaudreault
> >>> 
> >>> CLOUDSTACK-1836
> >>> License header failures for ja-JP .po translation files
> >>> sebastien goasguen   Chandan Purushothama
> >> 
> >> I fixed that one yesterday night. I did not close the ticket because I was 
> >> waiting for the next build to run.
> >> Can you tell me if the RAT passed properly ?
> >> 
> > 
> > Hey Sebastien, builds.a.o was down all day y'day. I see that RAT passes now
> > https://builds.apache.org/view/A-F/view/Cloudstack/job/cloudstack-rat-41/205/
> 
> Ok so we are good now ?
> 
> I am going to be offline till Tuesday morning. So shout now if there is still 
> an issue.
> 

Assuming ja-JP files are handled by default RAT build (which I'm unsure of) I 
think
we're good. :)

-- 
Prasanna.,

RE: 4.1 Patches on review board

9946Referenced bug is fixed Pradeep 
Update the patch with commit details and mark the patch as submitted

This patch has not been applied. I have sent the request once again with the 
latest diff to apply it.

Thanks,
Pradeep S

-Original Message-
From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] 
Sent: Friday, March 29, 2013 11:41 AM
To: dev@cloudstack.apache.org
Subject: 4.1 Patches on review board

I was reviewing the patches on the review board and following 4.1 patch needs 
some action from the submitters? Kelven, Kishan, Pradeep please review your 
action items.


Review# Status  Who to take Action  
Action Item
9860Committed   Kelven  
Mark as submitted
9888Referenced bug is fixed Kelven  
Is the patch still needed   
10145   Committed in 4.1Kelven  
Mark as submitted and commit changes to master  
10059   Not applied, pending submitter response Kishan  Is this 
needed for 4.1  
9946Referenced bug is fixed Pradeep 
Update the patch with commit details and mark the patch as submitted


Thanks
Animesh

Re: Review Request: BUG-ID: CLOUDSTACK-1689


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

(Updated March 29, 2013, 10:41 a.m.)


Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, and 
Wido den Hollander.


Changes
---

I have noticed this patch has not been applied. Please apply it.


Description
---

BUG-ID: CLOUDSTACK-1689

Summary: KVM agent install.sh should install ipset command

Added ipset in cloud.spec as part of agent install.

Signed-off-by: Pradeep 


Diffs (updated)
-

  packaging/centos63/cloud.spec e10ae1e 

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


Testing
---

Packaging ipset properly after this change. 


Thanks,

Pradeep Soundararajan

Re: Jetty Failing on latest master due to class UCS manager not Found

On 29/03/13 12:45 AM, "Chiradeep Vittal" 
wrote:

>For optional dependencies, JSR330 injection requires using setter
>injection.
>You should write
>
>@Inject void setGslbProvider(GslbProvider provider) {
>  this._gslbProvider = provider;
>}

Thanks Chiradeep for suggestion. But due to Spring default behaviour with
@Inject, both setter and interface injection fails if there is no
implementation. 

For now, I am just initializing through the bean property configured in
componentContext.xml and nonossComponentContext.xml.

>
>
>
>On 3/27/13 8:36 PM, "Murali Reddy"  wrote:
>
>>On 28/03/13 9:00 AM, "Rajesh Battala"  wrote:
>>
>>>Jetty is failing to start on latest master building oss.
>>>Am on the latest master, building oss and starting jetty is failing with
>>>below error. 
>>>
>>>Error creating bean with name 'globalLoadBalancingRulesServiceImpl':
>>>Injection of autowired dependencies failed; nested exception is
>>>org.springframewor
>>>  k.beans.factory.BeanCreationException: Could not autowire field:
>>>protected org.apache.cloudstack.region.
>>>   gslb.GslbServiceProvider
>>>org.apache.cloudstack.region.gslb.GlobalLoadBalancingRulesServiceImpl._g
>>>s
>>>l
>>>bProv  ider;
>>>nested exception is
>>>org.springframework.beans.factory.NoSuchBeanDefinitionException: No
>>>matching b  ean
>>>of type [org.apache.cloudstack.region.gslb.GslbServiceProvider] found
>>>for
>>>dependency: expected at le
>>>  ast 1 bean which qualifies as autowire candidate for this
>>>dependency. Dependency annotations: {@javax.in
>>>  ject.Inject()}
>>>at 
>>>org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPost
>>>P
>>>r
>>>ocessor.postProcess
>>>   PropertyValues(AutowiredAnnotationBeanPostProcessor.java:287)
>>>at org.springframe
>>
>>Please pull latest master, I pushed a fix.
>>
>
>

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!


On Mar 29, 2013, at 5:55 AM, prasanna  wrote:

> On 29 March 2013 15:16, Sebastien Goasguen  wrote:
>> 
>> On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
>> 
>>> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
>>> 
>>> Blocker:
>>> CLOUDSTACK-1746
>>> Cloudstack Usage Server won't start
>>> Kishan KavalaFrancois Gaudreault
>>> 
>>> CLOUDSTACK-1836
>>> License header failures for ja-JP .po translation files
>>> sebastien goasguen   Chandan Purushothama
>> 
>> I fixed that one yesterday night. I did not close the ticket because I was 
>> waiting for the next build to run.
>> Can you tell me if the RAT passed properly ?
>> 
> 
> Hey Sebastien, builds.a.o was down all day y'day. I see that RAT passes now
> https://builds.apache.org/view/A-F/view/Cloudstack/job/cloudstack-rat-41/205/

Ok so we are good now ?

I am going to be offline till Tuesday morning. So shout now if there is still 
an issue.

thanks,

-Sebastien

Re: [DOCS] Can't build Japanese docs


On Mar 29, 2013, at 6:25 AM, Hiroaki KAWAI  wrote:

> 
> (2013/03/28 20:38), Sebastien Goasguen wrote:
>>> I fixed those on transifex. Please feel free to tell me if you still
>>> have trouble in building ja-JP docs. :-)
>> 
>> thanks, I pulled all new translations. I am not sure what you did in 
>> transifex but it did not fix the issue :)
> -snip-
> 
> There was an invalid UTF-8 sequence and invalid xml markup
> in the translations.
> 
> -snip-
>> I ended fixing it in remove-vpx.xml , the  tags need to be on a new 
>> line…
> -snip-
> 
> That's strange. I could successfully run publican. As it is an xml,
> new line should not matter...
> 
> What I did:
> cd docs
> tx pull -l ja_JP
> publican build --formats xml --langs ja_JP \
>  --config publican-installation.cfg

ah yes, but --formats=pdf was failing. Should have been more precise in the bug 
report.


> 
> -snip-
>> Feel free to have a look at the runbook in transifex, it needs a little more 
>> love in Japanese (couple hours of work :) ).
> -snip-
> 
> ok :-)
> 
>> -sebastien
>> 
>> 
>>> 
>>> Thanks
>>> 
>>> (2013/03/24 19:28), Sebastien Goasguen wrote:
 I filed https://issues.apache.org/jira/browse/CLOUDSTACK-1796
 
 I can't publican build japanese docs. I managed to fix chinese and 
 portuguese.
 
 But I cannot figure out how to edit the problematic string in the .po file 
 to solve the issue.
 
 Anyone cares to take a look ?
 
 -sebastien
 
>>> 
>> 
>

[ACS41][Patch Request]

commit 48311363d600264cb3542992a615706cb92f90ad
Author: Murali Reddy 
Date:   Fri Mar 29 15:53:21 2013 +0530

CLOUDSTACK-1834: Events are not generated for registerUserKeys(),
Enabling account and Editing account.

Re: [DOCS] Can't build Japanese docs

2013-03-29 Thread Hiroaki KAWAI



(2013/03/28 20:38), Sebastien Goasguen wrote:

I fixed those on transifex. Please feel free to tell me if you still
have trouble in building ja-JP docs. :-)


thanks, I pulled all new translations. I am not sure what you did in transifex 
but it did not fix the issue :)

-snip-

There was an invalid UTF-8 sequence and invalid xml markup
in the translations.

-snip-

I ended fixing it in remove-vpx.xml , the  tags need to be on a new line…

-snip-

That's strange. I could successfully run publican. As it is an xml,
new line should not matter...

What I did:
 cd docs
 tx pull -l ja_JP
 publican build --formats xml --langs ja_JP \
  --config publican-installation.cfg

-snip-

Feel free to have a look at the runbook in transifex, it needs a little more 
love in Japanese (couple hours of work :) ).

-snip-

ok :-)


-sebastien




Thanks

(2013/03/24 19:28), Sebastien Goasguen wrote:

I filed https://issues.apache.org/jira/browse/CLOUDSTACK-1796

I can't publican build japanese docs. I managed to fix chinese and portuguese.

But I cannot figure out how to edit the problematic string in the .po file to 
solve the issue.

Anyone cares to take a look ?

-sebastien

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

On Fri, Mar 29, 2013 at 11:57:17AM +0530, Prasanna Santhanam wrote:
> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
> 
> CLOUDSTACK-1841 
> ASF 4.0 to 4.1 Upgrade: Missing Few Global Configuration parameters on the
> Upgraded Setup
> Unassigned   Chandan Purushothama 

Sent a patch request for this.

-- 
Prasanna.,

Re: Review Request: Feature Syslog Enhancements

2013-03-29 Thread Sateesh Chodapuneedi


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

Ship it!


Branch: refs/heads/master
Commit: 6a7156ad9af6ca173c0b685d69d5724d91d0a401
Parents: 2c176ab
Author: Anshul Gangwar 
Authored: Wed Mar 6 14:11:47 2013 +0530
Committer: Sateesh Chodapuneedi 
Committed: Fri Mar 29 09:53:05 2013 +0530


- Sateesh Chodapuneedi


On March 29, 2013, 9:50 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9870/
> ---
> 
> (Updated March 29, 2013, 9:50 a.m.)
> 
> 
> Review request for cloudstack, Rajesh Battala, Murali Reddy, and Sateesh 
> Chodapuneedi.
> 
> 
> Description
> ---
> 
> This patch implements the Syslog Enhancements in CloudStack. With this 
> feature one can configure log4j-cloud.xml to get the Syslog messages for 
> alerts. This feature is implemented by extending log4j Appeneder. Multiple 
> Syslog Hosts can be specified. To configure multiple Syslog Hosts one needs 
> to modify following portion in log4j-cloud.xml 
> class="org.apache.cloudstack.syslog.AlertsSyslogAppender">
>   
>   
>   
>   
>  
>   
>
> 
> To specify multiple  Syslog Hosts one has to give as follows in above code 
> fragment with each Syslog Hosts separated by ,
> 
> class="org.apache.cloudstack.syslog.AlertsSyslogAppender">
>   
>   
>   
>   
>  
>   
>
> 
> 
> This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-712.
> 
> 
> Diffs
> -
> 
>   client/pom.xml 23892e1 
>   client/tomcatconf/log4j-cloud.xml.in ce4079f 
>   plugins/alert-handlers/syslog-alerts/pom.xml PRE-CREATION 
>   
> plugins/alert-handlers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java
>  PRE-CREATION 
>   
> plugins/alert-handlers/syslog-alerts/test/org/apache/cloudstack/syslog/AlertsSyslogAppenderTest.java
>  PRE-CREATION 
>   plugins/pom.xml 39d9907 
> 
> Diff: https://reviews.apache.org/r/9870/diff/
> 
> 
> Testing
> ---
> 
> Units tests for functionality are written and manual testing for syslog 
> message generation for few alerts 
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.



> On March 29, 2013, 5:52 a.m., Prasanna Santhanam wrote:
> > setup/db/create-schema.sql, line 2370
> > 
> >
> > Kawai-san, If this is applied to master, I think this section should go 
> > in the schema-40to410.sql, instead of create-schema. Changes to the 4.0 
> > schema after the database creator was introduced shouldn't be required.
> 
> Hiroaki Kawai wrote:
> This patch was created for 4.0 branch. So where to put the sql seems to 
> depend on when to merge this patch. Do you think this patch should be in 
> master, not 4.0? If so, I'll rewrite the patch again.

That should be okay then. Since the bug exists on all versions just wanted to 
call out that the fix differs when applied to master, 4.1. For 4.0 this is 
fine. Once the server side code is reviewed master/4.1 patch can be made. Thanks


- Prasanna


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


On March 29, 2013, 1:49 a.m., Hiroaki Kawai wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9871/
> ---
> 
> (Updated March 29, 2013, 1:49 a.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
> 
> 
> Description
> ---
> 
> The location of the virtual machine is provided by DeployDestination, which 
> will be passed in NetworkGuru#reserve and NetworkElement#prepare. 
> 
> During the virtual machine migration, it actually changes DeployDestination 
> and it looks like that it will tell that event to network components as it 
> has NetworkManager#prepareNicForMigration. The problem is that althogh the 
> interface has that method, NetworkManagerImpl does not tell the 
> DeployDestination changes to network components. 
> 
> So IMHO, we need to add calls of NetworkGuru#reserve and 
> NetworkElement#prepare in NetworkManagerImpl#prepareNicForMigration . And 
> then, we also need to add calls NetworkGuru#release and 
> NetworkElement#release after the migration, otherwise the network resources 
> that plugin reserved will be kept even when the vm leaves off.
> 
> Created a first minimum patch to show the concept.
> 
> 
> This addresses bug CLOUDSTACK-1638.
> 
> 
> Diffs
> -
> 
>   docs/en-US/plugin-niciranvp-tables.xml 4f81655 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/NiciraNvpNicMappingVO.java
>  0779e69 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/element/NiciraNvpElement.java
>  1fcccdb 
>   server/src/com/cloud/network/NetworkManager.java 4124b19 
>   server/src/com/cloud/network/NetworkManagerImpl.java a98bdd4 
>   server/src/com/cloud/network/guru/ControlNetworkGuru.java 934cd70 
>   server/src/com/cloud/network/guru/DirectNetworkGuru.java ee824af 
>   server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java 354d7ed 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java 24d24f8 
>   server/src/com/cloud/network/guru/GuestNetworkGuru.java cebfb08 
>   server/src/com/cloud/network/guru/PodBasedNetworkGuru.java b513325 
>   server/src/com/cloud/network/guru/StorageNetworkGuru.java 879d0cd 
>   server/src/com/cloud/vm/VirtualMachineManagerImpl.java 9230f4a 
>   setup/db/create-schema.sql 5b6dc04 
> 
> Diff: https://reviews.apache.org/r/9871/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Hiroaki Kawai
> 
>

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!

On 29 March 2013 15:16, Sebastien Goasguen  wrote:
>
> On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:
>
>> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
>>
>> Blocker:
>> CLOUDSTACK-1746
>> Cloudstack Usage Server won't start
>> Kishan KavalaFrancois Gaudreault
>>
>> CLOUDSTACK-1836
>> License header failures for ja-JP .po translation files
>> sebastien goasguen   Chandan Purushothama
>
> I fixed that one yesterday night. I did not close the ticket because I was 
> waiting for the next build to run.
> Can you tell me if the RAT passed properly ?
>

Hey Sebastien, builds.a.o was down all day y'day. I see that RAT passes now
https://builds.apache.org/view/A-F/view/Cloudstack/job/cloudstack-rat-41/205/

Re: Review Request: Feature Syslog Enhancements

2013-03-29 Thread Anshul Gangwar


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

(Updated March 29, 2013, 9:50 a.m.)


Review request for cloudstack, Rajesh Battala, Murali Reddy, and Sateesh 
Chodapuneedi.


Changes
---

rebased with current master


Description
---

This patch implements the Syslog Enhancements in CloudStack. With this feature 
one can configure log4j-cloud.xml to get the Syslog messages for alerts. This 
feature is implemented by extending log4j Appeneder. Multiple Syslog Hosts can 
be specified. To configure multiple Syslog Hosts one needs to modify following 
portion in log4j-cloud.xml 
   
  
  
  
  
 
  
   

To specify multiple  Syslog Hosts one has to give as follows in above code 
fragment with each Syslog Hosts separated by ,

   
  
  
  
  
 
  
   


This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-712.


Diffs (updated)
-

  client/pom.xml 23892e1 
  client/tomcatconf/log4j-cloud.xml.in ce4079f 
  plugins/alert-handlers/syslog-alerts/pom.xml PRE-CREATION 
  
plugins/alert-handlers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java
 PRE-CREATION 
  
plugins/alert-handlers/syslog-alerts/test/org/apache/cloudstack/syslog/AlertsSyslogAppenderTest.java
 PRE-CREATION 
  plugins/pom.xml 39d9907 

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


Testing
---

Units tests for functionality are written and manual testing for syslog message 
generation for few alerts 


Thanks,

Anshul Gangwar

Re: [ACS41] Today's 4.1 blocker and crit list! Go get 'em!


On Mar 29, 2013, at 2:27 AM, Prasanna Santhanam  wrote:

> Here's the latest Blockers and criticals (UTC +0530, Mar 29)
> 
> Blocker:
> CLOUDSTACK-1746 
> Cloudstack Usage Server won't start
> Kishan KavalaFrancois Gaudreault 
> 
> CLOUDSTACK-1836 
> License header failures for ja-JP .po translation files
> sebastien goasguen   Chandan Purushothama

I fixed that one yesterday night. I did not close the ticket because I was 
waiting for the next build to run.
Can you tell me if the RAT passed properly ?

> 
> 
> Critical:
> CLOUDSTACK-1846
> KVM - storage pools can silently fail to be unregistered, leading to failure 
> to
> register later
> Marcus Sorensen 
> 
> CLOUDSTACK-1845 
> KVM - storage migration often fails
> Marcus Sorensen
> 
> CLOUDSTACK-1842 
> ASF 4.0 to 4.1 Upgrade: Missing Ubuntu 12.04 Guest OS Types on the Upgraded
> Setup
> Unassigned   Chandan Purushothama
> 
> CLOUDSTACK-1841 
> ASF 4.0 to 4.1 Upgrade: Missing Few Global Configuration parameters on the
> Upgraded Setup
> Unassigned   Chandan Purushothama 
> 
> CLOUDSTACK-1839 
> Upgrade 4.0 -> 4.1 - Upgraded DB has lot more keys and indexes for many tables
> compare to the fresh installed 4.1 DB.
> Rohit Yadav  Sangeetha Hariharan 
> CLOUDSTACK-1834 
> Events are not generated for registerUserKeys(), Enabling account and Editing
> account.
> Murali Reddy Sangeetha Hariharan 
> 
> CLOUDSTACK-1827  (This *may* already have been fixed by Sheng)
> Redundant router - When VR Master was stopped failover to VR Backup did not
> occur
> Sheng Yang   angeline shen
> 
> CLOUDSTACK-1819 
> AWS Regions - Issues seen when trying to move a zone from 1 region to another.
> Kishan KavalaSangeetha Hariharan
> 
> CLOUDSTACK-1805 
> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after long time 
> of
> inactivity resulting in not being able to log in to thr management server.
> Kelven Yang  Sangeetha Hariharan
> 
> 
> On Thu, Mar 28, 2013 at 05:05:22PM -0400, Chip Childers wrote:
>> Hi all,
>> 
>> Here's the list as of right now (and we are still pending a clear
>> statement from Sudha / others that upgrade testing has been completed).
>> 
>> Shout if you want help, or if there is a pending patch that I missed!
>> 
>> Blocker:
>> CLOUDSTACK-1836 
>> License header failures for ja-JP .po translation files 
>> sebastien goasguen
>> 
>> Critical:
>> CLOUDSTACK-1805 
>> com.mysql.jdbc.exceptions.jdbc4.CommunicationsException seen after long time 
>> of inactivity resulting in not being able to log in to thr management 
>> server.  
>> Kelven Yang
>> 
>> CLOUDSTACK-1819 
>> AWS Regions - Issues seen when trying to move a zone from 1 region to 
>> another.  
>> Kishan Kavala
>> 
>> CLOUDSTACK-1834 
>> Events are not generated for registerUserKeys(), Enabling account and 
>> Editing account.  
>> Murali Reddy
>> 
>> -chip
> 
> -- 
> Prasanna.,

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-03-29 Thread Hiroaki Kawai



> On March 29, 2013, 5:52 a.m., Prasanna Santhanam wrote:
> > setup/db/create-schema.sql, line 2370
> > 
> >
> > Kawai-san, If this is applied to master, I think this section should go 
> > in the schema-40to410.sql, instead of create-schema. Changes to the 4.0 
> > schema after the database creator was introduced shouldn't be required.

This patch was created for 4.0 branch. So where to put the sql seems to depend 
on when to merge this patch. Do you think this patch should be in master, not 
4.0? If so, I'll rewrite the patch again.


- Hiroaki


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


On March 29, 2013, 1:49 a.m., Hiroaki Kawai wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9871/
> ---
> 
> (Updated March 29, 2013, 1:49 a.m.)
> 
> 
> Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
> 
> 
> Description
> ---
> 
> The location of the virtual machine is provided by DeployDestination, which 
> will be passed in NetworkGuru#reserve and NetworkElement#prepare. 
> 
> During the virtual machine migration, it actually changes DeployDestination 
> and it looks like that it will tell that event to network components as it 
> has NetworkManager#prepareNicForMigration. The problem is that althogh the 
> interface has that method, NetworkManagerImpl does not tell the 
> DeployDestination changes to network components. 
> 
> So IMHO, we need to add calls of NetworkGuru#reserve and 
> NetworkElement#prepare in NetworkManagerImpl#prepareNicForMigration . And 
> then, we also need to add calls NetworkGuru#release and 
> NetworkElement#release after the migration, otherwise the network resources 
> that plugin reserved will be kept even when the vm leaves off.
> 
> Created a first minimum patch to show the concept.
> 
> 
> This addresses bug CLOUDSTACK-1638.
> 
> 
> Diffs
> -
> 
>   docs/en-US/plugin-niciranvp-tables.xml 4f81655 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/NiciraNvpNicMappingVO.java
>  0779e69 
>   
> plugins/network-elements/nicira-nvp/src/com/cloud/network/element/NiciraNvpElement.java
>  1fcccdb 
>   server/src/com/cloud/network/NetworkManager.java 4124b19 
>   server/src/com/cloud/network/NetworkManagerImpl.java a98bdd4 
>   server/src/com/cloud/network/guru/ControlNetworkGuru.java 934cd70 
>   server/src/com/cloud/network/guru/DirectNetworkGuru.java ee824af 
>   server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java 354d7ed 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java 24d24f8 
>   server/src/com/cloud/network/guru/GuestNetworkGuru.java cebfb08 
>   server/src/com/cloud/network/guru/PodBasedNetworkGuru.java b513325 
>   server/src/com/cloud/network/guru/StorageNetworkGuru.java 879d0cd 
>   server/src/com/cloud/vm/VirtualMachineManagerImpl.java 9230f4a 
>   setup/db/create-schema.sql 5b6dc04 
> 
> Diff: https://reviews.apache.org/r/9871/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Hiroaki Kawai
> 
>

RE: MASTER NONOSS BUILD FAILURE

Please find the surefire report 
https://www.dropbox.com/s/vyq610sdqzs7pt9/surefire-reports.zip

As discussed, I will create a new node for master-nonoss build in Jenkins.cs.o 
once I am free. 

Thanks,
Pradeep S

-Original Message-
From: prasanna [mailto:srivatsav.prasa...@gmail.com] On Behalf Of Prasanna 
Santhanam
Sent: Friday, March 29, 2013 12:36 PM
To: dev@cloudstack.apache.org
Subject: Re: MASTER NONOSS BUILD FAILURE

This will need the stacktrace from the maven-surefire report in server. Can you 
post those?

Also - there should've been a job on jenkins.cs.o looking at the nonoss build. 
Can you refresh that with your latest nonoss jars? I've given you access on 
jenkins. If the job's running it'll be easier to track these build breaks.

--
Prasanna.,

On Fri, Mar 29, 2013 at 06:50:24AM +, Pradeep Soundararajan wrote:
> [INFO] Apache CloudStack Server .. FAILURE 
> [36.121s]
> 
> 
> [ERROR] Please refer to 
> /root/jenkins/build/workspace/ASF-build-master-nonoss-rhel63/server/target/surefire-reports
>  for the individual test results.
>

[OFFLINE] 30March-30April

2013-03-29 Thread Rohit Yadav

Taking time off, will check ML if possible and work on few blockers. From
now onwards, I would only be able to work on ACS in my free time.
Pl. email me on bhais...@baagi.org for something urgent.

I'll be only following dev ML from now.

Cheers.

Re: Review Request: CLOUDSTACK-1592: [UI] Add support to limit newly added resourcetypes


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


Commit 2c176ab9ea6216b7c9da25ce0b0d2b34cbc2a776 in branch refs/heads/master 
from Pranav Saxena 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2c176ab ]

CLOUDSTACK-1592:[UI] Add support to limit newly added resourcetypes


- ASF Subversion and Git Services


On March 29, 2013, 8:35 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9816/
> ---
> 
> (Updated March 29, 2013, 8:35 a.m.)
> 
> 
> Review request for cloudstack, Pranav Saxena, Brian Federle, and Jessica Wang.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-1592: [UI] Add support to limit newly added resourcetypes.
> 
> This is the UI implementation for limiting resources feature. This will 
> display and update resources (CPU, Memory, Primary storage, Secondary storage 
> space) limits in the UI.
> 
> 
> This addresses bug CLOUDSTACK-1592.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 66b32ac 
>   ui/dictionary.jsp cd8d732 
>   ui/scripts/accounts.js 3403337 
>   ui/scripts/domains.js 991e37d 
>   ui/scripts/projects.js b62dcb4 
> 
> Diff: https://reviews.apache.org/r/9816/diff/
> 
> 
> Testing
> ---
> 
> Tests:
> 1. create a account and go to detail view, user will get the Max resource 
> count for the new resourcetypes along with the previously added resource 
> types.
> 2. create a domain and go to detail view, admin will get the details of all 
> the resourcestypes.
> 3. deploy a VM, resource count will update accordingly.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>

Re: Review Request: CLOUDSTACK-1592: [UI] Add support to limit newly added resourcetypes

2013-03-29 Thread Sanjay Tripathi


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

(Updated March 29, 2013, 8:35 a.m.)


Review request for cloudstack, Pranav Saxena, Brian Federle, and Jessica Wang.


Changes
---

Resolved conflicts with the latest code.


Description
---

CLOUDSTACK-1592: [UI] Add support to limit newly added resourcetypes.

This is the UI implementation for limiting resources feature. This will display 
and update resources (CPU, Memory, Primary storage, Secondary storage space) 
limits in the UI.


This addresses bug CLOUDSTACK-1592.


Diffs (updated)
-

  client/WEB-INF/classes/resources/messages.properties 66b32ac 
  ui/dictionary.jsp cd8d732 
  ui/scripts/accounts.js 3403337 
  ui/scripts/domains.js 991e37d 
  ui/scripts/projects.js b62dcb4 

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


Testing
---

Tests:
1. create a account and go to detail view, user will get the Max resource count 
for the new resourcetypes along with the previously added resource types.
2. create a domain and go to detail view, admin will get the details of all the 
resourcestypes.
3. deploy a VM, resource count will update accordingly.


Thanks,

Sanjay Tripathi

Re: Feature Request: Password Reset / Change UI

On 29 March 2013 13:39, prasanna  wrote:
> On 29 March 2013 12:49, Maurice Lawler  wrote:
>> Hello Gang,
>>
>> I was wondering short of rendering this with the API. Is there plans to 
>> allow end users on the CS UI to be permitted to easily change their login 
>> password ?
>>
>> I noticed at this time I (being the admin) can change them, but someone was 
>> asking me; if this would be implemented in future updates.
>>
>> Thank you for your time.
>>
>> Maurice
>
> There were several requests and JIRA tickets for this. I vaguely
> recall seeing a patch on reviewboard as well.

See https://issues.apache.org/jira/browse/CLOUDSTACK-648

Re: Feature Request: Password Reset / Change UI

On 29 March 2013 12:49, Maurice Lawler  wrote:
> Hello Gang,
>
> I was wondering short of rendering this with the API. Is there plans to allow 
> end users on the CS UI to be permitted to easily change their login password ?
>
> I noticed at this time I (being the admin) can change them, but someone was 
> asking me; if this would be implemented in future updates.
>
> Thank you for your time.
>
> Maurice

There were several requests and JIRA tickets for this. I vaguely
recall seeing a patch on reviewboard as well.

[ACS41][Patch Request]

commit d13c18516a30d035e2251d187979e805670aab81
Author: Prasanna Santhanam 
Date:   Fri Mar 29 13:25:20 2013 +0530

CLOUDSTACK-1841: Missing `cloud`.`configuration` stmts in upgrade

Following global configurations introduced b/w 3.0.2 and 4.1 were
missed.

concurrent.snapshots.threshold.perhost | NULL
network.ipv6.search.retry.max | 1
traffic.sentinel.exclude.zones |
traffic.sentinel.include.zones | EXTERNAL

Signed-off-by: Prasanna Santhanam

Assigned IPs

2013-03-29 Thread Maurice Lawler

In the account setup in CS it states allotted IPs allowed. Is it possible to 
automatically from a template made to pull two IPs from the pool. 

I would think simply make a ifcfg-eth0:1 with DHCP settings would accomplish 
this.

Am I wrong in my thinking?

[ACS42][QA]Test Plan for "Multiple IP Ranges support"

2013-03-29 Thread Sanjeev Neelarapu

Hi,

I have identified test cases for "Multiple IP Ranges support" in cloudstack and 
the test plan is @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+IP+Ranges+support

Please review and let me know your feedback.

Reference FS:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+Ip+ranges+FS

Thanks,
Sanjeev

[ACS41][QA] Issues with upgrade from 4.0 to 4.1 (Ubuntu 12.04)

2013-03-29 Thread Sailaja Mada

Hi,

I tried upgrade from 4.0 to 4.1 with Management server installed on Ubuntu 
12.04 .

Instructions are : 1) 
http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.1-incubating/html/Installation_Guide/sect-source-builddebs.html
  2) 
http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.1-incubating/html/Release_Notes/upgrade-instructions.html#upgrade-from-4.0.0-to-4.0.1



1.   Downloaded deb packages from 
http://jenkins.cloudstack.org/job/package-deb-4.1/

2.   Update APT repo:

sudo mkdir -p /var/www/cloudstack/repo/binary

sudo cp *.deb /var/www/cloudstack/repo/binary

sudo cd /var/www/cloudstack/repo/binary

sudo dpkg-scanpackages . /dev/null | tee Packages | gzip -9 > Packages.gz

3.   vi /etc/apt/sources.list.d/cloudstack.list and add below line

deb http://server.url/cloudstack/repo/binary ./

4.   sudo apt-get update

5.   apt-get upgrade cloud-*

6.   service cloud-management start



Observations :



1.   No packages got updated to 4.1

2.   Management server as 4.0 server and there is no execution of SQL 
scripts to update schema



Please share your thoughts if there are any instructions got missed.



Thanks,

Sailaja.M

Feature Request: Password Reset / Change UI

2013-03-29 Thread Maurice Lawler

Hello Gang,

I was wondering short of rendering this with the API. Is there plans to allow 
end users on the CS UI to be permitted to easily change their login password ?

I noticed at this time I (being the admin) can change them, but someone was 
asking me; if this would be implemented in future updates.

Thank you for your time.

Maurice

Re: MASTER NONOSS BUILD FAILURE