| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-15 | |||
| 13:11:26 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Introduce config class for QEMU's "tb-cache" https://review.opendev.org/c/openstack/nova/+/816823 | |
| 13:12:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 13:12:17 | sean-k-mooney | dmitriis: yes more or less | |
| 13:12:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 13:12:33 | sean-k-mooney | i belive that shoudl eb using the plugtime event funciton | |
| 13:12:51 | sean-k-mooney | dmitriis: since we have already passed binding the port | |
| 13:13:02 | sean-k-mooney | dmitriis: currently it is https://github.com/openstack/nova/blob/e28afc564700a1a35e3bf0269687d5734251b88a/nova/virt/libvirt/driver.py#L7191-L7198 | |
| 13:15:01 | sean-k-mooney | dmitriis: effectvly it s implementaiton shoudl be like this https://github.com/openstack/nova/blob/0e0196d979cf1b8e63b9656358116a36f1f09ede/nova/network/model.py#L581-L586 | |
| 13:15:09 | sean-k-mooney | i suspect that is a latent bug actully | |
| 13:16:06 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove use of 'bind' arguments https://review.opendev.org/c/openstack/nova/+/817747 | |
| 13:16:06 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove unnecessary warning filters https://review.opendev.org/c/openstack/nova/+/817748 | |
| 13:17:08 | sean-k-mooney | actully dmitriis :) this might be the cause of our revert resize issue | |
| 13:17:09 | dmitriis | sean-k-mooney: the VIF object itself has has_bind_time_event actually https://github.com/openstack/nova/blob/e28afc564700a1a35e3bf0269687d5734251b88a/nova/network/model.py#L488-L497 . So I can extend it to accept `None` for the migration object and add handling for VNIC_TYPE since that's available in the VIF itself. | |
| 13:17:35 | dmitriis | sean-k-mooney: oh well, good thing we looked at this code then :^) | |
| 13:18:58 | sean-k-mooney | yes i would extend the vif class | |
| 13:20:32 | sean-k-mooney | artom has a repoducer patch up for the resize bug i think | |
| 13:20:53 | sean-k-mooney | so we might fix that issue for you in a seperate patch so we can backport it | |
| 13:22:30 | sean-k-mooney | this is where we repoduced our downstream bug https://review.opendev.org/c/openstack/nova/+/817303 | |
| 13:23:14 | sean-k-mooney | the revert resize test fail in a very specific situation whic i think is basially the same thing you are seeing | |
| 13:23:16 | sean-k-mooney | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f60/817303/2/check/nova-ovs-hybrid-plug/f60d54c/testr_results.html | |
| 13:24:51 | dmitriis | sean-k-mooney: ok, I'll fix it in my WIP spec implementation proposal https://review.opendev.org/c/openstack/nova/+/812111 and try it out on the actual test environment. We can have a separate patch to fix the issue and that will leave my patch with an extension to has_bind_time_event to handle VNIC_TYPE_SMARTNIC. | |
| 13:25:15 | sean-k-mooney | yep that makes sense to me | |
| 13:29:52 | sean-k-mooney | hum actully for the revert case we pass in the filtered events list | |
| 13:29:54 | sean-k-mooney | https://github.com/openstack/nova/blob/e28afc564700a1a35e3bf0269687d5734251b88a/nova/virt/libvirt/driver.py#L11204-L11210 | |
| 13:30:07 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Deprecate support for floppy drives https://review.opendev.org/c/openstack/nova/+/802022 | |
| 13:30:08 | sean-k-mooney | dmitriis: but this is still broken for your usecase | |
| 13:30:13 | sean-k-mooney | so we should still fix it | |
| 13:31:10 | dmitriis | sean-k-mooney: yes, it looks like the "active" state check might be there for that case but it doesn't help in my case | |
| 13:31:47 | sean-k-mooney | the active state chagne on the neutron port? | |
| 13:32:39 | dmitriis | sean-k-mooney: yes, on this line https://github.com/openstack/nova/blob/e28afc564700a1a35e3bf0269687d5734251b88a/nova/virt/libvirt/driver.py#L7198 | |
| 13:32:56 | dmitriis | this state gets updated via the info cache from what I understand | |
| 13:33:15 | sean-k-mooney | not quite | |
| 13:33:35 | sean-k-mooney | it will but that is not related | |
| 13:33:40 | sean-k-mooney | this is the admin state | |
| 13:33:56 | dmitriis | I see | |
| 13:34:03 | sean-k-mooney | we skip port that have been marked as down | |
| 13:34:52 | dmitriis | sean-k-mooney: is it OK if I extend has_bind_time_event to accept `None` for the `migration` argument? https://github.com/openstack/nova/blob/0e0196d979cf1b8e63b9656358116a36f1f09ede/nova/network/model.py#L488-L497 | |
| 13:35:37 | dmitriis | I don't have it in _get_neutron_events https://github.com/openstack/nova/blob/e28afc564700a1a35e3bf0269687d5734251b88a/nova/virt/libvirt/driver.py#L7191-L7198 and I'd rather not try to instantiate one there | |
| 13:35:40 | sean-k-mooney | i think so i have been trying to figure out what that should look like and or if you should jsut replace migration with same_host=True/False | |
| 13:36:20 | kashyap | sean-k-mooney: Hmm, this page should also list "bochs" as an option, right? - https://docs.openstack.org/glance/xena/admin/useful-image-properties.html | |
| 13:36:37 | kashyap | I think I forgot to updte the docs for that :-( I'm going to do that now | |
| 13:36:49 | sean-k-mooney | yes eventually and you should update the glance metadefs if you have not already done so | |
| 13:36:51 | kashyap | For hw_video_model, I mean | |
| 13:37:24 | sean-k-mooney | https://github.com/openstack/glance/blob/master/etc/metadefs/compute-libvirt-image.json#L71-L84 | |
| 13:37:44 | sean-k-mooney | that used to generate the horizon and heat dropdown menues | |
| 13:38:16 | sean-k-mooney | you should be able to do both in the same patch | |
| 13:39:22 | kashyap | Right; I updated this file: etc/metadefs/compute-libvirt-image.json and also this file: doc/source/admin/useful-image-properties.rst | |
| 13:39:44 | dmitriis | sean-k-mooney: yes, making `has_bind_time_event` accept a boolean seems cleaner than passing None. There aren't any other migration attributes used so far in it. | |
| 13:40:43 | kashyap | sean-k-mooney: I wonder if I should add a belated release note to the Glance docs? | |
| 13:41:31 | dmitriis | sean-k-mooney: I've put generic wording into the spec for now https://review.opendev.org/c/openstack/nova-specs/+/787458/12/specs/yoga/approved/integration-with-off-path-network-backends.rst#343 I think it encompasses whatever we decide implementation-wise. | |
| 13:41:43 | sean-k-mooney | dmitriis: right it also an encapsulation thing e.g. pass the minium it need rather then large objects | |
| 13:41:55 | dmitriis | sean-k-mooney: yes, agreed | |
| 13:43:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 13:45:27 | sean-k-mooney | dmitriis: something like this i think woudl make sense https://paste.opendev.org/show/811000/ | |
| 13:46:06 | sean-k-mooney | actuly result &= self['vnic_type'] != 'smartnic' shoudl be result |= self['vnic_type'] == 'smartnic' | |
| 13:46:30 | kashyap | Posted the docs: https://review.opendev.org/c/openstack/glance/+/817950 (Add "bochs" as a possible video model) | |
| 13:47:07 | sean-k-mooney | i.e. if the vnic_type is smartnic then we know the vif has bind time events | |
| 13:49:01 | dmitriis | sean-k-mooney: yeah, that's certainly better than my original workaround https://git.launchpad.net/~dmitriis/+git/nova/commit/?h=2021-09-10-off-path-net-backends&id=76bd42c59a33d3c9d410444ac32d3babde472ba4 | |
| 13:52:18 | kashyap | sean-k-mooney: Thank you for the quick review | |
| 15:37:56 | Zer0Byte | hey guys | |
| 15:39:04 | Zer0Byte | im having a issue with nova for one reason i can't live migrate vms from one node to a specific node im getting this error while im trying to move to these node Failed to retrieve allocations for consumer d9f8c2ed-74b0-4eb9-8b1d-30f397ec2102: Expected to find allocations for source node resource provider f7751b40-7f22-4b2f-8d40-b1479a369969. Retry the operation without forcing a destination host. | |
| 15:39:17 | Zer0Byte | checking openstack resource provider | |
| 15:39:31 | Zer0Byte | don't show the specifc node | |
| 15:39:49 | Zer0Byte | who should create on openstack resource provider these record? | |
| 15:43:52 | sean-k-mooney | Zer0Byte: nova | |
| 15:44:12 | sean-k-mooney | are you upgrading form a deployment without placement to one with placment | |
| 15:44:38 | sean-k-mooney | if so there are some specific steps you need to do to enusre the allcoation are created | |
| 15:45:10 | Zer0Byte | no was runinng perfect i notice some performance degradation to keystone because switft that make me belive that nova can't communicate with placement on these moment | |
| 15:46:22 | sean-k-mooney | hum you suspect that the keystone authention some how did not work when it was trying to retive the allcoations | |
| 15:46:36 | sean-k-mooney | have you check the instnace in quetion to see if it has allcoation manually | |
| 15:46:49 | Zer0Byte | when the node was registered i suspect keystone was unavailable | |
| 15:47:01 | Zer0Byte | i see the node registered on nova service | |
| 15:47:07 | Zer0Byte | but not on resource provider | |
| 15:47:41 | sean-k-mooney | is the node in "openstack hypervisor list" | |
| 15:48:08 | Zer0Byte | yep | |
| 15:48:37 | sean-k-mooney | but if you do a openstack resouce provider show <compute node uuid> | |
| 15:48:41 | sean-k-mooney | it does not find the RP | |
| 15:49:22 | EugenMayer | what is the right way to segment an network into 2 subnets and let those instances reach eachother? (OVN) do i really need an router, a gateway in both subnets and the add a port of each subnet to that router? | |
| 15:49:45 | Zer0Byte | dont listed on resource provider list | |
| 15:49:57 | sean-k-mooney | EugenMayer: you need a neutorn router to interconnect the subnets | |
| 15:50:06 | Zer0Byte | don't appears on resource provider list | |
| 15:50:14 | sean-k-mooney | EugenMayer: unless you are using provider networking which you wont be if your using OVN | |
| 15:50:22 | EugenMayer | sean-k-mooney ok so that is the only option. | |
| 15:51:12 | sean-k-mooney | EugenMayer: more or less yes. if they are on the same neutron network with ovn technically they should have l2 conenctivty and be able to comunicate using the mac adress only | |
| 15:52:09 | sean-k-mooney | but arp for a non local subnet will always be sent to the current subnets routeer and the network stack withotuh a static route will assume a upstream reouter can interconnect the subnets | |
| 15:52:21 | EugenMayer | sean-k-mooney: i have an issue with the fact that i can only add a router with those 2 interfaces, if a gateway is presnet. I have a network A (sub1/sub2) and network B (sub1/sub2). I want an instance to be in A sub1 with the default gateway of Asub1, while this instnace is also in Bsub1. The problem is, if Bsub1 also has a gateway (which i need to | |
| 15:52:21 | EugenMayer | create the router so Bsub1 can reach Bsub2) the gateway of Bsub1 is pushed | |
| 15:52:26 | EugenMayer | (default gateway) | |
| 15:53:20 | sean-k-mooney | so in generall you will create a tenant network and a rounter to connect the tenant network to the external network | |
| 15:53:43 | sean-k-mooney | and if you want to have 2 subnets on that tenant network you just add a second interface to the existing router for the second subnet | |
| 15:54:35 | sean-k-mooney | Zer0Byte: ok if it does not appear in the RP list then if you check the compaute agent log on that host it should have an error | |
| 15:55:08 | sean-k-mooney | Zer0Byte: in the logs o fthe update_avaiable_resouces periodic task if not earlier | |
| 15:56:12 | EugenMayer | sean-k-mooney my problem is with the default gateway pushed, if an instance has 2 interfaces which both have a gateway | |
| 15:56:49 | sean-k-mooney | right that is a common problem | |
| 15:56:57 | sean-k-mooney | you can disable the gateway on the subnet | |
| 15:57:17 | sean-k-mooney | which should prevent that | |
| 15:58:53 | sean-k-mooney | EugenMayer: i think you do "openstack subnet unset --gateway <subnet>" | |
| 15:59:00 | EugenMayer | sean-k-mooney yes, that is what i did, then i cannot route Bsub1 to Bsub2 - you cannot add the interface | |
| 15:59:21 | EugenMayer | if a subnet has not gateway, it cannot be added to the router to route to the subnets | |
| 15:59:30 | sean-k-mooney | you can add the interface but you have to specify the ip manually to be what would be the normal gateway ip | |