)]}'
{"id":"openstack%2Fmetalsmith~676525","triplet_id":"openstack%2Fmetalsmith~master~Ibaeb99f37a044bbffb3afab7d9511e35a3263ab0","project":"openstack/metalsmith","branch":"master","topic":"bp/nova-less-deploy","hashtags":[],"change_id":"Ibaeb99f37a044bbffb3afab7d9511e35a3263ab0","subject":"Allow reserve_node to backfill from existing node","status":"ABANDONED","created":"2019-08-14 22:32:50.000000000","updated":"2019-08-26 23:00:01.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"bc350a85cd93d9abdbe3fa0f1308c1168da8028b","_number":676525,"virtual_id_number":676525,"owner":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"value":1,"date":"2019-08-15 23:12:08.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2019-08-19 17:50:03.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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"rejected":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"all":[{"value":-1,"date":"2019-08-26 22:09:50.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"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,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-15 23:12:08.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"},{"updated":"2019-08-19 17:50:03.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"}],"messages":[{"id":"b9163e1afca18e8df9c4632f82a2a6d2c6412b61","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-14 22:32:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"06bae48fdfffc592de0ae4f561e8487255407790","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-14 22:36:20.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"0b91ebc8057221da9d11a5bd222d3750279545c5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-14 23:59:10.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9fdc00319c7e44d2ae252e060e55bb97 : FAILURE in 2m 35s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/85b437260bab43e9a7f3b1d992f10b10 : SUCCESS in 3m 22s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/19329daa242e4eeab31d2831fda7858a : SUCCESS in 2m 44s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/24b423452eca47e6adeb2ad6ff4ba68c : SUCCESS in 3m 46s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/24b02a4f695c4227b497004dfd4550b4 : SUCCESS in 3m 29s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/e3d2633d95c84c63a01ff8f3a023f23f : SUCCESS in 4m 34s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d7ab5d39d84d44d68634329ec5a6f2f4 : SUCCESS in 3m 39s\n- metalsmith-integration-glance-localboot-centos7 https://zuul.opendev.org/t/openstack/build/40b20bfb8c3d462cb3260ebbb4e9506d : SUCCESS in 1h 16m 50s\n- metalsmith-integration-glance-netboot-cirros-iscsi-py3 https://zuul.opendev.org/t/openstack/build/364e7690d05a4e4098d8c78f7c2b67ab : SUCCESS in 48m 05s\n- metalsmith-integration-glance-netboot-cirros-direct https://zuul.opendev.org/t/openstack/build/788a7aa900084d83bb5975cd06477a67 : SUCCESS in 47m 36s\n- metalsmith-integration-http-netboot-cirros-direct-py3 https://zuul.opendev.org/t/openstack/build/7e2a400de2d54789846274545b528d2c : SUCCESS in 46m 04s","accounts_in_message":[],"_revision_number":2},{"id":"183853c04b2ef142c6436ab5ea3e2ab6177a5131","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-15 20:41:29.000000000","message":"Topic changed from nova-less-deploy to bp/nova-less-deploy","accounts_in_message":[],"_revision_number":2},{"id":"d8f7ff62b026ff57d929d08200ccbbfa976f35aa","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-15 21:43:15.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"a23247e014944eab20ab1f3bc944a3d610d31400","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-15 23:12:08.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2580bfbd72d440a882990c9bbadf0afe : SUCCESS in 5m 33s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/86baaeed542b477c95b24a2c8b5d36cf : SUCCESS in 4m 39s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/8321a94b1b7140d4afbd5f43b6a1439a : SUCCESS in 3m 41s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/7869621a0e46470880dd46a710400863 : SUCCESS in 5m 09s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ec3f2504891a4d539065e1226c13e257 : SUCCESS in 4m 27s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5c3588befe634fdf9d9315a4ebf987f0 : SUCCESS in 5m 59s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5a4aaf92e4cb429d925aae674aeadff7 : SUCCESS in 6m 37s\n- metalsmith-integration-glance-localboot-centos7 https://zuul.opendev.org/t/openstack/build/0cd2490c19a24e1eab639e84a2a4625c : SUCCESS in 1h 12m 13s\n- metalsmith-integration-glance-netboot-cirros-iscsi-py3 https://zuul.opendev.org/t/openstack/build/e0eedd73cf1f44ecae853da6448505e8 : SUCCESS in 51m 54s\n- metalsmith-integration-glance-netboot-cirros-direct https://zuul.opendev.org/t/openstack/build/e1732caad721497fa646a33590078650 : SUCCESS in 57m 33s\n- metalsmith-integration-http-netboot-cirros-direct-py3 https://zuul.opendev.org/t/openstack/build/6a5777e98e34484a8cf2a335adc1f7a8 : SUCCESS in 48m 59s","accounts_in_message":[],"_revision_number":3},{"id":"a92688ac0110e136a54d1f809f7a72270a4b744e","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2019-08-19 17:50:03.000000000","message":"Patch Set 3: Code-Review-1\n\nLet\u0027s not mix backfilling and allocations, these are different processes. We could have a separate backfill_node call, although I\u0027m not sure if it\u0027s useful to bring it in metalsmith, given that all other logic has to be skipped.","accounts_in_message":[],"_revision_number":3},{"id":"0afdc1af18200c98868854daf6cd2f61e0045a4b","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-26 22:09:50.000000000","message":"Patch Set 3: Workflow-1\n\nI\u0027m working on a change to call allocation create directly in baremetal_deploy.py, just finishing the unit tests. As soon as I\u0027ve proposed that I\u0027ll abandon this","accounts_in_message":[],"_revision_number":3},{"id":"bc350a85cd93d9abdbe3fa0f1308c1168da8028b","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2019-08-26 23:00:01.000000000","message":"Abandoned\n\nThis is no longer required https://review.opendev.org/678698","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"7d2d07afbbcc1092d26e9f1ce53437d97c3a5685","revisions":{"6937a0485393093d54b62e2ad232897172f957cc":{"kind":"REWORK","_number":1,"created":"2019-08-14 22:32:50.000000000","uploader":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"ref":"refs/changes/25/676525/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/metalsmith","ref":"refs/changes/25/676525/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/1"}}},"commit":{"parents":[{"commit":"a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e","subject":"Set useful port name when hostname is provided","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e"}]}],"author":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-14 22:19:08.000000000","tz":0},"committer":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-14 22:32:49.000000000","tz":0},"subject":"Allow reserve_node to backfill from existing node","message":"Allow reserve_node to backfill from existing node\n\nMetalsmith works fine when the allocation record is missing for an\nexisting active node, but for nova-less upgrade cases it would be nice\nif a proper allocation record can be created. This becomes possible by\nexposing the allocation create node mechanism.\n\nChange-Id: Ibaeb99f37a044bbffb3afab7d9511e35a3263ab0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/6937a0485393093d54b62e2ad232897172f957cc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/6937a0485393093d54b62e2ad232897172f957cc"}]},"branch":"refs/heads/master"},"41e9b56f79c44fb6a8092f976f38af130a3d5d88":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2019-08-14 22:36:20.000000000","uploader":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"ref":"refs/changes/25/676525/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/metalsmith","ref":"refs/changes/25/676525/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/2"}}},"commit":{"parents":[{"commit":"a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e","subject":"Set useful port name when hostname is provided","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e"}]}],"author":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-14 22:19:08.000000000","tz":0},"committer":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-14 22:35:42.000000000","tz":0},"subject":"Allow reserve_node to backfill from existing node","message":"Allow reserve_node to backfill from existing node\n\nMetalsmith works fine when the allocation record is missing for an\nexisting active node, but for nova-less upgrade cases it would be nice\nif a proper allocation record can be created. This becomes possible by\nexposing the allocation create node mechanism.\n\nChange-Id: Ibaeb99f37a044bbffb3afab7d9511e35a3263ab0\nDepends-On: https://review.opendev.org/676521\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/41e9b56f79c44fb6a8092f976f38af130a3d5d88"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/41e9b56f79c44fb6a8092f976f38af130a3d5d88"}]},"branch":"refs/heads/master"},"7d2d07afbbcc1092d26e9f1ce53437d97c3a5685":{"kind":"REWORK","_number":3,"created":"2019-08-15 21:43:15.000000000","uploader":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"ref":"refs/changes/25/676525/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/metalsmith","ref":"refs/changes/25/676525/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/metalsmith refs/changes/25/676525/3"}}},"commit":{"parents":[{"commit":"a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e","subject":"Set useful port name when hostname is provided","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e"}]}],"author":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-14 22:19:08.000000000","tz":0},"committer":{"name":"Steve Baker","email":"sbaker@redhat.com","date":"2019-08-15 21:43:04.000000000","tz":0},"subject":"Allow reserve_node to backfill from existing node","message":"Allow reserve_node to backfill from existing node\n\nMetalsmith works fine when the allocation record is missing for an\nexisting active node, but for nova-less upgrade cases it would be nice\nif a proper allocation record can be created. This becomes possible by\nexposing the allocation create node mechanism.\n\nChange-Id: Ibaeb99f37a044bbffb3afab7d9511e35a3263ab0\nDepends-On: https://review.opendev.org/676521\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/7d2d07afbbcc1092d26e9f1ce53437d97c3a5685"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/metalsmith/commit/7d2d07afbbcc1092d26e9f1ce53437d97c3a5685"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}