)]}'
{"id":"openstack%2Fnova~694906","triplet_id":"openstack%2Fnova~master~I121febe845e9e3906e858edbe7a34551a5c30338","project":"openstack/nova","branch":"master","topic":"bp/nova-cyborg-interaction","hashtags":[],"change_id":"I121febe845e9e3906e858edbe7a34551a5c30338","subject":"Refactor to extract Placement helper functions for functional tests.","status":"ABANDONED","created":"2019-11-19 06:05:49.000000000","updated":"2019-12-04 09:13:12.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ad2ca316d131aca21c0037cc7c9bf5ecb67b6816","_number":694906,"virtual_id_number":694906,"owner":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2019-11-19 08:31:55.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2019-11-27 02:25:15.000000000","_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"date":"2019-11-19 06:36:37.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"date":"2019-11-19 07:13:41.000000000","_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"date":"2019-11-19 08:32:50.000000000","_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"date":"2019-11-19 09:29:58.000000000","_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"date":"2019-11-19 06:53:00.000000000","_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"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":{"rejected":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":-2,"date":"2019-11-26 20:17:18.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":-1,"date":"2019-11-26 20:05:44.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"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":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-11-19 06:36:37.000000000","updated_by":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"reviewer":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"state":"REVIEWER"},{"updated":"2019-11-19 06:53:00.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"REVIEWER"},{"updated":"2019-11-19 07:13:41.000000000","updated_by":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"reviewer":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"state":"REVIEWER"},{"updated":"2019-11-19 08:31:55.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-11-19 08:32:50.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-11-19 09:29:58.000000000","updated_by":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-11-26 20:05:44.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2019-11-26 20:17:18.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"}],"messages":[{"id":"6de10616c332437bb025a7f47bfdfbec22534781","author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"date":"2019-11-19 06:05:49.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a6e179e3c828cd650cfdb3f116a4a35b6c6e85e9","author":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"date":"2019-11-19 06:36:37.000000000","message":"Patch Set 1:\n\nTesting completed on the zVM Driver CI system check-nova pipeline and failed.  To recheck only the zVM driver plugins, submit a comment with only  zvm: recheck in the comment.. Contact information: zvmosci@us.ibm.com. For information see https://wiki.openstack.org/wiki/ZVMDriver.\n\n- check-nova-master http://extbasicopstackcilog01.podc.sl.edst.ibm.com/test_logs/jenkins-check-nova-master-39247 : FAILURE in 15m 28s","accounts_in_message":[],"_revision_number":1},{"id":"b447b4369095669d0b5a6564b92cb40a5a653fa0","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-11-19 06:53:00.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/694906/1 : FAILURE in 27m 44s","accounts_in_message":[],"_revision_number":1},{"id":"8c02f73bb93c6f16c808bcd0017cdcabe8ff794f","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2019-11-19 07:13:41.000000000","message":"Patch Set 1:\n\nBuild SUCCESSFUL (check pipeline).\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/06/694906/1/check/pmem-tempest-plugin-filtered/4862618/ : SUCCESS in 1h 00m 00s","accounts_in_message":[],"_revision_number":1},{"id":"0066d79468ea28a99c361b04288f200210d11127","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-11-19 08:31:55.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/647249bbdca5471ea8b2251396fb766c : SUCCESS in 16m 07s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/729f209d027e446283ac6031644fff97 : SUCCESS in 15m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5d091faf26654dabaca167b984fc714f : SUCCESS in 10m 24s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cae83ba8c1c74674b646ad7a9a73eded : SUCCESS in 16m 29s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/72d89c4b517c46dbb4ae8a5d5fcf51c7 : SUCCESS in 15m 24s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b15833ce8a9a4c698f6551c6baf76f62 : SUCCESS in 9m 41s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/c37f7ce552a747fcbad2c32a77fbf141 : SUCCESS in 18m 28s","accounts_in_message":[],"_revision_number":1},{"id":"810dad80fe8a2bcec64b542be96ec48e3f10546d","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2019-11-19 08:32:50.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_VxFlexOS_NOVA http://publiclogs.emc.com/06/694906/1/check/EMC_VxFlexOS_NOVA/baf56b7/EMC_VxFlexOS_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC VxFlexOS CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VxFlexOS_CI","accounts_in_message":[],"_revision_number":1},{"id":"ca7cc60833f71342d2f8913a139ad4852769ab2d","author":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"date":"2019-11-19 09:29:58.000000000","message":"Patch Set 1:\n\n* nova-quobyteci-dsvm-volume http://78.46.57.153:8081/refs-changes-06-694906-1 : FAILURE \n\nSee https://wiki.openstack.org/wiki/ThirdPartySystems/Quobyte_CI for rechecking and info.","accounts_in_message":[],"_revision_number":1},{"id":"effd06c3f69a81ee77543b79d535faec4bbaac85","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-11-25 18:27:14.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"dc8a1d600a592a228aa1f5e81d83caaff4a446bf","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-11-26 20:05:44.000000000","message":"Patch Set 1: Code-Review-1\n\nWhy do we even need this? It\u0027s just noise IMO. If you like 85% of the base class for your tests but not using the other 15% then don\u0027t use the other 15% - why split everything up? I have the same thoughts about the same kind of thing being proposed here:\n\nhttps://review.opendev.org/#/c/689186/","accounts_in_message":[],"_revision_number":1},{"id":"a8481cb3955f88d7a6b7c1652cffdeceeb898998","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-11-26 20:06:03.000000000","message":"Patch Set 1:\n\nIOW, this is just unnecessary noise for this series IMO.","accounts_in_message":[],"_revision_number":1},{"id":"5452a25054766d6d374fe4d4a2a207789faf6e5b","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-11-26 20:17:18.000000000","message":"Patch Set 1: Code-Review-2\n\n\u003e Why do we even need this? It\u0027s just noise IMO. If you like 85% of\n \u003e the base class for your tests but not using the other 15% then\n \u003e don\u0027t use the other 15% - why split everything up? I have the same\n \u003e thoughts about the same kind of thing being proposed here:\n \u003e \n \u003e https://review.opendev.org/#/c/689186/\n\nAck yeah, I hadn\u0027t even gotten far enough to see why this was being done, and was just hung up on the comment that seemed to change the state of the world. I agree, this is unnecessary. I\u0027ll drop a -2 on it to make it clear, especially since there\u0027s another similar thing (probably needs a -2 as well, wanna do that one?).","accounts_in_message":[],"_revision_number":1},{"id":"f3338b9b56d0b0a4c7dbaf4cae5299799bd5a631","author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"date":"2019-11-27 02:25:15.000000000","message":"Patch Set 1:\n\n\u003e Why do we even need this? It\u0027s just noise IMO. If you like 85% of\n \u003e the base class for your tests but not using the other 15% then\n \u003e don\u0027t use the other 15% - why split everything up?\n\nBecause I am using the ServersTestBase as the base class for the tests [1], not ProviderUsageBaseTestCase. That is because:\n* ServersTestBase provides utility functions for state change, server deletion etc.\n* ServersTestBase is what is used by other test classes in this file.\n* The tests/methods in ProviderUsageBaseTestCase are oriented towards base RCs like VCPU, RAM and disk.\n\nFurther:\n* ProviderUsageBaseTestCase combines 2 different things: utilities to access Placement API, and actual tests. Separating the concerns will help future test writers too.\n* There is a precedent: InstanceHelperMixin, which does a similar separation of concerns. \n\n[1] https://review.opendev.org/#/c/631243/43/nova/tests/functional/test_servers.py@7392","accounts_in_message":[],"_revision_number":1},{"id":"4f0db1c27d8c228e9554dd6075e553e5f1b72ac4","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-11-27 14:42:34.000000000","message":"Patch Set 1:\n\n\u003e Because I am using the ServersTestBase as the base class for the\n \u003e tests [1], not ProviderUsageBaseTestCase. That is because:\n \u003e * ServersTestBase provides utility functions for state change,\n \u003e server deletion etc.\n \u003e * ServersTestBase is what is used by other test classes in this\n \u003e file.\n\n% grep ProviderUsageBase -r nova/tests/functional/test_servers.py -c\n11\n\n \u003e * The tests/methods in ProviderUsageBaseTestCase are oriented\n \u003e towards base RCs like VCPU, RAM and disk.\n \u003e \n \u003e Further:\n \u003e * ProviderUsageBaseTestCase combines 2 different things: utilities\n \u003e to access Placement API, and actual tests. Separating the concerns\n \u003e will help future test writers too.\n\nWhat tests are in ProviderUsageBasedTestCase? Because I don\u0027t see any, and it would be particularly strange to have a test with _actual_ tests in integrated_helpers.py.\n\nYou\u0027re literally just splitting apart some utility methods where you see an arbitrary line, but I don\u0027t see that it gains you anything, and it further splinters the plethora of mixins we have where it becomes difficult to know what combination is required to do anything useful. Further, the split also apparently confused you because you separated some of the utility methods from the definition of the microversion\u003d\u0027latest\u0027 which caused you to think that os-hypervisors was returning an integer ID to you. That was a direct result of you splitting out part of this test base, but not enough.","accounts_in_message":[],"_revision_number":1},{"id":"10dcbbf70910d149c1da4464cbe5d256a59918ed","author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"date":"2019-12-04 09:13:12.000000000","message":"Abandoned\n\nDropped after code review and adoption of a different approach.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"e2051544411264cc6bee73648d0704bd364473de","revisions":{"e2051544411264cc6bee73648d0704bd364473de":{"kind":"REWORK","_number":1,"created":"2019-11-19 06:05:49.000000000","uploader":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"ref":"refs/changes/06/694906/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/06/694906/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/06/694906/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/06/694906/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/06/694906/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/06/694906/1"}}},"commit":{"parents":[{"commit":"38e57fd1c1114b0a188bef08f157e9372e17a511","subject":"ksa auth conf and client for Cyborg access","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/38e57fd1c1114b0a188bef08f157e9372e17a511"}]}],"author":{"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","date":"2019-11-16 09:59:09.000000000","tz":-480},"committer":{"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","date":"2019-11-19 02:12:02.000000000","tz":-480},"subject":"Refactor to extract Placement helper functions for functional tests.","message":"Refactor to extract Placement helper functions for functional tests.\n\nChange-Id: I121febe845e9e3906e858edbe7a34551a5c30338\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2051544411264cc6bee73648d0704bd364473de"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2051544411264cc6bee73648d0704bd364473de"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
