| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-25 | |||
| 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 networking concepts https://review.opendev.org/695777 | |
| 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:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: remove admin/manage-users https://review.opendev.org/695779 | |
| 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: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 | openstack | Launchpad bug 1853635 in OpenStack Compute (nova) "Config_drive file throws Request is too large" [Undecided,New] | |
| 14:41:01 | mriedem | https://bugs.launchpad.net/nova/+bug/1853635 | |
| 14:41:04 | mriedem | was trying to triage that | |
| 14:44:32 | sean-k-mooney | we have a downstream bug for that i think that got reported like 2 weeks ago | |
| 14:44:42 | sean-k-mooney | ill see if i can find it | |
| 14:46:34 | sean-k-mooney | artom: you were looking at ^ with dvd or dhill 2 weeks ago right | |
| 14:48:37 | artom | sean-k-mooney, it rings a bell | |
| 14:48:52 | artom | With dhill, though I don't know how that ended | |
| 14:48:57 | artom | The logging made no sense | |
| 14:49:08 | artom | It was a 413, but no other details in the API logs | |
| 14:49:12 | artom | Which were in debug | |
| 14:49:23 | artom | So it made me think maybe a middleware somewhere was doing it | |
| 14:49:42 | artom | And I think efried as kind enough to offer to help since he had played with paste | |
| 14:49:48 | artom | ... and then we all forgot about it | |
| 14:49:56 | sean-k-mooney | i think we found a place where there awas a hardcoded limit | |
| 14:50:32 | efried | still happy to help there, lmk what you need | |
| 14:50:41 | artom | efried, ack, appreciated :) | |
| 14:50:55 | artom | Though at this point we're still speculatively rambling, methinks | |
| 14:53:12 | sean-k-mooney | i cant find it in my history or via BZ so i dont know if i still have the data on it | |
| 14:53:41 | artom | sean-k-mooney, did he ever actually file a bz though? | |
| 14:53:44 | artom | I'll ask | |
| 14:54:02 | sean-k-mooney | maybe not that could be why i cant find it | |
| 14:55:51 | fungi | quick question, mhen mentioned in the image encryption pop-up team meeting an hour ago that during the ptg some nova folks were discussing the spec reproposal with luzi and suggested that they wanted someone to fix ephemeral disk encryption to use luks before adding image encryption support. this is all third-hand so i'm sure i'm missing a bunch of details, but curious if anyone can fill in the gaps | |
| 14:55:53 | fungi | there (the two features don't, at face value, seem to be directly related to one another) | |
| 14:55:56 | fungi | http://eavesdrop.openstack.org/meetings/image_encryption/2019/image_encryption.2019-11-25-13.00.log.html | |
| 14:56:29 | sean-k-mooney | fungi: you want to speak to mdbooth | |
| 14:57:05 | fungi | ahh, so he's working on overhauling ephemeral disk encryption in nova? | |
| 14:57:13 | sean-k-mooney | fungi: mdbooth wanted epmeral disk encryption to be a prequisite for image encryption | |
| 14:57:33 | sean-k-mooney | im not sure if he is working on it or not | |