)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"e449bfd94cb2047ace2c950fbc1ce8808ade8071","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"537b1c73_64440ba3","updated":"2022-03-04 16:01:24.000000000","message":"I actually prefer the original code. I realize Red Hat and Debian manage their glance packaging differently, but I find it a little more difficult to see the big picture when the spec tests for each os is in a different file (mainly because the files themselves don\u0027t appear to be directly associated with a specific os).\n\n+1 for now, but I\u0027m happy to go along with this change if others feel positive about it.","commit_id":"5dd2f27a7e2d4ce4a2eea7124400d91705284dcc"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fe7e9584073c95d265f48c6b68982f3ef69465e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cd950ab7_ce48969b","updated":"2022-03-11 12:14:18.000000000","message":"recheck","commit_id":"5dd2f27a7e2d4ce4a2eea7124400d91705284dcc"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"786f701f7f6b0ef7e005a29a3c1455f2b3765867","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2d9d8723_7c903db6","in_reply_to":"537b1c73_64440ba3","updated":"2022-03-04 16:27:18.000000000","message":"That\u0027s a fair point. Actually this is not a complete improvement but a kind of pros-cons thin, and I\u0027d admit this might look overkilling considering how simple the base glance class is.\n\nWhile we\u0027ll wait for additional feedback from the others, let me write down some motivations to make this cahnge.\n\n- The corrent code doesn\u0027t allow us to test the parameter in the base glance class.\n\n- This requires additional consideration that any change in the base glance class can affect the spec file for glance::api, which we don\u0027t usually think are linked.","commit_id":"5dd2f27a7e2d4ce4a2eea7124400d91705284dcc"}]}