)]}'
{"whitebox_tempest_plugin/api/compute/test_uefi.py":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"451735e3f03e77db8cf1346ab0e0220b4de4d15f","unresolved":true,"context_lines":[{"line_number":61,"context_line":"            # Secure boot does not need to be explicitly set to \u0027no\u0027 if it"},{"line_number":62,"context_line":"            # is not being used by the guest. If the parameter is present"},{"line_number":63,"context_line":"            # confirm it is set to \u0027no\u0027."},{"line_number":64,"context_line":"            secure_boot_present \u003d loader.get(\u0027secure\u0027)"},{"line_number":65,"context_line":"            if secure_boot_present:"},{"line_number":66,"context_line":"                self.assertEqual("},{"line_number":67,"context_line":"                    \u0027no\u0027, loader.get(\u0027secure\u0027), \u0027Secure boot should be \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"23ac19d0_75f873d4","line":64,"updated":"2022-12-13 20:40:47.000000000","message":"This could be made more pythonic by just giving a default value of \u0027no\u0027 for get():\n\n  loader.get(\u0027secure\u0027, \u0027no)","commit_id":"997043d3aa5080657a7ca78241ee6f3ce92bd4a0"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"c055b6bf58b76cb7754d1c40b77972c457dcd534","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            # Secure boot does not need to be explicitly set to \u0027no\u0027 if it"},{"line_number":62,"context_line":"            # is not being used by the guest. If the parameter is present"},{"line_number":63,"context_line":"            # confirm it is set to \u0027no\u0027."},{"line_number":64,"context_line":"            secure_boot_present \u003d loader.get(\u0027secure\u0027)"},{"line_number":65,"context_line":"            if secure_boot_present:"},{"line_number":66,"context_line":"                self.assertEqual("},{"line_number":67,"context_line":"                    \u0027no\u0027, loader.get(\u0027secure\u0027), \u0027Secure boot should be \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"587181d8_6370d83c","line":64,"in_reply_to":"1442f0c5_c3576ab9","updated":"2022-12-13 21:08:09.000000000","message":"Yes much better I\u0027ll change based around that.","commit_id":"997043d3aa5080657a7ca78241ee6f3ce92bd4a0"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"cb9d9713663caf784826457cd089bd76a3b061e8","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            # Secure boot does not need to be explicitly set to \u0027no\u0027 if it"},{"line_number":62,"context_line":"            # is not being used by the guest. If the parameter is present"},{"line_number":63,"context_line":"            # confirm it is set to \u0027no\u0027."},{"line_number":64,"context_line":"            secure_boot_present \u003d loader.get(\u0027secure\u0027)"},{"line_number":65,"context_line":"            if secure_boot_present:"},{"line_number":66,"context_line":"                self.assertEqual("},{"line_number":67,"context_line":"                    \u0027no\u0027, loader.get(\u0027secure\u0027), \u0027Secure boot should be \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"68faed67_40d6b208","line":64,"in_reply_to":"23ac19d0_75f873d4","updated":"2022-12-13 20:42:05.000000000","message":"Good point, I\u0027ll update accordingly.","commit_id":"997043d3aa5080657a7ca78241ee6f3ce92bd4a0"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"12d58e241cdf81004cd3d8c28a8e3a3830c282d4","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            # Secure boot does not need to be explicitly set to \u0027no\u0027 if it"},{"line_number":62,"context_line":"            # is not being used by the guest. If the parameter is present"},{"line_number":63,"context_line":"            # confirm it is set to \u0027no\u0027."},{"line_number":64,"context_line":"            secure_boot_present \u003d loader.get(\u0027secure\u0027)"},{"line_number":65,"context_line":"            if secure_boot_present:"},{"line_number":66,"context_line":"                self.assertEqual("},{"line_number":67,"context_line":"                    \u0027no\u0027, loader.get(\u0027secure\u0027), \u0027Secure boot should be \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"1442f0c5_c3576ab9","line":64,"in_reply_to":"68faed67_40d6b208","updated":"2022-12-13 20:43:29.000000000","message":"In fact if we just add the default to L59, that\u0027s the only change we need.","commit_id":"997043d3aa5080657a7ca78241ee6f3ce92bd4a0"}]}