)]}'
{"keystone_tempest_plugin/README.rst":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":1,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Tempest Integration of keystone"},{"line_number":3,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba0121b8_50b6028b","line":1,"updated":"2016-03-30 21:27:10.000000000","message":"Can we change the directory to tempest_plugin? There\u0027s no need to have keystone in the directory name since that\u0027s the project.\n\nAlternatively, just call the directory tempest, or tempest/plugin. No need to use _ to separate parts since / is available.","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"534556765bbae1496ebbfcdb02ae00d9a0adbed5","unresolved":false,"context_lines":[{"line_number":1,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Tempest Integration of keystone"},{"line_number":3,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba0121b8_f005f210","line":1,"in_reply_to":"ba0121b8_50b6028b","updated":"2016-03-31 13:58:51.000000000","message":"makes sense, but {project_name}_tempest_plugin/tests is the \"pattern\" being used across projects [1] [2]. I guess makes sense to use the same pattern?\n\n[1] https://github.com/openstack/ironic/tree/master/ironic_tempest_plugin\n[2] https://github.com/openstack/manila/tree/master/manila_tempest_tests","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":1,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Tempest Integration of keystone"},{"line_number":3,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"This directory contains Tempest tests to cover the keystone project."}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba0121b8_d0817291","line":2,"range":{"start_line":2,"start_character":20,"end_line":2,"end_character":22},"updated":"2016-03-30 21:27:10.000000000","message":"change to Tests for Keystone","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"534556765bbae1496ebbfcdb02ae00d9a0adbed5","unresolved":false,"context_lines":[{"line_number":1,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Tempest Integration of keystone"},{"line_number":3,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"This directory contains Tempest tests to cover the keystone project."}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba0121b8_300c8a3a","line":2,"range":{"start_line":2,"start_character":20,"end_line":2,"end_character":22},"in_reply_to":"ba0121b8_d0817291","updated":"2016-03-31 13:58:51.000000000","message":"Done","commit_id":"96e789493d4a9890562a527595421f4085c1169f"}],"keystone_tempest_plugin/config.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from oslo_config import cfg"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest import config  # noqa"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"identity_group \u003d cfg.OptGroup(name\u003d\u0027identity\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_702226bb","line":19,"updated":"2016-03-30 21:27:10.000000000","message":"why noqa here? If it\u0027s not used just remove it.","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"534556765bbae1496ebbfcdb02ae00d9a0adbed5","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from oslo_config import cfg"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest import config  # noqa"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"identity_group \u003d cfg.OptGroup(name\u003d\u0027identity\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_b046da4c","line":19,"in_reply_to":"ba0121b8_702226bb","updated":"2016-03-31 13:58:51.000000000","message":"Done","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":23,"context_line":"                              title\u003d\"Keystone Configuration Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"IdentityGroup \u003d ["},{"line_number":26,"context_line":"    cfg.StrOpt(\u0027catalog_type\u0027,"},{"line_number":27,"context_line":"               default\u003d\u0027identity\u0027,"},{"line_number":28,"context_line":"               help\u003d\"Catalog type of the Identity service.\"),"},{"line_number":29,"context_line":"    cfg.BoolOpt(\u0027disable_ssl_certificate_validation\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_b0031e0f","line":26,"updated":"2016-03-30 21:27:10.000000000","message":"I don\u0027t think we should be adding config options until we need them.","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"534556765bbae1496ebbfcdb02ae00d9a0adbed5","unresolved":false,"context_lines":[{"line_number":23,"context_line":"                              title\u003d\"Keystone Configuration Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"IdentityGroup \u003d ["},{"line_number":26,"context_line":"    cfg.StrOpt(\u0027catalog_type\u0027,"},{"line_number":27,"context_line":"               default\u003d\u0027identity\u0027,"},{"line_number":28,"context_line":"               help\u003d\"Catalog type of the Identity service.\"),"},{"line_number":29,"context_line":"    cfg.BoolOpt(\u0027disable_ssl_certificate_validation\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_b0b41a35","line":26,"in_reply_to":"ba0121b8_b0031e0f","updated":"2016-03-31 13:58:51.000000000","message":"Done","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":118,"context_line":"                help\u003d\"A list of enabled identity extensions with a special \""},{"line_number":119,"context_line":"                     \"entry all which indicates every extension is enabled. \""},{"line_number":120,"context_line":"                     \"Empty list indicates all extensions are disabled. \""},{"line_number":121,"context_line":"                     \"To get the list of extensions run: \u0027keystone discover\u0027\")"},{"line_number":122,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_50128263","line":121,"updated":"2016-03-30 21:27:10.000000000","message":"What is this oddity? keystone is going to removed soon so this isn\u0027t going to work.","commit_id":"96e789493d4a9890562a527595421f4085c1169f"}],"keystone_tempest_plugin/plugin.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"696a1401a0ea9fbf1e6caa1c4adbaf1189c98030","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone_tempest_plugin import config as project_config"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class KeystoneTempestPlugin(plugins.TempestPlugin):"},{"line_number":26,"context_line":"    def load_tests(self):"},{"line_number":27,"context_line":"        base_path \u003d os.path.split(os.path.dirname("},{"line_number":28,"context_line":"            os.path.abspath(__file__)))[0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_f03596f7","line":25,"updated":"2016-03-30 21:27:10.000000000","message":"suggest remove \"Keystone\" -- what else would it be for?","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"534556765bbae1496ebbfcdb02ae00d9a0adbed5","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone_tempest_plugin import config as project_config"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class KeystoneTempestPlugin(plugins.TempestPlugin):"},{"line_number":26,"context_line":"    def load_tests(self):"},{"line_number":27,"context_line":"        base_path \u003d os.path.split(os.path.dirname("},{"line_number":28,"context_line":"            os.path.abspath(__file__)))[0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_50977e39","line":25,"in_reply_to":"ba0121b8_f03596f7","updated":"2016-03-31 13:58:51.000000000","message":"It inherits from TempestPlugin, seems confusing to have a subclass with the same name.","commit_id":"96e789493d4a9890562a527595421f4085c1169f"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"32dc7006ff0acd47f1cdf23f330a195150002e66","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2016 Red Hat, Inc."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba0121b8_9eca7d19","line":1,"updated":"2016-04-01 19:36:06.000000000","message":"\"keystone\" should be removed from the directory name. It\u0027s already in keystone so that\u0027s unnecessary.","commit_id":"088393d5216b49f909abcb1758eb4d3ebd88c980"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"32dc7006ff0acd47f1cdf23f330a195150002e66","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone_tempest_plugin import config as project_config"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class KeystoneTempestPlugin(plugins.TempestPlugin):"},{"line_number":26,"context_line":"    def load_tests(self):"},{"line_number":27,"context_line":"        base_path \u003d os.path.split(os.path.dirname("},{"line_number":28,"context_line":"            os.path.abspath(__file__)))[0]"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba0121b8_de5d157b","line":25,"updated":"2016-04-01 19:36:06.000000000","message":"remove Keystone since this is in keystone already.","commit_id":"088393d5216b49f909abcb1758eb4d3ebd88c980"}]}