Re: Review Request 12818: Update LDAPPlugin branch with master

2013-07-22 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12818/ --- (Updated July 23, 2013, 6:51 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Reformatting UI code

2013-07-22 Thread Ian Duffy
Hi, Very sorry about that! Glad to hear you got the conflicts resolved. On 22 July 2013 23:54, Jessica Wang wrote: > Sebastien, > > Brian and I are fix 4.2 bugs. > Any fix for 4.2 bugs should go to both 4.2 branch and master branch. > That's why Brian and I still need to merge our check-in betwe

Re: Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-07-22 Thread Toshiaki Hatano
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12623/ --- (Updated July 23, 2013, 6:45 a.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 12838: CLOUDSTACK-3488: listStoragePools throws NPE if any of the storage pools are not in the right state.

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12838/#review23680 --- Review 12838 PASSED the build test The url of build cloudstack-maste

Re: Regression failures

2013-07-22 Thread Prasanna Santhanam
Thanks Sowmya, I see more issues in the list now when the component is Automation. On Tue, Jul 23, 2013 at 06:23:04AM +, Sowmya Krishnan wrote: > This gives better results (specifically for the test script issues): > """ > project = CloudStack AND issuetype = Bug and (status = Open or status

RE: Regression failures

2013-07-22 Thread Sowmya Krishnan
This gives better results (specifically for the test script issues): """ project = CloudStack AND issuetype = Bug and (status = Open or status = Reopened or status = "Ready To Review") and (summary ~ Automation OR component = Automation) order by key desc """ > -Original Message- > F

Re: Regression failures

2013-07-22 Thread Prasanna Santhanam
We've been working regularly on fixing the regression tests trying to get it into good shape for 4.2. You will see bugs being filed from regular test runs on citrix internal environments. The following filter can be used in JIRA if you would like to help in fixing the bugs and/or review fixed bugs

Review Request 12838: CLOUDSTACK-3488: listStoragePools throws NPE if any of the storage pools are not in the right state.

2013-07-22 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12838/ --- Review request for cloudstack and Devdeep Singh. Bugs: CLOUDSTACK-3488 Reposi

Re: CloudStack 4.2 – Build and Packaging on RHEL 5.4

2013-07-22 Thread Hugo Trippaers
If i remember correctly these directories are not automagically created on redhat 5.x systems. You need to use the default location in the var directory probably so you need to make some changes to the package script to deal with that. Cheers, Hugo Sent from my iPhone On 23 jul. 2013, at 05:

Re: Review Request 12819: WIP: Add LDAP Account addition UI. Feedback and Help wanted.

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12819/#review23679 --- Review 12819 PASSED the build test The url of build cloudstack-maste

RE: CloudStack 4.2 – Build and Packaging on RHEL 5.4

2013-07-22 Thread Rayees Namathponnan
Re sending ; seems mail got bounced last time. From: Rayees Namathponnan Sent: Monday, July 22, 2013 8:24 PM To: dev@cloudstack.apache.org Subject: CloudStack 4.2 – Build and Packaging on RHEL 5.4 Hi All, I am trying to create 4.2 build and then package on RHEL 5.4, initially faced some i

Re: Tracing registerTemplate API, is it possible to connect Eclipse debugger to System VMs?

2013-07-22 Thread Prasanna Santhanam
Saved to a wiki: https://cwiki.apache.org/confluence/x/kQ-8AQ On Mon, Jul 22, 2013 at 08:53:36PM +, Edison Su wrote: > Yah, you need Eclipse connects to the java agent running on SSVM. > You can use the following steps: > 1. login into ssvm, either console proxy or ssh(port 3922) > 2. kill all

Re: Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12623/#review23677 --- Review 12623 failed the build test : FAILURE The url of build clouds

Re: Review Request 12823: CLOUDSTACK-3720. Fix resource types in description of listResourceLimits API command.

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12823/#review23676 --- Review 12823 failed the build test : FAILURE The url of build clouds

Re: Review Request 12818: Update LDAPPlugin branch with master

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12818/#review23675 --- Review 12818 failed the build test : FAILURE The url of build clouds

CloudStack 4.2 – Build and Packaging on RHEL 5.4

2013-07-22 Thread Rayees Namathponnan
Hi All, I am trying to create 4.2 build and then package on RHEL 5.4, initially faced some issue with RPM “topdir”, fixed those issue in local machine (changes are below) mkdir -p $RPMDIR/SPECS +mkdir -p $RPMDIR/BUILD mkdir -p $RPMDIR/SOURCES/$PACK_PROJECT-$VERSION (cd ../../; tar -c --

Re: Basic Zone network configuration

2013-07-22 Thread Ahmad Emneina
Hey Thomas, Basic zones dont explicitly work with vlans. If you have vlan's behind the scenes, say native vlans, it shouldnt be an issue as long as your relevant networks can route to each other. Say for example I have my management server sitting on the 192.168.10.0/24segment with hosts, system

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

2013-07-22 Thread Alex Huang
+1 to do this for any enum that was designed to be added to by the plugins. I actually wrote a class for this before called Constant that did exactly this. The problem I had was that there was no way to guarantee compile time enumeration of all of the values like Enum did so it's possible for

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

2013-07-22 Thread Frank Zhang
Frankly speaking, if we are going to change enum, I would suggest not using enmu anymore, instead, defining our own class like: public class VmType { private static Map types = Collections.synchronizedMap(new HashMap()); private final String typeName; public VmType (Stri

RE: Inclusion of commons-httpclient-contrib jar

2013-07-22 Thread Frank Zhang
I prefer to just use httpclient-contribs.jar as it has been proven by lots of users. Reimplementing such code brings no gain but maintenance effort. > -Original Message- > From: Amogh Vasekar [mailto:amogh.vase...@citrix.com] > Sent: Monday, July 22, 2013 5:17 PM > To: dev@cloudstack.apa

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

2013-07-22 Thread Alex Huang
BTW, this code already shows a bug that stems from the static method usage. It says ElasticIpVm is not a system vm (which I don't believe is true). Probably because whoever added elastic ip vm didn't see the static method and so didn't add the vm into the method. --Alex > -Original Messa

Inclusion of commons-httpclient-contrib jar

2013-07-22 Thread Amogh Vasekar
Hi All, I was working on a HTTPS related fix, and found this package in the codebase : https://github.com/apache/cloudstack/tree/master/plugins/network-elements/c isco-vnmc/src/org/apache/commons/httpclient/contrib/ssl I need to use the same classes, which are already a part of httpclient-contri

[DESIGN] Why is enum a class...

2013-07-22 Thread Alex Huang
I just went over this code and thought it was related and might be interested to other developers. What's the difference between declaring a enum like this public enum Type { User, DomainRouter, ConsoleProxy, SecondaryStorageVm, ElasticIpVm,

Coolest Clouds Video Proposal (Time Sensitive)

2013-07-22 Thread Gregg Witkin
Hello Everyone, Some of you know me and some of you do not. For those who do not, my name is Gregg Witkin and I am a teacher fellow for the summer working on developing video products for CloudStack and I am working under Jessica Tomechak. The last video I plan on making this summer is a short v

RE: Reformatting UI code

2013-07-22 Thread Jessica Wang
Sebastien, Brian and I are fix 4.2 bugs. Any fix for 4.2 bugs should go to both 4.2 branch and master branch. That's why Brian and I still need to merge our check-in between master branch and 4.2 branch. Jessica -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent

Re: Reformatting UI code

2013-07-22 Thread Sebastien Goasguen
On 23 Jul 2013, at 00:34, Brian Federle wrote: > In this case it was fine, I could resolve the conflicts by git's > -Xignore-space-change and reformatting accordingly. > > The main issue was that I didn't see any indication that this was being > committed from reading the discussion thread.

Re: Review Request 12782: Add encryption for Ucs password

2013-07-22 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12782/#review23661 --- Ship it! shipped in both master/4.2 - Frank Zhang On July 19, 20

Re: Review Request 12782: Add encryption for Ucs password

2013-07-22 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12782/#review23660 --- Ship it! Ship It! - Frank Zhang On July 19, 2013, 11:28 p.m., Am

RE: Reformatting UI code

2013-07-22 Thread Brian Federle
> suppose u could revert the patch . But for features for 4.3 i think it should > be rebase of your feature branches . I dont think this change should be > impacting 4.2 not sure why u say thats the case. It's OK, I've got the conflicts sorted out now. I don't have a problem with the new format

RE: Reformatting UI code

2013-07-22 Thread Brian Federle
In this case it was fine, I could resolve the conflicts by git's -Xignore-space-change and reformatting accordingly. The main issue was that I didn't see any indication that this was being committed from reading the discussion thread. Since we're doing a lot of bugfixing right now for 4.2, a la

Re: Reformatting UI code

2013-07-22 Thread Sebastien Goasguen
Hi jessica, Sorry this is causing you trouble. Ian us developing his ldap plugin in a feature branch . This was a UI cosmetic code change, i saw that pranav shipped it in RB but did not apply the patch. Since the three if us work in more or less the same time zone i applied the patch quickly to

Re: Review Request 12810: DnsMasqConfigurator need to be rewrite in bash script

2013-07-22 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12810/#review23649 --- patches/systemvm/debian/config/root/dnsmasq.sh

RE: Reformatting UI code

2013-07-22 Thread Jessica Wang
Pranav, Ian, Sebastien, The problem is the duration between the time Ian brought it up on the mailing list and the time Ian/Sebastien checked in the change to master branch is too short (less than 4 hours). Ian brought it up on the mailing list at Thu 7/18/2013 5:44 AM (email subject is "Auto

RE: 4.2 KVM agent can't communicate with 4.1 management server

2013-07-22 Thread Toshiaki Hatano
I got a question from this issue. Should cloudstack allow version mismatch between management and agent? I think it's very good to have for ops. But it looks very difficult for dev, since every little change could break the interoperability. (We have to test it?) -- Toshiaki -Original Messag

RE: Tracing registerTemplate API, is it possible to connect Eclipse debugger to System VMs?

2013-07-22 Thread Edison Su
Yah, you need Eclipse connects to the java agent running on SSVM. You can use the following steps: 1. login into ssvm, either console proxy or ssh(port 3922) 2. kill all the processes named as(run.sh/_run.sh, and java) 3. cd /usr/local/cloud/systemvm 4. add the " -Xdebug -Xrunjdwp:transport=dt_soc

Re: Review Request 12823: CLOUDSTACK-3720. Fix resource types in description of listResourceLimits API command.

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12823/#review23653 --- Commit 4e59c4daf99f96784df79210d0071d5f12d5b3ce in branch refs/heads

Re: Review Request 12823: CLOUDSTACK-3720. Fix resource types in description of listResourceLimits API command.

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12823/#review23652 --- Commit 6c22c7b6141e627ff18e6edc737d0dd314209f04 in branch refs/heads

Review Request 12823: CLOUDSTACK-3720. Fix resource types in description of listResourceLimits API command.

2013-07-22 Thread Andrew Bayer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12823/ --- Review request for cloudstack. Bugs: CLOUDSTACK-3720 Repository: cloudstack-g

RE: deleteAffinityGroup API

2013-07-22 Thread Prachi Damle
Alex, In that case if all test calls are over 8080, this looks to be a bug with CallContext then returning the system user. -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Thursday, July 18, 2013 10:34 PM To: dev@cloudstack.apache.org Cc: Alena Prokharchyk; Ale

Re: Reformatting UI code

2013-07-22 Thread Pranav Saxena
Hey Brian, Sorry to hear that it caused merge conflicts for you . But Ian did bring it up on the mailing list and I suggested him to use the js beautifier tool for reformatting the js code which I guess you missed probably because of the "heavy" traffic on the dev list and thereafter Sebastien mer

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Satoshi Shimazaki
Hi, It does not exist as below. [root@rx200s7-07m ~]# rbd -p libvirt-pool info 5e5d9b40-270b-44af-9479-782175556c47 rbd: error opening image 5e5d9b40-270b-44af-9479-782175556c47: (2) No such file or directory 2013-07-23 04:05:59.903162 7ff4d6b74760 -1 librbd::ImageCtx: error finding header: (2) N

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Wido den Hollander
Hi, On 07/22/2013 08:55 PM, Satoshi Shimazaki wrote: Hi Wido, Thank you for your comment. What I see is "No such file or directory", so that RBD image does not exist. It seems like a copy didn't succeed but now CloudStack thinks that the image does exist. Does "libvirt-pool" have a RBD im

RE: Reformatting UI code

2013-07-22 Thread Jessica Wang
+1 -Original Message- From: Brian Federle [mailto:brian.fede...@citrix.com] Sent: Monday, July 22, 2013 11:33 AM To: dev@cloudstack.apache.org Cc: Ian Duffy (i...@ianduffy.ie) Subject: Reformatting UI code Hello, Recently I discovered that all JS and UI code have been reformatted to 4 s

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Satoshi Shimazaki
Hi Wido, Thank you for your comment. > What I see is "No such file or directory", so that RBD image does not exist. > It seems like a copy didn't succeed but now CloudStack thinks that the image does exist. > Does "libvirt-pool" have a RBD image with the name 5e5d9b40-270b-44af-9479-782175556c47

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

2013-07-22 Thread Daan Hoogland
Ignore my remark about master. I have been looking At an old log. I looked some more into this and it seems the even simpler if (scheme == Vlan.scheme) { return new URI(scheme + "://" + value); } would do the trick. I have not been able to reprodu

Reformatting UI code

2013-07-22 Thread Brian Federle
Hello, Recently I discovered that all JS and UI code have been reformatted to 4 spaces: commit ad69bc8da3244b783dd003ddf3184fca2762c514 Author: Ian Duffy Date: Thu Jul 18 15:39:28 2013 +0100 Format JS While I do appreciate people coming in to help clean up the UI code, and don't mind if

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

2013-07-22 Thread Koushik Das
Daan, After manually updating the database entries to 'vlan://', the system VMs started working. So I don't think using a database before the commit is a problem. One piece of code that seems broken is CitrixResourceBase.getNetwork(). Refer to the line of code long vlan = Long.parseLong(broadc

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Wido den Hollander
Hi, On 07/22/2013 07:56 PM, Satoshi Shimazaki wrote: Hi Wido, I'm in the project with Kimi and Nakajima-san. [root@rx200s7-07m ~]# ceph -v ceph version 0.61.4 (1669132fcfc27d0c0b5e5bb93ade59d147e23404) Same version is installed into all the hosts (KVM host and Ceph nodes). Here is KVM Agent

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Satoshi Shimazaki
Hi Wido, I'm in the project with Kimi and Nakajima-san. [root@rx200s7-07m ~]# ceph -v ceph version 0.61.4 (1669132fcfc27d0c0b5e5bb93ade59d147e23404) Same version is installed into all the hosts (KVM host and Ceph nodes). Here is KVM Agent log. http://pastebin.com/5yG1uBuj I had set the log leve

Re: Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/#review23641 --- Commit b44c6208506791c4af6363f7471b1c367d37cc1a in branch refs/heads

Re: [ACS411] CLOUDSTACK-2188: patch request

2013-07-22 Thread Simon Weller
Issue filed as CLOUDSTACK-3716 . - Original Message - From: "Simon Weller" To: dev@cloudstack.apache.org Sent: Monday, July 22, 2013 11:52:08 AM Subject: Re: [ACS411] CLOUDSTACK-2188: patch request Ok, will do. The reason I hadn't brought this up until now is that we only upgrade

Re: Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-07-22 Thread Toshiaki Hatano
> On July 19, 2013, 4:56 p.m., Chiradeep Vittal wrote: > > scripts/vm/network/vnet/modifyvxlan.sh, line 28 > > > > > > I think there is a need to prevent the guest vm from spoofing the > > multicast? We don't need to.

Re: Review Request 12805: CLOUDSTACK-3695: test_project_configs.TestProjectInviteRequired.test_add_user_to_project failed with missing attribute "email"

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12805/#review23642 --- Commit 7d6c091c05420582d833be7d5eecd58c69b62e39 in branch refs/heads

Re: Review Request 12805: CLOUDSTACK-3695: test_project_configs.TestProjectInviteRequired.test_add_user_to_project failed with missing attribute "email"

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12805/#review23640 --- Commit 95e1f6324f75f873e277b27e6c449d3ae197f6c6 in branch refs/heads

Re: Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/#review23639 --- Commit 18fbbcfc5a3bea98576cc819a37b00b227616cae in branch refs/heads

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

2013-07-22 Thread Koushik Das
Standard system VMs with systemvm.iso from latest master. But that shouldn't matter as the issue is in the deploy VM code. From: Donal Lafferty Sent: 22/07/2013 6:37 PM To: 'dev@cloudstack.apache.org'; cloudstack-...@incubator.apache.org Subject: RE: System VMs no

RE: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Kimihiko Kitase
Hi Wido Thanks for you comment. If we create vm on the NFS primary storage and mount additional disk on the RBD storage, it works fine. If we check vm from virt manager, there is no virtual disk. So we believe the problem should be vm configuration... We will check ceph version tomorrow. Th

Re: Review Request 12819: WIP: Add LDAP Account addition UI. Feedback and Help wanted.

2013-07-22 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12819/ --- (Updated July 22, 2013, 5 p.m.) Review request for cloudstack, Abhinandan Prate

Re: Review Request 12819: WIP: Add LDAP Account addition UI. Feedback and Help wanted.

2013-07-22 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12819/ --- (Updated July 22, 2013, 4:59 p.m.) Review request for cloudstack, Abhinandan Pr

Review Request 12819: WIP: Add LDAP Account addition UI. Feedback and Help wanted.

2013-07-22 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12819/ --- Review request for cloudstack, Abhinandan Prateek, Brian Federle, Jessica Wang,

Re: [ACS411] CLOUDSTACK-2188: patch request

2013-07-22 Thread Simon Weller
Ok, will do. The reason I hadn't brought this up until now is that we only upgraded to 4.1 in production last week. That's when I started digging into this NPE and found the reported related issues. - Original Message - From: "Ilya Musayev" To: dev@cloudstack.apache.org Sent: Mond

Basic Zone network configuration

2013-07-22 Thread Thomas Schneider
Hi Do you have some examples of Basic Zone network configuration ? Is it possible to configure it without using different vlan ? At this stage I configured manually the network without using vlans and the consequence is that the System VM is stuck in starting mode. Best Regards, -- *Thomas Sch

RE: [ACS411] CLOUDSTACK-2188: patch request

2013-07-22 Thread Musayev, Ilya
Simon, Since this issue has been resolved, please open a separate issue for ACS 4.1 and link CLOUDSTACK-2177 and CLOUDSTACK-2188 to it. Please mention your environment and steps to reproduce along with error trace. Thanks ilya > -Original Message- > From: Musayev, Ilya [mailto:imusa...

Review Request 12818: Update LDAPPlugin branch with master

2013-07-22 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12818/ --- Review request for cloudstack, Abhinandan Prateek and Sebastien Goasguen. Repos

Re: Review Request 12817: CLOUDSTACK-3689: Test case test_accounts.TesttemplateHierarchy.test_01_template_hierarchy failed during ListTemplate call

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12817/#review23635 --- Review 12817 PASSED the build test The url of build cloudstack-maste

[ACS411] CLOUDSTACK-2188: patch request

2013-07-22 Thread Simon Weller
Hi, I'd like to request a patch for Cloudstack-2188 for ACS 4.1.1. The master commit was e56d2a401c40b4208d062c0a0ce1ec01df73dd08, but it appears the code has been greatly re-factored since 4.1 was originally branched. This NPE appears to be causing a memory leak in our production environment

RE: [ACS411] CLOUDSTACK-2188: patch request

2013-07-22 Thread Musayev, Ilya
Simon, I've looked at patch and yes - it's been refactored greatly. I wish this would have been brought up earlier :( While due to time constraint, we probably wont be able to make into 4.1.1, if we can get the patch, we will try to push it into next 4.1.2 update. Please update the ticket and

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

2013-07-22 Thread Daan Hoogland
Koushik, thanks for reporting this. You are probably using a database form before this commit. This should work but I suspect it doesn't. public URI toUri(T value) { try { // do we need to check that value does not contain a scheme // part?

Re: Review Request 12811: CLOUDSTACK-3612: 3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12811/#review23632 --- Review 12811 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/#review23631 --- Review 12812 PASSED the build test The url of build cloudstack-maste

Review Request 12817: CLOUDSTACK-3689: Test case test_accounts.TesttemplateHierarchy.test_01_template_hierarchy failed during ListTemplate call

2013-07-22 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12817/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: CLOUDSTACK-3689 R

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Wido den Hollander
Hi, On 07/22/2013 02:25 PM, Kimihiko Kitase wrote: Wido, Thank you very much. CloudStack: 4.1.0 QEMU: 1.5.50 Libvirt: 0.10.2 What version of Ceph on the nodes? $ ceph -v We will set "DEBUG" on the agent tomorrow. But the following is command CloudStack issue. We got this command at KVM h

Re: Using the ACS simulator for jclouds "live" tests - some questions. =)

2013-07-22 Thread Andrew Bayer
Woot, making progress - unsurprisingly, all of the jclouds live tests that expect to be able to create a VM and then SSH into it fail, but that's entirely reasonable. =) I'm making progress through the other failing tests - catching some issues in the jclouds CloudStack implementation along the way

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-22 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/ --- (Updated July 22, 2013, 1:42 p.m.) Review request for cloudstack, Koushik Das a

Review Request 12811: CLOUDSTACK-3612: 3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup

2013-07-22 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12811/ --- Review request for cloudstack, Alena Prokharchyk and Kishan Kavala. Bugs: 3612

Re: Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/#review23630 --- Ship it! Ship It! - Prasanna Santhanam On July 22, 2013, 1:52 p.

Re: Reviewboard and jenkins

2013-07-22 Thread Prasanna Santhanam
On Mon, Jul 22, 2013 at 12:06:31PM +0200, Hugo Trippaers wrote: > The job is running on jenkins.cloudstack.org > > This is de job doing the work: > > http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/ > > This is the job being executed with the patch: >

4.2 KVM agent can't communicate with 4.1 management server

2013-07-22 Thread Wido den Hollander
Hi, While reviewing 12775 I upgraded my Agents from 4.1 to 4.2, but kept my management server at 4.1 When the Agent starts it sends a StartupRoutingCommand to the management server, but this has changed it seems: In 4.1 the Agent sends this JSON: Sending Startup: Seq 4-0: { Cmd , MgmtId:

[Discuss] Apply rules on Virtual Router

2013-07-22 Thread Nguyen Anh Tu
Hi guys, While working with L3 network services, I found a problem in the process of applying iptables rules. It currently works not good in my opinion. When you apply a new rule (eg. StaticNat or Egress rule), Virtual Router backups old rules and re-apply all of non-revoked rules related to sourc

Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/ --- Review request for cloudstack, Parth Jagirdar and Prasanna Santhanam. Bugs: CLO

UI Query: createForm

2013-07-22 Thread Ian Duffy
Hi, I'm having an issue with the below gisted snippets. For whatever reason they do not get validated. Any idea why? https://gist.github.com/imduffy15/6053967 Thanks, Ian

Re: Review Request 12810: DnsMasqConfigurator need to be rewrite in bash script

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12810/#review23629 --- Review 12810 PASSED the build test The url of build cloudstack-maste

Re: secondary storage questions

2013-07-22 Thread Nitin Mehta
On 19/07/13 8:47 AM, "Mike Tutkowski" wrote: >When we use the word "template" here, do we mean both templates (like >OVAs) >and ISOs? Yes, that¹s correct. > >Thanks! > > >On Wed, Jun 5, 2013 at 5:07 PM, Marcus Sorensen >wrote: > >> Thanks, I have one follow-up: >> >> On Tue, Jun 4, 2013 at 1

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

2013-07-22 Thread Donal Lafferty
Where are you getting your system VM from? DL > -Original Message- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: 22 July 2013 11:09 AM > To: cloudstack-...@incubator.apache.org > Subject: System VMs not coming up due to > https://reviews.apache.org/r/12685/ > > Commit id: 2

Re: Review Request 12810: DnsMasqConfigurator need to be rewrite in bash script

2013-07-22 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12810/ --- (Updated July 22, 2013, 12:54 p.m.) Review request for cloudstack, Alena Prokha

Review Request 12810: DnsMasqConfigurator need to be rewrite in bash script

2013-07-22 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12810/ --- Review request for cloudstack, Alena Prokharchyk and Sheng Yang. Bugs: CLOUDSTA

RE: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Kimihiko Kitase
Wido, Thank you very much. CloudStack: 4.1.0 QEMU: 1.5.50 Libvirt: 0.10.2 We will set "DEBUG" on the agent tomorrow. But the following is command CloudStack issue. We got this command at KVM host. [root@rx200s7-07m ~]# ps -ef|grep 1517 root 16099 1 27 19:36 ?00:00:12 /usr/libex

Re: Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12808/#review23627 --- Review 12808 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-22 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review23626 --- Review 12809 failed the build test : UNSTABLE The url of build cloud

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-22 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review23625 --- Ship it! Ship It! - Nitin Mehta On July 22, 2013, 11:20 a.m., Ha

Re: Review Request 12720: CLOUDSTACK: 3382 Unable to Migrate VM's If the hosts are implicitly or explicitly dedicated.

2013-07-22 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12720/ --- (Updated July 22, 2013, 11:28 a.m.) Review request for cloudstack and Devdeep S

Re: Review Request 12720: CLOUDSTACK: 3382 Unable to Migrate VM's If the hosts are implicitly or explicitly dedicated.

2013-07-22 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12720/ --- (Updated July 22, 2013, 11:28 a.m.) Review request for cloudstack and Devdeep S

Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-22 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/ --- Review request for cloudstack, Koushik Das and Nitin Mehta. Bugs: CLOUDSTACK-37

RE: ip are not getting assigned to VM's created from accounts apart from admin

2013-07-22 Thread Rajesh Battala
Rebasing to latest 4.2 head, this issue is resolved now. Am getting ip's for VM's created from accounts Thanks Rajesh Battala > -Original Message- > From: Rajesh Battala [mailto:rajesh.batt...@citrix.com] > Sent: Monday, July 22, 2013 3:24 PM > To: dev@cloudstack.apache.org > Subject: i

Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-22 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12808/ --- Review request for cloudstack, Koushik Das and Nitin Mehta. Bugs: CLOUDSTACK-25

Re: GlusterFS QEMU libgfapi

2013-07-22 Thread Wido den Hollander
Hi John, On 07/21/2013 03:32 PM, John Mark Walker wrote: Greetings, CloudStackers. I noticed there was a question re: libgfapi as it pertains to libvirt and QEMU. As of last January/February, all new versions of the KVM/QEMU/libvirt/oVirt virtualization stack have been "glusterized" and use t

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Wido den Hollander
Hi, On 07/22/2013 12:43 PM, Kimihiko Kitase wrote: It seems secondary storage vm could copy template to primary storage successfully, but created VM doesn't point this vol.. If we create vm manually and add this vol as boot vol, it works fine.. Which version of CloudStack are you using? Wha

RE: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Kimihiko Kitase
It seems secondary storage vm could copy template to primary storage successfully, but created VM doesn't point this vol.. If we create vm manually and add this vol as boot vol, it works fine.. So it seems cloudstack cannot configure VM correctly in ceph rbd environment. Any idea? Thanks Kimi

Fwd: Issues with LDAP (git will be down for pushes/reverts)

2013-07-22 Thread Prasanna Santhanam
What's broken is broken for now. -- Prasanna., Powered by BigRock.com --- Begin Message --- Good morning, We are currently investigating an issue with LDAP as we have seen a few problem reports this morning with ability to use credentials. You may find you are unable

RE: Problem in adding Ceph RBD storage to CloudStack

2013-07-22 Thread Kimihiko Kitase
Hello I am in the project with Nakajima san. We succeeded to add RBD storage to primary storage. But when we try to boot centos as user instance, it fail during system logger process. It works fine when we boot centos using NFS storage. It works fine when we boot centos using NFS storage and add

  1   2   >