)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"d315a800f6229e361ba8f51f37ee125ec4c3237c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47e31040_12823c75","updated":"2025-09-25 17:44:30.000000000","message":"This will probably still fail for almost all the fake computes per worker as the state path is shared across them so other than the last compute the rest\u0027s compute_id is overwritten.","commit_id":"16ff61097de9c88f2093ab353c91a078985b839f"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"35eba43e556be4f7b85b20787695e9f6a7021773","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5e159537_3d094cfe","updated":"2025-09-26 13:43:04.000000000","message":"Suggestion inline, but otherwise this looks fine.","commit_id":"010f90c8daa68d534325bfef0448230b3c77acf5"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"e435238fd2f438e4d75e55143e1d4b294001c685","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8ecc74cb_a7c1242e","updated":"2025-10-28 15:41:38.000000000","message":"Change in a fake driver, looks fine.","commit_id":"f8b3c902bef911938e03d4fc4f13fc9851a843f5"}],"lib/nova":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"35eba43e556be4f7b85b20787695e9f6a7021773","unresolved":true,"context_lines":[{"line_number":1088,"context_line":"            # Ensure that each fake compute has its own state path so that it"},{"line_number":1089,"context_line":"            # can have its own compute_id file"},{"line_number":1090,"context_line":"            COMPUTE_ID\u003d$(uuidgen)"},{"line_number":1091,"context_line":"            sudo mkdir -p $NOVA_STATE_PATH/$COMPUTE_ID"},{"line_number":1092,"context_line":"            iniset $fake_conf DEFAULT state_path \"$NOVA_STATE_PATH/$COMPUTE_ID\""},{"line_number":1093,"context_line":"            # use the generated UUID as the stable compute node UUID"},{"line_number":1094,"context_line":"            echo \"$COMPUTE_ID\" | sudo tee \"$NOVA_STATE_PATH/$COMPUTE_ID/compute_id\""}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"0ca18daa_828aa1d8","line":1091,"updated":"2025-09-26 13:43:04.000000000","message":"I wonder if it might be easier for development to use `compute_${i}` here instead of just a bunch of UUID directories? Or `${HOSTNAME}${i}` to match DEFAULT.host for easier matching?","commit_id":"010f90c8daa68d534325bfef0448230b3c77acf5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"7368d76ee0733831ad8bec4466f35c9dd823bf58","unresolved":false,"context_lines":[{"line_number":1088,"context_line":"            # Ensure that each fake compute has its own state path so that it"},{"line_number":1089,"context_line":"            # can have its own compute_id file"},{"line_number":1090,"context_line":"            COMPUTE_ID\u003d$(uuidgen)"},{"line_number":1091,"context_line":"            sudo mkdir -p $NOVA_STATE_PATH/$COMPUTE_ID"},{"line_number":1092,"context_line":"            iniset $fake_conf DEFAULT state_path \"$NOVA_STATE_PATH/$COMPUTE_ID\""},{"line_number":1093,"context_line":"            # use the generated UUID as the stable compute node UUID"},{"line_number":1094,"context_line":"            echo \"$COMPUTE_ID\" | sudo tee \"$NOVA_STATE_PATH/$COMPUTE_ID/compute_id\""}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"07d7df8a_b98a44f3","line":1091,"in_reply_to":"0ca18daa_828aa1d8","updated":"2025-09-26 15:00:21.000000000","message":"Done","commit_id":"010f90c8daa68d534325bfef0448230b3c77acf5"}]}
