)]}'
{"config_tempest/services/orchestration.py":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"060bedda02a6f5cfcd49ed470e9382e39dd724de","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        if \u0027v3\u0027 not in uri:"},{"line_number":32,"context_line":"            return"},{"line_number":33,"context_line":"        sec \u003d \u0027heat_plugin\u0027"},{"line_number":34,"context_line":"        conf.set(sec, \u0027username\u0027, conf.get(\u0027identity\u0027, \u0027username\u0027))"},{"line_number":35,"context_line":"        conf.set(sec, \u0027password\u0027, conf.get(\u0027identity\u0027, \u0027password\u0027))"},{"line_number":36,"context_line":"        conf.set(sec, \u0027admin_username\u0027, conf.get(\u0027auth\u0027, \u0027admin_username\u0027))"},{"line_number":37,"context_line":"        conf.set(sec, \u0027admin_password\u0027, conf.get(\u0027auth\u0027, \u0027admin_password\u0027))"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_df636719","line":34,"updated":"2019-07-30 12:49:05.000000000","message":"In the other file, heat_integrationtests/prepare_test_env.sh is called, and it sets those values. \nI think that the heat tempest plugin shows various issues (it shouldn\u0027t duplicate values already set somewhere else in tempest.conf), but then I\u0027m not sure what is the best course of action. Maybe, if all the needed values are set here, it may be possible to drop the call to that script.","commit_id":"a6c4a753551982a13bb021dde2f14741bd28ebc2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3042096480862535383caf95d48ec264b7d07557","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def set_default_tempest_options(self, conf):"},{"line_number":33,"context_line":"        try:"},{"line_number":34,"context_line":"            uri \u003d conf.get(\u0027identity\u0027, \u0027uri\u0027)"},{"line_number":35,"context_line":"            if \u0027v3\u0027 not in uri:"},{"line_number":36,"context_line":"                return"},{"line_number":37,"context_line":"            sec \u003d \u0027heat_plugin\u0027"},{"line_number":38,"context_line":"            conf.set(sec, \u0027username\u0027, conf.get(\u0027identity\u0027, \u0027username\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_73715076","line":35,"range":{"start_line":34,"start_character":12,"end_line":35,"end_character":31},"updated":"2020-03-19 10:24:05.000000000","message":"this doesn\u0027t seem right, in the section there are \u0027uri\u0027 and \u0027uri_v3\u0027 options, \u0027uri\u0027 doesn\u0027t contain \u0027v3\u0027, so the condition is wrong","commit_id":"a858becb2db2e9c278df461ef028aa18116383b9"}]}