)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9f0d2fd1820aca3db23d519580c3725d030416c5","unresolved":true,"context_lines":[{"line_number":15,"context_line":"forever and when loglevel is not debug it\u0027s not clear"},{"line_number":16,"context_line":"to operators why it\u0027s stopped and why it hasn\u0027t"},{"line_number":17,"context_line":"progressed to start the resource tracker and initialize"},{"line_number":18,"context_line":"the compute node."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This adds a 60 seconds timeout and changes the code"},{"line_number":21,"context_line":"to catch the timeout exception and log it. We must"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"0ae1717d_447a3cb0","line":18,"updated":"2026-02-09 11:44:45.000000000","message":"yes this is a known problem\n\nnova will deadlock later if we dont fail here when trying to do other volume operartions  if the osd is not accable.\n\nwe have 2 way to interact with ceph one via the cli and the ohter via the python lib\n\nthe python lib does not support timeouts in the blocking mode\nits async api does but we do not use that currently and likely wont until after we compelte the eventlet removeal work.","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4d5d55ac13f029f6e8eee7f128c8298cc21ace18","unresolved":false,"context_lines":[{"line_number":15,"context_line":"forever and when loglevel is not debug it\u0027s not clear"},{"line_number":16,"context_line":"to operators why it\u0027s stopped and why it hasn\u0027t"},{"line_number":17,"context_line":"progressed to start the resource tracker and initialize"},{"line_number":18,"context_line":"the compute node."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This adds a 60 seconds timeout and changes the code"},{"line_number":21,"context_line":"to catch the timeout exception and log it. We must"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"346703e3_0ec24e66","line":18,"in_reply_to":"0ae1717d_447a3cb0","updated":"2026-02-09 14:11:47.000000000","message":"Done","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9f0d2fd1820aca3db23d519580c3725d030416c5","unresolved":true,"context_lines":[{"line_number":25,"context_line":"We catch all exceptions because when in eventlet the"},{"line_number":26,"context_line":"exception is thrown as eventlet.green.subprocess.TimeoutExpired"},{"line_number":27,"context_line":"and not subprocess.TimeoutExpired and to not introduce"},{"line_number":28,"context_line":"any eventlet imports we catch all, which is also fair here"},{"line_number":29,"context_line":"to not leak any information, and we always log the exception"},{"line_number":30,"context_line":"to help operators."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"125188e0_ca28f969","line":28,"updated":"2026-02-09 11:44:45.000000000","message":"if we are to do this i would want a fixme or todo comment to remove this when we only supprot threaded mode.","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9f0d2fd1820aca3db23d519580c3725d030416c5","unresolved":true,"context_lines":[{"line_number":28,"context_line":"any eventlet imports we catch all, which is also fair here"},{"line_number":29,"context_line":"to not leak any information, and we always log the exception"},{"line_number":30,"context_line":"to help operators."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Signed-off-by: Tobias Urdin \u003ctobias.urdin@binero.com\u003e"},{"line_number":33,"context_line":"Change-Id: Id07b3e8d5c73775f266bad97c7427b5ec238d285"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1c1c29cb_3584c8b1","line":31,"updated":"2026-02-09 11:44:45.000000000","message":"it would be good to have a bug tracker\n\ni knwo there were some in the past for this bug i cant find it\n\nhttps://bugs.launchpad.net/nova/+bug/1925144\nis relateed to the python side fo this where the timeout will only work for the async apis\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d2271314 is one of the downstream tracker we had related to the rados side which is covered by https://bugs.launchpad.net/nova/+bug/1925144\n\nbut we dont seam to have a trakcer for the commandline side of this probelm\n\nits not the first tiem this has come up but since we didnt have a plan for hwo to fix the python side we never spent time on the cli side.","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4d5d55ac13f029f6e8eee7f128c8298cc21ace18","unresolved":false,"context_lines":[{"line_number":28,"context_line":"any eventlet imports we catch all, which is also fair here"},{"line_number":29,"context_line":"to not leak any information, and we always log the exception"},{"line_number":30,"context_line":"to help operators."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Signed-off-by: Tobias Urdin \u003ctobias.urdin@binero.com\u003e"},{"line_number":33,"context_line":"Change-Id: Id07b3e8d5c73775f266bad97c7427b5ec238d285"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"47fa5ac8_95682260","line":31,"in_reply_to":"1c1c29cb_3584c8b1","updated":"2026-02-09 14:11:47.000000000","message":"Done","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"e63d1a8177348977425cdd599277bfcac73dde98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"de16118e_b458874d","updated":"2026-02-11 09:20:08.000000000","message":"recheck timeout for tempest.api.compute.servers.test_server_actions.ServerActionsTestOtherA.test_resize_volume_backed_server_confirm","commit_id":"d1280856ca5e1c5184e473f038b406515565b02c"}],"nova/storage/rbd_utils.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9f0d2fd1820aca3db23d519580c3725d030416c5","unresolved":true,"context_lines":[{"line_number":428,"context_line":""},{"line_number":429,"context_line":"        try:"},{"line_number":430,"context_line":"            out, _ \u003d processutils.execute(*args, timeout\u003d60)"},{"line_number":431,"context_line":"        except Exception:"},{"line_number":432,"context_line":"            LOG.exception(\u0027Could not determine disk usage\u0027)"},{"line_number":433,"context_line":"            raise exception.StorageError("},{"line_number":434,"context_line":"                reason\u003d\u0027Could not determine disk usage\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"f9d71797_ce539c70","line":431,"updated":"2026-02-09 11:44:45.000000000","message":"we generally do not allow excpet Excpetion\nwe etiehr add two excpt for btoh types of excptions or you can catch multipel togehter.","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"aac420b7ec0f58489ede000c29977bf9f59b53eb","unresolved":false,"context_lines":[{"line_number":428,"context_line":""},{"line_number":429,"context_line":"        try:"},{"line_number":430,"context_line":"            out, _ \u003d processutils.execute(*args, timeout\u003d60)"},{"line_number":431,"context_line":"        except Exception:"},{"line_number":432,"context_line":"            LOG.exception(\u0027Could not determine disk usage\u0027)"},{"line_number":433,"context_line":"            raise exception.StorageError("},{"line_number":434,"context_line":"                reason\u003d\u0027Could not determine disk usage\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bdf81f35_e62b1595","line":431,"in_reply_to":"f9d71797_ce539c70","updated":"2026-02-09 13:25:28.000000000","message":"Done","commit_id":"508f4324d9e7862403d6d3446009200f0b9451bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4d5d55ac13f029f6e8eee7f128c8298cc21ace18","unresolved":true,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import urllib"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import eventlet.green.subprocess as eventlet_subprocess"},{"line_number":20,"context_line":"import subprocess"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from oslo_concurrency import processutils"}],"source_content_type":"text/x-python","patch_set":4,"id":"177ccf43_1296129b","line":19,"updated":"2026-02-09 14:11:47.000000000","message":"i think we have hacking check that will block this but we are trying to get to the point were nova compute can run without eventlet installed this cycle.\n\nso at the minimum we whould need to make this a condtional import","commit_id":"be371c115f9fb9f3551ef4098b2b95fcf28d2eb5"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"56aecc4db5b78a218628a3ac8d557b1b51f70022","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import urllib"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import eventlet.green.subprocess as eventlet_subprocess"},{"line_number":20,"context_line":"import subprocess"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from oslo_concurrency import processutils"}],"source_content_type":"text/x-python","patch_set":4,"id":"74fbe813_a50805c0","line":19,"in_reply_to":"177ccf43_1296129b","updated":"2026-02-11 13:43:05.000000000","message":"Done","commit_id":"be371c115f9fb9f3551ef4098b2b95fcf28d2eb5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4d5d55ac13f029f6e8eee7f128c8298cc21ace18","unresolved":true,"context_lines":[{"line_number":429,"context_line":"        # calculation."},{"line_number":430,"context_line":"        args \u003d [\u0027ceph\u0027, \u0027df\u0027, \u0027--format\u003djson\u0027] + self.ceph_args()"},{"line_number":431,"context_line":""},{"line_number":432,"context_line":"        # TODO(tobias-urdin): Remove eventlet exception handling"},{"line_number":433,"context_line":"        try:"},{"line_number":434,"context_line":"            out, _ \u003d processutils.execute(*args, timeout\u003d60)"},{"line_number":435,"context_line":"        except (processutils.ProcessExecutionError,"}],"source_content_type":"text/x-python","patch_set":4,"id":"a5aba5a2_ab698f3f","line":432,"updated":"2026-02-09 14:11:47.000000000","message":"actully i was suggesting keeping the except Excpetiopn but just leaving a TODO\nto revert back to explcity expction once eventlet is removed","commit_id":"be371c115f9fb9f3551ef4098b2b95fcf28d2eb5"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"56aecc4db5b78a218628a3ac8d557b1b51f70022","unresolved":false,"context_lines":[{"line_number":429,"context_line":"        # calculation."},{"line_number":430,"context_line":"        args \u003d [\u0027ceph\u0027, \u0027df\u0027, \u0027--format\u003djson\u0027] + self.ceph_args()"},{"line_number":431,"context_line":""},{"line_number":432,"context_line":"        # TODO(tobias-urdin): Remove eventlet exception handling"},{"line_number":433,"context_line":"        try:"},{"line_number":434,"context_line":"            out, _ \u003d processutils.execute(*args, timeout\u003d60)"},{"line_number":435,"context_line":"        except (processutils.ProcessExecutionError,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1e7f7cac_1b893626","line":432,"in_reply_to":"a5aba5a2_ab698f3f","updated":"2026-02-11 13:43:05.000000000","message":"Done","commit_id":"be371c115f9fb9f3551ef4098b2b95fcf28d2eb5"}]}