| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-09 | |||
| 15:50:01 | CeeMac | makes sense | |
| 15:58:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Only add a USB controller if it's necessary https://review.opendev.org/756549 | |
| 15:58:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove support for '[libvirt] use_usb_tablet' https://review.opendev.org/756550 | |
| 15:58:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Rationalize attachment of USB tablet https://review.opendev.org/756551 | |
| 15:58:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add support for virtio-based input devices https://review.opendev.org/756552 | |
| 15:59:59 | sean-k-mooney | stephenfin: did you add None to the pointer device. the original requirement diring this is to have no input device attached to the guest | |
| 16:00:11 | sean-k-mooney | so using virtio wont actully help in the realtime case | |
| 16:00:27 | sean-k-mooney | i assume that is just a nice to have while your here | |
| 16:03:11 | admin0 | hi guys .. can the default domain be null or "" blank ? | |
| 16:03:15 | admin0 | insetad of .novalocal ? | |
| 16:03:33 | sean-k-mooney | dns domain | |
| 16:03:44 | admin0 | can it be null ? | |
| 16:03:56 | admin0 | or some value has to be there | |
| 16:04:14 | sean-k-mooney | its set by https://docs.openstack.org/nova/latest/configuration/config.html#api.dhcp_domain | |
| 16:04:51 | admin0 | it does not say if it can be left null | |
| 16:05:00 | sean-k-mooney | i dont think it can be | |
| 16:05:48 | sean-k-mooney | although we have exampel of it being empty | |
| 16:06:27 | sean-k-mooney | we use it like this | |
| 16:06:28 | sean-k-mooney | https://github.com/openstack/nova/blob/58636d34ec68d44952b22947bac01d4b88ae103a/nova/api/metadata/base.py#L533-L536 | |
| 16:06:56 | sean-k-mooney | you would have to set it to "" | |
| 16:06:58 | sean-k-mooney | https://github.com/openstack/nova/blob/85c35b4d9d6386866ddf197e4a751f729b726188/nova/conf/api.py#L218-L222 | |
| 16:07:04 | sean-k-mooney | but i dont know if that wil work or not | |
| 16:07:14 | sean-k-mooney | default might still replace "" | |
| 16:07:18 | sean-k-mooney | but it would be easy to test | |
| 16:07:34 | sean-k-mooney | just set [api]/dhcp_domain="" | |
| 16:12:42 | melwitt | it can be blank. we had it set that way at yahoo | |
| 16:35:07 | stephenfin | sean-k-mooney: No, it wasn't helpful. You can't disable the built-in PS2 devices | |
| 16:35:23 | stephenfin | However | |
| 16:35:38 | sean-k-mooney | you cant by generating model=none in the xml | |
| 16:36:00 | stephenfin | I don't think that's supported. The only RFE I found for it was DEFERRED | |
| 16:36:40 | sean-k-mooney | well the perfromacne team teh filed teh orignal downstream issue had said they disabeld it manually by editing the xml | |
| 16:37:02 | stephenfin | they were talking about USB devices | |
| 16:37:15 | sean-k-mooney | yes | |
| 16:37:32 | stephenfin | you won't get a USB keyboard or pointer device on x86 unless you request them via image metadata properties or nova.conf | |
| 16:37:52 | sean-k-mooney | no libvirt will add it | |
| 16:37:52 | stephenfin | so all that's necessary is to remove the superfluous USB bus | |
| 16:37:59 | stephenfin | not a USB one, it won't | |
| 16:38:08 | stephenfin | it'll add a PS2 mouse and keyboard | |
| 16:38:11 | stephenfin | not USB | |
| 16:38:22 | sean-k-mooney | ok but i think they wanted to remvoe that too | |
| 16:38:41 | stephenfin | See above. I don't think that's supported | |
| 16:39:01 | sean-k-mooney | ya so your proposing what use virtio? | |
| 16:39:04 | sean-k-mooney | or ps2 | |
| 16:39:05 | stephenfin | nope | |
| 16:39:22 | stephenfin | I've four patches | |
| 16:39:23 | sean-k-mooney | they are hoping to remove all interupts related to the input device | |
| 16:39:40 | stephenfin | first one is to remove the unnecessary USB bus | |
| 16:40:54 | stephenfin | With that, you can set '[compute] pointer_model' to 'none' or 'ps2mouse' and, unset the 'hw_pointer_model' image metadata property prop, and ensure you haven't set 'hw_*_bus' to 'usb' | |
| 16:41:05 | stephenfin | and you won't have any USB devices or a USB bus | |
| 16:41:18 | stephenfin | the second is a cleanup patch | |
| 16:41:31 | sean-k-mooney | well we dont want this to be in the nova.conf really | |
| 16:41:38 | sean-k-mooney | that will cause issue for live migation | |
| 16:41:51 | stephenfin | the third makes it so that we only attach a USB tablet if graphics (SPICE or VNC) are enabled | |
| 16:41:58 | stephenfin | because it's useless otherwise | |
| 16:42:20 | stephenfin | and the fourth adds support for virtio devices because they're supposed to be far more performant and it was an easy win | |
| 16:43:00 | stephenfin | Agreed. However, I can't remove it until you add that system metadata thing for Q35 | |
| 16:43:37 | sean-k-mooney | :( ya | |
| 16:44:18 | sean-k-mooney | ok we might want to update the downstream bz to say the input device cant be removed but we are goign to do this instead | |
| 16:44:53 | stephenfin | I don't think the PS2 device is a massive issue. It's the USB bus that was doing most of the damage | |
| 16:45:01 | stephenfin | But I should check again to be sure | |
| 16:45:10 | stephenfin | Monday! For now, it's sunny outside so o/ :) | |
| 16:45:51 | sean-k-mooney | i guess thre expect result was actully "no USB and no graphics" | |
| 16:46:09 | sean-k-mooney | so ps2 device or virtio might be fine actully | |
| 16:46:18 | sean-k-mooney | based on https://bugzilla.redhat.com/show_bug.cgi?id=1787442 | |
| 16:46:18 | openstack | bugzilla.redhat.com bug 1787442 in documentation ""Configuring Real-Time Compute" lacks guest device configuration tuning" [Unspecified,Closed: wontfix] - Assigned to stephenfin | |
| 16:50:53 | admin0 | sean-k-mooney, i made it blank and it still gives me a .novalocal | |
| 16:51:00 | admin0 | is that something from neutron in dhcp ? | |
| 16:51:47 | sean-k-mooney | no i think since novalocal is the default of that coning option you have to do ="" | |
| 16:51:55 | sean-k-mooney | i dont think blank will actully work | |
| 16:52:40 | sean-k-mooney | melwitt: do you recal if ye set dhcp_domain= or dhcp_domain="" | |
| 16:52:56 | melwitt | dhcp_domain="" | |
| 16:53:07 | melwitt | set to empty string | |
| 16:53:30 | melwitt | sorry, that's what I meant when I said "blank" earlier | |
| 16:55:21 | sean-k-mooney | ya ok that makes sense | |
| 16:55:39 | sean-k-mooney | dhcp_domain= i think will be treated like its unset and the default will be used | |
| 16:56:06 | sean-k-mooney | admin0: this might only take effect for new vms too | |
| 16:56:35 | admin0 | unless i can modify the dhcp config manually | |
| 16:56:38 | sean-k-mooney | for existing vm i think we store there hostname in the db | |
| 16:57:18 | sean-k-mooney | you would need to update the instnace.hostname filed in the db | |
| 16:57:21 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L130 | |
| 18:00:08 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 18:00:08 | openstack | bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt) | |
| 18:00:08 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637 | |
| 18:03:20 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 18:03:20 | openstack | bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt) | |
| 18:03:21 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637 | |
| 18:03:43 | melwitt | omg uuughh | |
| 18:05:27 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 18:05:27 | openstack | bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt) | |
| 18:05:28 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637 | |
| 18:06:19 | melwitt | sorry for the spam | |
| 19:05:31 | openstackgerrit | melanie witt proposed openstack/nova stable/train: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/757177 | |
| 19:47:44 | melwitt | mnaser: is ^ something you would find useful/appropriate for sta | |
| 19:47:55 | melwitt | *stable/train or nah? | |
| #openstack-nova - 2020-10-12 | |||
| 01:00:45 | openstackgerrit | Jorhson Deng proposed openstack/nova master: add task state for attach volume and detach volume https://review.opendev.org/754258 | |
| 05:56:04 | openstackgerrit | MaAoyu proposed openstack/os-traits master: bump py37 to py38 in tox.ini https://review.opendev.org/757432 | |
| 06:27:41 | openstackgerrit | zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448 | |
| 07:00:36 | gibi | good morning nova | |
| 07:26:51 | openstackgerrit | Merged openstack/nova master: Follow up for cherry-pick check for merge patch https://review.opendev.org/756639 | |
| 08:23:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Revert "Temporarily disable parts of heal port allocation test" https://review.opendev.org/755995 | |