)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"3764e79599f5bfc221e34912687bea6deed75ba0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7cf8a7ea_ab34d7e5","updated":"2022-02-28 13:03:10.000000000","message":"recheck","commit_id":"90fd3c41f0af27355262e5fb5a0759dfcae1ebbb"}],"tripleo_common/utils/overcloudrc.py":[{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"263e44196fd89f4e8fde9bfc89614c46a29dba38","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    # Remove duplicated entries"},{"line_number":99,"context_line":"    no_proxy_list \u003d sorted(list(set(no_proxy_list)))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    rc_params \u003d {"},{"line_number":102,"context_line":"        \u0027OS_USERNAME\u0027: \u0027admin\u0027,"},{"line_number":103,"context_line":"        \u0027OS_PROJECT_NAME\u0027: \u0027admin\u0027,"},{"line_number":104,"context_line":"        \u0027OS_USER_DOMAIN_NAME\u0027: \u0027Default\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"e06f0fed_ad59563b","line":101,"updated":"2022-04-07 23:07:02.000000000","message":"Can\u0027t we just set OS_CLOUD\u003dovercloud for this now? Since we\u0027re writing a clouds.yaml file. We should probably settle on just one, if we want to use clouds.yaml, simply setting OS_CLOUD\u003dovercloud would be all we need to maintain compatibility for people that want to source overcloudrc.","commit_id":"90fd3c41f0af27355262e5fb5a0759dfcae1ebbb"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"e1fcad665fd763feea465f96fa6172ac0a9bde72","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    # Remove duplicated entries"},{"line_number":99,"context_line":"    no_proxy_list \u003d sorted(list(set(no_proxy_list)))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    rc_params \u003d {"},{"line_number":102,"context_line":"        \u0027OS_USERNAME\u0027: \u0027admin\u0027,"},{"line_number":103,"context_line":"        \u0027OS_PROJECT_NAME\u0027: \u0027admin\u0027,"},{"line_number":104,"context_line":"        \u0027OS_USER_DOMAIN_NAME\u0027: \u0027Default\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fbf7f693_9258c5ce","line":101,"in_reply_to":"8cc654da_a5b0bc61","updated":"2022-04-14 11:11:08.000000000","message":"could we followup with this change in a future patch if it\u0027s desired? for now, I\u0027d like to just keep the existing functionality but make it work with ephemeral Heat.","commit_id":"90fd3c41f0af27355262e5fb5a0759dfcae1ebbb"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6579b17540de11ebd23febebc974cdb996162f06","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    # Remove duplicated entries"},{"line_number":99,"context_line":"    no_proxy_list \u003d sorted(list(set(no_proxy_list)))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    rc_params \u003d {"},{"line_number":102,"context_line":"        \u0027OS_USERNAME\u0027: \u0027admin\u0027,"},{"line_number":103,"context_line":"        \u0027OS_PROJECT_NAME\u0027: \u0027admin\u0027,"},{"line_number":104,"context_line":"        \u0027OS_USER_DOMAIN_NAME\u0027: \u0027Default\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"8cc654da_a5b0bc61","line":101,"in_reply_to":"e06f0fed_ad59563b","updated":"2022-04-08 06:00:25.000000000","message":"The main concern with OS_CLOUD\u003dovercloud is that it doesn\u0027t work with the legacy CLIs like nova, cinder and etc. We might be able to switch to OS_CLOUD completely considering the improved coverage of recnet openstack cli but I tend to leave this or create a separate file with individual environments so that legacy CLI can be still used without setting OS_* manually.","commit_id":"90fd3c41f0af27355262e5fb5a0759dfcae1ebbb"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"1ffb2b5c3f6e6b8557be03b70958e25a83ab61ca","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    # Remove duplicated entries"},{"line_number":99,"context_line":"    no_proxy_list \u003d sorted(list(set(no_proxy_list)))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    rc_params \u003d {"},{"line_number":102,"context_line":"        \u0027OS_USERNAME\u0027: \u0027admin\u0027,"},{"line_number":103,"context_line":"        \u0027OS_PROJECT_NAME\u0027: \u0027admin\u0027,"},{"line_number":104,"context_line":"        \u0027OS_USER_DOMAIN_NAME\u0027: \u0027Default\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a75bde9_2fb9563a","line":101,"in_reply_to":"fbf7f693_9258c5ce","updated":"2022-04-14 11:17:23.000000000","message":"Absolutely, no objections from me. Just raising it as a discussion point. Takashi had a good point. We can just leave it until the individual service clients are deprecated and then look at it.","commit_id":"90fd3c41f0af27355262e5fb5a0759dfcae1ebbb"}]}