)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"36dd1092e2331c99fbe3ea048cb69db2a05907ea","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add basic tests for OSC plugin baremetal chassis commands"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add smoke tests for commands:"},{"line_number":10,"context_line":"openstack baremetal chassis delete,"},{"line_number":11,"context_line":"openstack baremetal chassis list,"},{"line_number":12,"context_line":"openstack baremetal chassis show,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_0c8caed8","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":29},"updated":"2016-11-21 03:10:56.000000000","message":"removing \"openstack baremetal chassis create,\" is not good.\n\nhttps://review.openstack.org/#/c/366158/2..5//COMMIT_MSG\n\nI think we can update \"def test_create(self):\" which creates another chassis.\n\nhttps://review.openstack.org/#/c/366158/2..5/ironicclient/tests/functional/osc/v1/test_baremetal_chassis_basic.py@50","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"}],"ironicclient/tests/functional/osc/v1/base.py":[{"author":{"_account_id":20522,"name":"Aline Bousquet","email":"albousque@gmail.com","username":"alineb"},"change_message_id":"ad58bbb30e6f9d78abd25aa186b7d86a22d183ef","unresolved":false,"context_lines":[{"line_number":208,"context_line":"        :return: JSON object of chassis"},{"line_number":209,"context_line":"        \"\"\""},{"line_number":210,"context_line":"        opts \u003d self.get_opts(fields)"},{"line_number":211,"context_line":"        chassis \u003d self.openstack(\u0027baremetal chassis show {0} {1}\u0027"},{"line_number":212,"context_line":"                                 .format(opts, uuid, params))"},{"line_number":213,"context_line":"        return json.loads(chassis)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_6f34e3dd","line":211,"updated":"2016-11-22 15:27:46.000000000","message":"Missing \u0027{2}\u0027 here to pass \u0027params\u0027 to the command","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c6d31e66f85818b607ec56b27752102b654858fe","unresolved":false,"context_lines":[{"line_number":208,"context_line":"        :return: JSON object of chassis"},{"line_number":209,"context_line":"        \"\"\""},{"line_number":210,"context_line":"        opts \u003d self.get_opts(fields)"},{"line_number":211,"context_line":"        chassis \u003d self.openstack(\u0027baremetal chassis show {0} {1}\u0027"},{"line_number":212,"context_line":"                                 .format(opts, uuid, params))"},{"line_number":213,"context_line":"        return json.loads(chassis)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_9be8a118","line":211,"in_reply_to":"7a77a97e_6f34e3dd","updated":"2016-11-22 16:25:20.000000000","message":"Thanks for catch!","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"39d7738a72bd3c59a63409090897e53e834c653a","unresolved":false,"context_lines":[{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        :param String uuid: UUID of the chassis"},{"line_number":180,"context_line":"        :param Bool ignore_exceptions: Ignore exception (needed for cleanUp)"},{"line_number":181,"context_line":"        :return: raw values output"},{"line_number":182,"context_line":"        \"\"\""},{"line_number":183,"context_line":"        try:"},{"line_number":184,"context_line":"            return self.openstack(\u0027baremetal chassis delete {0}\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a71b18c_529e6a07","line":181,"updated":"2016-12-06 15:34:28.000000000","message":"Nit: Documentation should explicitly call out possible exceptions raised, although I see that other documentation in this file does not follow this advice. In an ideal world it would.","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"8dfc7076018aa84830014f91f05805bac042387e","unresolved":false,"context_lines":[{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        :param String uuid: UUID of the chassis"},{"line_number":180,"context_line":"        :param Bool ignore_exceptions: Ignore exception (needed for cleanUp)"},{"line_number":181,"context_line":"        :return: raw values output"},{"line_number":182,"context_line":"        \"\"\""},{"line_number":183,"context_line":"        try:"},{"line_number":184,"context_line":"            return self.openstack(\u0027baremetal chassis delete {0}\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_3e77973b","line":181,"in_reply_to":"3a71b18c_529e6a07","updated":"2016-12-13 14:43:47.000000000","message":"Done","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b08b8202000043fd2e08786bfa135fcbbd89ad48","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                                 .format(opts, params))"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"        chassis \u003d json.loads(chassis)"},{"line_number":242,"context_line":"        self.addCleanup(self.chassis_delete, chassis[\u0027uuid\u0027], True)"},{"line_number":243,"context_line":"        if not chassis:"},{"line_number":244,"context_line":"            self.fail(\u0027Baremetal chassis has not been created!\u0027)"},{"line_number":245,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"df140735_cf60dd35","line":242,"updated":"2017-05-30 07:15:27.000000000","message":"this should go after the check that chassis is not empty","commit_id":"fe583fde0553405faed7e3ff82844eb3655594fa"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"d5d8dac3331ddb640e2defa11e57649a1de25644","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                                 .format(opts, params))"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"        chassis \u003d json.loads(chassis)"},{"line_number":242,"context_line":"        self.addCleanup(self.chassis_delete, chassis[\u0027uuid\u0027], True)"},{"line_number":243,"context_line":"        if not chassis:"},{"line_number":244,"context_line":"            self.fail(\u0027Baremetal chassis has not been created!\u0027)"},{"line_number":245,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"df140735_5403f80f","line":242,"in_reply_to":"df140735_cf60dd35","updated":"2017-06-01 12:00:28.000000000","message":"Done","commit_id":"fe583fde0553405faed7e3ff82844eb3655594fa"}],"ironicclient/tests/functional/osc/v1/test_baremetal_chassis_basic.py":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e50a6349a9dcbdf3e885f5de7ce4877470b916f3","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        1) Create baremetal chassis in setUp."},{"line_number":52,"context_line":"        3) Check that chassis successfully created."},{"line_number":53,"context_line":"        \"\"\""},{"line_number":54,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], self.chassis_list([\u0027UUID\u0027]))"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    def test_delete(self):"},{"line_number":57,"context_line":"        \"\"\"Check baremetal chassis delete command."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_bf421cac","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":72},"updated":"2016-09-27 07:28:52.000000000","message":"nit: it would be redundant, because it is almost identical with test_list() at L.25.","commit_id":"4e388f499ab9b62af0491afdbb21815439e5e5a9"},{"author":{"_account_id":20522,"name":"Aline Bousquet","email":"albousque@gmail.com","username":"alineb"},"change_message_id":"ad58bbb30e6f9d78abd25aa186b7d86a22d183ef","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        Test steps:"},{"line_number":43,"context_line":"        1) Create baremetal chassis in setUp."},{"line_number":44,"context_line":"        2) Show baremetal chassis."},{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_4f3ac735","line":45,"updated":"2016-11-22 15:27:46.000000000","message":"nit: s/list/show ?","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c6d31e66f85818b607ec56b27752102b654858fe","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        Test steps:"},{"line_number":43,"context_line":"        1) Create baremetal chassis in setUp."},{"line_number":44,"context_line":"        2) Show baremetal chassis."},{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_bb847d5d","line":45,"in_reply_to":"7a77a97e_4f3ac735","updated":"2016-11-22 16:25:20.000000000","message":"Done","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":20522,"name":"Aline Bousquet","email":"albousque@gmail.com","username":"alineb"},"change_message_id":"ad58bbb30e6f9d78abd25aa186b7d86a22d183ef","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_delete(self):"},{"line_number":51,"context_line":"        \"\"\"Check baremetal chassis delete command."}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_2c13d95d","line":48,"range":{"start_line":48,"start_character":13,"end_line":48,"end_character":21},"updated":"2016-11-22 15:27:46.000000000","message":"Wouldn\u0027t it be better to use assertEqual() here?","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"c6d31e66f85818b607ec56b27752102b654858fe","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_delete(self):"},{"line_number":51,"context_line":"        \"\"\"Check baremetal chassis delete command."}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_fb7ef568","line":48,"range":{"start_line":48,"start_character":13,"end_line":48,"end_character":21},"in_reply_to":"7a77a97e_2c13d95d","updated":"2016-11-22 16:25:20.000000000","message":"Thanks!","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"36dd1092e2331c99fbe3ea048cb69db2a05907ea","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_delete(self):"},{"line_number":51,"context_line":"        \"\"\"Check baremetal chassis delete command."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        Test steps:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_8cbefe62","line":50,"range":{"start_line":48,"start_character":8,"end_line":50,"end_character":26},"updated":"2016-11-21 03:10:56.000000000","message":"This is a comment of removing \"def test_create(self):\".\n\nhttps://review.openstack.org/#/c/366158/2..5/ironicclient/tests/functional/osc/v1/test_baremetal_chassis_basic.py\n\nI think we can avoid the redundancy with \"test_list()\" if \"test_create(self)\" created another chassis and assert there are two chassis.\n\nWDYT?","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"7321c28446cda549e0de7269fa95daa92517e94b","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_delete(self):"},{"line_number":51,"context_line":"        \"\"\"Check baremetal chassis delete command."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        Test steps:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_eb2ac72b","line":50,"range":{"start_line":48,"start_character":8,"end_line":50,"end_character":26},"in_reply_to":"7a77a97e_44710ce0","updated":"2016-11-22 04:19:21.000000000","message":"Kyrylo,\n\nOh, great! thanks :-)\n\nNaohiro","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"5357df1f4d13bfaca239d0096e5fdd993be900cc","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        3) Check chassis in chassis list."},{"line_number":46,"context_line":"        \"\"\""},{"line_number":47,"context_line":"        chassis \u003d self.chassis_show(self.chassis[\u0027uuid\u0027])"},{"line_number":48,"context_line":"        self.assertIn(self.chassis[\u0027uuid\u0027], chassis[\u0027uuid\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_delete(self):"},{"line_number":51,"context_line":"        \"\"\"Check baremetal chassis delete command."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        Test steps:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_44710ce0","line":50,"range":{"start_line":48,"start_character":8,"end_line":50,"end_character":26},"in_reply_to":"7a77a97e_8cbefe62","updated":"2016-11-21 13:20:31.000000000","message":"Hello, Naohiro! Here i have a patch for `chassis create` subcommand with options: https://review.openstack.org/#/c/382496/","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":20522,"name":"Aline Bousquet","email":"albousque@gmail.com","username":"alineb"},"change_message_id":"ad58bbb30e6f9d78abd25aa186b7d86a22d183ef","unresolved":false,"context_lines":[{"line_number":76,"context_line":"                       .format(extra_key, extra_value, self.chassis[\u0027uuid\u0027]))"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        show_prop \u003d self.chassis_show(self.chassis[\u0027uuid\u0027], [\u0027extra\u0027])"},{"line_number":79,"context_line":"        self.assertIn(extra_value, show_prop[\u0027extra\u0027][extra_key])"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        self.openstack(\u0027baremetal chassis unset --extra {0} {1}\u0027"},{"line_number":82,"context_line":"                       .format(extra_key, self.chassis[\u0027uuid\u0027]))"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_cfeb3711","line":79,"range":{"start_line":79,"start_character":13,"end_line":79,"end_character":21},"updated":"2016-11-22 15:27:46.000000000","message":"Maybe use assertEqual() here too?","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"df108176c077813ee06d6ebb45726a197d26e00e","unresolved":false,"context_lines":[{"line_number":76,"context_line":"                       .format(extra_key, extra_value, self.chassis[\u0027uuid\u0027]))"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        show_prop \u003d self.chassis_show(self.chassis[\u0027uuid\u0027], [\u0027extra\u0027])"},{"line_number":79,"context_line":"        self.assertIn(extra_value, show_prop[\u0027extra\u0027][extra_key])"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        self.openstack(\u0027baremetal chassis unset --extra {0} {1}\u0027"},{"line_number":82,"context_line":"                       .format(extra_key, self.chassis[\u0027uuid\u0027]))"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_7b72e512","line":79,"range":{"start_line":79,"start_character":13,"end_line":79,"end_character":21},"in_reply_to":"7a77a97e_cfeb3711","updated":"2016-11-22 16:33:21.000000000","message":"Done","commit_id":"4a751f0fdea0692aaf20778b66dd5ff11266f773"},{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"39d7738a72bd3c59a63409090897e53e834c653a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"                      [x[\u0027Description\u0027] for x in chassis_list])"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_show(self):"},{"line_number":40,"context_line":"        \"\"\"Check baremetal chassis list command."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        Test steps:"},{"line_number":43,"context_line":"        1) Create baremetal chassis in setUp."}],"source_content_type":"text/x-python","patch_set":6,"id":"3a71b18c_32c2cee9","line":40,"updated":"2016-12-06 15:34:28.000000000","message":"s/list/show/","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"8dfc7076018aa84830014f91f05805bac042387e","unresolved":false,"context_lines":[{"line_number":37,"context_line":"                      [x[\u0027Description\u0027] for x in chassis_list])"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_show(self):"},{"line_number":40,"context_line":"        \"\"\"Check baremetal chassis list command."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        Test steps:"},{"line_number":43,"context_line":"        1) Create baremetal chassis in setUp."}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_de9ea355","line":40,"in_reply_to":"3a71b18c_32c2cee9","updated":"2016-12-13 14:43:47.000000000","message":"Done","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"39d7738a72bd3c59a63409090897e53e834c653a","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                       .format(extra_key, self.chassis[\u0027uuid\u0027]))"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        show_prop \u003d self.chassis_show(self.chassis[\u0027uuid\u0027], [\u0027extra\u0027])"},{"line_number":86,"context_line":"        self.assertIsNot(extra_key, show_prop[\u0027extra\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a71b18c_d22efaa3","line":86,"updated":"2016-12-06 15:34:28.000000000","message":"What exactly should show_prop[\u0027extra\u0027] be at this point? None? I think self.assertIsNone(show_prop[\u0027extra\u0027]) is preferable if that\u0027s the case.","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"8dfc7076018aa84830014f91f05805bac042387e","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                       .format(extra_key, self.chassis[\u0027uuid\u0027]))"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        show_prop \u003d self.chassis_show(self.chassis[\u0027uuid\u0027], [\u0027extra\u0027])"},{"line_number":86,"context_line":"        self.assertIsNot(extra_key, show_prop[\u0027extra\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_1ec81b55","line":86,"in_reply_to":"3a71b18c_d22efaa3","updated":"2016-12-13 14:43:47.000000000","message":"Thanks, Clif!","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"0585631cb5e221c5ddae0fadf36bf10073fa5642","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                       .format(extra_key, self.chassis[\u0027uuid\u0027]))"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        show_prop \u003d self.chassis_show(self.chassis[\u0027uuid\u0027], [\u0027extra\u0027])"},{"line_number":86,"context_line":"        self.assertIsNot(extra_key, show_prop[\u0027extra\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_88e230d4","line":86,"in_reply_to":"3a71b18c_d22efaa3","updated":"2016-12-13 15:32:39.000000000","message":"oh, this became a mismatch testtools.matchers._impl.MismatchError: {} is not None\nbecause empty dictionary is not none :)","commit_id":"8942eda50ff325320619a9edcea7e2f4634f3e7a"}]}