)]}'
{"id":"openstack%2Fdevstack~20249","triplet_id":"openstack%2Fdevstack~master~I1404cc5c754f878e32a2d10254840d092211e6e6","project":"openstack/devstack","branch":"master","topic":"devstack-ldap2","hashtags":[],"change_id":"I1404cc5c754f878e32a2d10254840d092211e6e6","subject":"Add optional silent install and config of ldap to devstack","status":"MERGED","created":"2013-01-22 16:21:16.000000000","updated":"2013-02-07 17:07:10.000000000","submitted":"2013-02-07 17:07:10.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":46,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d6ae5bae638cdf120e84161d012ef8f647f771c5","_number":20249,"virtual_id_number":20249,"owner":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":2,"date":"2013-02-07 17:07:10.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},{"value":0,"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":2,"date":"2013-02-07 16:01:55.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},{"value":0,"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},{"value":1,"date":"2013-02-06 15:49:51.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-02-06 11:37:45.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":1,"date":"2013-02-06 13:45:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},{"value":2,"date":"2013-02-06 11:26:35.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":1,"date":"2013-02-07 16:01:55.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},{"value":0,"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},{"value":0,"date":"2013-02-06 11:26:35.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},{"value":0,"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-02-01 21:34:41.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-02-04 16:53:24.000000000","updated_by":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"reviewer":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"state":"REVIEWER"},{"updated":"2013-02-04 18:25:43.000000000","updated_by":{"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},"reviewer":{"_account_id":5575,"name":"Jose Castro Leon","email":"jose.castro.leon@cern.ch","username":"jose-castro-leon"},"state":"REVIEWER"},{"updated":"2013-02-04 18:25:50.000000000","updated_by":{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},"reviewer":{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},"state":"REVIEWER"},{"updated":"2013-02-04 18:27:51.000000000","updated_by":{"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},"reviewer":{"_account_id":1155,"name":"sleepsonthefloor","username":"sleepsonthefloor"},"state":"REVIEWER"},{"updated":"2013-02-05 16:06:07.000000000","updated_by":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"reviewer":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"state":"REVIEWER"},{"updated":"2013-02-06 11:26:35.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-02-06 11:37:45.000000000","updated_by":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"reviewer":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"state":"REVIEWER"},{"updated":"2013-02-06 13:45:56.000000000","updated_by":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"reviewer":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"state":"REVIEWER"},{"updated":"2013-02-06 15:49:51.000000000","updated_by":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"reviewer":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"state":"REVIEWER"},{"updated":"2013-02-07 16:01:55.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2013-02-07 17:07:10.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"4af4165c32e4255e991604f7860b4dfcba42570d","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-01-22 16:23:45.000000000","message":"Patch Set 1: Work In Progress\n\nThis is a work in progress to start adding silent install and config of ldap to devstack","accounts_in_message":[],"_revision_number":1},{"id":"e1ccc134ba6505ca5e045855de345ac324325cb7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 16:58:48.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/20249/1/check/gate-tempest-devstack-vm/27784 : SUCCESS\n- http://logs.openstack.org/20249/1/check/gate-tempest-devstack-vm-postgres/931 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/1/check/gate-tempest-devstack-vm-quantum/2154 : FAILURE\n","accounts_in_message":[],"_revision_number":1},{"id":"a9f34771ee6b905999c04b4b3767858f068310c8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-23 00:29:02.000000000","message":"Patch Set 1:\n\nas this is going to be behind keystone, I\u0027d actually move to put this into lib/keystone","accounts_in_message":[],"_revision_number":1},{"id":"96cf5050d30abc7bc31d0b721439a137ec3654e1","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-01-25 18:30:05.000000000","message":"Patch Set 1: (4 inline comments)\n\nGood progress","accounts_in_message":[],"_revision_number":1},{"id":"2c652cec1db101298eae84175d6a6279c8c9ecdf","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-01-25 21:22:27.000000000","message":"Patch Set 1:\n\nDisagree on putting this under keystone.  LDAP is a stand alone service.   Keystone consumes LDAP, but it is possible that we will use LDAP for DNS in the future.","accounts_in_message":[],"_revision_number":1},{"id":"80d86f8108a317d2f8efe75a14b808bbaff5dc2f","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-01-25 21:29:11.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"00d5bb53135b444948486be6051c09e7a9193b09","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-01-25 22:00:48.000000000","message":"Patch Set 1:\n\nInteresting stuff...\n\nPlease structure this similarly to the other projects in lib/, it should be in its own file (lib/ldap).  The .ldif files should go in files/ or files/ldap/ if you prefer.\n\nI saw a comment regarding making it a part of keystone; I\u0027d prefer to not do that as it looks like you are using the packaged openldap server so it is a system service.  If configured to use ldap keystone should test for it being available but I\u0027d think that is all.","accounts_in_message":[],"_revision_number":1},{"id":"a33fa19e440b75530a7615d1dbe5d30e777ee1dc","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-28 15:01:19.000000000","message":"Patch Set 1:\n\nif this isn\u0027t going under keystone, even though it\u0027s the only service that uses it, the it at least needs to look like the rest of the services.\n\nbe lib/ldap, have install, configure, etc functions called from the other functions.\n\nfiles installed via files/apts/ldap\n\nThe system service argument about what we might do in the future doesn\u0027t really hold for me though. Quantum starts all kinds of system services, depending on plugin. ryu integration is probably a much closer analogy here, because at this time the only direct ldap consumer is keystone.\n\nIt can always be refactored later if we start consuming ldap in multiple services like we do the database.","accounts_in_message":[],"_revision_number":1},{"id":"89f19fa26d07c630504e3443ce01ff6144f63360","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-01-28 21:26:55.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"13e3a83bf94eb4df51b8901e24fd254e074a1d1f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-28 21:49:53.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/2/check/gate-tempest-devstack-vm/28527 : SUCCESS\n- http://logs.openstack.org/20249/2/check/gate-tempest-devstack-vm-postgres/967 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/2/check/gate-tempest-devstack-vm-quantum/3100 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"3cea0b68b02346649e447c16d020a4e8d9f8a44a","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-01-29 23:42:29.000000000","message":"Patch Set 2: (7 inline comments)\n\nGood progress, I\u0027ve noted some minor formatting things.","accounts_in_message":[],"_revision_number":2},{"id":"e683afc3c81c93261ccf8fe11003dd6fa2582f7f","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-30 11:16:40.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nin addition to dtroyer\u0027s comments, a couple other on indenting.\n\nAt a conceptual level I still think we should call install_ldap in lib/keystone, not in stack.sh until the service is used by another core project.\n\nAlso, how will one trigger keystone being backed with ldap? The way as written if someone enables ldap as a system servce, it seems that keystone converts to ldap under the covers, but all the same database setup is required as well?\n\nGiven that devstack is also living documentation, it would be good to expose what the best practice would be here.","accounts_in_message":[],"_revision_number":2},{"id":"57ba812c6a2adc1ab41e0b97c49e15a7d3a6e1a7","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-01-30 14:59:01.000000000","message":"Patch Set 2:\n\nSean,  Yes, it is intentional that LDAP triggers the Keystone config, and still requires the Database.  You cannot run Keystone without a Database, even if you use LDAP for the Identity store, as LDAP does not store Policy, Service Catalog nor the Tokens.\n\nLDAP should be an optional component, and not installed by default, but if it is is indicated that it is enabled, the above logic should trigger it.  It is much like postgres or qpid;  they are not installed by default, but used if they are indicated on the command line.","accounts_in_message":[],"_revision_number":2},{"id":"58bd4f2604303cb5c407ed8ba22bb5ce83d94194","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-30 15:51:47.000000000","message":"Patch Set 2:\n\nright, though it is a little different. In the qpid or pg case they are implementations of a feature (database or queue) that you can\u0027t function without. ldap is a keystone backend implementation detail, like the quantum plugins.\n\nSo it seems like KEYSTONE_BACKEND\u003dldap (or something) should be the stackrc and that triggers this to run.","accounts_in_message":[],"_revision_number":2},{"id":"8732ed274ef29e090667f340758af427b7ae063f","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-01-30 16:07:59.000000000","message":"Patch Set 2:\n\nWe\u0027re adding KEYSTONE_TOKEN_BACKEND, I suppose adding KEYSTONE_IDENTITY_BACKEND to lib/keystone would complete the trifecta.","accounts_in_message":[],"_revision_number":2},{"id":"ab91d011ee70a71be0c17c4aa818a76e4f9d593b","author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"date":"2013-01-31 20:58:23.000000000","message":"Patch Set 2:\n\n@Dean: don\u0027t forget KEYSTONE_POLICY_BACKEND and KEYSTONE_TRUSTS_BACKEND, both new for grizzly (although we\u0027ll only have sql drivers)","accounts_in_message":[],"_revision_number":2},{"id":"0b43ecf04f6dee490d68fc473d03613e360e14b0","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-01 00:37:56.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9f5d6fec6d9a4d96b37216a2ee311d2955091eba","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-01 01:07:07.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/3/check/gate-tempest-devstack-vm/29402 : SUCCESS\n- http://logs.openstack.org/20249/3/check/gate-tempest-devstack-vm-postgres/1024 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/3/check/gate-tempest-devstack-vm-quantum/4110 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"d7e6449f57c1c0611beffaff1e706033b2d3ecb2","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-02-01 19:46:21.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\nplease add in the deb/rpm dependencies for python-ldap.  Keystone does not yet install them by default, as LDAP is considered optional.","accounts_in_message":[],"_revision_number":3},{"id":"20af94e46cc1f9b40781d82d2e818260e1099416","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-02-01 21:34:41.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(7 inline comments)\n\nit looks like files/ldap/manager.ldif shouldn\u0027t be here?  it\u0027s generated in lib/ldap.\n\nGetting close...","accounts_in_message":[],"_revision_number":3},{"id":"ac4fbca7154e19388cd0fe2c204e3d7a4e191655","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-01 23:22:43.000000000","message":"Patch Set 3: (7 inline comments)\n\nI am addressing all your comments in my next patch version. See my comments","accounts_in_message":[],"_revision_number":3},{"id":"8058d460f0823359cce565f3aacdcf6246795438","author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"date":"2013-02-01 23:49:38.000000000","message":"Patch Set 3:\n\n(remove WIP from the commit summary if you\u0027re going to leave the status as Review in Progress)","accounts_in_message":[],"_revision_number":3},{"id":"645a66169c8eb58b4c49cad49d357616cc4303cc","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-02-01 23:53:52.000000000","message":"Patch Set 3: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"65f5c94dee094cf7de75ec0de29595ec10163b91","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-02 00:47:52.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"aaa1a3298035cd29dc43d88c82be2a53fe8519f8","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-02 00:58:06.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"47e44533c5066f6fd692a558d16d98c5a07da1c7","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-02-02 00:58:12.000000000","message":"Patch Set 5:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":5},{"id":"7ea77e3ab69b956cad15892c032e08a72156891d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-02 02:07:59.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/5/check/gate-tempest-devstack-vm-full/467 : SUCCESS\n- http://logs.openstack.org/20249/5/check/gate-tempest-devstack-vm-postgres-full/195 : FAILURE (non-voting)\n- http://logs.openstack.org/20249/5/check/gate-tempest-devstack-vm-quantum/4287 : SUCCESS\n","accounts_in_message":[],"_revision_number":5},{"id":"356a992ff5404d3f29a3b5e203805ce2afb4cd57","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-02 12:19:33.000000000","message":"Patch Set 5: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nCouple of comments on just moving around where things are called.\n\nAlso, I\u0027m a little concerned that there isn\u0027t a state clearing function. For the databases we have a recreate call so that we ensure we start from a blank schema on every run. Seems like something equivalent might be needed for ldap.","accounts_in_message":[],"_revision_number":5},{"id":"dd121cabe6b2d1215c39eb21c8b51a4cf67bf1ff","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-04 06:49:17.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"bf1e5e57f22df2389d2c4ad791c3ec526a23fd30","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-04 07:29:39.000000000","message":"Patch Set 6: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/6/check/gate-tempest-devstack-vm-full/639 : SUCCESS\n- http://logs.openstack.org/20249/6/check/gate-tempest-devstack-vm-postgres-full/367 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/6/check/gate-tempest-devstack-vm-quantum/4493 : SUCCESS\n","accounts_in_message":[],"_revision_number":6},{"id":"13c7074a2be0eb0e4744ad7c14ee8979fca37ca3","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-04 13:01:12.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\nI think the only thing that\u0027s missing is documentation on the 2 variables you need to enable to make this work. I\u0027m going to +1 for now, +2 once we get something in here to let people know how they would enable the feature.","accounts_in_message":[],"_revision_number":6},{"id":"15c5619521df3d09ad06f8544803f6c2950c7ff8","author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"date":"2013-02-04 16:53:24.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":6},{"id":"7ebbfa0bb5ab18519f79fc6571887d74971c41ff","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-02-04 18:24:29.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\nNice work.","accounts_in_message":[],"_revision_number":6},{"id":"ad6582f5cffec5f930a3c2993dc3b7347a40267f","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-02-04 18:55:45.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nFounds some things in live testing","accounts_in_message":[],"_revision_number":6},{"id":"2f7764ff4f042d11094200eee65b95393967f6e8","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-04 20:21:09.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"0364bea0fe506e7fe7d72ca56f46aebace585caa","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-04 20:34:51.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"0a603831b99bbb3128a4f330cb4a9eab133fb2e4","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-02-04 20:34:58.000000000","message":"Patch Set 8:\n\nNew patchset patch-id matches previous patchset, but whitespace content has changed.","accounts_in_message":[],"_revision_number":8},{"id":"4ef9a9bd54b5476ad77f1861857896b282124541","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-04 21:30:19.000000000","message":"Patch Set 8: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/8/check/gate-tempest-devstack-vm-full/799 : SUCCESS\n- http://logs.openstack.org/20249/8/check/gate-tempest-devstack-vm-postgres-full/526 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/8/check/gate-tempest-devstack-vm-quantum/4679 : SUCCESS\n","accounts_in_message":[],"_revision_number":8},{"id":"028a7c6d5c002b884cb24b2f1bdf9051c3bb7c07","author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"date":"2013-02-05 16:06:07.000000000","message":"Patch Set 8: Looks good to me, but someone else must approve\n\nlgtm","accounts_in_message":[],"_revision_number":8},{"id":"5f9a74f36b4ae2e0d1dc91143f57698d0b8883db","author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"date":"2013-02-05 16:08:06.000000000","message":"Patch Set 8:\n\noh, please mention the bug report which seems related - https://bugs.launchpad.net/devstack/+bug/1021319","accounts_in_message":[],"_revision_number":8},{"id":"86c515c530e393540b14372d24b6d7f30cdc85ef","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2013-02-05 18:39:29.000000000","message":"Patch Set 8: I would prefer that you didn\u0027t merge this\n\n(8 inline comments)\n\n","accounts_in_message":[],"_revision_number":8},{"id":"3061d24758f1195609ba70f1a3ffbfb6bc3b5644","author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"date":"2013-02-06 02:02:31.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"f0bcd4acb07077c31b1232cee26e2ee1a880df85","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-06 03:27:13.000000000","message":"Patch Set 9: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/9/check/gate-tempest-devstack-vm-full/1118 : SUCCESS\n- http://logs.openstack.org/20249/9/check/gate-tempest-devstack-vm-postgres-full/845 : SUCCESS (non-voting)\n- http://logs.openstack.org/20249/9/check/gate-tempest-devstack-vm-quantum/5067 : SUCCESS\n","accounts_in_message":[],"_revision_number":9},{"id":"af9f8e1251956dcc1fe3a4edc8d7456c418a6536","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-06 11:26:35.000000000","message":"Patch Set 9: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":9},{"id":"bcde7c40f9d3f03a3311b6e952f0d99dab30c958","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2013-02-06 11:37:45.000000000","message":"Patch Set 9: Looks good to me, but someone else must approve\n\nNice work!","accounts_in_message":[],"_revision_number":9},{"id":"225f5c56148b58aa5a2eff013ea112ea681f02b5","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2013-02-06 13:45:56.000000000","message":"Patch Set 9: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":9},{"id":"28929b8facebd81de3beaff80b30728d0df5803a","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2013-02-06 15:49:51.000000000","message":"Patch Set 9: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":9},{"id":"03223e01f6d49e36e9ee11bbfe748a7f22327a1a","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-07 16:01:55.000000000","message":"Patch Set 9: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":9},{"id":"e2758f62c3a7e9d3c6dd33cf06fb5a488ab95b66","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-07 16:01:56.000000000","message":"Patch Set 9: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":9},{"id":"6e29edfc620f05bd1825f90bf25c0c44fe20f354","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-07 17:07:10.000000000","message":"Patch Set 9: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/20249/9/gate/gate-tempest-devstack-vm-full/1432 : SUCCESS\n- http://logs.openstack.org/20249/9/gate/gate-tempest-devstack-vm-postgres-full/1158 : SUCCESS\n- http://logs.openstack.org/20249/9/gate/gate-tempest-devstack-vm-quantum/5458 : SUCCESS\n","accounts_in_message":[],"_revision_number":9},{"id":"897bfcf0145c88b34f4e3c74f5e0cb57b9be5dc0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-07 17:07:10.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":9}],"current_revision_number":9,"current_revision":"f127e2f316f1161bacdf4cccdbc3e56b2b8a54a8","revisions":{"03e2931142ce8d96ce9170efacee14e179431928":{"kind":"REWORK","_number":1,"created":"2013-01-22 16:21:16.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/1"}}},"commit":{"parents":[{"commit":"0dd34df455637ee29176525974d6dab93f530e66","subject":"lib/quantum: refactor quantum plugins and third party","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0dd34df455637ee29176525974d6dab93f530e66"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:21:06.000000000","tz":-360},"subject":"Work In progress to add optional silent install and config of ldap to devstack","message":"Work In progress to add optional silent install and config of ldap to devstack\n\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/03e2931142ce8d96ce9170efacee14e179431928"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/03e2931142ce8d96ce9170efacee14e179431928"}]},"branch":"refs/heads/master"},"2c19441c684a7a47262d8e7c2c15205d50cb3f7d":{"kind":"REWORK","_number":2,"created":"2013-01-28 21:26:55.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/2"}}},"commit":{"parents":[{"commit":"3b3a6d78a63e93b43d3815f7e32a32458a82a3a9","subject":"Merge \"Set MySQL slow log to record every query\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3b3a6d78a63e93b43d3815f7e32a32458a82a3a9"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-28 21:26:42.000000000","tz":-360},"subject":"Work In progress to add optional silent install and config of ldap to devstack","message":"Work In progress to add optional silent install and config of ldap to devstack\n\nI restructured the code to be more devstack compliant as suggested.  ldif files are now in files/ldap and the main script is in lib.  I hooked this into stack.sh and updated keystone with new inisets to add ldap properties to keystone.conf\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2c19441c684a7a47262d8e7c2c15205d50cb3f7d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2c19441c684a7a47262d8e7c2c15205d50cb3f7d"}]},"branch":"refs/heads/master"},"adb194fbe0b844295b334d9c4b2c76d6345702f1":{"kind":"REWORK","_number":3,"created":"2013-02-01 00:37:56.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/3"}}},"commit":{"parents":[{"commit":"a814f22ce49a3674fd6f266f52bf7de990521adc","subject":"Minor fix to readme - corrected indentation for some commands","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a814f22ce49a3674fd6f266f52bf7de990521adc"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-01 00:37:50.000000000","tz":-360},"subject":"Work In progress to add optional silent install and config of ldap to devstack","message":"Work In progress to add optional silent install and config of ldap to devstack\n\nI cleaned up formatting of the code, added new cache files, and addressed\nall comments. I also fixed a bug where ldapadd would halt devstack by\nreturning an error code if the ldap was previously configured. I also added\nKEYSTONE_IDENTITY_BACKEND to control the driver selection independent of the\nldap install option.\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/adb194fbe0b844295b334d9c4b2c76d6345702f1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/adb194fbe0b844295b334d9c4b2c76d6345702f1"}]},"branch":"refs/heads/master"},"046954a9b47d05557aad69bc02493aaf69ce441f":{"kind":"REWORK","_number":4,"created":"2013-02-02 00:47:52.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/4"}}},"commit":{"parents":[{"commit":"a814f22ce49a3674fd6f266f52bf7de990521adc","subject":"Minor fix to readme - corrected indentation for some commands","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a814f22ce49a3674fd6f266f52bf7de990521adc"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-02 00:45:46.000000000","tz":-360},"subject":"Work In progress to add optional silent install and config of ldap to devstack","message":"Work In progress to add optional silent install and config of ldap to devstack\n\nAddressed all comments from the previous patch iteration.\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/046954a9b47d05557aad69bc02493aaf69ce441f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/046954a9b47d05557aad69bc02493aaf69ce441f"}]},"branch":"refs/heads/master"},"78e9e964aa0866ae3a414637517ee5db24da5960":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2013-02-02 00:58:06.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/5"}}},"commit":{"parents":[{"commit":"a814f22ce49a3674fd6f266f52bf7de990521adc","subject":"Minor fix to readme - corrected indentation for some commands","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a814f22ce49a3674fd6f266f52bf7de990521adc"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-02 00:55:59.000000000","tz":-360},"subject":"Add optional silent install and config of ldap to devstack","message":"Add optional silent install and config of ldap to devstack\n\nAddressed all comments from the previous patch iteration.\nFixed title of patch\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/78e9e964aa0866ae3a414637517ee5db24da5960"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/78e9e964aa0866ae3a414637517ee5db24da5960"}]},"branch":"refs/heads/master"},"0d1825aae6b8c4af5c052af708c2a5c1c1b2fa24":{"kind":"REWORK","_number":6,"created":"2013-02-04 06:49:17.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/6"}}},"commit":{"parents":[{"commit":"f3da41a5eea19f429a296405bfa3b72d899c3141","subject":"Merge \"Introduce get_python_exec_prefix function.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f3da41a5eea19f429a296405bfa3b72d899c3141"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-04 06:48:45.000000000","tz":-360},"subject":"Add optional silent install and config of ldap to devstack","message":"Add optional silent install and config of ldap to devstack\n\nMade code changes suggested by Sean and added\nKEYSTONE_CLEAR_LDAP option to refresh an already installed\nldap\n\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0d1825aae6b8c4af5c052af708c2a5c1c1b2fa24"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0d1825aae6b8c4af5c052af708c2a5c1c1b2fa24"}]},"branch":"refs/heads/master"},"5abc04c7fe2a448c97b2e6148248f01fa63e652c":{"kind":"REWORK","_number":7,"created":"2013-02-04 20:21:09.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/7"}}},"commit":{"parents":[{"commit":"f3da41a5eea19f429a296405bfa3b72d899c3141","subject":"Merge \"Introduce get_python_exec_prefix function.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f3da41a5eea19f429a296405bfa3b72d899c3141"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-04 20:18:25.000000000","tz":-360},"subject":"Add optional silent install and config of ldap to devstack","message":"Add optional silent install and config of ldap to devstack\n\nAdded documentation of new variables I added (requested by Sean)\nand made all code changes suggested by ayoung.\n\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5abc04c7fe2a448c97b2e6148248f01fa63e652c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5abc04c7fe2a448c97b2e6148248f01fa63e652c"}]},"branch":"refs/heads/master"},"658bf9c61683f6d5193c8cf0fb732995d29056d0":{"kind":"REWORK","_number":8,"created":"2013-02-04 20:34:51.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/8"}}},"commit":{"parents":[{"commit":"f3da41a5eea19f429a296405bfa3b72d899c3141","subject":"Merge \"Introduce get_python_exec_prefix function.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f3da41a5eea19f429a296405bfa3b72d899c3141"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-04 20:32:45.000000000","tz":-360},"subject":"Add optional silent install and config of ldap to devstack","message":"Add optional silent install and config of ldap to devstack\n\nRemoved trailing white spaces. The previous patch\nadded documentation of new variables I added (requested by Sean)\nand made all code changes suggested by ayoung and all of this\nshould be reflected in this patch.\n\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/658bf9c61683f6d5193c8cf0fb732995d29056d0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/658bf9c61683f6d5193c8cf0fb732995d29056d0"}]},"branch":"refs/heads/master"},"f127e2f316f1161bacdf4cccdbc3e56b2b8a54a8":{"kind":"REWORK","_number":9,"created":"2013-02-06 02:02:31.000000000","uploader":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"ref":"refs/changes/49/20249/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/49/20249/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/49/20249/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/49/20249/9"}}},"commit":{"parents":[{"commit":"f3da41a5eea19f429a296405bfa3b72d899c3141","subject":"Merge \"Introduce get_python_exec_prefix function.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f3da41a5eea19f429a296405bfa3b72d899c3141"}]}],"author":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-01-22 16:17:50.000000000","tz":-360},"committer":{"name":"Brad Topol","email":"btopol@us.ibm.com","date":"2013-02-06 02:00:15.000000000","tz":-360},"subject":"Add optional silent install and config of ldap to devstack","message":"Add optional silent install and config of ldap to devstack\n\nEdited initial ldap entries and olcdb template file as recommended by Brant.\n\nChange-Id: I1404cc5c754f878e32a2d10254840d092211e6e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f127e2f316f1161bacdf4cccdbc3e56b2b8a54a8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f127e2f316f1161bacdf4cccdbc3e56b2b8a54a8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}