| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-08 | |||
| 15:51:56 | bauzas | gibi: stephenfin: I'm done for the day but I left your two main changes not accepted yet, but I'll look at them tomorrow | |
| 15:52:04 | stephenfin | ack | |
| 15:52:09 | gibi | bauzas: ack | |
| 15:52:21 | bauzas | (vTPM main change, and SRIOV attach/detach hairy patch) | |
| 15:52:37 | bauzas | hopefully, we can close two blueprints tomorrow | |
| 16:16:38 | openstackgerrit | Elod Illes proposed openstack/nova stable/train: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747610 | |
| 16:20:54 | openstackgerrit | Elod Illes proposed openstack/nova stable/train: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747610 | |
| 16:44:28 | stephenfin | sean-k-mooney: Does my understanding of https://review.opendev.org/#/c/738432/ make sense? | |
| 16:44:37 | stephenfin | Or rather is it correct? | |
| 16:45:30 | sean-k-mooney | reading your commnet on sec | |
| 16:46:12 | sean-k-mooney | yes we are uring the name lookup on the source host with the adress form the dest | |
| 16:46:16 | sean-k-mooney | which is totally broken | |
| 16:46:28 | stephenfin | okay, phew | |
| 16:46:37 | stephenfin | I've been staring at that patch for weeks, heh | |
| 16:47:47 | sean-k-mooney | and yes if the destination device does not exist on the source things explode | |
| 16:48:27 | sean-k-mooney | and more annoyingly sometimes the device on the souce and dest match and this just work by acident | |
| 16:48:35 | sean-k-mooney | untill they dont | |
| 16:48:54 | sean-k-mooney | this is only an issue for macvtap since it does not have the pci adress in teh xml | |
| 16:49:02 | sean-k-mooney | it uses the nic name istead | |
| 16:51:30 | sean-k-mooney | stephenfin: ya moving thie logic up one level would also make sense | |
| 16:51:56 | sean-k-mooney | i.e. caluatle the devname before calling set_vif_host_backend_hw_veb | |
| 16:52:34 | openstackgerrit | Merged openstack/nova master: add functional regression test for bug #1888395 https://review.opendev.org/747454 | |
| 16:52:34 | openstack | bug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to sean mooney (sean-k-mooney) | |
| 17:04:13 | openstackgerrit | Merged openstack/nova master: Expand generic reproducer for bug #1879878 https://review.opendev.org/750186 | |
| 17:04:13 | openstack | bug 1879878 in OpenStack Compute (nova) "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Medium,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane) | |
| 17:33:20 | ganso | Hi Nova folks, could someone please review this small bugfix if you have a few minutes https://review.opendev.org/#/c/748533 thanks in advance! | |
| 17:57:53 | openstackgerrit | Merged openstack/nova master: Set 'old_flavor', 'new_flavor' on source before resize https://review.opendev.org/750187 | |
| 18:40:28 | openstackgerrit | Merged openstack/nova master: Move equality check into LibvirtConfigGuestInterface https://review.opendev.org/744524 | |
| 20:49:00 | artom | This is not making any sense... | |
| 20:49:16 | artom | USE_NEUTRON = True | |
| 20:49:55 | artom | if self.USE_NEUTRON: | |
| 20:49:57 | artom | self.neutron = self.useFixture(nova_fixtures.NeutronFixture(self)) | |
| 20:50:29 | artom | self.test.stub_out( | |
| 20:50:31 | artom | 'nova.network.neutronv2.api.API.setup_networks_on_host' | |
| 20:50:46 | artom | And yet | |
| 20:50:56 | artom | File "nova/network/neutronv2/api.py", line 332, in setup_networks_on_host | |
| 20:50:57 | artom | data = self.list_ports(context, **search_opts) | |
| 20:51:01 | artom | So it didn't get stubbed out | |
| 20:57:20 | artom | Ah, I think it's happening async in the rollback, presumably after the tearDown() has cleanup up the stubs/mocks | |
| 20:59:28 | artom | Aha, yep, adding self.test.flags(vif_plugging_timeout=0) has done it. | |
| 20:59:39 | artom | (From https://review.opendev.org/#/c/696245/2/nova/tests/fixtures.py, which I'm not even going to attempt to backprot) | |
| 21:20:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: nova-evacuate https://review.opendev.org/744883 | |
| 23:26:26 | openstackgerrit | melanie witt proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676 | |
| #openstack-nova - 2020-09-09 | |||
| 03:24:21 | openstackgerrit | MIKI Nobuhiro proposed openstack/nova master: libvirt: add IP address to libvirt guest metadata https://review.opendev.org/750552 | |
| 03:42:16 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Track error migrations in resource tracker https://review.opendev.org/745281 | |
| 03:42:17 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Rollback when reset running live migration https://review.opendev.org/747899 | |
| 03:42:17 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282 | |
| 06:16:28 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282 | |
| 08:17:36 | openstackgerrit | Xinliang Liu proposed openstack/nova master: Always pass image source info to ironic https://review.opendev.org/746626 | |
| 08:19:03 | bauzas | whoops my manners | |
| 08:19:10 | bauzas | good morning Nova | |
| 08:20:00 | gibi | good morning bauzas | |
| 08:25:12 | bauzas | gibi: you're next in the queue, finishing to review the last patch of the vTPM series | |
| 08:25:22 | gibi | OK, thanks | |
| 08:25:47 | gibi | I will prioritize comments on the SRIOV series today and tomorrow (I want to get it merged) | |
| 08:34:42 | openstackgerrit | Liang Fang proposed openstack/nova master: Add volume local cache support https://review.opendev.org/663542 | |
| 08:48:37 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow up for Ibf8dca4bd57b3bddb39955b53cc03564506f5754 https://review.opendev.org/745095 | |
| 08:58:58 | luyao | stephenfin: Hi, FYI. I abandoned the separate patch for driver interface change, I merge it to https://review.opendev.org/#/c/745281/ | |
| 08:59:35 | stephenfin | ack | |
| 08:59:53 | stephenfin | again, apologies for sending you the wrong way on that one /o\ | |
| 09:05:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Enable real policy fixture by default https://review.opendev.org/745906 | |
| 09:05:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add and use 'GlanceFixture' https://review.opendev.org/745907 | |
| 09:05:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove '_FakeImageService' https://review.opendev.org/745908 | |
| 09:14:42 | luyao | stephenfin: it's ok, I thought your suggestion was better at that time, we both didn't notice the issue. :D | |
| 09:16:27 | luyao | stephenfin: and always thanks for your review. :) | |
| 09:39:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Add type hints https://review.opendev.org/744869 | |
| 09:39:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Remove 'kwargs' from 'get_instance_nw_info' https://review.opendev.org/744872 | |
| 09:39:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Remove unused 'affect_auto_assigned' parameter https://review.opendev.org/744871 | |
| 09:39:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Add type hints for various allocate functions https://review.opendev.org/744870 | |
| 09:41:36 | yingjisun | gibi: the latest result is here. http://207.189.188.190/logs/07/742407/6/check-vote/ext-nova-zuul/5e91989/ | |
| 09:41:49 | bauzas | stephenfin: hammered you https://review.opendev.org/#/c/639934/57 | |
| 09:42:14 | yingjisun | the apache2 logs are in the apache2 folder. Please have a look. | |
| 09:42:22 | bauzas | stephenfin: usually I wouldn't bother for this and would be OK for a FUP, but given we're like 24 hours away from FF, please respin quickly | |
| 09:42:38 | bauzas | typey, typey | |
| 09:42:54 | yingjisun | However there is a random failure case. We have recheck it. | |
| 10:55:36 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | |
| 10:59:46 | brinzhang_ | sean-k-mooney, gibi: the change of https://review.opendev.org/#/c/729563/12/nova/conductor/manager.py@1273 was restored, and I left comments in this patch | |
| 12:06:19 | openstackgerrit | Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175 | |
| 12:49:52 | pawan-gupta | Hi, I have setup a Compute Cluster and I am trying to push a small script using vendor data, It is working good in most of the distributions except CentOS-7. when I do cloud-id, it gives me ci-b64:b3B..... instead of openstack, I am not sure, what I am missing here. Cloud-Init version is 18.5. | |
| 12:55:22 | sean-k-mooney | brinzhang_: yep just saw it. ya your right we delete the arqs in shelve offload so in unshelve we just need to create them not delete and create. | |
| 12:56:02 | brinzhang_ | sean-k-monney: yeah | |
| 12:56:20 | brinzhang_ | sean-k-mooney: Ask a questions | |
| 12:57:31 | brinzhang_ | gibi, sean-k-mooney: If we used Nova in Rocky release, and we want to make placement project join the openstack Rocky release, is there any risk? | |
| 12:57:53 | brinzhang_ | In rocky release, the placement still in Nova | |
| 13:00:10 | sean-k-mooney | you mean what use standalone placement fomr a later release instead of in tree placemnt on rocky | |
| 13:00:36 | sean-k-mooney | that should work fine unless we messed up the placement microversions in some way | |
| 13:00:41 | brinzhang_ | We used docker to deploy the OpenStack env, if pull the placement-Ussuri join, it will be a independent docker | |
| 13:01:10 | sean-k-mooney | brinzhang_: ah instead of hosted form the nova api? | |
| 13:01:26 | sean-k-mooney | brinzhang_: i tought it was always a seperate container | |
| 13:01:38 | brinzhang_ | sean-k-mooney: yeah | |
| 13:01:56 | sean-k-mooney | as in i though kolla for example always ran placment in its own container even when it was in tree | |
| 13:02:18 | brinzhang_ | if so I think it work ok, the Ussuri placement works in the OpenStack Rocky env | |
| 13:02:33 | sean-k-mooney | ya it should | |
| 13:02:49 | brinzhang_ | ack thanks sean-k-mooney | |
| 13:02:58 | sean-k-mooney | we havent really tested that but that what microverions are for and they have pretty good microverion testing in placment | |
| 13:03:31 | brinzhang_ | I know the placement microversion, Iwill see it later | |
| 13:04:12 | yingjisun | gibi: here is the latest result of patch | |
| 13:04:24 | yingjisun | http://207.189.188.190/logs/07/742407/6/check-vote/ext-nova-zuul/664e0fe/ | |
| 13:04:25 | gibi | yingjisun: yes I saw it thanks. It looks good | |
| 13:04:42 | yingjisun | :) | |