| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-30 | |||
| 14:41:26 | zigo | sean-k-mooney: I'd prefer if it was virtio-cd *always*, yes. | |
| 14:41:50 | sean-k-mooney | so if we were to make that change it would only take effect for new vms | |
| 14:42:15 | zigo | Yeah, I get that point. | |
| 14:42:23 | sean-k-mooney | when we made teh other changes we added code to recored the currnt model in the db | |
| 14:42:32 | sean-k-mooney | we did that so we could chagne the default in the future | |
| 14:43:31 | zigo | BTW. what is the use of rabbit_qos_prefetch_count? I've set use_queue_manager and rabbit_stream_fanout, but now nova-compute complains I haven't set rabbit_qos_prefetch_count and it is to its default of zero. What is a good value for this then ? | |
| 14:43:38 | sean-k-mooney | we bassically pretent the glance image properlty is set and add it in isntance sysmtem metaadat table as if it came form teh glance image when the vm is first created | |
| 14:44:07 | sean-k-mooney | zigo: no idea i dont think we sue that in nvoa today so its proably directly coming form oslo.messaging | |
| 14:44:55 | sean-k-mooney | zigo: we directly include the oslo.messaging cofnig options in our config because all the parsieng fo those exctra are done by oslo not nova | |
| 14:45:07 | zigo | Yeah, like everyone else does! :) | |
| 14:45:26 | sean-k-mooney | well excpet swift... | |
| 14:45:52 | zigo | Which is a real pain. | |
| 14:46:14 | zigo | We tried switching to json log to send all to Elasticsearch, it worked great ... except for swift ! :/ | |
| 14:46:22 | sean-k-mooney | so i am not sure what that does but you might get a better answer for the oslo folks | |
| 14:46:43 | sean-k-mooney | zigo: i both love and hate that that exists | |
| 14:46:51 | sean-k-mooney | its really nice for suecase like that | |
| 14:47:12 | sean-k-mooney | but when our custoerm enabel that and i have to ready the raw logs in json format it sucks | |
| 14:47:44 | sean-k-mooney | i wish there was a tool to take teh json format and print them normally in human reablae forrmat in oslo | |
| 14:47:46 | zigo | It was commited together with the fanout and stream stuff by Arnaud Morin <arnaud.morin@ovhcloud.com> | |
| 14:47:54 | zigo | https://review.opendev.org/c/openstack/oslo.messaging/+/890825 | |
| 14:48:00 | sean-k-mooney | ya so nova does not supprot any of that offically | |
| 14:48:01 | zigo | Though absolutely no doc about it ... :/ | |
| 14:48:28 | sean-k-mooney | im vagly aware that stream supprot has been added but no work has been doen to use nova with it | |
| 14:48:41 | sean-k-mooney | it might work but no one has done the work to test it | |
| 14:48:55 | zigo | https://www.rabbitmq.com/docs/consumer-prefetch | |
| 14:48:58 | sean-k-mooney | there has been no cross project work to support it that im aware off | |
| 14:49:46 | sean-k-mooney | ok so it sound like itx a limit of how many unacked message a clinet can have at a time | |
| 14:51:53 | sean-k-mooney | https://bugs.launchpad.net/oslo.messaging/+bug/2031497 really shoudl have had an oslo spec IMO | |
| 14:52:12 | sean-k-mooney | maybe they had one for stream supprot in general | |
| 14:52:24 | zigo | Agreed. | |
| 14:52:52 | sean-k-mooney | zigo: i hope your not plannig to use that in debian 13 by default | |
| 14:53:04 | sean-k-mooney | the stream supprot i mean | |
| 14:53:49 | sean-k-mooney | its disabled by default in oslo https://docs.openstack.org/nova/latest/configuration/config.html#oslo_messaging_rabbit.rabbit_stream_fanout | |
| 14:53:58 | sean-k-mooney | and we do not ahve any testing with it at all for nova | |
| 14:54:26 | sean-k-mooney | we also do not test with rabbit_quorum_queue set to ture either | |
| 14:54:40 | sean-k-mooney | it might work but again no work has been doen in nova to offically supprot it | |
| 14:55:09 | sean-k-mooney | we could close that gap by enablign testing in nova-next | |
| 14:55:23 | zigo | Well, it's time to move to it: rabbitmq does *not* support HA queue starting at version 4.0.x | |
| 14:55:28 | zigo | So no choice for operators. | |
| 14:55:30 | sean-k-mooney | but we would also need to update the docs before offically supproting it | |
| 14:55:56 | sean-k-mooney | im aware although flamingo is the first realee we are usign 4.0.0 i think | |
| 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 | |