| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-14 | |||
| 15:43:13 | mgoddard | try #openstack-release | |
| 15:43:19 | sean-k-mooney | in this case nova branced on time but ya | |
| 15:43:19 | gibi | ack | |
| 15:45:07 | sean-k-mooney | gibi: are you goign to update the release repo? if not i can do it after my call finishes | |
| 15:45:39 | gibi | sean-k-mooney: I pinge smcginnis about it as it is not just nova that is missing | |
| 15:45:52 | gibi | maybe there is just a script that needs to be run | |
| 15:46:06 | smcginnis | gibi: We have a script to add them, but it checks that the actual landing page exists first. So if a project hadn't merged the bot patches to add it, that would explain why it doesn't show up. (from releases channel) | |
| 15:51:53 | gibi | sean-k-mooney: smcginnis solved the issue with https://review.opendev.org/#/c/758197/ | |
| 15:54:55 | sean-k-mooney | ok cool | |
| 15:55:12 | sean-k-mooney | looks like half of puppet was skipped | |
| 15:55:59 | melwitt | https://zuul.opendev.org/t/openstack/build/f2608672b6c74108ad07d8cdd0ec63ae/log/logs/screen-n-cpu.txt#17674 | |
| 15:55:59 | melwitt | I've been noticing quite a few libvirtError during live migration in the gate, recent example: "Live Migration failure: internal error: missing block job data for disk 'vda': libvirt.libvirtError: internal error: missing block job data for disk 'vda'" in ova-grenade-multinode | |
| 15:56:31 | melwitt | I'm not aware there's anything we can do about it but it's a pain | |
| 16:04:32 | gibi | melwitt: the only recent libvirt driver change I'm aware of is https://review.opendev.org/#/q/Ib9ed7069cef5b73033351f7a78a3fb566753970d but I don't know if it is related | |
| 16:06:43 | openstackgerrit | Mark Goddard proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/710847 | |
| 16:07:19 | melwitt | thanks gibi. I doubt it's related but I will have a look in logstash to see if there's any time correlation. also noting that e-r indexing is behind by 198hrs http://status.openstack.org/elastic-recheck/ I've discussed the e-r indexing stuff with -infra and I'm going to try a change there to help with that | |
| 16:08:53 | gibi | melwitt: other change is the Focal migration that brought in new libvirt version | |
| 16:09:55 | melwitt | ohh right. that might be more likely | |
| 16:38:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use _evacuate_server helper in func test https://review.opendev.org/758210 | |
| 16:38:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove self.server dependency form _evacuate_server https://review.opendev.org/758209 | |
| 16:38:29 | gibi | stephenfin: ^^ these might be interesting for you | |
| 18:02:27 | openstackgerrit | Merged openstack/nova master: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/746873 | |
| 20:17:05 | openstackgerrit | Lee Yarwood proposed openstack/nova master: CinderFixture: Return a driver_type of `fake` https://review.opendev.org/758238 | |
| 20:17:07 | openstack | bug 1899835 in OpenStack Compute (nova) "disconnect_volume called during rollback from pre_live_migration failure on destination" [Undecided,New] https://launchpad.net/bugs/1899835 - Assigned to Lee Yarwood (lyarwood) | |
| 20:17:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1899835 https://review.opendev.org/758239 | |
| 20:18:19 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758240 | |
| 20:24:38 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758241 | |
| 20:25:06 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758242 | |
| 20:25:23 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758243 | |
| 20:25:39 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758244 | |
| 20:28:39 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758245 | |
| 20:36:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: CinderFixture: Return a driver_type of `fake` within connection_info https://review.opendev.org/758238 | |
| 20:36:08 | openstack | bug 1899835 in OpenStack Compute (nova) "disconnect_volume called during early rollback from pre_live_migration failure on destination" [Undecided,New] https://launchpad.net/bugs/1899835 - Assigned to Lee Yarwood (lyarwood) | |
| 20:36:08 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1899835 https://review.opendev.org/758239 | |
| 21:00:32 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: make mdev types name attribute be optional https://review.opendev.org/754401 | |
| 23:15:23 | openstack | bug 1894966 in OpenStack Compute (nova) train "Create servergroup failed with unexpected error" [Undecided,In progress] https://launchpad.net/bugs/1894966 - Assigned to Stephen Finucane (stephenfinucane) | |
| 23:15:23 | openstackgerrit | Merged openstack/nova stable/train: tests: Add regression test for bug 1894966 https://review.opendev.org/752706 | |
| #openstack-nova - 2020-10-15 | |||
| 00:05:42 | openstackgerrit | Merged openstack/nova stable/train: api: Set min, maxItems for server_group.policies field https://review.opendev.org/752707 | |
| 00:06:45 | openstackgerrit | Merged openstack/nova stable/rocky: post live migration: don't call Neutron needlessly https://review.opendev.org/750673 | |
| 00:06:55 | openstackgerrit | Merged openstack/nova stable/queens: [placement] Add status and links fields to version document at / https://review.opendev.org/751239 | |
| 01:29:08 | openstackgerrit | Merged openstack/nova stable/stein: Set different VirtualDevice.key https://review.opendev.org/749415 | |
| 04:15:34 | openstackgerrit | Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175 | |
| 07:04:08 | openstackgerrit | Mamduh proposed openstack/os-vif master: [WIP] get_ifname_by_pci_address compare phys_port_id https://review.opendev.org/758352 | |
| 07:41:22 | openstackgerrit | Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175 | |
| 09:24:30 | bauzas | good late morning Nova | |
| 09:37:02 | gibi | o/ | |
| 09:55:01 | lyarwood | Morning morning | |
| 10:11:09 | stephenfin | gibi: Heads up that I've added specless BP approval for https://blueprints.launchpad.net/nova/+spec/compact-havana-db-migrations to the agenda for today's meeting. I won't be around though so I can move to next week if it proves to be any way contentious | |
| 10:11:53 | stephenfin | "...and make it easier to move to Alembic in the future." /o\ | |
| 10:12:17 | stephenfin | that went super well, evidently | |
| 10:15:42 | tosky | stephenfin: is that related to the usage of sqlalchemy-migrate? | |
| 10:15:54 | stephenfin | tosky: Dumping the use of it, yes | |
| 10:16:52 | tosky | so sqlalchemy-migrate is still a dependency, but it's kind of abandoned (x namespace), I've tried to fix the zuul jobs but of course most of the people marked as maintainers are not around | |
| 10:24:32 | stephenfin | tosky: It's dead. Alembic is apparently a far better designed tool that we should have switched to ages ago but, you know, ENOTIME | |
| 10:25:17 | stephenfin | but our number of DB migrations has plummeted in recent releases so no better time, I'm thinking. We shall see | |
| 10:31:05 | openstackgerrit | Merged openstack/nova stable/victoria: Follow up for cherry-pick check for merge patch https://review.opendev.org/757613 | |
| 10:47:06 | gibi | stephenfin: ack. I don't expect too much pushback on that | |
| 10:50:28 | lyarwood | stephenfin: ^ another pre_live_migration functional test if you have time today | |
| 10:50:28 | openstack | bug 1899835 in OpenStack Compute (nova) "n-cpu attempts to disconnect volumes after early pre_live_migration failures on the destination during a live migration" [Undecided,New] https://launchpad.net/bugs/1899835 - Assigned to Lee Yarwood (lyarwood) | |
| 10:50:28 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1899835 https://review.opendev.org/758239 | |
| 11:09:15 | tosky | stephenfin: I know it's dead, but as long as projects still depend on it (because ENOTIME), it's not really dead :) | |
| 11:17:29 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Compact pre-Juno database migrations https://review.opendev.org/758394 | |
| 11:17:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Compact pre-Liberty database migrations https://review.opendev.org/758396 | |
| 11:17:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: TODO: Compact pre-Kilo database migration https://review.opendev.org/758395 | |
| 11:17:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Compact pre-Newton database migrations https://review.opendev.org/758398 | |
| 11:17:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: TODO: Compact pre-Mitaka database migrations https://review.opendev.org/758397 | |
| 11:20:31 | stephenfin | gibi: Think you could take a spin through https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:remove-xenapi maybe early next week too? I assume we can merge now that V is out the door | |
| 11:22:25 | stephenfin | lyarwood: +2 to that and its predecessor | |
| 11:22:28 | stephenfin | <3 functional tests | |
| 11:22:55 | stephenfin | lyarwood: Not nova, but could you buzz through https://review.opendev.org/#/c/755887/ and https://review.opendev.org/#/c/755888/ | |
| 11:23:01 | gibi | stephenfin: thanks for the reminder, will do | |
| 11:24:15 | stephenfin | Does anyone know where that docs that mriedem did on novaclient-OSC gaps is gone to? Think I'm going to spend some time this cycle closing that out too | |
| 11:25:09 | gibi | it is an etherpad... looking.. | |
| 11:25:16 | gibi | https://etherpad.opendev.org/p/compute-api-microversion-gap-in-osc | |
| 11:28:30 | stephenfin | gibi++ thanks! | |
| 11:31:21 | gibi | stephenfin: I also thought about the relationship of o.vo and mypy. o.vo handles the fields by generating properties for it runtime so that static checker is clueless. But I think w might be able to generate a .pyi interface file for o.vos that has the fields with static types to help mypy | |
| 11:47:59 | sean-k-mooney | that makes sense althoug we can proably also extend mypy to pars ovo but the pyi file is proably simpler | |
| 11:49:16 | gibi | sean-k-mooney: yeah a plugin to mypy is also a possibility but as far as I see the plugin interface of mypy is unstable | |
| 11:50:21 | sean-k-mooney | i have not really looked at it | |
| 11:51:13 | gibi | https://mypy.readthedocs.io/en/stable/extending_mypy.html#extending-mypy-using-plugins "The plugin system is experimental and prone to change. I" | |
| 12:04:30 | sean-k-mooney | we could likely write a small script that would loop over the nova/objects dir and produce the pyi files and run that in the tox env | |
| 12:05:04 | sean-k-mooney | we could optionally commit them or just have them generated on the fly | |
| 12:14:32 | gibi | sean-k-mooney: yeah, that is the direction of my thinknig | |
| 12:16:24 | lyarwood | stephenfin: FWIW I wanted to sort the evacuate command out in osc | |
| 12:16:58 | lyarwood | TIL sean-k-mooney had a change for this | |
| 12:19:24 | lyarwood | added to the PTG pad | |
| 12:28:20 | sean-k-mooney | lyarwood: yep it more or less done but we want to rename it back to evacuate | |
| 12:28:34 | sean-k-mooney | its been on my todo list for a whiel if you feel like taking it over | |
| 12:28:43 | sean-k-mooney | if not i can try and do it before the ptg | |
| 12:29:40 | lyarwood | sean-k-mooney: so was that outcome, that we wanted to stay with evacuate for now? | |
| 12:29:48 | lyarwood | sean-k-mooney: if so I can rework it now | |
| 12:30:19 | sean-k-mooney | lyarwood: thats what dansmith wanted | |
| 12:30:36 | sean-k-mooney | also since i want to add recreate as a new action it proably better not to rename | |
| 12:30:43 | brinzhang | gibi, sean-k-mooney: I encounter a question, I want to backport https://review.opendev.org/#/q/status:merged+project:openstack/nova+branch:master+topic:bp/numa-aware-live-migration to Rocky release | |
| 12:30:53 | brinzhang | gibi, sean-k-moooney: Because there is a big gap between its minor versions, I only merged the changes below the api, including the RPC Version changes, but changed rpc_version=5.3 to rpc_version=5.1 to adapt to the version number of Rocky release. | |
| 12:31:00 | sean-k-mooney | lyarwood: the main concen was people thinking it was a new thing calling it resurect | |
| 12:31:15 | sean-k-mooney | brinzhang: that a feature not a bug | |
| 12:31:21 | sean-k-mooney | brinzhang: so you cant backprot it | |
| 12:31:40 | lyarwood | sean-k-mooney: ack yup np, I'll rework it now in that case | |
| 12:31:42 | brinzhang | gibi: but, when I request live migration the rpcverion=5.1 can not valid | |