)]}'
{"doc/source/indexingservice.rst":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"569ba81f19f20e088b0f3f6ed466fb1fa12b763e","unresolved":false,"context_lines":[{"line_number":191,"context_line":"associated with each index is also shown. A sample output will look"},{"line_number":192,"context_line":"like this::"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    $ searchlight-manage index aliases"},{"line_number":195,"context_line":"    List of Elasticsearch indices (and their associated aliases) used by Searchlight."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"    Indices are denoted with a \u0027*\u0027"}],"source_content_type":"text/x-rst","patch_set":7,"id":"dada55a8_1aebe9b1","line":194,"updated":"2016-07-19 04:41:57.000000000","message":"This example doesn\u0027t match the output with the Note about configuration files, but I\u0027m okay with that, since you explain it further below.","commit_id":"bad07b6f912e981251ffd1b7491f72befdcda223"}],"searchlight/cmd/manage.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"b6a657fb7d2e0d387e7eee71a05d7326c3713aaf","unresolved":false,"context_lines":[{"line_number":292,"context_line":"                LOG.error(encodeutils.exception_to_unicode(e))"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"class AliasCommands(object):"},{"line_number":296,"context_line":"    def __init__(self):"},{"line_number":297,"context_line":"        utils.register_plugin_opts()"},{"line_number":298,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_df043016","line":295,"range":{"start_line":295,"start_character":6,"end_line":295,"end_character":19},"updated":"2016-07-12 17:02:07.000000000","message":"Rather than add a separate command (leaving us with `searchlight-manage index sync` and `searchlight-manage alias alias`) maybe neater to add this to the existing command since it\u0027s strongly related - `searchlight-manage index aliases`? \u0027index\u0027 is sort of ambiguous; I don\u0027t think we have to restrict it to meaning \u0027insert data\u0027 but instead treat it as \u0027operations referring to indexing data\u0027.","commit_id":"1a625f71c51b76503ac52f054a5637b7d60393cf"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"b6a657fb7d2e0d387e7eee71a05d7326c3713aaf","unresolved":false,"context_lines":[{"line_number":296,"context_line":"    def __init__(self):"},{"line_number":297,"context_line":"        utils.register_plugin_opts()"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def alias(self, group\u003dNone, _type\u003dNone, force\u003dFalse):"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        # Grab a list of aliases used by Searchlight."},{"line_number":302,"context_line":"        aliases \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_7f3b7cd1","line":299,"range":{"start_line":299,"start_character":50,"end_line":299,"end_character":55},"updated":"2016-07-12 17:02:07.000000000","message":"I don\u0027t see where any of these parameters are used - copy n paste?","commit_id":"1a625f71c51b76503ac52f054a5637b7d60393cf"},{"author":{"_account_id":16150,"name":"Lei Zhang","email":"lei.a.zhang@intel.com","username":"leizhang"},"change_message_id":"0e1b4670550663aacbe66b06fb282e1057ec3de2","unresolved":false,"context_lines":[{"line_number":306,"context_line":"        for alias in set(aliases):"},{"line_number":307,"context_line":"            try:"},{"line_number":308,"context_line":"                response \u003d es_utils.get_indices(alias)"},{"line_number":309,"context_line":"            except Exception:"},{"line_number":310,"context_line":"                # Ignore and continue."},{"line_number":311,"context_line":"                response \u003d {}"},{"line_number":312,"context_line":"            for index in response.keys():"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_55e72046","line":309,"updated":"2016-07-14 09:11:23.000000000","message":"Is it better to catch only NotFoundError exception here? Otherwise we may miss something useful.","commit_id":"0f84fd3f2e9d897e003da5193a5a8e8b9ba412de"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"f1c26b015b294b4aa296c3b1bf2e8b87185b4a60","unresolved":false,"context_lines":[{"line_number":306,"context_line":"        for alias in set(aliases):"},{"line_number":307,"context_line":"            try:"},{"line_number":308,"context_line":"                response \u003d es_utils.get_indices(alias)"},{"line_number":309,"context_line":"            except Exception:"},{"line_number":310,"context_line":"                # Ignore and continue."},{"line_number":311,"context_line":"                response \u003d {}"},{"line_number":312,"context_line":"            for index in response.keys():"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_5dcba34a","line":309,"in_reply_to":"1aa78d24_03a7060c","updated":"2016-07-14 21:22:05.000000000","message":"The only other exceptions will be related to connection issues with ES. I can modify the get_indices to ignore 404 errors. Then we can catch all others, display a message to the user and exit. That way the user can try to resolve any ES issues and retry.","commit_id":"0f84fd3f2e9d897e003da5193a5a8e8b9ba412de"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"1e86a07fa83919ae986ce2e75866388794960ee9","unresolved":false,"context_lines":[{"line_number":306,"context_line":"        for alias in set(aliases):"},{"line_number":307,"context_line":"            try:"},{"line_number":308,"context_line":"                response \u003d es_utils.get_indices(alias)"},{"line_number":309,"context_line":"            except Exception:"},{"line_number":310,"context_line":"                # Ignore and continue."},{"line_number":311,"context_line":"                response \u003d {}"},{"line_number":312,"context_line":"            for index in response.keys():"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_03a7060c","line":309,"in_reply_to":"1aa78d24_55e72046","updated":"2016-07-14 18:48:55.000000000","message":"Agree we should only catch errors we might reasonably expect. I\u0027m ok with ignoring the error if we\u0027re clear that this only outputs indices actually in use.","commit_id":"0f84fd3f2e9d897e003da5193a5a8e8b9ba412de"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"870e9386743d19f2b61beae4c65fcd06825c191d","unresolved":false,"context_lines":[{"line_number":344,"context_line":"            print(\"\\nNo Elasticsearch indices for Searchlight exist.\")"},{"line_number":345,"context_line":"        else:"},{"line_number":346,"context_line":"            print(\"\\nList of Elasticsearch indices (and their associated\""},{"line_number":347,"context_line":"                  \" aliases) used by Searchlight.\\n\")"},{"line_number":348,"context_line":"            print(\"Indices are denoted with a \u0027*\u0027\")"},{"line_number":349,"context_line":"            print(\"Aliases are denoted with a \u0027+\u0027\\n\")"},{"line_number":350,"context_line":"            for index in indices:"}],"source_content_type":"text/x-python","patch_set":6,"id":"dada55a8_f5d1d1fc","line":347,"updated":"2016-07-18 23:12:45.000000000","message":"Is there a simple way to indicate to the user that this depends on the configuration file?  Maybe could say at the end of the listing \"To check the indexes and aliases used by a different Searchlight configuration file, use the --config-file option.\"","commit_id":"06f79a6c457dcec7b0534a602c5b2eb7ba487585"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"eb84b2020e8f9912f651380e8e4d03811a5d0eaf","unresolved":false,"context_lines":[{"line_number":345,"context_line":"            config_file \u003d CONF.find_file(\"searchlight.conf\")"},{"line_number":346,"context_line":"            print(\"\\nList of Elasticsearch indices (and their associated\""},{"line_number":347,"context_line":"                  \" aliases) used by Searchlight.\\n\")"},{"line_number":348,"context_line":"            print(\"Note:\")"},{"line_number":349,"context_line":"            if config_file:"},{"line_number":350,"context_line":"                print(\"The indices are based on the config file:\")"},{"line_number":351,"context_line":"                print(\"    \" + str(config_file))"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_29befb64","line":348,"range":{"start_line":348,"start_character":17,"end_line":348,"end_character":19},"updated":"2016-07-20 21:15:33.000000000","message":"This isn\u0027t documentation - just output what\u0027s necessary","commit_id":"5cfe31578a5156abd3363fe56d503ee338d948a3"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"0a184db8427962bbd5adbbf4e5bff761bc2ec193","unresolved":false,"context_lines":[{"line_number":345,"context_line":"            config_file \u003d CONF.find_file(\"searchlight.conf\")"},{"line_number":346,"context_line":"            print(\"\\nList of Elasticsearch indices (and their associated\""},{"line_number":347,"context_line":"                  \" aliases) used by Searchlight.\\n\")"},{"line_number":348,"context_line":"            print(\"Note:\")"},{"line_number":349,"context_line":"            if config_file:"},{"line_number":350,"context_line":"                print(\"The indices are based on the config file:\")"},{"line_number":351,"context_line":"                print(\"    \" + str(config_file))"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_00a301b3","line":348,"range":{"start_line":348,"start_character":17,"end_line":348,"end_character":19},"in_reply_to":"dada55a8_29befb64","updated":"2016-07-21 03:26:52.000000000","message":"manage.py has been denoted.","commit_id":"5cfe31578a5156abd3363fe56d503ee338d948a3"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"eb84b2020e8f9912f651380e8e4d03811a5d0eaf","unresolved":false,"context_lines":[{"line_number":348,"context_line":"            print(\"Note:\")"},{"line_number":349,"context_line":"            if config_file:"},{"line_number":350,"context_line":"                print(\"The indices are based on the config file:\")"},{"line_number":351,"context_line":"                print(\"    \" + str(config_file))"},{"line_number":352,"context_line":"            else:"},{"line_number":353,"context_line":"                print(\"The indices are based on the current config file.\")"},{"line_number":354,"context_line":"            print(\"To view indices used by other Searchlight config \""}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_8988c7c6","line":351,"range":{"start_line":351,"start_character":23,"end_line":351,"end_character":27},"updated":"2016-07-20 21:15:33.000000000","message":"This is wrong when you specify a config file.\n\nsearchlight-manage --config-file etc/searchlight-swift.conf index aliases\n\nThe indices are based on the config file:\n    /Users/mclellst/openstack/searchlight/etc/searchlight.conf\nTo view indices used by other Searchlight config files, use the --config-file option.","commit_id":"5cfe31578a5156abd3363fe56d503ee338d948a3"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"0a184db8427962bbd5adbbf4e5bff761bc2ec193","unresolved":false,"context_lines":[{"line_number":348,"context_line":"            print(\"Note:\")"},{"line_number":349,"context_line":"            if config_file:"},{"line_number":350,"context_line":"                print(\"The indices are based on the config file:\")"},{"line_number":351,"context_line":"                print(\"    \" + str(config_file))"},{"line_number":352,"context_line":"            else:"},{"line_number":353,"context_line":"                print(\"The indices are based on the current config file.\")"},{"line_number":354,"context_line":"            print(\"To view indices used by other Searchlight config \""}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_408059f7","line":351,"range":{"start_line":351,"start_character":23,"end_line":351,"end_character":27},"in_reply_to":"dada55a8_8988c7c6","updated":"2016-07-21 03:26:52.000000000","message":"Good find. Looking at the oslo_config code, I do not see a reliable way (for all circumstances) to get the config file name. Instead of parsing the command line and trying default values, we will just print a generic message about the \"current config file\". This should satisfy the vast majority of users.","commit_id":"5cfe31578a5156abd3363fe56d503ee338d948a3"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"c9ab8f28993e1baf774beb13621aabb760d66b48","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        else:"},{"line_number":451,"context_line":"            print(\"\\nList of Elasticsearch indices (and their associated\""},{"line_number":452,"context_line":"                  \" aliases) used by Searchlight.\\n\")"},{"line_number":453,"context_line":"            print(\"The indices are based on the config file.\")"},{"line_number":454,"context_line":"            print(\"To view indices used by other Searchlight config \""},{"line_number":455,"context_line":"                  \"files, use the --config-file option.\\n\")"},{"line_number":456,"context_line":"            print(\"Indices are denoted with a \u0027*\u0027\")"}],"source_content_type":"text/x-python","patch_set":9,"id":"dada55a8_7ad5e1b7","line":453,"range":{"start_line":453,"start_character":48,"end_line":453,"end_character":54},"updated":"2016-07-22 18:54:19.000000000","message":"Nit - \u0027current config file\u0027","commit_id":"558a6c567405dca19dae2d4ed5f001dc90c73a38"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"c9ab8f28993e1baf774beb13621aabb760d66b48","unresolved":false,"context_lines":[{"line_number":451,"context_line":"            print(\"\\nList of Elasticsearch indices (and their associated\""},{"line_number":452,"context_line":"                  \" aliases) used by Searchlight.\\n\")"},{"line_number":453,"context_line":"            print(\"The indices are based on the config file.\")"},{"line_number":454,"context_line":"            print(\"To view indices used by other Searchlight config \""},{"line_number":455,"context_line":"                  \"files, use the --config-file option.\\n\")"},{"line_number":456,"context_line":"            print(\"Indices are denoted with a \u0027*\u0027\")"},{"line_number":457,"context_line":"            print(\"Aliases are denoted with a \u0027+\u0027\\n\")"}],"source_content_type":"text/x-python","patch_set":9,"id":"dada55a8_1afca53d","line":454,"range":{"start_line":454,"start_character":40,"end_line":454,"end_character":42},"updated":"2016-07-22 18:54:19.000000000","message":"This line and the one above seem redundant, but not a huge deal.","commit_id":"558a6c567405dca19dae2d4ed5f001dc90c73a38"}],"searchlight/elasticsearch/plugins/utils.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"b6a657fb7d2e0d387e7eee71a05d7326c3713aaf","unresolved":false,"context_lines":[{"line_number":375,"context_line":"def get_indices(alias):"},{"line_number":376,"context_line":"    \"\"\"Return a list of indices associated with the specified alias."},{"line_number":377,"context_line":"    \"\"\""},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    es_engine \u003d searchlight.elasticsearch.get_api()"},{"line_number":380,"context_line":""},{"line_number":381,"context_line":"    return es_engine.indices.get_alias(name\u003dalias)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_c4a7494e","line":378,"updated":"2016-07-12 17:02:07.000000000","message":"Tiny nit, empty line\u0027s not necessary","commit_id":"1a625f71c51b76503ac52f054a5637b7d60393cf"}]}