Posted	Nick	Remark
#openstack-nova - 2019-10-11
04:32:31	openstackgerrit	Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
04:36:43	openstackgerrit	Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
04:37:19	openstackgerrit	Brin Zhang proposed openstack/nova master: Fix legacy issues in Filter migrations by user_id/project_id https://review.opendev.org/682198
04:38:58	openstackgerrit	Brin Zhang proposed openstack/nova master: Fix legacy issues in filter migrations by user_id/project_id https://review.opendev.org/682198
06:00:26	openstackgerrit	ya.wang proposed openstack/nova master: libvirt: Change _compare_cpu to raise InternalError https://review.opendev.org/687808
06:20:32	openstackgerrit	Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
06:45:38	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Change the primary assignee to the mainly contributor https://review.opendev.org/688042
06:46:55	openstackgerrit	Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
06:46:55	openstackgerrit	Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912
08:07:08	openstackgerrit	Merged openstack/nova master: Fix exception translation when creating volume https://review.opendev.org/678991
08:17:42	openstackgerrit	Merged openstack/python-novaclient master: PDF documentation build https://review.opendev.org/687966
08:20:32	openstackgerrit	Merged openstack/nova master: Fix unit of hw_rng:rate_period https://review.opendev.org/683051
08:32:14	openstackgerrit	Lee Yarwood proposed openstack/nova stable/stein: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688067
08:33:52	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688068
08:34:28	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688069
08:39:49	openstack	Launchpad bug 1741364 in OpenStack Compute (nova) "ceph ephemeral info not updated during live migrate" [Medium,Confirmed]
08:39:49	trident	Any thoughts on https://bugs.launchpad.net/nova/+bug/1741364 ?
08:41:19	openstackgerrit	Fan Zhang proposed openstack/nova stable/train: Fix exception translation when creating volume https://review.opendev.org/688072
08:43:05	openstackgerrit	Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
08:43:53	openstackgerrit	Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
09:03:58	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/train: Fix unit of hw_rng:rate_period https://review.opendev.org/688083
09:08:42	openstackgerrit	ya.wang proposed openstack/nova master: libvirt: Change _compare_cpu to raise InvalidCPUInfo https://review.opendev.org/687808
09:21:27	openstackgerrit	Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
09:26:59	openstackgerrit	Fan Zhang proposed openstack/nova stable/rocky: Fix exception translation when creating volume https://review.opendev.org/688089
09:31:40	openstackgerrit	Nicolas Belouin proposed openstack/nova master: Add support for Xen PVH VM Mode in libvirt driver https://review.opendev.org/687827
09:39:13	stephenfin	did we not have a meeting last night
09:41:10	openstackgerrit	Fan Zhang proposed openstack/nova stable/queens: Fix exception translation when creating volume https://review.opendev.org/688090
11:20:27	kashyap	Any stable cores, mind having a look at this stable/queens thing? It has also merged in other stable branches: https://review.opendev.org/#/c/668111/ (libvirt: Rework 'EBUSY' (SIGKILL) error handling code path
11:20:31	kashyap	)
11:21:03	kashyap	(It has merged in Rocky and Stein.)
11:34:26	lyarwood	kashyap: yup sorry looking now
11:34:47	kashyap	Gracias
11:56:27	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/pike: Error out interrupted builds https://review.opendev.org/687918
11:59:21	kashyap	Needs a second stable core +2 here: https://review.opendev.org/#/c/668111/
12:21:39	openstack	Launchpad bug 1847759 in tempest "reboot_server_hard can trigger corrupted file system" [Undecided,New]
12:21:39	frickler	this bug is targeted on tempest, but I'd like to know from nova ppl how valuable testing the hard reboot is to you, as I'm considering to propose dropping this test if it can't be made more reliable https://bugs.launchpad.net/tempest/+bug/1847759
12:25:46	lyarwood	frickler: IMHO it's useful and that sounds worrying as a bug report
12:28:02	frickler	lyarwood: I'm even more worried after seeing how many matches my logstash query has, though I haven't verified yet how many of those match the bug http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22self._test_reboot_server('HARD')%5C%22
12:29:41	lyarwood	frickler: I wonder if we even need to boot an instance using an image to test hard reboot
12:30:25	lyarwood	frickler: if corruption of the image by the guest OS dying suddenly is the issue here
12:36:06	frickler	lyarwood: o.k., I checked some of the failure logs. 2/3 seem to fail to connect to the instance on the initial boot already, 1/3 looks like fs corruption after the hard reboot, so still a relevant set. if you have any idea how to make that more stable, I think that would be pretty useful
12:57:42	bauzas	stephenfin: nope, the nova meeting was cancelled, see ML
13:23:30	stephenfin	bauzas: ah, yes, thanks
13:24:02	stephenfin	bauzas: Would you do me the honours? https://review.opendev.org/#/c/684402/ https://review.opendev.org/#/c/685929/ https://review.opendev.org/#/c/687264// ?
13:25:30	openstackgerrit	Matthew Booth proposed openstack/nova master: Don't take a lock when fetching a cached cell mapping https://review.opendev.org/688122
13:29:15	stephenfin	bauzas: ta
13:35:40	openstackgerrit	Merged openstack/nova master: api-guide: Fix available info in handling down cells https://review.opendev.org/686520
13:41:54	openstackgerrit	Merged openstack/nova-specs master: Change the primary assignee to the mainly contributor https://review.opendev.org/688042
13:41:55	openstackgerrit	Merged openstack/nova-specs master: Fix followup comments of policy-defaults-refresh spec https://review.opendev.org/669196
13:54:13	mdbooth	lyarwood: Re https://code.engineering.redhat.com/gerrit/182841 GAH! and thanks. Mind if we don't wait for tempest after I update the commit message?
13:54:59	lyarwood	mdbooth: wrong window but yeah of course
13:55:28	mdbooth	lyarwood: Hah, so it is
14:11:18	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Avoid using image with kernel in BDM large request func test https://review.opendev.org/688132
14:40:08	gibi	stephenfin: quick question in https://review.opendev.org/#/c/686802/6//COMMIT_MSG@18
14:47:00	stephenfin	gibi: replied
14:47:51	gibi	stephenfin: +2
14:47:58	stephenfin	ta
14:55:54	stephenfin	gibi: If you're on a reviewing streak, I'd appreciate your eyes on the following patch too since it's NeutronFixture'y and you know that stuff, heh https://review.opendev.org/#/c/684344/
14:56:04	stephenfin	Feel free to chuck something my way too
14:56:10	gibi	stephenfin: reviewing it right no
14:56:13	gibi	w
14:56:20	stephenfin	nice :D
14:56:43	dansmith	gibi: there are a few patches in this stack for which you reviewed the spec that you could +W if you want :)
14:56:44	dansmith	gibi: https://review.opendev.org/#/c/687137/4
14:57:05	gibi	dansmith: ack, I will look at it after stephenfin's patch
14:57:28	dansmith	gibi: thanks
14:57:48	gibi	stephenfin: -1 due to https://review.opendev.org/#/c/684344/15/nova/tests/functional/api_sample_tests/test_floating_ips.py@158
14:58:23	openstackgerrit	Dan Smith proposed openstack/nova master: Add image caching API for aggregates https://review.opendev.org/687140
14:58:23	openstackgerrit	Dan Smith proposed openstack/nova master: Add cache_images() to conductor https://review.opendev.org/687139
14:58:24	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Add image precaching docs for aggregates https://review.opendev.org/687348
14:58:58	stephenfin	gibi: Would a follow-up be okay? Can post that now
14:59:05	gibi	stephenfin: sure
14:59:10	gibi	let me change my vote
15:03:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Use deepcopy on value returned by NeutronFixture https://review.opendev.org/688139
15:29:29	stephenfin	gibi: Back at you https://review.opendev.org/#/c/688132/
15:30:26	gibi	stephenfin: thanks. Make sense. I need to get back to that
15:53:44	gibi	dansmith: reviewd the image chache series left some comments here and there
15:55:19	dansmith	gibi: so the return dict thing is leaving us some room for improvement in the future without having to bump the compute rpc version again
15:55:41	dansmith	gibi: are you -1 on that or the logging asserts?
15:55:54	dansmith	oh you said, because the driver bit is unused
15:56:06	gibi	dansmith: you mean that now the rpc returns a dict so later we can add whatever we want to that dict without rpc bump?
15:56:36	dansmith	gibi: no, it's returning that so that I could follow on with some stats logging in conductor.. not expecting to add stuff to the dict later, just expecting to use it
15:56:54	dansmith	gibi: there was some discussion in those patches (IIRC) but definitely some here about what we might do in the future to log some stats,
15:57:15	dansmith	gibi: like "image X seemed to fail on every host" or "34 new downloads, 127 existing", that kind of thing
15:57:18	gibi	dansmith: if it will be used in the future then I'm OK with it
15:57:51	dansmith	gibi: yep, plan is to use it, just trying to keep the base functionality to these patches and then we can hem and haw over how to calculate some stats in a future patch
16:01:16	gibi	dansmith: cool. changed my votes as the rest of my comments can be done in a fup
16:01:43	dansmith	gibi: roger, just saw as I was replying, I'll post a fup for the nits and get a WIP enqueued for the stats so I can point at that if the question comes up again :D
16:02:21	gibi	dansmith: thanks
16:04:44	dansmith	gibi: oh, heh on that libvirt test name... it's totally opposite, I dunno how I did that :P
16:05:53	gibi	:)
16:06:24	dansmith	I'll call it a "testing gibi's attention to detail" easter egg
16:06:31	gibi	it worked :)
16:06:56	dansmith	yeah you passed the test this time
16:07:41	gibi	I'm wondering was there other tests I did not even notice?!
16:09:05	dansmith	gibi: muahah :D
16:09:12	gibi	stephenfin: do you want me to refactor the fake image service ? https://review.opendev.org/#/c/688132/1/nova/tests/functional/test_boot_from_volume.py@203
16:11:34	gibi	stephenfin: https://github.com/openstack/nova/blob/ef6e49d5bc721840b331c87c6391a69309253ade/nova/tests/unit/image/fake.py#L45
16:12:18	stephenfin	gibi: If you have time, but I won't block on that now
16:12:48	gibi	stephenfin: I can do that later. making a todo...

Earlier   Later