| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-30 | |||
| 16:57:48 | sean-k-mooney | we stilll supprot 1.4.13+ | |
| 16:57:54 | sean-k-mooney | https://github.com/openstack/nova/blob/master/requirements.txt#L6 | |
| 16:58:09 | sean-k-mooney | so to do that we woudl need to bump our min version to 2.0 | |
| 16:58:20 | sean-k-mooney | we coudl do that but that will need a release note | |
| 16:58:26 | sean-k-mooney | to capture the upgrade impact | |
| 16:58:51 | sean-k-mooney | oh actully no | |
| 16:59:01 | sean-k-mooney | https://github.com/sqlalchemy/sqlalchemy/commit/450f5c0d6519a439f40 was in 1.4.0 | |
| 16:59:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 16:59:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC executor selectively using threading https://review.opendev.org/c/openstack/nova/+/953815 | |
| 16:59:30 | sean-k-mooney | so so it does nto force min version bump above our current min | |
| 17:07:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 17:24:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 17:24:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 17:24:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 17:24:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 17:24:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 17:24:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 17:24:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 17:24:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 17:24:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 17:24:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 17:24:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 17:24:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 17:24:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 17:27:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 17:28:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/953338 | |
| 17:28:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/953339 | |
| 17:56:00 | sean-k-mooney | as an fyi the nova gate should become unblocked once https://review.opendev.org/c/openstack/neutron/+/953768 lands | |
| 18:10:37 | sean-k-mooney | gmaan: im holding +w to make sure you agree with that my interperation of host info alings with yours | |
| 18:11:02 | gmaan | sean-k-mooney: checking | |
| 18:14:11 | sean-k-mooney | proably shoudl have pinged with the link to the comment https://review.opendev.org/c/openstack/nova-specs/+/953722/comment/a91a7f7a_c8837144/ | |
| 18:16:40 | gmaan | sean-k-mooney: replied, yes, host related fields will be 'None' for non-admin | |
| 18:17:36 | sean-k-mooney | ok if we are treatign includign the query parma as a error for non admins | |
| 18:17:43 | sean-k-mooney | then that ok | |
| 18:17:48 | sean-k-mooney | and ack on the rest | |
| 18:17:57 | sean-k-mooney | None is fin i guess or empty string | |
| 18:18:02 | gmaan | yeah | |
| 18:18:05 | sean-k-mooney | which ever is allowed by the schema | |
| 18:18:14 | gmaan | for request, you mean to error ? | |
| 18:18:40 | gmaan | yeah, there will be no change in request, whatever it is allowed now will be same | |
| 18:18:44 | sean-k-mooney | well i dont want to allwo them to probel by sarting a resize and then guessign the source node ectra | |
| 18:18:58 | sean-k-mooney | so ok i think im happy we are on the same page | |
| 18:19:20 | sean-k-mooney | im ok to move to the impleation review so ill add +w | |
| 18:20:32 | gmaan | sean-k-mooney: this is the implementation will looks like with more granular policies https://review.opendev.org/c/openstack/nova/+/953063/7/nova/api/openstack/compute/migrations.py | |
| 18:21:21 | gmaan | and next patch in that series change the policy to know what all things will be open/allowed for manager https://review.opendev.org/c/openstack/nova/+/941347 | |
| 18:23:25 | sean-k-mooney | ya that looks reasonable | |
| 18:24:15 | sean-k-mooney | i left a comment on the asymetry with dest_host and the duplciatvie info that it contains with dest_compute | |
| 18:24:30 | sean-k-mooney | but that is just an observation not a request to change anythign | |
| 18:24:53 | gmaan | ack | |
| 18:25:24 | sean-k-mooney | we are a littel inconssitent with how we fileter on the query side and how we respsent this in the responce | |
| 18:26:13 | sean-k-mooney | like host in the query can match the source or dest host but source_compute only mages the soucee but you cant query by only dest compute or node | |
| 18:26:32 | sean-k-mooney | i dont feeel like it worth haveing a microversion to fix that right now | |
| 18:27:10 | sean-k-mooney | after we finish the openapi serise and we have clollected a bunch of similar odities it might make sense to do another cleanup microversion to normalise things | |
| 18:28:56 | sean-k-mooney | gmaan: you have 2 serise this cycle right service role and manger role. do you have a prefered order for those in terms of review | |
| 18:29:29 | gmaan | host in query is either one, just checked if we do same in code or not and yes it match either one https://github.com/openstack/nova/blob/43d57ae63d1ecda24d8707b4750d404daadc980f/nova/db/main/api.py#L3425 | |
| 18:29:56 | gmaan | sean-k-mooney: yeah, I am targeting to finish manager role first | |
| 18:30:02 | sean-k-mooney | ack | |
| 18:30:19 | sean-k-mooney | hopeflyy we can do both but i think doing one then the other is more likely to work out | |
| 18:30:32 | sean-k-mooney | ill try and do a pass over the manager role then this week | |
| 18:30:49 | gmaan | yeah, once I am done with manager role I am going to resume my service role also | |
| 18:30:50 | sean-k-mooney | ping me if i havnt by firday | |
| 18:30:56 | gmaan | cool, ack | |
| 18:30:59 | opendevreview | Merged openstack/nova-specs master: Allow list migrations policy to manager role https://review.opendev.org/c/openstack/nova-specs/+/953722 | |
| 19:40:14 | sean-k-mooney | Uggla: i left some comments on your memfd spec https://review.opendev.org/c/openstack/nova-specs/+/951689 i dont see any other reviwe on that so beign realsitc that proably wont land this cycle | |
| 19:40:25 | sean-k-mooney | it could but its definetly at risk | |
| 19:40:54 | sean-k-mooney | we might want to punt it premtivly so we can discss the upgrade impact with less time pressure | |
| 19:41:24 | sean-k-mooney | there are some details i would like to see captured in the spec that are missing and im not sure we should rush it but im +1 on the overall propsoal | |
| #openstack-nova - 2025-07-01 | |||
| 08:18:47 | ralonsoh | ok folks, nova-ovs-hybrid-plug is green again: https://zuul.opendev.org/t/openstack/builds?job_name=nova-ovs-hybrid-plug&skip=0 | |
| 08:18:59 | ralonsoh | sorry for the inconveniences | |
| 08:32:26 | gibi | ralonsoh: thanks for the quick fix1 | |
| 09:13:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 09:13:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 09:13:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 09:13:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 09:13:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 09:13:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 09:13:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 09:13:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 09:13:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 09:13:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 09:13:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 09:13:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 09:13:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 09:13:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 09:13:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 09:17:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 10:04:34 | mikal | Herm, now the ceph CI test appears broken? | |
| 10:04:34 | mikal | Some packages could not be installed. This may mean that you have | |
| 10:04:34 | mikal | requested an impossible situation or if you are using the unstable | |
| 10:04:34 | mikal | distribution that some required packages have not yet been created | |
| 10:04:34 | mikal | or been moved out of Incoming. | |
| 10:04:36 | mikal | The following information may help to resolve the situation: | |
| 10:04:36 | mikal | The following packages have unmet dependencies: | |
| 10:04:39 | mikal | qemu-block-extra : Depends: librbd1 (>= 19.2.1-0ubuntu0.24.04.1) but 19.2.0-0ubuntu0.24.04.2 is to be installed | |
| 10:04:39 | mikal | E: Unable to correct problems, you have held broken packages. | |
| 10:04:42 | mikal | + functions-common:apt_get:1 : exit_trap | |
| 10:04:42 | mikal | + ./stack.sh:exit_trap:543 : local r=100 | |
| 10:04:43 | mikal | ++ ./stack.sh:exit_trap:544 : jobs -p | |
| 10:04:43 | mikal | + ./stack.sh:exit_trap:544 : jobs= | |
| 10:04:46 | mikal | + ./stack.sh:exit_trap:547 : [[ -n '' ]] | |