Re: [PROPOSAL] Ability to add new guest OS

2014-03-04 Thread Daan Hoogland
+1 for the feature, Did you consider systemvms? Or the default vm-template? Or are those marked as undeletable or even hardcoded in case the table gets editted outside of ACS? The delete feature is vital. You marked it as open for discussion but I don't agree. The admin should delete when he make

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Daan Hoogland
H Alena, You are right. I was under the impression that response format was being extended. I saw you already reverted. If the response was extended with the extra value it would be alright, would it? On Mon, Mar 3, 2014 at 7:06 PM, Alena Prokharchyk wrote: > Daan, this fix break API compatibil

Re: 4.3 createVMSnapshot doesn't check volume state

2014-03-04 Thread Daan Hoogland
H Qian, Submit a bug report in jira [1]. If you have a fix for the 'master' branch submit a review request in the review board [2] be sure to add enough contents to the fields and set the group to 'cloudstack'. Find the people most involved with the code you changed and ask them to review. If you

Re: Review Request 16688: console-proxy add support of AltGr key and FR azerty keyboard

2014-03-04 Thread Axel Delahaye
> On March 4, 2014, 7:35 a.m., daan Hoogland wrote: > > This is ust for 4.2.x, does it apply to higher version as well or do you > > want to submit a separate patch for those? I don't try it with newest cloudstack version (I don't have a 4.3) but if the console-proxy doesn't change a lot it ma

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

2014-03-04 Thread Antonio Fornié Casarrubios
Alena, I saw it, at first I thought it would be a problem in a certain cmd and then I saw it's the same for all of them. Actually Cmd#getCommandName() should give us what we want here, the command name, right? Why are we returning the cmdNameResponse instead? On top of that, if we continue this wa

Re: Review Request 18616: added delete iso api to the simulator

2014-03-04 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18616/#review36102 --- Ship it! master -> 8e2c4c810557005995f8d49ed55e903d6bc4126d - Kous

Re: Review Request 18680: added mock handlers for VpnUsersCfgCommand and RemoteAccessVpnCfgCommand in the simulator

2014-03-04 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18680/#review36103 --- Ship it! master -> 9be51e1961aa3be09948202e3244e13343d567f9 - Kous

Re: Review Request 16688: console-proxy add support of AltGr key and FR azerty keyboard

2014-03-04 Thread daan Hoogland
> On March 4, 2014, 7:35 a.m., daan Hoogland wrote: > > This is ust for 4.2.x, does it apply to higher version as well or do you > > want to submit a separate patch for those? > > Axel Delahaye wrote: > I don't try it with newest cloudstack version (I don't have a 4.3) but if > the console

Re: Review Request 16688: console-proxy add support of AltGr key and FR azerty keyboard

2014-03-04 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16688/#review36105 --- Ship it! eb2f323c48ac4d26d268cc5ee566520b475c74cf (note that needs

Re: [4.4] Feature request: root resize (FAO Marcus)

2014-03-04 Thread Nux!
On 27.02.2014 09:29, Nux! wrote: Hi, Having talked about this issue recently, I'd like to request this feature so as to avoid template duplication. Marcus, you said you have a working patch, if you get it to apply to 4.4 I will invest time in testing it. I have opened an issue with rationale, e

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

2014-03-04 Thread Bharat Kumar
Hi ALL, There are many other APIs that use Map like createNetworkOffering, updateZone, createTemplate, in most of the cases we do not say how to use maps, one way would be to write this in the description or to use the same way to access maps of all APIs. BTW the way to use details in deploy vm

Re: [PROPOSAL] Ability to add new guest OS

2014-03-04 Thread Abhinandan Prateek
On 04/03/14 2:14 am, "Amogh Vasekar" wrote: >Hi, > >CloudStack currently does not allow an easy way to add new guest OS types, >for example, a standard way to add say, CentOS 6.5 even though a >hypervisor may support it. >Part of the reason is since the OS to hypervisor-specific platform >mapping

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread France
Hi Marcus and others. There is no need to kill of the entire hypervisor, if one of the primary storages fail. You just need to kill the VMs and probably disable SR on XenServer, because all other SRs and VMs have no problems. if you kill those, then you can safely start them elsewhere. On XenS

RE: master branch DbUtilTest failure

2014-03-04 Thread Alex Hitchins
I too have run into this however not yet managed to sort it out. Regards Alex Hitchins D: +44 1892 523 587 | S: +44 2036 030 540 | M: +44 7788 423 969 alex.hitch...@shapeblue.com -Original Message- From: Prachi Damle [mailto:prachi.da...@citrix.com] Sent: 03 March 2014 23:35 To: dev@c

CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

2014-03-04 Thread Mandar Barve
Hi, I tried to reproduce the issue but couldn't get this to fail for insufficient space. I then injected an exception trying to list files from a non existent path (added this code in the "try" block). This landed me into the exception handling code. It raised correct exception saying "file

Re: Review Request 17506: mvn clean - test logs- contrail -cleanps up only target, dist directories; testclient.log file should be generated inside target directory

2014-03-04 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17506/#review36106 --- Ship it! e2f9c11ad3851ab5fd5749474cd11abe970db997 - daan Hoogland

Re: Removing TAGS

2014-03-04 Thread David Nalley
This really isn't 'our' history though. Most of the tags Sebastien referred to came in on January 8th[1]. When we moved to the ASF, we removed the vast majority of tags and branches as they simply didn't have a lot of context here, and increased the already-bewildering confusion most people experie

Re: 4.4 Feature Freeze

2014-03-04 Thread Sebastien Goasguen
On Mar 3, 2014, at 8:18 PM, Animesh Chaturvedi wrote: > > >> -Original Message- >> From: Sebastien Goasguen [mailto:run...@gmail.com] >> Sent: Sunday, March 02, 2014 10:13 AM >> To: dev@cloudstack.apache.org >> Subject: Re: 4.4 Feature Freeze >> >> My take is that we are slipping on

Re: Review Request 18623: CLOUDSTACK-6147: Adding first set of test cases for Dynamic Compute Offering feature

2014-03-04 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18623/#review36107 --- Gentle reminder - Gaurav Aradhye On Feb. 28, 2014, 7:17 p.m., Gau

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread Andrei Mikhailovsky
I agree with France, sounds like a more sensible idea and killing hosts left, right and centre with live vms. I now understand the reasons behind killing the troubled host server, however, this should be done without killing live vms with fully working volumes. Regarding having nfs and ceph

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread Nux!
On 04.03.2014 12:55, Andrei Mikhailovsky wrote: Regarding having nfs and ceph storage in different clusters - sounds like a good idea for majority of cases, however, my setup will not allow me to do that just yet. I am using ceph for my root and data volumes and NFS for backup volumes. Having

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread David Grizzanti
Alena, Have the same question as Daan. If we left the "success => true" key/value in the response and added the project in addition, would that be OK? On Tue, Mar 4, 2014 at 3:05 AM, Daan Hoogland wrote: > H Alena, > > You are right. I was under the impression that response format was > being e

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread Marcus
On Tue, Mar 4, 2014 at 3:34 AM, France wrote: > Hi Marcus and others. > > There is no need to kill of the entire hypervisor, if one of the primary > storages fail. > You just need to kill the VMs and probably disable SR on XenServer, because > all other SRs and VMs have no problems. > if you kill

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread Wido den Hollander
On 03/04/2014 03:38 PM, Marcus wrote: On Tue, Mar 4, 2014 at 3:34 AM, France wrote: Hi Marcus and others. There is no need to kill of the entire hypervisor, if one of the primary storages fail. You just need to kill the VMs and probably disable SR on XenServer, because all other SRs and VMs ha

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-03-04 Thread Marcus
If you get a moment, please look at resize-root branch and see if it works for Gluster. I didn't see any code specific to creating volumes in your patch, mostly just pool stuff, so hopefully it's covered. Also, Wido, as mentioned on the bug for resize-root I'd prefer it if you could review the cha

Re: 4.4 Feature Freeze

2014-03-04 Thread Daan Hoogland
Animesh, You are doing a great job as guard on our present release and I don't recall features slipping in either. instead those should now be added to the master branch. If Sebastien made it sound that way it is probably because of something else that is bothering him. However, Sebastien makes a

[Events] CloudStack Bangalore March Meetup @ InMobi

2014-03-04 Thread iliyas shirol
Greetings! We are happy to announce the Apache CloudStack Bangalore March MeetUp. RSVP @ http://www.meetup.com/CloudStack-Bangalore-Group/events/169340552/ PFB the agenda of the meetup, - Agenda: 4.00 - 4.30 PM Networking with CloudStackians (have fun !) 4.30 - 5.15 PM

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread Marcus
Of course, in the event that the NFS *server* is truly out, users may prefer that no action is taken, everything stays as-is so that when it comes back things can continue running. The main issue I can think of with that is that NFS clients may think things were flushed to disk on the NFS server th

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

2014-03-04 Thread Mike Tutkowski
Hi, Can you tell me what release you tested this with? I noticed the problem while developing on CloudStack 4.3. Thanks! On Tue, Mar 4, 2014 at 3:43 AM, Mandar Barve wrote: > Hi, > I tried to reproduce the issue but couldn't get this to fail for > insufficient space. I then injected an

4.2.1 Simulator - DeployVirtualMachine gives different keypair than specified

2014-03-04 Thread Ian Duffy
Hi All, I'm running against the 4.2.1 Simulator. I'm executing a DeployVirtualMachine command with KeyPair of test example: client/api?apikey=sZOWCNSEqYmqhZGswOPpAq-snv_YoRzq6ocdb4FDKodSxni7VgI9t3g4K30q_-Y3PAYkQsq-Sp3LJHPQbiSh3w&command=deployVirtualMachine&keypair=test&response=json&serviceoffe

Re: ALARM - ACS reboots host servers!!!

2014-03-04 Thread France
On Mar 4, 2014, at 3:38 PM, Marcus wrote: > On Tue, Mar 4, 2014 at 3:34 AM, France wrote: >> Hi Marcus and others. >> >> There is no need to kill of the entire hypervisor, if one of the primary >> storages fail. >> You just need to kill the VMs and probably disable SR on XenServer, because >> a

RE: 4.4 Feature Freeze

2014-03-04 Thread Giles Sirett
I'm coming in late to this debate, but have been watching closely. I am -1 on moving the freeze date To put it more generally: I am -1 on ANYTHING that will have a potential impact on quality right now. Whatever anybody says, I can bet that a shorter code freeze (even temporary) will have an imp

Review Request 18733: Refactor DbUpgradeUtils

2014-03-04 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18733/ --- Review request for cloudstack, Alena Prokharchyk, daan Hoogland, and Hugo Trippa

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

2014-03-04 Thread jenkins
See

Re: 4.4 Feature Freeze

2014-03-04 Thread SuichII, Christopher
I’d be a fan of using 4.4 as a learning process for the community and an attempt to meet our proposed schedule. While I can appreciate the desire to get features into 4.4, there appears to be a significant backlog on JIRA and we’ve now had two consecutive versions (I’m not sure about 4.1) not me

Re: 4.4 Feature Freeze

2014-03-04 Thread Jayapal Reddy Uradi
Hi, The below feature is proposed in dev list but somehow the fix version is not set correctly, update it now. CLOUDSTACK-2692 Add load balancing support for multiple ips to nic Thanks, Jayapal On 04-Mar-2014, at 9:37 PM, Giles Sirett mai

Re: [PROPOSAL] Ability to add new guest OS

2014-03-04 Thread John Kinsella
+1 for the feature, but usually I just go for “other paravirtualized (64 bit)” (paraphrasing, mistakes mine) nowadays. What would be more useful, IMHO, is to better document the what the selection OS type selection choice means for a VM. For KVM, it’s a hell of a lot more than SCSI vs IDE root d

Review Request 18735: CLOUDSTACK-4840: Final set of test cases for Multiple IPs per NIC feature

2014-03-04 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18735/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUDSTACK-4840

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Alena Prokharchyk
Daan, you can¹t extend the response with more parameters as its a generic SuccessResponse used by many Apis, and you can¹t add anything a particular call specific to that. I think we shouldn¹t fix the bug at this point. You can always get the additional information you need by executing correspond

RE: 4.4 Feature Freeze

2014-03-04 Thread Paul Angus
To stick my oar in, I think that 3 days is too short a time for people to get round to doing meaningful testing, particularly for those of us doing it on a purely voluntary basis. Also I got the feeling that we were spinning up new RCs before it was clear that all issues with the previous RC h

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

2014-03-04 Thread Alena Prokharchyk
Antonio, I don¹t have a history about why we return cmdNameResponse instead of returning actual command name. We might change the method name in the future. And yes, please go ahead and create a new method in BaseCmd. -Alena. On 3/4/14, 1:02 AM, "Antonio Fornié Casarrubios" wrote: >Alena, I sa

RE: 4.4 Feature Freeze

2014-03-04 Thread Animesh Chaturvedi
> -Original Message- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: Tuesday, March 04, 2014 4:45 AM > To: dev@cloudstack.apache.org > Subject: Re: 4.4 Feature Freeze > > > On Mar 3, 2014, at 8:18 PM, Animesh Chaturvedi > wrote: > > > > > > >> -Original Message- >

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-04 Thread Daan Hoogland
This week? should be ok. end of next week is too late! On Tue, Mar 4, 2014 at 6:17 PM, Alex Ough wrote: > Do you think it is ok to wrap up the implementation until the end of this > week to make this included in 4.4 release? > > Thanks > Alex Ough > > > On Sun, Mar 2, 2014 at 2:10 PM, Daan Hoogla

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Daan Hoogland
We can not change the response for this api call? so other apis aren't touched? On Tue, Mar 4, 2014 at 6:31 PM, Alena Prokharchyk wrote: > Daan, you can¹t extend the response with more parameters as its a generic > SuccessResponse used by many Apis, and you can¹t add anything a particular > call

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Alena Prokharchyk
We are not gonna change the response for API calls mentioned in the bug filed by David. We will just leave Apis the way it is. The original bug should be closed as WontFix, and I already reverted the commit. I didn’t quite get your question about other APIs, could you please elaborate? -Alena.

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Daan Hoogland
I mean not returning a SuccessResponse but some custom variety to it. On Tue, Mar 4, 2014 at 7:44 PM, Alena Prokharchyk wrote: > We are not gonna change the response for API calls mentioned in the bug > filed by David. We will just leave Apis the way it is. The original bug > should be closed as

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Alena Prokharchyk
Daan, I don’t see the need for doing that. Take deleteAccountFromProject API. Every delete command in CloudStack returns the success response; seeing true/false is more than enough to figure out if the deletion was successful. We follow this format in every delete* command in CS. Now take addAccou

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-04 Thread Alex Ough
Do you think it is ok to wrap up the implementation until the end of this week to make this included in 4.4 release? Thanks Alex Ough On Sun, Mar 2, 2014 at 2:10 PM, Daan Hoogland wrote: > two weeks from now, but under debate. Let's go with that, though. > > On Sun, Mar 2, 2014 at 1:17 AM, Alex

Re: [4.4] Feature request: root resize (FAO Marcus)

2014-03-04 Thread Mike Tutkowski
Just updated the ticket. Thanks for checking! On Tue, Mar 4, 2014 at 2:46 AM, Nux! wrote: > On 27.02.2014 09:29, Nux! wrote: > >> Hi, >> >> Having talked about this issue recently, I'd like to request this >> feature so as to avoid template duplication. >> Marcus, you said you have a working p

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-04 Thread Alex Ough
Yes, by 3/7. Thanks Alex Ough On Tue, Mar 4, 2014 at 12:30 PM, Daan Hoogland wrote: > This week? should be ok. end of next week is too late! > > On Tue, Mar 4, 2014 at 6:17 PM, Alex Ough wrote: > > Do you think it is ok to wrap up the implementation until the end of this > > week to make this

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread Daan Hoogland
On Tue, Mar 4, 2014 at 7:57 PM, Alena Prokharchyk wrote: > Although > the response name "success" that we have to preserve, So if we fi it this way the reponse should contain displaytext : any text associated with the success or failure success : true if operation is executed successfully and t

SSO

2014-03-04 Thread María Noelia Gil
Hello, I am studying the operation of single sign-on in CloudStack. Does anyone can provide me information about this topic? I want to know if you can use SAML, OpenID, ..., and as it should be. Thank you.

RE: Removing TAGS

2014-03-04 Thread Alex Huang
Thanks Dave. I think that makes it much clearer. I'm +1 on it, especially given that it got pushed in this year. --Alex > -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Tuesday, March 4, 2014 4:00 AM > To: dev@cloudstack.apache.org > Subject: Re: Removing TAGS

Re: Review Request 18733: Refactor DbUpgradeUtils

2014-03-04 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18733/#review36151 --- tests fail: Running com.cloud.upgrade.dao.DatabaseAccessObjectTest

RE: Exception adding XenServer host to CS

2014-03-04 Thread Anthony Xu
>Do people have to run xe-switch-network-backend bridge before upgrading to >4.4 (if they are using a Basic Zone)? Correct. >In my case, since I'm developing and able to start with a fresh DB and >XenServer hosts, it looks like I can just run xe-switch-network-backend bridge >before I create my

RE: Exception adding XenServer host to CS

2014-03-04 Thread Paul Angus
I haven't managed to read every post in this thread, but I think I get the gist of what's being asked. Since XenServer 6.1 openvswitch has become the default stack rather than bridge mode. To enable bridge mode for security groups you need to do the xe-switch-network-backend bridge command and

Re: Review Request 18673: Return failure for StartCommand and PrepareForMigrationCommand if storage adaptor fails to connect disks

2014-03-04 Thread Brian Angus
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18673/#review36162 --- Ship it! Ship It! - Brian Angus On March 3, 2014, 12:09 a.m., Ma

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

2014-03-04 Thread Suresh Balineni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18552/ --- (Updated March 4, 2014, 10:10 p.m.) Review request for cloudstack. Changes --

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

2014-03-04 Thread Suresh Balineni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18552/ --- (Updated March 4, 2014, 10:11 p.m.) Review request for cloudstack. Changes --

[4.4] Root volume resize for XenServer

2014-03-04 Thread Nux!
Hello, Would someone be interested in this for Xenserver? Would be a valuable feature for all hypervisors. https://issues.apache.org/jira/browse/CLOUDSTACK-6181?focusedCommentId=13918796&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13918796 (point 6) Lucian

Re: [4.4] Root volume resize for XenServer

2014-03-04 Thread Mike Tutkowski
Is this feature currently KVM only? On Tue, Mar 4, 2014 at 3:11 PM, Nux! wrote: > Hello, > > Would someone be interested in this for Xenserver? Would be a valuable > feature for all hypervisors. > > https://issues.apache.org/jira/browse/CLOUDSTACK-6181? > focusedCommentId=13918796&page=com.atla

Re: [4.4] Root volume resize for XenServer

2014-03-04 Thread Marcus
resizeVolume has been extended to work with root disks. If resizeVolume works on XenServer (which I believe it does), then it should just work. Also, if someone looks at the CopyCommand for CitrixResourceBase (or whatever handles the storage commands for Xen now) that is used to create new disks f

Re: [4.4] Root volume resize for XenServer

2014-03-04 Thread Marcus
It is, but not yet merged. We're investigating whether anyone who codes on the other hypervisors are interested in supporting it for the release. If not, it will be KVM-only for now. On Tue, Mar 4, 2014 at 3:15 PM, Mike Tutkowski wrote: > Is this feature currently KVM only? > > > On Tue, Mar 4, 2

Re: [4.4] Root volume resize for XenServer

2014-03-04 Thread Mike Tutkowski
If no one picks it up for Xen, I should be able to look into it. Problem is it might have to be after Feature Freeze if that's just in two weeks. On Tue, Mar 4, 2014 at 3:16 PM, Marcus wrote: > It is, but not yet merged. We're investigating whether anyone who > codes on the other hypervisors ar

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

2014-03-04 Thread Marcus
Ok, sounds like there needs to be some work done to make these more consistent, perhaps. Can you comment on why rootdisksize was made from a parameter into a part of the details map? On Tue, Mar 4, 2014 at 3:12 AM, Bharat Kumar wrote: > Hi ALL, > There are many other APIs that use Map like create

Re: Review Request 17591: CLOUDSTACK-5872: Async response from addAccountToProject doesn't contain useful information

2014-03-04 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17591/#review36192 --- Commit 444e8f6bb3447a7eeb97f7c5d9b46abba3150072 in cloudstack's bran

Review Request 18759: HTTP support for console proxy and making it default

2014-03-04 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18759/ --- Review request for cloudstack, Demetrius Tsitrelis and John Kinsella. Repositor

Re: Review Request 18759: HTTP support for console proxy and making it default

2014-03-04 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18759/ --- (Updated March 5, 2014, 1:42 a.m.) Review request for cloudstack, Demetrius Tsi

Re: [DISCUSS] realhostip.com going away

2014-03-04 Thread Amogh Vasekar
Hello, I have created a review request at : https://reviews.apache.org/r/18759/ that partially address the issue. It has a link to the wiki describing the changes in detail. Thanks, Amogh On 3/3/14 8:58 AM, "John Kinsella" wrote: >I talked with some of the Citrix folk over the weekendŠtheir po

RE: OVS plugin in CloudStack 4.3.0

2014-03-04 Thread Animesh Chaturvedi
Tuna I look at your commits in 4.3-forwards and seems the changes are a lot, this would have to come in the next release Thanks Animesh > -Original Message- > From: ng.t...@gmail.com [mailto:ng.t...@gmail.com] On Behalf Of Nguyen > Anh Tu > Sent: Monday, February 24, 2014 8:31 AM > To: d

Build failed in Jenkins: build-master #402

2014-03-04 Thread jenkins
See -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on cloudstack-buildslave-centos6-28c in workspace

Jenkins build is back to normal : build-master #403

2014-03-04 Thread jenkins
See

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

2014-03-04 Thread Mandar Barve
I tested this with CS 4.3. Thanks, Mandar On Tue, Mar 4, 2014 at 9:09 PM, Mike Tutkowski wrote: > Hi, > > Can you tell me what release you tested this with? I noticed the problem > while developing on CloudStack 4.3. > > Thanks! > > > On Tue, Mar 4, 2014 at 3:43 AM, Mandar Barve wrote: > >> Hi

RE: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

2014-03-04 Thread Saksham Srivastava
Marcus, as a committer can you kindly commit it too. -Original Message- From: Marcus Sorensen [mailto:nore...@reviews.apache.org] On Behalf Of Marcus Sorensen Sent: Monday, March 3, 2014 10:09 PM To: Marcus Sorensen; Sateesh Chodapuneedi Cc: Saksham Srivastava; cloudstack Subject: Re: Rev

Re: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

2014-03-04 Thread Marcus
sure, I lose track of who is committer, as many times committers post to reviewboard as well if the code is unfamiliar. On Tue, Mar 4, 2014 at 9:46 PM, Saksham Srivastava wrote: > Marcus, as a committer can you kindly commit it too. > > -Original Message- > From: Marcus Sorensen [mailto:n

Re: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

2014-03-04 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18683/#review36209 --- Commit b8413b9413eb39053c378309195469a4d5d4fec0 in cloudstack's bran

Re: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

2014-03-04 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18683/#review36210 --- Commit cd0d16f05a2be02e3de1a327bba9a5acd112e4f2 in cloudstack's bran

RE: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

2014-03-04 Thread Saksham Srivastava
Thanks, I am still not there :) -Original Message- From: Marcus [mailto:shadow...@gmail.com] Sent: Wednesday, March 5, 2014 10:48 AM To: Saksham Srivastava Cc: dev@cloudstack.apache.org; Sateesh Chodapuneedi Subject: Re: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared ne

Re: [Proposal] Virtual Router service failure alerting

2014-03-04 Thread Harikrishna Patnala
Hi, My comments inline. On 03-Mar-2014, at 1:16 pm, Chiradeep Vittal mailto:chiradeep.vit...@citrix.com>> wrote: Hi, What happens if there is an extended disconnect from MS? What is the rollover strategy for the new log files? In case of extended disconnect, we record the last received timestamp

Re: [PROPOSAL] Ability to add new guest OS

2014-03-04 Thread Amogh Vasekar
Hi, Responses inline. Thanks, Amogh On 3/4/14 12:00 AM, "Daan Hoogland" wrote: >+1 for the feature, > >Did you consider systemvms? Or the default vm-template? Or are those >marked as undeletable or even hardcoded in case the table gets editted >outside of ACS? [Amogh] Sorry, I don't think I get

Re: [PROPOSAL] Ability to add new guest OS

2014-03-04 Thread Amogh Vasekar
On 3/4/14 2:33 AM, "Abhinandan Prateek" wrote: >Is the initial list of supported os coming from Hypervisor Yes. More specifically, for now the list is coming from : KVM : http://bit.ly/1kY6msQ Xen : http://bit.ly/1mUei3f VMW : http://bit.ly/MMsAlF

Review Request 18763: CLOUDSTACK-6201: createVMSnapshot doesn't check volume state

2014-03-04 Thread Qian Shaohua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18763/ --- Review request for cloudstack, daan Hoogland and Kelven Yang. Bugs: CLOUDSTACK-

Re: Review Request 18759: HTTP support for console proxy and making it default

2014-03-04 Thread John Kinsella
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18759/#review36212 --- Amogh - thanks for the patch. Some small issues on this patch. I've

RE: [4.3] [Cherry-pick] CLOUDSTACK-6146

2014-03-04 Thread Animesh Chaturvedi
Done > -Original Message- > From: Likitha Shetty > Sent: Wednesday, February 26, 2014 12:39 AM > To: dev@cloudstack.apache.org > Cc: Animesh Chaturvedi > Subject: [4.3] [Cherry-pick] CLOUDSTACK-6146 > > Animesh, > > Can you cherry pick 8cb03ddb2387c7aebf20dc0aa011e41f227e9f68 and > c652f

RE: [4.3] [Cherry-pick] CLOUDSTACK-6174, CLOUDSTACK-6175

2014-03-04 Thread Animesh Chaturvedi
Done > -Original Message- > From: Min Chen > Sent: Wednesday, February 26, 2014 3:25 PM > To: dev@cloudstack.apache.org > Cc: Animesh Chaturvedi > Subject: Re: [4.3] [Cherry-pick] CLOUDSTACK-6174, CLOUDSTACK-6175 > > Animesh, > > Can you cherry pick the following commits from 4.3-f

RE: [4.3] [Cherry-pick] CLOUDSTACK-6186

2014-03-04 Thread Animesh Chaturvedi
Done > -Original Message- > From: Nitin Mehta > Sent: Friday, February 28, 2014 2:45 PM > To: dev@cloudstack.apache.org > Cc: Animesh Chaturvedi > Subject: [4.3] [Cherry-pick] CLOUDSTACK-6186 > > Hi Animesh - Can you please cherry pick the following commit from 4.3- > forward to 4.3? > >

Re: [DISCUSS] realhostip.com going away

2014-03-04 Thread John Kinsella
While this patch might address the direct problem, a quick string search of the source shows “realhostip.com” in a few more places that also need to be cleaned up, including in the db. I haven’t actually applied the patch yet, just read over it a few times. Will try to app