| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-06-20 | |||
| 18:16:04 | frickler | ack | |
| 18:19:11 | sean-k-mooney | pbr still depends on distutils by the way | |
| 18:19:12 | sean-k-mooney | https://opendev.org/openstack/pbr/src/branch/master/pbr/core.py#L51 | |
| 18:20:34 | sean-k-mooney | so until we drop that or we start adding seutuptools > whatever | |
| 18:20:49 | sean-k-mooney | to the relevent project that depend on distutils | |
| 18:20:57 | sean-k-mooney | we will need to keep the workarouncx in devstack | |
| 18:33:20 | opendevreview | Rafael Weingartner proposed openstack/nova master: Block VM instance resize to flavor with different aggregates https://review.opendev.org/c/openstack/nova/+/922421 | |
| 18:35:37 | opendevreview | Rafael Weingartner proposed openstack/nova master: Block VM instance resize to flavor with different aggregates https://review.opendev.org/c/openstack/nova/+/922421 | |
| 18:38:45 | clarkb | sean-k-mooney: PBR also depends on setuptools (which caries distutils these days) so you're depending on both really | |
| 18:39:11 | sean-k-mooney | clarkb: i didnt see the requrieemtns file for pbr in the repo | |
| 18:39:20 | sean-k-mooney | did i miss it or is that depenciy specified elsewher | |
| 18:39:47 | clarkb | sean-k-mooney: its implicit as pbr is a setuptools setup_requires hook or whatever its called | |
| 18:39:50 | sean-k-mooney | there are no deps in https://github.com/openstack/pbr/blob/d03d617c09e7ba8ddf62d1e53d71685cd708e2da/setup.cfg or https://github.com/openstack/pbr/blob/d03d617c09e7ba8ddf62d1e53d71685cd708e2da/setup.py | |
| 18:40:04 | clarkb | you can't list it as a requirment because of chicke and egg issues. The pyproject.toml documentation does explicitly list setuptools though | |
| 18:40:15 | sean-k-mooney | clarkb: ok but without an explict depency we can specify a min version | |
| 18:40:36 | sean-k-mooney | right here https://github.com/openstack/pbr/blob/d03d617c09e7ba8ddf62d1e53d71685cd708e2da/pyproject.toml.future#L7 | |
| 18:40:44 | clarkb | pbr is expected to run with a large range of setuptools versions | |
| 18:40:54 | clarkb | there probably is some min version but its going to be old | |
| 18:41:29 | sean-k-mooney | we basically need to say if python 3.12 reuqire setuptools >= the one with distutils | |
| 18:41:43 | sean-k-mooney | which we would normaly do in a requriemtes.txt | |
| 18:41:53 | sean-k-mooney | with the version spericifc marker | |
| 18:42:31 | clarkb | right but you can't because of the chicken and egg problem. The only solution to that is to use pyproject.toml instead | |
| 18:42:39 | clarkb | but no projects are doing that yet as far as I know | |
| 18:42:39 | sean-k-mooney | i.e. setuptools>=70;python_version='3.12' | |
| 18:43:09 | sean-k-mooney | clarkb: well i started adding pyproject.toml support | |
| 18:43:25 | sean-k-mooney | but it need the wsgi_srcipts to be replaced | |
| 18:43:32 | sean-k-mooney | so that work kind of stalled | |
| 18:43:36 | clarkb | basically the dependency may exist but you can't encode that in a requirements file in a way that works/makes sense. You can encode it in project.toml build requires though | |
| 18:44:27 | sean-k-mooney | well for now we can just have devstack install it | |
| 18:44:56 | sean-k-mooney | for porject other then pbr if they have a dep on distutils they can sepciry the requireemnt in the normal way | |
| 18:45:31 | clarkb | yes, the workaround there is appropriate. I just wanted to call out that it isn't just for distutils. You also need setuptools proper which isn't included by default in modern python either | |
| 18:45:55 | sean-k-mooney | ack ya thats also true | |
| 18:47:29 | sean-k-mooney | looking at codesearch/hound there are more project then i would like importing distutils | |
| 18:48:02 | sean-k-mooney | .... inlcuing oslo.versionedobjects | |
| 18:48:38 | sean-k-mooney | we proably should look to remove that dep and replace it with somethign else | |
| 18:49:25 | sean-k-mooney | mostly we seam to be suing it for version parsing | |
| 18:49:34 | sean-k-mooney | which we can replace with microversion_parse | |
| 18:51:20 | sean-k-mooney | frickler: the noble run is currently on tempet https://zuul.openstack.org/stream/0b61b3b596554e4093acd6e17f61b26c?logfile=console.log | |
| 18:51:37 | sean-k-mooney | so thats looking promising | |
| 18:51:43 | sean-k-mooney | im going to leave it there for today | |
| 22:48:13 | opendevreview | sean mooney proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| #openstack-nova - 2024-06-21 | |||
| 07:55:36 | jopdorp_ | Hi | |
| 07:56:13 | jopdorp_ | I was wondering if anyone has experience with H100 HGX systems , or maybe DGX and GPU passthrough to KVM VMs in nova | |
| 07:56:28 | jopdorp_ | We're seeing an issue where the nvidia-smi command hangs | |
| 07:57:06 | jopdorp_ | we do isntall fabricmanager, and I've tried some different configs of it. | |
| 07:57:29 | jopdorp_ | We're trying to pass one gpu per vm, on a host that has 8xH100 SXM gpus | |
| 07:57:49 | jopdorp_ | nv_open_q takes 100% of a single cpu core when nvidia-smi is invoked and hangs | |
| 11:45:59 | sean-k-mooney | jopdorp_: even if our downstream customers ask that question we would have to direct them to nvida supprot | |
| 11:46:22 | sean-k-mooney | what nvidia-smi does internally is really a black box that only they understand | |
| 14:02:39 | opendevreview | sean mooney proposed openstack/nova master: refactor cmd dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 16:47:15 | opendevreview | sean mooney proposed openstack/nova master: update nova-next to use ubuntu 24.04 https://review.opendev.org/c/openstack/nova/+/922492 | |
| 17:35:04 | opendevreview | sean mooney proposed openstack/nova master: Remove nova debugger funcitonality https://review.opendev.org/c/openstack/nova/+/922496 | |
| 17:35:05 | opendevreview | sean mooney proposed openstack/nova master: Add IO Thread Pool Executor https://review.opendev.org/c/openstack/nova/+/922497 | |
| 18:03:39 | opendevreview | sean mooney proposed openstack/nova master: remove eventlet.tpool https://review.opendev.org/c/openstack/nova/+/905287 | |
| #openstack-nova - 2024-06-22 | |||
| 03:10:45 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 03:10:45 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/922545 | |
| 03:10:46 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 03:10:46 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 03:10:47 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 03:10:47 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 07:13:02 | opendevreview | Michael Still proposed openstack/nova-specs master: Propose a new API microversion for a spice-direct console. https://review.opendev.org/c/openstack/nova-specs/+/915190 | |
| #openstack-nova - 2024-06-23 | |||
| 05:53:58 | opendevreview | Michael Still proposed openstack/nova-specs master: Propose a new API microversion for a spice-direct console. https://review.opendev.org/c/openstack/nova-specs/+/915190 | |
| #openstack-nova - 2024-06-24 | |||
| 01:55:39 | opendevreview | Michael Still proposed openstack/nova-specs master: Propose a new API microversion for a spice-direct console. https://review.opendev.org/c/openstack/nova-specs/+/915190 | |
| 04:02:22 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | |
| 04:06:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | |
| 05:03:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | |
| 07:30:58 | sahid | o/ morning all | |
| 07:31:10 | sahid | I this we are good with this one, CI is green now :-) | |
| 07:31:13 | sahid | https://review.opendev.org/c/openstack/nova/+/921665 | |
| 07:31:28 | sahid | sean-k-mooney, gibi if you have a moment | |
| 09:14:31 | mikal | Hey sean-k-mooney, sorry for the horrendous delay with that SPICE spec. I blame CentOS 7. I've replied to your various comments and done some updates. | |
| 09:14:44 | mikal | So whenever you're up for it I'd appreciate your thoughts. | |
| 09:33:21 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 09:41:27 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 10:22:38 | ralonsoh | sean-k-mooney, hello! maybe you'll like to check https://review.opendev.org/c/openstack/oslo-specs/+/922597 | |
| 10:23:11 | sean-k-mooney | ralonsoh: nice find ill take a look | |
| 10:23:43 | ralonsoh | sean-k-mooney, I talked to Daniel this morning, so I was warned | |
| 11:01:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | |
| 11:01:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Don't do version check if nothing required https://review.opendev.org/c/openstack/nova/+/915737 | |
| 11:01:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 11:01:51 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| 11:01:52 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | |
| 11:01:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | |
| 11:01:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server action APIs https://review.opendev.org/c/openstack/nova/+/915742 | |
| 11:01:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remaining server action APIs https://review.opendev.org/c/openstack/nova/+/915743 | |
| 12:15:54 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | |
| 12:18:18 | sean-k-mooney | mikal: i did a quick review and responed to some of the comments | |
| 12:20:02 | sean-k-mooney | mikal: this is my main feedback https://review.opendev.org/c/openstack/nova-specs/+/915190/comment/2ba5ce88_5e260fbf/ | |
| 12:20:47 | LarsErikP | Hi. Anyone running 2023.1, and a server with SapphireRapids-noTSX ? I got this weird behavior: https://paste.opendev.org/show/b8BM3HtnTKlQEqtv6i4I/ | |
| 12:20:48 | sean-k-mooney | i would love if we coudl make it so we can just return '"url":"spice://${KERBSIDE_HOST}/?token=f9906a48-b71e-4f18-baca-c987da3ebdb3"' | |
| 12:21:08 | LarsErikP | not really sure what happens here. I did not act like that when it was running zed... | |
| 12:21:45 | sean-k-mooney | LarsErikP: its likely a combination of 2 things 1 the qemu version your using and 2 the way nova is using the older libvirt api to detect comaptiablity | |
| 12:22:35 | sean-k-mooney | we have started to use the newer api recommeneded by the libvirt folk around 2023.1 ish | |
| 12:23:04 | LarsErikP | right.. so maybe I need a newer qemu version? | |
| 12:23:04 | sean-k-mooney | but saphire rapid in particalar is causing issues with the places we still use the old api | |
| 12:23:38 | LarsErikP | qemu-system-x86 1:6.2+dfsg-2ubuntu6.21 | |
| 12:24:17 | sean-k-mooney | we have 2 workaroudn options that can be used to side step the issue at the expecse fo delegating all cpu compatiablity checkign to libvirt | |
| 12:24:28 | sean-k-mooney | https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.skip_cpu_compare_on_dest and https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.skip_cpu_compare_at_startup | |
| 12:25:25 | LarsErikP | ah. ok. I'll try those | |
| 12:26:06 | sean-k-mooney | setting skip_cpu_compare_on_dest=true means tha tif the dest cpu is not compatiable instead of failing in pre-live migrate we will fail when we call migrate which is much later in the live-migrate workflow | |