Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-09-08
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: Improve orphans tracking in resource tracker https://review.opendev.org/745282
03:42:17 openstackgerrit Luyao Zhong proposed openstack/nova master: Rollback when reset running live migration https://review.opendev.org/747899
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: Add and use 'GlanceFixture' https://review.opendev.org/745907
09:05:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Enable real policy fixture by default https://review.opendev.org/745906
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: Remove 'kwargs' from 'get_instance_nw_info' https://review.opendev.org/744872
09:39:06 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add type hints https://review.opendev.org/744869
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 :)
13:04:47 gibi yingjisun: I will approve the undeprecate patch on Thursday
13:05:02 yingjisun I see. Thank you.
13:05:07 lyarwood gmann: https://review.opendev.org/#/c/744883/ - I'm trying to wrap the existing gate script for evacuation in an ansible role as you have done for the parts of grenade
13:05:31 lyarwood gmann: any idea what I'm doing wrong here as zuul isn't able to find the role I've added in the same change
13:05:48 gmann lyarwood: checking..
13:06:34 gmann 1 min, fixing policy patch pep8
13:06:46 tosky lyarwood: - run-nova-evacuate-hook but the role is roles/run-evacuate-hook/README.rst
13:06:57 lyarwood argh!
13:07:04 tosky :)
13:07:12 lyarwood tosky: thanks :)
13:09:24 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
13:10:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
13:12:16 gmann stephenfin: updated^^, sorry about that, it did not hit me locally due to type annotation checks
13:13:11 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
13:25:18 openstackgerrit Federico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain https://review.opendev.org/750652
14:04:34 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
14:06:02 sean-k-mooney lyarwood: is that a job ^
14:06:14 sean-k-mooney i assume its not some new api action right
14:06:25 sean-k-mooney it sjust the job where we will test evacuate
14:06:46 tosky it's part of the removal of the remaining live-migration job
14:06:58 sean-k-mooney ya that is what i was assuming
14:07:38 sean-k-mooney i know lee has been workign on that for a while and we discussed that in the past but have not been following this too closely
14:17:26 openstackgerrit Artom Lifshitz proposed openstack/nova stable/train: post live migration: don't call Neutron needlessly https://review.opendev.org/750670
14:21:25 bauzas gibi: fwiw, looking at your change, no question yet
14:21:32 bauzas stephenfin: have you seen my point ?
14:21:39 gibi bauzas: cool, thanks
14:24:10 stephenfin bauzas: oh, sorry, I missed that

Earlier   Later