)]}'
{"tempest/api/volume/test_volumes_actions.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"416889cfa164def03d2bce8b9b220051e841c477","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    @test.idempotent_id(\u002763e21b4c-0a0c-41f6-bfc3-7c2816815599\u0027)"},{"line_number":73,"context_line":"    def test_volume_bootable(self):"},{"line_number":74,"context_line":"        # Verify that a volume bootable flag is retrieved"},{"line_number":75,"context_line":"        if not CONF.volume_feature_enabled.bootable:"},{"line_number":76,"context_line":"            raise self.skipException(\"Update bootable status of a volume\""},{"line_number":77,"context_line":"                                     \" is not enabled\")"},{"line_number":78,"context_line":"        for bool_bootable in [True, False]:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a0c5dc1_582672bb","line":75,"range":{"start_line":75,"start_character":14,"end_line":75,"end_character":16},"updated":"2015-05-12 14:16:29.000000000","message":"You might want to use the \u0027@testtools.skipUnless\u0027 decorator.","commit_id":"cf2779a3097245b49c515dac9af22ac0d6ad504c"}],"tempest/config.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"416889cfa164def03d2bce8b9b220051e841c477","unresolved":false,"context_lines":[{"line_number":657,"context_line":"    cfg.BoolOpt(\u0027bootable\u0027,"},{"line_number":658,"context_line":"                default\u003dFalse,"},{"line_number":659,"context_line":"                help\u003d\u0027Update bootable status of a volume \u0027"},{"line_number":660,"context_line":"                     \u0027Not implemented on ice-house \u0027)"},{"line_number":661,"context_line":"]"},{"line_number":662,"context_line":""},{"line_number":663,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a0c5dc1_e5b0e38d","line":660,"updated":"2015-05-12 14:16:29.000000000","message":"Nit: you should spell it \u0027Icehouse\u0027.","commit_id":"cf2779a3097245b49c515dac9af22ac0d6ad504c"}],"tempest/services/volume/json/volumes_client.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"9a33fac5e741fded65fd843e03c1fc16f25b11bf","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2012 OpenStack Foundation"},{"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"},{"line_number":5,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":6,"context_line":"#    a copy of the License at"},{"line_number":7,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":1,"id":"9af37de9_a4ad4e33","line":4,"updated":"2015-05-11 09:11:38.000000000","message":"Unexpected white space change.","commit_id":"1f476fec6b8399cc85b94dbdf61c0870c56cb18c"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"416889cfa164def03d2bce8b9b220051e841c477","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        resp, body \u003d self.post(url, post_body)"},{"line_number":123,"context_line":"        self.expected_success(202, resp.status)"},{"line_number":124,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    def bootable_volume(self, volume_id, bootable):"},{"line_number":127,"context_line":"        \"\"\"set a bootable flag for a volume - true or false.\"\"\""},{"line_number":128,"context_line":"        post_body \u003d {\"bootable\": bootable}"},{"line_number":129,"context_line":"        post_body \u003d json.dumps({\u0027os-set_bootable\u0027: post_body})"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a0c5dc1_c5227fb9","line":126,"range":{"start_line":125,"start_character":0,"end_line":126,"end_character":18},"updated":"2015-05-12 14:16:29.000000000","message":"The method name is not optimal. I suggest \u0027set_bootable_flag\u0027.","commit_id":"cf2779a3097245b49c515dac9af22ac0d6ad504c"}]}