)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"89624d57817b2c3d7e686ee9328eef396118da01","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Test nonexistent volume type extra_spec name instead of id"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"volume type extra_spec doesn\u0027t have \u0027id\u0027, so this is to"},{"line_number":10,"context_line":"rename test_get_nonexistent_extra_spec_id to"},{"line_number":11,"context_line":"test_get_nonexistent_extra_spec_name, testing nonexistent"},{"line_number":12,"context_line":"volume type extra_spec."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f436f4f_a939d53f","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":41},"updated":"2017-08-22 02:46:11.000000000","message":"it has actually - https://github.com/openstack/cinder/blob/7f69969345ec027595d2530d646cc5fd820be65a/cinder/db/sqlalchemy/models.py#L462","commit_id":"2cdb48c3669bf452b545fe4376d1046727ac6d1a"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"fc99dad16469bc9362f962b9773834750a2d240e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Test nonexistent volume type extra_spec name instead of id"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"volume type extra_spec doesn\u0027t have \u0027id\u0027, so this is to"},{"line_number":10,"context_line":"rename test_get_nonexistent_extra_spec_id to"},{"line_number":11,"context_line":"test_get_nonexistent_extra_spec_name, testing nonexistent"},{"line_number":12,"context_line":"volume type extra_spec."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f436f4f_2934c5fa","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":41},"in_reply_to":"9f436f4f_a939d53f","updated":"2017-08-22 03:02:39.000000000","message":"Thanks for the clarification, I didn\u0027t know that before:)","commit_id":"2cdb48c3669bf452b545fe4376d1046727ac6d1a"}],"tempest/api/volume/admin/test_volume_types_extra_specs_negative.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"89624d57817b2c3d7e686ee9328eef396118da01","unresolved":false,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"},{"line_number":130,"context_line":"    @decorators.idempotent_id(\u0027c881797d-12ff-4f1a-b09d-9f6212159753\u0027)"},{"line_number":131,"context_line":"    def test_get_nonexistent_extra_spec_name(self):"},{"line_number":132,"context_line":"        # Should not get volume type extra spec for nonexistent extra spec"},{"line_number":133,"context_line":"        # name."},{"line_number":134,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":1,"id":"9f436f4f_894851cc","line":131,"range":{"start_line":131,"start_character":8,"end_line":131,"end_character":51},"updated":"2017-08-22 02:46:11.000000000","message":"Yea, it only work with spec key and API implementation looks like taking \u0027id\u0027 but cinder internally search with key only :)- https://github.com/openstack/cinder/blob/4b9bc833d19f01bf102e6b469af72e1ce7c06cca/cinder/api/contrib/types_extra_specs.py#L147\n\nI am ok to rename it as defore does not use it.","commit_id":"2cdb48c3669bf452b545fe4376d1046727ac6d1a"}]}