)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e0f3f9e71f204f19cc9ccbf7f8da29c5811115e6","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Swapnil Machikar \u003cswapnil.machikar@yahoo.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-06-10 13:56:26 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Deployment test using virtual media boot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This adds test for virtual media boot deployment"},{"line_number":10,"context_line":"using iDRAC redfish virtual media"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fd327cd_0c7bfead","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"updated":"2022-06-10 14:34:10.000000000","message":"as the test is specific to iDRAC, suggest to add \u0027iDRAC\u0027 to the title too:\n\n\"Add iDRAC Redfish virtual media boot deploy test\"\n\nand can remove the description (line 9-10) altogether as it does not say anything new.","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"},{"author":{"_account_id":34694,"name":"Swapnil Machikar","email":"swapnil.machikar@yahoo.com","username":"smachi"},"change_message_id":"9878570a6afe4bc9f75e6f0e880447159d1ec015","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Swapnil Machikar \u003cswapnil.machikar@yahoo.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-06-10 13:56:26 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Deployment test using virtual media boot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This adds test for virtual media boot deployment"},{"line_number":10,"context_line":"using iDRAC redfish virtual media"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d24f32af_caa846b0","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"in_reply_to":"3fd327cd_0c7bfead","updated":"2022-06-13 09:47:27.000000000","message":"Done change to commit message as suggested","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e0f3f9e71f204f19cc9ccbf7f8da29c5811115e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24e54431_25fe5ec4","updated":"2022-06-10 14:34:10.000000000","message":"thanks for the patch, some comments inline.","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"},{"author":{"_account_id":34694,"name":"Swapnil Machikar","email":"swapnil.machikar@yahoo.com","username":"smachi"},"change_message_id":"9878570a6afe4bc9f75e6f0e880447159d1ec015","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c99f35de_f069154c","updated":"2022-06-13 09:47:27.000000000","message":"Hi, \n\nHave published the changes as per the review comments\n\nThanks ","commit_id":"29efeb7a62ec35db4214c679876e76771472d077"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"738e109e4e69d2b0fc10358c275abc8d955b9919","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"642b90e3_e2d9a804","updated":"2022-07-11 15:30:04.000000000","message":"Going to check this later today","commit_id":"b1c8a2b86f84e975817c9a7824752404d747eb2d"}],"ironic_tempest_plugin/tests/scenario/ironic_standalone/test_basic_ops.py":[{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e0f3f9e71f204f19cc9ccbf7f8da29c5811115e6","unresolved":true,"context_lines":[{"line_number":440,"context_line":"        self.boot_and_verify_node()"},{"line_number":441,"context_line":""},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"class BaremetalIdracVirtualMedia("},{"line_number":444,"context_line":"        bsm.BaremetalStandaloneScenarioTest):"},{"line_number":445,"context_line":""},{"line_number":446,"context_line":"    mandatory_attr \u003d [\u0027driver\u0027, \u0027boot_interface\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5e937266_5f90d2bc","line":443,"updated":"2022-06-10 14:34:10.000000000","message":"should add `Wholedisk` to the name like other tests?","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"},{"author":{"_account_id":34694,"name":"Swapnil Machikar","email":"swapnil.machikar@yahoo.com","username":"smachi"},"change_message_id":"9878570a6afe4bc9f75e6f0e880447159d1ec015","unresolved":false,"context_lines":[{"line_number":440,"context_line":"        self.boot_and_verify_node()"},{"line_number":441,"context_line":""},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"class BaremetalIdracVirtualMedia("},{"line_number":444,"context_line":"        bsm.BaremetalStandaloneScenarioTest):"},{"line_number":445,"context_line":""},{"line_number":446,"context_line":"    mandatory_attr \u003d [\u0027driver\u0027, \u0027boot_interface\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"a08102a9_868f785e","line":443,"in_reply_to":"5e937266_5f90d2bc","updated":"2022-06-13 09:47:27.000000000","message":"Added \u0027Wholedisk\u0027 to the class name","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e0f3f9e71f204f19cc9ccbf7f8da29c5811115e6","unresolved":true,"context_lines":[{"line_number":449,"context_line":"    boot_interface \u003d \u0027idrac-redfish-virtual-media\u0027"},{"line_number":450,"context_line":"    image_ref \u003d CONF.baremetal.whole_disk_image_ref"},{"line_number":451,"context_line":"    deploy_interface \u003d \u0027direct\u0027"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"    @decorators.idempotent_id(\u0027b0bc87a5-4324-4134-bd5f-4bb1cf549e5c\u0027)"},{"line_number":454,"context_line":"    @utils.services(\u0027image\u0027, \u0027network\u0027)"},{"line_number":455,"context_line":"    def test_deploy_virtual_media_boot(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"6060ea2b_0dd26579","line":452,"updated":"2022-06-10 14:34:10.000000000","message":"should this also have `wholedisk_image \u003d True` like other tests?","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"},{"author":{"_account_id":34694,"name":"Swapnil Machikar","email":"swapnil.machikar@yahoo.com","username":"smachi"},"change_message_id":"9878570a6afe4bc9f75e6f0e880447159d1ec015","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    boot_interface \u003d \u0027idrac-redfish-virtual-media\u0027"},{"line_number":450,"context_line":"    image_ref \u003d CONF.baremetal.whole_disk_image_ref"},{"line_number":451,"context_line":"    deploy_interface \u003d \u0027direct\u0027"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"    @decorators.idempotent_id(\u0027b0bc87a5-4324-4134-bd5f-4bb1cf549e5c\u0027)"},{"line_number":454,"context_line":"    @utils.services(\u0027image\u0027, \u0027network\u0027)"},{"line_number":455,"context_line":"    def test_deploy_virtual_media_boot(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b2f50b27_436ce6fb","line":452,"in_reply_to":"6060ea2b_0dd26579","updated":"2022-06-13 09:47:27.000000000","message":"Added \u0027Wholedisk \u003d True\u0027","commit_id":"68c08171d8a9c658dd33b3408c364cea5d40fd21"}]}
