Posted	Nick	Remark
#openstack-nova - 2019-12-17
21:03:32	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: DNM: test that the ceph job is fixed on pike https://review.opendev.org/699500
21:36:14	mriedem	things in EM branches won't be released, and fixes in OSC would need to be released to be used by someone
21:36:21	mriedem	so i don't see much point in backporting osc stuff past rocky
21:36:27	mriedem	upstream anyway
21:38:39	KeithMnemonic_	ok thanks
22:00:04	KeithMnemonic_	mriedem can i run that issue with the "prev" button by you again. i know you said it was sort_dir/sort_key and in fact we found something interesting
22:00:28	KeithMnemonic_	using the nova cli and sort_key uuid and marker show correct results when using asc or desc
22:01:01	KeithMnemonic_	using the default sort created_at does not and that would explain why horizons "prev" button is not working
22:36:24	KeithMnemonic	mriedem: so would you expect this command to work? nova list --sort created_at:desc and nova list --sort created_at:asc
22:36:48	KeithMnemonic	they work with any field that is not a time field
22:37:42	KeithMnemonic	but with a time field it does not sort correctly
22:42:53	KeithMnemonic	if anyone has any ideas http://paste.openstack.org/show/787694/
22:43:11	KeithMnemonic	uuid works fine, created_at does not for the sort_key
22:43:38	KeithMnemonic	and this was run as admin in devstack
22:56:13	openstackgerrit	Curt Moore proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
23:03:11	openstackgerrit	Curt Moore proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
23:42:07	openstackgerrit	Luigi Toscano proposed openstack/nova master: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699516
#openstack-nova - 2019-12-18
01:20:21	openstackgerrit	Merged openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047
03:45:34	openstack	bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
03:45:34	openstackgerrit	melanie witt proposed openstack/nova master: Add functional regression test for bug 1817833 https://review.opendev.org/699538
05:23:37	openstackgerrit	Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
05:23:38	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Define Cyborg ARQ binding notification event. https://review.opendev.org/692707
05:23:38	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
05:23:39	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581
05:23:39	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
05:23:40	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
05:23:40	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245
05:23:41	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553
05:23:41	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
05:23:42	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
07:02:11	openstackgerrit	Shilpa Devharakar proposed openstack/nova master: Handle new is_volume_backend join column query https://review.opendev.org/694462
07:02:12	openstackgerrit	Shilpa Devharakar proposed openstack/nova master: Ignore root_gb if instance is booted from volume https://review.opendev.org/612626
07:02:12	openstackgerrit	Shilpa Devharakar proposed openstack/nova master: Instance object changes for the new 'is_volume_backed' expected_attr https://review.opendev.org/694463
07:30:17	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update keypairs in saving an instance object https://review.opendev.org/683043
07:30:50	openstackgerrit	ya.wang proposed openstack/nova-specs master: Add "live migration without performance impact" spec. https://review.opendev.org/693655
09:25:01	openstackgerrit	Merged openstack/nova stable/train: Cache security group driver https://review.opendev.org/697475
10:40:56	frickler	KeithMnemonic: the sorting works fine for me, both with master and rocky.
11:35:22	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669
11:56:20	stephenfin	lyarwood: Could you hit https://review.opendev.org/#/c/699522/ please?
11:59:15	lyarwood	stephenfin: I'm only nova-stable not stable-maint so I can't +2 that sorry
11:59:26	stephenfin	ah, np
12:06:02	stephenfin	melwitt: Could you review https://review.opendev.org/#/c/687909/ today when you're around. Would love to close that BP out before the end of the year
12:06:08	stephenfin	will ask mriedem too when he's about
13:00:13	openstackgerrit	Nathan Kinder proposed openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502
13:37:23	openstackgerrit	Merged openstack/nova stable/pike: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/666865
13:37:28	openstackgerrit	Merged openstack/nova stable/ocata: Exclude build request marker from server listing https://review.opendev.org/677201
14:19:39	KeithMnemonic1	frickler thanks, i found the problem i was seeing at least, when you bulk create 5 instances for example unless you are really lucky the created time is the same
14:30:38	KeithMnemonic1	frickler, it is broken when you use a marker
14:46:30	KeithMnemonic1	frickler ok i figured it out now, sorry for the noise, it is back to a horizon issue
14:56:28	frickler	KeithMnemonic1: ok, great, thanks for the update
15:03:20	openstack	bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] https://launchpad.net/bugs/1829479 - Assigned to Matt Riedemann (mriedem)
15:03:20	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/699698
15:03:21	openstack	bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
15:35:50	openstack	bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Medium,In progress] https://launchpad.net/bugs/1852610 - Assigned to Matt Riedemann (mriedem)
15:35:50	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1852610 https://review.opendev.org/699705
15:40:27	mriedem	stephenfin: same vendor approval huh https://review.opendev.org/#/c/679502/
15:41:06	stephenfin	I didn't even see they worked for Red Hat /o\
15:41:54	stephenfin	I can pull it out of the gate, but do we have anyone other than Mel and I that can review console stuff now?
15:43:11	openstack	bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Medium,In progress] https://launchpad.net/bugs/1852610 - Assigned to Matt Riedemann (mriedem)
15:43:11	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate revert resize test for bug 1852610 https://review.opendev.org/699708
15:43:45	mriedem	i reviewed that once before and spotted some stuff - don't have to be a console expert to be a core and review things
15:44:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502
15:44:22	mriedem	pull it out, +2 it and i'll look at it today
15:44:35	stephenfin	fair. done and done
15:44:41	mriedem	thanks
15:45:08	stephenfin	while you're on console stuff, could you add [1] to the queue too? https://review.opendev.org/#/c/687909/
15:45:30	stephenfin	I pinged Mel about it earlier too
15:47:37	mriedem	have you pushed stable branch changes to enable xvpvnc? https://review.opendev.org/#/c/695853/
15:48:31	mriedem	i see https://review.opendev.org/#/c/698591/1 but it's wrong
15:55:50	lyarwood	mriedem: I'll rework that now
15:56:51	mriedem	the tempest patch has issues as well
15:57:41	mriedem	btw, i thought you guys were all on pto?
15:58:06	lyarwood	ha I wish
15:58:10	lyarwood	zero PTO left for the year
15:58:18	lyarwood	and I think stephenfin is the same
15:58:30	mriedem	huh, ok
15:59:04	stephenfin	Sorry, had to sanity check how that Tempest check was actually working
15:59:44	lyarwood	stephenfin: I'll respin after the standup
15:59:57	stephenfin	Nah, I mean the original test
16:00:19	stephenfin	I still don't know how it does. XVP console is Xen-only. That shouldn't work unless Tempest was run with a XenServer backend
16:00:50	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Block deleting compute services with in-progress migrations https://review.opendev.org/699718
16:03:59	mriedem	stephenfin: it's not actually trying to get a console i don't think, just a url
16:04:44	mriedem	so if there isn't any special token handling for xvpvnc it doesn't seem like it is treated any different from vnc
16:04:53	mriedem	the url would be different, but the test isn't checking to see if the url can be used
16:15:17	lyarwood	so is it even worth keeping the tempest test in tree for older releases in that case?
16:15:51	lyarwood	stephenfin: ^ ?
16:26:20	stephenfin	Sorry, meeting
16:26:34	stephenfin	lyarwood, mriedem: So I think that test is junk because nova doesn't seem to check whether it can actually support the requested console type
16:27:12	stephenfin	nova just requests _a_ console type from the virt driver https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6681
16:27:57	stephenfin	so if you request the xvp type on the libvirt driver, it'll return a connection_info object with a URL that would never work
16:28:31	mriedem	i think i said all of that above
16:28:32	lyarwood	yup, do you want to push something to nuke it and I'll drop my change?
16:28:49	stephenfin	lyarwood: To answer your question, given that we're deprecating the XVP console type and using it previously would require Xen, which no one's tested in who knows how long, I'd nuke it from Tempest
16:28:54	stephenfin	no need for the config option
16:29:16	stephenfin	mriedem: and I'm agreeing with you :)
16:29:18	lyarwood	stephenfin: ack, I'll do that now then
16:29:19	mriedem	i'd support nuking that from tempest
16:29:32	mriedem	if someone using xen comes along and wants that later, they could write a tempest plugin
16:29:46	stephenfin	If they want it on < Ussuri, that is
16:29:52	stephenfin	since it'll be gone from then on, hopefully

Earlier   Later