Posted	Nick	Remark
#openstack-nova - 2019-11-22
22:59:04	openstackgerrit	Matt Riedemann proposed openstack/nova master: doc: remove admin/manage-users https://review.opendev.org/695779
23:13:26	efried	dansmith: you said something about hard reboot being admin-only by default, but I can't find how that's the case in the code, where it looks like soft/hard are under the same umbrella. Did I misunderstand?
23:17:24	openstackgerrit	Merged openstack/nova master: test cleanup: Use oslotest's Timeout fixture https://review.opendev.org/685389
23:17:30	openstackgerrit	Merged openstack/nova master: test cleanup: Use oslotest's CaptureOutput fixture https://review.opendev.org/685392
23:37:09	dansmith	efried: that wasn't me
23:38:14	dansmith	at least, not the hard reboot nova action
23:38:21	dansmith	I talked a lot about *host* reboot...
23:38:33	efried	dammit, that's probably where I got confused.
23:38:41	efried	thanks.
#openstack-nova - 2019-11-23
00:08:59	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
00:09:00	openstackgerrit	Dustin Cowles proposed openstack/nova master: WIP: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
00:09:00	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
02:40:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
05:31:46	openstackgerrit	Merged openstack/nova master: test cleanup: Make base TestCase subclass oslotest https://review.opendev.org/685399
06:28:19	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
13:03:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
13:08:13	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813
13:08:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815
13:08:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814
13:08:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817
13:08:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816
13:08:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
14:05:32	openstackgerrit	Merged openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
15:55:28	openstackgerrit	Merged openstack/nova master: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/594527
#openstack-nova - 2019-11-24
00:31:50	openstackgerrit	Merged openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813
06:46:00	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
#openstack-nova - 2019-11-25
00:45:33	openstackgerrit	Eric Xie proposed openstack/nova master: Fix incorrect command examples https://review.opendev.org/695832
05:12:16	openstackgerrit	Merged openstack/nova master: api-guide: flesh out BUILD and ACTIVE server create transitions https://review.opendev.org/695771
05:13:28	openstackgerrit	Merged openstack/nova master: api-guide: fix the file injection considerations drift https://review.opendev.org/695773
06:30:41	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
08:19:04	bauzas	good morning Nova
08:44:35	openstackgerrit	Lee Yarwood proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
08:45:03	lyarwood	stephenfin: ^ hope you don't mind, thought I'd sort out the last few failures as the removal came up in RDO over the weekend.
08:53:29	stephenfin	lyarwood: fine by me, but I think it needs to be rebased again
08:53:35	stephenfin	"This change depends on a change that failed to merge."
08:53:55	lyarwood	stephenfin: ah it's the tempest change?
08:54:22	lyarwood	stephenfin: hmm no that's also up to date
08:54:23	lyarwood	weird
09:07:07	lyarwood	stephenfin: oh that was my bad, I pushed another PS to the tempest change after pushing the nova change, apologies!
09:07:16	lyarwood	recheked
09:07:22	lyarwood	rechecked*
09:38:43	openstackgerrit	Luyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory https://review.opendev.org/695863
09:51:14	tssurya	dansmith: whenever you are awake and have some time, review appreciated on this https://review.opendev.org/#/c/689422/ thanks in advance
09:52:48	openstackgerrit	Luyao Zhong proposed openstack/nova-specs master: spec update: virtual persistent memory https://review.opendev.org/695868
10:15:06	openstackgerrit	Elod Illes proposed openstack/nova stable/stein: Use admin neutron client to see if instance has qos ports https://review.opendev.org/694668
10:19:04	stephenfin	dansmith: Could I get some input on RPC versioning issues here, when you're online? https://review.opendev.org/#/c/686814/11
10:59:35	stephenfin	johnthetubaguy: Could you weigh in on https://review.opendev.org/#/c/686814/11/nova/network/rpcapi.py@a211, assuming you know RPC better than I do?
11:11:53	kashyap	stephenfin: Hi, when you get a moment, want to ACK/NACK these? -- https://review.opendev.org/#/c/694821/3
11:12:28	kashyap	stephenfin: I'd prefer to keep them seperate, although I know they _could_ be squashed. But if that's the "bone of contention" and you insist, can squash :-)
12:32:03	sean-k-mooney	stephenfin: can you take a look at https://review.opendev.org/#/c/674072/ today if you get a chance
12:51:23	gibi	sean-k-mooney, mriedem: hi! do we have live migration functional tests where the server has at least on neutron port? did we added such functional test when the bp neutron-new-port-binding-api has been implemented?
12:51:35	gibi	s/at least on/at least one/
12:55:22	kashyap	gibi: Hi, any chance you can have a look at this low-hanging-fruit (that lets us clean up dead code): https://review.opendev.org/#/q/topic:Bump_min_libvirt_and_QEMU_for_Ussuri+(status:open+OR+status:merged)
12:56:25	sean-k-mooney	gibi: i would have to check but i dont think the neutron fixture implements the multiple port bindigns workflow
12:56:48	sean-k-mooney	gibi: so i dont think we ahve live migration test that succeed.
12:57:02	sean-k-mooney	we have 2 negitive tests i belive
12:57:10	gibi	kashyap: added to my list but no promises for today
12:57:36	kashyap	gibi: No worries, this week is fine; it's also advertized on the mailing list, etc. (FWIW.)
12:58:18	gibi	sean-k-mooney: cool, then I understand why my test with qos + live migration fails with the neutron fixture with vif plug timeout
12:58:56	gibi	sean-k-mooney: thanks. I think I will try to enhance the NeutronFixture to cover live migration with qos
12:59:20	tobias-urdin	do i understand it correctly that nova refreshes the rbd in the libvirt XML by *adding any new* monitors to the config based on the image backend/connection_info from cinder but doesn't purge old entries? And that this patch tries to make sure it completely refreshes the entries by clearing it and adding the existing ones back in? https://review.opendev.org/#/c/579004/
13:01:52	sean-k-mooney	gibi: right i dont think you wll get the network-vif-plugged event
13:03:13	gibi	sean-k-mooney: yeah, now I was able to make an existing test fail with the same problem by simply adding networks="auto" to the boot request
13:04:00	sean-k-mooney	that would normally be set by the ovs agent once that tap is create by libvirt. so you need to mock out the activate binding api endpoint in the fixture to send the vif plugged event to neutron
13:04:22	gibi	sean-k-mooney: ack
13:04:23	gibi	thanks
13:05:01	sean-k-mooney	*form neutorn to nova
13:05:26	sean-k-mooney	it would be useful to have that by they way
13:05:48	sean-k-mooney	so thanks for taking a look at adding it
13:06:30	sean-k-mooney	gibi: on a related note did any of the neutron folks respond on that tread regarding backport the agent changes for finding the RP
13:07:03	gibi	sean-k-mooney: not yet. I have a face 2 face discussion with rubasov this morning. I hope he will reply at some point
13:07:11	gibi	s/have/had/
13:08:01	sean-k-mooney	ok i can poke some of the redhat folks too to comment on it
13:33:39	gibi	sean-k-mooney: I opened a neutron bug https://bugs.launchpad.net/neutron/+bug/1853840
13:33:40	openstack	Launchpad bug 1853840 in neutron "Neutron fails to create bandwidth providers if CONF.host is set" [Undecided,New]
13:34:54	sean-k-mooney	gibi: thanks flagged this internally they will try to discuss it in the neutron team meeting if they have time
13:40:06	openstackgerrit	Matt Riedemann proposed openstack/nova stable/train: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/695900
13:41:02	stephenfin	sean-k-mooney: Sure, I can do that
13:41:34	stephenfin	sean-k-mooney: out of curiosity, am I correct in assuming the firewall implementations found in nova only apply when using nova-network?
13:42:05	sean-k-mooney	yes
13:42:11	stephenfin	cool
13:42:16	stephenfin	Deeeeeelete.
13:42:26	sean-k-mooney	although we used to have some anit spoof rules we enforced
13:42:44	sean-k-mooney	we should be useing the noop driver when using neutron for like 3 years at this point
13:43:08	stephenfin	that's what I was thinking
13:44:05	sean-k-mooney	it was ok to haver it enabled for a time but the allowed adress paris extentaion in neutron allow the vm ot have multipel mac and ip addresses on a singel port
13:44:20	sean-k-mooney	and the nova anti spoof rules dont know about those
13:44:40	sean-k-mooney	so since that was added you shoudl disable teh nova ones and rely on neutrons
13:48:33	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION https://review.opendev.org/695901
13:48:44	gibi	sean-k-mooney: thanks
13:49:28	kashyap	Whoops, I linked to the wrong Change-Id, /me updates...
13:50:46	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION https://review.opendev.org/695901
14:08:06	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
14:10:32	mriedem	dansmith: you might want to weigh in on this https://review.opendev.org/#/c/686814/11/nova/network/rpcapi.py
14:10:52	mriedem	the client rpc api methods being removed are not called from anywhere, but that add_network_to_project one just looks weird since it never used a version
14:11:46	shilpasd	gibi: hi, how i can edit agenda for next nova meeting 'https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting'?
14:12:55	gibi	shilpasd: you can edit the wiki and add your item to the agenda. please add your name as well to know who to ping on the meeting
14:13:24	gibi	shilpasd: you can add to the Open discussion topic
14:15:52	shilpasd	gibi: thanks will check that
14:16:08	mriedem	stephenfin: on https://review.opendev.org/#/c/686816/ you remove the nova-network binary but not the actual module code until nova/network/* - is that coming in later changes?

Earlier   Later