)]}'
{"keystone/conf/ldap.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":33,"context_line":"user \u003d cfg.StrOpt("},{"line_number":34,"context_line":"    \u0027user\u0027,"},{"line_number":35,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":36,"context_line":"The user name of the administrator bind DN to use when querying the LDAP"},{"line_number":37,"context_line":"server."},{"line_number":38,"context_line":"\"\"\"))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"password \u003d cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9f8c2337","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":7},"updated":"2016-07-02 02:32:30.000000000","message":"just a thought - Do we want to mention that these are not necessary for most LDAPs? Or that most support anonymous querying? I\u0027ve often responded to requests asking what username and password should be.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":33,"context_line":"user \u003d cfg.StrOpt("},{"line_number":34,"context_line":"    \u0027user\u0027,"},{"line_number":35,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":36,"context_line":"The user name of the administrator bind DN to use when querying the LDAP"},{"line_number":37,"context_line":"server."},{"line_number":38,"context_line":"\"\"\"))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"password \u003d cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_f4862912","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":7},"in_reply_to":"1aa78d24_9f8c2337","updated":"2016-07-02 08:05:48.000000000","message":"I believe most LDAP servers require user/pwd to do any kind of connection with encryption.  Pretty sure AD does.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    \u0027suffix\u0027,"},{"line_number":49,"context_line":"    default\u003d\u0027cn\u003dexample,cn\u003dcom\u0027,"},{"line_number":50,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":51,"context_line":"The default LDAP server suffix to use, if a DN is not defined via either"},{"line_number":52,"context_line":"`user_tree_dn` or `group_tree_dn`."},{"line_number":53,"context_line":"\"\"\"))"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"use_dumb_member \u003d cfg.BoolOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7ff177cd","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":34},"updated":"2016-07-02 02:32:30.000000000","message":"so much better","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    default\u003d\u0027cn\u003dexample,cn\u003dcom\u0027,"},{"line_number":50,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":51,"context_line":"The default LDAP server suffix to use, if a DN is not defined via either"},{"line_number":52,"context_line":"`user_tree_dn` or `group_tree_dn`."},{"line_number":53,"context_line":"\"\"\"))"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"use_dumb_member \u003d cfg.BoolOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_dfb86be8","line":52,"range":{"start_line":52,"start_character":0,"end_line":52,"end_character":33},"updated":"2016-07-02 02:32:30.000000000","message":"so not a big deal, but you prefix \"dumb_member\" with [ldap] on line 59, but these two are not prefixed with [ldap], if you are tossing up another patch, consider adding it","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    \u0027use_dumb_member\u0027,"},{"line_number":57,"context_line":"    default\u003dFalse,"},{"line_number":58,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":59,"context_line":"If true, Keystone will add a dummy member based on the `[ldap] dumb_member`"},{"line_number":60,"context_line":"option when creating new groups. This is required if the `objectClass` for"},{"line_number":61,"context_line":"groups requires the `member` attribute. This option is only used for write"},{"line_number":62,"context_line":"operations."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_5fee3bed","line":59,"range":{"start_line":59,"start_character":9,"end_line":59,"end_character":17},"updated":"2016-07-02 02:32:30.000000000","message":"keystone*","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":59,"context_line":"If true, Keystone will add a dummy member based on the `[ldap] dumb_member`"},{"line_number":60,"context_line":"option when creating new groups. This is required if the `objectClass` for"},{"line_number":61,"context_line":"groups requires the `member` attribute. This option is only used for write"},{"line_number":62,"context_line":"operations."},{"line_number":63,"context_line":"\"\"\"))"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"dumb_member \u003d cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_1499356f","line":62,"range":{"start_line":61,"start_character":40,"end_line":62,"end_character":11},"updated":"2016-07-02 08:05:48.000000000","message":"Should be marked deprecated then.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":66,"context_line":"    \u0027dumb_member\u0027,"},{"line_number":67,"context_line":"    default\u003d\u0027cn\u003ddumb,dc\u003dnonexistent\u0027,"},{"line_number":68,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":69,"context_line":"DN of the \"dummy member\" to use when `[ldap] use_dumb_member` is enabled. This"},{"line_number":70,"context_line":"option is only used for write operations."},{"line_number":71,"context_line":"\"\"\"))"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"allow_subtree_delete \u003d cfg.BoolOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_bfa0ffa6","line":70,"range":{"start_line":69,"start_character":74,"end_line":70,"end_character":40},"updated":"2016-07-02 02:32:30.000000000","message":"we should mark this option as deprecated and add the deprecation message here..","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    default\u003dFalse,"},{"line_number":76,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":77,"context_line":"Delete subtrees using the subtree delete control. Only enable this option if"},{"line_number":78,"context_line":"your LDAP server supports subtree deletion. This option is only used for write"},{"line_number":79,"context_line":"operations."},{"line_number":80,"context_line":"\"\"\"))"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"query_scope \u003d cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9fa543b7","line":79,"range":{"start_line":78,"start_character":44,"end_line":79,"end_character":11},"updated":"2016-07-02 02:32:30.000000000","message":"we should mark this option as deprecated and add the deprecation message here..","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":93,"context_line":"\"\"\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"page_size \u003d cfg.IntOpt("},{"line_number":96,"context_line":"    \u0027page_size\u0027,"},{"line_number":97,"context_line":"    default\u003d0,"},{"line_number":98,"context_line":"    min\u003d0,"},{"line_number":99,"context_line":"    help\u003dutils.fmt(\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_df86ab12","line":96,"range":{"start_line":96,"start_character":5,"end_line":96,"end_character":14},"updated":"2016-07-02 02:32:30.000000000","message":"i believe specifying this option is only meaning if username and password are set, and that username has sufficient privileges to actually perform paging - consider adding that if you think it\u0027s useful","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"d2dc0043add7c5452c18f6fbec11b7f7016d971a","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    \u0027debug_level\u0027,"},{"line_number":120,"context_line":"    min\u003d-1,"},{"line_number":121,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":122,"context_line":"Sets the LDAP debugging level for LDAP calls. A value of 0 means that debugging"},{"line_number":123,"context_line":"is not enabled. This value is a bitmask, consult your LDAP documentation for"},{"line_number":124,"context_line":"possible values."},{"line_number":125,"context_line":"\"\"\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_eb1f7159","line":122,"updated":"2016-07-01 21:51:29.000000000","message":"Since the min value is -1 it would be good to say what a -1 value means as a debug level similar to how you mention what a 0 value means.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"71d1b51f752efa76865414fa4d0861f10f501f72","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    \u0027debug_level\u0027,"},{"line_number":120,"context_line":"    min\u003d-1,"},{"line_number":121,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":122,"context_line":"Sets the LDAP debugging level for LDAP calls. A value of 0 means that debugging"},{"line_number":123,"context_line":"is not enabled. This value is a bitmask, consult your LDAP documentation for"},{"line_number":124,"context_line":"possible values."},{"line_number":125,"context_line":"\"\"\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_d49df25a","line":122,"in_reply_to":"1aa78d24_ab2ba932","updated":"2016-07-02 00:41:21.000000000","message":"K, Wasn\u0027t a big deal either way","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"26e201bc19233a06b1c5500b9e2fd4e12eb09caf","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    \u0027debug_level\u0027,"},{"line_number":120,"context_line":"    min\u003d-1,"},{"line_number":121,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":122,"context_line":"Sets the LDAP debugging level for LDAP calls. A value of 0 means that debugging"},{"line_number":123,"context_line":"is not enabled. This value is a bitmask, consult your LDAP documentation for"},{"line_number":124,"context_line":"possible values."},{"line_number":125,"context_line":"\"\"\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_ab2ba932","line":122,"in_reply_to":"1aa78d24_eb1f7159","updated":"2016-07-01 21:59:53.000000000","message":"The meaning of -1 depends on the implementation of LDAP :( It\u0027s actually meaningless is most implementations, but supported by OpenLDAP and it\u0027s derivatives.\n\n0 is the only value with a common meaning across implementations, AFAIK, but I\u0027m not sure why it\u0027s not the default value here either.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":127,"context_line":"chase_referrals \u003d cfg.BoolOpt("},{"line_number":128,"context_line":"    \u0027chase_referrals\u0027,"},{"line_number":129,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":130,"context_line":"Sets Keystone\u0027s referral chasing behavior across directory partitions. If left"},{"line_number":131,"context_line":"unset, the system\u0027s default behavior will be used."},{"line_number":132,"context_line":"\"\"\"))"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_5f73db4d","line":130,"range":{"start_line":130,"start_character":5,"end_line":130,"end_character":13},"updated":"2016-07-02 02:32:30.000000000","message":"keystone* there are a bunch of these below, i\u0027m okay with either Keystone or keystone, i forget what we\u0027ve been using in your patches so far.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    \u0027chase_referrals\u0027,"},{"line_number":129,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":130,"context_line":"Sets Keystone\u0027s referral chasing behavior across directory partitions. If left"},{"line_number":131,"context_line":"unset, the system\u0027s default behavior will be used."},{"line_number":132,"context_line":"\"\"\"))"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"user_tree_dn \u003d cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_f4b469d2","line":131,"range":{"start_line":131,"start_character":11,"end_line":131,"end_character":17},"updated":"2016-07-02 08:05:48.000000000","message":"What is meant by system?  The underlying ldap native client? Or the LDAP server?","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":234,"context_line":"\"\"\"))"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"user_attribute_ignore \u003d cfg.ListOpt("},{"line_number":237,"context_line":"    \u0027user_attribute_ignore\u0027,"},{"line_number":238,"context_line":"    default\u003d[\u0027default_project_id\u0027],"},{"line_number":239,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":240,"context_line":"List of user attributes to ignore on create and update. This is only used for"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7a34e5dd","line":237,"range":{"start_line":237,"start_character":5,"end_line":237,"end_character":26},"updated":"2016-07-02 02:32:30.000000000","message":"we should deprecate this option too","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":329,"context_line":"    \u0027group_objectclass\u0027,"},{"line_number":330,"context_line":"    default\u003d\u0027groupOfNames\u0027,"},{"line_number":331,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":332,"context_line":"The LDAP `objectclass` to use for groups. If setting this option to"},{"line_number":333,"context_line":"`posixGroup`, you may also be interested in enabling the `[ldap]"},{"line_number":334,"context_line":"group_members_are_ids` option."},{"line_number":335,"context_line":"\"\"\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_da5b592d","line":332,"range":{"start_line":332,"start_character":10,"end_line":332,"end_character":21},"updated":"2016-07-02 02:32:30.000000000","message":"see line 365","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":362,"context_line":"    \u0027group_members_are_ids\u0027,"},{"line_number":363,"context_line":"    default\u003dFalse,"},{"line_number":364,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":365,"context_line":"Enable this option if the members of the group `objectClass` are Keystone user"},{"line_number":366,"context_line":"IDs rather than LDAP DNs. This is the case when using `posixGroup` as the group"},{"line_number":367,"context_line":"`objectClass` in Open Directory."},{"line_number":368,"context_line":"\"\"\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_1a3e81b8","line":365,"range":{"start_line":365,"start_character":48,"end_line":365,"end_character":59},"updated":"2016-07-02 02:32:30.000000000","message":"objectclass vs objectClass (theres another example of this for users)","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":378,"context_line":"    \u0027group_attribute_ignore\u0027,"},{"line_number":379,"context_line":"    default\u003d[],"},{"line_number":380,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":381,"context_line":"List of group attributes to ignore on create and update. This is only used for"},{"line_number":382,"context_line":"write operations."},{"line_number":383,"context_line":"\"\"\"))"},{"line_number":384,"context_line":""},{"line_number":385,"context_line":"group_allow_create \u003d cfg.BoolOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_ba73edb3","line":382,"range":{"start_line":381,"start_character":57,"end_line":382,"end_character":16},"updated":"2016-07-02 02:32:30.000000000","message":"we should deprecate this option","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":428,"context_line":"\"\"\"))"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"tls_cacertdir \u003d cfg.StrOpt("},{"line_number":431,"context_line":"    \u0027tls_cacertdir\u0027,"},{"line_number":432,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":433,"context_line":"An absolute path to a CA certificate directory to use when communicating with"},{"line_number":434,"context_line":"LDAP servers."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_b42f81f5","line":431,"range":{"start_line":431,"start_character":5,"end_line":431,"end_character":18},"updated":"2016-07-02 08:05:48.000000000","message":"Its not quite clear if both tls_cacertfile and tls_cacertdir have to be set, or just one.  Maybe you can add a blurb to clarify","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1cb82067fff00f8f40a86202e722c4475d643d36","unresolved":false,"context_lines":[{"line_number":439,"context_line":"    default\u003dFalse,"},{"line_number":440,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":441,"context_line":"Enable TLS when communicating with LDAP servers. You should also set the"},{"line_number":442,"context_line":"`[ldap] tls_cacertfile` and `[ldap] tls_cacertdir` options when using the"},{"line_number":443,"context_line":"option."},{"line_number":444,"context_line":"\"\"\"))"},{"line_number":445,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9a78b191","line":442,"range":{"start_line":442,"start_character":70,"end_line":442,"end_character":73},"updated":"2016-07-02 02:32:30.000000000","message":"this?","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    choices\u003d[\u0027demand\u0027, \u0027never\u0027, \u0027allow\u0027],"},{"line_number":450,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":451,"context_line":"Specifies which checks to perform against client certificates on incoming TLS"},{"line_number":452,"context_line":"sessions."},{"line_number":453,"context_line":"\"\"\"))"},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"use_pool \u003d cfg.BoolOpt("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_d4116db1","line":452,"updated":"2016-07-02 08:05:48.000000000","message":"I think it would be useful to detail more about the choices.","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":472,"context_line":"pool_retry_max \u003d cfg.IntOpt("},{"line_number":473,"context_line":"    \u0027pool_retry_max\u0027,"},{"line_number":474,"context_line":"    default\u003d3,"},{"line_number":475,"context_line":"    min\u003d0,"},{"line_number":476,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":477,"context_line":"The maximum number of times to attempt reconnecting to the LDAP server before"},{"line_number":478,"context_line":"aborting. This option has no effect unless `[ldap] use_pool` is also enabled."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9449a5d3","line":475,"range":{"start_line":475,"start_character":8,"end_line":475,"end_character":9},"updated":"2016-07-02 08:05:48.000000000","message":"What does a value of 0 indicate? Infinite retries? Or no retries?","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"0a81882ac2ea125e46f4a6f1f7e404d192dc1c34","unresolved":false,"context_lines":[{"line_number":481,"context_line":"pool_retry_delay \u003d cfg.FloatOpt("},{"line_number":482,"context_line":"    \u0027pool_retry_delay\u0027,"},{"line_number":483,"context_line":"    default\u003d0.1,"},{"line_number":484,"context_line":"    min\u003d0,"},{"line_number":485,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":486,"context_line":"The number of seconds to wait before attempting to reconnect to the LDAP"},{"line_number":487,"context_line":"server. This option has no effect unless `[ldap] use_pool` is also enabled."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7446b9c0","line":484,"range":{"start_line":484,"start_character":8,"end_line":484,"end_character":9},"updated":"2016-07-02 08:05:48.000000000","message":"0.0","commit_id":"c7aa00225467164e044b500ba3f109682a7fa8f8"}]}