| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix non-existent method of Mock https://review.opendev.org/676840 | 00:02 |
|---|---|---|
| openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | 00:04 |
| *** igordc has quit IRC | 00:11 | |
| *** igordc has joined #openstack-nova | 00:11 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix non-existent method of Mock https://review.opendev.org/676840 | 00:15 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider config file schema and loader https://review.opendev.org/673341 | 00:23 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP: Public method to retrieve custom resource providers https://review.opendev.org/676029 | 00:23 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP: Load the custom resource providers to resource tracker https://review.opendev.org/676522 | 00:23 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix non-existent method of Mock https://review.opendev.org/676844 | 00:32 |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | 00:33 |
| *** gyee has quit IRC | 00:34 | |
| *** licanwei has quit IRC | 00:40 | |
| *** igordc has quit IRC | 00:44 | |
| *** igordc has joined #openstack-nova | 00:45 | |
| *** markvoelker has joined #openstack-nova | 00:45 | |
| sean-k-mooney | mriedem nice music choice as always | 00:50 |
| *** markvoelker has quit IRC | 00:50 | |
| sean-k-mooney | ill proably fix the vpmu stuff before your back online | 00:50 |
| *** ricolin has joined #openstack-nova | 00:58 | |
| *** brinzhang_ has joined #openstack-nova | 01:00 | |
| *** markvoelker has joined #openstack-nova | 01:03 | |
| *** brinzhang has quit IRC | 01:03 | |
| *** spsurya has joined #openstack-nova | 01:14 | |
| *** brinzhang has joined #openstack-nova | 01:16 | |
| *** dklyle has quit IRC | 01:18 | |
| *** brinzhang_ has quit IRC | 01:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix non-existent method of Mock https://review.opendev.org/676844 | 01:29 |
| *** tbachman has quit IRC | 01:31 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452 | 01:38 |
| *** dave-mccowan has joined #openstack-nova | 01:40 | |
| *** tbachman has joined #openstack-nova | 01:42 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Fix non-existent method of Mock https://review.opendev.org/676848 | 01:49 |
| *** markvoelker has quit IRC | 01:51 | |
| openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.77: Support Specifying AZ to unshelve https://review.opendev.org/665136 | 01:51 |
| *** slaweq has joined #openstack-nova | 02:11 | |
| *** tinwood has quit IRC | 02:12 | |
| *** tinwood has joined #openstack-nova | 02:13 | |
| *** slaweq has quit IRC | 02:16 | |
| *** bbowen has joined #openstack-nova | 02:23 | |
| *** markvoelker has joined #openstack-nova | 02:35 | |
| *** BjoernT has joined #openstack-nova | 02:37 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | 02:38 |
| *** markvoelker has quit IRC | 02:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix non-existent method of Mock https://review.opendev.org/676850 | 02:40 |
| *** ash2307 has joined #openstack-nova | 02:46 | |
| *** takashin has left #openstack-nova | 03:02 | |
| *** slaweq has joined #openstack-nova | 03:11 | |
| *** dave-mccowan has quit IRC | 03:12 | |
| *** slaweq has quit IRC | 03:16 | |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | 03:23 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Refactor some methods for reuse by Cyborg-related code. https://review.opendev.org/673734 | 03:23 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | 03:23 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | 03:23 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | 03:23 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | 03:23 |
| openstackgerrit | Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466 | 03:38 |
| openstackgerrit | Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466 | 03:40 |
| *** boxiang has quit IRC | 03:46 | |
| *** boxiang has joined #openstack-nova | 03:47 | |
| openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | 03:57 |
| *** markvoelker has joined #openstack-nova | 04:00 | |
| *** markvoelker has quit IRC | 04:04 | |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | 04:10 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | 04:10 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | 04:10 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | 04:10 |
| *** igordc has quit IRC | 04:11 | |
| *** markvoelker has joined #openstack-nova | 04:30 | |
| *** markvoelker has quit IRC | 04:35 | |
| *** jaosorior has quit IRC | 04:40 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** slaweq has quit IRC | 05:16 | |
| gmann | gibi: i replied on 675371. resize is difficult to say smoke operation. IMO too we should go with job side regex option | 05:23 |
| *** boxiang has quit IRC | 05:24 | |
| *** boxiang has joined #openstack-nova | 05:24 | |
| *** BjoernT has quit IRC | 05:26 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670 | 05:26 |
| *** ratailor has joined #openstack-nova | 05:32 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670 | 05:34 |
| *** boxiang has quit IRC | 05:40 | |
| *** boxiang has joined #openstack-nova | 05:40 | |
| *** dpawlik has joined #openstack-nova | 05:43 | |
| openstackgerrit | Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466 | 05:46 |
| openstackgerrit | YumengBao proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | 05:55 |
| openstackgerrit | YumengBao proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | 05:55 |
| openstackgerrit | YumengBao proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | 05:56 |
| *** phasespace has quit IRC | 06:05 | |
| *** belmoreira has joined #openstack-nova | 06:15 | |
| *** belmoreira has quit IRC | 06:16 | |
| *** boxiang has quit IRC | 06:21 | |
| *** boxiang has joined #openstack-nova | 06:21 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452 | 06:22 |
| *** ociuhandu has joined #openstack-nova | 06:31 | |
| *** ociuhandu has quit IRC | 06:35 | |
| *** slaweq has joined #openstack-nova | 06:51 | |
| *** dpawlik has quit IRC | 06:51 | |
| *** brinzhang_ has joined #openstack-nova | 06:56 | |
| *** brinzhang has quit IRC | 06:59 | |
| *** dpawlik has joined #openstack-nova | 07:00 | |
| *** bhagyashris has joined #openstack-nova | 07:02 | |
| *** markvoelker has joined #openstack-nova | 07:05 | |
| *** smcginnis|pto has quit IRC | 07:08 | |
| *** markvoelker has quit IRC | 07:10 | |
| *** brinzhang_ has quit IRC | 07:17 | |
| *** brinzhang_ has joined #openstack-nova | 07:18 | |
| openstackgerrit | Merged openstack/nova master: config: remove deprecated checksum options https://review.opendev.org/676539 | 07:25 |
| *** trident has quit IRC | 08:03 | |
| *** ivve has joined #openstack-nova | 08:07 | |
| *** trident has joined #openstack-nova | 08:11 | |
| *** jaosorior has joined #openstack-nova | 08:11 | |
| *** jaosorior has quit IRC | 08:11 | |
| *** tkajinam has quit IRC | 08:27 | |
| gibi | gmann: thanks for the feedback in 675371 . I will try to go with the job side solution | 08:28 |
| *** derekh has joined #openstack-nova | 08:37 | |
| *** ircuser-1 has joined #openstack-nova | 08:43 | |
| *** kaisers has quit IRC | 08:44 | |
| *** kaisers has joined #openstack-nova | 08:45 | |
| *** jaosorior has joined #openstack-nova | 09:02 | |
| *** kaisers has quit IRC | 09:31 | |
| *** kaisers has joined #openstack-nova | 09:41 | |
| *** jaosorior has quit IRC | 09:41 | |
| *** jaosorior has joined #openstack-nova | 09:44 | |
| *** cdent has joined #openstack-nova | 09:46 | |
| *** ociuhandu has joined #openstack-nova | 09:48 | |
| *** ociuhandu has quit IRC | 09:50 | |
| *** bhagyashris has quit IRC | 09:59 | |
| *** markvoelker has joined #openstack-nova | 10:01 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Rename 'nova.common.config' module to 'nova.middleware' https://review.opendev.org/676898 | 10:05 |
| *** markvoelker has quit IRC | 10:10 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Mock libvirt'y things in setUp https://review.opendev.org/676727 | 10:14 |
| *** brinzhang_ has quit IRC | 10:20 | |
| *** ociuhandu has joined #openstack-nova | 10:25 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.opendev.org/636508 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.opendev.org/639280 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826 | 10:27 |
| *** jaosorior has quit IRC | 10:35 | |
| *** tbarron has joined #openstack-nova | 10:38 | |
| *** ociuhandu has quit IRC | 10:38 | |
| *** ociuhandu has joined #openstack-nova | 10:39 | |
| *** ociuhandu has quit IRC | 10:42 | |
| *** ociuhandu has joined #openstack-nova | 10:42 | |
| *** markvoelker has joined #openstack-nova | 10:45 | |
| *** markvoelker has quit IRC | 10:50 | |
| *** ociuhandu has quit IRC | 10:53 | |
| *** tbachman has quit IRC | 10:57 | |
| *** ociuhandu has joined #openstack-nova | 10:58 | |
| tbarron | when one live-migrates an instance that has a floating IP does one need to do anything to bring over the FIP ? | 11:03 |
| sean-k-mooney | no | 11:04 |
| tbarron | sean-k-mooney: thanks for confirming | 11:04 |
| tbarron | sean-k-mooney: even with DVR, right? | 11:04 |
| tbarron | not reporting a bug, just checking :) | 11:04 |
| sean-k-mooney | the floating ip translation happens on the networking nodes on in the dvr fip namespeace. that will be set up in all routers that service the neutron router | 11:04 |
| sean-k-mooney | tbarron: this is all handeled in neutron by the way not nova. but for dvr it will jsut work too | 11:05 |
| tbarron | sean-k-mooney: yeah i realize i might ought to be asking in neutron but behaviorally it's a nova migration so | 11:06 |
| tbarron | I figured people here would know the answer | 11:06 |
| tbarron | sean-k-mooney: thanks much | 11:06 |
| sean-k-mooney | worst case senarior your upstream router/switch might send the packet to the old host which will do the translation but when the vm replies it will go direct form the new node and it should learn that the mac/fip has moved port on the TOR | 11:07 |
| tbarron | sean-k-mooney: makes sense | 11:08 |
| sean-k-mooney | stephenfin: that privsep seriese still has a lot of conflit. do you think we should just bit the bullet and try and merge it all n one go | 11:09 |
| stephenfin | sean-k-mooney: I imagine we should, yeah | 11:09 |
| stephenfin | well, it's really just the last patch that causes most of them, looking at it now | 11:11 |
| sean-k-mooney | cool it would be nice if we could merge it today/over teh weekend | 11:11 |
| stephenfin | the first has a lot of conflicts but with older patches | 11:11 |
| stephenfin | efried is our man, I suspect | 11:11 |
| sean-k-mooney | or perhas gibi i saw him around earlier | 11:12 |
| sean-k-mooney | it would be nice to finally be rid of rootwrap in train. then we can start using privsep properly | 11:12 |
| *** cdent has quit IRC | 11:25 | |
| *** tbachman has joined #openstack-nova | 11:37 | |
| *** ociuhandu has quit IRC | 11:38 | |
| *** ociuhandu has joined #openstack-nova | 11:40 | |
| gibi | sean-k-mooney, stephenfin: I will try to look at it but no promises for today | 11:40 |
| *** artom has joined #openstack-nova | 11:42 | |
| sean-k-mooney | gibi: its no urgent its just it would be nice to finally be done with that series | 11:42 |
| sean-k-mooney | so no worries if you dont get to it | 11:42 |
| *** tbachman has quit IRC | 11:44 | |
| *** ociuhandu has quit IRC | 11:44 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338 | 11:46 |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338 | 11:47 |
| *** markvoelker has joined #openstack-nova | 11:48 | |
| *** ociuhandu has joined #openstack-nova | 11:50 | |
| *** ratailor has quit IRC | 11:51 | |
| *** mgoddard has quit IRC | 11:52 | |
| *** belmoreira has joined #openstack-nova | 11:53 | |
| *** ociuhandu has quit IRC | 11:55 | |
| *** mgoddard has joined #openstack-nova | 11:59 | |
| sean-k-mooney | why do we have some unit test that take 160s to run | 12:04 |
| sean-k-mooney | there has to be something wrong it nova.tests.unit.api.openstack.compute.test_services.ServicesTestV21.test_services_disable | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338 | 12:06 |
| *** maciejjozefczyk has joined #openstack-nova | 12:08 | |
| *** cdent has joined #openstack-nova | 12:10 | |
| *** maciejjozefczyk has quit IRC | 12:10 | |
| gibi | sean-k-mooney: that rings a bell to me | 12:13 |
| gibi | sean-k-mooney: https://review.opendev.org/#/c/675935/ | 12:14 |
| sean-k-mooney | there are several that at least locally are several minutes | 12:14 |
| gibi | sean-k-mooney: I fixed a very same issue in the functional test suite | 12:14 |
| * sean-k-mooney clicks | 12:14 | |
| * gibi starts a unit test run locally | 12:15 | |
| sean-k-mooney | im running them on my workstation where i dont have devstack now to see if that makes a difference | 12:15 |
| sean-k-mooney | well actully i did not have devstack on my laptop... | 12:15 |
| gibi | it seems it takes long on my end too | 12:17 |
| sean-k-mooney | but ya it could be a similar missing stub | 12:17 |
| gibi | let me mock the RPC call to see if that helps | 12:17 |
| gibi | sean-k-mooney: yep the same mock helps | 12:19 |
| gibi | pushing a fix... | 12:19 |
| sean-k-mooney | cool | 12:19 |
| sean-k-mooney | so it was just trying to hit rabbit and timeing out or somehting wehn enableding/disabling rabbit | 12:20 |
| gibi | sean-k-mooney: yes. since https://review.opendev.org/#/q/If32bca070185937ef83f689b7163d965a89ec10a the service disable/enable api call call down to the compute so the compute can update traits in placement | 12:20 |
| gibi | in these tests there is no compute so we are waiting for a timeout that is then catched and droped by the caller | 12:21 |
| gibi | s/catched/caught/ | 12:21 |
| sean-k-mooney | im surpised no one completed about long running ci jobs | 12:22 |
| sean-k-mooney | although i guess tempest will always be longer | 12:22 |
| gibi | I saw the slow functional tests locally but somehow I did not found the same for the unit test, I guess functional takes longer so I always waited for that | 12:27 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Avoid timeout from service update api unit tests https://review.opendev.org/676923 | 12:28 |
| sean-k-mooney | i just noticed because i went to grab my phone form my bedroom and then got a glass of water and noticed it was still running and it looked like it was hung. | 12:29 |
| sean-k-mooney | then it updated with a long test | 12:29 |
| sean-k-mooney | im seeing some other errors too. i just checkout out the latest master and ran them and im getting "TypeError: missing a required argument: 'project_id'" | 12:30 |
| sean-k-mooney | i wonder if i could have some stale pyc files? | 12:31 |
| efried | stephenfin: Thanks for catching up the privsep series. Hopefully I'll get a chance to +A those suckers today. | 12:32 |
| *** tbachman has joined #openstack-nova | 12:33 | |
| *** tbachman has quit IRC | 12:34 | |
| gibi | sean-k-mooney: the rest of the py27 run was clean for me top of master | 12:36 |
| sean-k-mooney | gibi: im seeing this on py36 http://paste.openstack.org/show/757964/ | 12:37 |
| sean-k-mooney | im cloneing a fresh repo to see if it makes any difference | 12:37 |
| sean-k-mooney | there are 3 other failure all in the sam test class | 12:37 |
| * gibi starts a py36 run | 12:38 | |
| sean-k-mooney | the project id is not set in the context object | 12:40 |
| sean-k-mooney | oh it is | 12:40 |
| *** ociuhandu has joined #openstack-nova | 12:40 | |
| sean-k-mooney | its uuids.my_tenant | 12:40 |
| *** luksky has joined #openstack-nova | 12:41 | |
| gibi | sean-k-mooney: py36 was clean for me too | 12:44 |
| sean-k-mooney | ill know in a second if the clean repo is clean | 12:45 |
| sean-k-mooney | i delete the pyc files and it still happened so i dont know what it could be in the other one | 12:45 |
| sean-k-mooney | oh ya i have to wait for those tests without the rpc stub... | 12:46 |
| sean-k-mooney | nope same 4 failtures on a clean clone | 12:47 |
| gibi | if it still fails for you then it might be the virtualenv. I did not recreate that | 12:48 |
| *** tbachman has joined #openstack-nova | 12:48 | |
| sean-k-mooney | http://paste.openstack.org/show/757965/ | 12:49 |
| openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Test sdk Connection(strict_proxies) https://review.opendev.org/676926 | 12:50 |
| efried | mordred: There's the sniffer ^ | 12:50 |
| mordred | efried: fingers crossed | 12:50 |
| sean-k-mooney | gibi it looks like an issue with " mocked_client = mock.create_autospec(client.Client)" | 12:56 |
| sean-k-mooney | i progess passted that but when the client is used we get teh exception | 12:57 |
| efried | what? Autospec causing problems? Never! | 12:57 |
| gibi | sean-k-mooney: I still cannot reproduce it with a recreated virtualenv. Im using Python 3.6.9 | 12:57 |
| *** ociuhandu has quit IRC | 12:57 | |
| sean-k-mooney | Python 3.6.8 | 12:58 |
| sean-k-mooney | that is on rhel8 | 12:58 |
| sean-k-mooney | ill try it on a ubunut system | 12:58 |
| gibi | sean-k-mooney: I'm on 16.04 | 13:00 |
| gibi | sean-k-mooney: with ppa:deadsnakes/ppa as the python source | 13:01 |
| sean-k-mooney | it passed for my on python 3.6.8 on popos which is based ubuntu 18.04 | 13:01 |
| sean-k-mooney | so this might be a rhel8 issue | 13:01 |
| sean-k-mooney | i keep finding rhel8 issues that make me wonder why im trying to using... | 13:02 |
| sean-k-mooney | i am so tempted to just install ubunut 18.04 but not quite yet... | 13:02 |
| *** nweinber has joined #openstack-nova | 13:05 | |
| sean-k-mooney | gibi: thanks for checking :) | 13:06 |
| *** tbachman_ has joined #openstack-nova | 13:06 | |
| *** tbachman has quit IRC | 13:07 | |
| *** tbachman_ is now known as tbachman | 13:07 | |
| *** tbachman has quit IRC | 13:14 | |
| *** tbachman has joined #openstack-nova | 13:18 | |
| KeithMnemonic | Can someone please see if they can WF+1 this change now https://review.opendev.org/#/c/674916/ ? | 13:21 |
| *** tbachman has quit IRC | 13:24 | |
| *** dpawlik has quit IRC | 13:24 | |
| efried | KeithMnemonic: You'll need a stable. mriedem is on vacation. dansmith or melwitt should be able to help once they arrive on the scene. (They're US Pacific, so it might be a bit.) | 13:25 |
| KeithMnemonic | efried: thanks! | 13:27 |
| dansmith | efried: 'scuse me? | 13:33 |
| efried | dansmith: KeithMnemonic is looking for a second stable to push https://review.opendev.org/#/c/674916/ | 13:34 |
| dansmith | efried: I know, | 13:34 |
| efried | and yes, I know you're an early bird | 13:34 |
| dansmith | mkay. | 13:34 |
| efried | I was giving you an excuse to finish your coffee | 13:35 |
| dansmith | hrmf. | 13:35 |
| efried | you're welcome. | 13:35 |
| *** ociuhandu has joined #openstack-nova | 13:38 | |
| *** ociuhandu has quit IRC | 13:42 | |
| KeithMnemonic | dansmith: Thanks! | 13:43 |
| dansmith | yup | 13:44 |
| *** ociuhandu has joined #openstack-nova | 13:45 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Recalculate 'RequestSpec.numa_topology' on resize https://review.opendev.org/662522 | 13:47 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Update docstring of 'revert_resize' function https://review.opendev.org/665976 | 13:47 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Cleanup of '_test_resize' helper test https://review.opendev.org/664245 | 13:47 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Add '_setup_compute_services' helper https://review.opendev.org/663102 | 13:47 |
| aspiers | stephenfin: I've extracted this as requested https://review.opendev.org/#/c/638680/38//COMMIT_MSG@43 - will submit soon | 13:51 |
| aspiers | stephenfin: you said you have another use for it https://review.opendev.org/#/c/638680/38/nova/tests/functional/libvirt/integrated_helpers.py@24 so thought I'd mention | 13:51 |
| stephenfin | aspiers: Sweet | 13:52 |
| *** ratailor has joined #openstack-nova | 13:54 | |
| *** mlavalle has joined #openstack-nova | 13:54 | |
| yonglihe | stephenfin: do you have bandwidth to review https://review.opendev.org/#/c/621476/ , and you are NUMA guy. 😃 | 14:09 |
| yonglihe | Hope this not too surprise 😄 | 14:12 |
| *** dave-mccowan has joined #openstack-nova | 14:22 | |
| *** tbachman has joined #openstack-nova | 14:26 | |
| *** liuyulong has joined #openstack-nova | 14:31 | |
| *** ratailor has quit IRC | 14:32 | |
| *** dave-mccowan has quit IRC | 14:35 | |
| *** belmoreira has quit IRC | 14:36 | |
| *** cfriesen has joined #openstack-nova | 14:39 | |
| efried | mordred, dustinc: results are in https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b | 14:42 |
| efried | (failed, just starting to dig now) | 14:42 |
| *** nweinber has quit IRC | 14:42 | |
| dustinc | Sigh.. | 14:43 |
| mordred | efried: poop | 14:43 |
| mordred | efried: I blame ... uhm ... blast, jaypipes isn't here anymore | 14:43 |
| aspiers | :) | 14:44 |
| sean-k-mooney | mordred: blame friday and the gates hate of our weekends | 14:44 |
| sean-k-mooney | or at least efried weekend | 14:44 |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 14:46 |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964 | 14:46 |
| aspiers | stephenfin: ^^^ | 14:47 |
| *** bnemec is now known as beekneemech | 14:48 | |
| efried | We got the right sdk, I think https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/job-output.txt#924 | 14:49 |
| sean-k-mooney | has the battle for 2.76 concluded by the way or is the power-update patch still under review | 14:49 |
| efried | It looks like we're now raising VirtDriverNotReady properly when we get the failure to connect to ironic ++ | 14:52 |
| efried | https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/controller/logs/screen-n-cpu.txt.gz?severity=1#935 | 14:52 |
| *** nweinber has joined #openstack-nova | 14:53 | |
| efried | ...but then we stop getting that error, which implies that we got the connection good, which is what we expect --- but we still get ComputeHostNotFound | 14:54 |
| efried | dustinc: It's possible that this ^ is an actual bug on the nova side | 14:55 |
| *** lyarwood has quit IRC | 14:56 | |
| *** ivve has quit IRC | 14:57 | |
| mordred | efried: yay! that does seem like the sequence of actions we want | 14:57 |
| *** BjoernT has joined #openstack-nova | 14:58 | |
| efried | Reckon a little extra logging would help out here. | 14:58 |
| *** BjoernT_ has joined #openstack-nova | 15:03 | |
| *** BjoernT has quit IRC | 15:04 | |
| *** dklyle has joined #openstack-nova | 15:05 | |
| *** lee1 has joined #openstack-nova | 15:05 | |
| *** lee1 is now known as lyarwood | 15:06 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate server with ports having resource request https://review.opendev.org/655113 | 15:08 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138 | 15:08 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140 | 15:08 |
| *** bbowen has quit IRC | 15:10 | |
| *** gyee has joined #openstack-nova | 15:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | 15:15 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972 | 15:15 |
| *** Garyx has joined #openstack-nova | 15:18 | |
| *** hemna has quit IRC | 15:18 | |
| openstackgerrit | Chris Dent proposed openstack/nova stable/stein: Add an issue releasenote for placement eventlet stall https://review.opendev.org/676973 | 15:22 |
| cdent | efried, belmoreira : first draft ^ | 15:23 |
| *** hemna has joined #openstack-nova | 15:27 | |
| melwitt | cdent: fyi I had this similar reno going, is it for the same thing or no? https://review.opendev.org/662095 | 15:33 |
| cdent | same issue, different context, so maybe worth having both? | 15:34 |
| *** tbachman has quit IRC | 15:34 | |
| cdent | Not sure whether combining they will be useful or not | 15:34 |
| melwitt | ok. I'm on a call right now but skimmed over | 15:34 |
| melwitt | if they are different then that's fine. wasn't sure at a glance | 15:35 |
| openstackgerrit | sean mooney proposed openstack/nova master: Deprecate Aggregate[Core|Ram|Disk]Filters https://review.opendev.org/673496 | 15:35 |
| *** gyee has quit IRC | 15:35 | |
| *** luksky has quit IRC | 15:37 | |
| *** ociuhandu has quit IRC | 15:40 | |
| *** ociuhandu has joined #openstack-nova | 15:41 | |
| *** eharney has joined #openstack-nova | 15:42 | |
| *** ociuhandu has quit IRC | 15:45 | |
| *** gyee has joined #openstack-nova | 15:48 | |
| *** ociuhandu has joined #openstack-nova | 15:50 | |
| *** gyee has quit IRC | 15:52 | |
| *** ociuhandu has quit IRC | 15:54 | |
| *** ociuhandu has joined #openstack-nova | 15:55 | |
| openstackgerrit | Merged openstack/nova stable/rocky: Add 'path' query parameter to console access url https://review.opendev.org/674916 | 15:55 |
| openstackgerrit | Merged openstack/nova master: Avoid timeout from service update api unit tests https://review.opendev.org/676923 | 15:55 |
| efried | mordred, dustinc: I can't figure it out so far. | 15:56 |
| efried | I'm not very experienced looking at ironic logs (maybe we can bring in a ringer) but afaict we make requests for /nodes that return nothing (I assume 13 bytes is "no results") for a while, and then start returning stuff, whereupon we actually start requesting and receiving real data. | 15:56 |
| efried | And yet, the nova side never seems to "get there". | 15:56 |
| efried | The devstack times out waiting for n-api to respond, which is weird, because n-api looks active https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/controller/logs/screen-n-api.txt.gz | 15:56 |
| efried | Not sure if that /flavors/baremetal business is "normal" or not. | 15:56 |
| efried | Could it really be that we got a spurious failure? That would seem a terrible coincidence. I'll recheck just in case. | 15:56 |
| efried | I need to run for ~1h. donnyd is spinning us up an env where we can play with this more realtime-ish. | 15:56 |
| *** efried is now known as efried_afk | 15:56 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980 | 16:03 |
| *** gyee has joined #openstack-nova | 16:08 | |
| dustinc | efried: awesome, thanks | 16:10 |
| *** tbachman has joined #openstack-nova | 16:17 | |
| *** bnemec has joined #openstack-nova | 16:19 | |
| *** beekneemech has quit IRC | 16:19 | |
| *** bnemec is now known as beekneemech | 16:20 | |
| *** cdent has quit IRC | 16:22 | |
| *** ociuhandu_ has joined #openstack-nova | 16:28 | |
| *** nweinber has quit IRC | 16:29 | |
| *** ociuhandu has quit IRC | 16:31 | |
| *** beekneemech has quit IRC | 16:32 | |
| *** ociuhandu_ has quit IRC | 16:32 | |
| *** boxiang has quit IRC | 16:35 | |
| *** bnemec has joined #openstack-nova | 16:35 | |
| *** boxiang has joined #openstack-nova | 16:36 | |
| *** macz has joined #openstack-nova | 16:37 | |
| *** boxiang has quit IRC | 16:37 | |
| *** boxiang has joined #openstack-nova | 16:38 | |
| *** boxiang has quit IRC | 16:39 | |
| *** boxiang has joined #openstack-nova | 16:40 | |
| donnyd | I should have something to use in a bit, maybe an hour or so | 16:40 |
| *** bnemec has quit IRC | 16:44 | |
| *** bnemec has joined #openstack-nova | 16:45 | |
| *** derekh has quit IRC | 16:48 | |
| *** ivve has joined #openstack-nova | 16:49 | |
| *** ivve has quit IRC | 16:49 | |
| *** eharney has quit IRC | 17:00 | |
| *** panda has quit IRC | 17:00 | |
| *** bnemec has quit IRC | 17:02 | |
| *** panda has joined #openstack-nova | 17:02 | |
| *** bnemec has joined #openstack-nova | 17:03 | |
| *** igordc has joined #openstack-nova | 17:04 | |
| *** ociuhandu has joined #openstack-nova | 17:05 | |
| *** mgoddard has quit IRC | 17:07 | |
| *** igordc has quit IRC | 17:08 | |
| *** ociuhandu has quit IRC | 17:09 | |
| *** ivve has joined #openstack-nova | 17:09 | |
| *** markvoelker has quit IRC | 17:10 | |
| *** mgoddard has joined #openstack-nova | 17:11 | |
| stephenfin | yonglihe: Sorry, missed the ping. I'll take a look at that Monday, sure :) | 17:12 |
| *** yaawang has quit IRC | 17:14 | |
| *** bnemec has quit IRC | 17:14 | |
| *** yaawang has joined #openstack-nova | 17:14 | |
| *** bnemec has joined #openstack-nova | 17:15 | |
| *** markvoelker has joined #openstack-nova | 17:18 | |
| *** mgoddard has quit IRC | 17:22 | |
| openstackgerrit | ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299 | 17:23 |
| openstackgerrit | ya.wang proposed openstack/nova master: vCPU model selection https://review.opendev.org/670298 | 17:23 |
| openstackgerrit | ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300 | 17:23 |
| *** mgoddard has joined #openstack-nova | 17:23 | |
| *** nweinber has joined #openstack-nova | 17:24 | |
| *** igordc has joined #openstack-nova | 17:26 | |
| *** bnemec has quit IRC | 17:31 | |
| *** bnemec has joined #openstack-nova | 17:34 | |
| *** dklyle has quit IRC | 17:40 | |
| *** bnemec has quit IRC | 17:40 | |
| *** bnemec has joined #openstack-nova | 17:42 | |
| sean-k-mooney | hum look like gerrit is down | 17:42 |
| sean-k-mooney | i think that is a sign | 17:42 |
| mnaser | sean-k-mooney: wfm | 17:42 |
| sean-k-mooney | nope cant open any of the links above | 17:43 |
| mnaser | i can so many its your side :) | 17:43 |
| mnaser | and also a sign :) | 17:43 |
| sean-k-mooney | yep its friday and 6:43 | 17:43 |
| sean-k-mooney | time to not be working | 17:43 |
| *** igordc has quit IRC | 17:49 | |
| *** igordc has joined #openstack-nova | 17:50 | |
| *** nweinber has quit IRC | 18:02 | |
| *** ivve has quit IRC | 18:07 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Mock libvirt'y things in setUp https://review.opendev.org/676727 | 18:07 |
| *** liuyulong has quit IRC | 18:15 | |
| *** eharney has joined #openstack-nova | 18:21 | |
| *** ricolin has quit IRC | 18:24 | |
| *** tbachman has quit IRC | 18:47 | |
| efried_afk | alex_xu: I had to give up on https://review.opendev.org/#/c/673151/ -- would you please review it when you get a chance? | 18:52 |
| *** efried_afk is now known as efried | 18:52 | |
| *** nweinber has joined #openstack-nova | 18:53 | |
| openstackgerrit | Merged openstack/nova master: Allow assertXmlEqual() to pass options to matchers.XMLMatches https://review.opendev.org/674628 | 19:12 |
| openstackgerrit | Merged openstack/nova master: Fix libvirt driver tests to use LibvirtConfigCapsGuest instances https://review.opendev.org/674629 | 19:12 |
| artom | dansmith, thanks for all the reviews - I won't get to them all this afternoon, obviously, but I'm aiming for Monday | 19:12 |
| dansmith | ack | 19:13 |
| efried | aspiers: https://review.opendev.org/#/c/638680/ qq if you're still around. | 19:16 |
| *** bbowen has joined #openstack-nova | 19:16 | |
| *** ociuhandu has joined #openstack-nova | 19:28 | |
| *** ociuhandu has quit IRC | 19:29 | |
| *** ociuhandu has joined #openstack-nova | 19:31 | |
| *** ociuhandu has quit IRC | 19:34 | |
| *** ociuhandu has joined #openstack-nova | 19:34 | |
| *** nweinber has quit IRC | 19:38 | |
| *** tbachman has joined #openstack-nova | 19:40 | |
| *** tbachman has quit IRC | 19:45 | |
| *** ociuhandu has quit IRC | 19:48 | |
| *** ociuhandu has joined #openstack-nova | 19:48 | |
| *** ociuhandu has quit IRC | 19:56 | |
| *** nweinber has joined #openstack-nova | 20:02 | |
| *** bnemec is now known as beekneemech | 20:14 | |
| aspiers | efried: yup? | 20:16 |
| efried | aspiers: see inline, want to make sure caching-only-False is intentional | 20:16 |
| aspiers | ok | 20:17 |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151 | 20:22 |
| *** nweinber has quit IRC | 20:23 | |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151 | 20:23 |
| aspiers | efried: are there questions I can answer which would make reviewing https://review.opendev.org/#/c/673151/ a possibility for you? | 20:24 |
| aspiers | efried: I really doubt I know significantly more than you in this area - you're probably just missing one or two key facts | 20:25 |
| *** markvoelker has quit IRC | 20:25 | |
| efried | aspiers: I am missing domain (so to speak) knowledge of libvirt+xml | 20:25 |
| efried | like, the whole thing. | 20:25 |
| efried | and machine types | 20:25 |
| efried | and architectures | 20:25 |
| efried | It's just all an enormous hole in my brain. | 20:26 |
| aspiers | haha | 20:26 |
| aspiers | it's really not that hard, honest :) | 20:26 |
| efried | be like asking me to review, I dunno, a Windows kernel patch. | 20:26 |
| aspiers | LOL | 20:26 |
| aspiers | I doubt it | 20:26 |
| efried | aspiers: But it's this one that I was asking about https://review.opendev.org/#/c/638680/41 | 20:27 |
| efried | specifically https://review.opendev.org/#/c/638680/41/nova/virt/libvirt/host.py@1186 | 20:27 |
| aspiers | oh, looks like I screwed it up | 20:27 |
| *** tbachman has joined #openstack-nova | 20:30 | |
| *** markvoelker has joined #openstack-nova | 20:30 | |
| openstackgerrit | James E. Blair proposed openstack/nova master: WIP: sphinx latex build tests https://review.opendev.org/677028 | 20:31 |
| efried | okay | 20:33 |
| *** markvoelker has quit IRC | 20:33 | |
| *** markvoelker has joined #openstack-nova | 20:33 | |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 20:44 |
| efried | dustinc, mordred: Okay, donnyd set up us the cloud, and I've got it configured to reproduce the failure | 20:44 |
| efried | ...which afaict is, we're talking to ironic okay, but for some reason the node.list is coming back with 0 nodes. | 20:44 |
| efried | so dustinc this might be where you need to jump in and debug. | 20:44 |
| *** ociuhandu has joined #openstack-nova | 20:48 | |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964 | 20:49 |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 20:49 |
| *** spatel has joined #openstack-nova | 20:53 | |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 20:54 |
| aspiers | efried: what's the world record for the most patchsets in a nova review? :-o | 20:54 |
| aspiers | am I even close yet? | 20:54 |
| aspiers | seems like a very dubious achievement | 20:54 |
| efried | you are nowhere close. | 20:55 |
| efried | Well into three figures | 20:55 |
| aspiers | hah, phew | 20:55 |
| aspiers | wow | 20:55 |
| *** spatel has quit IRC | 20:57 | |
| *** pcaruana has quit IRC | 20:57 | |
| *** ociuhandu has quit IRC | 20:58 | |
| *** eharney has quit IRC | 20:59 | |
| *** igordc has quit IRC | 21:01 | |
| *** igordc has joined #openstack-nova | 21:01 | |
| *** KeithMnemonic has quit IRC | 21:02 | |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964 | 21:12 |
| openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | 21:12 |
| *** markvoelker has quit IRC | 21:12 | |
| aspiers | Damn, I intended to not work at all today, and here I am still going ... time for late dinner | 21:14 |
| aspiers | Just desperate to get this damn series all to +2 | 21:14 |
| efried | aspiers: https://review.opendev.org/#/c/676964/3/nova/tests/functional/libvirt/test_report_cpu_traits.py show me where STUB_INIT_HOST is being used? | 21:16 |
| aspiers | efried: ah sorry, it *is* in the next patch | 21:19 |
| efried | It's not worth respinning stuff for, just want to make sure I'm not blind. | 21:19 |
| aspiers | yeah | 21:19 |
| efried | (I got glasses day before yesterday, for the first time in my life) | 21:20 |
| aspiers | I ran my grep with the wrong commit checked out | 21:20 |
| aspiers | hah, welcome to nerd club | 21:20 |
| aspiers | I've been in it since I was 14 :-( | 21:20 |
| aspiers | well, since I was born I suppose | 21:20 |
| aspiers | OK, dinner | 21:27 |
| aspiers | thanks and catch you next week! o/ | 21:27 |
| efried | mordred, dustinc: I found the bug | 21:27 |
| efried | It looks like you can only run through the generator returned by conn.baremetal.nodes() once. | 21:28 |
| *** mgoddard has quit IRC | 21:29 | |
| efried | which, now that I think about it, makes sense. | 21:29 |
| efried | I mean, now that I've seen this thing in action and think about it. | 21:29 |
| efried | Because having done that I learned 1) that it's a generator, and 2) that it doesn't actually do anything, like go out over the wire, until you start iterating over it. | 21:30 |
| efried | dustinc: Ima fix that up in the node.list patch (and rebase the rest on top, cause they definitely won't work without that) and let's see where we get to. | 21:31 |
| efried | (that will probably break UT; but I'll leave you to fix that :P ) | 21:31 |
| *** mgoddard has joined #openstack-nova | 21:32 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027 | 21:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028 | 21:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690 | 21:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | 21:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | 21:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Test sdk Connection(strict_proxies) https://review.opendev.org/676926 | 21:33 |
| efried | I... don't actually know whether strict_proxies is gold here. But that last patch ^ ought to tell us. | 21:34 |
| * donnyd starts the weekending | 21:38 | |
| efried | Thanks for your help donnyd | 21:42 |
| *** BjoernT_ has quit IRC | 21:45 | |
| donnyd | glad I can be helpful efried | 22:04 |
| *** bbowen has quit IRC | 22:06 | |
| *** markvoelker has joined #openstack-nova | 22:10 | |
| *** markvoelker has quit IRC | 22:15 | |
| *** spsurya has quit IRC | 22:23 | |
| *** BjoernT has joined #openstack-nova | 22:24 | |
| *** ociuhandu has joined #openstack-nova | 22:29 | |
| *** ociuhandu has quit IRC | 22:33 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Limit get_sdk_adapter to requested service type https://review.opendev.org/674678 | 22:36 |
| *** igordc has quit IRC | 22:46 | |
| *** macz has quit IRC | 22:58 | |
| *** ociuhandu has joined #openstack-nova | 23:00 | |
| *** ociuhandu has quit IRC | 23:05 | |
| *** BjoernT has quit IRC | 23:07 | |
| *** vesper11 has quit IRC | 23:32 | |
| *** dviroel has joined #openstack-nova | 23:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!