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

See

Re: Create GRE tunnel failed in 4.2 with XenServer.

On 01/02/14 9:39 PM, "Paul Angus" wrote: >Hey Tuna, (and anyone else working on this) > >I filed bug 5967 which looks like it might relate to this - I was trying >to use GRE tunnel isolation in 4.3 and got VM_REQUIRES_NETWORK errors >from the XenServer when trying to start a new virtual router. >

Review Request 17533: Added changes as per CLOUDSTACK-6006

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17533/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-6006 h

Re: [PROPOSAL] Introduce API returning you an answer from CloudStack storage/host allocators whethere there is enough resources for vm deployment

I agree with Santhosh. Returning appropriate error codes might be enough. Thanks, ~Rajani On 02-Feb-2014, at 11:17 pm, Santhosh Edukulla wrote: > Just a note: > > Instead of making two separate api calls one to check and then for deploy, > may be it can be part of deploy only and check th

RE: CLOUDSTACK-5406

Francois, First can you check if there are disks waiting to expunge. If there are, then the primary storage count will get reduced once the disks gets expunged. Also, apart from primary_storage count, can you check the volume count. --Sanjay -Original Message- From: Francois Gaudreault

Re: Findbugs report on 4.3-forward

Hi Daan/Ian, looks like the change is only in 4.3. Can you please do it on 4.3-forward and master as well. Thanks, ~Rajani On 03-Feb-2014, at 12:21 am, Daan Hoogland wrote: > just pushed :( > > > On Sun, Feb 2, 2014 at 7:50 PM, Ian Duffy wrote: >> Daan, >> >> Have this done already, jus

Re: cloudstack-agent on ubuntu

Hi Anirban, Are you sure you have used "apt-get install cloudstack-agent" and not "apt-get install cloud-agent". I had encountered the same issue and then realized I had installed an outdated version of the agent. The link displayed in google search results takes you to installation guide of 4.0.2

RE: Review Request 15280: CLOUDSTACK-5062: Deleting Load Balancing Rule fails when generating usage events are enabled

Pulled this one into latest 4.3 too > -Original Message- > From: daan Hoogland [mailto:nore...@reviews.apache.org] On Behalf Of > daan Hoogland > Sent: Friday, January 31, 2014 2:27 AM > To: daan Hoogland > Cc: David Grizzanti; Dennis John; cloudstack > Subject: Re: Review Request 15280: C

Re: Review Request 15280: CLOUDSTACK-5062: Deleting Load Balancing Rule fails when generating usage events are enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15280/#review33403 --- Commit 91ec548fc0b08aa4ff8e6aa57fab013ab9892893 in branch refs/heads

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

See

Re: Findbugs report on 4.3-forward

just pushed :( On Sun, Feb 2, 2014 at 7:50 PM, Ian Duffy wrote: > Daan, > > Have this done already, just about to push. > > > On 2 February 2014 18:47, Daan Hoogland wrote: > >> never mind, I thought you are committer. I'll fix >> >> On Sun, Feb 2, 2014 at 6:52 PM, Rajani Karuturi >> wrote: >>

Re: Findbugs report on 4.3-forward

Daan, Have this done already, just about to push. On 2 February 2014 18:47, Daan Hoogland wrote: > never mind, I thought you are committer. I'll fix > > On Sun, Feb 2, 2014 at 6:52 PM, Rajani Karuturi > wrote: > > Sure Daan. Will submit a review request for master and 4.3-forward > tomorrow m

Re: Findbugs report on 4.3-forward

never mind, I thought you are committer. I'll fix On Sun, Feb 2, 2014 at 6:52 PM, Rajani Karuturi wrote: > Sure Daan. Will submit a review request for master and 4.3-forward tomorrow > morning IST. > > Thanks, > ~Rajani > > > > On 02-Feb-2014, at 8:17 pm, Daan Hoogland wrote: > >> On Sun, Feb 2

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

> On Dec. 31, 2013, 2:32 a.m., Amogh Vasekar wrote: > > Reminder- > > Hi, > > The review has been pending for long. Please update the patch since the > > build fails. > > Thanks i think this can be closed (searched master for the old classes) - daan -

Re: Review Request 12228: wip: static resource compression

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review33400 --- client/WEB-INF/web.xml

RE: Jenkins bacd: Simulator-build Job Failing

No changes to be done majorly as such, Thought of using this view of jobs to test any changes for simulator or bug fixes for that matter, but some jobs were not working since many days or weeks, i thought if no body is monitoring this job, i can check to see if i have access. Thanks! Santhosh

Re: Jenkins bacd: Simulator-build Job Failing

On Sun, Feb 02, 2014 at 05:50:28PM +, Santhosh Edukulla wrote: > I have seen few of these links, but can i get access to this node? some of the committers within citrix already have access to the node at this point. what would you like changed on the node? -- Prasanna.,

Re: Findbugs report on 4.3-forward

Sure Daan. Will submit a review request for master and 4.3-forward tomorrow morning IST. Thanks, ~Rajani On 02-Feb-2014, at 8:17 pm, Daan Hoogland wrote: > On Sun, Feb 2, 2014 at 12:48 PM, Rajani Karuturi > wrote: >> what we should use is new String(bytes[]) > > > ok, makes sense. Can you

RE: Jenkins bacd: Simulator-build Job Failing

I have seen few of these links, but can i get access to this node? Santhosh From: Prasanna Santhanam [t...@apache.org] Sent: Sunday, February 02, 2014 12:35 PM To: dev@cloudstack.apache.org Subject: Re: Jenkins bacd: Simulator-build Job Failing On Fri, Jan

RE: [PROPOSAL] Introduce API returning you an answer from CloudStack storage/host allocators whethere there is enough resources for vm deployment

Just a note: Instead of making two separate api calls one to check and then for deploy, may be it can be part of deploy only and check these conditions for resources\any other information and return appropriate codes? This check can be called firsthand as part of deploy\other commands applicabl

Re: Jenkins bacd: Simulator-build Job Failing

On Fri, Jan 31, 2014 at 05:02:16PM +, Santhosh Edukulla wrote: > Team, > > 1. It seems the simulator-gate project in that view, was not able to > fetch the code using jenkins git plugin and is stuck as per below > message. This job is failing continuously with no use as of now. > > 2. If poss

Build failed in Jenkins: build-master #209

See -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on cloudstack-buildslave-centos6-a23 in workspace

Re: Mailing list search still broken (Was: Re: Mailing list search)

On Sun, Feb 2, 2014 at 9:49 AM, Noah Slater wrote: > Hi folks, > > Our mailing list search is still broken. Search isn't broken... it's setup to work the only way that it can work with markmail. > Is there any reason why we're > still pointing people to the Incubator archives? Because [1] != [

Build failed in Jenkins: cloudstack-4.3-maven-build #462

See -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on rpmbuilder-2 in workspace

Mailing list search still broken (Was: Re: Mailing list search)

Hi folks, Our mailing list search is still broken. Is there any reason why we're still pointing people to the Incubator archives? On 25 October 2013 20:58, Noah Slater wrote: > Hi, > > Mailing list search on our site still points to the incubator: > > http://cloudstack.apache.org/mailing-lists.h

Build failed in Jenkins: build-master #208

See -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on cloudstack-buildslave-centos6-a23 in workspace

Re: Review Request 12228: wip: static resource compression

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated Feb. 2, 2014, 5:04 p.m.) Review request for cloudstack, Brian Federle,

Re: Review Request 17335: cleanup around seemingly impossible cast

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17335/ --- (Updated Feb. 2, 2014, 5:03 p.m.) Review request for cloudstack, Alena Prokharc

Re: Add reviewer to Review requests (was: ReviewBoard)

Ah yes, this is how I worked so far. I will keep doing so, I just add the address from the maintainer list as well. However the more reviewer, the less work done. We will see if this helps. On Sun, Feb 2, 2014 at 5:18 PM, Prasanna Santhanam wrote: > the best way is to find the person who change

Re: Add reviewer to Review requests (was: ReviewBoard)

Good advice Prasanna, better then the maintainers list!! On Sun, Feb 2, 2014 at 5:19 PM, Laszlo Hornyak wrote: > Bookmarked, thank you! > > This developer guide I worked with so far is seriously misleading. > > > On Sun, Feb 2, 2014 at 4:27 PM, Daan Hoogland wrote: > >> It should be on the wiki :

Re: Add reviewer to Review requests (was: ReviewBoard)

Bookmarked, thank you! This developer guide I worked with so far is seriously misleading. On Sun, Feb 2, 2014 at 4:27 PM, Daan Hoogland wrote: > It should be on the wiki : > > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Current+Maintainers+Per+Component > > On Sun, Feb 2, 2014 at 4:1

Re: Add reviewer to Review requests (was: ReviewBoard)

the best way is to find the person who changed the file you changed most and include them as reviewer. `git blame` FTW. On Sun, Feb 02, 2014 at 04:29:39PM +0100, Daan Hoogland wrote: > but I would give preference to whoever you discussed a thingy with. > > On Sun, Feb 2, 2014 at 4:27 PM, Daan Hoo

Re: Add reviewer to Review requests (was: ReviewBoard)

but I would give preference to whoever you discussed a thingy with. On Sun, Feb 2, 2014 at 4:27 PM, Daan Hoogland wrote: > It should be on the wiki : > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Current+Maintainers+Per+Component > > On Sun, Feb 2, 2014 at 4:19 PM, Laszlo Hornyak > w

Re: Add reviewer to Review requests (was: ReviewBoard)

It should be on the wiki : https://cwiki.apache.org/confluence/display/CLOUDSTACK/Current+Maintainers+Per+Component On Sun, Feb 2, 2014 at 4:19 PM, Laszlo Hornyak wrote: > Hi Daan, > > I would start doing so right now, but I have never heard of a list of > component maintainers and google didn't

Re: Add reviewer to Review requests (was: ReviewBoard)

Hi Daan, I would start doing so right now, but I have never heard of a list of component maintainers and google didn't find one for me either. Ideally, such list should be here: http://cloudstack.apache.org/developers.html or it should be at least linked from this page. Thank you, Laszlo On Sun

Re: Review Request 16523: Unit tests for Juniper contrail VM Interface Model and Instance Ip Model class.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16523/#review33398 --- please add a license header to both files and resubmit - daan Hoogl

Re: Findbugs report on 4.3-forward

On Sun, Feb 2, 2014 at 12:48 PM, Rajani Karuturi wrote: > what we should use is new String(bytes[]) ok, makes sense. Can you revert my commit and make a new patch? thanks, Daan

Re: Findbugs report on 4.3-forward

I agree that we shouldn’t be using Arrays.toString() here. It would output CSV of values enclosed in ‘[]’ (http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7u40-b43/java/util/Arrays.java/#Arrays.toString%28byte%5B%5D%29

Re: Review Request 17638: Rest client moved to utils. Nvp extended.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17638/#review33396 --- pom.xml Har

Re: Review Request 17497: CLOUDSTACK-5977 - Incorrect String Quotations

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17497/#review33395 --- Ship it! did a litle tidy up 9cb2458a5871307efa6f18143c241b32a9e795

Re: Review Request 17497: CLOUDSTACK-5977 - Incorrect String Quotations

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17497/#review33394 --- Commit 9cb2458a5871307efa6f18143c241b32a9e79540 in branch refs/heads

old review request regarding NetworkManager (now NetworkOrchestrator)

H Murali, You commented to Soheils commit (https://reviews.apache.org/r/14023) That it should be redone for present day master. Is this still valid, meaning I just did should I check it in? regards, Daan

Add reviewer to Review requests (was: ReviewBoard)

LS, I would like to see contributers add reviewers to their submissions. I noticed quite soe old review requests that don't have anyone's attention. There are component 'owners' for everything and I have easy speach as I am not listed as contact for any component but I wouldn't mind being assinged

Re: ReviewBoard

OK - this is now done - we shrank from 93 to 44 reviews due to this. Lets see if we can get these reviewed or rejected and keep up with this. --David On Mon, Jan 27, 2014 at 2:59 PM, David Nalley wrote: > Hi folks: > > ReviewBoard is pretty bloated. There are currently 93 reviews that are open.

Re: Review Request 16424: some net util for dealing with isolation ids

> On Dec. 23, 2013, 1:10 p.m., daan Hoogland wrote: > > added as 94abbb1367bc817bae98f369e78679f0ddb7727f to 4.3 (not in master yet) Commit 04570eefed9a0ee1eca1fd700ed5732ba67150ce in master - daan --- This is an automatically generated

Re: ReviewBoard

I have not. Yesterday RB went down for some period of time, though AFAIK, infra restored it. --David On Sat, Feb 1, 2014 at 4:11 PM, Daan Hoogland wrote: > H David, > > have you started on this yet? > I can't reach the review board at the moment. > > On Mon, Jan 27, 2014 at 8:59 PM, David Nalley