| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-10-17 | |||
| 13:11:04 | bauzas | the fact that we return 'nova' per the user AZ list ? | |
| 13:11:06 | sean-k-mooney | bauzas: as a non admin if you do an az list you get nova | |
| 13:11:16 | bauzas | if so, we should *not* return it, I agree | |
| 13:11:22 | sean-k-mooney | which mean it looks valid to and end suer | |
| 13:11:33 | sean-k-mooney | because they cant no what the defautl az is | |
| 13:11:42 | sean-k-mooney | *default internal az is | |
| 13:12:03 | sean-k-mooney | actuilly no i was right the first time https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.default_availability_zone | |
| 13:12:32 | bauzas | okay, then I agree: we shouldn't return the default AZ in that list | |
| 13:12:35 | bauzas | https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.default_availability_zone | |
| 13:12:48 | sean-k-mooney | you can actully create a real az callded nova and change that to something else | |
| 13:13:03 | sean-k-mooney | the defautl az can be returned if its an actual az | |
| 13:13:07 | bauzas | for defaulting to pin a specific AZ, this is https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.default_schedule_zone | |
| 13:13:28 | bauzas | we could even make default_az to None, I don't see why we need to have 'nova' | |
| 13:13:37 | sean-k-mooney | i was just thinking about that | |
| 13:13:41 | bauzas | but we could and should discuss this at the PTG | |
| 13:13:55 | sean-k-mooney | ya i dont know of the top of my head anything that would break | |
| 13:14:27 | sean-k-mooney | it would stop instance reporting an az if they are on a host that are not part of any az | |
| 13:14:34 | sean-k-mooney | but that is more correct then "nova" | |
| 13:14:36 | pas-ha[m] | as I said, api ref already promises that non-admin should not see it | |
| 13:14:37 | pas-ha[m] | > The default availability zone is named nova. This AZ is only shown when listing the availability zones as an admin. | |
| 13:14:37 | pas-ha[m] | https://docs.openstack.org/api-ref/compute/#create-server | |
| 13:15:02 | sean-k-mooney | pas-ha[m] reight beu twe have a bug because that is not what the api is doing | |
| 13:16:22 | sean-k-mooney | http://192.168.50.7:80 "GET /compute/v2.1/os-availability-zone HTTP/1.1" 200 97 | |
| 13:16:24 | sean-k-mooney | RESP: [200] Connection: close Content-Length: 97 Content-Type: application/json Date: Thu, 17 Oct 2024 13:15:54 GMT OpenStack-API-Version: compute 2.1 Server: Apache/2.4.58 (Ubuntu) Vary: OpenStack-API-Version,X-OpenStack-Nova-API-Version X-OpenStack-Nova-API-Version: 2.1 x-compute-request-id: req-17af8a83-82f0-45ce-81fd-c6d54a0c0ad9 x-openstack-request-id: | |
| 13:16:26 | sean-k-mooney | req-17af8a83-82f0-45ce-81fd-c6d54a0c0ad9 | |
| 13:16:28 | sean-k-mooney | RESP BODY: {"availabilityZoneInfo": [{"zoneName": "nova", "zoneState": {"available": true}, "hosts": null}]} | |
| 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 | |