)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"96550e21156bf85641b23b1b9993454f36781cdf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2d51328e_98c90eb0","updated":"2021-11-03 09:28:06.000000000","message":"makes sense","commit_id":"eb37657d8e34e1d0f8ee639c3953a752ba615d8d"}],"lib/keystone":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f814ecce4ef459b994ec2c12054813bec9f138a6","unresolved":true,"context_lines":[{"line_number":565,"context_line":"            --os-username admin \\"},{"line_number":566,"context_line":"            --os-user-domain-id default \\"},{"line_number":567,"context_line":"            --os-password \"$ADMIN_PASSWORD\" \\"},{"line_number":568,"context_line":"            --os-project-name admin \\"},{"line_number":569,"context_line":"            --os-project-domain-id default \\"},{"line_number":570,"context_line":"            keystone admin \"$KEYSTONE_SERVICE_URI\""},{"line_number":571,"context_line":"    fi"},{"line_number":572,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"f2080bb6_a186c4ce","line":569,"range":{"start_line":568,"start_character":0,"end_line":569,"end_character":44},"updated":"2021-10-25 17:32:26.000000000","message":"is it needed for endpoint creation?","commit_id":"eb37657d8e34e1d0f8ee639c3953a752ba615d8d"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"771f886d8aad10851c97e5a1233538630b2a7ede","unresolved":true,"context_lines":[{"line_number":565,"context_line":"            --os-username admin \\"},{"line_number":566,"context_line":"            --os-user-domain-id default \\"},{"line_number":567,"context_line":"            --os-password \"$ADMIN_PASSWORD\" \\"},{"line_number":568,"context_line":"            --os-project-name admin \\"},{"line_number":569,"context_line":"            --os-project-domain-id default \\"},{"line_number":570,"context_line":"            keystone admin \"$KEYSTONE_SERVICE_URI\""},{"line_number":571,"context_line":"    fi"},{"line_number":572,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"2c4cd82c_38b56b9b","line":569,"range":{"start_line":568,"start_character":0,"end_line":569,"end_character":44},"in_reply_to":"6ac2acc4_e490b49f","updated":"2021-10-25 17:54:38.000000000","message":"No, I think it\u0027s ok the way it is.","commit_id":"eb37657d8e34e1d0f8ee639c3953a752ba615d8d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"20abe890fe4bf4fc642e542ecc1a3186bd5d123b","unresolved":true,"context_lines":[{"line_number":565,"context_line":"            --os-username admin \\"},{"line_number":566,"context_line":"            --os-user-domain-id default \\"},{"line_number":567,"context_line":"            --os-password \"$ADMIN_PASSWORD\" \\"},{"line_number":568,"context_line":"            --os-project-name admin \\"},{"line_number":569,"context_line":"            --os-project-domain-id default \\"},{"line_number":570,"context_line":"            keystone admin \"$KEYSTONE_SERVICE_URI\""},{"line_number":571,"context_line":"    fi"},{"line_number":572,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"6ac2acc4_e490b49f","line":569,"range":{"start_line":568,"start_character":0,"end_line":569,"end_character":44},"in_reply_to":"f2080bb6_a186c4ce","updated":"2021-10-25 17:49:13.000000000","message":"Either this or switch to using system-admin credentials instead, though I\u0027m not sure if we have access to those at this point in time.\n\nThe other option if you don\u0027t like this would be to modify the bootstrap command above, to still have the admin endpoint creation done there, modifying the argument list dynamically. I considered that when doing this patch but thought this extra step would be cleaner. I\u0027m not so sure about this any longer though, wdyt?","commit_id":"eb37657d8e34e1d0f8ee639c3953a752ba615d8d"}]}