Review Request 12849: added backwards compatibility code to Networks enums

2013-07-23 Thread daan Hoogland
://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: System VMs not coming up due to https://reviews.apache.org/r/12685/

2013-07-23 Thread Daan Hoogland
Koushik, please review https://reviews.apache.org/r/12849/ It should solve the problem. regards, Daan On Mon, Jul 22, 2013 at 8:40 PM, Daan Hoogland wrote: > Ignore my remark about master. I have been looking > At an old log. > I looked some more into this and it seems the eve

Re: [DESIGN] Why is enum a class...

2013-07-24 Thread Daan Hoogland
Thanks Frank and John, I've been strugling with the enums in Networks.java. A discussion on those is needed I think. BroadcastDomainType seems to be a hybrid of a stable and a plugin extendible enum. It breaks my mind and my every change. regards, Daan On Tue, Jul 23, 2013 at 8:37 PM,

[DISCUSS} enums in Networks,.java

2013-07-24 Thread Daan Hoogland
there has been some discussion on the proper use of enums recently. I've been strugling with the enums in Networks.java. A discussion on those is needed I think. BroadcastDomainType seems to be a hybrid of a stable and a plugin extendible enum. It breaks my mind and my every change. There are more

Re: [DESIGN] Why is enum a class...

2013-07-24 Thread Daan Hoogland
Burwell wrote: > Daan and Frank, > > First, I think its important to separate when and where an enumeration > should be used from how to best employ enums when their use is appropriate. > Enums are a finite set of values defined at compile-time. Therefore, they > shouldn't

RE: Review Request 12685: CLOUDSTACK-1532 added utility functions to scan URIs

2013-07-24 Thread Daan Hoogland
school. Op 24 jul. 2013 19:19 schreef "Alex Huang" het volgende: > Daan, > > ** ** > > Sorry for the late reply. Now, it kinda goes into code/design > philosophy. I’ll tell you what mine is. > > ** ** > > There’s one school of thought that code sh

Re: Physical Network / Guest Network / Service Offering Interactions for Basic Networking

2013-07-26 Thread Daan Hoogland
Soheil, I noticed your post not being replied to so I wondered what it was about, I can make a guess but it doesn't contain a question, only an observation. Do you mean to have some question about the behavior you are seeing? regards, Daan Hoogland On Fri, Jul 19, 2013 at 7:57 PM, S

unanswerred questions 15-21 juli

2013-07-26 Thread Daan Hoogland
question. given the volume of this list I don't think that is a bad score. It is still 4 missed opportunities of course. Over the time I have been part of this community it has been my impression that this number is rather stable. The week after ccc it was 6. greeting from earth, Daan

Re: unanswerred questions 15-21 juli

2013-07-26 Thread Daan Hoogland
Do you want me to include links to those four messages or do you state that there are more, that I have missed? Daan On Fri, Jul 26, 2013 at 11:55 AM, Prasanna Santhanam wrote: > On Fri, Jul 26, 2013 at 11:47:55AM +0200, Daan Hoogland wrote: > > H, > > > > Some

Re: unanswerred questions 15-21 juli

2013-07-26 Thread Daan Hoogland
essage/acdyq637gdlmch4g storage.max.volume.size by Damien Mannix And I asked for clarification on the question which was then picked up by Chiradeep so I am leaving the fourth one out On Fri, Jul 26, 2013 at 1:07 PM, Prasanna Santhanam wrote: > On Fri, Jul 26, 2013 at 01:02:49PM +0200, Daan

Re: Build failure on master...anyone know what this is?

2013-07-26 Thread Daan Hoogland
Donal, The page (found after following instructions on the page not found page) says, mvn clean install -P developer,systemvm How is this deprecated? Or do you mean that not adding the -P option is? On Fri, Jul 26, 2013 at 10:51 AM, Donal Lafferty wrote: > AFAIK, using 'mvn clean install' to b

Re: unanswerred questions 15-21 juli

2013-07-26 Thread Daan Hoogland
I could for the coming time. At the moment I use the gmail label and thread functionality. I am only subscribed to the user list with my schubergphilis.com address so will have a look over the weekend about what is wise and convenient. regards, Daan On Fri, Jul 26, 2013 at 5:19 PM, Sebastien

Re: [DISCUSS] maven dependencies...

2013-07-27 Thread Daan Hoogland
ship it;) On Sat, Jul 27, 2013 at 12:09 AM, Alex Huang wrote: > Everyone, > > After looking around in the maven documentation, I realized the way we are > specifying dependencies is not quite right for a large project such as ours. > > Currently, almost every project declare their own dependenc

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-07-27 Thread daan Hoogland
well. tested by instantiating a zone. There are no upgrade (database) aspects to this as far as I can see. - daan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12849/#review23686

Re: [DISCUSS] Compatibility issue between network plugins and hypervisors

2013-07-27 Thread Daan Hoogland
H, isn't it the responsibility of the administrator to setup the cloud in a proper way? hypervisor and network implementations may enhance their capabilities at minor upgrades so it will not be easy to keep checks on this up to date in cloudstack. Am I missing the point here? regards, Daan

Re: Review Request 12985: CLOUDSTACK-3682: NPE in BridgeVifDriver causing systemvm startup failure in KVM

2013-07-27 Thread daan Hoogland
review that patch this one does not take into account that some uri shouldn't have // after : - daan Hoogland On July 27, 2013, 12:25 a.m., Toshiaki Hatano wrote: > > --- > This is an automatically generated e-mail. T

rejected mail by dev.apache.org

2013-07-27 Thread Daan Hoogland
Hello, I replied to a message a minute ago and got Delivery to the following recipient failed permanently: dev@cloudstack.apache.org Technical details of permanent failure: Google tried to deliver your message, but it was rejected by the server for the recipient domain cloudstack.apache.or

Re: rejected mail by dev.apache.org

2013-07-27 Thread Daan Hoogland
thanks, i c&p'd a link from another mail, big sin of course. On Sat, Jul 27, 2013 at 1:23 PM, Ian Duffy wrote: > Just check your not sending emails in HTML. They get rejected unless > they are plain text :-) > > On 27 July 2013 12:21, Daan Hoogland wrote: > > He

Re: [DISCUSS] vlan uri format change

2013-07-27 Thread Daan Hoogland
The original patch was committed under Alex' account. Work on the networks enums is far from done with https://reviews.apache.org/r/12849/. I commented on the one you are mentioning, Prasanna. regards, Daan On Sat, Jul 27, 2013 at 12:41 PM, Koushik Das wrote: > I raised this in a

Re: Review Request 13004: Bug:advance zone, create public Network with vlan id specified, but the portgroup is cloud.public.untagged.0.1-vSwitch0 not cloud.public.[vlanid].0.1-vSwitch0

2013-07-29 Thread daan Hoogland
called when BroadcastDomainType is Vlan, Storage or Mido and not when it is for instance Lswitch. - daan Hoogland On July 28, 2013, 11:49 a.m., Jijun wrote: > > --- > This is an automatically generated e-mail. To reply, visit

uuid lost on xenserver?

2013-07-29 Thread Daan Hoogland
LS, while trying an update path from 4.0.2 to 4.1.1 I get the appended exceptions. It does not seem to have to do with the upgrade itself as after starting a 4.0 management server with the old database the problem persists. Can anyone tell me what this means? thanks, Daan 2013-07-29 18:59

Re: Review Request 13004: Bug:advance zone, create public Network with vlan id specified, but the portgroup is cloud.public.untagged.0.1-vSwitch0 not cloud.public.[vlanid].0.1-vSwitch0

2013-07-29 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13004/#review24213 --- Ship it! Ship It! - daan Hoogland On July 30, 2013, 12:58 a.m

Re: [DISCUSS] Compatibility issue between network plugins and hypervisors

2013-07-30 Thread Daan Hoogland
2:50 AM, Toshiaki Hatano wrote: > Daan, > > In my opinion, it's responsibility of the developer to provide > information and to stop Ops from failing over cliff. > Obviously Dev knows how code works (sometime not :), but Ops doesn't. > So, if Dev deny to help Ops, ho

Re: [DISCUSS] Should we be releasing -beta releases?

2013-07-31 Thread Daan Hoogland
H Ilya, I am working on a paralel track. At Schuberg Philis we have a version that contains some code from 4.2 and a patch that I cannot get committed to master. I don't want to release this in the open, but am interested in your considerations on the subject. regards, Daan On Wed, Jul 31,

Re: [DISCUSS] Should we be releasing -beta releases?

2013-08-01 Thread Daan Hoogland
merged in master. > What features are we talking about? I gues our focus is hastening our own development and not backporting. regards, Daan

Re: Review Request 12185: Remove dependency on backport-util.concurrent, and just use built in classes

2013-08-01 Thread daan Hoogland
h and build it. - daan Hoogland On June 29, 2013, 2:34 a.m., Dave Brosius wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Review Request 11861: double slash fix for windows based nfs servers [CLOUDSTACK-2968]

2013-08-01 Thread daan Hoogland
, daan Hoogland

upgrade path 4.1.1 -> 4.2/master

2013-08-01 Thread Daan Hoogland
s this not automatic this time? sorry if I missed the mail thread explaining all of this, Daan Hoogland

Re: How to create Port Forwarding rule for VM through UI?

2013-08-01 Thread Daan Hoogland
network -> -> IP Adresses -> -> configuration -> Port Forwarding On Mon, Jul 22, 2013 at 12:09 PM, Gaurav Aradhye wrote: > Hi all, > > Can anybody tell me how to create port forwarding rule for a specific vm > through Cloudstack UI? > You can just mention the navigational steps. > > Regards, >

unanwerred questions on dev

2013-08-01 Thread Daan Hoogland
? alternatively they could go to marketing or dev to dev and users to users. That last one is not good marketing i think :P. regards, Daan

we like dev better then users :(

2013-08-01 Thread Daan Hoogland
Marc regards, Daan

Re: upgrade path 4.1.1 -> 4.2/master

2013-08-01 Thread Daan Hoogland
Harikrishna, I suppose I have to register those templates before upgrading; i.e. in the 4.1.1 version management server. Is this procedure described somewhere? Do I bake the templates myself first or should I use publicly offered templates? thanks, Daan On Thu, Aug 1, 2013 at 3:16 PM

Re: we like dev better then users :(

2013-08-01 Thread Daan Hoogland
Wido, I am not urging you or anybody to keep better track of this. It is a marketing issue. That is the only point I am trying to make. I am notorious at not reading user lists ever, myself. DaanOps On Thu, Aug 1, 2013 at 3:36 PM, Wido den Hollander wrote: > On 08/01/2013 03:31 PM, D

Review Request 13192: create private network test

2013-08-01 Thread daan Hoogland
://reviews.apache.org/r/13192/diff/ Testing --- Thanks, daan Hoogland

Re: upgrade path 4.1.1 -> 4.2/master

2013-08-01 Thread Daan Hoogland
d) any change I can salvage this installation to run with master? I.E. add the templates outside cloudstack somehow? (and promise I will never do it again) On Thu, Aug 1, 2013 at 4:43 PM, Harikrishna Patnala wrote: > My comments inline. > > Thanks > Harikrishna > On 01-Aug-2013,

Re: [DISCUSS] Should we be releasing -beta releases?

2013-08-01 Thread Daan Hoogland
9:51 AM >> To: dev@cloudstack.apache.org >> Subject: RE: [DISCUSS] Should we be releasing -beta releases? >> >> Daan and Hugo, >> >> This is just my opinion, you should consider merging this feature into master >> when time allows - this way you don'

Re: we like dev better then users :(

2013-08-01 Thread Daan Hoogland
list first, before tone down their mails. I do not know if we are equipped to handle this properly. One thing I can do is find an earlier moment next week to make my report. regards, Daan On Thu, Aug 1, 2013 at 7:24 PM, Kelcey Jamison Damage wrote: > You can always send them over to IRC where

Re: upgrade path 4.1.1 -> 4.2/master

2013-08-01 Thread Daan Hoogland
> On 01-Aug-2013, at 8:42 PM, Daan Hoogland wrote: > >> I have allready run master against this dev env, and added the >> mentioned entry in the upgrade map. Now I cannot access the database >> using 4.1.1 anymore because of >> ERROR [agent.manager.AgentManagerImpl] (A

Re: Do we allow ENUM database types?

2013-08-01 Thread Daan Hoogland
I agree with the string as db value. reordering or inserting a value, which might make for a more logical order, will break backwards compatibility using ordinals. Daan On Thu, Aug 1, 2013 at 10:43 PM, Alex Huang wrote: > Wido, > > We actually have plenty of places that uses enums.

Re: Review Request 13192: create private network test

2013-08-01 Thread Daan Hoogland
tack-master-with-patch #102 is : > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/102/ > > > - Jenkins Cloudstack.org > > > On August 1st, 2013, 2:52 p.m. UTC, daan Hoogland wrote: > > Review request for cloudstack, Koushik Das and Sheng Yang. > By daan Hoogla

[Responsiveness report] 2013w30 (was: we like dev better then users :( )

2013-08-01 Thread Daan Hoogland
sure, this one allright? On Fri, Aug 2, 2013 at 12:54 AM, Alex Huang wrote: > Hi Daan, > > Sorry I meant to subject tag your email so people who's interested in helping > can easily get to them. > > --Alex > >> -Original Message- >> From: Daan Ho

reviewer wanted

2013-08-02 Thread Daan Hoogland
Can https://reviews.apache.org/r/13192/ be reviewed please. regards, Daan

Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-02 Thread daan Hoogland
--- dealt with some warnings in NetworkServiceImpl Diffs - server/src/com/cloud/network/NetworkServiceImpl.java ff753f4 Diff: https://reviews.apache.org/r/13240/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
kTrace? This way the logfile will > > contain the error. will do - daan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13192/#review24547 ---

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
/network/CreatePrivateNetworkTest.java PRE-CREATION Diff: https://reviews.apache.org/r/13192/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
/CreatePrivateNetworkTest.java PRE-CREATION Diff: https://reviews.apache.org/r/13192/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-03 Thread daan Hoogland
://reviews.apache.org/r/13240/diff/ Testing --- Thanks, daan Hoogland

Re: Physical Network / Guest Network / Service Offering Interactions for Basic Networking

2013-08-03 Thread Daan Hoogland
:) great, I had to step through the createNetwork code quite a lot for creating a private vpc gateway, before I found out it worked ok. Daan On Fri, Aug 2, 2013 at 8:32 PM, Soheil Eizadi wrote: > Following up this thread, I initialized my database and created a new > deployment and ve

api calls in 4.1.1

2013-08-05 Thread Daan Hoogland
H, What is the status of 4.1.1 at this moment. I am getting several reports of api calls not working as before. I saw the vote passing by but didn't give them much attention. In jira I don't see much mention of such issues. Is this noticed by anybosy else? regards, Daan

Re: Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-05 Thread daan Hoogland
. Changes --- adding Sheng Repository: cloudstack-git Description --- dealt with some warnings in NetworkServiceImpl Diffs - server/src/com/cloud/network/NetworkServiceImpl.java ff753f4 Diff: https://reviews.apache.org/r/13240/diff/ Testing --- Thanks, daan

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-05 Thread daan Hoogland
/com/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-05 Thread daan Hoogland
--- Thanks, daan Hoogland

database upgrade

2013-08-05 Thread Daan Hoogland
H, What is the policy for adding upgrade paths? I am working on upgrade from 4.1.1 already. So when do we add _upgradeMap.put("4.1.1", new DbUpgrade[] { new Upgrade410to420(), new Upgrade420to430() }); to DatabaseUpgradeChecker? thanks, Daan

Re: unable to view content of cwiki

2013-08-05 Thread Daan Hoogland
I've able to access the wiki all day so far. On Mon, Aug 5, 2013 at 12:43 PM, Rajesh Battala wrote: > Hi All, > > Am not able to view any link of cwiki. > Am getting permission denied to view any content. > Please let me know how to resolve this issue. > > Thanks > Rajesh Battala

Re: [DISCUSS] Should we be releasing -beta releases?

2013-08-05 Thread Daan Hoogland
For this the jenkins builds might be suitable. I think if those are running on all active release branches. On Mon, Aug 5, 2013 at 4:42 PM, Nux! wrote: > On 31.07.2013 16:30, Chip Childers wrote: >> >> On Wed, Jul 31, 2013 at 12:04:50AM +0100, Nux! wrote: >>> >>> On 14.05.2013 15:41, Chip Childer

Re: [DISCUSS] Should we be releasing -beta releases?

2013-08-05 Thread Daan Hoogland
think was the keyword there, ... but I am looking for ya. moment On Mon, Aug 5, 2013 at 5:21 PM, Nux! wrote: > On 05.08.2013 16:09, Daan Hoogland wrote: >> >> For this the jenkins builds might be suitable. I think if those are >> running on all active release branches. >

Re: [DISCUSS] Should we be releasing -beta releases?

2013-08-05 Thread Daan Hoogland
40 PM, Daan Hoogland wrote: > think was the keyword there, ... but I am looking for ya. moment > > On Mon, Aug 5, 2013 at 5:21 PM, Nux! wrote: >> On 05.08.2013 16:09, Daan Hoogland wrote: >>> >>> For this the jenkins builds might be suitable. I think if those

Re: database upgrade

2013-08-05 Thread Daan Hoogland
:) meaning now, i suppose. On Mon, Aug 5, 2013 at 7:12 PM, Chip Childers wrote: > On Mon, Aug 05, 2013 at 12:32:19PM +0200, Daan Hoogland wrote: >> H, >> >> What is the policy for adding upgrade paths? >> >> I am working on upgrade from 4.

Re: api calls in 4.1.1

2013-08-05 Thread Daan Hoogland
AM +0200, Daan Hoogland wrote: >> H, >> >> What is the status of 4.1.1 at this moment. I am getting several >> reports of api calls not working as before. I saw the vote passing by >> but didn't give them much attention. In jira I don't see much mention &g

Re: unable to view content of cwiki

2013-08-06 Thread Daan Hoogland
Rajesh, You must reside behind a proxy or firwall that prohibits you from working. I can access the page you linked us to. regards, Daan On Tue, Aug 6, 2013 at 5:24 AM, Rajesh Battala wrote: > Am still not able to access any pages. > Ex : https://cwiki.apache.org/confluence

Re: Breaking docs out

2013-08-06 Thread Daan Hoogland
one litle spark here; why does every thing need to go in its own repo. I would rather go for putting it all in one repo. not much of a flame but here you go. Daan On Tue, Aug 6, 2013 at 6:25 AM, Radhika Puthiyetath wrote: > Make perfect sense. > + 1 > > -Radhika > > ---

Re: upgrade path 4.1.1 -> 4.2/master

2013-08-06 Thread Daan Hoogland
Thanks Harikrishna, it worked just fine On Thu, Aug 1, 2013 at 11:12 PM, Daan Hoogland wrote: > I did a 4.0 -> 4.1.1 -> master > I can/will go back to 4.0 and report my findings tomorrow > > On Thu, Aug 1, 2013 at 8:02 PM, Harikrishna Patnala > wrote: >> Have y

Review Request 13317: db upgrade 4.1.1 to master

2013-08-06 Thread daan Hoogland
--- not worth much review but have a look if you like Diffs - engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java eb987ea Diff: https://reviews.apache.org/r/13317/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread daan Hoogland
/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread daan Hoogland
/src/com/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: database upgrade

2013-08-06 Thread Daan Hoogland
I did a submission, https://reviews.apache.org/r/13317/ On Mon, Aug 5, 2013 at 8:27 PM, Chip Childers wrote: > On Mon, Aug 05, 2013 at 08:23:48PM +0200, Daan Hoogland wrote: >> :) meaning now, i suppose. >> >> On Mon, Aug 5, 2013 at 7:12 PM, Chip Childers >> wrote: &

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread daan Hoogland
compatibility Diffs (updated) - api/src/com/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread daan Hoogland
/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

weird test failure

2013-08-06 Thread Daan Hoogland
etDuration() >= 1000); s_logger.info("testProfiler() stopped"); } } any clue welcome, Daan

Re: api calls in 4.1.1

2013-08-06 Thread Daan Hoogland
PM, Musayev, Ilya wrote: > Daan, > > The commit CLOUDSTACK-3080 seems like the only change that was made - that > would be related. > > I'm about to try this out and find it - if that's what broke it. > > Thanks > ilya > >> -Original Message-

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread Daan Hoogland
Hi Dave, I see what you mean by colons wouldn't be allowed. Nothing changed but I went for this solution for consistency (over backward compatibility). I will change again. Let me get to work first. nice working around the globe;) Daan On Wed, Aug 7, 2013 at 3:57 AM, Sheng Yang

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-06 Thread Daan Hoogland
not to be used. regards, Daan On Wed, Aug 7, 2013 at 8:52 AM, Daan Hoogland wrote: > Hi Dave, I see what you mean by colons wouldn't be allowed. Nothing > changed but I went for this solution for consistency (over backward > compatibility). I will change again. Let me get to wo

Re: Breaking docs out

2013-08-07 Thread Daan Hoogland
ok, +0 then, I'm off for a coffee. On Tue, Aug 6, 2013 at 10:21 PM, Jessica Tomechak wrote: > I'm +0 on it. Don't really mind either way. > > Jessica T. > > From: Radhika Puthiyetath [radhika.puthiyet...@citrix.com] > Sent: Monday, August 05, 2013 9:25 PM

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
ng in this case? > > Who would require "//" and who wouldn't? Only the mido code needs the explicit // the rest can deal with URI constructors - daan --- This is an automatically generated e-m

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
/com/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
Does this function need to be public? I think getSchemeValue() and > > getTypeOf(String str) should be enough? see the comment above - daan --- This is an automatically generated

Re: weird test failure

2013-08-07 Thread Daan Hoogland
ere > http://www.javatuning.com/why-is-thread-sleep-inherently-inaccurate/ > > > On 8/6/13 8:16 AM, "Daan Hoogland" wrote: > >>H, >> >>I just had a time skew that seems impossible: >> >>if in the folowing test the sleep is 1000 it reports a

Re: IP Address Allocation

2013-08-07 Thread Daan Hoogland
things are falling into place in slow-motion. Thanks Alex! And Soheil for bringing this up. Let's have a hack session on this at the next CCC. On Tue, Aug 6, 2013 at 10:10 PM, Alex Huang wrote: > I'll say it's a bad design. He he I designed it. :( We had to make > some compromises due to

Re: api calls in 4.1.1

2013-08-07 Thread Daan Hoogland
will do, to busy now but is bound to get priority without further pushing from my side:| On Wed, Aug 7, 2013 at 1:37 AM, Musayev, Ilya wrote: > Daan, > > Take this back - the error was on my part it seems (I've used cloudmonkey > with api/secret keys for another test user). >

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Daan Hoogland
Chiradeep, Sheng, Can you guys have a look at this? I cleaned up a bit with Davids guidance. thanks, Daan On Wed, Aug 7, 2013 at 11:40 AM, daan Hoogland wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12849/ > Review re

[Responsiveness report] users 2013w31

2013-08-07 Thread Daan Hoogland
S6.4 + Adv Network + VLAN = ' by Noel Kendall http://markmail.org/message/s75sdw65r6xfa6o7 'Web gui - id instead of name' by Andreas Huser http://markmail.org/message/iacthvbslyema2zg 'cannot create instance from new created template--error "unable to create a deployment for VM"' by William Jiang regards, Daan

Re: Unable to create image on Cloudcentral

2013-08-07 Thread Daan Hoogland
Prabakaran, Can you make 'I got null pointer exception' more explicit? regards, On Thu, Jul 25, 2013 at 12:56 PM, Prabakaran R wrote: > Hi, > > I am trying to create image of virtual machine in cloudcentral using JClouds. > Following is my code snippet > > > CreateTemplateOptions templateOptio

Re: zone create wizard issue with create storagepool

2013-08-07 Thread Daan Hoogland
Talluri, did you check the history of the files involved in creating a zone? regards, On Fri, Jul 26, 2013 at 4:57 AM, Srikanteswararao Talluri wrote: > Createstoragepool fails during zone creation while creating zone using zone > create wizard. This is happening because add host is taking too

Re: Force delete domain through test case

2013-08-07 Thread Daan Hoogland
n't find the specific domain. > Gaurav, as you describe it it sounds as if you are trying to delete the domain twice. Does this make sense? regards, Daan

[Responsiveness report] 2013w31 dev

2013-08-07 Thread Daan Hoogland
with a witty question a minute ago. net result: no mails from week 31 where without response, Daan Hoogland

Re: [Responsiveness report] 2013w31 dev

2013-08-07 Thread Daan Hoogland
less and less a problem :P On Wed, Aug 7, 2013 at 4:34 PM, Musayev, Ilya wrote: > Daan, thanks a million for doing this! > >> -Original Message----- >> From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] >> Sent: Wednesday, August 07, 2013 6:24 AM >>

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Daan Hoogland
you refer to expects something like '://'. The reason mido needs the string concatination is because it can contain 'mido://:'. Daan On Wed, Aug 7, 2013 at 7:12 PM, Donal Lafferty wrote: >This is an automatically generated e-mail. To reply, visit: > https://

Re: weird test failure

2013-08-07 Thread Daan Hoogland
ok, in the mean while please have another look at https://reviews.apache.org/r/12849/ please? On Wed, Aug 7, 2013 at 8:01 PM, Chiradeep Vittal wrote: > I'd say the test needs to be changed to allow for a negative skew. > > On 8/7/13 2:05 AM, "Daan Hoogland" wrote:

Re: Review Board messages

2013-08-07 Thread Daan Hoogland
Along the same line, the mail being archived at markmail is still the incubator list On Thu, Aug 8, 2013 at 3:14 AM, Dave Cahill wrote: > Hi, > > I noticed that Review Board messages are still going to > cloudstack-...@incubator.apache.org. > > Should we switch that to dev@cloudstack.apache.org

Re: [OFFLINE] Offline from Aug 9 to 19

2013-08-07 Thread Daan Hoogland
hide in the mountains or on sea somewhere! On Thu, Aug 8, 2013 at 4:21 AM, Marcus Sorensen wrote: > Is that allowed? > On Aug 7, 2013 8:21 PM, "Chip Childers" wrote: > >> I'm offline from this Friday to the Monday after next (Aug 9 through >> 19). >> >> Happy summer! >>

Re: Review Board messages

2013-08-07 Thread Daan Hoogland
post a ticket on >> http://issues.apache.org/jira/browse/INFRA >> >> On Thu, Aug 08, 2013 at 05:05:48AM +0200, Daan Hoogland wrote: >> > Along the same line, >> > >> > the mail being archived at markmail is still the incubator list >> > >> > On Thu,

db upgrade 4.1.1 -> master

2013-08-08 Thread Daan Hoogland
Can someone please have a look at https://reviews.apache.org/r/13317/ thanks, Daan

firewall rules application

2013-08-08 Thread Daan Hoogland
erRun(Unknown Source) at java.util.concurrent.FutureTask.run(Unknown Source) at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) at java.lang.Thread.run(Unknown Source) Can someone confirm my suspicion? thanks, Daan

Re: Review Request 13426: Update Maven repo location for MidoNet dependencies

2013-08-08 Thread daan Hoogland
tps://reviews.apache.org/r/13426/#comment49032> same as 13425!?! - daan Hoogland On Aug. 8, 2013, 10:05 a.m., Dave Cahill wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

anyone please: firewall rules application

2013-08-08 Thread Daan Hoogland
I feel I am on a ghost hunt. On Thu, Aug 8, 2013 at 10:32 AM, Daan Hoogland wrote: > H, > > I noted that in some of the 4.1 versions I have been testing setting a > firewall rule fails. This seems to be when a router is not fully > initialized, is it? > > the stack trace

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-08 Thread daan Hoogland
) - api/src/com/cloud/network/Networks.java c76c3d4 api/test/com/cloud/network/NetworksTest.java 31114e8 Diff: https://reviews.apache.org/r/12849/diff/ Testing --- Thanks, daan Hoogland

Re: anyone please: firewall rules application

2013-08-08 Thread Daan Hoogland
One extra nuances is that after going from one version to another the error message popped up while creation had actually succeeded. Deleting the rule as a next step failed. anyone been there, done that? On Thu, Aug 8, 2013 at 1:13 PM, Daan Hoogland wrote: > I feel I am on a ghost hunt. >

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-08 Thread daan Hoogland
(updated) --- unit tests for different kind of BroadcastDomainType.values Thanks, daan Hoogland

<    1   2   3   4   5   6   7   8   9   10   >