)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"26364bbb291f9c415888f4864a2ba4d4c2c73d4b","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch replaces the use of Eventlet’s"},{"line_number":10,"context_line":"timeout context manager with a standard Python implementation"},{"line_number":11,"context_line":"using the signal module (signal.alarm)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Icbe1a5af179c9940dff901352126de7b8594a9c5"},{"line_number":14,"context_line":"Signed-off-by: Kamil Sambor \u003ckamil.sambor@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f8f84374_375c1a75","line":11,"updated":"2025-07-08 13:20:56.000000000","message":"-1 we cannout use singals\n\nyou can only have one signal handler for sig alarm per process so this is a non starter.\n\nwe evalcuated and discsarded this approch a long time ago because we need the ablity to have multipel timeouts potically.","commit_id":"ae70eeef29de889e4a232729d2832c27bee1ef68"}],"nova/compute/manager.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"af28c6eff49df34c3bc757d734334b339c076624","unresolved":true,"context_lines":[{"line_number":573,"context_line":"        sw \u003d timeutils.StopWatch()"},{"line_number":574,"context_line":"        sw.start()"},{"line_number":575,"context_line":"        try:"},{"line_number":576,"context_line":"            with eventlet.timeout.Timeout(deadline):"},{"line_number":577,"context_line":"                self._wait_for_instance_events("},{"line_number":578,"context_line":"                    instance, events, error_callback)"},{"line_number":579,"context_line":"        except eventlet.timeout.Timeout:"}],"source_content_type":"text/x-python","patch_set":1,"id":"41b996aa_19440791","side":"PARENT","line":576,"updated":"2025-07-10 14:34:48.000000000","message":"```\n\u003e\u003e\u003e import eventlet\n\u003e\u003e\u003e eventlet.monkey_patch()\n\u003e\u003e\u003e e \u003d eventlet.Event()\n\u003e\u003e\u003e str(e.wait(timeout\u003d10))\n\u0027None\u0027\n\u003e\u003e\u003e with eventlet.Timeout(10):\n...   str(e.wait())\n... \nTraceback (most recent call last):\n  File \"\u003cstdin\u003e\", line 2, in \u003cmodule\u003e\n  File \"/home/gibi/upstream/git/openstack/nova/.tox/py312/lib/python3.12/site-packages/eventlet/event.py\", line 124, in wait\n    result \u003d hub.switch()\n             ^^^^^^^^^^^^\n  File \"/home/gibi/upstream/git/openstack/nova/.tox/py312/lib/python3.12/site-packages/eventlet/hubs/hub.py\", line 310, in switch\n    return self.greenlet.switch()\n           ^^^^^^^^^^^^^^^^^^^^^^\neventlet.timeout.Timeout: 10 seconds\n\u003e\u003e\u003e \n```\nThis shows that we do have an exception raised from e.wait() when the Timeout context manager is used. (I hate eventlet).","commit_id":"68eb3bc40517f8aea3b61727668c023c77e0e0ae"}],"nova/compute/multi_cell_list.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a8badaf4480483f2c6f4dee3ddf84fb6318a82fb","unresolved":true,"context_lines":[{"line_number":100,"context_line":"    "},{"line_number":101,"context_line":"    with eventlet.timeout.Timeout(context.CELL_TIMEOUT, exception.CellTimeout):"},{"line_number":102,"context_line":"        signal.signal(signal.SIGALRM, compute_utils.timeout_handler)"},{"line_number":103,"context_line":"        signal.alarm(context.CELL_TIMEOUT) "},{"line_number":104,"context_line":"        try:"},{"line_number":105,"context_line":"            for record in fn(ctx, *args, **kwargs):"},{"line_number":106,"context_line":"                yield record"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b919053_46df2840","line":103,"updated":"2025-07-08 13:21:40.000000000","message":"the intent here is to move this to just using the db timeout.","commit_id":"ae70eeef29de889e4a232729d2832c27bee1ef68"}]}
