| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-10-17 | |||
| 13:16:56 | sean-k-mooney | ill quickly checkc but im 99% shure i dont have any hostaggrate defiend on this devstack | |
| 13:17:24 | pas-ha[m] | And I am afraid something would break if we start doing that. This would effectively force operators to always put hosts into some explicit AZ, because users who are used to pick the AZ from the list will just use one of those the list presents them, making hosts not in any AZ underused | |
| 13:18:30 | sean-k-mooney | well az are optional | |
| 13:18:42 | sean-k-mooney | i would expect most user not to sepcify one | |
| 13:18:59 | sean-k-mooney | unless its via horizon in shich case they still dont need to select one | |
| 13:24:54 | sean-k-mooney | pas-ha[m]: the alaswer ot this is AZ shoudl be replacced with a first class api object and we shoudl have some way to auto enrole comptue services in AZ | |
| 13:25:23 | sean-k-mooney | the problem is we have never found an accpatbale solution to do that but i know other are interested in that direction | |
| 13:27:08 | pas-ha[m] | just to re-iterate, for the exact use case that we had broken for us:... (full message at <https://matrix.org/oftc/media/v1/media/download/AZi6ENlpQm3FPa8euVAaf8oGaS7R6Eq6HS5W4Rwq_LvrIBUm5dYilERrZnNQr5tnKVfobGYw-LXZy-WaqFYBXT5CeS4sIl5AAG1hdHJpeC5vcmcva0RHSEVMREVleFlzVUF6SldadkRXeWlW>) | |
| 13:28:43 | sean-k-mooney | an AZ is nto the correct way to to enabel that usecase | |
| 13:28:51 | sean-k-mooney | ti can be doen with az but its error prone | |
| 13:31:06 | sean-k-mooney | pas-ha[m]: just to make sure i understand correctly do you allow hugepage flavor to be used elsewhere or rather what is the cirtia for allowing it on a dpdk host | |
| 13:33:17 | pas-ha[m] | only those hosts are configured with huge pages, so scheduler filters other hosts out. the problem is to limit flavors that not use huge pages out of those hosts. | |
| 13:34:27 | pas-ha[m] | probably adding some extra spec like forbidden=... to any other flavor would work too, but that's more cumbersome than what was working before. | |
| 13:34:38 | sean-k-mooney | you can map specific flavor to host with the AggregateTypeAffinityFilter you can use can also use AggregateInstanceExtraSpecsFilter or aggregate_image_properties_isolation to enfoce extra specs are requested in teh flavor/image or you can use the palcemnet feature ath was created explcitly for this type of grouping | |
| 13:34:40 | sean-k-mooney | https://docs.openstack.org/nova/latest/reference/isolate-aggregates.html | |
| 13:35:00 | sean-k-mooney | if you use ^ | |
| 13:35:54 | sean-k-mooney | you can make so that if a falvor does not request a custome trait it cant be shcheduled to the host | |
| 13:37:30 | sean-k-mooney | you just create a CUSTOM_DPDK trait. create a host aggate with your dpdk host and add trait:CUSTOM_DPDK=required | |
| 13:37:46 | sean-k-mooney | and request trait:CUSTOM_DPDK=required in your dpdk flavors | |
| 13:40:32 | sean-k-mooney | pas-ha[m]: that was added in train https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/placement-req-filter-forbidden-aggregates.html | |
| 18:39:55 | opendevreview | Ghanshyam proposed openstack/nova master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/nova/+/932648 | |
| 20:00:19 | opendevreview | Mengyang Zhang proposed openstack/nova-specs master: Add Burst Length Support to Cinder QoS https://review.opendev.org/c/openstack/nova-specs/+/932653 | |
| 20:25:29 | opendevreview | Mengyang Zhang proposed openstack/nova-specs master: Add Burst Length Support to Cinder QoS https://review.opendev.org/c/openstack/nova-specs/+/932653 | |
| #openstack-nova - 2024-10-18 | |||
| 03:21:38 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | |
| 04:42:18 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | |
| 05:26:31 | opendevreview | norman shen proposed openstack/nova master: Add timeout to ceph util's command line if necessary https://review.opendev.org/c/openstack/nova/+/932674 | |
| 08:51:12 | opendevreview | norman shen proposed openstack/nova master: Add timeout to ceph util's command line if necessary https://review.opendev.org/c/openstack/nova/+/932674 | |
| 13:12:41 | haleyb | bauzas: do you think we need a nova/neutron cross-project next week? the only thing i know about is multiple port bindings, the thing we never seem to get to :( | |
| 13:13:10 | bauzas | haleyb: I have no ask in the etherpad | |
| 13:13:59 | haleyb | bauzas: ack, if something changes we probably have some extra time slots | |
| 13:14:31 | sean-k-mooney | haleyb: the one about using service tokens or the ceanup we shoudl eb doing about shelve or makign it required for all neutron deploymeents | |
| 13:14:52 | sean-k-mooney | haleyb: there are seveal multiple portbiding topics | |
| 13:15:05 | sean-k-mooney | but i dont think talking about them hleps | |
| 13:15:22 | sean-k-mooney | we shoudl just deprecation the support for not having muptipel port binding in nova | |
| 13:15:52 | sean-k-mooney | since its a slrup band figure out if we ever have time to enforce it after we do all the other higher priortiy things | |
| 13:18:10 | sean-k-mooney | haleyb: there is one topic i guess but it does not have to be in the ptg | |
| 13:18:35 | sean-k-mooney | we need a decisoin on how to move forward with https://review.opendev.org/c/openstack/os-vif/+/923036 so we can merge that and finally fix it | |
| 13:18:53 | haleyb | sean-k-mooney: ack regarding the service tokens, i'll at least add it to our agenda as we should plan it | |
| 13:19:25 | sean-k-mooney | haleyb: for now i think you made port bining admin_or_service | |
| 13:19:47 | sean-k-mooney | i think nova already passes a service token alays wen talkign to neutorn | |
| 13:20:07 | sean-k-mooney | so it might already work if we removed the admin role form nova but i dont think anywone has tried | |
| 13:20:23 | sean-k-mooney | i kind o include this in "nova should use the sdk to talk to neutron" topic | |
| 13:20:57 | sean-k-mooney | i.e. so ye can delete python-neutornclient | |
| 13:21:07 | sean-k-mooney | but i dont think that is contoverial | |
| 13:21:20 | sean-k-mooney | it just need people to do it so not sure it need a ptg discussion | |
| 13:21:32 | haleyb | sean-k-mooney: yes, that deprecation is ongoing | |
| 16:44:09 | opendevreview | Stephen Finucane proposed openstack/nova master: [codespell] Fixes for latest version https://review.opendev.org/c/openstack/nova/+/923738 | |
| 16:44:09 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Bump versions https://review.opendev.org/c/openstack/nova/+/923739 | |
| #openstack-nova - 2024-10-19 | |||
| 01:10:56 | opendevreview | norman shen proposed openstack/nova master: Add timeout config for Rados python object https://review.opendev.org/c/openstack/nova/+/932766 | |
| 08:51:03 | opendevreview | norman shen proposed openstack/nova master: Add timeout config for Rados python object https://review.opendev.org/c/openstack/nova/+/932766 | |
| #openstack-nova - 2024-10-20 | |||
| 00:21:13 | opendevreview | norman shen proposed openstack/nova master: Add timeout config for Rados python object https://review.opendev.org/c/openstack/nova/+/932766 | |
| 01:54:49 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 01:54:50 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 01:54:50 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 01:54:51 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 01:54:51 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| #openstack-nova - 2024-10-21 | |||
| 11:25:25 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: Drop unnecessary 'x' bit from doc config file https://review.opendev.org/c/openstack/os-resource-classes/+/932878 | |
| 11:25:38 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Drop unnecessary 'x' bit from doc config file https://review.opendev.org/c/openstack/os-traits/+/932879 | |
| 11:26:41 | opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Drop unnecessary 'x' bit from doc config file https://review.opendev.org/c/openstack/osc-placement/+/932881 | |
| 12:40:14 | sean-k-mooney | HTTP exception thrown: Flavor has hw:virtio_packed_ring extra spec explicitly set to True, conflicting with image which has hw_virtio_packed_ring explicitly set to true. | |
| 12:40:19 | sean-k-mooney | ... well thats a bug | |
| 12:40:37 | sean-k-mooney | we are doing a sting compare somewhere instead of comparing the boolean values | |
| 12:43:27 | tkajinam | https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1120 | |
| 12:43:35 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/2085124 | |
| 12:44:04 | tkajinam | this is used not only for boolean properties but also for string properties. We probably have to add a flat to indicate whether the value should be converted before comparison | |
| 12:44:37 | sean-k-mooney | perhaps we have other helpers too | |
| 12:44:51 | sean-k-mooney | i normally expect us to call _get_flavor_image_meta | |
| 12:45:05 | sean-k-mooney | and hten have a spereate funciton per extra spec to chekc it | |
| 12:45:12 | sean-k-mooney | like get_mem_encryption_constraint | |
| 12:45:30 | sean-k-mooney | so we proably used _get_unique_flavor_image_meta to make it simpler but didnt think about this usecase | |
| 12:45:43 | sean-k-mooney | rather then a flavor for conversion we proably need to pass an expected type | |
| 12:45:59 | sean-k-mooney | or convertion function for the flavor value. the image ones are already typed | |
| 12:46:17 | tkajinam | yeah | |
| 12:46:25 | tkajinam | ok the logic for packed_ring is specifically implemented in get_packed_virtqueue_constraint | |
| 12:46:30 | sean-k-mooney | i.e. pass in bool or oslo.utils.str_to_bool whatever taht is called | |
| 12:47:48 | sean-k-mooney | ya so the issue is proably here https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1985 | |
| 12:47:57 | sean-k-mooney | in that path we are not converting it to the correct type | |
| 12:48:16 | sean-k-mooney | we convert the flavor to the correct type here https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1990-L1992 | |
| 12:48:28 | sean-k-mooney | but in the dict case the image_value is just a string | |
| 12:49:15 | sean-k-mooney | so we need to call strutils.bool_from_string here https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1985 | |
| 13:54:13 | jlejeune | hello, I have a question regarding host_mapping, is it "normal" that when we delete a nova-compute service, the 'mapped' field in nova_cell1.compute_nodes table stays at 1 instead of 0 while nova_api.host_mappings related entry is removed ? | |
| 13:55:23 | jlejeune | by letting that 'mapped' field to 1 prevent nova-manage cell_v2 discover_hosts command to create the missing mapping | |
| 13:56:37 | gibi | jlejeune: it sounds like bug #1 in scenario 1 in https://bugs.launchpad.net/nova/+bug/2077070/comments/7 | |
| 13:57:17 | gibi | jlejeune: could you check if adding --by-service to the discover_hosts command helps you or not | |
| 13:57:21 | gibi | ? | |
| 13:57:41 | jlejeune | I've already test it and indeed, it "fix" the issue by creating the missing mapping | |
| 13:58:14 | gibi | OK. Then I think it is the same issue. | |
| 13:58:32 | gibi | so yeah, nova should set mapped to 0 when the host mapping is deleted. | |
| 13:58:40 | jlejeune | but not sure I want to do it by service, given the number of computes we have in production | |
| 13:59:01 | jlejeune | agree | |
| 13:59:35 | gibi | if you have time to propose a fix then I'm happy to review it. If not then eventually I get to it myself to propose the fix. | |
| 14:01:46 | jlejeune | I will have a look ! | |
| 14:02:26 | gibi | jlejeune: thanks! | |
| 14:27:27 | jlejeune | gibi: maybe we should split the bug in two, one for pci devices and another one for the mapping issue ? | |
| 14:31:56 | gibi | jlejeune: I don't think there is anythin else in the pci side, but if you want a separate bug to fix the mapping issue then sure go ahead and file one. I agree that not mentioning pci devices in the separate bug will help the reviewers not bog down on the pci stuff. | |
| 14:34:00 | jlejeune | yes exactly :) | |
| 14:35:43 | gibi | ping me with the separate bug and I will quickly do the triage of it | |
| 14:37:21 | jlejeune | ok | |
| 14:55:02 | jlejeune | gibi: https://bugs.launchpad.net/nova/+bug/2085135 | |
| 15:09:46 | gibi | jlejeune: thank | |