| *** __ministry is now known as Guest1303 | 01:06 | |
| *** __ministry is now known as Guest1320 | 04:10 | |
| opendevreview | Nisha Brahmankar proposed openstack/nova master: Retain port binding profile after live migration https://review.opendev.org/c/openstack/nova/+/936338 | 04:28 |
|---|---|---|
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | 10:24 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | 10:24 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | 10: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 | 10:24 |
| 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 |
| 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 |
| 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:17 |
| zigo | (of course, I'm not expecting a reply before you wake up ... :P ) | 11:23 |
| 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:28 |
| opendevreview | Merged openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | 13:37 |
| 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 | 13:50 |
| ishanwar | Hi team, | 14:24 |
| 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:25 |
| ishanwar | Kindly suggest @Sean-k-mooney and @bauzas ? | 14:26 |
| sean-k-mooney | the nova codebase is quite large btu we have alot of contibutor adn refence docs decibing the archetecture and subssytems | 14:33 |
| sean-k-mooney | ishanwar: the main entry point to our docs is https://docs.openstack.org/nova/latest/ | 14:34 |
| sean-k-mooney | i woudl first start with https://docs.openstack.org/nova/latest/admin/architecture.html | 14:34 |
| 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:35 |
| sean-k-mooney | beyond that there are a number of youtube videos form previosu summits on specific topics | 14:36 |
| 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:38 |
| 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 |
| sean-k-mooney | speicifclly shelve_offoloaded | 14:39 |
| 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:40 |
| 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:51 |
| 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 | 14:56 |
| 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 |
| 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 |
| sean-k-mooney | *vm | 15:09 |
| 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:18 |
| zigo | FYI, that's form Antelope to Caracal. | 15:19 |
| zigo | from | 15:19 |
| sean-k-mooney | populate_instance_compute_id should be using data form within the cell db | 15:28 |
| sean-k-mooney | so if your doign this on a local backup ectra then it should not cause issues that im aware of | 15:29 |
| sean-k-mooney | this is what its actully doing https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L1357-L1377 | 15:30 |
| 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:31 |
| sean-k-mooney | so that shoudl be relitivly quick although if you have a lot of instnaces then it will take a while | 15:32 |
| zigo | That cluster has ~8k VM, though many deleted, I guess. | 15:33 |
| zigo | It's rather fast, what I'm worried about, is the None thingy, plus compute nodes that we decomitionned. | 15:33 |
| sean-k-mooney | we are reading deleted isntanfe too | 15:33 |
| zigo | Yeah, and I don't understand why. | 15:34 |
| sean-k-mooney | its backpopulating deleted but not archived isntances | 15:34 |
| sean-k-mooney | to keep it consitent | 15:34 |
| zigo | Oh, maybe I should just archive them then ... | 15:34 |
| sean-k-mooney | so if the compute node is delete then this will skip them | 15:35 |
| sean-k-mooney | but if your rungin this in a loop it will never fully complete | 15:35 |
| zigo | Hum... I do have the ::nova::cron::archive_deleted_rows thingy ... :P | 15:37 |
| 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 |
| sean-k-mooney | but there are bugs that can lead to instanfce beign in a wereid state sometimes | 15:37 |
| 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 |
| sean-k-mooney | bauzas: proably needs to be done on the other delvierables too | 16:09 |
| bauzas | hmmm, no, we already have blueprints for 2025.1 | 16:10 |
| bauzas | but indeed, maybe the active release is still 2024.2 | 16:10 |
| bauzas | sean-k-mooney: okay, now fixed | 16:13 |
| bauzas | https://launchpad.net/nova/+series | 16:13 |
| sean-k-mooney | cool | 16:15 |
| sean-k-mooney | im just looking at fixing watchers trackign and im using novas config as a refence | 16:15 |
| sean-k-mooney | so i just noticed that when comparing | 16:15 |
| opendevreview | Merged openstack/nova master: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/909122 | 16:40 |
| sean-k-mooney | woot now to backport that... | 16:56 |
| 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 | 16:56 |
| opendevreview | ribaudr proposed openstack/nova master: Refactor response schemas for share API https://review.opendev.org/c/openstack/nova/+/936656 | 18:26 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: WIP: Test mitigations for candidate explosion https://review.opendev.org/c/openstack/placement/+/936658 | 19:23 |
| opendevreview | Merged openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | 21:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!