[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/755#issuecomment-136946071 @karuturi LGTM, though one question - once an LDAP group is linked with a cloudstack domain, and then say if I manually create a user in cloudstack's domain first

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-02 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-136962555 tested this on Xen 6.5 advanced zone with isolated and VPC. verified that the default policies are set to drop. I am not sure if its related to this. But,

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

2015-09-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-136962823 @miguelaferreira @wilderrodrigues @wido @jburwell @abhinandanprateek @kishankavala hi, can you help review this, thanks --- If your project is set up for it, you

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

2015-09-02 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-136981967 @bhaisaab There are 2 reviews already. @wido and @miguelaferreira gave :+1: I think you can merge this and #753 --- If your project is set up for it, you can

Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread anil lakineni
Hi All, We are facing boot issues when VMs have DATA drives. *After VM was shut down we were unable to power on server unless we detach Data Drive.* We are using Citrix Cloud Portal 4.5.0 and XenServer 6.2 *Note: *We have not facing this issue on some of the VM's which also have DATA drives

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38512226 --- Diff: server/src/com/cloud/user/AccountManagerImpl.java --- @@ -2145,14 +2145,10 @@ private UserAccount getUserAccount(String username, String

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/755#issuecomment-136980464 users are added based on the username. So, in this case LDAP user wont be added(it will error out). CS user will exist and will continue to work with earlier

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38511573 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LinkDomainToLdapCmd.java --- @@ -0,0 +1,114 @@ +/* + *

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38511898 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -52,21 +58,71 @@ public

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38511940 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -52,21 +58,71 @@ public

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38512420 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LinkDomainToLdapCmd.java --- @@ -0,0 +1,114 @@ +/* + *

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38512312 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -52,21 +58,71 @@ public LdapAuthenticator(final

[GitHub] cloudstack pull request: CLOUDSTACK-8755: xs-tools.iso missing fro...

2015-09-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/727 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: Quota master WIP (just to follow changes)

2015-09-02 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-136950359 Previously, I expressed concerns that the pattern of opening and immediately closing transactions on ``TransactionLegacy`` was placing undue pressure on the

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-02 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/772 CLOUDSTACK-8690:Added remote access vpn and vpn users configuration @remibergsma @wilderrodrigues Added the VR configuration changes for 1. Remote access vpn 2. Remote access vpn

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-136951585 I will push a test today to cover the iptables default policies. Do you need help finding the options on the UI whilst the test gets cooked?

[GitHub] cloudstack pull request: CLOUDSTACK-8678: Reserve RAM for KVM host...

2015-09-02 Thread kmccormick
Github user kmccormick commented on the pull request: https://github.com/apache/cloudstack/pull/766#issuecomment-137155599 Regarding the upgrade scenario, I believe that this would not affect any running VMs. Can anyone confirm? Besides just running VMs, it is still a potential

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread anil lakineni
Dave, Thanks for the replay and suggestion. I believe in the CS when we deploy a fresh VM from Template or ISO a drive is created which is ROOT drive (C drive) and it will acts as primary drive (surely an entry will be created in Cloud DataBase tables to maintain this record) for the VM until

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/755#issuecomment-137344109 @koushik-das I created two issue for the open items. I will track these later https://issues.apache.org/jira/browse/CLOUDSTACK-8797

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread Mike Tutkowski
Do you have any management server log file you can share? On Wed, Sep 2, 2015 at 10:45 AM, anil lakineni < anilkumar459.lakin...@gmail.com> wrote: > Dave, > > Thanks for the replay and suggestion. > > I believe in the CS when we deploy a fresh VM from Template or ISO a drive > is created which

[GitHub] cloudstack pull request: CLOUDSTACK-8678: Reserve RAM for KVM host...

2015-09-02 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/766#issuecomment-137243367 @kmccormick Agreed. A release note would be enough since running VMs will not be affected. CPU is a different problem indeed. Stuff becomes slow, but going Out

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-09-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-137254065 so now we are back to, "I would like to see two unit tests for checkForKeyByPublicKey. The succes and the failure cases. I am not sure if this check shows a

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread Dave Dunaway
Check the boot order in the BIOS and make sure the VM is not trying to boot off the DATA disk. This is maybe why you don't see this issue when there is only one disk attached. hth. dave. On Wed, Sep 2, 2015 at 7:38 AM, anil lakineni < anilkumar459.lakin...@gmail.com> wrote: > Hi Rajani, > >

RE: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread Patrick Chevalley
Have you done any previous manipulation (e.g. storage migration, detach/attach, etc.) Do you have more than 1 primary storage? What is the state of the impacted volume(s) in the DB? As already asked, look at the mgt logs for more details, or send the error. -Original Message- From: Mike

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread giljae o
Hi, How many are DATA disk attached to the VM? If DATA disk is more than, try boot the VM with only 2 data disks. James. On Thursday, September 3, 2015, Patrick Chevalley wrote: > Have you done any previous manipulation (e.g. storage migration, > detach/attach,

[GitHub] cloudstack pull request: Fixed issue in adding vm SG rules on vm r...

2015-09-02 Thread csghuser
Github user csghuser commented on the pull request: https://github.com/apache/cloudstack/pull/479#issuecomment-137047840 Would it be possible to tell me which cloudstack release this fix is included in? --- If your project is set up for it, you can reply to this email and have your

Re: Instance reboot issue - CS 4.5.1 - Xen 6.5

2015-09-02 Thread cs user
Hi Folks, Looks like this is fixed in https://github.com/apache/cloudstack/pull/479 ? Which cloudstack release contains this fix? Many thanks! On Wed, Sep 2, 2015 at 11:16 AM, cs user wrote: > Forwarding to users channel in case anyone else has seen this... > > > > Hi

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-137040420 Thanks for testing it, @karuturi, much appreciated! I'm writing marvin tests for this PR and the other issue (CLOUDSTACK-8759). Once done, I will

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread anil lakineni
Hi Rajani, Host is *XenServer 6.2* Guest OS is *Windows 2008 R2 & Windows 2012 R2* Regards, Anil. On Wed, Sep 2, 2015 at 3:42 PM, Rajani Karuturi wrote: > whats the host and guest os? > > ~Rajani > > On Wed, Sep 2, 2015 at 2:33 PM, anil lakineni < >

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-02 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-137048186 Ok. Here is the new issue https://issues.apache.org/jira/browse/CLOUDSTACK-8795 :+1: for this PR --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

2015-09-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-136995082 Thanks @karuturi merging now --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38515428 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -52,21 +58,71 @@ public

Instance reboot issue - CS 4.5.1 - Xen 6.5

2015-09-02 Thread cs user
Hi All, We have seen this in 2 separate environments, both running the same versions of Cloudstack and Xenserver. When we reboot an instance, we lose access to it. Looking at the iptables config on the xen host, we can see that the vif is incremented for the bridged entries, but not updated for

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

2015-09-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/753 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38513021 --- Diff: server/src/com/cloud/user/AccountManagerImpl.java --- @@ -2145,14 +2145,10 @@ private UserAccount getUserAccount(String username, String

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

2015-09-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/754 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Build failed in Jenkins: simulator-4.5-singlerun #271

2015-09-02 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 325 originally caused by: Started by upstream project "build-4.5" build number 523 originally caused by: Started by

[GitHub] cloudstack pull request: Quota

2015-09-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-137026221 The quota feature has three major packages; - framework/quota - plugin/database/quota - usage/*Quota*Impls Unit test coverage on framework/quota

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38512709 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManagerImpl.java --- @@ -243,4 +263,22 @@ public boolean

[GitHub] cloudstack pull request: Cloudstack:8647 LDAP Trust AD and Autoimp...

2015-09-02 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/755#discussion_r38513100 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManagerImpl.java --- @@ -243,4 +263,22 @@ public boolean isLdapEnabled()

[GitHub] cloudstack pull request: CLOUDSTACK-8678: Reserve RAM for KVM host...

2015-09-02 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/766#issuecomment-137004807 What happens in case of an upgrade? Looks like there is a possibility that the host may get over-provisioned. Should there be some scripts to let the user know

Re: Facing BOOT issues (if DATA drives are attached) on VMs in CCP 4.5.0

2015-09-02 Thread Rajani Karuturi
whats the host and guest os? ~Rajani On Wed, Sep 2, 2015 at 2:33 PM, anil lakineni < anilkumar459.lakin...@gmail.com> wrote: > Hi All, > > We are facing boot issues when VMs have DATA drives. > > > *After VM was shut down we were unable to power on server unless we detach > Data Drive.* > > We