| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-19 | |||
| 09:07:38 | bauzas | honestly, that simple series became way more tricky than expected | |
| 09:07:55 | bauzas | because the kernel doesn't act like I was expecting | |
| 09:08:07 | bauzas | I just wish this was more documented | |
| 09:08:43 | opendevreview | Merged openstack/os-traits master: Add a new trait for AMD SEV-ES https://review.opendev.org/c/openstack/os-traits/+/908966 | |
| 09:10:28 | gibi | turning on cpus during nova-compute shutdown is leaky, what if the hypervisor is rebooted out of the blue? | |
| 09:13:11 | gibi | if we want to go this direction the it is safer to do the following in init_host: turn on all dedicated cores, collect governor data, run off all unused dedicated cores. | |
| 09:13:19 | opendevreview | Fabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load https://review.opendev.org/c/openstack/nova-specs/+/922201 | |
| 09:15:49 | bauzas | I need to leave now but I'm cool with discussing this later on this afternoon | |
| 09:16:23 | sean-k-mooney | gibi: if the hyperiofr is reboot all the cores would be one but we coudl turn them all on, gather the numa info once and then turn them off | |
| 09:16:27 | sean-k-mooney | when we start | |
| 09:16:47 | sean-k-mooney | so yes im fine with that in init_host | |
| 09:17:44 | sean-k-mooney | so the toploty info of a core (socket, numa node, cluster, die) shoudl not change while the agent is running | |
| 09:17:53 | sean-k-mooney | that would iply the phsical hardware changed | |
| 09:18:40 | gibi | yeah I agree that we should be able to cache the topology at startup and not re-query it. | |
| 09:19:03 | sean-k-mooney | so if we can ensure we are collecting good data, (by turning on all the cores we manage), we can look that up once and stop doing it every 5 mins for no real value | |
| 09:20:04 | sean-k-mooney | i think in the short term that is the solution to the bug, | |
| 09:20:50 | sean-k-mooney | in the long term as i said to bauzas i think we have usecase wehre not could benifit form a persitient datastore that is local to the nova-compute agent fo some kind | |
| 09:21:18 | gibi | yeah, we can discuss the persistent store for long term. | |
| 09:21:18 | sean-k-mooney | so we have a tool we can use to solve these type of problems in a more robust way | |
| 09:22:45 | sean-k-mooney | too be clear im alo not opposed ot useing the filesytem as a db. that works very well and is simple to debug and understand but i think we shoudl gater some usecase and see why might be more approate and talk about this more before or during the ptg | |
| 09:35:43 | gibi | bauzas: I left a simple request in the power mgmt fix https://review.opendev.org/c/openstack/nova/+/924427 | |
| 09:40:37 | sean-k-mooney | gibi: yep i agree with % | |
| 09:40:39 | sean-k-mooney | ^ | |
| 09:41:43 | gibi | bauzas: and thanks for quickly jumping on this bug | |
| 10:23:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924427 | |
| 10:23:20 | bauzas | sean-k-mooney: gibi: patch updated | |
| 10:23:34 | bauzas | once merged, I'll provide the backports to Antelope | |
| 10:37:33 | sean-k-mooney | the previous results were green and the change is trivial so +2 | |
| 10:45:34 | gibi | bauzas: thanks +A | |
| 11:32:32 | bauzas | tkajinam: cores, fwiw os-traits release patch https://review.opendev.org/c/openstack/releases/+/924492 | |
| 11:56:17 | stephenfin | gmann: Lemme know when you're around. I'm not sure I understand what you mean in https://review.opendev.org/c/openstack/nova/+/915738 | |
| 11:56:29 | opendevreview | Stefan Hoffmann proposed openstack/nova master: wait for ovn network at migration https://review.opendev.org/c/openstack/nova/+/924500 | |
| 12:00:25 | songwenping_ | sean-k-mooney: hi, the cmd 'openstack server migration show f4078cee-71c0-4a47-a0e6-76b67ab625fa 845cca1c-ec99-43bd-a7e8-ce588c0e1dbf' raise exception 'In-progress live migration ddf01c0f-d0f3-469e-a320-ebcb6732420e is not found for server f4078cee-71c0-4a47-a0e6-76b67ab625fa.' | |
| 12:00:37 | songwenping_ | the migration status is failed | |
| 12:01:07 | songwenping_ | so we cannot get the detail of failed migration? | |
| 12:01:54 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Clarify '[api] response_validation help' text https://review.opendev.org/c/openstack/nova/+/924501 | |
| 13:48:16 | atmark | hello, we have bunch of windows VMs that are boot from volume and got impacted by crowdstrike incident today. As per doc, `Rescuing a volume-backed instance is not supported with this mode`. Is there any method to rescue an bfv instance? | |
| 14:13:07 | sean-k-mooney | atmark: it depend on the release | |
| 14:15:19 | sean-k-mooney | rescuse with boot form voluem was added in ussuri https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-bfv-instance-rescue.html | |
| 14:22:43 | atmark | sean-k-mooney: thought it wasn't supported. i'm on xena | |
| 14:26:23 | sean-k-mooney | you need to use this with the stable rescue feature i think but its been supproted for several years now | |
| 14:32:57 | atmark | ok, i can't get it to work. I uploaded with cirros image with this property `--property hw_rescue_device=cdrom` | |
| 14:33:57 | atmark | then I tried `openstack --os-compute-api-version 2.87 server rescue --image e0b042a3-ad6a-4f6e-ad5d-f6f150b5287e a11283a0-55bb-46cb-b0be-b8c3df79a6cf` | |
| 14:34:19 | atmark | it returns ` cannot be rescued: Cannot rescue a volume-backed instance HTTP 404` | |
| 14:34:30 | atmark | s/404/400 | |
| 14:34:49 | sean-k-mooney | you need an newer api micorversion and you need to set other image properites on the image | |
| 14:49:27 | opendevreview | Merged openstack/nova master: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924427 | |
| 14:53:54 | sean-k-mooney | atmark: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#maximum-in-ussuri-and-victoria | |
| 14:54:04 | sean-k-mooney | you need micro version 2.87 | |
| 14:55:10 | sean-k-mooney | and you should set hw_rescue_bus i recommend hw_rescue_bus=usb | |
| 14:56:22 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2024.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924514 | |
| 14:57:54 | sean-k-mooney | atmark: these are the tempest test in case that helps https://github.com/openstack/tempest/blob/1af21705c53bc9911ea467eaeee2bc12489a43ed/tempest/api/compute/servers/test_server_rescue.py#L259-L310 | |
| 15:05:21 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2023.2: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924517 | |
| 15:08:37 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2023.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924518 | |
| 15:32:58 | gibi | stephenfin: what would be the way to get an openstack sdk client not based on the service user in the config but based on the user token from a nova context? I tried naivly https://paste.opendev.org/show/bnuA5WLIZvlJGR4cSHUt/ but it fails with `Jul 19 15:29:53 gibi-devstack-aio-jammy devstack@n-api.service[1814794]: ERROR nova.api.openstack.wsgi openstack.exceptions.NotSupported: The | |
| 15:33:04 | gibi | shared-file-system service for :None exists but does not have any supported versions. | |
| 15:33:07 | gibi | ` | |
| 15:34:45 | gibi | for the nova - manila integration we try to use the sdk to talk to manila but we want to make sure that the user who request attaching a manial share from nova has the creds to read the share in the manila api | |
| 16:57:47 | atmark | sean-k-mooney: Thanks. Got it working now. I set hw_rescue_device=cdrom and hw_rescue_bus=ide for ISO. Upstream doc https://docs.openstack.org/nova/latest/user/rescue.html#stable-device-instance-rescue says `or` instead of `and` https://docs.openstack.org/nova/latest/user/rescue.html#stable-device-instance-rescue | |
| #openstack-nova - 2024-07-20 | |||
| 00:49:29 | gmann | stephenfin: +2 now, sorry I missed to check that you added empty schema only for those GET Action APIs which is not actually query schema but can be counted as action schema only so all good https://review.opendev.org/c/openstack/nova/+/915738/6/nova/api/openstack/compute/schemas/hosts.py | |
| 04:40:27 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 04:40:28 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 04:40:28 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 04:40:29 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 04:40:29 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 04:56:39 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 04:56:40 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 04:56:40 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 04:56:41 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 04:56:41 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 08:50:34 | opendevreview | Merged openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 09:13:13 | opendevreview | Merged openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| #openstack-nova - 2024-07-21 | |||
| 13:14:22 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 13:20:52 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 13:26:52 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 15:05:36 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| #openstack-nova - 2024-07-22 | |||
| 05:25:49 | frickler | stephenfin: how much of https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22 do you think can we get still done this cycle? I have little hope for cinder, but placement could be fine if someone reviews https://review.opendev.org/c/openstack/placement/+/919569 | |
| 07:16:41 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Correct info about volume-backed server rebuild https://review.opendev.org/c/openstack/nova/+/924574 | |
| 09:40:33 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2024.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924514 | |
| 10:53:48 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for host aggregate APIs https://review.opendev.org/c/openstack/nova/+/924585 | |
| 10:53:48 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for host aggregate actions API https://review.opendev.org/c/openstack/nova/+/924584 | |
| 10:53:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for versions APIs https://review.opendev.org/c/openstack/nova/+/924587 | |
| 10:53:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for extensions API https://review.opendev.org/c/openstack/nova/+/924586 | |
| 10:53:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for port interface APIs https://review.opendev.org/c/openstack/nova/+/924589 | |
| 10:53:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | |
| 10:53:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for availability zone APIs https://review.opendev.org/c/openstack/nova/+/924590 | |
| 10:53:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for console auth token APIs https://review.opendev.org/c/openstack/nova/+/924592 | |
| 10:53:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for bare metal node APIs https://review.opendev.org/c/openstack/nova/+/924591 | |
| 10:53:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor extra specs APIs https://review.opendev.org/c/openstack/nova/+/924594 | |
| 10:53:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor access API https://review.opendev.org/c/openstack/nova/+/924593 | |
| 10:53:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP pool APIs https://review.opendev.org/c/openstack/nova/+/924596 | |
| 10:53:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavors APIs https://review.opendev.org/c/openstack/nova/+/924595 | |
| 10:53:58 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 10:53:58 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add contributor docs for response body validation https://review.opendev.org/c/openstack/nova/+/924597 | |
| 10:54:00 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: tests: Temporarily ignore missing schemas for removed APIs https://review.opendev.org/c/openstack/nova/+/924599 | |
| 10:54:49 | stephenfin | frickler: There's nothing technically challenging about any of them: it's simply a case of getting them reviewed and merged | |
| 10:57:21 | stephenfin | gibi: You could try calling 'connect_as' on the connection object? That doesn't accept a context object (yet) so you'll need to set the various fields manually but | |
| 10:58:21 | stephenfin | ...that shouldn't be too difficult to do | |
| 10:58:38 | stephenfin | gtema might have a better idea on #openstack-sdks | |
| 11:04:14 | sean-k-mooney | frickler: stephenfin i reviewd the placement patch only the first of the two is requried and can merge this cycle | |