)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"051fc7b30056ddbcf96aa86163552d4fae8bf886","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4f8fe27c_4e57dc5d","updated":"2023-06-07 07:49:41.000000000","message":"Thanks for submitting this patch. This needs more work before it can be merged. As a start, could you please remove all the IBM comments, change the cafile default to None, and make the patch only change the cafile configuration, not the way the endpoint URL is built? Thanks.","commit_id":"f417e7a7899fbc56b12d8a801b042f83f4cefc09"},{"author":{"_account_id":36003,"name":"Amar Rajeev","display_name":"amar_rajeev","email":"Amar.Rajeev.Singuluri1@partner.ibm.com","username":"amar_rajeev"},"change_message_id":"23cfc9e8c1db3d809e21eb7be24c5e5a20d5b7f3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5e509eb0_3ca4f963","updated":"2023-05-26 09:47:37.000000000","message":"recheck","commit_id":"f417e7a7899fbc56b12d8a801b042f83f4cefc09"}],"blazar/config.py":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"051fc7b30056ddbcf96aa86163552d4fae8bf886","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               help\u003d\u0027A domain name the os_admin_project_name belongs to\u0027),"},{"line_number":69,"context_line":"    # IBM-only start"},{"line_number":70,"context_line":"    # RTC 93973"},{"line_number":71,"context_line":"    cfg.StrOpt(\u0027cafile\u0027,"},{"line_number":72,"context_line":"               default\u003d\u0027/etc/pki/tls/certs/powervc.crt\u0027,"},{"line_number":73,"context_line":"               help\u003d\u0027The certificate authority filename\u0027)"},{"line_number":74,"context_line":"    # IBM-only end"},{"line_number":75,"context_line":"]"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"cada9020_9ac2d85d","line":73,"range":{"start_line":71,"start_character":0,"end_line":73,"end_character":57},"updated":"2023-06-07 07:49:41.000000000","message":"We may want to follow what is done in some other services, for example Ironic, and have a different setting per service that we need to reach.","commit_id":"f417e7a7899fbc56b12d8a801b042f83f4cefc09"}],"blazar/utils/openstack/base.py":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"051fc7b30056ddbcf96aa86163552d4fae8bf886","unresolved":true,"context_lines":[{"line_number":64,"context_line":"    if ctx is not None:"},{"line_number":65,"context_line":"        kwargs.setdefault(\u0027global_request_id\u0027, ctx.global_request_id)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    if auth_url is None:"},{"line_number":68,"context_line":"        auth_url \u003d \"%s://%s:%s/%s/%s\" % (CONF.os_auth_protocol,"},{"line_number":69,"context_line":"                                         get_os_auth_host(CONF),"},{"line_number":70,"context_line":"                                         CONF.os_auth_port,"},{"line_number":71,"context_line":"                                         CONF.os_auth_prefix,"},{"line_number":72,"context_line":"                                         CONF.os_auth_version)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    auth_kwargs \u003d dict("},{"line_number":75,"context_line":"        auth_url\u003dauth_url,"}],"source_content_type":"text/x-python","patch_set":5,"id":"364ed929_66122114","side":"PARENT","line":72,"range":{"start_line":67,"start_character":0,"end_line":72,"end_character":62},"updated":"2023-06-07 07:49:41.000000000","message":"This is required to be compatible with the way DevStack configures OpenStack endpoints.","commit_id":"735d43eccc240ef03a1a0489db183a485f9fb71c"}]}
