)]}'
{"id":"openstack%2Fwhitebox-tempest-plugin~704420","triplet_id":"openstack%2Fwhitebox-tempest-plugin~master~If9eab53861079a1ca1530f6dd56ee0f39216077f","project":"openstack/whitebox-tempest-plugin","branch":"master","topic":"inst_cpu_colocate","hashtags":[],"change_id":"If9eab53861079a1ca1530f6dd56ee0f39216077f","subject":"Added instance collocation migration test","status":"MERGED","created":"2020-01-27 20:28:57.000000000","updated":"2020-09-02 16:26:24.000000000","submitted":"2020-09-02 16:26:24.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":106,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"704420-1599063984985-33708db1","meta_rev_id":"3c3348149c684d5b0151e1bc8e2e16fc5e536e97","_number":704420,"virtual_id_number":704420,"owner":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-09-02 16:26:24.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},{"value":0,"date":"2020-09-02 15:28:57.000000000","_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"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":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},{"value":0,"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":2,"date":"2020-09-02 14:54:14.000000000","_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"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":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},{"value":0,"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":1,"date":"2020-09-02 15:22:15.000000000","_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-01-27 20:32:51.000000000","updated_by":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"reviewer":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"state":"REVIEWER"},{"updated":"2020-01-27 20:32:57.000000000","updated_by":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"reviewer":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"state":"REVIEWER"},{"updated":"2020-02-13 09:32:05.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2020-03-18 17:59:02.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2020-09-02 15:22:15.000000000","updated_by":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"reviewer":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"state":"REVIEWER"},{"updated":"2020-09-02 16:26:24.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":"bd40d1c4b353351524606bf65b77a5d7f9afddb5","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-01-27 20:28:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a800281f51d3c1e0844cf2fafa990db892ede255","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-27 21:53:53.000000000","message":"Patch Set 1: 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/0e91736716e6406a88de1b0f63f8828b : FAILURE in 3m 48s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/6ca5740a86fe4b7f9aaafb8b7b635eec : SUCCESS in 5m 26s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/35b69f82c4d540bbb49ae4c34689ed9b : SUCCESS in 5m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cb83e23d2a8141d58445a1edc2f8201f : SUCCESS in 4m 08s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a4fa88926e0d4786bcd4baab33009385 : SUCCESS in 4m 42s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/8575d3e1892d4d9ab1bf096aac6b2ab4 : FAILURE in 1h 17m 22s","accounts_in_message":[],"_revision_number":1},{"id":"34171c3681cf9df4847e044d8ec7ad4e75ace8b0","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-01-28 22:24:53.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b0fda99c68dd39d02332de18958986f4b3a1bfa1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-28 23:57:23.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f9207796b3254b7aa569d680563520f1 : SUCCESS in 4m 13s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/7a2248e6742e4781ae8381d3f45b8db0 : SUCCESS in 4m 18s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/825f946a90534368914e148a5a7addc4 : SUCCESS in 5m 29s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bcc025e137c54928857e834fad230cf7 : SUCCESS in 5m 20s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/9b1eca012466487e859ecf02c945c5f8 : SUCCESS in 6m 40s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/321c8da847e44480ac597f11f76f17e4 : SUCCESS in 1h 23m 14s","accounts_in_message":[],"_revision_number":2},{"id":"51e632c71d85511b0d0777360519d4949139e81a","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-01-31 09:36:53.000000000","message":"Patch Set 2: Code-Review-1\n\n(21 comments)\n\nGood start. I think we can make this a good bit more generic if we drop unnecessary assertions like physical pinsets needing to be identical. That should make these less fragile.","accounts_in_message":[],"_revision_number":2},{"id":"98500d5e999377e61bdf6ed64567f265a761a809","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-01-31 19:13:42.000000000","message":"Patch Set 2:\n\n(18 comments)\n\nAgree with your points about focusing on NUMA nodes, will instead just collect total pCPU count and do the setup/comparisons accordingly.","accounts_in_message":[],"_revision_number":2},{"id":"40c03e27c9e9ca2a1a7586fafcd5ee3f08084a77","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-02-03 21:56:18.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"edb228436b266d65460aefabbe0206802d535328","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 23:21:30.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/597db67b10c347e79ec7b54e90a69f5b : SUCCESS in 3m 35s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/62f6170f3e024fdcb067a17bfbb76e39 : SUCCESS in 4m 11s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/d9b82b4b0adb4a8486dfe0443091be09 : SUCCESS in 4m 17s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d02f042b094d41e5b7877d7634c47fd4 : SUCCESS in 4m 03s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/46cc0dfc022a407e84ebd31749b15ce8 : SUCCESS in 5m 36s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/5384140e9f974394a30df19100d9c847 : SUCCESS in 1h 22m 46s","accounts_in_message":[],"_revision_number":3},{"id":"8bf5f75d4f0dee9d64d0f1949d09f6bcc073eef0","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-02-13 09:32:05.000000000","message":"Patch Set 3: Code-Review-1\n\n(5 comments)\n\nAgain, pretty close but there\u0027s a bit of complexity here that I think we can eliminate. Let me know if you disagree :)","accounts_in_message":[],"_revision_number":3},{"id":"bc480b5715cc54b598edc9c65045cd4cfbab8941","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-02-13 13:35:39.000000000","message":"Patch Set 3:\n\n(5 comments)","accounts_in_message":[],"_revision_number":3},{"id":"8ed297e5a048c8e3a95712ecbd1c801c878b8e60","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-02-17 17:22:09.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"4facd781d9f61ac77ca410f9e4ed7cf616e2fc06","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2020-03-18 17:59:02.000000000","message":"Patch Set 3:\n\njames i notices you commented the changes were done a month ago but have not pushed them can you push the new version?","accounts_in_message":[],"_revision_number":3},{"id":"5220683aad424252c715206f367361dced7e4075","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-20 20:54:58.000000000","message":"Patch Set 3:\n\n@sean, apologize for the delay, I have been focusing on zStream/baremetal automation until this past week. I\u0027ve been addressing all of my patch reviews and hope to get to this early next week.  For this test I need to handle tracking nova.conf configs in cases of a heterogeneous cpu layout between nodes and should have something up afterwards.","accounts_in_message":[],"_revision_number":3},{"id":"eff71505dbb158936fcc580e2b7f9dd7b1fbdaaa","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-24 20:20:25.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"14654d936d8e2e5e2e360260f7728574478e0857","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-24 20:26:13.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"16a35a2154820ff7d6049da1f336a68afd345fb4","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-03-24 20:31:12.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)\n\nThe overall test logic is good, a few notes:\n\n1. You can use the \u0027host\u0027 boot server request arg (new in 2.74) to force a server to land on a host.\n\n2. If we assume that whitebox gains something like CONF.cpus_per_node and CONF.nodes_per_host, your test needs... 4 (?) CPUs on each host. 2 dedicated, 2 shared,. Each dedicated VM gets 1 CPU (to allow for live migration), each shaared VM can get 2.\n\nGiven all of the above, you can skip your entire test if CONF.cpus_per_node * CONF.nodes_per_host (iow, total CPUs) \u003c 4\n\n3. You can then set dedicated\u003d0,1 and shared\u003d2,3 on both hosts\n\n4. With the host thing from 1, you can force VMs to land where you want them. You can pull hosts from get_all_hypervisors(), and have server_dedicated_host1, server_shared_host1, server_dedicated_host2, server_shared_host2\n\n5. With all of the above, it becomes much easier to do the test.\n\nPS: see inline for parse_cpu_spec :)","accounts_in_message":[],"_revision_number":4},{"id":"29918c0bd3e56321f4ee183fc5c8fd09e14116fc","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-24 20:33:10.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"357bf03069d4f52e1fc20653b2febd222fcb04fa","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-24 21:34:34.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1d1808c47007488e93cd75071784e609 : SUCCESS in 4m 28s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/c8090878b1ba48a9818b27b8805a05d6 : SUCCESS in 6m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bd38932e5b414f5098325b62bfc54128 : SUCCESS in 6m 52s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/e41e262969a94a478e0ebf80a047b7d1 : SUCCESS in 9m 28s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/3a3d920df7424d648584e13ada9e6040 : FAILURE in 1h 07m 55s","accounts_in_message":[],"_revision_number":4},{"id":"5dc004e7fa5cdcd18e2a4df84711361c14200b9d","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-25 00:10:31.000000000","message":"Patch Set 4:\n\nQuick thought based on our IRC discussion. Maybe instead of exposing two functions to the test classes from BaseWhiteboxComputeTest, we keep the same basic logic of get_all_hypervisors() but always return the hostname.  Then for the SSHClient() when the hostname is passed in do a check similar to get_hypervisor_ip() just without the server checks. If an IP is listed in the tempest.conf use it, otherwise use the hostname.","accounts_in_message":[],"_revision_number":4},{"id":"134ccf46d0ffee13ccc0003f5ee909335c23da6c","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-26 18:31:23.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"a463808a748c2f5588a7a376c22b6cdb8ba86cb4","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-03-26 19:00:29.000000000","message":"Patch Set 5: Code-Review-1\n\n(7 comments)\n\nOverall I think this is much much better than before - few small things inline.","accounts_in_message":[],"_revision_number":5},{"id":"1f0d627624fb7b1f341ea9c88680a4c244c1b7f4","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-26 19:13:59.000000000","message":"Patch Set 5:\n\n(7 comments)","accounts_in_message":[],"_revision_number":5},{"id":"854b392390592434ab089307d9db681042452a5d","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-03-26 19:18:33.000000000","message":"Patch Set 5:\n\n(3 comments)","accounts_in_message":[],"_revision_number":5},{"id":"e878176c5781d2493faa093211d8bcbbba7786dd","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-26 19:34:21.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0f31bb25e7a84be68f1eac3a0fb57d8b : SUCCESS in 5m 00s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4e109b536c1442e68eecb3b77530d13d : FAILURE in 2m 45s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/eafd421cd28640288cd00e4f9062b80f : SUCCESS in 5m 04s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/e54e9197a27443e2813b93c9175b1cb5 : SUCCESS in 4m 00s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/c61c54a188b84e1aafb754228a1ecc9f : SUCCESS in 1h 01m 22s","accounts_in_message":[],"_revision_number":5},{"id":"64548ffcc44cd325343fefad36c79e26afcc8a18","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-26 19:45:40.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"2502134ba72bac61e83aa66bbf46bca2c2126ae0","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-26 20:11:59.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"e2771eafbdfcf91d40d8c9f6d9061676f6fe40ae","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-26 21:15:28.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d2c61be53a5646219f903965dc4811f3 : SUCCESS in 3m 41s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/93523ef419d247ebb497032875d09f5b : FAILURE in 3m 16s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/ceb60072a1d14c2f8fed9201c37cf468 : SUCCESS in 4m 06s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/23f51642261d47c99904c2a807b6cd10 : SUCCESS in 5m 15s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/be7122754ecd4393aa7adc995bff88bf : SUCCESS in 1h 01m 35s","accounts_in_message":[],"_revision_number":6},{"id":"bbf8ae9a1bc80819ae013dddfddb12f70bd976d7","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-03-26 21:27:39.000000000","message":"Patch Set 6: Code-Review-1\n\n(6 comments)\n\nI think you have a bunch of assertions left over from the time that cpu_(shared|dedicated)_set was different on both hosts. Suggestion inline for how to make that happen.","accounts_in_message":[],"_revision_number":6},{"id":"4a4ba9a916a84a48a72bfad322ac7af44fe15e97","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-03-26 21:28:22.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"459e7addbcb957ffea53372329abf79f8a88c5b3","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-03-27 15:51:47.000000000","message":"Patch Set 6:\n\n(6 comments)","accounts_in_message":[],"_revision_number":6},{"id":"90f3616d8758c60bc9fb8af76f839ecb94ab8bd3","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-04-01 18:05:40.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"1e6eb6f7edb0fab3abb39d29793c266dbf1614fe","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-01 19:07:31.000000000","message":"Patch Set 7: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/80f6b996f9f64fb292e35747cb9847f4 : SUCCESS in 4m 45s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4631629be86f45a883408f30ad42cd57 : SUCCESS in 4m 56s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/19d4aec2ca004b1795fe00047b634b4c : SUCCESS in 5m 34s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/0f743d68cc1446c08a18e765b25b3708 : SUCCESS in 5m 12s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/cb379a35523d489baedf1ee27672f614 : FAILURE in 59m 44s","accounts_in_message":[],"_revision_number":7},{"id":"132662dcb7aca20587db22ed7590bb872ec99eef","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-04-03 19:44:58.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"909a33a1d2012544a95a5ea4759d0f29b84f881b","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-04-03 19:49:24.000000000","message":"Patch Set 8: Workflow-1","accounts_in_message":[],"_revision_number":8},{"id":"1e271b5bbb7be9d10bbde3705ac94f5a3ff42a40","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-03 21:12:11.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b54c8c6cb40a426faff6d91a0ee02e91 : SUCCESS in 4m 34s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/20d359887cfe4a2fbfbcfe03f0d1893c : SUCCESS in 6m 01s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d6a2a7d4debd45c6b8a16d3aa98ae6df : SUCCESS in 4m 52s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/77f3769dac8e4270a833c5624c2098a4 : SUCCESS in 5m 11s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/88c9d5748c904187a02942a094ed9f11 : SUCCESS in 1h 24m 17s","accounts_in_message":[],"_revision_number":8},{"id":"6d33ea32498ee8a4fa777101fac165993668d3b7","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-06-24 20:01:19.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"d5d910755d26e82dad8900f010ff252a922e39d2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-24 20:52:30.000000000","message":"Patch Set 9: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/686f798d1c25422ea0eb76f9c03129d1 : SUCCESS in 5m 34s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4f5dd8a588c34fccb3bc0ba58cd9eee2 : SUCCESS in 5m 43s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/92ec943daef0486591af31864dc54509 : SUCCESS in 11m 48s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/2f508195cc22420ebea74f4c559f0709 : SUCCESS in 12m 02s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/b265a2b1b3d748b3983f380892dd9341 : FAILURE in 47m 05s","accounts_in_message":[],"_revision_number":9},{"id":"02660d5ae0881459b52a9874792fe1a20ef4c3f4","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-06-24 22:35:46.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"d771c178f455e2744dad75d15a5fbdb496a810a8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-24 23:25:47.000000000","message":"Patch Set 10: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e1a33ee4b8244b4cbb3e3e4f5796b0de : SUCCESS in 4m 24s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/e3a07861e89c4578b77881f7974f6c18 : SUCCESS in 5m 11s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b57a7020282d42d19409b91c892e2365 : SUCCESS in 4m 28s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/5bcf95ad5bc54cd5bc47a1c09999fd25 : SUCCESS in 5m 09s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/ce953f5f62b740b49834f61424731436 : FAILURE in 38m 09s","accounts_in_message":[],"_revision_number":10},{"id":"1e73d834667f713510b656d8adef6fa12d4780a8","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-06-25 00:14:49.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"ceecde574868e7d66be83c65f4f546502b9c4849","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-25 01:06:54.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/12a0f6ea4bab49dcace5f4dcda19243b : SUCCESS in 3m 26s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/210e48ca76464ffdbea145bfba83b192 : SUCCESS in 4m 33s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/45b9d2f214bf44c1a1649c06d6372d87 : SUCCESS in 4m 41s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/5542588517b74879aed81d5d6f4e701f : SUCCESS in 4m 59s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/1c46bdfcce074cf6ab2bc848d050e39b : SUCCESS in 50m 29s","accounts_in_message":[],"_revision_number":11},{"id":"0969b721df7cf0ffdb6087b7f4dda391d073ef1b","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-06-25 19:30:05.000000000","message":"Patch Set 11:\n\nrecheck","accounts_in_message":[],"_revision_number":11},{"id":"fa18407e0fffc6a2a788665ec5099e9b411ac6e0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-25 20:35:33.000000000","message":"Patch Set 11: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7b46bad45de24ff78181be436cb86eb2 : SUCCESS in 4m 49s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/25a627c525b94f7f84e940277b53daa7 : SUCCESS in 4m 40s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/05f4c6307da2416ca0838b85b17a708c : SUCCESS in 3m 41s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/4aa3591a6ffa42b5a6a6795e8f913c30 : SUCCESS in 4m 25s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/e476bfa496034753a803e133775b50f0 : FAILURE in 1h 03m 59s","accounts_in_message":[],"_revision_number":11},{"id":"cb3d00fc62a28152fcc7f46fef7bae423bb2ad79","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-06-26 14:41:01.000000000","message":"Uploaded patch set 12.","accounts_in_message":[],"_revision_number":12},{"id":"f01c18425107dd5ca0a98474f70ea2bdbbbe3ff2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-26 15:51:56.000000000","message":"Patch Set 12: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b4c76aca4c0e4e5b97c55ae0c5ae1bc4 : SUCCESS in 3m 41s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/5bf45d16d39541a68fdf52dae5302012 : SUCCESS in 4m 07s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/910dcde451d1463a9fade6fa33a22685 : SUCCESS in 7m 01s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/9e60d66a1a484647844b63d5b4158b24 : SUCCESS in 4m 25s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/0867e5cb8db74e93a245ee82b2bfcf11 : SUCCESS in 1h 02m 03s","accounts_in_message":[],"_revision_number":12},{"id":"fc55d1a3264d7051e30461316566e4a973b10cc4","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-08-24 19:15:50.000000000","message":"Patch Set 12: Code-Review-1\n\n(6 comments)\n\nSome nits inline, but I mostly think we need to simplify this. If we just add a helper that gives us all the CPUs in the deployment as such:\n\n  def get_all_cpus():\n    topology_dict \u003d CONF.whitebox_hardware.cpu_topology\n    cpus \u003d []\n    [cpus.extend(c) for c in topology_dict.values()]\n    return cpus\n\nThen we\u0027d need to:\n\n1. Require a min of 4 CPUs total (1 per instance)\n2. Not require multi-NUMA :)\n3. Assume the CPUs are numbered sequentially\n4. Hard-code values for cpu_dedicated_set and cpu_shared_set on each host\n5. Keep the rest of the main test logic the same, but our assertions would become easier.\n\nSounds sensible?","accounts_in_message":[],"_revision_number":12},{"id":"cc325622c314bdd68e3844b0bfc9214a18e630c8","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-08-24 19:31:50.000000000","message":"Patch Set 12:\n\n(6 comments)\n\nAgreed, I\u0027ll simplify the approach to allow for both upstream/downstream execution.","accounts_in_message":[],"_revision_number":12},{"id":"cb9a54e26d8d14c39529f9db52faf38ff250b1d6","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-08-31 19:48:10.000000000","message":"Uploaded patch set 13.","accounts_in_message":[],"_revision_number":13},{"id":"9e5f9aa915d7f01fb9f71faf483fa0d3dcd1f7a1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-31 20:21:47.000000000","message":"Patch Set 13: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b0ae73c591c0474686b9ce1ebeb66ace : SUCCESS in 3m 47s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/cfff0c6a57b14cc1ad923ab28f3c1253 : SUCCESS in 4m 58s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cfcd5599793043dda43bc272214b59c2 : SUCCESS in 4m 08s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/99786fe5897b4d3eb84764e728b52e87 : SUCCESS in 5m 06s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/e4071b8012e24ab181d173dfebf0c87a : FAILURE in 25m 17s\n- whitebox-multinode-multinuma-devstack https://zuul.opendev.org/t/openstack/build/f65e1f33882041aebe6ca3a79c0463c8 : RETRY_LIMIT in 8m 25s (non-voting)","accounts_in_message":[],"_revision_number":13},{"id":"d99e788f0141d2724f3ed938c4ea85b228b6802d","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-09-01 14:49:06.000000000","message":"Patch Set 13: Code-Review-1\n\n(4 comments)\n\n_1 for the wrong suffix, otherwise it\u0027s just nits.","accounts_in_message":[],"_revision_number":13},{"id":"dfbe232c4da8cdd5d037d3a65a0fb7735b212961","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-09-01 14:57:19.000000000","message":"Patch Set 13:\n\n(4 comments)","accounts_in_message":[],"_revision_number":13},{"id":"0b4932b4336ef33a6c5ea8e77a8123346699f1c8","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-09-01 14:59:41.000000000","message":"Patch Set 13:\n\n(2 comments)","accounts_in_message":[],"_revision_number":13},{"id":"b068f7fe4b6e9625aba8927eadb5331bd6a51d84","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-09-02 14:13:29.000000000","message":"Uploaded patch set 14.","accounts_in_message":[],"_revision_number":14},{"id":"54be7429966c98d86da6507a283ecb0c209178c3","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-09-02 14:54:14.000000000","message":"Patch Set 14: Code-Review+2\n\nLooks good. Now to unbreak CI :(","accounts_in_message":[],"_revision_number":14},{"id":"93307c3094b3f2ff941248433d323ce7627a7d6a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-02 15:20:45.000000000","message":"Patch Set 14: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ba03dccd1c094bddbe40a23d152e2d8f : SUCCESS in 3m 56s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/59d512fa4eda4121a44ec5df7bf4d74b : SUCCESS in 5m 06s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/3f598f5fea4944fbaa6123dadc7c7bad : SUCCESS in 4m 53s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/00c8679826af42d18a4f1984b499dc66 : SUCCESS in 5m 38s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/859e84c5bb284d19b6c7d5cd7272b136 : SUCCESS in 59m 48s\n- whitebox-multinode-multinuma-devstack https://zuul.opendev.org/t/openstack/build/3d546dc7188d4d73a22c7b6821333280 : RETRY_LIMIT in 13m 11s (non-voting)","accounts_in_message":[],"_revision_number":14},{"id":"8b6ea5d5751d73046892159381b670c4fdab7a25","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2020-09-02 15:22:15.000000000","message":"Patch Set 14: Workflow+1\n\nDid CI just magically fix itself?","accounts_in_message":[],"_revision_number":14},{"id":"4f010eb05183e75f53e030e82530f3fd92605ba1","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-02 15:22:27.000000000","message":"Patch Set 14: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":14},{"id":"524597b1a6e1c1db9116124136ad9be4262706ac","author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"date":"2020-09-02 15:28:57.000000000","message":"Patch Set 14:\n\nhttps://review.opendev.org/#/c/749253/ merged after I had already submitted PS13 and I was reading the ssh key error might be related to the pip installation methodology so maybe that did it?  PS14 ran with it merged and appears to be working now, but I could be completely off the mark.","accounts_in_message":[],"_revision_number":14},{"id":"d0aed0c0703343f9a58519eaebdec5ca12bfc9c3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-02 16:26:24.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":14},{"id":"c2fac9ef49d77fb089a8bc6b2ed4bdd7669dc6d1","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-02 16:26:24.000000000","message":"Patch Set 14: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fd26fb49d8bc44189ad524618d1542e0 : SUCCESS in 3m 53s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/c96a06feaa5043fda589534ec2bb2c2a : SUCCESS in 3m 47s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/8c3fbab3eeed412aa366caae9486bfa6 : SUCCESS in 4m 53s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/b217866cf11e40cc895d1b8f881d3eac : SUCCESS in 4m 27s\n- whitebox-multinode-devstack https://zuul.opendev.org/t/openstack/build/a7c595d5b7644f27ad1d2289a0f84df3 : SUCCESS in 1h 00m 26s\n- whitebox-multinode-multinuma-devstack https://zuul.opendev.org/t/openstack/build/76aa731353f64360847f437eadcae762 : RETRY_LIMIT in 11m 03s (non-voting)","accounts_in_message":[],"_revision_number":14}],"current_revision_number":14,"current_revision":"5d4120cbaef53aa660dc76152bd373673118114e","revisions":{"caf683d46fac2628cb7bbe577597175962b6ff46":{"kind":"REWORK","_number":1,"created":"2020-01-27 20:28:57.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/1"}}},"commit":{"parents":[{"commit":"45d12603821355424fd578d9447a45275e098ac6","subject":"Merge \"NUMA rebuild tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/45d12603821355424fd578d9447a45275e098ac6"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-27 20:26:10.000000000","tz":-300},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The method is designed to test blueprint [1].\nTest requires a deployment of two compute nodes with at least two NUMA\nnodes of identical topology. Test configures cpu_dedicated_set to use\nall pCPUs from the first NUMA node of each host. The pCPUs of the second\nNUMA node are used for the cpu_shared_set. Two instances will be created\non the first compute node, one will require cpu pinning and the other\nwill simply float on the available shared pCPUs. Each instance should\n‘fill’ their respective NUMA node. This procedure is then repeated on\nthe second host.\n\nAfter validating all pinning schemes for each instance is correct, the\nfloating instance on the first host will be migrated to the second. The\ndedicated instance on the target host will be deleted and the remaining\ndedicated instance will be migrated to the target host. The test method\nwill validate all instance migrations were successful and the three\ninstances are collocated on the same host. Finally it validates the\nunderlying pin scheme of the instances to confirm they are still\naccurate after migration\n\nA helper method _get_shared_cpuset was added to NUMALiveMigrationTest as\nwell. It parses an instance’s xml for the ‘cpuset’ attribute found in\nthe vcpu tag. It then converts and returns the ‘cpuset’ value into a set\nof pCPUs.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/caf683d46fac2628cb7bbe577597175962b6ff46"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/caf683d46fac2628cb7bbe577597175962b6ff46"}]},"branch":"refs/heads/master"},"b1ac10ae3e057259a0e0ea46eb152949313e50be":{"kind":"REWORK","_number":2,"created":"2020-01-28 22:24:53.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/2"}}},"commit":{"parents":[{"commit":"45d12603821355424fd578d9447a45275e098ac6","subject":"Merge \"NUMA rebuild tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/45d12603821355424fd578d9447a45275e098ac6"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-28 22:24:27.000000000","tz":-300},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The method is designed to test blueprint [1].\nDownstream RedHat BZ can be referenced here [2].  Test requires a\ndeployment of two compute nodes with at least two NUMA nodes of\nidentical topology. Test configures cpu_dedicated_set to use all pCPUs\nfrom the first NUMA node of each host. The pCPUs of the second NUMA node\nare used for the cpu_shared_set. Two instances will be created on the\nfirst compute node, one will require cpu pinning and the other will\nsimply float on the available shared pCPUs. Each instance should ‘fill’\ntheir respective NUMA node. This procedure is then repeated on the\nsecond host.\n\nAfter validating all pinning schemes for each instance is correct, the\nfloating instance on the first host will be migrated to the second. The\ndedicated instance on the target host will be deleted and the remaining\ndedicated instance will be migrated to the target host. The test method\nwill validate all instance migrations were successful and the three\ninstances are collocated on the same host. Finally it validates the\nunderlying pin scheme of the instances to confirm they are still\naccurate after migration\n\nA helper method _get_shared_cpuset was added to NUMALiveMigrationTest as\nwell. It parses an instance’s xml for the ‘cpuset’ attribute found in\nthe vcpu tag. It then converts and returns the ‘cpuset’ value into a set\nof pCPUs.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1693372\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b1ac10ae3e057259a0e0ea46eb152949313e50be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b1ac10ae3e057259a0e0ea46eb152949313e50be"}]},"branch":"refs/heads/master"},"b3f16a50e120900d52504aba8c80404a88c1108c":{"kind":"REWORK","_number":3,"created":"2020-02-03 21:56:18.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/3"}}},"commit":{"parents":[{"commit":"45d12603821355424fd578d9447a45275e098ac6","subject":"Merge \"NUMA rebuild tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/45d12603821355424fd578d9447a45275e098ac6"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-02-03 21:23:51.000000000","tz":-300},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The method is designed to test blueprint [1].\nThe downstream Red Hat BZ can be referenced here [2]. Test also touches\non Red Hat BZ [3]. Test requires a deployment of two compute nodes. Each\ncompute node’s nova.conf will update their cpu_dedicated_set and\ncpu_shared_set parameters based on the provided vcpu sizes of\n‘dedicated_flavor_vcpus’ and  ‘shared_flavor_vcpus’ respectively. For\nthe test to execute, each host must be capable of holding two dedicated\ninstances and still have available host CPUs to configure the\ncpu_shared_set.\n\nTest will first deploy four instances total, a pinned instance and a\nshared instance for each host. After validating all servers are up and\ntheir pin scheme is correct, all instances from the first host are live\nmigrated to the second. The pin schemes are then revalidated to confirm\npinned instances do not intersect with each nor do they intersect with\nthe host CPUs configured in cpu_shared_set.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationTest.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag. It then converts and returns the ‘cpuset’ value as a set of\nhost CPUs.  Another method, _get_cpu_configs, was repurposed and added\nto BasePinningTest from [4]. It calculates the minimum number of host\nCPUs needed to fit a required cpu_dedicated_set and cpu_shared_set. It\ndetermines the host CPUs needed for each parameter and generates the\nappropriate string configuration to add to nova.conf.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1693372\n[3] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1222414\n[4] https://review.rdoproject.org/r/gitweb?p\u003dopenstack/whitebox-tempest-plugin.git;a\u003dblob_plain;f\u003dwhitebox_tempest_plugin/api/compute/test_cpu_pinning.py;hb\u003drefs/changes/70/18070/52\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b3f16a50e120900d52504aba8c80404a88c1108c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b3f16a50e120900d52504aba8c80404a88c1108c"}]},"branch":"refs/heads/master"},"d11dfc0d34d046a3a74f29ec0ef20494d19052f5":{"kind":"REWORK","_number":4,"created":"2020-03-24 20:20:25.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/4"}}},"commit":{"parents":[{"commit":"1e124bd995e3b2f26a68a6897826b0fbde1584f7","subject":"conf: give available_cinder_storage a sensible default","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/1e124bd995e3b2f26a68a6897826b0fbde1584f7"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-03-24 20:20:16.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nstandard policy. Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nThe commit adds a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The test will update the cpu_shared_set and\ncpu_dedicated_set of each compute node’s nova.conf If the host is single\nNUMA, half of its cores will be allocated to dedicated cpus and the\nother half for shared. If the host has multiple NUMA nodes, then all\ncores from NUMA 0 will be used for the shared cpu pool and all cores\nfrom NUMA 1 will be used for the dedicated cpu pool. This logic is\nhandled by the test method\nNUMALiveMigrationTest._get_dedicated_and_shared_sets().\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationTest.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag. It then converts and returns the ‘cpuset’ value as a set of\nhost CPUs.\n\nIn order to correctly map the expected cpu configuration with what is\nrunning live on the host, the method VirshXMLClient.get_fqdn_of_host()\nwas added to clients.py. This uses the connection address provided by\nwhitebox.hypervisors in tempest.conf to gather the hypervisor hostname.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d11dfc0d34d046a3a74f29ec0ef20494d19052f5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d11dfc0d34d046a3a74f29ec0ef20494d19052f5"}]},"branch":"refs/heads/master"},"8ed7c20d43750a60e8dcc2af8161479a4ba9405b":{"kind":"REWORK","_number":5,"created":"2020-03-26 18:31:23.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/5"}}},"commit":{"parents":[{"commit":"1e124bd995e3b2f26a68a6897826b0fbde1584f7","subject":"conf: give available_cinder_storage a sensible default","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/1e124bd995e3b2f26a68a6897826b0fbde1584f7"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-03-26 18:30:09.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as Nova bug\n1289064. The first stage tests basic colocation of instances using shared and\ndedicated cpu policies. With a two compute host deployment, create two\ninstances on each host. On each host, one instance should use a standard cpu\npolicy and the other should use a dedicated cpu policy. Validate all instances\ncome up correctly and the respective cpu mappings do not intersect with other\npools.\n\nThe second stage involves migrating all instances from the first host to the\nsecond. When migration is finalized a single host should have four instances\ntotal, two with a dedicated cpu policy and two using the standard policy.\nValidate the dedicated instance\u0027s cpu’s do not intersect with each other or the\nshared pool of the host.\n\nThe commit adds a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The test has a skip condition based on two new whitebox\nconfig parameters: whitebox.cpus_per_numa_node and\nwhitebox.numa_nodes_per_compute_host. If the multiplication of these two amount\nto four or more host CPU\u0027s the test will execute. Since the test assumes four\nor more host cpus are available it will update each hosts cpu_dedicated_set to\nuse 0,1 and cpu_shared_set to use 2,3. To avoid guests being deployed cross\nNUMA, flavor size for all guests is restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationTest. It\nparses an instance’s XML for the ‘cpuset’ attribute found in the ‘vcpu’ tag and\npass the result to parse_cpu_spec.\n\ncpus_per_numa_node and numa_nodes_per_compute_host are both defaulted to one\nin config.py and devstack/settings parameters WHITEBOX_CPUS_PER_NUMA_NODE and\nWHITEBOX_NUMA_NODES_PER_COMPUTE_HOST were added for upstream configuration.\nThese values were set to four and two respectively.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/8ed7c20d43750a60e8dcc2af8161479a4ba9405b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/8ed7c20d43750a60e8dcc2af8161479a4ba9405b"}]},"branch":"refs/heads/master"},"b84f43979622570b0b5902303c7a73be3030b7e7":{"kind":"REWORK","_number":6,"created":"2020-03-26 20:11:59.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/6"}}},"commit":{"parents":[{"commit":"1e124bd995e3b2f26a68a6897826b0fbde1584f7","subject":"conf: give available_cinder_storage a sensible default","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/1e124bd995e3b2f26a68a6897826b0fbde1584f7"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-01-24 20:20:50.000000000","tz":-300},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-03-26 20:11:46.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as Nova bug\n1289064. The first stage tests basic colocation of instances using shared and\ndedicated cpu policies. With a two compute host deployment, create two\ninstances on each host. On each host, one instance should use a standard cpu\npolicy and the other should use a dedicated cpu policy. Validate all instances\ncome up correctly and the respective cpu mappings do not intersect with other\npools.\n\nThe second stage involves migrating all instances from the first host to the\nsecond. When migration is finalized a single host should have four instances\ntotal, two with a dedicated cpu policy and two using the standard policy.\nValidate the dedicated instance\u0027s cpu’s do not intersect with each other or the\nshared pool of the host.\n\nThe commit adds a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The test has a skip condition based on two new whitebox\nconfig parameters: whitebox.cpus_per_numa_node and\nwhitebox.numa_nodes_per_compute_host. If the multiplication of these two amount\nto four or more host CPU\u0027s the test will execute. Since the test assumes four\nor more host cpus are available it will update each hosts cpu_dedicated_set to\nuse 0,1 and cpu_shared_set to use 2,3. To avoid guests being deployed cross\nNUMA, flavor size for all guests is restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationTest. It\nparses an instance’s XML for the ‘cpuset’ attribute found in the ‘vcpu’ tag and\npass the result to parse_cpu_spec.\n\ncpus_per_numa_node and numa_nodes_per_compute_host are both defaulted to one\nin config.py and devstack/settings parameters WHITEBOX_CPUS_PER_NUMA_NODE and\nWHITEBOX_NUMA_NODES_PER_COMPUTE_HOST were added for upstream configuration.\nThese values were set to four and two respectively.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b84f43979622570b0b5902303c7a73be3030b7e7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/b84f43979622570b0b5902303c7a73be3030b7e7"}]},"branch":"refs/heads/master"},"058aca5f7ea2fa09904fb17b57416a5f9ef33c18":{"kind":"REWORK","_number":7,"created":"2020-04-01 18:05:40.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/7"}}},"commit":{"parents":[{"commit":"d53db73cab5ae3890747f13aad844f8ea8d9a4ca","subject":"Refactor NUMALive tests for legacy and current","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d53db73cab5ae3890747f13aad844f8ea8d9a4ca"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 18:01:04.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nstandard policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nThe commit adds a new test method, test_colloc_migration, to\nNUMALiveMigrationTest. The test has a skip condition based on two new\nwhitebox config parameters: whitebox.cpus_per_numa_node and\nwhitebox.numa_nodes_per_compute_host. If the multiplication of these two\namount to four or more host CPU\u0027s the test will execute. Since the test\nassumes four or more host cpus are available it will update each host\nand leverage host CPUs 0-3. The first host will use 0,1 for its\ncpu_dedicated_set and 2,3 for its cpu_shared_set and the second will use\nthe inverse configuration (dedicated: 2,3 and shared: 0,1). To avoid\nguests being deployed cross NUMA, flavor size for all guests is\nrestricted to one vcpu, dictated by the variable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationTest.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\ncpus_per_numa_node and numa_nodes_per_compute_host are both defaulted to\none in config.py and devstack/settings parameters\nWHITEBOX_CPUS_PER_NUMA_NODE and WHITEBOX_NUMA_NODES_PER_COMPUTE_HOST\nwere added for upstream configuration. These values were set to four\nand two respectively.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n\nDepends-on: https://review.opendev.org/#/c/716344/\nDepends-on: https://review.opendev.org/#/c/716311/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/058aca5f7ea2fa09904fb17b57416a5f9ef33c18"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/058aca5f7ea2fa09904fb17b57416a5f9ef33c18"}]},"branch":"refs/heads/master"},"d58f188d3ee1b05dc8255789834614026f0665e7":{"kind":"REWORK","_number":8,"created":"2020-04-03 19:44:58.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/8"}}},"commit":{"parents":[{"commit":"c4ec528269def17d9e9d409f2e50a6ac8079d571","subject":"Normalize numa_topology for consistent equality tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/c4ec528269def17d9e9d409f2e50a6ac8079d571"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-03 19:35:07.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nstandard policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a chile of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe test method  has a skip condition based on two new whitebox config\nparameters: whitebox.cpus_per_numa_node and\nwhitebox.numa_nodes_per_compute_host. If the multiplication of these two\namount to four or more host CPU\u0027s the test will execute. Since the test\nassumes four or more host cpus are available it will update each host\nand leverage host CPUs 0-3. The first host will use 0,1 for its\ncpu_dedicated_set and 2,3 for its cpu_shared_set and the second will use\nthe inverse configuration (dedicated: 2,3 and shared: 0,1). To avoid\nguests being deployed cross NUMA, flavor size for all guests is\nrestricted to one vcpu, dictated by the variable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationBase.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\ncpus_per_numa_node and numa_nodes_per_compute_host are both defaulted to\none in config.py and devstack/settings parameters\nWHITEBOX_CPUS_PER_NUMA_NODE and WHITEBOX_NUMA_NODES_PER_COMPUTE_HOST\nwere added for upstream configuration. These values were set to four and\ntwo respectively.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nDepends-on: https://review.opendev.org/#/c/717355/\nDepends-on: https://review.opendev.org/#/c/716344/\nDepends-on: https://review.opendev.org/#/c/716311/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d58f188d3ee1b05dc8255789834614026f0665e7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d58f188d3ee1b05dc8255789834614026f0665e7"}]},"branch":"refs/heads/master"},"51fe0aae6bc15cf42adedbaa333bb408686aa923":{"kind":"REWORK","_number":9,"created":"2020-06-24 20:01:19.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/9"}}},"commit":{"parents":[{"commit":"12eca6f75e2b966ddae0c1fe11d8bed6992c71fd","subject":"Merge \"Rename nova-libvirt to libvirt\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/12eca6f75e2b966ddae0c1fe11d8bed6992c71fd"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-06-24 19:24:21.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has two skip checks based around\ncpu_topology, validating there are at least two NUMA nodes per host and\neach node has at least two or more pCPUs.\n\nThe test method uses all pCPUs from NUMA node 0 as the dedicated cpu set\nfor the first host and the shared set for the second host. NUMA node 1\nprovides the pCPUs for the dedicated range on host two and the shared\nset range for host one. To avoid guests being deployed cross NUMA,\nflavor size for all guests are restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationBase.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\nFor devstack deployments cpu_topology is set via\nWHITEBOX_HARDWARE_CPU_TOPOLOGY which defaults to \u00270: [0,1,2,3], 1:\n[4,5,6,7]\u0027.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nDepends-on: https://review.opendev.org/#/c/731557/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/51fe0aae6bc15cf42adedbaa333bb408686aa923"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/51fe0aae6bc15cf42adedbaa333bb408686aa923"}]},"branch":"refs/heads/master"},"63bb13ef5241eef947011079d431af560e501fab":{"kind":"REWORK","_number":10,"created":"2020-06-24 22:35:46.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/10"}}},"commit":{"parents":[{"commit":"12eca6f75e2b966ddae0c1fe11d8bed6992c71fd","subject":"Merge \"Rename nova-libvirt to libvirt\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/12eca6f75e2b966ddae0c1fe11d8bed6992c71fd"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-06-24 22:35:35.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has two skip checks based around\ncpu_topology, validating there are at least two NUMA nodes per host and\neach node has at least two or more pCPUs.\n\nThe test method uses all pCPUs from NUMA node 0 as the dedicated cpu set\nfor the first host and the shared set for the second host. NUMA node 1\nprovides the pCPUs for the dedicated range on host two and the shared\nset range for host one. To avoid guests being deployed cross NUMA,\nflavor size for all guests are restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationBase.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\nFor devstack deployments cpu_topology is set via\nWHITEBOX_HARDWARE_CPU_TOPOLOGY which defaults to \u00270: [0,1,2,3], 1:\n[4,5,6,7]\u0027.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nDepends-on: https://review.opendev.org/#/c/731557/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/63bb13ef5241eef947011079d431af560e501fab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/63bb13ef5241eef947011079d431af560e501fab"}]},"branch":"refs/heads/master"},"3d76b5756ebad1847e0079ef9113ffd6fa902c54":{"kind":"REWORK","_number":11,"created":"2020-06-25 00:14:49.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/11"}}},"commit":{"parents":[{"commit":"12eca6f75e2b966ddae0c1fe11d8bed6992c71fd","subject":"Merge \"Rename nova-libvirt to libvirt\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/12eca6f75e2b966ddae0c1fe11d8bed6992c71fd"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-06-25 00:13:39.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has two skip checks based around\ncpu_topology, validating there are at least two NUMA nodes per host and\neach node has at least two or more pCPUs.\n\nThe test method uses all pCPUs from NUMA node 0 as the dedicated cpu set\nfor the first host and the shared set for the second host. NUMA node 1\nprovides the pCPUs for the dedicated range on host two and the shared\nset range for host one. To avoid guests being deployed cross NUMA,\nflavor size for all guests are restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationBase.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nDepends-on: https://review.opendev.org/#/c/731557/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/3d76b5756ebad1847e0079ef9113ffd6fa902c54"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/3d76b5756ebad1847e0079ef9113ffd6fa902c54"}]},"branch":"refs/heads/master"},"d8bac3210b3d5b707b35d9aa28f00c21f22fd049":{"kind":"REWORK","_number":12,"created":"2020-06-26 14:41:01.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/12"}}},"commit":{"parents":[{"commit":"14b70736b97e0110c81795f537dd74b788b6a3bf","subject":"Update compute cpu-model to kvm64","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/14b70736b97e0110c81795f537dd74b788b6a3bf"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-06-26 14:40:11.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nNova bug 1289064. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has two skip checks based around\ncpu_topology, validating there are at least two NUMA nodes per host and\neach node has at least two or more pCPUs.\n\nThe test method uses all pCPUs from NUMA node 0 as the dedicated cpu set\nfor the first host and the shared set for the second host. NUMA node 1\nprovides the pCPUs for the dedicated range on host two and the shared\nset range for host one. To avoid guests being deployed cross NUMA,\nflavor size for all guests are restricted to one vcpu, dictated by the\nvariable flavor_vcpu_size.\n\nA helper method, _get_shared_cpuset, was added to NUMALiveMigrationBase.\nIt parses an instance’s XML for the ‘cpuset’ attribute found in the\n‘vcpu’ tag and pass the result to parse_cpu_spec.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nDepends-on: https://review.opendev.org/#/c/731557/\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d8bac3210b3d5b707b35d9aa28f00c21f22fd049"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/d8bac3210b3d5b707b35d9aa28f00c21f22fd049"}]},"branch":"refs/heads/master"},"7f465bdf3055605b759b4081b0787e3cb7a71509":{"kind":"REWORK","_number":13,"created":"2020-08-31 19:48:10.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/13"}}},"commit":{"parents":[{"commit":"17185972008937fd45bc0fe3755a8e69f991a524","subject":"Pass hostname to NovaServiceManager","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/17185972008937fd45bc0fe3755a8e69f991a524"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-08-31 19:45:44.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nlive-migration. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has one skip check based around\ncpu_topology, that validates each host has four or more pCPUs.\n\nThe test method uses the first two pCPUs from cpu_topology as the\ndedicated pCPUs for the first host and the shared set for the second\nhost. pCPU id\u0027s three and four are for the dedicated range on host two\nand the shared set range for host one. To avoid guests being deployed\ncross NUMA, flavor size for all guests are restricted to one vcpu,\ndictated by the variable flavor_vcpu_size.\n\nTwo helper methods, _get_shared_cpuset and get_all_cpus, were added to\nNUMALiveMigrationBase.  _get_shared_cpuset parses an instance’s XML for\nthe ‘cpuset’ attribute found in the ‘vcpu’ tag and pass the result to\nparse_cpu_spec. Method get_all_cpus aggregates all cpus to a single\nlist from cpu_topology in found in section whitebox-hardware of tempest.conf.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/7f465bdf3055605b759b4081b0787e3cb7a71509"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/7f465bdf3055605b759b4081b0787e3cb7a71509"}]},"branch":"refs/heads/master"},"5d4120cbaef53aa660dc76152bd373673118114e":{"kind":"REWORK","_number":14,"created":"2020-09-02 14:13:29.000000000","uploader":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"ref":"refs/changes/20/704420/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/whitebox-tempest-plugin","ref":"refs/changes/20/704420/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/whitebox-tempest-plugin refs/changes/20/704420/14"}}},"commit":{"parents":[{"commit":"17185972008937fd45bc0fe3755a8e69f991a524","subject":"Pass hostname to NovaServiceManager","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/17185972008937fd45bc0fe3755a8e69f991a524"}]}],"author":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-04-01 17:51:47.000000000","tz":-240},"committer":{"name":"James Parker","email":"jparker@redhat.com","date":"2020-09-01 15:33:11.000000000","tz":-240},"subject":"Added instance collocation migration test","message":"Added instance collocation migration test\n\nAdded a two stage test meant to test both Nova feature [1] as well as\nlive-migration. The first stage tests basic colocation of instances\nusing shared and dedicated cpu policies. With a two compute host\ndeployment, create two instances on each host. On each host, one\ninstance should use a standard cpu policy and the other should use a\ndedicated cpu policy. Validate all instances come up correctly and the\nrespective cpu mappings do not intersect with other pools.\n\nThe second stage involves migrating all instances from the first host to\nthe second. When migration is finalized a single host should have four\ninstances total, two with a dedicated cpu policy and two using the\nshared policy.  Validate the dedicated instance\u0027s cpu’s do not\nintersect with each other or the shared pool of the host.\n\nIn order to schedule the instances to their respective hosts necessary\nto execute the procedure, the test takes advantage of the host parameter\nadded in 2.74 [2]. Since a large portion of the test method leverages\nfunctions from NUMALiveMigrationTest, but the minimum microversion for\nthe class is 2.25, the class was refactored. There is now a base class\nNUMALiveMigrationBase which holds all of the common helper methods and\nNUMALiveMigrationTest is a child of that base. A second child class has\nbeen created, NUMACPUDedicatedLiveMigrationTest, which contains the new\ntest method, test_collocation_migration, and sets the minimum\nmicroversion to 2.74.\n\nThe class NUMACPUDedicatedLiveMigrationTest has a skip check to validate\nwhitebox-hardware/cpu_topology is set in tempest.conf. The new method\ntest_collocation_migration has one skip check based around\ncpu_topology, that validates each host has four or more pCPUs.\n\nThe test method uses the first two pCPUs from cpu_topology as the\ndedicated pCPUs for the first host and the shared set for the second\nhost. pCPU id\u0027s three and four are for the dedicated range on host two\nand the shared set range for host one. To avoid guests being deployed\ncross NUMA, flavor size for all guests are restricted to one vcpu,\ndictated by the variable flavor_vcpu_size.\n\nTwo helper methods, _get_shared_cpuset and get_all_cpus, were added to\nNUMALiveMigrationBase.  _get_shared_cpuset parses an instance’s XML for\nthe ‘cpuset’ attribute found in the ‘vcpu’ tag and pass the result to\nparse_cpu_spec. Method get_all_cpus aggregates all cpus to a single\nlist from cpu_topology in found in section whitebox-hardware of tempest.conf.\n\n[1] https://blueprints.launchpad.net/nova/+spec/use-pcpu-and-vcpu-in-one-instance\n[2] https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id67\n\nChange-Id: If9eab53861079a1ca1530f6dd56ee0f39216077f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/5d4120cbaef53aa660dc76152bd373673118114e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/whitebox-tempest-plugin/commit/5d4120cbaef53aa660dc76152bd373673118114e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}