| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-10-17 | |||
| 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 | |
| 15:44:06 | johnthetubaguy | dansmith: are you thinking threads for each task being better for what nova-compute is generally doing? I hadn’t really thought about that. I could see that limiting the code changes, I suppose? | |
| 15:44:29 | dansmith | johnthetubaguy: I want us to at least consider that yeah | |
| 15:44:35 | bauzas | johnthetubaguy: that was something we were considering from the last PTG | |
| 15:45:29 | dansmith | johnthetubaguy: I suspect we have some places where we think we're threadsafe, but aren't | |
| 15:45:38 | dansmith | so we'll need to fix that up, which we should have fixed anyway | |