| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-11-27 | |||
| 20:20:39 | sean-k-mooney | basically we have https://github.com/openstack/nova/blob/master/nova/objects/service.py#L316 | |
| 20:20:56 | sean-k-mooney | which tells use when the last heartbet was recived | |
| 20:21:15 | sean-k-mooney | and we jus tneed to compare that to the interval and allow missed heatbeats | |
| 20:22:07 | sean-k-mooney | or actully we can jsut check https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.service_down_time driectly | |
| 20:22:33 | s3rj1k | I see report_count in there | |
| 20:23:51 | sean-k-mooney | up = datetime.now() - CONF.service_down_time < last_seen_up | |
| 20:24:06 | sean-k-mooney | something like ^ would work | |
| 20:24:29 | s3rj1k | report_count is per-service right? | |
| 20:24:37 | sean-k-mooney | yes | |
| 20:25:07 | s3rj1k | so if thing goes down and up it gets nulled and count get started over? | |
| 20:25:25 | sean-k-mooney | i dont recall of the top of my head | |
| 20:25:44 | sean-k-mooney | in this context we can ignore i ti think | |
| 20:26:03 | sean-k-mooney | anyway i better finish for today | |
| 20:26:13 | s3rj1k | so basically there are solution how to make that selection more clever | |
| 20:26:27 | sean-k-mooney | yep if needed | |
| 20:26:27 | s3rj1k | thanks for you input | |
| 20:26:47 | sean-k-mooney | if we were trying to distibute load | |
| 20:26:54 | sean-k-mooney | we might want a more clever solution | |
| 20:27:08 | sean-k-mooney | but since we are trying to make sure only one runs i dont think that isneeded | |
| 20:27:44 | sean-k-mooney | we might want a minium uptime before the leader can change | |
| 20:27:58 | sean-k-mooney | i.e. double the perodic interval or somthing like that | |
| 20:28:25 | sean-k-mooney | just to prevent issues on start up | |
| 20:28:47 | s3rj1k | even of we miss, next run will fix that | |
| 20:28:49 | sean-k-mooney | or perhaps double the service downtime interval | |
| 20:29:02 | s3rj1k | s/of/if/ | |
| 20:29:24 | sean-k-mooney | but yes i think we can do this in a lock free way and it will self heal | |
| 20:29:39 | sean-k-mooney | actully droping now o/ | |
| 20:30:00 | s3rj1k | have a good night | |
| #openstack-nova - 2024-11-28 | |||
| 04:28:46 | opendevreview | Nisha Brahmankar proposed openstack/nova master: Retain port binding profile after live migration https://review.opendev.org/c/openstack/nova/+/936338 | |
| 10:24:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 10:24:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 10:24:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 10:24:24 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 11:17:53 | zigo | dansmith: Hi there! Got a question for you. I'm testing db upgrades, and populate_instance_compute_id is taking like forever. | |
| 11:17:53 | zigo | I wonder, what is this about? Your commit header says it's for "deleted, shelved, never scheduled or otherwise on a compute host that isn't updatating it", I don't really get it. | |
| 11:17:53 | zigo | Also, I have many warnings about: "Unable to migrate instance because host None with node None not found: nova.exception.ComputeHostNotFound: Compute host None could not be found.". Should I be worried? | |
| 11:23:27 | zigo | (of course, I'm not expecting a reply before you wake up ... :P ) | |
| 13:28:53 | opendevreview | Merged openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 13:37:57 | opendevreview | Merged openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 13:50:52 | opendevreview | Merged openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 14:24:29 | ishanwar | Hi team, | |
| 14:25:16 | ishanwar | I wanted to ask what is a good entry point, to start understanding the nova implementation in the codebase. I could not find much resources about the implementations on the openstack website | |
| 14:26:17 | ishanwar | Kindly suggest @Sean-k-mooney and @bauzas ? | |
| 14:33:56 | sean-k-mooney | the nova codebase is quite large btu we have alot of contibutor adn refence docs decibing the archetecture and subssytems | |
| 14:34:29 | sean-k-mooney | ishanwar: the main entry point to our docs is https://docs.openstack.org/nova/latest/ | |
| 14:34:41 | sean-k-mooney | i woudl first start with https://docs.openstack.org/nova/latest/admin/architecture.html | |
| 14:35:26 | sean-k-mooney | oru contibutor docs are here https://docs.openstack.org/nova/latest/contributor/index.html and tecnial refence docs are here https://docs.openstack.org/nova/latest/reference/index.html | |
| 14:36:47 | sean-k-mooney | beyond that there are a number of youtube videos form previosu summits on specific topics | |
| 14:38:25 | sean-k-mooney | zigo: dansmith is on PTO until the end of the year more or less so expect a delay on any pings :) | |
| 14:39:33 | sean-k-mooney | zigo: if the instance has host and node none and is not deleted or in cell0 then that is only valid if its shelved | |
| 14:39:50 | sean-k-mooney | speicifclly shelve_offoloaded | |
| 14:40:26 | sean-k-mooney | if you have an isntnace that is not shelve_offloaded and has host or node None then you have db currption | |
| 14:51:01 | ishanwar | sean-k-mooney: thanks for the docs, I had a look at the architecture, I was wondering if there is any specific command to start the apis for testing locally in the codebase | |
| 14:56:12 | ishanwar | Also, @Sean-k-mooney @bauzas, it would be great if you could point me to any low hanging bugs to solve like good-first-issues in the codebase | |
| 15:09:08 | sean-k-mooney | ishanwar: you can techinally do that but its not hwo developemnt is normally done. nova is a distirbute system with several external depencies | |
| 15:09:41 | sean-k-mooney | so the normall approch is to use devstack https://github.com/openstack/devstack to create a devleopment enviornment in a vam | |
| 15:09:43 | sean-k-mooney | *vm | |
| 15:18:38 | zigo | sean-k-mooney: My db is comming from another cluster, just testing db upgrades and online_data_migrations. Could that be the reason? | |
| 15:19:37 | zigo | FYI, that's form Antelope to Caracal. | |
| 15:19:42 | zigo | from | |
| 15:28:50 | sean-k-mooney | populate_instance_compute_id should be using data form within the cell db | |
| 15:29:31 | sean-k-mooney | so if your doign this on a local backup ectra then it should not cause issues that im aware of | |
| 15:30:47 | sean-k-mooney | this is what its actully doing https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L1357-L1377 | |
| 15:31:47 | sean-k-mooney | for each instance we get the node and host values and lookup the compute node id then set that in teh instance compute_id column | |
| 15:32:32 | sean-k-mooney | so that shoudl be relitivly quick although if you have a lot of instnaces then it will take a while | |
| 15:33:16 | zigo | That cluster has ~8k VM, though many deleted, I guess. | |
| 15:33:42 | zigo | It's rather fast, what I'm worried about, is the None thingy, plus compute nodes that we decomitionned. | |
| 15:33:44 | sean-k-mooney | we are reading deleted isntanfe too | |
| 15:34:01 | zigo | Yeah, and I don't understand why. | |
| 15:34:25 | sean-k-mooney | its backpopulating deleted but not archived isntances | |
| 15:34:33 | sean-k-mooney | to keep it consitent | |
| 15:34:44 | zigo | Oh, maybe I should just archive them then ... | |
| 15:35:30 | sean-k-mooney | so if the compute node is delete then this will skip them | |
| 15:35:46 | sean-k-mooney | but if your rungin this in a loop it will never fully complete | |
| 15:37:12 | zigo | Hum... I do have the ::nova::cron::archive_deleted_rows thingy ... :P | |
| 15:37:14 | sean-k-mooney | in theory if you are deleting a compute node shoudl shoudl also be deleting any instance that are on it or move them to a diffent host first | |
| 15:37:41 | sean-k-mooney | but there are bugs that can lead to instanfce beign in a wereid state sometimes | |
| 16:09:38 | sean-k-mooney | bauzas: as an fyi you have not created the 2025.1 series in https://launchpad.net/nova yet or set it as the active developemnt release, it still says 2024.2 | |
| 16:09:50 | sean-k-mooney | bauzas: proably needs to be done on the other delvierables too | |
| 16:10:05 | bauzas | hmmm, no, we already have blueprints for 2025.1 | |
| 16:10:20 | bauzas | but indeed, maybe the active release is still 2024.2 | |
| 16:13:19 | bauzas | sean-k-mooney: okay, now fixed | |
| 16:13:25 | bauzas | https://launchpad.net/nova/+series | |
| 16:15:23 | sean-k-mooney | cool | |
| 16:15:45 | sean-k-mooney | im just looking at fixing watchers trackign and im using novas config as a refence | |
| 16:15:54 | sean-k-mooney | so i just noticed that when comparing | |
| 16:40:25 | opendevreview | Merged openstack/nova master: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/909122 | |
| 16:56:10 | sean-k-mooney | woot now to backport that... | |
| 16:56:30 | opendevreview | sean mooney proposed openstack/nova stable/2024.2: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/936645 | |
| 18:26:12 | opendevreview | ribaudr proposed openstack/nova master: Refactor response schemas for share API https://review.opendev.org/c/openstack/nova/+/936656 | |
| 19:23:16 | opendevreview | Balazs Gibizer proposed openstack/placement master: WIP: Test mitigations for candidate explosion https://review.opendev.org/c/openstack/placement/+/936658 | |
| 21:41:45 | opendevreview | Merged openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| #openstack-nova - 2024-11-29 | |||
| 04:22:04 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 04:22:04 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 04:22:05 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 04:22:05 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 04:22:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 10:35:16 | r-taketn | Hello. We are planning to contribute a new feature for building Arm-Confidential Computing Architecture(CCA)-enabled instances. Could you please review the following e-mail? (sorry, mail title is bad) https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/244U52T3HINVWUFSMPMA45A67BUPAQGK/ I'm not sure it is acceptable for OpenStack because Arm-CCA enabled hardware is not yet available to our knowledge. | |
| 10:35:47 | r-taketn | Your feedback on the acceptance of this contribution would be greatly appreciated. | |
| 14:47:02 | opendevreview | Merged openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |