)]}'
{"devstack/README.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac9a10674adac067abcdfd9445e02e665e0af7a","unresolved":false,"context_lines":[{"line_number":40,"context_line":"services. Each integrated service may require additional configuration"},{"line_number":41,"context_line":"settings. For example, typically, you will need to add the"},{"line_number":42,"context_line":"``searchlight_indexer`` notification topic to each service\u0027s configuration."},{"line_number":43,"context_line":"Please review the plugins and add configuration appropriately."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"   http://docs.openstack.org/developer/searchlight/plugins.html"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_65b5ed2f","line":43,"updated":"2015-10-01 16:47:49.000000000","message":"I\u0027d suggest making this sentence a new paragraph (ending with a colon) so that the hyperlink isn\u0027t hanging out there all by itself.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":40,"context_line":"services. Each integrated service may require additional configuration"},{"line_number":41,"context_line":"settings. For example, typically, you will need to add the"},{"line_number":42,"context_line":"``searchlight_indexer`` notification topic to each service\u0027s configuration."},{"line_number":43,"context_line":"Please review the plugins and add configuration appropriately."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"   http://docs.openstack.org/developer/searchlight/plugins.html"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_92fe0637","line":43,"in_reply_to":"9a1a9d01_355b9930","updated":"2015-10-07 05:35:30.000000000","message":"Done","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"56eee8b2e2669340345c0d7af8c6c674704c32d3","unresolved":false,"context_lines":[{"line_number":40,"context_line":"services. Each integrated service may require additional configuration"},{"line_number":41,"context_line":"settings. For example, typically, you will need to add the"},{"line_number":42,"context_line":"``searchlight_indexer`` notification topic to each service\u0027s configuration."},{"line_number":43,"context_line":"Please review the plugins and add configuration appropriately."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"   http://docs.openstack.org/developer/searchlight/plugins.html"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_355b9930","line":43,"in_reply_to":"9a1a9d01_65b5ed2f","updated":"2015-10-01 20:38:48.000000000","message":"Agreed, or \u0027Please review the plugin `documentation`_ and ...\u0027 and link in a RSTy way.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":40,"context_line":"services. Each integrated service may require additional configuration"},{"line_number":41,"context_line":"settings. For example, typically, you will need to add the"},{"line_number":42,"context_line":"``searchlight_indexer`` notification topic to each service\u0027s configuration."},{"line_number":43,"context_line":"Please review the plugins and add configuration appropriately."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"   http://docs.openstack.org/developer/searchlight/plugins.html"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_32fdb23a","line":43,"in_reply_to":"9a1a9d01_65b5ed2f","updated":"2015-10-07 05:35:30.000000000","message":"Done","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"}],"doc/source/index.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac9a10674adac067abcdfd9445e02e665e0af7a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"This documentation is generated by the Sphinx toolkit using `tox -e docs` and"},{"line_number":38,"context_line":"lives in the source tree."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Additional documentation on Searchlight can be found on the `wiki`_."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":".. _`wiki`: https://wiki.openstack.org/wiki/Searchlight"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_455469c9","line":40,"updated":"2015-10-01 16:47:49.000000000","message":"Just curious why you dropped the \u0027OpenStack\u0027 before \u0027wiki\u0027.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":37,"context_line":"This documentation is generated by the Sphinx toolkit using `tox -e docs` and"},{"line_number":38,"context_line":"lives in the source tree."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Additional documentation on Searchlight can be found on the `wiki`_."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":".. _`wiki`: https://wiki.openstack.org/wiki/Searchlight"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_b2dd4244","line":40,"in_reply_to":"9a1a9d01_455469c9","updated":"2015-10-07 05:35:30.000000000","message":"Lack of oxygen to the brain, probably. Added back.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"}],"doc/source/indexingservice.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    $ searchlight-manage index sync --index searchlight"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"You will be prompted to confirm unless --force is provided."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"The ``searchlight-manage index sync`` command may be re-run at any time to"},{"line_number":82,"context_line":"perform a full re-index of the data. This will delete the data, any mappings,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_763eacb9","line":79,"updated":"2015-10-07 13:46:51.000000000","message":"I\u0027d suggest double backticks so that the two dashes are obvious in --force when the rst is rendered.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    $ searchlight-manage index sync --index searchlight"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"You will be prompted to confirm unless --force is provided."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"The ``searchlight-manage index sync`` command may be re-run at any time to"},{"line_number":82,"context_line":"perform a full re-index of the data. This will delete the data, any mappings,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1fb18847","line":79,"in_reply_to":"7a2fa921_763eacb9","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":89,"context_line":"supplement for services that do not produce intermediate status change"},{"line_number":90,"context_line":"notifications."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_f6863c64","line":92,"updated":"2015-10-07 13:46:51.000000000","message":"Those bastards!  But my point is, there should be an uppercase L in \u0027liberty\u0027.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":89,"context_line":"supplement for services that do not produce intermediate status change"},{"line_number":90,"context_line":"notifications."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_9f835856","line":92,"in_reply_to":"7a2fa921_f6863c64","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"},{"line_number":96,"context_line":"information for indexed images::"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_569850c0","line":94,"updated":"2015-10-07 13:46:51.000000000","message":"\u0027fo\u0027 should be \u0027to\u0027","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"},{"line_number":96,"context_line":"information for indexed images::"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_7f80bc46","line":94,"in_reply_to":"7a2fa921_569850c0","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"},{"line_number":96,"context_line":"information for indexed images::"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    searchlight-manage index sync --type OS::Glance::Image --force --no-delete"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_f6d71c21","line":95,"updated":"2015-10-07 13:46:51.000000000","message":"I think \u0027setup\u0027 should be \u0027set up\u0027 here.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":92,"context_line":"For example, in the liberty release, the Glance service did not provide image"},{"line_number":93,"context_line":"membership update notifications, even though it provided image update and"},{"line_number":94,"context_line":"delete notifications. In order fo provide accurate membership information, a"},{"line_number":95,"context_line":"cron job could be setup with the following command to get the most up to date"},{"line_number":96,"context_line":"information for indexed images::"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    searchlight-manage index sync --type OS::Glance::Image --force --no-delete"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1f98a8bf","line":95,"in_reply_to":"7a2fa921_f6d71c21","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Once a resource has been indexed, typically you will only need to consume"},{"line_number":104,"context_line":"incremental updates rather than re-index the entire data set again. The"},{"line_number":105,"context_line":"preferred methodolgy is to setup notification listening."},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"Notifications"},{"line_number":108,"context_line":"^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_b62a042b","line":105,"updated":"2015-10-07 13:46:51.000000000","message":"I think \u0027setup\u0027 should be \u0027set up\u0027 here.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Once a resource has been indexed, typically you will only need to consume"},{"line_number":104,"context_line":"incremental updates rather than re-index the entire data set again. The"},{"line_number":105,"context_line":"preferred methodolgy is to setup notification listening."},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"Notifications"},{"line_number":108,"context_line":"^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_7ff75cf5","line":105,"in_reply_to":"7a2fa921_b62a042b","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"d55ba2e5dbb5f46d8ac0217fccd75cba932a40f0","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Bulk indexing"},{"line_number":61,"context_line":"-------------"},{"line_number":62,"context_line":"To initially create the catalog index (or add new resource typs to it later),"},{"line_number":63,"context_line":"run the following command::"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    $ searchlight-manage index sync"}],"source_content_type":"text/x-rst","patch_set":8,"id":"7a2fa921_ad4d7f3f","line":62,"updated":"2015-10-09 15:16:39.000000000","message":"\u0027types\u0027","commit_id":"c0dac2599323a547dd46c133c22b08379c364b9e"}],"doc/source/plugins.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac9a10674adac067abcdfd9445e02e665e0af7a","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The search service is driven using a plugin mechanism. Each integrated service"},{"line_number":21,"context_line":"may require additional configuration settings. For example, typically, you"},{"line_number":22,"context_line":"will need to add the `searchlight_indexer`` notification topic to each"},{"line_number":23,"context_line":"service\u0027s configuration."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Please review the plugins and add configuration appropriately. Information"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_c58079fa","line":22,"updated":"2015-10-01 16:47:49.000000000","message":"There should be 2 backticks before \u0027searchlight_indexer\u0027","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"56eee8b2e2669340345c0d7af8c6c674704c32d3","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The search service is driven using a plugin mechanism. Each integrated service"},{"line_number":21,"context_line":"may require additional configuration settings. For example, typically, you"},{"line_number":22,"context_line":"will need to add the `searchlight_indexer`` notification topic to each"},{"line_number":23,"context_line":"service\u0027s configuration."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Please review the plugins and add configuration appropriately. Information"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_f53d21bc","line":22,"in_reply_to":"9a1a9d01_c58079fa","updated":"2015-10-01 20:38:48.000000000","message":"You have eyes like an eagle.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The search service is driven using a plugin mechanism. Each integrated service"},{"line_number":21,"context_line":"may require additional configuration settings. For example, typically, you"},{"line_number":22,"context_line":"will need to add the `searchlight_indexer`` notification topic to each"},{"line_number":23,"context_line":"service\u0027s configuration."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Please review the plugins and add configuration appropriately. Information"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_32f292d3","line":22,"in_reply_to":"9a1a9d01_f53d21bc","updated":"2015-10-07 05:35:30.000000000","message":"Done","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"}],"doc/source/plugins/designate.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_114bc2ca","line":30,"updated":"2015-10-07 13:46:51.000000000","message":"Nit: \u0027it\u0027 is referring to \u0027options\u0027, so you have singular referring to plural (there\u0027s a technical term but I don\u0027t recall it).  Maybe replace \u0027it\u0027 with \u0027any\u0027?  (Or just ignore me.)","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_9f06f8cc","line":30,"in_reply_to":"7a2fa921_114bc2ca","updated":"2015-10-07 14:57:04.000000000","message":"Good suggestion... My midnight doc writing skills are especially suspect, so I appreciate the close read.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":44,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":45,"context_line":"::"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    [resource_plugin:os_designate_zone]"},{"line_number":48,"context_line":"    enabled \u003d true"},{"line_number":49,"context_line":"    index_name \u003d searchlight"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_16c65848","line":47,"updated":"2015-10-07 13:46:51.000000000","message":"I think \u0027os_designate_zone\u0027 should be \u0027os_designate_recordset\u0027.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":44,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":45,"context_line":"::"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    [resource_plugin:os_designate_zone]"},{"line_number":48,"context_line":"    enabled \u003d true"},{"line_number":49,"context_line":"    index_name \u003d searchlight"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1ffbc8c2","line":47,"in_reply_to":"7a2fa921_16c65848","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":51,"context_line":".. warning::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    You need to have both *os_designate_zone* and *os_designate_recordset*"},{"line_number":54,"context_line":"    enabled at because of a parent child relationship between them."},{"line_number":55,"context_line":"    They must also have the same index_name."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Designate Configuration"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1695f8dc","line":54,"updated":"2015-10-07 13:46:51.000000000","message":"Probably change \u0027enabled at\u0027 to \u0027enabled together\u0027.\n\nYou might also want to say which is the parent and which is the child.  I take it that the parent could be \"on\" and the kid \"off\", but not vice-versa.  Because of the placement of the warning, I imagine that \u0027recordset\u0027 is the child, but might as well state it explicitly.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":51,"context_line":".. warning::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    You need to have both *os_designate_zone* and *os_designate_recordset*"},{"line_number":54,"context_line":"    enabled at because of a parent child relationship between them."},{"line_number":55,"context_line":"    They must also have the same index_name."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Designate Configuration"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_7f9d1c58","line":54,"in_reply_to":"7a2fa921_1695f8dc","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    You need to have both *os_designate_zone* and *os_designate_recordset*"},{"line_number":54,"context_line":"    enabled at because of a parent child relationship between them."},{"line_number":55,"context_line":"    They must also have the same index_name."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Designate Configuration"},{"line_number":58,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_367ef4ae","line":55,"updated":"2015-10-07 13:46:51.000000000","message":"I know it seems silly, but I\u0027d pull this out into a separate warning to make it clear that you\u0027re making two points here.  Or you could keep them in one box if you bullet them or something, whatever you think would be best.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    You need to have both *os_designate_zone* and *os_designate_recordset*"},{"line_number":54,"context_line":"    enabled at because of a parent child relationship between them."},{"line_number":55,"context_line":"    They must also have the same index_name."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Designate Configuration"},{"line_number":58,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_bf93b44e","line":55,"in_reply_to":"7a2fa921_367ef4ae","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":106,"context_line":"temporarily not containing any data (which would happen with a full bulk"},{"line_number":107,"context_line":"indexing job)::"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    searchlight-manage index sync --type OS::Designate::Zone,OS::Designate::RecordSet --force --no-delete"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_319cbe68","line":109,"updated":"2015-10-07 13:46:51.000000000","message":"I thought the --no-delete option was removed from searchlight-manage?  IIIRC the idea was that there was no use case, if you really wanted an old index, you\u0027d save it before re-indexing.\n\nI don\u0027t see --no-delete in the help output for searchlight-manage, but maybe I\u0027m looking at an outdated version.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":106,"context_line":"temporarily not containing any data (which would happen with a full bulk"},{"line_number":107,"context_line":"indexing job)::"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    searchlight-manage index sync --type OS::Designate::Zone,OS::Designate::RecordSet --force --no-delete"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_df4a30c5","line":109,"in_reply_to":"7a2fa921_319cbe68","updated":"2015-10-07 14:57:04.000000000","message":"We had to add it back in because of the glance membership notifications situation and to help with this situation.  It just seemed that we didn\u0027t have an alternative way to do this in Liberty that wouldn\u0027t require a downtime of the index.  In Mitaka, I want to enable zero downtime indexing (with delete).  I\u0027ve opened that blueprint here: https://blueprints.launchpad.net/searchlight/+spec/zero-downtime-reindexing\n\nSee line 81 - 99 here: https://review.openstack.org/#/c/228664/3/doc/source/indexingservice.rst\n\nThere should be help output:\n\nhttps://github.com/openstack/searchlight/blob/master/searchlight/cmd/manage.py#L53","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fe840594a47dd55bac402eba02e82bc8c97de882","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    file after stacking or modify the searchlight post config section in"},{"line_number":82,"context_line":"    ``local.conf`` prior to stacking by adding a"},{"line_number":83,"context_line":"    ``[[post-config|$SEARCHLIGHT_CONF]]`` section."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"The Designate plugin must be enabled and run with devstack to include Designate"},{"line_number":86,"context_line":"with your devstack deployment. Follow the instructions here:"},{"line_number":87,"context_line":"http://docs.openstack.org/developer/designate/devstack.html"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a2fa921_047930c8","line":84,"updated":"2015-10-07 17:51:21.000000000","message":"I think the Note could use a rewrite.  How\u0027s this:\n\n\n.. note::\n\n    Designate resource types are *not* enabled by default (``enabled \u003d false``)\n    in the searchlight devstack script because designate is not\n    installed by default in devstack.  You have two options for enabling\n    designate resource types:\n\n    1. Prior to stacking: modify the searchlight post config section in\n    ``local.conf`` by adding a ``[[post-config|$SEARCHLIGHT_CONF]]`` section.\n\n    2. After stacking: manually edit the ``searchlight-api.conf`` file","commit_id":"2cf9c693bbdec9c11d2ea1829427abc6871ade7d"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"7db453eba4b698bce54028c3cde3b6345fac7dd5","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    file after stacking or modify the searchlight post config section in"},{"line_number":82,"context_line":"    ``local.conf`` prior to stacking by adding a"},{"line_number":83,"context_line":"    ``[[post-config|$SEARCHLIGHT_CONF]]`` section."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"The Designate plugin must be enabled and run with devstack to include Designate"},{"line_number":86,"context_line":"with your devstack deployment. Follow the instructions here:"},{"line_number":87,"context_line":"http://docs.openstack.org/developer/designate/devstack.html"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a2fa921_ea307dea","line":84,"in_reply_to":"7a2fa921_047930c8","updated":"2015-10-07 18:20:22.000000000","message":"Definitely better.","commit_id":"2cf9c693bbdec9c11d2ea1829427abc6871ade7d"}],"doc/source/plugins/glance.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_3174befb","line":30,"updated":"2015-10-07 13:46:51.000000000","message":"Nit: \u0027it\u0027 is referring to \u0027options\u0027, so you have singular referring to plural (there\u0027s a technical term but I don\u0027t recall it).  Maybe replace \u0027it\u0027 with \u0027any\u0027?  (Or just ignore me.)","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_9f1358b8","line":30,"in_reply_to":"7a2fa921_3174befb","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":51,"context_line":"To configure it, add a property_protection_file property with a path"},{"line_number":52,"context_line":"to the file in searchlight-api.conf. For example::"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    property_protection_file \u003d ~/openstack/searchlight/etc/property-protections-roles.conf"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Plugin: OS::Glance::Metadef"},{"line_number":57,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1183c299","line":54,"updated":"2015-10-07 13:46:51.000000000","message":"I\u0027d change this to an absolute path ... then if someone uses a relative path and it breaks, it\u0027s their fault.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"To configure it, add a property_protection_file property with a path"},{"line_number":52,"context_line":"to the file in searchlight-api.conf. For example::"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    property_protection_file \u003d ~/openstack/searchlight/etc/property-protections-roles.conf"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Plugin: OS::Glance::Metadef"},{"line_number":57,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_df21d0fc","line":54,"in_reply_to":"7a2fa921_1183c299","updated":"2015-10-07 14:57:04.000000000","message":"Good call. Done.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":100,"context_line":"-----------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Glance did not generate notifications for Image Member updates up to and"},{"line_number":103,"context_line":"including Liberty release."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"Search results will include correct results when the visibility is ``public``"},{"line_number":106,"context_line":"or ``private``, but ``shared`` images will only be included in search results"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_5127fac9","line":103,"updated":"2015-10-07 13:46:51.000000000","message":"\"the\" Liberty release?","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":100,"context_line":"-----------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Glance did not generate notifications for Image Member updates up to and"},{"line_number":103,"context_line":"including Liberty release."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"Search results will include correct results when the visibility is ``public``"},{"line_number":106,"context_line":"or ``private``, but ``shared`` images will only be included in search results"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_3f6a441b","line":103,"in_reply_to":"7a2fa921_5127fac9","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Glance did not generate notifications for Image Member updates up to and"},{"line_number":103,"context_line":"including Liberty release."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"Search results will include correct results when the visibility is ``public``"},{"line_number":106,"context_line":"or ``private``, but ``shared`` images will only be included in search results"},{"line_number":107,"context_line":"for the owning project without additional deployment configuration."},{"line_number":108,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_7109d630","line":105,"updated":"2015-10-07 13:46:51.000000000","message":"maybe change \u0027visibility\u0027 to \u0027image visibility\u0027","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":120,"context_line":"indexing job)::"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    searchlight-manage index sync --type OS::Glance::Image --force --no-delete"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_9cb5a9ef","line":123,"updated":"2015-10-07 13:46:51.000000000","message":"See my comment about --no-delete on the designate page.","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":120,"context_line":"indexing job)::"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    searchlight-manage index sync --type OS::Glance::Image --force --no-delete"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1fdd08e0","line":123,"in_reply_to":"7a2fa921_9cb5a9ef","updated":"2015-10-07 14:57:04.000000000","message":"Yep, I put a comment back. Thanks!","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"}],"doc/source/plugins/nova.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac9a10674adac067abcdfd9445e02e665e0af7a","unresolved":false,"context_lines":[{"line_number":45,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Since changes to Neutron can affect Nova instances you may optionally turn on"},{"line_number":48,"context_line":"notifications for neutron.  If you do not, networking changes will only be"},{"line_number":49,"context_line":"picked up by Searchlight when notifications are received from Nova."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Open neutron.conf and make the following changes::"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_8525a10b","line":48,"updated":"2015-10-01 16:47:49.000000000","message":"This isn\u0027t part of your change, but should \u0027neutron\u0027 be capitalized here?","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":45,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Since changes to Neutron can affect Nova instances you may optionally turn on"},{"line_number":48,"context_line":"notifications for neutron.  If you do not, networking changes will only be"},{"line_number":49,"context_line":"picked up by Searchlight when notifications are received from Nova."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Open neutron.conf and make the following changes::"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_12be76d4","line":48,"in_reply_to":"9a1a9d01_8525a10b","updated":"2015-10-07 05:35:30.000000000","message":"honestly not sure... but I\u0027ll do it for consistency.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac9a10674adac067abcdfd9445e02e665e0af7a","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    notification_topics \u003d searchlight_indexer"},{"line_number":55,"context_line":"    rpc_backend \u003d \u0027rabbit\u0027"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Restart neutron service (q-svc)."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Neutron Devstack Config"},{"line_number":60,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_6573ad16","line":57,"updated":"2015-10-01 16:47:49.000000000","message":"For consistency with the other services, don\u0027t want to capitalize it here.","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a33a1e876ef99fe4950c71021a2bee9b91798139","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    notification_topics \u003d searchlight_indexer"},{"line_number":55,"context_line":"    rpc_backend \u003d \u0027rabbit\u0027"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Restart neutron service (q-svc)."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Neutron Devstack Config"},{"line_number":60,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_d2b79ef3","line":57,"in_reply_to":"9a1a9d01_6573ad16","updated":"2015-10-07 05:35:30.000000000","message":"Done","commit_id":"edf64a72c88c57bd649555c79c42f45c4ef8fd27"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1526ad8a27aeab9b68656e28e2b9aadff35070c3","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_1122e200","line":30,"updated":"2015-10-07 13:46:51.000000000","message":"Nit: \u0027it\u0027 is referring to \u0027options\u0027, so you have singular referring to plural (there\u0027s a technical term but I don\u0027t recall it).  Maybe replace \u0027it\u0027 with \u0027any\u0027?  (Or just ignore me.)","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"6749474290510ccb6b71c259f7f3555d6ba30c0f","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Searchlight resource configuration options are shown below with their"},{"line_number":29,"context_line":"configuration file and default values. You only need to update the"},{"line_number":30,"context_line":"below configuration options if you decide to change it to a non-default value."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"searchlight-api.conf"},{"line_number":33,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a2fa921_7fec3c8f","line":30,"in_reply_to":"7a2fa921_1122e200","updated":"2015-10-07 14:57:04.000000000","message":"Done","commit_id":"575b7ca9ecbd1c4d38843e29fd73151022cdf32a"}]}