It's for 4.2 . That's why I committed them only to master . @Abhi - Can you please confirm ?
-----Original Message----- From: Suresh Sadhu Sent: Thursday, February 21, 2013 4:06 PM To: Pranav Saxena; cloudstack-dev@incubator.apache.org; Abhinandan Prateek Cc: Sudha Ponnaganti Subject: RE: [ACS4.2] LDAP UI branch Merged Thanks for your review comments ..I didn't see your UI changes in 4.1. branch .is it expected to be in 4.1 or 4.2?can you please confirm? Regards Sadhu -----Original Message----- From: Pranav Saxena Sent: 20 February 2013 00:45 To: cloudstack-dev@incubator.apache.org; Suresh Sadhu; Abhinandan Prateek Subject: RE: [ACS4.2] LDAP UI branch Merged Btw, For the last test case you mentioned - 1.Open the LDAP configuration Wizard(Navigate Global Setting-LdapConfiguration) 2. .Without providing any values,Click on "Ok" button You won't be allowed to do this , since to configure LDAP , you need to provide some set of mandatory parameters and hence your expected result i.e (dialogue window should prompt with proper error message) is not valid . Regards, Pranav -----Original Message----- From: Pranav Saxena [mailto:pranav.sax...@citrix.com] Sent: Wednesday, February 20, 2013 12:32 AM To: Suresh Sadhu; cloudstack-dev@incubator.apache.org; Abhinandan Prateek Subject: RE: [ACS4.2] LDAP UI branch Merged Hi Sadhu , The LDAP test plan looks good to me from the LDAP-UI integration point of view . In case of any issues , please feel free to create a JIRA ticket and assign it to me . Thanks, Pranav -----Original Message----- From: Suresh Sadhu Sent: Tuesday, February 19, 2013 11:32 PM To: cloudstack-dev@incubator.apache.org; Abhinandan Prateek Cc: Pranav Saxena Subject: RE: [ACS4.2] LDAP UI branch Merged Hi, Please review the following test plan for the feature "LDAP" and let me know the review comments. https://cwiki.apache.org/confluence/display/CLOUDSTACK/LDAP Regards Sadhu -----Original Message----- From: Pranav Saxena [mailto:pranav.sax...@citrix.com] Sent: 19 February 2013 16:56 To: Abhinandan Prateek; cloudstack-dev@incubator.apache.org Subject: RE: [ACS4.2] LDAP UI branch Merged Abhi , I just completed the UI integration for this feature on my feature branch asf/LDAP-UI and have merged it with asf/master . You can go ahead and test the functionality . Thanks, Pranav -----Original Message----- From: Abhinandan Prateek Sent: Saturday, February 16, 2013 8:59 AM To: Pranav Saxena; cloudstack-dev@incubator.apache.org Subject: Re: [ACS4.2] LDAP UI Pranav, I was wondering we have API to get configuration, can't we reuse the same to get ldap config ? -abhi On 15/02/13 7:36 PM, "Pranav Saxena" <pranav.sax...@citrix.com> wrote: >Abhi , > >Just wanted to check with you if we have a list LDAP configuration API >call in CS ? Because I would need this to list the current LDAP >configuration on the UI , if it exists . > >Regards, >Pranav > >-----Original Message----- >From: Abhinandan Prateek [mailto:abhinandan.prat...@citrix.com] >Sent: Thursday, February 14, 2013 12:27 PM >To: cloudstack-dev@incubator.apache.org >Subject: Re: [ACS4.2] LDAP UI > > >On 14/02/13 11:30 AM, "David Nalley" <da...@gnsa.us> wrote: > >>On Thu, Feb 14, 2013 at 12:51 AM, Pranav Saxena >><pranav.sax...@citrix.com> wrote: >>> To configure LDAP , we need to pass in few multiple mandatory >>>parameters - >>> >>> hostname Hostname or ip address of the ldap server eg: >>>my.ldap.com >>> queryfilter You specify a query filter here, which narrows down the >>>users, who can be part of this domain. >>> searchbase The search base defines the starting point for the >>>search in the directory tree >>> >>> If you are referring to Global settings , that can be done but then >>>we'll have to have three Ldap config parameters there . if that is a >>>good design to handle this , then yes we can do that. Perhaps , the >>>idea is to have a single dialog box where a user could supply three >>>values and configure and debug them if something goes wrong. >> >> >>Those are the mandatory API inputs for CloudStack. >>But almost all environments will require username/password for binding >>at a minimum, and you should probably, and prolly offer the SSL option >>as well. Port should probably be an option too. >> >>Without at least bind creds, the API configuration is practically >>useless on any modern LDAP server. >> >>--David > >The admin guide documents the LDAP API. SSL is supported. >http://incubator.apache.org/cloudstack/docs/en-US/Apache_CloudStack/4.0 >.0- >i >ncubating/pdf/Admin_Guide/Apache_CloudStack-4.0.0-incubating-Admin_Guid >e-e >n >-US.pdf > >Look for LDAP configuration. In short you have following config params: > >hostname >searchbase >queryfilter >binddn >bindpass >port >ssl >truststore >truststorepass >response > > > >-abhi > > > >> >
