| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-30 | |||
| 14:56:03 | sean-k-mooney | although that may have been epoxy | |
| 14:56:16 | sean-k-mooney | it depend on what is in ubuntu 24.04 | |
| 14:56:20 | zigo | Ahh... Please everyone: stop that bullshit again. Upstream does not get to choose what's in the distro. :( | |
| 14:56:34 | sean-k-mooney | we get to choose what we supprot | |
| 14:56:47 | sean-k-mooney | and if we dont have it aviabel in our ci to test with we cant claim supprot | |
| 14:56:50 | zigo | Debian Trixie has RabbitMQ 4.0.5, and it has Epoxy. | |
| 14:57:00 | sean-k-mooney | ack | |
| 14:57:17 | zigo | Just like I have to make Epoxy run on Python 3.13: no choice... :/ | |
| 14:57:18 | sean-k-mooney | i think its goign to be imporant to get trixy in our ci as soon as practical | |
| 14:57:30 | sean-k-mooney | we offically only supprot debian 12 cuyrrently | |
| 14:57:36 | zigo | I need to fix this rabbitmq config first, so I can continue my tests. | |
| 14:57:38 | sean-k-mooney | trixy woudl be add next cycle | |
| 14:58:03 | zigo | Trixie is out in a few weeks... :) | |
| 14:58:05 | sean-k-mooney | ack, i think must of those feature are aviabel in the older release of rabbit too | |
| 14:58:21 | sean-k-mooney | so we can try and use them in our existing jobs provide they work | |
| 14:58:22 | zigo | It's always the same story anyways, so I'm used to it. :) | |
| 16:36:09 | gmaan | sean-k-mooney: you might have noticed but pinging just in case you missed that manager role series is ready for review https://review.opendev.org/q/topic:%22bp/policy-manager-role-default%22+status:open | |
| 16:42:26 | sean-k-mooney | gmaan: its proably in my gerrit folder but this week and last i have not been on top of burning that down to 0 each day | |
| 16:42:43 | sean-k-mooney | so tanks for the ping ill take a look at the spec today | |
| 16:42:54 | gmaan | sean-k-mooney: thanks | |
| 16:54:50 | opendevreview | Takashi Kajinami proposed openstack/nova master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/nova/+/953751 | |
| 16:56:41 | sean-k-mooney | tkajinam: so ^ is not nessiarly a bad thign but it need to have a tracker, at the miniume this shoudl be a tech debt but IMO. | |
| 16:57:40 | sean-k-mooney | the other thing to condier is while we supprot sqlachemy 2.0 | |
| 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 | |