| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-04 | |||
| 20:03:53 | mriedem | note that devstack sets it to ps2mouse explicitly https://github.com/openstack/devstack/blob/93d22d82988ee781feddf84535d65adf44472379/lib/nova_plugins/hypervisor-libvirt#L44 | |
| 20:04:05 | mriedem | so you could just check one of the gate job logs for a guest xml to see if the pointer device is not set | |
| 20:04:46 | melwitt | oh nice | |
| 20:07:17 | mriedem | code looks basically the same since newton https://github.com/openstack/nova/blob/newton-eol/nova/virt/libvirt/driver.py#L4675 | |
| 20:07:50 | melwitt | here's a guest xml https://zuul.opendev.org/t/openstack/build/fcc8380b76924d0ca8acf158f7571ad8/log/controller/logs/screen-n-cpu.txt#1888 | |
| 20:09:46 | ozzzo | I see the USB lines missing from the XML now, but I still see the PS2 mouse and keyboard lines. How can I get rid of those? | |
| 20:11:34 | mriedem | melwitt: i don't see any input devices in there | |
| 20:11:38 | mriedem | so seems that ps2mouse is doing the deed | |
| 20:11:50 | melwitt | yeah I don't either | |
| 20:11:55 | mriedem | ozzzo: you set pointer_model=ps2mouse on your computes? | |
| 20:12:07 | ozzzo | only on the HV | |
| 20:12:20 | mriedem | HV = nova-compute nodes? | |
| 20:12:24 | ozzzo | yes | |
| 20:12:34 | mriedem | restarted the service and booted a new vm on that node? | |
| 20:12:36 | ozzzo | just the one I'm testing | |
| 20:12:48 | ozzzo | no, I stopped and started the VM, but that rewrites the XML | |
| 20:12:57 | ozzzo | do I need to delete it and build a new one? | |
| 20:12:58 | mriedem | you have to restart the compute service to refresh the config | |
| 20:13:09 | mriedem | then you could reboot the vm | |
| 20:13:10 | melwitt | ozzzo: that won't refresh the nova config though | |
| 20:13:29 | ozzzo | I stop/started nova-compute on the compute node | |
| 20:13:30 | melwitt | that reminds me, did SIGHUP ever get fixed? | |
| 20:13:34 | ozzzo | then stop/started the VM | |
| 20:13:38 | melwitt | oh ok | |
| 20:13:45 | mriedem | paste your guest xml i guess | |
| 20:13:55 | mriedem | in a paste/gist | |
| 20:14:16 | melwitt | well, I'm wondering if this is what the config help is describing "'Uses default behavior provided by drivers (mouse on PS2 for libvirt x86)" | |
| 20:14:32 | mriedem | but we're running libvirt x86 in the gate | |
| 20:14:40 | melwitt | ... true | |
| 20:14:48 | melwitt | hm what the heck | |
| 20:15:08 | mriedem | unless y'all moved over to ARM in the last few months | |
| 20:15:27 | ozzzo | http://paste.openstack.org/show/790313/ | |
| 20:16:45 | mriedem | huh yeah - which version of libvirt? | |
| 20:22:26 | melwitt | looks like usb tablet is the only input device we're capable of adding https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L1802 | |
| 20:23:33 | melwitt | I mean, I guess we already knew that but there's where all the configs are | |
| 20:25:49 | melwitt | here's danpb stating "We don't specify anything actually. QEMU always gives you a ps2 keyboard + mouse no matter what & AFAIK, there's no way to disable that. Any other mouse should override the PS2 mouse, provided the guest OS actually initializes the non-ps2 mouse." https://bugzilla.redhat.com/show_bug.cgi?id=953064#c4 | |
| 20:25:50 | openstack | bugzilla.redhat.com bug 953064 in libvirt "allow disabling ps2 mouse, it overrides tablet on macosx guests" [Medium,Closed: deferred] - Assigned to libvirt-maint | |
| 20:27:33 | mriedem | ok so ozzzo's bug is invalid / won't fix, just need to set pointer_model=ps2mouse | |
| 20:27:45 | mriedem | and remove that None option as a choice since it's invalid - i guess that's the bug | |
| 20:28:31 | melwitt | yeah, it's weird. None is supposed to mean "use the libvirt defaults" which it is doing | |
| 20:29:24 | melwitt | and setting ps2mouse does the same thing, uses the defaults. almost seems like the choices should be 'usbtablet' and 'default' instead or something | |
| 20:30:11 | melwitt | I mean, if None worked, it would be doing what it says, using the defaults | |
| 20:30:13 | ozzzo | so it's not possible to not load the PS2 driver, and that has nothing to do with openstack; it is a qemu limitation? | |
| 20:30:55 | melwitt | looks like it yeah, according to that libvirt bz | |
| 20:34:00 | ozzzo | ok I'll talk to the guys in #centos again I guess; they are telling me to not load the USB and PS2 drivers because it appears that they choke on large RAM | |
| 20:35:15 | melwitt | yeah ... we need to chat with a libvirt/qemu person I think too | |
| 20:35:25 | ozzzo | thanks for your help and advice! | |
| 20:36:25 | melwitt | https://libvirt.org/contact.html#irc | |
| 20:36:55 | melwitt | they have a mailing list too | |
| 20:37:09 | ozzzo | ok I'll try them, ty | |
| 21:00:53 | openstackgerrit | Merged openstack/nova master: Pass the actual target in os-availability-zone policy https://review.opendev.org/706691 | |
| 21:01:03 | openstackgerrit | Merged openstack/nova master: Fix os-os-deferred-delete policy to be admin_or_owner https://review.opendev.org/707457 | |
| 21:01:11 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing deferred_delete policies https://review.opendev.org/707455 | |
| 21:05:07 | melwitt | bus https://libvirt.org/formatdomain.html#elementsInput | |
| 21:05:07 | melwitt | ozzzo: this might be totally useless but ... I googled around out of curiosity and found this reddit post https://www.reddit.com/r/VFIO/comments/8qo468/stuttering_caused_by_evdev_with_ps2_input_devices/ where they changed their keyboard and mouse bus to 'virtio' and installed the necessary drivers. they're calling this "passing through the mouse and keyboard". and I saw that as of libvirt 1.3.0, 'virtio' is an option for input device | |
| 21:09:24 | melwitt | I would probably try something like that in a test/lab environment to see if it helps (if you don't find any other info when you inquire) by making manual updates to the xml. but I really don't know what I'm talking about so you should probably ignore me. this problem is interesting | |
| 21:11:11 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing attach_interfaces policies https://review.opendev.org/705126 | |
| 21:18:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-attach-interfaces https://review.opendev.org/705799 | |
| 21:18:42 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-atttach-inerfaces policies https://review.opendev.org/706672 | |
| 21:50:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546 | |
| 21:50:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into own mixin https://review.opendev.org/705545 | |
| 21:50:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 21:50:54 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 21:50:54 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 22:00:58 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-deferred_delete https://review.opendev.org/707476 | |
| 22:43:14 | openstackgerrit | melanie witt proposed openstack/nova master: Remove broken CONF.pointer_model choice None https://review.opendev.org/711315 | |
| 23:06:49 | ozzzo | melwitt: that might help, I'll take a look | |
| 23:20:03 | ozzzo | reading further into that thread; they are repeating what was said here: "It doesn't seem possible to disable them QEMU side." | |
| 23:20:18 | melwitt | yeah | |
| 23:21:33 | melwitt | I found a few other threads in various places too while googling, saying the same | |
| 23:25:42 | melwitt | so I got curious how people get around the perf issues that seem to crop up | |
| 23:33:50 | openstackgerrit | Merged openstack/nova stable/train: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701525 | |
| 23:33:55 | openstackgerrit | Merged openstack/nova master: functional: Avoid race and fix use of self.api within test_bug_1831771 https://review.opendev.org/711211 | |
| 23:47:49 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-atttach-inerfaces policies https://review.opendev.org/706672 | |
| 23:54:19 | brinzhang | gmann: https://review.opendev.org/#/c/706672/3/nova/tests/unit/policies/test_attach_interfaces.py@85 | |
| 23:55:08 | openstackgerrit | Eric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363 | |
| 23:55:35 | brinzhang | gmann: why the os-volume-attachments policy can unauthorized the self.project_foo_context context? that you can review https://review.opendev.org/#/c/710190/3/nova/tests/unit/policies/test_volumes.py@131 | |
| 23:56:59 | gmann | brinzhang: just replied on review for instance action policy | |
| 23:57:18 | gmann | i will check volume policy tomorrow. | |
| 23:57:36 | brinzhang | gmann: thanks, I will look | |
| 23:58:27 | gmann | because of deprecation rules, we do not see the behavior of new defaults when we stop supporting the old one. | |
| 23:59:03 | gmann | also we cannot remove the old rules otherwise it break the deployment. | |
| 23:59:51 | brinzhang | gmann: sure | |
| #openstack-nova - 2020-03-05 | |||
| 00:03:59 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-deferred_delete policies https://review.opendev.org/707485 | |
| 00:53:11 | openstackgerrit | Brin Zhang proposed openstack/nova master: Correct the actual target in os-instance-actions policy https://review.opendev.org/710411 | |
| 00:56:58 | openstackgerrit | Brin Zhang proposed openstack/nova master: Correct the actual target in os-instance-actions policy https://review.opendev.org/710411 | |
| 02:15:37 | openstackgerrit | Merged openstack/nova stable/train: Make RBD imagebackend flatten method idempotent https://review.opendev.org/707650 | |
| 02:28:24 | openstackgerrit | Merged openstack/nova master: Remove oslo_db.sqlalchemy.compat reference https://review.opendev.org/711093 | |
| 02:47:21 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in os-agents policy https://review.opendev.org/701645 | |
| 04:44:43 | openstackgerrit | Merged openstack/nova master: libvirt: Remove native LUKS compat code https://review.opendev.org/669121 | |
| 07:22:24 | openstackgerrit | Eric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363 | |
| 09:59:51 | raghavendrat | Hi, | |
| 10:00:03 | raghavendrat | In nova, unable to attach interface to instance. | |
| 10:00:16 | raghavendrat | Output is available here: http://paste.openstack.org/show/790328/ | |
| 10:00:40 | raghavendrat | Appreciate if anyone can provide any pointers. | |
| 10:38:06 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add test coverage of existing os-instance-actions policies https://review.opendev.org/707777 | |
| 11:08:07 | openstackgerrit | Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751 | |
| 11:22:36 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/711210 | |
| 11:22:37 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Unplug VIFs as part of cleanup of networks https://review.opendev.org/711251 | |
| 11:22:38 | stephenfin | lyarwood: have squashed in your fix for the race there now ^ | |