About the 4.5 Compile Error

2014-10-16 Thread Star Guo
Hello, When I compile -b 4.5 , it show the error as bellow: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile (default-compile) on project cloud-server: Compilation failure: Compilation failure: [ERROR]

[EVENT] CloudStack Conference Budapest 2014

2014-10-16 Thread sebgoa
Hi everyone, Don't forget to register for the CloudStack conference in Budapest nov 19-21. http://events.linuxfoundation.org/events/cloudstack-collaboration-conference-europe If you are a committer, email me privately to get the registration code. cheers, -Sebastien

Jenkins build is back to normal : build-4.5 #25

2014-10-16 Thread jenkins
See http://jenkins.buildacloud.org/job/build-4.5/25/changes

RE: About the 4.5 Compile Error

2014-10-16 Thread Sanjay Tripathi
Fixed the error, and 4.5 builds are coming up successfully. --Sanjay -Original Message- From: Star Guo [mailto:st...@ceph.me] Sent: Thursday, October 16, 2014 12:56 PM Cc: dev@cloudstack.apache.org Subject: About the 4.5 Compile Error Hello, When I compile -b 4.5 , it show the error as

merging versus cherry-picking

2014-10-16 Thread Daan Hoogland
H, I noticed a lot of commits on master and 4.5 without any links between them. These could have all been committed on 4.5 and merged back into master leaving a trail of prove that the same code is in both branches. It hurts to see this happen. we are at a brink where we can improve on our way

答复: About the 4.5 Compile Error

2014-10-16 Thread Star Guo
It builds successfully. Star Guo -邮件原件- 发件人: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.com] 发送时间: 2014年10月16日 17:28 收件人: dev@cloudstack.apache.org 主题: RE: About the 4.5 Compile Error Fixed the error, and 4.5 builds are coming up successfully. --Sanjay -Original Message-

Re: merging versus cherry-picking

2014-10-16 Thread sebgoa
On Oct 16, 2014, at 11:34 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: H, I noticed a lot of commits on master and 4.5 without any links between them. These could have all been committed on 4.5 and merged back into master leaving a trail of prove that the same code is in both branches.

Re: merging versus cherry-picking

2014-10-16 Thread Daan Hoogland
​+1​ On Thu, Oct 16, 2014 at 12:02 PM, sebgoa run...@gmail.com wrote: On Oct 16, 2014, at 11:34 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: Proposal: All commits come through github PR, *even* for committers. We declare a moratorium period (agreed suspension of activity)

Re: merging versus cherry-picking

2014-10-16 Thread Daan Hoogland
On Thu, Oct 16, 2014 at 11:34 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: ​...​ I will do a test merge-back of 4.5 to master to see how big the damage is​. ​So I did locally and found:​ commit 148efbb73f0e084614eff62f48ea9fa964c64da8 Merge: 1f8cf0b 420d4e0 Author: Daan Hoogland

Re: merging versus cherry-picking

2014-10-16 Thread Rohit Yadav
Hi, On 16-Oct-2014, at 3:32 pm, sebgoa run...@gmail.com wrote: Proposal: All commits come through github PR, *even* for committers. We declare a moratorium period (agreed suspension of activity) during which direct commit to master is forbidden. Only the master RM is allowed to merge

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/23 Merge 4.5 back into master It would be good to have this as a daily practice You can merge this pull request into a Git repository by running: $ git pull

Re: [VOTE][ACS44]Apache CloudStack 4.4.1 RC 3 in branch 4.4-RC20141014T2316

2014-10-16 Thread Rohit Yadav
Hi, -0 Upgraded from 4.3.1 to 4.4.1, following the docs (http://cloudstack-release-notes.readthedocs.org/en/latest/upgrade/upgrade-4.3.html) it worked and usual VM lifecycle (create, start, stop, reboot etc), networking etc in basic zone. The issue is that, the upgrade docs ask you to

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/23#discussion_r18949755 --- Diff: api/src/com/cloud/event/EventTypes.java --- @@ -454,7 +454,6 @@ public static final String EVENT_BAREMETAL_PXE_SERVER_ADD =

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/23#discussion_r18949772 --- Diff: plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetalKickStartServiceImpl.java --- @@ -238,7 +232,7 @@ private

Re: merging versus cherry-picking

2014-10-16 Thread Rajani Karuturi
+1 on merging(and the proposal). Unlike the belief that merging adds to dev time it actually makes life easier to track a commit (after all, it has the same number of commands and the same conflicts as cherry-pick :) ) We should also define merge strategy for LTS(4.3 and 4.4) branches. ~Rajani

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/23#issuecomment-59347522 Mostly LGTM. I found a pattern that all the baremetal changes are causing issues, from the JIRA issue [1] this feature is in progress and I would only

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/23#issuecomment-59348377 Seems like you are the RM for 4.6/master now Rohit, congratulations. I suggest that you and @ke4qqq do a merge like this on a daily basis. Of course I would like

Re: Review Request 26760: Skip various BVT tests on LXC

2014-10-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26760/#review56916 --- Ship it! 20d3cf1acc535c3380e52e92a4108444a0b593e3 master

[GitHub] cloudstack pull request: Merge 4.5 back into master

2014-10-16 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/23#issuecomment-59356773 Hi Daan, I can help merging things on daily basis but not sure about official RM position :) I'm going ahead with the merge. I was just waiting for Travis to

Re: [VOTE][ACS44]Apache CloudStack 4.4.1 RC 3 in branch 4.4-RC20141014T2316

2014-10-16 Thread Daan Hoogland
On Thu, Oct 16, 2014 at 1:09 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: - Hardcoded systemvm template names in Upgrade440to441 :: updateSystemVmTemplates ​I don't care about this one as the names are a choice and not bound to any real name outside of ACS, the url is but it is under our

RE: [VOTE][ACS44]Apache CloudStack 4.4.1 RC 3 in branch 4.4-RC20141014T2316

2014-10-16 Thread Geoff Higginbottom
+1 Tested Advanced Networking with XenServer 6.2 Previous Cluster Details issue has been resolved Minor bug in the UI with regards to VM Snapshots. If a VM is stopped, and I go to take a VM Snapshot I still get the option of 'Snapshot Memory' which obviously is not possible and results in an

master broken

2014-10-16 Thread Marcus
It seems master is broken since this commit. We probably shouldn't be pushing code into the repo that we haven't tried to compile. commit 148efbb73f0e084614eff62f48ea9fa964c64da8 Merge: 1f8cf0b 420d4e0 Date: Thu Oct 16 12:29:37 2014 +0200 Merge remote-tracking branch 'origin/4.5' into

Re: master broken

2014-10-16 Thread Daan Hoogland
On Thu, Oct 16, 2014 at 6:57 PM, Marcus shadow...@gmail.com wrote: We probably shouldn't be pushing code into the repo that we haven't tried to compile. ​as I understand it it passed travis​, I agree otherwise -- Daan

RE: Help in Adding XenServer to CS Management Server

2014-10-16 Thread Ritu Sabharwal
Thanks Yitao and Pierre for replying! I enabled the Zone and monitored the CS server logs. I did not configured the Secondary Storage earlier and did that. After that I could see that Secondary Storage and Console Proxy VMs are starting and stopping again and again. In the logs: 2014-10-16

Re: master broken

2014-10-16 Thread Nguyen Anh Tu
Yes, I get the same error. --Tuna On Fri, Oct 17, 2014 at 12:03 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Thu, Oct 16, 2014 at 6:57 PM, Marcus shadow...@gmail.com wrote: We probably shouldn't be pushing code into the repo that we haven't tried to compile. ​as I understand it

Re: master broken

2014-10-16 Thread Marcus
I can't rule out that it's specific to me at the moment, but I cleaned my repo and rebuilt with the same failure, then went back to the commit just before this and it succeeded. Wouldn't it have to be pushed first to pass any CI testing? So in theory we should never have CI catch compile errors

Re: master broken

2014-10-16 Thread Daan Hoogland
Marcus did you try the commit just before the merge? given the location of the problem it might well be in the vpc refactor work or a combination of the both. I am trying myself as well now On Thu, Oct 16, 2014 at 7:07 PM, Marcus shadow...@gmail.com wrote: I can't rule out that it's specific

Re: master broken

2014-10-16 Thread Nguyen Anh Tu
I've just tried the commit before HEAD. It worked! --Tuna On Fri, Oct 17, 2014 at 12:10 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: Marcus did you try the commit just before the merge? given the location of the problem it might well be in the vpc refactor work or a combination of the

Re: master broken

2014-10-16 Thread Marcus
The commit just before and just after the merge seem to work. They're both from the 15th. On Thu, Oct 16, 2014 at 11:12 AM, Nguyen Anh Tu ng.t...@gmail.com wrote: I've just tried the commit before HEAD. It worked! --Tuna On Fri, Oct 17, 2014 at 12:10 AM, Daan Hoogland

Re: master broken

2014-10-16 Thread Daan Hoogland
what do you mean by just after the merge? On Thu, Oct 16, 2014 at 7:28 PM, Marcus shadow...@gmail.com wrote: The commit just before and just after the merge seem to work. They're both from the 15th. On Thu, Oct 16, 2014 at 11:12 AM, Nguyen Anh Tu ng.t...@gmail.com wrote: I've just tried

RE: merging versus cherry-picking

2014-10-16 Thread Sheng Yang
I totally agree that we should have some staging tree for this purpose. But I would prefer deploying gerrit on this rather than using github. Also, using gerrit would make it possible to host our own Jenkins instance for testing, rather than depends on TravisCI which is very limited and shared

RE: merging versus cherry-picking

2014-10-16 Thread Sheng Yang
Oh, sorry Daan, seems I am a little out dated. Just found 4.5 has been branched without the refactor. --Sheng On Oct 16, 2014 11:08 AM, Sheng Yang sh...@yasker.org wrote: I totally agree that we should have some staging tree for this purpose. But I would prefer deploying gerrit on this rather

Re: master broken

2014-10-16 Thread Marcus
I mean as they are ordered in the log. The commits before and after the merge are both working points. commit 20d3cf1acc535c3380e52e92a4108444a0b593e3 Author: Alex Brett alex.br...@citrix.com Date: Wed Oct 15 18:50:57 2014 +0100 CLOUDSTACK-7727 Skip various BVT tests on LXC A number

Re: merging versus cherry-picking

2014-10-16 Thread Daan Hoogland
Sheng, I merged after the branch not before. Did this end up in 4.5? On Thu, Oct 16, 2014 at 8:08 PM, Sheng Yang sh...@yasker.org wrote: I totally agree that we should have some staging tree for this purpose. But I would prefer deploying gerrit on this rather than using github. Also, using

Re: master broken

2014-10-16 Thread Daan Hoogland
I found some culprits, it is in partly reverted code that was committed before the cut of master and 4.5. All to do with Baremetal. I am having a look. On Thu, Oct 16, 2014 at 8:54 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Thu, Oct 16, 2014 at 8:17 PM, Marcus shadow...@gmail.com

Re: [VOTE][ACS44]Apache CloudStack 4.4.1 RC 3 in branch 4.4-RC20141014T2316

2014-10-16 Thread Mike Tutkowski
+1 (binding) (based on the regression tests I typically run) On Thu, Oct 16, 2014 at 7:50 AM, Geoff Higginbottom geoff.higginbot...@shapeblue.com wrote: +1 Tested Advanced Networking with XenServer 6.2 Previous Cluster Details issue has been resolved Minor bug in the UI with regards to

Re: master broken

2014-10-16 Thread Daan Hoogland
I can't seem to get it working by reverting, neither the merge commit nor individual related commits. @Frank: can we disable baremetal all together until you are done? On Thu, Oct 16, 2014 at 9:43 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: I found some culprits, it is in partly reverted

RE: master broken

2014-10-16 Thread Frank Zhang
Do you know what part of baremetal causes the issue? I built master on Wed and didn't see any issue. I will give it a try right now. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thursday, October 16, 2014 2:28 PM To: dev; Frank Zhang Subject: Re:

Re: master broken

2014-10-16 Thread Daan Hoogland
On Thu, Oct 16, 2014 at 11:37 PM, Frank Zhang frank.zh...@citrix.com wrote: Do you know what part of baremetal causes the issue? No it is due to the merge back of the 4.5 branch I built master on Wed and didn't see any issue. I will give it a try right now. ​I probably resolved some

Review Request 26842: CLOUDSTACK-7745: Fixed the script 'test_ssvm.py' - Private IP of System VMs can change on systemVMs reboot

2014-10-16 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26842/ --- Review request for cloudstack, edison su, sangeetha hariharan, and

Re: merging versus cherry-picking

2014-10-16 Thread David Nalley
I think this needs it's own thread as a [PROPOSAL] So a couple of comments: 1. I think we need to do things in a uniform way. Having patches committed directly, having patches hit RB and having patches hit GH is fail. 2. I think we should gate patches, even from committers. I'd personally like to

RE: System VM build job slaves are down

2014-10-16 Thread Rayees Namathponnan
Hi Leo, I have restored Citrix systemvm build machine and added as slave in Jenkins.buildacloud.org; node systemvmbuilder Systemvm builds hung, while copying VBoxGuestAdditions_4.2.6.iso from Jenkins master, Build machine added as jnpl client, do we need to make any other configuration to

Re: Issue VM Access

2014-10-16 Thread Yitao Jiang
Try loging into vms each other using ssh ,maybe your template play trick. Or Using vnc-viewer logged into vm to see if the sshd deamon running --- Thanks, Yitao(依涛 姜) jiangyt.github.io On Fri, Oct 17, 2014 at 5:48 AM, Bikas mbik...@uic.edu wrote: Thanks for the replies. I created another VM

Re: Review Request 26842: CLOUDSTACK-7745: Fixed the script 'test_ssvm.py' - Private IP of System VMs can change on systemVMs reboot

2014-10-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26842/#review57111 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Oct. 16, 2014,