| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-08-21 | |||
| 08:50:07 | alex_xu | s/guess/guess so/ | |
| 08:50:47 | stephenfin | alex_xu: The ResourceTracker is so broken :'( | |
| 08:51:02 | stephenfin | Spent all day merely trying to figure out how broken it is, heh | |
| 08:51:14 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747360 | |
| 08:51:17 | alex_xu | hah | |
| 08:52:44 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747361 | |
| 08:55:17 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747362 | |
| 08:56:22 | lyarwood | elod_off: https://review.opendev.org/#/q/topic:bug/1887946+status:open - When you're back this series will need some stable reviews :) | |
| 10:06:37 | openstackgerrit | Merged openstack/nova master: functional: Move single-use function to its caller https://review.opendev.org/745904 | |
| 10:06:43 | openstackgerrit | Merged openstack/nova master: functional: Drop '_api' suffix from placement fixture https://review.opendev.org/745905 | |
| 11:04:00 | lyarwood | https://github.com/openstack/nova/blob/835440e3f9dace4074511036473ebdae22f72c25/nova/context.py#L268-L279 - does anyone know if there's a way to get an admin context with a populated service catalog? | |
| 11:05:23 | lyarwood | ^ at the moment we are using this during self.driver.resume_state_on_host_boot and failing when attempting to hard reboot instances with encrypted volumes as the key_manager service isn't listed in the service catalog | |
| 11:06:07 | lyarwood | appreciate that even if it was the admin context wouldn't be able to fetch user secrets without changing policy but I'd still like to fix this if possible | |
| 11:49:32 | artom | sean-k-mooney, well my VPN just died, but https://github.com/openstack/nova/blob/master/nova/virt/libvirt/migration.py#L96 | |
| 11:50:07 | artom | And https://github.com/openstack/nova/blob/master/nova/virt/libvirt/migration.py#L144 | |
| 12:40:01 | sean-k-mooney | artom that only does it for numa migration | |
| 12:40:11 | sean-k-mooney | artom: it does not print them for all migrations | |
| 12:40:32 | sean-k-mooney | because of https://github.com/openstack/nova/blob/835440e3f9dace4074511036473ebdae22f72c25/nova/virt/libvirt/migration.py#L66 | |
| 12:40:55 | sean-k-mooney | we should move the loging to the start and end of get_updated_guest_xml | |
| 12:41:06 | sean-k-mooney | so it does it for all migrations | |
| 12:43:38 | artom | sean-k-mooney, ah, in that sense, yeah | |
| 12:44:01 | openstack | Launchpad bug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [High,In progress] - Assigned to sean mooney (sean-k-mooney) | |
| 12:44:01 | sean-k-mooney | dansmith: by the way i figured out why https://bugs.launchpad.net/nova/+bug/1888395 happens im working on a repoducer now but the fix is basicaly https://review.opendev.org/#/c/742180/ | |
| 12:46:39 | sean-k-mooney | stephenfin: is the only functional test with live migration and port your new one or do you know if gibi added some for bandwidth based schduling | |
| 12:46:53 | sean-k-mooney | i think he did but just going to look for them now | |
| 12:47:58 | sean-k-mooney | artom: you siad your numa live migation test dont use interface right | |
| 12:48:05 | artom | sean-k-mooney, nope | |
| 12:48:07 | sean-k-mooney | ah yes server = self._create_server(flavor_id=flavor, host=host, | |
| 12:48:10 | sean-k-mooney | networks='none') | |
| 12:48:27 | artom | sean-k-mooney, I'm wondering if it wouldn't be easier to reproduce that migrate_data.vifs bug with a DNM CI patch | |
| 12:48:43 | artom | Is there a way we can deploy Neutron in CI without the multiple port bindings extension? | |
| 12:49:29 | sean-k-mooney | no but i can hack the nova neutron client to always return false | |
| 12:50:05 | sean-k-mooney | that may or may not tirgger it i guess | |
| 12:50:45 | sean-k-mooney | i might however ill try that quickly i would prefer to have a regression test however | |
| 12:51:02 | artom | sean-k-mooney, yeah, long-term something merged in-tree would be better | |
| 12:51:05 | artom | Or a real CI job for it | |
| 12:51:18 | sean-k-mooney | im not standing up a contrial ci :P | |
| 12:52:06 | sean-k-mooney | live migration is really uncloudy we should maybe just remove it instead. it would be a lot less work and headaces then a contrail ci :) | |
| 12:53:24 | artom | lulz | |
| 12:53:43 | sean-k-mooney | it would "fix" all the live migration bugs :) | |
| 12:58:10 | sean-k-mooney | so there is a live migration test with the old microvertion that does not support it for bandwith | |
| 12:58:13 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L5993-L6029 | |
| 12:58:41 | sean-k-mooney | but not the new one so ya i think i would need stephenfin patches to cover the gaps in the neutron fixture | |
| 12:59:38 | sean-k-mooney | oh there are more test for move operatoins later | |
| 13:01:03 | sean-k-mooney | ah ha https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L7299 | |
| 13:01:08 | sean-k-mooney | test_live_migrate_with_qos_port | |
| 13:02:43 | sean-k-mooney | so even without stephenfin patches there is technically enough of a fixture to do the migration with a port | |
| 13:02:49 | sean-k-mooney | just not sriov which is fine | |
| 13:03:23 | sean-k-mooney | i just need to mock the fixture or alter it in the regression test. | |
| 13:04:44 | sean-k-mooney | i need to redefine https://github.com/openstack/nova/blob/835440e3f9dace4074511036473ebdae22f72c25/nova/tests/fixtures.py#L1739-L1752 | |
| 13:05:15 | sean-k-mooney | which i think i can do with a mock | |
| 13:22:51 | stephenfin | you could totally modify the fixture to remove "support" for multiple port bindings | |
| 13:26:14 | sean-k-mooney | im just replaceing the list_extensions function on my instance of the fixture object before i activate it | |
| 13:26:31 | sean-k-mooney | it looks like that shoudl work and is trivial to do | |
| 13:26:56 | sean-k-mooney | self.neutron = nova_fixtures.NeutronFixture(self) | |
| 13:26:59 | sean-k-mooney | self.neutron.list_extensions = self.list_extensions | |
| 13:27:01 | sean-k-mooney | self.useFixture(self.neutron) | |
| 13:27:03 | sean-k-mooney | basicly that | |
| 13:27:38 | sean-k-mooney | im basically modifing https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1862633.py | |
| 13:27:41 | dansmith | sean-k-mooney: I saw the discussion, thanks for chasing that down | |
| 13:27:52 | sean-k-mooney | well not modifing just copying it and creating a new one | |
| 13:28:59 | sean-k-mooney | dansmith: it was not obvious at first but i knew the code path it was taking was for multipel portbindings | |
| 13:29:51 | sean-k-mooney | anyway hopefully ill have a functional regression test for this today and we can do the too patch thing were i assert the behavior is broken and then that the patch fixes it | |
| 13:30:05 | dansmith | sweet | |
| 15:41:41 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | |
| 15:42:41 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | |
| 15:43:07 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | |
| 15:43:25 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | |
| 15:43:51 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | |
| 15:44:08 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | |
| 15:49:19 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Few todo fixes for API new policies https://review.opendev.org/746610 | |
| 16:16:55 | sean-k-mooney | anyone know of the top of your head how i get a valid admin context in a functional test | |
| 16:19:13 | stephenfin | sean-k-mooney: nova.context.get_admin_context() | |
| 16:19:28 | stephenfin | look for 'self.ctxt' in nova/tests/functional | |
| 16:19:59 | sean-k-mooney | thanks | |
| 16:39:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Few todo fixes for API new policies https://review.opendev.org/746610 | |
| 16:40:12 | sean-k-mooney | oh i need to use the libvirt driver in this regression test | |
| 16:40:27 | sean-k-mooney | that explains why its passing wehn it shoudl not be. | |
| 16:40:37 | sean-k-mooney | ok that is still doable | |
| 16:52:49 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert "Handle Neutron errors in _post_live_migration()" https://review.opendev.org/747443 | |
| 16:53:28 | artom | ... which I screwed up. | |
| 16:55:57 | sean-k-mooney | it certenly rivial some of the epics stephenfin has written lately | |
| 16:56:36 | sean-k-mooney | :) http://paste.openstack.org/show/797050/ | |
| 16:56:54 | sean-k-mooney | there we go repoduced it in a functional test | |
| 16:57:10 | stephenfin | not so fast | |
| 16:57:17 | artom | Noice. | |
| 16:57:50 | sean-k-mooney | i feel like im mocking more then i then i need to and could remove some of the fixture im using | |
| 16:57:52 | stephenfin | actually, yeah, that'd make sense for this bug | |
| 16:58:08 | sean-k-mooney | but on the other hand its working so.... | |
| 16:58:16 | stephenfin | was going to say that was failing for me because the NeutronFixture wasn't setting that field like neutron would in real-life | |
| 16:58:32 | stephenfin | but of course it'll only set it if the port-binding extension is present | |
| 16:58:49 | sean-k-mooney | actully that the correct behavior | |
| 16:58:56 | stephenfin | see the changes I made to https://review.opendev.org/#/c/746947/2/nova/conductor/tasks/live_migrate.py | |
| 16:59:10 | sean-k-mooney | currenlty we are setting migrate_data.vif unconditionally | |
| 16:59:15 | sean-k-mooney | im fixint that | |
| 16:59:25 | stephenfin | already fixed ^ | |
| 16:59:25 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert "Handle Neutron errors in _post_live_migration()" https://review.opendev.org/747443 | |
| 16:59:30 | sean-k-mooney | no that is not the fix | |
| 16:59:47 | sean-k-mooney | stephenfin: that code we determined can never be hit | |
| 17:00:03 | sean-k-mooney | the if will always be false | |