Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

Doesn't sound like a guarantee, If the stack is build otherwise there might come in a different (non-integer-representing) string. On Thu, Nov 7, 2013 at 6:08 PM, Min Chen wrote: > Yes. From callstack, > BroadcastDomainType.getValue(BroadcastDomainType.fromString(vlanId) is > already called befor

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

Then we need to clearly define the semantic of parameter "vlanId" of HypervisorHostHelper.prepareNetwork is indeed the vlan Id string, not other format. The invoker method should pass the correct one to this utility to make it work. Thanks -min On 11/7/13 3:43 PM, "Daan Hoogland" wrote: >Doesn'

Router requires upgrade. Unable to send command to router Error

Unable to deploy vms in the latest master because of the commit below. Is anyone noticing this on the latest master. I checked the code and there was a commit made recently 3f5b8f which is where the exception points to. WARN [o.a.c.alerts] (CapacityChecker:ctx-4f1ef01f) alertType:: 2 // data

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

Hi Daan, I think your patch is completely wrong. The BroadcastDomainType.getValue() would accept format of URI, not the number. I don't think your change would work, either in code or by semantic. I think your commit would break all elements your code touched. The current assumption of vlanId an

Re: Router requires upgrade. Unable to send command to router Error

Nitin, I had the same problem, but I fixed it by uploading 4.2 system templates to my secondary storage. Make sure you have the latest too. -alena. From: Nitin Mehta mailto:nitin.me...@citrix.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apa

Re: Reviewboard needs your attention

2013/11/8 Animesh Chaturvedi > Given that we are approaching the 4.3 feature freeze tomorrow if you have > submitted patches and are still pending reviews please reach out to your > reviewers Dear Sebastien & Hugo, I have two pending patches on review board need to review: Rebase OVS (sdnexte

Jenkins build is back to normal : build-master-noredist #1740

See

Re: [PROPOSAL] Liaison with ETSI NFV ISG

2013/11/7 Chiradeep Vittal > While I appreciate the confidence, there are weekly calls to attend. > The particularly interesting one (MANO WG) happens on Wednesdays at 6am > PST. > I've tried but there's very little chance that I can attend these > meetings. > Anybody in a more agreeable timezone

Re: Review Request 13072: Print agent host name in logging of agent commands

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13072/#review28482 --- Reminder - Hi, This review has been pending for long. Request the r

Re: Reviewboard needs your attention

Hi Animesh, For whatever reason, I can no longer edit https://reviews.apache.org/r/13865/. Any thoughts on this? Thanks! On Thu, Nov 7, 2013 at 6:41 PM, Nguyen Anh Tu wrote: > 2013/11/8 Animesh Chaturvedi > > > Given that we are approaching the 4.3 feature freeze tomorrow if you have > > su

Re: Review Request 13865: Extend support of CloudStack-managed storage to KVM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13865/#review28483 --- Reminder - Hi, This review has been pending for long. Please update

Re: Review Request 12098: Add PCI Passthrough to CloudStack on KVM Hypervisor

> On Aug. 7, 2013, 10:30 a.m., Pawit Pornkitprasan wrote: > > I will rebase it in a few days. > > > > This patch only includes support for KVM, but the concepts used can apply > > for all hypervisors supporting PCI Passthrough so additional patches can > > add support for XenServer. (Just to b

Re: Review Request 14023: NetworkManager Exception Fix Due to call to getDhcpServiceProvider()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14023/#review28485 --- Reminder - Hi, This review has been pending for long. Please update

Re: Review Request 13865: Extend support of CloudStack-managed storage to KVM

Hi, I just sent an e-mail to Animesh about this as I can no longer edit this review for some reason. Thanks! On Thu, Nov 7, 2013 at 7:04 PM, Amogh Vasekar wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13865/ > > Reminder - > Hi, > This r

Re: Review Request 12883: DNS value in Zone is not getting set in the Network created for the Zone

> On July 30, 2013, 3:57 a.m., Soheil Eizadi wrote: > > In researching this more look like there is a code in NetworkManager that > > will set the Network DNS Domain, given proper DNS Capabilities > > (Capability.AllowDnsSuffixModification) is set in the DNS Offering. I need > > to investigate

RE: [ACS 421] Upgrade testing

Hi, Looks like Schuberg Philis is covering 4. 1.1 -> 4.2.1 upgrade We ( Citrix systems) would cover 4.2 -> 4.2.1 I think these two paths might be enough for this minor release. I think RC is going to cut soon so these need to be covered before that. Thanks /sudha -Original Message-

Re: ComponentContext :: getComponent() ?

Not really. There is a away but it's intended for only framework code. The API framework uses it. For regular business logic, I've yet to see any good reason on why you need to look up things in the application context. There is typically a cleaner way. Darren > On Nov 6, 2013, at 2:19 PM,

Re: Storage overprovisioning for RBD / disable threshold?

Please find answers inline On 07/11/13 3:32 AM, "Indra Pramana" wrote: >Dear all, > >I am using CloudStack 4.2.0, KVM hypervisors, Ceph RBD primary storage and >NFS secondary storage. > >I tried to change the storage.overprovisioning.factor from 1 to 2 under >Global Settings, and it doesn't seem

Re: Review Request 15263: Added fix for bug 5056.

> On Nov. 7, 2013, 5:05 p.m., SrikanteswaraRao Talluri wrote: > > tools/marvin/marvin/remoteSSHClient.py, line 116 > > > > > > it should be self.keyPairFiles instead of keyPairFiles > > > > again keyPairFiles

Re: Review Request 15263: Added fix for bug 5056.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15263/#review28494 --- Ship it! - SrikanteswaraRao Talluri On Nov. 6, 2013, 1:44 p.m., S

Re: Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15241/#review28495 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Nov. 6, 2013, 2:

Re: Review Request 14872: CLOUDSTACK-4934: Rename Limit resources tests so that they have unique names

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14872/#review28496 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Oct. 23, 2013, 8

[ACS4.2.1]RC Date

We are coming close to a RC. There are still some criticals open, have spoken to assignees and they are confident that these will be closed this week. Sudha has already asked for volunteers to test 4.1.1->4.2.1 upgrade paths and we have folks who have volunteered. I will also request the commun

Re: Review Request 14872: CLOUDSTACK-4934: Rename Limit resources tests so that they have unique names

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14872/#review28497 --- Commit 5d6d14bc1f1579006406a68cd069fba14ba3fa0c in branch refs/heads

Re: Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

> On Nov. 8, 2013, 4:30 a.m., SrikanteswaraRao Talluri wrote: > > Ship It! These two files are already covered as part of https://reviews.apache.org/r/14872/ - SrikanteswaraRao --- This is an automatically generated e-mail. To reply, v

RE: Reviewboard needs your attention

I can see it fine, were you logged in at that time > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Thursday, November 07, 2013 6:03 PM > To: dev@cloudstack.apache.org > Subject: Re: Reviewboard needs your attention > > Hi Animesh, > > For whateve

Re: Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

> On Nov. 8, 2013, 4:30 a.m., SrikanteswaraRao Talluri wrote: > > Ship It! > > SrikanteswaraRao Talluri wrote: > These two files are already covered as part of > https://reviews.apache.org/r/14872/ ok. - Santhosh --- This is an au

Re: Review Request 14872: CLOUDSTACK-4934: Rename Limit resources tests so that they have unique names

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14872/#review28500 --- Commit 8f76c49a994a630bfdd583c3515ac06c11c7c662 in branch refs/heads

Re: pool.storage.capacity.disablethreshold global config

>From what I understand, now there is a new interface Configurable that exposes >the config parameters. In this case if you look at the >CapacityManagerImpl.java, there is a method getCongigKeys() that return the >config parameters. During MS startup all these are automatically populated in >th

Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/ --- Review request for cloudstack and SrikanteswaraRao Talluri. Bugs: CLOUDSTACK-50

Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15342/ --- Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri. B

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28506 --- i can't see the diff - SrikanteswaraRao Talluri On Nov. 8, 2013,

Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15342/#review28507 --- Ship it! Ship It! - Girish Shilamkar On Nov. 8, 2013, 7:07 a.m.,

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/ --- (Updated Nov. 8, 2013, 7:16 a.m.) Review request for cloudstack and Srikanteswa

Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15342/#review28508 --- Commit b9547b07c059d229e6aed6213366cc3ab59fa26e in branch refs/heads

Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15342/#review28509 --- Commit 3c350ab0b8ce1687a8f7e27e9e2f8e8d0fa68c47 in branch refs/heads

Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14874/#review28510 --- 4.2: ae852adabb70f43d6d349dd4ab56dc3116e10aed - Girish Shilamkar

zone-wide primary storage: 2 possible UI bugs

I have observed two possible UI bugs that have to do with 4.2's new feature "zone-wide primary storage": 1. Volumes with zone-wide primary storage show no Hypervisor type in Storage tab. In the Storage tab, volumes that are placed in zone-wide primary storage have their Hypervisor field empty. So

<    1   2