Posted	Nick	Remark
#openstack-nova - 2021-01-03
01:15:08	openstackgerrit	zhangboye proposed openstack/os-traits master: Add Train Ussuri Victoria release info https://review.opendev.org/c/openstack/os-traits/+/768889
01:20:57	openstackgerrit	zhangboye proposed openstack/os-resource-classes master: Add Train Ussuri Victoria release info https://review.opendev.org/c/openstack/os-resource-classes/+/768891
01:31:21	openstackgerrit	zhangboye proposed openstack/osc-placement master: Add Victoria release info https://review.opendev.org/c/openstack/osc-placement/+/768893
02:53:28	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Replace tenant with project in nova CLI https://review.opendev.org/c/openstack/python-novaclient/+/768921
07:15:44	openstackgerrit	Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852
07:18:05	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Replace tenant with project in nova CLI https://review.opendev.org/c/openstack/python-novaclient/+/768921
08:22:30	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in flavor-access commands https://review.opendev.org/c/openstack/python-novaclient/+/769019
09:09:48	brinzhang0	gibi, sean-k-mooney, stephenfin, gmann: happy new year ^^
09:11:04	brinzhang0	gibi, sean-k-mooney, stephenfin, gmann:I would like to use microversion 2.88 for remove tenant_id, I have submited all nova apis change for this feature, pls see https://review.opendev.org/q/topic:%22bp%252Fremove-tenant-id%22+(status:open%20OR%20status:merged)
09:11:56	brinzhang0	of course, the novaclient nad sdk need to be change, and it will be done in next week.
09:12:24	brinzhang0	gibi, sean-k-mooney, stephenfin, gmann: please consider my request, thanks.
09:16:18	brinzhang0	gibi_pto ^^
09:41:57	openstackgerrit	Reason li proposed openstack/osc-placement master: remove unicode from code https://review.opendev.org/c/openstack/osc-placement/+/769059
09:44:20	openstackgerrit	Reason li proposed openstack/osc-placement master: remove unicode from code https://review.opendev.org/c/openstack/osc-placement/+/769062
12:51:17	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Fix a functional test for 'nova agent-list' https://review.opendev.org/c/openstack/python-novaclient/+/769066
13:13:04	openstackgerrit	Dawood Khorsandi proposed openstack/nova master: Reason message added to NoValidHost exception https://review.opendev.org/c/openstack/nova/+/769067
13:18:29	openstackgerrit	Dawood Khorsandi proposed openstack/nova master: Reason message added to NoValidHost exception https://review.opendev.org/c/openstack/nova/+/769067
14:18:08	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Deprecate agent commands and APIs https://review.opendev.org/c/openstack/python-novaclient/+/769068
16:10:20	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Deprecate agent commands and APIs https://review.opendev.org/c/openstack/python-novaclient/+/769068
16:35:44	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Deprecate agent commands and APIs https://review.opendev.org/c/openstack/python-novaclient/+/769068
#openstack-nova - 2021-01-04
01:48:08	openstackgerrit	Arthur Dayne proposed openstack/nova master: Libvirt: fix the bug volume attaching failure when the VM domain is in transient status. https://review.opendev.org/c/openstack/nova/+/769074
04:50:55	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Fix undesirable raw Python error https://review.opendev.org/c/openstack/python-novaclient/+/769082
07:58:21	gibi	good morning
08:04:49	bauzas	good morning Nova and happy new year !
08:43:57	openstackgerrit	Lee Yarwood proposed openstack/nova master: zuul: Deploy novnc from source in nova-next https://review.opendev.org/c/openstack/nova/+/768334
08:48:13	gibi	bauzas: happy new year to you too
08:52:37	lyarwood	morning all and a happy new year
08:52:56	gibi	lyarwood: happy new year to you too
09:04:33	openstackgerrit	Lee Yarwood proposed openstack/nova master: zuul: Deploy novnc from source in nova-next https://review.opendev.org/c/openstack/nova/+/768334
09:07:56	stephenfin	happy new year
09:09:21	gibi	stephenfin: o/
09:10:13	kashyap	New year's greetings to you all too
09:11:17	bauzas	maybe I should tell "Better New Year" instead of "Happy new year" ?
09:12:29	gibi	kashyap: \o
09:12:44	gibi	bauzas: or simply just different new year :D
09:12:53	lyarwood	bauzas: that's a pretty low bar after 2020 ;)
09:13:32	kashyap	bauzas: I ended my new year w/ this video: https://www.youtube.com/watch?v=fn3KWM1kuAw
09:13:50	kashyap	Err, the old year, I mean.
09:13:54	bauzas	kashyap: saw it ;) <3
09:15:11	lyarwood	I for one welcome our new dad dancing overlords
09:20:18	stephenfin	gibi: What do we want to do about this? https://review.opendev.org/c/openstack/python-novaclient/+/769066/
09:28:26	openstackgerrit	Merged openstack/osc-placement master: remove unicode from code https://review.opendev.org/c/openstack/osc-placement/+/769059
09:30:02	openstackgerrit	Merged openstack/osc-placement master: remove unicode from code https://review.opendev.org/c/openstack/osc-placement/+/769062
09:32:09	gibi	stephenfin: looking
09:32:54	stephenfin	Ta. Historically, mriedem had removed the novaclient features once the nova API was removed, however, since novaclient is used as a library by things like OSC, I'm not sure if we want to keep doing that
09:34:47	gibi	stephenfin: I think it is OK to keep the nova client as is even for remove APIs as this code is not a big burden to maintain and as you said a new novaclient might be used towards an old nova API endpoint
09:36:55	gibi	stephenfin: I don't even sure we want a deprecation message from the client itself as the server response is pretty obvious
09:37:27	stephenfin	Okay, cool. Let's keep it so
09:39:03	brinzhang0	gibi, stephenfin: happy new year
09:39:50	brinzhang0	gibi, stephenfin: did you have seen my news above? pinged you yesterday
09:40:53	gibi	brinzhang0: saw it but haven't looked the patches yet
09:41:29	stephenfin	same
09:41:38	brinzhang0	gibi: ack, yeah, I just want to use v2.88 as the remove tenant_id microversion
09:42:28	gibi	brinzhang0: I have to see what are the other patches fighting for the same microversion and how ready these patches are
09:42:33	stephenfin	I was pretty sure we'd said we'd use 2.88 for the os-hypervisor work, since that's the smallest of the three items. I could have picked that up wrong though
09:43:14	stephenfin	It's okay though. I think we're all aware there are competing microversion changes and can be sure to finish one before starting on the next
09:43:40	bauzas	gibi: stephenfin: MHO about client changes is that we can remove old features if the API is no longer supporting it
09:44:13	bauzas	but then we would have to say that the minor version supported by the client would change
09:44:16	gibi	bauzas: sure we can, it is more the question if we want to remove.
09:44:24	brinzhang0	stephenfin: yeah, if possible, I can help you to change, because the remove tenant_id patach depends on 2.88, and there are some place use v2.88
09:44:50	stephenfin	bauzas: what do we do about things like OSC though, that depend on the novaclient library implementation
09:45:44	bauzas	stephenfin: the question is, should we still support v2.1 for the client ?
09:45:57	bauzas	if yes, we *can't* remove
09:46:17	stephenfin	brinzhang0: That's true for all changes. Whatever gets punted to 2.89 just has to be rebased and references to 2.88 updated to 2.89
09:47:03	stephenfin	bauzas: OSC does, yes. It's supposed to support all OpenStack clouds, regardless of version (though I suspect we fall well short of that marker, and I also doubt many people are using new OSC with an e.g. Essex cloud)
09:48:36	brinzhang0	stephenfin: yeah, after you can review these patch, and pls consider waht I want to say, thanks
09:49:22	brinzhang0	and I am sure the remove tenant_id feature can be done in this release
09:50:05	brinzhang0	but as bauzas and you talked above, how to deal with the OSC changes?
09:51:04	brinzhang0	ah, maybe it's(OSC) ok for the latest changes :D
09:53:01	bauzas	stephenfin: brinzhang0: see how version discovery works https://docs.openstack.org/api-guide/compute/microversions.html#version-discovery
09:53:34	bauzas	given our API provides minimum as 2.0 (or 2.1, can't remember exactly which one given they are the same)
09:53:37	bauzas	,
09:54:00	bauzas	then the client would have to support the minimum too
09:54:07	bauzas	here, 2.1
09:54:48	bauzas	actually, wrong link, https://docs.openstack.org/api-guide/compute/microversions.html#client-interaction is better
09:59:15	gibi	so the conclusion could be that we can remove the CLI part but we cannot remove the lib part due to OSC
10:01:17	brinzhang0	I think yes
10:04:10	gibi	stephenfin, bauzas: summarized my view in https://review.opendev.org/c/openstack/python-novaclient/+/769068
10:10:07	bauzas	gibi: if we cut the support on the CLI, then we need to bump a major version when we release the client
10:10:23	bauzas	but that's ok IMHO
10:10:33	gibi	bauzas: correct, that is what the deprecation messages now describe in the patch
10:11:01	bauzas	ack
10:14:37	gibi	when (or even if ever) we cut a major version is open
10:14:52	bauzas	right
10:32:48	lyarwood	gibi / stephenfin ; https://review.opendev.org/c/openstack/nova/+/767590 - time to +W this?
10:33:32	openstackgerrit	Merged openstack/nova master: tests: Merge 'test_hypervisor_status' into 'test_hypervisors' https://review.opendev.org/c/openstack/nova/+/764039
10:33:35	stephenfin	I think so
10:35:11	lyarwood	done
10:56:07	openstackgerrit	Merged openstack/nova master: Run the db migration tests in the same test worker https://review.opendev.org/c/openstack/nova/+/767590
11:01:51	gibi	lyarwood,stephenfin: thanks
11:01:59	gibi	I will keep monitoring those tests
11:02:11	gibi	as we did not solved the issue we just decreased the likelyhood
11:02:37	gibi	stephen's db migration compression patch series will be an even better fix when it lands
11:11:48	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Add nova-live-migration-ceph job https://review.opendev.org/c/openstack/nova/+/768466
11:12:46	lyarwood	^ getting close with this btw, there are some random volume backup test failures at the moment but the live migration and evacuation parts are working
11:13:39	sean-k-mooney	oh nice
11:18:01	openstackgerrit	Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976
11:18:03	openstackgerrit	Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472
11:18:03	openstack	bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood)
11:19:14	gibi	stephenfin: I'm looking at the hypervisor series and wondering if the impl now in sync with the uptime change proposed in the yet unmerged spec additon here https://review.opendev.org/c/openstack/nova-specs/+/765797/1 ?

Earlier   Later