)]}'
{"id":"openstack%2Fmistral~1002463","triplet_id":"openstack%2Fmistral~master~Ib882c46272a56080cffa0e79baf71d2b061f32bb","project":"openstack/mistral","branch":"master","hashtags":[],"change_id":"Ib882c46272a56080cffa0e79baf71d2b061f32bb","subject":"Surface typed SSH ActionException errors to the caller","status":"MERGED","created":"2026-08-26 10:09:46.000000000","updated":"2026-08-26 12:46:29.000000000","submitted":"2026-08-26 12:45:07.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"1002463","meta_rev_id":"402e55b9bceb209de73a5bf7f1083c8bfcbb2f4e","_number":1002463,"virtual_id_number":1002463,"owner":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2026-08-26 12:45:06.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2026-08-26 11:58:33.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2026-08-26 11:58:33.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-08-26 10:54:27.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"3a461d120179cee7d7b0347ee5308213e26e5a76","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"date":"2026-08-26 10:09:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"aa6ebe0a245db4044f7a4020b4c07dd7fc5e5106","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-26 10:54:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/74a997b6a0a2488db5faeeae3a820ade\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d47eb80974fe4be98792cb7cdb07e049 : SUCCESS in 3m 56s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/66675b8952814cacb5e187f100bd3980 : SUCCESS in 10m 26s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/e38d6245ff0548e9b27e1173da4f8d44 : SUCCESS in 10m 05s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/28901733dc834a9f96f625530483d6a4 : SUCCESS in 12m 56s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8518f406672547f88a5a385fe0c907be : SUCCESS in 8m 29s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7104b92b513249ea8e687472d6cc87ec : SUCCESS in 4m 37s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/4898d591abee4b62bcda872434b7b046 : SUCCESS in 16m 05s (non-voting)\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/2ce7a65f5a964060a8b2eac2ca548886 : SUCCESS in 44m 02s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/bcf53a57030c411484e63ae5d7af2a92 : SUCCESS in 43m 30s","accounts_in_message":[],"_revision_number":1},{"id":"143cec4eef759c12c6cb7304e9e41ac09dc59846","author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"date":"2026-08-26 11:58:33.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"a22125749da20ae27e5521b9711360a3c8a8f34b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-26 11:59:01.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"9c95e3e059a151a9f30317d5114caa2fc769c6fb","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-26 12:45:06.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/e1351ba6e69a443ba49dcfb389106093\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f8e6f46c441b4f2783e43e3a805d9a9c : SUCCESS in 3m 52s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/1bb18fb6e6ae496082ae96f04f5b1bae : SUCCESS in 9m 44s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/f36b95fa2766406dbf9477fd8ca00b65 : SUCCESS in 10m 02s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4132eda6e6d44a638d45c018347f7eb8 : SUCCESS in 4m 50s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/8e03b2cf6dd641f1ba8dc2ffb9933ebb : SUCCESS in 5m 22s\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/7cf8e50b07104efcacf16c6c96526e2d : SUCCESS in 30m 28s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/1ebea6d3291d4b579f8c643ef9f39969 : SUCCESS in 40m 40s","accounts_in_message":[],"_revision_number":1},{"id":"7f81bb26e451c76c9d5be972e71f743bf885bc65","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-26 12:45:07.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":1},{"id":"402e55b9bceb209de73a5bf7f1083c8bfcbb2f4e","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-26 12:46:29.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/0e06ce65a4f5400081e8ca571f5ddc7f\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/45aceb41cb6b448abb20c5981972f425 : SUCCESS in 42s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3f53fb605cb77ad8c7ceddd6de94e140c39b4fd2","revisions":{"3f53fb605cb77ad8c7ceddd6de94e140c39b4fd2":{"kind":"REWORK","_number":1,"created":"2026-08-26 10:09:46.000000000","uploader":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"ref":"refs/changes/63/1002463/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/63/1002463/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/63/1002463/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/63/1002463/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/63/1002463/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/63/1002463/1"}}},"commit":{"parents":[{"commit":"3422e078abaeb1257d986369007966768c10d4c5","subject":"Do not leak the evaluation context in expression errors","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/3422e078abaeb1257d986369007966768c10d4c5"}]}],"author":{"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","date":"2026-08-25 08:42:22.000000000","tz":120},"committer":{"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","date":"2026-08-26 10:08:43.000000000","tz":120},"subject":"Surface typed SSH ActionException errors to the caller","message":"Surface typed SSH ActionException errors to the caller\n\nSSHAction.run() caught every exception and re-raised a generic\n\"Failed to execute ssh cmd\" message, deliberately masking the real\ncause to avoid leaking SSH/host details to the caller. That masking\nalso swallowed any deliberate, typed ActionException whose message is\nsafe and intended for the caller.\n\nLet an already typed ActionException propagate as-is (the same way the\nexpression evaluators keep their already-typed errors), while keeping\nthe generic masking for the underlying SSH/host errors that may leak\ninformation.\n\nSigned-off-by: Arnaud Morin \u003carnaud.morin@gmail.com\u003e\nChange-Id: Ib882c46272a56080cffa0e79baf71d2b061f32bb\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/3f53fb605cb77ad8c7ceddd6de94e140c39b4fd2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/3f53fb605cb77ad8c7ceddd6de94e140c39b4fd2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dMAX"],"failing_atoms":["label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dMAX"],"failing_atoms":["label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}