| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-23 | |||
| 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? | |
| 14:16:17 | mriedem | *under nova/network | |
| 14:16:19 | stephenfin | mriedem: Correct | |
| 14:16:31 | stephenfin | there's a _lot_ of unwinding to be done before I can remove all of that | |
| 14:16:34 | stephenfin | :( | |
| 14:16:55 | mriedem | so basically a lot of small(ish) patches like https://review.opendev.org/#/c/686814/ ? | |
| 14:17:03 | stephenfin | for the most part, yeah | |
| 14:17:09 | stephenfin | currently working on stripping out the firewall feature. That's a beefy one | |
| 14:17:17 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_QEMU_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/695915 | |
| 14:19:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-guide: flesh out flavor extra specs and image properties https://review.opendev.org/695776 | |
| 14:19:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-guide: flesh out networking concepts https://review.opendev.org/695777 | |
| 14:19:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-guide: flesh out todos in user doc https://review.opendev.org/695778 | |
| 14:19:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: remove admin/manage-users https://review.opendev.org/695779 | |
| 14:19:32 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-guide: remove empty sections about inter-service interactions https://review.opendev.org/695774 | |
| 14:38:59 | mriedem | anyone remember where the hard 64kib limit on injected_file_content_bytes is enforced? | |
| 14:39:38 | dansmith | mriedem: isn't it on the implied size of the text field in the db? | |
| 14:39:45 | mriedem | i guess i was thinking user_data | |
| 14:39:50 | mriedem | https://docs.openstack.org/nova/latest/user/metadata.html#metadata-userdata | |
| 14:39:56 | dansmith | oh right right | |
| 14:41:01 | mriedem | https://bugs.launchpad.net/nova/+bug/1853635 | |
| 14:41:01 | openstack | Launchpad bug 1853635 in OpenStack Compute (nova) "Config_drive file throws Request is too large" [Undecided,New] | |
| 14:41:04 | mriedem | was trying to triage that | |