)]}'
{"tempest/scenario/test_volume_boot_pattern.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"dbbedc9982df7a2892592c086d61faa3c405099a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"from oslo_log import log as logging"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_bbde333d","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":0},"updated":"2016-11-09 00:14:43.000000000","message":"nit: unnecessary additional space.","commit_id":"22f0ff3b905a25ac1ddb913b697fbb8f6aa772b9"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"c3c190c98acfba785195bf614ccda094a60c8474","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"from oslo_log import log as logging"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_35ff8c35","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":0},"in_reply_to":"9a629dbe_bbde333d","updated":"2016-11-09 04:45:26.000000000","message":"will fix it","commit_id":"22f0ff3b905a25ac1ddb913b697fbb8f6aa772b9"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"dbbedc9982df7a2892592c086d61faa3c405099a","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest import config"},{"line_number":17,"context_line":"from tempest import test"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest.common import waiters"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from tempest.common.utils import data_utils"},{"line_number":22,"context_line":"from tempest.scenario import manager"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import testtools"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_1b79df2f","line":22,"range":{"start_line":18,"start_character":0,"end_line":22,"end_character":36},"updated":"2016-11-09 00:14:43.000000000","message":"This change seems unnecessary at all. This patch seems necessary only to add \"import testtools\" and a check for the test.","commit_id":"22f0ff3b905a25ac1ddb913b697fbb8f6aa772b9"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"c3c190c98acfba785195bf614ccda094a60c8474","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest import config"},{"line_number":17,"context_line":"from tempest import test"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest.common import waiters"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from tempest.common.utils import data_utils"},{"line_number":22,"context_line":"from tempest.scenario import manager"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import testtools"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_75f9044d","line":22,"range":{"start_line":18,"start_character":0,"end_line":22,"end_character":36},"in_reply_to":"9a629dbe_1b79df2f","updated":"2016-11-09 04:45:26.000000000","message":"will fix it","commit_id":"22f0ff3b905a25ac1ddb913b697fbb8f6aa772b9"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"dbbedc9982df7a2892592c086d61faa3c405099a","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from tempest.common.utils import data_utils"},{"line_number":22,"context_line":"from tempest.scenario import manager"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import testtools"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"CONF \u003d config.CONF"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_5bc51759","line":24,"updated":"2016-11-09 00:14:43.000000000","message":"testtools module should be put after oslo_log because of 3rd party module.","commit_id":"22f0ff3b905a25ac1ddb913b697fbb8f6aa772b9"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8c5f47b1fa9efe06a590fe512483e563347628db","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_log import log as logging"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from tempest.common.utils import data_utils"},{"line_number":16,"context_line":"from tempest.common import waiters"},{"line_number":17,"context_line":"from tempest import config"},{"line_number":18,"context_line":"from tempest.scenario import manager"},{"line_number":19,"context_line":"from tempest import test"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"CONF \u003d config.CONF"},{"line_number":22,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_6775d996","side":"PARENT","line":19,"range":{"start_line":15,"start_character":0,"end_line":19,"end_character":24},"updated":"2016-11-09 06:36:07.000000000","message":"Please please don\u0027t change the order of line15-19.","commit_id":"cb640f515514fdb7633abe109b7f064b81b856bd"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d3e0da7287e6149cdabfa4f8beed9ac394501e42","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_log import log as logging"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import testtools"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_4731b5eb","line":14,"updated":"2016-11-09 06:35:02.000000000","message":"a space is not necessary between line 13 and 15.. because both are the same 3rd party lib.","commit_id":"397f9fe401d2d83ac9e8424fb1c1e72e4a604fb9"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"db6e9ae48ecb6533ee3600f8d7cbc6764c2369f4","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    @test.idempotent_id(\u0027557cd2c2-4eb8-4dce-98be-f86765ff311b\u0027)"},{"line_number":96,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":97,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027, \u0027network\u0027)"},{"line_number":98,"context_line":"    def test_volume_boot_pattern(self):"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        \"\"\"This test case attempts to reproduce the following steps:"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a629dbe_0f813d8f","line":97,"range":{"start_line":97,"start_character":4,"end_line":97,"end_character":59},"updated":"2016-11-10 18:50:07.000000000","message":"This service list means which components are used in each test.\nThat is the list of used service APIs like Nova, Cinder, Glance.\nIn this case, Neutron API is not used and nova-net env also can pass through this test. So this \u0027network\u0027 seems unnecessary IIUC.","commit_id":"5c4f5a5ec8c7bba98afbbc186b7ed3ebcfe53020"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f1841709eb84767d9b4a2f4e0248cc78f1f11ffb","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    @test.idempotent_id(\u0027557cd2c2-4eb8-4dce-98be-f86765ff311b\u0027)"},{"line_number":96,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":97,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027, \u0027network\u0027)"},{"line_number":98,"context_line":"    def test_volume_boot_pattern(self):"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        \"\"\"This test case attempts to reproduce the following steps:"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a629dbe_66f712d3","line":97,"range":{"start_line":97,"start_character":4,"end_line":97,"end_character":59},"in_reply_to":"9a629dbe_0f813d8f","updated":"2016-11-11 22:43:51.000000000","message":"Yea, this can be done using nova-net as of now so neutron is not actually needed as mandatory.","commit_id":"5c4f5a5ec8c7bba98afbbc186b7ed3ebcfe53020"}]}