)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5d0c93a2329afebf20d2b3e4a1a2c6c2044c2d4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1601ed41_13ebfb95","updated":"2021-11-10 05:27:58.000000000","message":"\u003e Patch Set 1:\n\u003e \n\u003e There seems to be several references to enabling this?  Are they all old?\n\u003e \n\u003e Also seems to be some tempest parts that could be cleaned up too?\n\u003e \n\u003e [1] https://codesearch.opendev.org/?q\u003dENABLE_IDENTITY_V2\u0026i\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d\n\nI\u0027ll clean up references to this, but they shouldn\u0027t have any effect, keystone cannot provide v2.0 any longer.","commit_id":"22d09eb77d868dcc4ac0e7d4d4649b5429e9a2b6"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"caa47114af559aea035d77b6f6bb462ff7e514f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"66c880f2_9f59005d","updated":"2021-11-10 05:28:35.000000000","message":"\u003e Patch Set 1:\n\u003e \n\u003e also: https://codesearch.opendev.org/?q\u003d%5CbIDENTITY_API_VERSION%5Cb\u0026i\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d\n\nI\u0027m keeping the variable definition now, marking it for later cleanup.","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"25eb456bdb14041c46a35c6784dbfa2cf3bc8caa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"850e1c78_415ec4ec","updated":"2021-11-30 14:53:20.000000000","message":"lgtm, thanks","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"688bbf0f92b9e63d7c1dcdbc43327c4bdcfffeee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7506c144_6cc958a2","in_reply_to":"66c880f2_9f59005d","updated":"2021-11-11 19:08:31.000000000","message":"+1","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"}],"openrc":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"435facab419693c69470b4ecc7fcb90867deb176","unresolved":true,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"# Currently, in order to use openstackclient with Identity API v3,"},{"line_number":95,"context_line":"# we need to set the domain which the user and project belong to."},{"line_number":96,"context_line":"if [ \"$OS_IDENTITY_API_VERSION\" \u003d \"3\" ]; then"},{"line_number":97,"context_line":"    export OS_USER_DOMAIN_ID\u003d${OS_USER_DOMAIN_ID:-\"default\"}"},{"line_number":98,"context_line":"    export OS_PROJECT_DOMAIN_ID\u003d${OS_PROJECT_DOMAIN_ID:-\"default\"}"},{"line_number":99,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"b75beca6_aee785ff","line":96,"range":{"start_line":96,"start_character":0,"end_line":96,"end_character":1},"updated":"2021-11-23 15:14:20.000000000","message":"we can also drop this condition, can\u0027t we?","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"25eb456bdb14041c46a35c6784dbfa2cf3bc8caa","unresolved":true,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"# Currently, in order to use openstackclient with Identity API v3,"},{"line_number":95,"context_line":"# we need to set the domain which the user and project belong to."},{"line_number":96,"context_line":"if [ \"$OS_IDENTITY_API_VERSION\" \u003d \"3\" ]; then"},{"line_number":97,"context_line":"    export OS_USER_DOMAIN_ID\u003d${OS_USER_DOMAIN_ID:-\"default\"}"},{"line_number":98,"context_line":"    export OS_PROJECT_DOMAIN_ID\u003d${OS_PROJECT_DOMAIN_ID:-\"default\"}"},{"line_number":99,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"cb81bc1d_d4c227d2","line":96,"range":{"start_line":96,"start_character":0,"end_line":96,"end_character":1},"in_reply_to":"9c86467b_25a8c1dd","updated":"2021-11-30 14:53:20.000000000","message":"ok, that can be done in a follow up patch, let\u0027s merge this one now","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1a5ab26363ffb45f957de447fb86ddc3d901f248","unresolved":true,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"# Currently, in order to use openstackclient with Identity API v3,"},{"line_number":95,"context_line":"# we need to set the domain which the user and project belong to."},{"line_number":96,"context_line":"if [ \"$OS_IDENTITY_API_VERSION\" \u003d \"3\" ]; then"},{"line_number":97,"context_line":"    export OS_USER_DOMAIN_ID\u003d${OS_USER_DOMAIN_ID:-\"default\"}"},{"line_number":98,"context_line":"    export OS_PROJECT_DOMAIN_ID\u003d${OS_PROJECT_DOMAIN_ID:-\"default\"}"},{"line_number":99,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"9c86467b_25a8c1dd","line":96,"range":{"start_line":96,"start_character":0,"end_line":96,"end_character":1},"in_reply_to":"b75beca6_aee785ff","updated":"2021-11-23 15:33:13.000000000","message":"Hmm, we could. But actually we should drop this whole thing almost completely and only set OS_CLOUD to some value. I wonder whether we even need a script for that any longer. But likely we\u0027ll need to deprecate it first.","commit_id":"95555ba39827e2f3648eb89bde91b1342c493306"}]}