Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/ --- (Updated June 11, 2013, 6:38 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/ --- (Updated June 11, 2013, 6:33 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread bharat kumar
> On June 11, 2013, 5:22 a.m., Koushik Das wrote: > > server/src/com/cloud/configuration/ConfigurationManagerImpl.java, line 3083 > > > > > > if result_final == false, why is txn committed? i missed the txn.rollback

Re: Review Request: Updated account and domain id for nic secondary ips for shared networks

2013-06-10 Thread Jayapal Reddy
> On June 4, 2013, 5:47 a.m., Murali Reddy wrote: > > Should set the owner to be entity owner passed as ipowner instead of caller. Updated to ipOwner - Jayapal --- This is an automatically generated e-mail. To reply, visit: https://revi

Re: Review Request: Updated account and domain id for nic secondary ips for shared networks

2013-06-10 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11458/ --- (Updated June 11, 2013, 6:24 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread bharat kumar
> On June 11, 2013, 5:19 a.m., Koushik Das wrote: > > server/src/com/cloud/configuration/ConfigurationManagerImpl.java, line 3037 > > > > > > Instead of using _accountDao.findById(Account.ACCOUNT_ID_SYSTEM) why > > d

Re: Object_Store storage refactor GoToMeeting Tomorrow

2013-06-10 Thread Min Chen
It is 11th June. John is not free between 9:15am to 10am, that is why we schedule it at 10:30am. Thanks -min On 6/10/13 10:52 PM, "Nitin Mehta" wrote: >Hi Min, >When you say tomorrow, what date is it 11th June or 12th ? Can the time be >preponed by an hour please - its too late here ? > >Thanks

Re: Object_Store storage refactor GoToMeeting Tomorrow

2013-06-10 Thread Nitin Mehta
Hi Min, When you say tomorrow, what date is it 11th June or 12th ? Can the time be preponed by an hour please - its too late here ? Thanks, -Nitin On 11/06/13 11:06 AM, "Min Chen" wrote: >Hi there, > >To reach consensus on some remaining NFS cache issues on object_store >storage refactor work i

RE: Build failed in Jenkins: cloudstack-rat-master #1468

2013-06-10 Thread Animesh Chaturvedi
Seeing this again, shouldn't jenkins just pull and compile? > -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Thursday, June 06, 2013 6:06 AM > To: dev@cloudstack.apache.org > Subject: Re: Build failed in Jenkins: cloudstack-rat-master #1468 > > Why is jenkins tryin

Object_Store storage refactor GoToMeeting Tomorrow

2013-06-10 Thread Min Chen
Hi there, To reach consensus on some remaining NFS cache issues on object_store storage refactor work in a more effective manner, John, Edison and I have scheduled a GoToMeeting tomorrow to discuss them over the phone, any interested parties are welcome to join and brainstorm. Here are detailed

Re: [DISCUSS] We have a videographer. What videos do we want to make?

2013-06-10 Thread Mike Tutkowski
Hi Jessica, It looks like the presentation I did in San Francisco last month is not yet available on Build a Cloud. I would expect it to be up soon. Just let me know if you'd like me to do something here. Thanks! On Mon, Jun 10, 2013 at 7:44 PM, Jessica Tomechak < jessica.tomec...@gmail.com>

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/#review21692 --- server/src/com/cloud/configuration/ConfigurationManagerImpl.java

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/#review21688 --- server/src/com/cloud/configuration/ConfigurationManagerImpl.java

Re: Review Request: Cloudstack-2511 Multiple_Ip_Ranges: Adding guest ip range in subset/superset to existing CIDR is allowed https://issues.apache.org/jira/browse/CLOUDSTACK-2511, Cloudstack-2651 [Sha

2013-06-10 Thread bharat kumar
> On June 10, 2013, 4:23 p.m., Koushik Das wrote: > > Don't combine multiple fixes in a single commit. Keep it simple -> 1 commit > > = 1 bug fix These issues are closely related to each other and so i think combining them is not bad - bharat ---

RE: Contributing as a non-committer

2013-06-10 Thread Soheil Eizadi
This was a good resource, I added a section for Remote Debugging to the Wiki. -Soheil From: Alex Huang [alex.hu...@citrix.com] Sent: Monday, June 10, 2013 8:00 PM To: dev@cloudstack.apache.org Subject: RE: Contributing as a non-committer Paul, That line is

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
Also, if we are good with Edison merging my code into his branch before going into master, I am good with that. We can remove the StoragePoolType.Dynamic code after his merge and we can deal with Burst IOPS then, as well. On Mon, Jun 10, 2013 at 9:08 PM, Mike Tutkowski < mike.tutkow...@solidfire

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
Let me make sure I follow where we're going here: 1) There should be NO references to hypervisor code in the storage plug-ins code (this includes the default storage plug-in, which currently sends several commands to the hypervisor in use (although it does not know which hypervisor (XenServer, ESX

RE: Contributing as a non-committer

2013-06-10 Thread Alex Huang
> Forget about eclipse for now :) just use vi :) Why don't we just go back to ed? --Alex

RE: Contributing as a non-committer

2013-06-10 Thread Alex Huang
Paul, That line is old. You should follow this page. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Setting+up+CloudStack+Development+Environment --Alex > -Original Message- > From: Paul Angus [mailto:paul.an...@shapeblue.com] > Sent: Monday, June 10, 2013 6:41 AM > To: dev@clo

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
"+1 -- Burst IOPS can be implemented while avoiding implementation attributes. I always wondered about the details field. I think we should beef up the description in the documentation regarding the expected format of the field. In 4.1, I noticed that the details are not returned on the createSt

Re: [DISCUSS] We have a videographer. What videos do we want to make?

2013-06-10 Thread Jessica Tomechak
Mike, thanks for the idea! A pointer to the existing presentation video would be great. Maybe Gregg can edit that together with animation, or demo footage from our lab to show the corresponding hardware setup...just brainstorming...Gregg is likely to have ideas of his own. This is in addition to t

Re: [GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Nguyen Anh Tu
Hi, I'll look around this bug. 2013/6/11 Chiradeep Vittal > Yes > > On 6/10/13 1:12 PM, "Sebastien Goasguen" wrote: > > > > >On Jun 10, 2013, at 3:07 PM, Chiradeep Vittal > > wrote: > > > >> The check for the config is incorrect. The check should be whether the > >> network in the input param

Re: Review Request: use commons-lang StringUtils

2013-06-10 Thread Alex Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11767/#review21678 --- The patch did not apply cleanly. Please resubmit. Since you have t

RE: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Edison Su
> -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Monday, June 10, 2013 4:07 PM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE] disk_io_throttling to MASTER > > That's a good point about the "managed" property being storable in the > storage

Re: Review Request: The equals method checks type against Pair

2013-06-10 Thread Alex Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11752/#review21676 --- Ship it! Commit ID on master: a1a68ed0c4b722f6f83733d0b110debde2a10

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
That's a good point about the "managed" property being storable in the storage_pool_details table. We don't need another column for it in the storage_pool table. In the current url field is where this kind of information can be passed along and it can be stored in the storage_pool_details table, if

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, From a CloudStack perspective, it will keep implementation specific concepts from the base data model, and provide a great test case to develop a mechanism to capture this information in 4.3. Ideally, I want CloudStack to exploit these implementation specific features. I just want to pr

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
I am pretty sure it is not a blocker for me to drop Burst IOPS. Let me run that past Product Management, but I don't think it's a problem. On Mon, Jun 10, 2013 at 4:01 PM, John Burwell wrote: > Mike, > > Yes. I realize my other reply did not explicitly state leaving the Min > and Max IOPS fiel

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
I see the unmanaged() method in the life cycle object now (didn't know this was there before). I also notice it is not being invoked anywhere. I can make use of this method to help the storage framework determine if it should send a message to the hypervisor in use to create an SR (or datastore fo

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, Yes. I realize my other reply did not explicitly state leaving the Min and Max IOPS fields in the data model as these seem to generic terms across storage implementations. Thanks, -John On Jun 10, 2013, at 5:57 PM, Mike Tutkowski wrote: > More generally speaking, you're looking to re

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, I am asking whether or not we can defer the entire notion of Burst IOPS in the data model (e.g. the driver could default it to Max IOPS when configuring a SAN volume) for 4.2. In 4.3, we design a facility for extended driver data that would allow drivers to expose these implementation-sp

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
More generally speaking, you're looking to remove Burst IOPS from CloudStack for 4.2, but we would keep Min and Max (and they would be displayed in the Disk Offering dialog as I've proposed)? On Mon, Jun 10, 2013 at 3:52 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Just to make su

[REVIEW] Merging of VMSync improvement into master

2013-06-10 Thread Kelven Yang
Hi there, Alex Huang and I are targeting to finish the debugging process on VMsync improvement by the end of this week. I'd like to encourage those who are interested or having concerns about this project to review it as soon as possible on branch vmsync. We are going to propose the official m

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
Just to make sure I understand your request, are you looking to display Min and Max (as long as Wei's feature is not in use), but not display Burst IOPS? On Mon, Jun 10, 2013 at 3:50 PM, John Burwell wrote: > Mike, > > My concern becomes that we start ballooning the data model and user > interf

Re: haproxy on VMWare systemVM template

2013-06-10 Thread Chiradeep Vittal
The jenkins build of the systemvm has been failing for a couple of days. Can someone clean it up? On 6/7/13 5:35 PM, "Chiradeep Vittal" wrote: >This is now done. >http://s.apache.org/wy > > > >On 5/28/13 3:35 PM, "Chiradeep Vittal" >wrote: > >>Thanks. I'll wait for the i386 bits to land as well

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, My concern becomes that we start ballooning the data model and user interface with a fields that are documented as, "If using SolidFire, then Burst IOPS is honored and foo and bar are not. For solution X, Burst IOPS is ignored, but foo and bar apply." It may have to hold until 4.3, but

Re: [GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Chiradeep Vittal
Yes On 6/10/13 1:12 PM, "Sebastien Goasguen" wrote: > >On Jun 10, 2013, at 3:07 PM, Chiradeep Vittal > wrote: > >> The check for the config is incorrect. The check should be whether the >> network in the input parameter has the isolation method 'GRE'. You can >>see >> the canHandle method on how

Re: [MERGE] disk_io_throttling to MASTER

2013-06-10 Thread Mike Tutkowski
Right...all of that makes sense to me, Edison. I think John's concern with my patch file (which implements the approach you outlined in your last paragraph) is that he didn't like the "attach" command performing logic to create an SR if the SR isn't present. He wanted a new hypervisor command to

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
Hi, I don't think we can restrict Burst to SolidFire only because there really is no way to know that the Disk Offering will be serviced by SolidFire. This really is a flaw with the way CloudStack handles storage tags. For example, the Disk Offering could use a storage tag that references SolidF

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Wei ZHOU
Mike, Glad to see this. This is what I have questioned about. I accepted John's and your opnion that hypervisor IOPS and storage IOPS are mutually exclusion. The way you mentioned is a good way to go. A small question is, will burst IOPS only appear when storage device is SolidFire? -Wei 201

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
A major issue for current QoS providers is not being able to utilize over a Max amount even when it is highly desirable and the storage system can support it. That's why I'm thinking it will be a feature others attempt to imitate. On Mon, Jun 10, 2013 at 2:44 PM, Mike Tutkowski < mike.tutkow...@s

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
My thinking is that Min and Max are industry standard and Burst is a new concept that could catch on. On Mon, Jun 10, 2013 at 2:29 PM, John Burwell wrote: > Wei, > > In this case, we can have the hypervisor or storage providing the quality > of service guarantee. Naively, it seems reasonable t

Re: Review Request: Windows 8 (64 bit) guest OS enablement for KVM deployments in cloudstack

2013-06-10 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11611/#review21670 --- The code looks fine to me, what about test results? - Min Chen On

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Wei, In this case, we can have the hypervisor or storage providing the quality of service guarantee. Naively, it seems reasonable to separate hypervisor and storage QoS parameters and enforce mutually exclusion. Not only does this approach simplify a whole range of operations, it also avoids

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
What if you specified multiple storage tags that mapped to different storage vendors' storage? I'm not sure how they could enter fields for all of those vendors if you can only select one vendors' fields. On Mon, Jun 10, 2013 at 2:21 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > I

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
I'm not sure how that would work if the user picks SolidFire, but then specifies a Storage Tag that doesn't include SolidFire. As far as I know, Min, Max, and Burst is a superset for current storage vendors that do provisioned IOPS of some sort. On Mon, Jun 10, 2013 at 1:59 PM, Wei ZHOU wrote:

Re: [GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Sebastien Goasguen
On Jun 10, 2013, at 3:07 PM, Chiradeep Vittal wrote: > The check for the config is incorrect. The check should be whether the > network in the input parameter has the isolation method 'GRE'. You can see > the canHandle method on how to check for this. Although, I don't see why > this check shou

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Wei ZHOU
Mike, I do not think users can select only one of them, as they are implemented on different sides. Have you investigated the parameters other storage devices support, besides min/max/burst IOPS? You'd better add all possible fields in your implementation. What do you think about this? Hypersivor

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
Here is my thinking: Two radio buttons (whatever we want to call them): 1) Hypervisor IOPS 2) Storage IOPS Leave them both un-checked by default. If the user checks one or the other, the relevant fields appear. On Mon, Jun 10, 2013 at 1:38 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wr

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
What do you think, Wei? Should we come up with a way for only one feature (yours or mine) to be used at a time on the new Disk Offering dialog? Since most storage-side provisioned IOPS don't break it down into separate read and write categories, I think that's the way to go (only one feature or t

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, I agree -- I can't image a situation where you would want to use IOPS provisioned by both the hypervisor and storage. There are two points of concern -- the UI and the management server. We have to ensure that the user can't create a VM from a compute/disk offering combination where hyp

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
Wei has sent me the screen shots. I don't support Compute Offerings for 4.2, so that's not an issue here. I do support Disk Offerings. It looks like Wei has added four new fields to the Disk Offering. I have added three (Min, Max, and Burst IOPS). We just need to decide if we should toggle bet

Re: Unable to adjust firewall rules on VR since upgrade

2013-06-10 Thread Chiradeep Vittal
Technically there is no agent running inside the VR. They are all handled by ssh scripts. But generally, communication problems between the mgmt server and the dom0 or a stuck (kernel OOM f.e) VR can cause this issue. On 6/10/13 10:45 AM, "Francois Gaudreault" wrote: >Interesting... I can't repr

Re: [GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Chiradeep Vittal
The check for the config is incorrect. The check should be whether the network in the input parameter has the isolation method 'GRE'. You can see the canHandle method on how to check for this. Although, I don't see why this check should even be necessary. The network will be associated with the gur

Re: [GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Sebastien Goasguen
On Jun 10, 2013, at 10:17 AM, Nguyen Anh Tu wrote: > Hi forks, > I think we say "folks" but that's not solving the NPE error. I cc Chiradeep who might have some insight here. -sebastien > I'm trying OVS tunnel with XCP 1.6 and get error NullPointerException when > executing deployVM. > > I

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Mike, Have Wei and you figured out the system level as well (e.g. allowing either storage provisioned IOPS or hypervisor throttling, but no both)? Thanks, -John On Jun 10, 2013, at 2:12 PM, Mike Tutkowski wrote: > Perhaps Wei could send me some screen shots of what he's changed in the GUI >

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Mike Tutkowski
Perhaps Wei could send me some screen shots of what he's changed in the GUI for his feature? Thanks! On Mon, Jun 10, 2013 at 11:56 AM, John Burwell wrote: > Wei, > > Have Mike Tutkowski and you reconciled the potential conflict between a > throttled I/O VM and a provisioned IOPs volume? If so

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread John Burwell
Wei, Have Mike Tutkowski and you reconciled the potential conflict between a throttled I/O VM and a provisioned IOPs volume? If so, what solution did you select? Thanks, -John On Jun 10, 2013, at 1:54 PM, Wei ZHOU wrote: > Guys, > > I would like to merge disk_io_throttling branch into mast

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-10 Thread Wei ZHOU
Guys, I would like to merge disk_io_throttling branch into master. Please review the code on https://reviews.apache.org/r/11782 If nobody object, I will merge into master in 72 hours. -Wei 2013/5/30 Wei ZHOU > Hi, > I would like to merge disk_io_throttling branch into master. > If nobody obje

Review Request: (CLOUDSTACK-1301) VM Disk I/O Throttling

2013-06-10 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11782/ --- Review request for cloudstack, Wido den Hollander and John Burwell. Description

Re: Unable to adjust firewall rules on VR since upgrade

2013-06-10 Thread Francois Gaudreault
Interesting... I can't reproduce it today :S So maybe that was just a temporary issue? (I hate those issues!) What would cause this actually? VR not running? Agent on the VR not running? Thanks! Francois On 2013-06-10 1:34 PM, Chiradeep Vittal wrote: Need more logs On 6/10/13 10:28 AM,

Re: Unable to adjust firewall rules on VR since upgrade

2013-06-10 Thread Chiradeep Vittal
Need more logs On 6/10/13 10:28 AM, "Francois Gaudreault" wrote: >Hi, > >I posted this on the users mailing list, but didn't get any reply. Maybe >I will get more attention here :) > >Let me know if I should open a bug report. > >Thanks! > > Original Message >Subject: Unab

Re: [DISCUSS] NFS cache storage issue on object_store

2013-06-10 Thread John Burwell
Chiradeep, Looks like I have a rookie mistake in S3-backed Secondary Storage. I will investigate, and send a patch to lock on the management server side. Thanks, -John On Jun 10, 2013, at 1:30 PM, Chiradeep Vittal wrote: > >> >> From a code perspective, I think it would behove us to imple

RE: Contributing as a non-committer

2013-06-10 Thread Soheil Eizadi
CloudStack uses Maven you need to use M2e - Eclipse Maven Plugin that will create the Projects for you. -Soheil From: Sebastien Goasguen [run...@gmail.com] Sent: Monday, June 10, 2013 6:45 AM To: dev@cloudstack.apache.org Subject: Re: Contributing as a non-

Re: [DISCUSS] NFS cache storage issue on object_store

2013-06-10 Thread Chiradeep Vittal
> >From a code perspective, I think it would behove us to implement a more >functional approach to command execution in order to ensure reference >counting, error handling, resource management are handled in a consistent >manner. I implemented such an approach in >com.cloud.utils.db.GlobalLock#ex

Fwd: Unable to adjust firewall rules on VR since upgrade

2013-06-10 Thread Francois Gaudreault
Hi, I posted this on the users mailing list, but didn't get any reply. Maybe I will get more attention here :) Let me know if I should open a bug report. Thanks! Original Message Subject:Unable to adjust firewall rules on VR since upgrade Date: Fri, 07 Jun 2013 12

Review Request: Fix for CLOUDSTACK-1622, CLOUDSTACK-1704

2013-06-10 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11780/ --- Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. Description

Re: Handling Self Signed Certs

2013-06-10 Thread Kelven Yang
Will, Thanks for the effort in getting a common wrapper into utils package. As for the policy decision(whether or not to make a global flag or a per-device option), both have pros and cons, we can wait and see the feedbacks from others in the community. Considering the legacy installations and t

Re: VPC and shared networks

2013-06-10 Thread Chiradeep Vittal
Not sure what the test case is trying to achieve, but you can have a VM with a nic on a VPC subnet and another nic on a shared subnet. On 6/10/13 3:18 AM, "Prasanna Santhanam" wrote: >On Mon, Jun 10, 2013 at 03:39:08PM +0530, Prasanna Santhanam wrote: >> Can we create shared networks inside a VP

Re: Handling Self Signed Certs

2013-06-10 Thread Chiradeep Vittal
+1 to per-device On 6/10/13 9:04 AM, "Will Stevens" wrote: >When I went looking in CS for the HTTP clients that were already >available, >I found the one that Soheil is using as well as the new apache one. I am >using the new apache one because I was assuming it was going to be the >preferred o

Re: Review Request: Cloudstack-2511 Multiple_Ip_Ranges: Adding guest ip range in subset/superset to existing CIDR is allowed https://issues.apache.org/jira/browse/CLOUDSTACK-2511, Cloudstack-2651 [Sha

2013-06-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11600/#review21654 --- Don't combine multiple fixes in a single commit. Keep it simple -> 1

buildbot success in ASF Buildbot on cloudstack-site-staging

2013-06-10 Thread buildbot
The Buildbot has detected a restored build on builder cloudstack-site-staging while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/cloudstack-site-staging/builds/168 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-cms-slave Build Reason: s

Re: Handling Self Signed Certs

2013-06-10 Thread Will Stevens
When I went looking in CS for the HTTP clients that were already available, I found the one that Soheil is using as well as the new apache one. I am using the new apache one because I was assuming it was going to be the preferred one going forward. I will clean up my wrapper and make it available

[GSOC] NullPointerException when deployVM using GRE isolation method. Help me!

2013-06-10 Thread Nguyen Anh Tu
Hi forks, I'm trying OVS tunnel with XCP 1.6 and get error NullPointerException when executing deployVM. I found this error starting from the below code. *NetworkManagerImpl.java* *NicProfile profile = guru.allocate(network, requested, vm); (1)* if (isDefaultNic != null) {

Re: Contributing as a non-committer

2013-06-10 Thread Sebastien Goasguen
On Jun 10, 2013, at 9:40 AM, Paul Angus wrote: > Thanks Seb, > I've bought pro git as extra reading too. > > Now I'm trying to setup my dev environment, the 'Setting Up an Apache > CloudStack Development Environment' says > "Most Apache CloudStack developers use Eclipse as their primary IDE.

RE: Contributing as a non-committer

2013-06-10 Thread Paul Angus
Thanks Seb, I've bought pro git as extra reading too. Now I'm trying to setup my dev environment, the 'Setting Up an Apache CloudStack Development Environment' says "Most Apache CloudStack developers use Eclipse as their primary IDE. CloudStack source code already includes Eclipse .project file

Re: Review Request: (CLOUDSTACK-2707) use executeBatch instead of persist when Usage Server createNetworkHelperEntry

2013-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11461/#review21649 --- Commit 76ce304411259dc4a29e0e070de829b79b46efe2 in branch refs/heads

[OFFLINE] 6/12 and 6/13

2013-06-10 Thread Prasanna Santhanam
Will be away mid-week and won't be able to make it to the meeting and/or check emails. Thanks, -- Prasanna., Powered by BigRock.com

Re: Runtime Error: "could not find systemvmiso" in master branch

2013-06-10 Thread Nguyen Anh Tu
@Ke4qqq: Okie I'll note to document it :-) 2013/6/10 Vijay Venkatachalam > Glad to know it is was useful! > > Thanks, > Vijay V. > > > -Original Message- > > From: Nguyen Anh Tu [mailto:ng.t...@gmail.com] > > Sent: Saturday, June 08, 2013 3:57 PM > > To: cloudstack-...@incubator.apache.

Re: VPC and shared networks

2013-06-10 Thread Prasanna Santhanam
On Mon, Jun 10, 2013 at 03:39:08PM +0530, Prasanna Santhanam wrote: > Can we create shared networks inside a VPC? If yes - why does the UI > filter out the network offerings in the VPC dialog on listing networks > using forvpc=true & guestiptype=Isolated? > > If no - we can get rid of a test suite

VPC and shared networks

2013-06-10 Thread Prasanna Santhanam
Can we create shared networks inside a VPC? If yes - why does the UI filter out the network offerings in the VPC dialog on listing networks using forvpc=true & guestiptype=Isolated? If no - we can get rid of a test suite - TestVMLifeCycleSharedNwVPC. But was it supported before and dropped recentl

Re: Review Request: Automation: Add testcases for Affinity/Anti-Affinity Rules

2013-06-10 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11067/ --- (Updated June 10, 2013, 10:03 a.m.) Review request for cloudstack, Prachi Damle

VMWare systemVM template

2013-06-10 Thread Abhinandan Prateek
Rohit, I have the new Debian 7 system VM template and related vmx file for VMWare. I am not sure how much of VMWare fusion stuff it can be automated via virtual box ? -abhi

Re: Review Request: generalisation of network code (needed for CLOUDSTACK-1532)

2013-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10970/ --- (Updated June 10, 2013, 9:20 a.m.) Review request for cloudstack, Murali Reddy,

Re: What are the standard commands to setup 4.1 development environment

2013-06-10 Thread Prasanna Santhanam
On Mon, Jun 10, 2013 at 04:52:22PM +0800, Ryan Lei wrote: > Thanks for the quick explanation. > Then should the cloud-install-sys-tmplt script be run before or after the > systemvm profile? Or does it matter at all? The install-sys-tmplt script still will need to be run (before you launch the mana

Re: What are the standard commands to setup 4.1 development environment

2013-06-10 Thread Ryan Lei
Thanks for the quick explanation. Then should the cloud-install-sys-tmplt script be run before or after the systemvm profile? Or does it matter at all? On Mon, Jun 10, 2013 at 4:03 PM, Prasanna Santhanam wrote: > On Mon, Jun 10, 2013 at 03:52:19PM +0800, Ryan Lei wrote: > > CloudStack 4.1 has b

Cloud Usage and API

2013-06-10 Thread CK
Can someone please provide further information on what the API call: generateUsageRecords actually does? I know it generates usage records, but: 1. Does the Usage Server service call the generateUsageRecords API when it runs at the scheduled time(=usage.stats.job.exec.time) or does the service do

Re: What are the standard commands to setup 4.1 development environment

2013-06-10 Thread Prasanna Santhanam
On Mon, Jun 10, 2013 at 03:52:19PM +0800, Ryan Lei wrote: > CloudStack 4.1 has been released, but I see two "versions" of instructions > in setting up the oss development environment. > > (1) From the official website: > http://cloudstack.apache.org/develop/environment.html > > # Compile > $ mvn

What are the standard commands to setup 4.1 development environment

2013-06-10 Thread Ryan Lei
CloudStack 4.1 has been released, but I see two "versions" of instructions in setting up the oss development environment. (1) From the official website: http://cloudstack.apache.org/develop/environment.html # Compile $ mvn -P developer clean install # Deploy both the management server and the dat

Re: Review Request: Meng's Proposal for GSOC 2013 in Docbook format

2013-06-10 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11775/#review21643 --- Ship it! Thanks, patch applied to the master branch with commit 6e