)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33560,"name":"Alexandre Horst","email":"alexandre.horst@encora.com","username":"ahorst"},"change_message_id":"bc3a07bb3d46cc27fe242158c4f6d5d5699a6d13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9e70f14c_84756b42","updated":"2021-12-09 18:51:05.000000000","message":"I changed the error message as suggested by Sabeel.","commit_id":"ff1b35283cddc1693cb62529a1d1fba5ab63ea34"},{"author":{"_account_id":33560,"name":"Alexandre Horst","email":"alexandre.horst@encora.com","username":"ahorst"},"change_message_id":"143a9109162d67dac23fd0cb915d7db6bce8f7b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2eea8a93_24d1a636","updated":"2021-12-10 13:57:43.000000000","message":"Added a space on end of string.","commit_id":"c605b620c39e5caca97f8729f216a86ab727f56b"},{"author":{"_account_id":33560,"name":"Alexandre Horst","email":"alexandre.horst@encora.com","username":"ahorst"},"change_message_id":"f86409a1b65de4e1b8077f848d6e37b145124878","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a6fdf93d_3d30c5db","updated":"2021-12-10 16:05:56.000000000","message":"I think I addressed all the comments, so is there anything to comment or to change?","commit_id":"c605b620c39e5caca97f8729f216a86ab727f56b"}],"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/certificate.py":[{"author":{"_account_id":31500,"name":"Sabeel Ansari","email":"Sabeel.Ansari@windriver.com","username":"sansari"},"change_message_id":"c2abd58a8d2a8e33c42e16d5b34ad45ec10d54f7","unresolved":true,"context_lines":[{"line_number":412,"context_line":"            # validation checking for ssl_ca"},{"line_number":413,"context_line":"            if mode \u003d\u003d constants.CERT_MODE_SSL_CA:"},{"line_number":414,"context_line":"                if not cutils.is_ca_cert(cert):"},{"line_number":415,"context_line":"                    msg \u003d \"Cannot install SSL certificate as SSL \" \\"},{"line_number":416,"context_line":"                          \"CA cert\""},{"line_number":417,"context_line":"                    return dict(success\u003d\"\", error\u003dmsg)"},{"line_number":418,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"22698b94_1d2cff3a","line":415,"updated":"2021-12-09 16:10:03.000000000","message":"I think it makes sense to change this to say that its not a CA-type certificate, instead of saying its a ssl certificate.\n\nSomething like \"Cannot install non-CA type certificate as SSL CA certificate\"","commit_id":"52a0ff9e45120b4eb5108b9812eddc7dd8f319fb"},{"author":{"_account_id":33560,"name":"Alexandre Horst","email":"alexandre.horst@encora.com","username":"ahorst"},"change_message_id":"bc3a07bb3d46cc27fe242158c4f6d5d5699a6d13","unresolved":false,"context_lines":[{"line_number":412,"context_line":"            # validation checking for ssl_ca"},{"line_number":413,"context_line":"            if mode \u003d\u003d constants.CERT_MODE_SSL_CA:"},{"line_number":414,"context_line":"                if not cutils.is_ca_cert(cert):"},{"line_number":415,"context_line":"                    msg \u003d \"Cannot install SSL certificate as SSL \" \\"},{"line_number":416,"context_line":"                          \"CA cert\""},{"line_number":417,"context_line":"                    return dict(success\u003d\"\", error\u003dmsg)"},{"line_number":418,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"805caba3_e84fefb9","line":415,"in_reply_to":"22698b94_1d2cff3a","updated":"2021-12-09 18:51:05.000000000","message":"Done","commit_id":"52a0ff9e45120b4eb5108b9812eddc7dd8f319fb"}]}