)]}'
{"id":"openstack%2Fironic-python-agent~443649","triplet_id":"openstack%2Fironic-python-agent~master~I10ca70d6a390ed802505c0d10d440dfb52beb56c","project":"openstack/ironic-python-agent","branch":"master","topic":"bug/1670916","hashtags":[],"change_id":"I10ca70d6a390ed802505c0d10d440dfb52beb56c","subject":"Fix waiting for target disk to appear","status":"MERGED","created":"2017-03-09 14:04:41.000000000","updated":"2017-10-31 14:20:46.000000000","submitted":"2017-10-17 14:24:05.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":13,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"443649-1508250245395-2a1f4ec0","meta_rev_id":"e3317438272924153cc31f16d0ad7a88c5050799","_number":443649,"virtual_id_number":443649,"owner":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},{"value":2,"date":"2017-10-17 14:24:05.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"value":0,"date":"2017-10-17 11:15:19.000000000","_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":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":2,"date":"2017-10-17 13:04:42.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":2,"date":"2017-10-16 23:46:29.000000000","_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"value":0,"_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":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":1,"date":"2017-10-17 13:04:42.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"value":0,"_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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-03-10 02:49:55.000000000","updated_by":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"reviewer":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"state":"REVIEWER"},{"updated":"2017-03-10 08:29:04.000000000","updated_by":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"reviewer":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"state":"REVIEWER"},{"updated":"2017-10-09 19:20:14.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-10-16 23:46:29.000000000","updated_by":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"reviewer":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"state":"REVIEWER"},{"updated":"2017-10-17 11:15:19.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":"2017-10-17 13:04:42.000000000","updated_by":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"reviewer":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"state":"REVIEWER"},{"updated":"2017-10-17 14:24:05.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":"10c74fba74027966f2754f707934f6d835351661","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-09 14:04:41.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"dbdc5b5a2d4d87ed7e181eb065fa7e6dbf1fa82b","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-09 14:04:59.000000000","message":"Patch Set 1: Workflow-1\n\nTesting","accounts_in_message":[],"_revision_number":1},{"id":"4f413eea55f6df0fd8979e9d98d97a78d8f481c2","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-09 14:51:37.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"311d6fe2bf7111d08435f858147fdb81a0d1b939","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-09 15:55:25.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/49/443649/2/check/gate-ironic-python-agent-docs-ubuntu-xenial/ad0b7a8//doc/build/html/ : SUCCESS in 4m 24s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-ironic-python-agent-pep8-ubuntu-xenial/6da355f/ : SUCCESS in 3m 50s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-ironic-python-agent-python27-ubuntu-xenial/ed77ee6/ : SUCCESS in 2m 32s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/49/443649/2/check/gate-ironic-python-agent-python35/6249fa3/ : SUCCESS in 1m 52s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/dd7e7de/ : FAILURE in 1m 59s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/8cbaef1/ : SUCCESS in 47m 04s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/b235f89/ : SUCCESS in 36m 57s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/2cdec26/ : SUCCESS in 47m 32s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/07c1a86/ : SUCCESS in 35m 56s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/0e52407/ : SUCCESS in 46m 39s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/ba1e733/ : SUCCESS in 33m 21s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/0ed27b0/ : SUCCESS in 45m 10s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/a80e508/ : SUCCESS in 34m 26s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/49/443649/2/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/bd7de48/ : SUCCESS in 1h 00m 40s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"1027f714c2b8c09d085c9f86f4ee150307503aa7","author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"date":"2017-03-10 02:49:55.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"f601b61ba5a9b46273b1989db50deb1551147b5f","author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"date":"2017-03-10 02:50:48.000000000","message":"Patch Set 2:\n\nThank you. LGTM","accounts_in_message":[],"_revision_number":2},{"id":"898c54fc7c8a38186cea28a15186e0fb84496068","author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"date":"2017-03-10 08:29:04.000000000","message":"Patch Set 2: Code-Review-1\n\nLucas, thanks for the patch, looks good!\n-1 as we need a release notes for this.","accounts_in_message":[],"_revision_number":2},{"id":"d814ef4697bf57f930a4fc5c8ddc7590c296ce11","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-10 10:23:40.000000000","message":"Patch Set 2: Workflow-1\n\nOps, thanks Vasyl and Shivanand for the review. I was suppose to mark this as WIP because I\u0027m testing it in a real env where the nodes has many disks.","accounts_in_message":[],"_revision_number":2},{"id":"c2bb98fd4e89a1a6373cd3874d449d50a88ba92a","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-03-10 13:10:55.000000000","message":"Patch Set 2: Code-Review-1\n\nI was considering it, but waiting for all disks is also required for inspection. If we do this, we need to update inspector.py to wait for all disks..","accounts_in_message":[],"_revision_number":2},{"id":"2d9e30100fcc19d903ba1e504bbdcfde32354b40","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-10 13:15:37.000000000","message":"Patch Set 2:\n\n@Dmitry,\n\nNot sure if I get it. It wasn\u0027t waiting for all disks before and neither is today. Before it always waited for any disk to show up, now if the root device hints is set it tries to wait for that specific disk to show up (otherwise it would wait for any, like before).","accounts_in_message":[],"_revision_number":2},{"id":"1b537429509fbd3367a0307e29146c38eacd8a4c","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-10 13:15:56.000000000","message":"Patch Set 2:\n\ns/today/now","accounts_in_message":[],"_revision_number":2},{"id":"cb652edd2e4352b6919258c4bf0d7c3975dba9f6","author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"date":"2017-03-10 13:21:33.000000000","message":"Patch Set 2:\n\n\u003e @Dmitry,\n \u003e \n \u003e Not sure if I get it. It wasn\u0027t waiting for all disks before and\n \u003e neither is today. Before it always waited for any disk to show up,\n \u003e now if the root device hints is set it tries to wait for that\n \u003e specific disk to show up (otherwise it would wait for any, like\n \u003e before).\n\nmight be consider make this configurable, and allow to set something like \u0027rootdelay\u0027 and sleep for that time before doing any actions related to disks?\n\nhttp://www.linuxtopia.org/online_books/linux_kernel/kernel_configuration/re58.html","accounts_in_message":[],"_revision_number":2},{"id":"f0e831dc99050bb32d671b2268508c92d03bdf83","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-03-10 13:21:50.000000000","message":"Patch Set 2: -Code-Review\n\nTrue. I can\u0027t remember why I ruled out using this call.","accounts_in_message":[],"_revision_number":2},{"id":"cc9216d3ef05b312509511eea57c2645641a6fa3","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-10 13:31:15.000000000","message":"Patch Set 2:\n\nHi Vasyl,\n\nI think it already is today [0] (at least partially), you can pass \"ipa-disk-wait-attempts\" and \"ipa-disk-wait-delay\" in the kernel cmd line when waiting the the disks to show up.\n\nOr you mean having a configuration which will sleep X number of seconds before even attempting to invoke _wait_for_disks() ?\n\n[0] https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L171-L185","accounts_in_message":[],"_revision_number":2},{"id":"b18034a45db15ee5ba72ad903542391905dc6ceb","author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"date":"2017-03-10 14:44:46.000000000","message":"Patch Set 2:\n\n\u003e Hi Vasyl,\n \u003e \n \u003e I think it already is today [0] (at least partially), you can pass\n \u003e \"ipa-disk-wait-attempts\" and \"ipa-disk-wait-delay\" in the kernel\n \u003e cmd line when waiting the the disks to show up.\n \u003e \n \u003e Or you mean having a configuration which will sleep X number of\n \u003e seconds before even attempting to invoke _wait_for_disks() ?\n \u003e \n \u003e [0] https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L171-L185\n\nyes, I meant that we need a delay before attempt to invoke _wait_for_disks()","accounts_in_message":[],"_revision_number":2},{"id":"fd470d0e6b25a54eb777d8b15a0d29a145c2f44a","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-03-10 16:24:27.000000000","message":"Patch Set 2:\n\n\u003e \u003e Hi Vasyl,\n \u003e \u003e\n \u003e \u003e I think it already is today [0] (at least partially), you can\n \u003e pass\n \u003e \u003e \"ipa-disk-wait-attempts\" and \"ipa-disk-wait-delay\" in the kernel\n \u003e \u003e cmd line when waiting the the disks to show up.\n \u003e \u003e\n \u003e \u003e Or you mean having a configuration which will sleep X number of\n \u003e \u003e seconds before even attempting to invoke _wait_for_disks() ?\n \u003e \u003e\n \u003e \u003e [0] https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L171-L185\n \u003e \n \u003e yes, I meant that we need a delay before attempt to invoke\n \u003e _wait_for_disks()\n\nI see, right I don\u0027t mind adding it. That would indeed make things more flexible. I just wonder if we need to enable it by default, cause we are seeming this problem with the disks not showing up but the nodes we are using has 30+ disks in them. So seems to be an edge case (but need a fix anyway)","accounts_in_message":[],"_revision_number":2},{"id":"8d929f8c0fbdecb43001dd88c7f0039ea45dc9b6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-15 04:26:16.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"7e57a72cd583f59a751d8686d010fb5c54eef07d","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-05-18 14:34:35.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"496aef76e095d09dfba5964c2257314ed7c7dfbe","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-05-18 15:12:37.000000000","message":"Patch Set 3: Code-Review-1\n\nOk, I remembered what the problem was. We load hardware managers very early in IPA start up, much before we know the root device hints. So this change does not change anything. I suspect we need to add retries to get_os_install_device instead..","accounts_in_message":[],"_revision_number":3},{"id":"2276a06aa54b9ea565bf7f2c9d209ebd4d413027","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-18 15:45:08.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/49/443649/3/check/gate-ironic-python-agent-docs-ubuntu-xenial/4dd6ee3//doc/build/html/ : SUCCESS in 3m 19s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-ironic-python-agent-pep8-ubuntu-xenial/a896d96/ : SUCCESS in 3m 19s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-ironic-python-agent-python27-ubuntu-xenial/bab791c/ : SUCCESS in 3m 17s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/49/443649/3/check/gate-ironic-python-agent-python35/41332a0/ : SUCCESS in 2m 52s\n- gate-ironic-python-agent-releasenotes http://docs-draft.openstack.org/49/443649/3/check/gate-ironic-python-agent-releasenotes/d1962dd//releasenotes/build/html/ : SUCCESS in 2m 52s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/b913302/ : FAILURE in 3m 04s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/3d56a59/ : SUCCESS in 43m 59s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/7688932/ : FAILURE in 1h 02m 58s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/0684c4b/ : SUCCESS in 33m 26s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/60e9c63/ : SUCCESS in 54m 58s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/1876a8b/ : SUCCESS in 35m 42s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/e513bbb/ : SUCCESS in 43m 09s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/1d925f3/ : SUCCESS in 47m 06s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/760f721/ : SUCCESS in 47m 14s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/49/443649/3/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/f58296a/ : SUCCESS in 1h 08m 07s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"e56289dcfa3673185cf7d94000fbc586abfa8e36","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-07-12 12:21:17.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":3},{"id":"4b316cf95f9922049ff05c2ba93761d0e7676c3f","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-09 11:52:13.000000000","message":"Restored\n\nI\u0027ll give this a go","accounts_in_message":[],"_revision_number":3},{"id":"3634d216287f6d0a672adba4122dca956d31e496","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-09 11:52:24.000000000","message":"Patch Set 3: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":3},{"id":"4aac7f2d88bbc16e8cb338ef38afdedf6c048119","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-10-09 11:53:47.000000000","message":"Patch Set 3:\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":3},{"id":"535ade6e8c6976324e7457a468165583924b95d5","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-09 13:10:51.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"1b4fd86bb474feb5ce94d1a9c8b2889fa0b40098","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-10-09 15:01:18.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/49/443649/4/check/gate-ironic-python-agent-docs-ubuntu-xenial/9b74168//doc/build/html/ : SUCCESS in 3m 57s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-ironic-python-agent-pep8-ubuntu-xenial/d9000f5/ : SUCCESS in 5m 11s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-ironic-python-agent-python27-ubuntu-xenial/9adcf55/ : SUCCESS in 5m 15s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/49/443649/4/check/gate-ironic-python-agent-python35/a6c1dfd/ : SUCCESS in 4m 47s\n- gate-ironic-python-agent-releasenotes http://docs-draft.openstack.org/49/443649/4/check/gate-ironic-python-agent-releasenotes/5688408//releasenotes/build/html/ : SUCCESS in 4m 49s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/8dc6b34/ : FAILURE in 3m 02s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/1d73752/ : SUCCESS in 48m 09s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/fc13948/ : SUCCESS in 59m 24s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/def3573/ : SUCCESS in 55m 39s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/f6f4712/ : SUCCESS in 55m 37s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/f2acf7a/ : SUCCESS in 45m 27s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/525f5eb/ : SUCCESS in 52m 13s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/36c656a/ : SUCCESS in 48m 08s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/d40b76c/ : SUCCESS in 58m 48s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/49/443649/4/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/a2fb20d/ : SUCCESS in 1h 12m 09s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"f3e161fe07eae2c2ae337fc566394bece7cf12f3","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-09 15:04:59.000000000","message":"Patch Set 4: Workflow-1\n\nbreaks functional tests","accounts_in_message":[],"_revision_number":4},{"id":"2d5a5ef8fc91d85ce57c9d2ab09674192b15cc87","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-09 15:53:59.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"5362f832cbd5ef73a373db9f86c7e0ad48a635f0","author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"date":"2017-10-09 16:04:03.000000000","message":"Patch Set 5: Code-Review+2\n\nLGTM","accounts_in_message":[],"_revision_number":5},{"id":"9c1e450306c0e41e24ef81cc960782c2f900d9db","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-10-09 19:20:14.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/49/443649/5/check/gate-ironic-python-agent-docs-ubuntu-xenial/20d9f1e//doc/build/html/ : SUCCESS in 3m 37s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-ironic-python-agent-pep8-ubuntu-xenial/7a148f4/ : SUCCESS in 4m 00s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-ironic-python-agent-python27-ubuntu-xenial/da9ac1a/ : SUCCESS in 3m 08s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/49/443649/5/check/gate-ironic-python-agent-python35/0322ba7/ : SUCCESS in 3m 46s\n- gate-ironic-python-agent-releasenotes http://docs-draft.openstack.org/49/443649/5/check/gate-ironic-python-agent-releasenotes/c4356a0//releasenotes/build/html/ : SUCCESS in 3m 53s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/f40d1e9/ : SUCCESS in 5m 03s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/7514f71/ : SUCCESS in 1h 05m 57s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/5e0bf28/ : SUCCESS in 1h 09m 37s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/f3f4d91/ : SUCCESS in 46m 37s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/33538a4/ : SUCCESS in 46m 07s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/f9c25a3/ : SUCCESS in 46m 32s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/edcde43/ : SUCCESS in 51m 07s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/a6cd215/ : SUCCESS in 48m 37s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/1e65d1e/ : SUCCESS in 1h 05m 11s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/49/443649/5/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/972a966/ : SUCCESS in 1h 08m 56s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"ddff7bb05b61e3d724a8b1b329f9bdd2a19794c8","author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"date":"2017-10-09 23:54:21.000000000","message":"Patch Set 5:\n\n(4 comments)","accounts_in_message":[],"_revision_number":5},{"id":"76ef30d6bfa45e22ea2209da0784ab51ccab2efb","author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"date":"2017-10-09 23:54:37.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"48b3f26d80561c3899d8a685e558430175908a09","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-10 17:41:08.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed.\n\n- openstack-tox-pep8 http://logs.openstack.org/49/443649/5/check/openstack-tox-pep8/9f81417/ : SUCCESS in 4m 07s\n- openstack-tox-py27 http://logs.openstack.org/49/443649/5/check/openstack-tox-py27/4616b8e/ : SUCCESS in 4m 05s\n- openstack-tox-py35 http://logs.openstack.org/49/443649/5/check/openstack-tox-py35/373a059/ : SUCCESS in 4m 07s\n- build-openstack-sphinx-docs http://logs.openstack.org/49/443649/5/check/build-openstack-sphinx-docs/eeb9d24/html/ : SUCCESS in 4m 02s\n- build-openstack-releasenotes http://logs.openstack.org/49/443649/5/check/build-openstack-releasenotes/c8c26b3/html/ : SUCCESS in 3m 40s\n- legacy-ironic-python-agent-tox-func http://logs.openstack.org/49/443649/5/check/legacy-ironic-python-agent-tox-func/eace845/ : SUCCESS in 5m 39s (non-voting)\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src/734a5c2/ : SUCCESS in 54m 19s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src/98d2b9c/ : FAILURE in 1h 22m 58s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src/9a477ff/ : SUCCESS in 49m 18s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src/6c20fe1/ : SUCCESS in 48m 44s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src/30d2e06/ : SUCCESS in 47m 34s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src/bf99803/ : SUCCESS in 51m 31s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src/77b79a3/ : SUCCESS in 55m 03s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src/cb934db/ : SUCCESS in 54m 05s\n- legacy-tempest-dsvm-ironic-inspector-src http://logs.openstack.org/49/443649/5/check/legacy-tempest-dsvm-ironic-inspector-src/2a5b214/ : SUCCESS in 1h 09m 50s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"28b2f86dca5d1e469b34602a5375b3f3e16e200d","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-16 13:38:19.000000000","message":"Patch Set 5: Workflow-1\n\n(4 comments)\n\nwill fix","accounts_in_message":[],"_revision_number":5},{"id":"f4e4bacb60cff6254a5dafd6268581caded19f46","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-16 13:42:01.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"3bcee6cf64450529dcba9e6dc92b761e39ea0b95","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-16 14:56:43.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed.\n\n- openstack-tox-pep8 http://logs.openstack.org/49/443649/6/check/openstack-tox-pep8/dfd9ddd/ : SUCCESS in 3m 13s\n- openstack-tox-py27 http://logs.openstack.org/49/443649/6/check/openstack-tox-py27/d5f9240/ : SUCCESS in 4m 02s\n- openstack-tox-py35 http://logs.openstack.org/49/443649/6/check/openstack-tox-py35/b0f10d3/ : SUCCESS in 2m 47s\n- build-openstack-sphinx-docs http://logs.openstack.org/49/443649/6/check/build-openstack-sphinx-docs/7997202/html/ : SUCCESS in 3m 01s\n- build-openstack-releasenotes http://logs.openstack.org/49/443649/6/check/build-openstack-releasenotes/a778655/ : RETRY_LIMIT in 2m 35s\n- legacy-ironic-python-agent-tox-func http://logs.openstack.org/49/443649/6/check/legacy-ironic-python-agent-tox-func/8939b2a/ : SUCCESS in 2m 01s (non-voting)\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src/4d803c7/ : SUCCESS in 56m 29s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src/11db9d5/ : SUCCESS in 52m 11s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src/5010196/ : SUCCESS in 51m 09s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src/d468b56/ : SUCCESS in 54m 29s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src/4c06c72/ : SUCCESS in 48m 48s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src/46e15fa/ : SUCCESS in 53m 21s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src/95fbc72/ : SUCCESS in 51m 30s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src/b0664e1/ : SUCCESS in 1h 00m 43s\n- legacy-tempest-dsvm-ironic-inspector-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-inspector-src/23bd99b/ : SUCCESS in 1h 09m 08s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"c5d66edb2eadd2a5c142eb8156fb68884381a03b","author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"date":"2017-10-16 17:19:11.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"165944512e531ea881bcb40743577c10b519a276","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-16 17:20:51.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nrecheck RETRY_LIMIT","accounts_in_message":[],"_revision_number":6},{"id":"ac70cb31cdcc55364e428acd84db443977b308d8","author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"date":"2017-10-16 23:46:29.000000000","message":"Patch Set 6: Code-Review+2","accounts_in_message":[],"_revision_number":6},{"id":"555edb1a79884e13be59e1d390b905198d802b59","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-17 08:45:06.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"2b8b4fc81dbfab59159a9899c4c8e020d8567ed5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-17 10:02:19.000000000","message":"Patch Set 6:\n\nBuild failed.\n\n- openstack-tox-pep8 http://logs.openstack.org/49/443649/6/check/openstack-tox-pep8/75b9f16/ : SUCCESS in 3m 03s\n- openstack-tox-py27 http://logs.openstack.org/49/443649/6/check/openstack-tox-py27/786ac98/ : SUCCESS in 2m 16s\n- openstack-tox-py35 http://logs.openstack.org/49/443649/6/check/openstack-tox-py35/99944dc/ : SUCCESS in 2m 23s\n- build-openstack-sphinx-docs http://logs.openstack.org/49/443649/6/check/build-openstack-sphinx-docs/d596312/html/ : SUCCESS in 2m 37s\n- build-openstack-releasenotes http://logs.openstack.org/49/443649/6/check/build-openstack-releasenotes/fdc9c15/ : POST_FAILURE in 2m 37s\n- legacy-ironic-python-agent-tox-func http://logs.openstack.org/49/443649/6/check/legacy-ironic-python-agent-tox-func/c0d838d/ : SUCCESS in 3m 25s (non-voting)\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src/0e5a1e0/ : SUCCESS in 50m 41s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src/67370ac/ : SUCCESS in 52m 55s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src/b1320a7/ : SUCCESS in 52m 22s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src/f93eaf1/ : FAILURE in 35m 03s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src/2658d8f/ : SUCCESS in 1h 15m 25s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src/ddb3984/ : SUCCESS in 53m 06s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src/95f7cbe/ : SUCCESS in 1h 05m 46s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src/16f6167/ : SUCCESS in 55m 36s\n- legacy-tempest-dsvm-ironic-inspector-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-inspector-src/ba1f153/ : SUCCESS in 1h 14m 29s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"62361540537c31c4c178f3a608354e60db0272ba","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-17 11:15:19.000000000","message":"Patch Set 6:\n\nrecheck the fix was merged","accounts_in_message":[],"_revision_number":6},{"id":"f9c1268753e014a4a9678dd701d8eb02428d01c8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-17 12:37:43.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded.\n\n- openstack-tox-pep8 http://logs.openstack.org/49/443649/6/check/openstack-tox-pep8/428ede7/ : SUCCESS in 3m 09s\n- openstack-tox-py27 http://logs.openstack.org/49/443649/6/check/openstack-tox-py27/f690b82/ : SUCCESS in 2m 10s\n- openstack-tox-py35 http://logs.openstack.org/49/443649/6/check/openstack-tox-py35/b338783/ : SUCCESS in 3m 23s\n- build-openstack-sphinx-docs http://logs.openstack.org/49/443649/6/check/build-openstack-sphinx-docs/5125887/html/ : SUCCESS in 2m 38s\n- build-openstack-releasenotes http://logs.openstack.org/49/443649/6/check/build-openstack-releasenotes/3461d18/html/ : SUCCESS in 2m 46s\n- legacy-ironic-python-agent-tox-func http://logs.openstack.org/49/443649/6/check/legacy-ironic-python-agent-tox-func/e084ddc/ : SUCCESS in 2m 30s (non-voting)\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src/4f6f5ba/ : SUCCESS in 56m 08s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src/b7ca6f3/ : SUCCESS in 1h 06m 32s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src/60f5b52/ : SUCCESS in 52m 08s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src/e696a23/ : SUCCESS in 49m 38s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src/6bf8346/ : SUCCESS in 49m 10s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src/0d7ac35/ : SUCCESS in 55m 56s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src/3d5d9a2/ : SUCCESS in 54m 40s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src/18cf447/ : SUCCESS in 49m 56s\n- legacy-tempest-dsvm-ironic-inspector-src http://logs.openstack.org/49/443649/6/check/legacy-tempest-dsvm-ironic-inspector-src/f2d4ce7/ : SUCCESS in 1h 15m 41s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"af61d98527797faacc393bf823345074e24f0ad7","author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"date":"2017-10-17 13:04:42.000000000","message":"Patch Set 6: Workflow+1 Code-Review+2","accounts_in_message":[],"_revision_number":6},{"id":"202bbfc1cb9834c26c9044d68a04e63a52ce7fa9","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-17 13:04:54.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"aed1c7e5b5aabf36f570015b39dbdb1994883d98","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-17 14:24:05.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":6},{"id":"af861e824631bb167cfcf99c8596ac8b0e9df9e0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-17 14:24:05.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/49/443649/6/gate/openstack-tox-pep8/0981d26/ : SUCCESS in 2m 39s\n- openstack-tox-py27 http://logs.openstack.org/49/443649/6/gate/openstack-tox-py27/85c3fa4/ : SUCCESS in 2m 54s\n- openstack-tox-py35 http://logs.openstack.org/49/443649/6/gate/openstack-tox-py35/3c80fe5/ : SUCCESS in 2m 41s\n- build-openstack-sphinx-docs http://logs.openstack.org/49/443649/6/gate/build-openstack-sphinx-docs/7bc89f5/html/ : SUCCESS in 2m 45s\n- build-openstack-releasenotes http://logs.openstack.org/49/443649/6/gate/build-openstack-releasenotes/5ecbb65/html/ : SUCCESS in 2m 30s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src/b5dabec/ : SUCCESS in 1h 04m 31s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src/45708d5/ : SUCCESS in 1h 06m 30s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src/cb3e359/ : SUCCESS in 1h 00m 45s\n- legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src/420f7e8/ : SUCCESS in 53m 19s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src/85072d5/ : SUCCESS in 50m 52s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src/997bbf3/ : SUCCESS in 1h 03m 04s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src/942eb36/ : SUCCESS in 55m 16s\n- legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src http://logs.openstack.org/49/443649/6/gate/legacy-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src/57a6825/ : SUCCESS in 51m 17s","accounts_in_message":[],"_revision_number":6},{"id":"d8d7b0716694c846f268922b2ddefa535631f2fd","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-17 14:25:05.000000000","message":"Patch Set 6: Cherry Picked\n\nThis patchset was cherry picked to branch stable/pike as commit bd8c6c7420c5a913ff407da569c1b8dfbc0a488a","accounts_in_message":[],"_revision_number":6},{"id":"ee69452df70f440a78d2094f51383958de76ead8","author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"date":"2017-10-17 17:19:26.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"4137b2dfcc14fee1bd8645ba5a11090d5df4e456","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2017-10-18 08:53:45.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"c3bf25097a8ec57008044c75fdb422ece9fc4d2e","author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"date":"2017-10-31 14:20:46.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"3189c16a5e95ade468fa8bc37302eb9979f5a8c9","revisions":{"4968f234c073790775d82edad7a0763c61e2b450":{"kind":"REWORK","_number":1,"created":"2017-03-09 14:04:41.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/49/443649/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/1"}}},"commit":{"parents":[{"commit":"9b77ba530a6038b31ca503bc088100d4d0bc2160","subject":"Merge \"imagebuild: coreos: Build sha256 checksum files\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/9b77ba530a6038b31ca503bc088100d4d0bc2160"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"subject":"Change the _wait_for_disks behavior to wait for a specific disk","message":"Change the _wait_for_disks behavior to wait for a specific disk\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/4968f234c073790775d82edad7a0763c61e2b450"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/4968f234c073790775d82edad7a0763c61e2b450"}]},"branch":"refs/heads/master"},"f471d4cc630fdfa02c9a11cc78611b0c30c2986f":{"kind":"REWORK","_number":2,"created":"2017-03-09 14:51:37.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/49/443649/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/2"}}},"commit":{"parents":[{"commit":"9b77ba530a6038b31ca503bc088100d4d0bc2160","subject":"Merge \"imagebuild: coreos: Build sha256 checksum files\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/9b77ba530a6038b31ca503bc088100d4d0bc2160"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:48:11.000000000","tz":0},"subject":"Change the _wait_for_disks behavior to wait for a specific disk","message":"Change the _wait_for_disks behavior to wait for a specific disk\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/f471d4cc630fdfa02c9a11cc78611b0c30c2986f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/f471d4cc630fdfa02c9a11cc78611b0c30c2986f"}]},"branch":"refs/heads/master"},"ca8702d9109d154519678cf7a1a91651e5a91141":{"kind":"REWORK","_number":3,"created":"2017-05-18 14:34:35.000000000","uploader":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"ref":"refs/changes/49/443649/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/3"}}},"commit":{"parents":[{"commit":"15878b7b181098db7abb6601ada0e245b6722d46","subject":"Merge \"Remove unused function _configdrive_location\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/15878b7b181098db7abb6601ada0e245b6722d46"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Dmitry Tantsur","email":"divius.inside@gmail.com","date":"2017-05-18 14:33:53.000000000","tz":120},"subject":"Change the _wait_for_disks behavior to wait for a specific disk","message":"Change the _wait_for_disks behavior to wait for a specific disk\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nCo-Authored-By: Dmitry Tantsur \u003cdtantsur@redhat.com\u003e\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/ca8702d9109d154519678cf7a1a91651e5a91141"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/ca8702d9109d154519678cf7a1a91651e5a91141"}]},"branch":"refs/heads/master"},"b02a633359f8fa863bce53dceb4044845636f8cf":{"kind":"REWORK","_number":4,"created":"2017-10-09 13:10:51.000000000","uploader":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"ref":"refs/changes/49/443649/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/4"}}},"commit":{"parents":[{"commit":"e87d03911139d84230fba4f8c1af048be57153e9","subject":"Merge \"Migrate to stestr as unit tests runner\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/e87d03911139d84230fba4f8c1af048be57153e9"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Dmitry Tantsur","email":"divius.inside@gmail.com","date":"2017-10-09 13:10:29.000000000","tz":120},"subject":"Fix waiting for target disk to appear","message":"Fix waiting for target disk to appear\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nThe _wait_for_disks call was made into a proper hardware manager method.\nIt is now also called each time the cached node is updated, not only\non start up. This is to ensure that we wait for the device, matching\nroot device hints (which are part of the node).\n\nFinally, this patch adds more logging around detecting the root device.\n\nCo-Authored-By: Dmitry Tantsur \u003cdtantsur@redhat.com\u003e\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/b02a633359f8fa863bce53dceb4044845636f8cf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/b02a633359f8fa863bce53dceb4044845636f8cf"}]},"branch":"refs/heads/master"},"ef0c6f701e9e9a166d2a040625bab94fa3fa354c":{"kind":"REWORK","_number":5,"created":"2017-10-09 15:53:59.000000000","uploader":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"ref":"refs/changes/49/443649/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/5"}}},"commit":{"parents":[{"commit":"e87d03911139d84230fba4f8c1af048be57153e9","subject":"Merge \"Migrate to stestr as unit tests runner\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/e87d03911139d84230fba4f8c1af048be57153e9"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Dmitry Tantsur","email":"divius.inside@gmail.com","date":"2017-10-09 15:53:42.000000000","tz":120},"subject":"Fix waiting for target disk to appear","message":"Fix waiting for target disk to appear\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nThe _wait_for_disks call was made into a proper hardware manager method.\nIt is now also called each time the cached node is updated, not only\non start up. This is to ensure that we wait for the device, matching\nroot device hints (which are part of the node).\n\nFinally, this patch adds more logging around detecting the root device.\n\nCo-Authored-By: Dmitry Tantsur \u003cdtantsur@redhat.com\u003e\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/ef0c6f701e9e9a166d2a040625bab94fa3fa354c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/ef0c6f701e9e9a166d2a040625bab94fa3fa354c"}]},"branch":"refs/heads/master"},"3189c16a5e95ade468fa8bc37302eb9979f5a8c9":{"kind":"REWORK","_number":6,"created":"2017-10-16 13:42:01.000000000","uploader":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"ref":"refs/changes/49/443649/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/49/443649/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/49/443649/6"}}},"commit":{"parents":[{"commit":"e87d03911139d84230fba4f8c1af048be57153e9","subject":"Merge \"Migrate to stestr as unit tests runner\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/e87d03911139d84230fba4f8c1af048be57153e9"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2017-03-09 14:02:06.000000000","tz":0},"committer":{"name":"Dmitry Tantsur","email":"divius.inside@gmail.com","date":"2017-10-16 13:39:25.000000000","tz":120},"subject":"Fix waiting for target disk to appear","message":"Fix waiting for target disk to appear\n\nThis patch is changing the _wait_for_disks() method behavior to wait to\na specific disk if any device hints is specified. There are cases where\nthe deployment might fail or succeed randomly depending on the order and\ntime that the disks shows up.\n\nIf no root device hints is specified, the method will just wait for any\nsuitable disk to show up, like before.\n\nThe _wait_for_disks call was made into a proper hardware manager method.\nIt is now also called each time the cached node is updated, not only\non start up. This is to ensure that we wait for the device, matching\nroot device hints (which are part of the node).\n\nThe loop was corrected to avoid redundant sleeps and warnings.\n\nFinally, this patch adds more logging around detecting the root device.\n\nCo-Authored-By: Dmitry Tantsur \u003cdtantsur@redhat.com\u003e\nChange-Id: I10ca70d6a390ed802505c0d10d440dfb52beb56c\nCloses-Bug: #1670916\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/3189c16a5e95ade468fa8bc37302eb9979f5a8c9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/3189c16a5e95ade468fa8bc37302eb9979f5a8c9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
