cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Duffy" <>
Subject Re: Review Request 12819: WIP: Add LDAP Account addition UI. Feedback and Help wanted.
Date Mon, 22 Jul 2013 16:59:58 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 22, 2013, 4:59 p.m.)

Review request for cloudstack, Abhinandan Prateek, Brian Federle, Jessica Wang, Pranav Saxena,
and Sebastien Goasguen.

Repository: cloudstack-git


This is purely a work in progress. I'm submitting it as I'd like somebody to give *detailed*
feedback/reviewal before I go much further.

The code works/does-what-I-want but I'm not sure if its "correct" and follows cloudstack UI
I had to do some ugly css in order to get the display I wanted.
I'm currently having issues with fields marked as "required" for some reason the requirement
doesn't seem to be enforced.
I'm not sure if a loader appears should the request to listAllLdapUsers be slow to respond.

I'm not sure how to add a if ldapEnabled display this view else display old view condition.

For testing purposes there is a ldap server included in this branch. You can launch it with:
mvn -pl :cloud-plugin-user-authenticator-ldap ldap:run

and then configure it at Global Settings -> LDAP Configuration, hostname: localhost, port:


  ui/css/cloudstack3.css 4545e96 
  ui/index.jsp 34f0c54 
  ui/scripts/accounts.js e5faf33 
  ui/scripts/accountsWizard.js PRE-CREATION 
  ui/scripts/ui-custom/accountsWizard.js PRE-CREATION 



Complied... view in browser, got expected results.

File Attachments (updated)



Ian Duffy

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message