)]}'
{"oslo_config/tests/test_get_location.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"3bcb8a32e1262f5b166ceb305584b4e9f4a784f6","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def setUp(self):"},{"line_number":49,"context_line":"        super(GetLocationTestCase, self).setUp()"},{"line_number":50,"context_line":"        self.useFixture(fixtures.NestedTempfile())"},{"line_number":51,"context_line":"        self.conf \u003d TestConfigOpts()"},{"line_number":52,"context_line":"        self.normal_opt \u003d cfg.StrOpt("},{"line_number":53,"context_line":"            \u0027normal_opt\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_8b568406","line":50,"updated":"2018-02-05 18:10:56.000000000","message":"Nit: This is already part of the oslotest base class: https://github.com/openstack/oslotest/blob/88e8edb93e409fe934f82a6cebdbd4fd27fa347f/oslotest/base.py#L111","commit_id":"33c2b286fd03ebeb9662f076a60747936892e730"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"5f53227be9859f2552a12bffe32a5863dde440f5","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def setUp(self):"},{"line_number":49,"context_line":"        super(GetLocationTestCase, self).setUp()"},{"line_number":50,"context_line":"        self.useFixture(fixtures.NestedTempfile())"},{"line_number":51,"context_line":"        self.conf \u003d TestConfigOpts()"},{"line_number":52,"context_line":"        self.normal_opt \u003d cfg.StrOpt("},{"line_number":53,"context_line":"            \u0027normal_opt\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_2956582b","line":50,"in_reply_to":"3fa0c359_8b568406","updated":"2018-02-05 19:28:11.000000000","message":"Done","commit_id":"33c2b286fd03ebeb9662f076a60747936892e730"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"3bcb8a32e1262f5b166ceb305584b4e9f4a784f6","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            default\u003d\u0027normal_opt_default\u0027,"},{"line_number":55,"context_line":"        )"},{"line_number":56,"context_line":"        self.conf.register_opt(self.normal_opt)"},{"line_number":57,"context_line":"        self.other_opt \u003d cfg.StrOpt("},{"line_number":58,"context_line":"            \u0027other_opt\u0027,"},{"line_number":59,"context_line":"            default\u003d\u0027other_opt_default\u0027,"},{"line_number":60,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_2b8e5078","line":57,"updated":"2018-02-05 18:10:56.000000000","message":"I see cli_opt is used in a follow up patch, but is this one ever used?","commit_id":"33c2b286fd03ebeb9662f076a60747936892e730"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"5f53227be9859f2552a12bffe32a5863dde440f5","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            default\u003d\u0027normal_opt_default\u0027,"},{"line_number":55,"context_line":"        )"},{"line_number":56,"context_line":"        self.conf.register_opt(self.normal_opt)"},{"line_number":57,"context_line":"        self.other_opt \u003d cfg.StrOpt("},{"line_number":58,"context_line":"            \u0027other_opt\u0027,"},{"line_number":59,"context_line":"            default\u003d\u0027other_opt_default\u0027,"},{"line_number":60,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_a9a3484f","line":57,"in_reply_to":"3fa0c359_2b8e5078","updated":"2018-02-05 19:28:11.000000000","message":"Done","commit_id":"33c2b286fd03ebeb9662f076a60747936892e730"}]}