Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2024-10-17
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
15:46:14 dansmith but I think much of nova (potentially conductor as well) could just move straight to real threads, ops can control how much parallelism they want (which they've asked for before) to scale performance, and then we can move on
15:47:17 johnthetubaguy Yeah, I had not thought of it like that. Its a good point.
15:49:06 clarkb just as a point of info zuul runs largely with real threads (I think the websocket lib we use may involve asyncio but I can't remember)
15:49:32 JayF ^ is what dtantsur has been pitching for IPA, although we've written no code in that direction

Earlier   Later