)]}'
{"id":"openstack%2Fironic-python-agent~745424","triplet_id":"openstack%2Fironic-python-agent~stable%2Ftrain~Ic4608b7c6ff9773d1403926efb3d59869c71343b","project":"openstack/ironic-python-agent","branch":"stable/train","hashtags":[],"change_id":"Ic4608b7c6ff9773d1403926efb3d59869c71343b","subject":"Fix TypeError on agent lookup failure","status":"MERGED","created":"2020-08-07 22:30:00.000000000","updated":"2020-08-09 12:13:40.000000000","submitted":"2020-08-09 12:12:15.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":"745424-1596975136173-c9d102ff","meta_rev_id":"2b7ac9f68662397a59e7050f06656b9e40b509fb","_number":745424,"virtual_id_number":745424,"owner":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-08-09 12:12:15.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2020-08-08 01:18:32.000000000","_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"value":2,"date":"2020-08-09 10:33:00.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"value":1,"date":"2020-08-09 10:33:00.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-08-08 01:18:32.000000000","updated_by":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"reviewer":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-08-09 10:33:00.000000000","updated_by":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"reviewer":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"state":"REVIEWER"},{"updated":"2020-08-09 12:12:15.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":"77f5ce917f11ac649c8b48cdf6939f92aad28644","author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"date":"2020-08-07 22:30:00.000000000","message":"Patch Set 1: Cherry Picked from branch stable/ussuri.","accounts_in_message":[],"_revision_number":1},{"id":"7befa9c74e6cca8369c63408efd6d3e9aca32ca7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-08 00:35:48.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5b963c9f75cd4e1eb3f87895dea1c0e1 : SUCCESS in 5m 01s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/e677759cde114c2c908b5661caae0688 : SUCCESS in 5m 37s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/8758249add81490ca7f2f88b39b39143 : SUCCESS in 4m 42s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/c050d996028c4232be585cbddd54a6a2 : SUCCESS in 5m 37s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/90ec2a89f6eb46a1977dfb423d499d00 : SUCCESS in 5m 11s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5187f98ba1b9472ab66736a9ed9f0007 : SUCCESS in 8m 51s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/f0c0ca37359e46f5b5a6b2d67fc4ec40 : SUCCESS in 4m 11s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src https://zuul.opendev.org/t/openstack/build/538b9bfaecc2496684341452eb41d1a1 : SUCCESS in 1h 21m 37s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src https://zuul.opendev.org/t/openstack/build/6e0fde38caa14503aba95fa33d4109cf : SUCCESS in 54m 17s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src https://zuul.opendev.org/t/openstack/build/73a0499433f04d1c8fa1a36a51e35488 : SUCCESS in 1h 20m 18s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src https://zuul.opendev.org/t/openstack/build/905cd3dcaf714ee1a74e4d9b10716d35 : SUCCESS in 1h 01m 51s\n- ironic-standalone-ipa-src https://zuul.opendev.org/t/openstack/build/af08c8f8b2ef41d4a932ed62a6ef4d64 : SUCCESS in 1h 16m 54s\n- ironic-python-agent-check-image-tinyipa https://zuul.opendev.org/t/openstack/build/f2224cf6ce68442480136a342dd17256 : SUCCESS in 12m 24s (non-voting)\n- ironic-python-agent-check-image-dib-centos7 https://zuul.opendev.org/t/openstack/build/769b54d9b7c042cb9ed4daec01bdfaae : SUCCESS in 10m 52s (non-voting)\n- ipa-tempest-ironic-inspector-src https://zuul.opendev.org/t/openstack/build/f51fed1e6cfc461abb7730d953aa8dc3 : SUCCESS in 2h 02m 45s (non-voting)\n- metalsmith-integration-ipa-src https://zuul.opendev.org/t/openstack/build/104ef9b58b72430eb3074c2142ea1cc9 : RETRY_LIMIT in 26m 58s (non-voting)\n- ipa-tempest-wholedisk-bios-ipmi-direct-dib-src https://zuul.opendev.org/t/openstack/build/42adbb4539374deca8c86314383cb461 : FAILURE in 1h 02m 19s (non-voting)\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/0ca7b9c953b1430ab3ffc7c6e9c4b549 : SUCCESS in 5m 00s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/8bdb048667994b3d9b21b5e761f0f971 : SUCCESS in 5m 04s\n- ipa-tox-bandit https://zuul.opendev.org/t/openstack/build/e85e9cfcd27b4d9e970cb7ebee163450 : SUCCESS in 3m 46s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"105596f5dd37db21c933a1f9309e914934392ef8","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2020-08-08 01:18:32.000000000","message":"Patch Set 1:\n\nBuild succeeded (check pipeline).\n\n- tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001 https://review.rdoproject.org/zuul/build/a3ee29e45ee5495294c6391eaf04a452 : SUCCESS in 2h 47m 35s","accounts_in_message":[],"_revision_number":1},{"id":"05d06ead87560d927d93fb736fb9e94f28bd4785","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2020-08-09 10:33:00.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"e702983c288a2c1ac62c1017afbb3b420f43ff0d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-09 10:33:13.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"53a43c558546260ab19a4452fe56d0267a924f40","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-09 12:12:15.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/bdb64cd06b6444e980231029c64a3ac5 : SUCCESS in 4m 03s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/daf438c255fb4e9390455c167f154f3b : SUCCESS in 3m 47s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a147204ed88e48c599bd45b81c9c2690 : SUCCESS in 3m 57s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a4e39260a1424b3491d1834b1c51b5a4 : SUCCESS in 4m 42s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/53e3d8b463a94211a50d853251aaaf56 : SUCCESS in 6m 11s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0cae19d2f72e4209a7df9c67e00f0ada : SUCCESS in 4m 38s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src https://zuul.opendev.org/t/openstack/build/03d950ff0beb4a4a94c8ec8034599aa3 : SUCCESS in 1h 02m 52s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src https://zuul.opendev.org/t/openstack/build/619114ad894b4998b6322d6efa8ff72f : SUCCESS in 1h 00m 14s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src https://zuul.opendev.org/t/openstack/build/4313259b160d4368bb0faf996dc5df0c : SUCCESS in 57m 15s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src https://zuul.opendev.org/t/openstack/build/4acd5d75047e4333b6531f0ae73e70db : SUCCESS in 1h 04m 35s\n- ironic-standalone-ipa-src https://zuul.opendev.org/t/openstack/build/845248b624e24045b14868226f88b675 : SUCCESS in 1h 30m 11s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/afa1b00bd26a4b40a463c3e1ff471d70 : SUCCESS in 4m 04s","accounts_in_message":[],"_revision_number":1},{"id":"fe9da445bf0d3e5af017b6f1a055a93776132a0a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-09 12:12:16.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"2b7ac9f68662397a59e7050f06656b9e40b509fb","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-09 12:13:40.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b644c949ac4d48b58d0e242e88b41af7 : SUCCESS in 55s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/3f50339b6a40463e9ddc426493740b0c : SUCCESS in 50s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"e45ee3f64221962ad243f7df8bc123510c632935","revisions":{"e45ee3f64221962ad243f7df8bc123510c632935":{"kind":"REWORK","_number":1,"created":"2020-08-07 22:30:00.000000000","uploader":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"ref":"refs/changes/24/745424/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/24/745424/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/24/745424/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/24/745424/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/24/745424/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/24/745424/1"}}},"commit":{"parents":[{"commit":"8d0c10ccc13318abba87f938aa6a3533e4a97a04","subject":"Merge \"Ignore devices with size 0 when collecting inventory\" into stable/train","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/8d0c10ccc13318abba87f938aa6a3533e4a97a04"}]}],"author":{"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","date":"2020-07-30 23:16:57.000000000","tz":-420},"committer":{"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","date":"2020-08-07 22:30:00.000000000","tz":0},"subject":"Fix TypeError on agent lookup failure","message":"Fix TypeError on agent lookup failure\n\nAgent lookups can fail as we presently use logging.exception,\nbetter known in our code as LOG.exception, which can also generate\nother fun issues on journald based systems where additional errors\ncould be raised resulting in us being unable to troubleshoot the\nthe actual issue.\n\nBecause of the mis-use of LOG.exception and the default behavior\nof the backoff retry handler, the retry logic was also not\nfunctional as any error no matter how small caused IPA to\njust exit.\n\nChange-Id: Ic4608b7c6ff9773d1403926efb3d59869c71343b\nStory: 2007968\nTask: 40465\n(cherry picked from commit 5eab9bced63b2b9a6753cbbf594dda7ef9d03a3a)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/e45ee3f64221962ad243f7df8bc123510c632935"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/e45ee3f64221962ad243f7df8bc123510c632935"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
