)]}'
{"zuul.d/ironic-jobs.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a53ffbceeaefa41cad66e7ffc7246ce663a8f38e","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        BUILD_TIMEOUT: 1200"},{"line_number":39,"context_line":"        IRONIC_BAREMETAL_BASIC_OPS: True"},{"line_number":40,"context_line":"        IRONIC_BUILD_DEPLOY_RAMDISK: False"},{"line_number":41,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 900"},{"line_number":42,"context_line":"        IRONIC_DEPLOY_DRIVER: ipmi"},{"line_number":43,"context_line":"        IRONIC_INSPECTOR_BUILD_RAMDISK: False"},{"line_number":44,"context_line":"        IRONIC_INSPECTOR_TEMPEST_INTROSPECTION_TIMEOUT: 1200"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_aacfa6b1","line":41,"updated":"2020-07-19 15:31:07.000000000","message":"IRONIC_PXE_BOOT_RETRY_TIMEOUT is 600 by default, maybe we want to increase it? and then set callback_timeout to 2x retry timeout?","commit_id":"517999670411b2847f20dd3acd6b7dd7a78b25d5"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"caf7a0d29946b3ca7c183d657a2ca6e1046e2ed2","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        BUILD_TIMEOUT: 1200"},{"line_number":39,"context_line":"        IRONIC_BAREMETAL_BASIC_OPS: True"},{"line_number":40,"context_line":"        IRONIC_BUILD_DEPLOY_RAMDISK: False"},{"line_number":41,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 900"},{"line_number":42,"context_line":"        IRONIC_DEPLOY_DRIVER: ipmi"},{"line_number":43,"context_line":"        IRONIC_INSPECTOR_BUILD_RAMDISK: False"},{"line_number":44,"context_line":"        IRONIC_INSPECTOR_TEMPEST_INTROSPECTION_TIMEOUT: 1200"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_651adf6f","line":41,"in_reply_to":"bf51134e_aacfa6b1","updated":"2020-07-19 21:23:36.000000000","message":"Ohh, yes, we should extend that too. Wouldn\u0027t we want them to instead be the same value? I\u0027d have to go double check the code, but don\u0027t we kick the machine if it is not up in the allotted amount of time?","commit_id":"517999670411b2847f20dd3acd6b7dd7a78b25d5"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"48d04d07f24872c882cb59a4cb7c1461ac7e32be","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        IRONIC_BAREMETAL_BASIC_OPS: True"},{"line_number":40,"context_line":"        IRONIC_BUILD_DEPLOY_RAMDISK: False"},{"line_number":41,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 900"},{"line_number":42,"context_line":"        IRONIC_PXE_BOOT_RETRY_TIMEOUT: 900"},{"line_number":43,"context_line":"        IRONIC_DEPLOY_DRIVER: ipmi"},{"line_number":44,"context_line":"        IRONIC_INSPECTOR_BUILD_RAMDISK: False"},{"line_number":45,"context_line":"        IRONIC_INSPECTOR_TEMPEST_INTROSPECTION_TIMEOUT: 1200"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bf51134e_5bb69a40","line":42,"updated":"2020-07-20 07:30:28.000000000","message":"You\u0027re disabling retries this way. The callback timeout should be at least 2x the retry timeout.","commit_id":"044280defa640e6b10364d1c500b9ce2c27f72d7"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c0d226885f26a78176f7743fe4745bb32ec6b77b","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        IRONIC_BAREMETAL_BASIC_OPS: True"},{"line_number":40,"context_line":"        IRONIC_BUILD_DEPLOY_RAMDISK: False"},{"line_number":41,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 900"},{"line_number":42,"context_line":"        IRONIC_PXE_BOOT_RETRY_TIMEOUT: 900"},{"line_number":43,"context_line":"        IRONIC_DEPLOY_DRIVER: ipmi"},{"line_number":44,"context_line":"        IRONIC_INSPECTOR_BUILD_RAMDISK: False"},{"line_number":45,"context_line":"        IRONIC_INSPECTOR_TEMPEST_INTROSPECTION_TIMEOUT: 1200"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bf51134e_94d5951e","line":42,"in_reply_to":"bf51134e_5bb69a40","updated":"2020-07-20 21:40:15.000000000","message":"I guess it makes sense to go ahead and let it run. I just wish we didn\u0027t need such a long window for CI. :( Enter why we did tinyipa.","commit_id":"044280defa640e6b10364d1c500b9ce2c27f72d7"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"f72b591f1949640cd7db6dc8794b5eb2be2318e4","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        FORCE_CONFIG_DRIVE: True"},{"line_number":36,"context_line":"        INSTALL_TEMPEST: False # Don\u0027t install a tempest package globaly"},{"line_number":37,"context_line":"        VIRT_DRIVER: ironic"},{"line_number":38,"context_line":"        BUILD_TIMEOUT: 1200"},{"line_number":39,"context_line":"        IRONIC_BAREMETAL_BASIC_OPS: True"},{"line_number":40,"context_line":"        IRONIC_BUILD_DEPLOY_RAMDISK: False"},{"line_number":41,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 1800"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bf51134e_75f43488","line":38,"updated":"2020-07-21 13:01:23.000000000","message":"Raise this to 1800 as well?","commit_id":"9cac9a02578527b9db2892fc33ef54f51b054566"}]}