)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"6d50ce3f1fad03b3b92901c5530503022d1b8634","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In \u0027setUpClass\u0027 method config groups were referenced"},{"line_number":10,"context_line":"before initialization using still empty oslo.config"},{"line_number":11,"context_line":"which is empty if use without other cli test suites."},{"line_number":12,"context_line":"Fixed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4f0109617bf98bbb375cfcf816c074762b9fc69a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAUH%2F%2F8l0%3D","line":11,"updated":"2014-01-06 11:40:07.000000000","message":"nit: if used","commit_id":"aa86a800b6b374ab74f66fa48b360d69b23629bd"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"6d50ce3f1fad03b3b92901c5530503022d1b8634","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In \u0027setUpClass\u0027 method config groups were referenced"},{"line_number":10,"context_line":"before initialization using still empty oslo.config"},{"line_number":11,"context_line":"which is empty if use without other cli test suites."},{"line_number":12,"context_line":"Fixed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4f0109617bf98bbb375cfcf816c074762b9fc69a"},{"line_number":15,"context_line":"Closes-Bug: #1265909"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAUH%2F%2F8lw%3D","line":12,"updated":"2014-01-06 11:40:07.000000000","message":"not really needed :)","commit_id":"aa86a800b6b374ab74f66fa48b360d69b23629bd"}],"tempest/cli/simple_read_only/test_ceilometer.py":[{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"c0151a96d91f049dbafb81eb95d560f9798e2d4f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from oslo.config import cfg"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import tempest.cli"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2Fns%3D","line":17,"updated":"2014-01-04 11:45:55.000000000","message":"Can we use \"from tempest import config\" instead?","commit_id":"c44111fec3653b86d1cffc82880c0a2dbae7a658"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"982f6e2168527986e7f4f5dfc381a2e38de92492","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from oslo.config import cfg"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import tempest.cli"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2Fl4%3D","line":17,"in_reply_to":"AAAAUH%2F%2F%2Fns%3D","updated":"2014-01-04 12:43:09.000000000","message":"Yes, you are right, it should be so, if import here, but it is already used in setUpClass method of inherited class.\n\nDone, look at patch-set 2","commit_id":"c44111fec3653b86d1cffc82880c0a2dbae7a658"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"c0151a96d91f049dbafb81eb95d560f9798e2d4f","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import tempest.cli"},{"line_number":21,"context_line":"from tempest.openstack.common import log as logging"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"CONF \u003d cfg.CONF"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2Fng%3D","line":22,"updated":"2014-01-04 11:45:55.000000000","message":"then we can use CONF \u003d config.CONF , here. The CONG will be initialized.","commit_id":"c44111fec3653b86d1cffc82880c0a2dbae7a658"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"982f6e2168527986e7f4f5dfc381a2e38de92492","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import tempest.cli"},{"line_number":21,"context_line":"from tempest.openstack.common import log as logging"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"CONF \u003d cfg.CONF"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2Fl0%3D","line":22,"in_reply_to":"AAAAUH%2F%2F%2Fng%3D","updated":"2014-01-04 12:43:09.000000000","message":"Same as above, it is redundant.\n\nRemoved.","commit_id":"c44111fec3653b86d1cffc82880c0a2dbae7a658"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"c556515767c75123094fea5705175d0ea34bede1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(SimpleReadOnlyCeilometerClientTest, cls).setUpClass()"},{"line_number":35,"context_line":"        if (not cls.config.service_available.ceilometer):"},{"line_number":36,"context_line":"            msg \u003d (\"Skiping all Ceilometer cli tests because it is\""},{"line_number":37,"context_line":"                   \"not available\")"},{"line_number":38,"context_line":"            raise cls.skipException(msg)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F90M%3D","line":35,"updated":"2014-01-06 05:15:39.000000000","message":"\"CONF.service_available.neutron\" is right here. We shouldn\u0027t use cls.config anymore. See: http://www.mail-archive.com/openstack-dev@lists.openstack.org/msg12412.html","commit_id":"e5c07d8226e5414dee61c9ab2b018575c85a2272"}],"tempest/cli/simple_read_only/test_neutron.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"c556515767c75123094fea5705175d0ea34bede1","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def setUpClass(cls):"},{"line_number":38,"context_line":"        super(SimpleReadOnlyNeutronClientTest, cls).setUpClass()"},{"line_number":39,"context_line":"        if (not cls.config.service_available.neutron):"},{"line_number":40,"context_line":"            msg \u003d \"Skiping all Neutron cli tests because it is not available\""},{"line_number":41,"context_line":"            raise cls.skipException(msg)"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F90Y%3D","line":39,"updated":"2014-01-06 05:15:39.000000000","message":"same comment in test_ceilometer.py","commit_id":"e5c07d8226e5414dee61c9ab2b018575c85a2272"}]}