| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-13 | |||
| 17:35:20 | sean-k-mooney | the defuatl qdisk was change about 3-4 year ago it used to be pfifo by default | |
| 17:35:41 | sean-k-mooney | ovs also change to not ignoring/removing the qdisc on port added to it by defualt | |
| 17:36:19 | gibi | gmaan: thanks! I added the upgrade reno | |
| 17:37:50 | sean-k-mooney | vsaienko: https://wiki.archlinux.org/title/Advanced_traffic_control#CoDel_and_Fair_Queueing_CoDel | |
| 17:38:09 | sean-k-mooney | that is some of the history on this changes it hapspen din systemd 217 | |
| 17:41:45 | opendevreview | Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682 | |
| 17:44:45 | gmaan | anyone face/know the issue of not seeing 'comoute.<hostname>' queue in devstack env ? I am listing all queues (sudo rabiitmqctl list_queues) and it show all queues except compute service queues (oslo.message create three queues per compute service 1. 'compute' 2. 'compute.<hostname>' 3. compute fanout with uuids) | |
| 17:45:32 | gmaan | it list all other queues for example sch, conductor, cinder queues | |
| 17:47:07 | opendevreview | Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682 | |
| 17:48:10 | nicolairuckel | sean-k-mooney, I found a few more places with code that wasn't necessary anymore | |
| 17:48:29 | nicolairuckel | I think it should be a lot cleaner now | |
| 17:48:37 | sean-k-mooney | yep im +1 on your patch after skimming it | |
| 17:48:51 | sean-k-mooney | ill upgrade to +2 after i have time to test it and read over it again | |
| 17:48:59 | nicolairuckel | perfect :) | |
| 17:50:51 | sean-k-mooney | gmaan: so it shoudl be "compute.{CONF.host}" but if the queue is not there it normally is a netwrokign or auth issue, i belive it can also happen if we exit because fo say the missing compute node id file | |
| 17:51:05 | sean-k-mooney | i.e. if the agent stop runing and the queue gets deleted | |
| 17:51:26 | gmaan | compute service is up, instance are booting fine so no issue on that side | |
| 17:51:53 | sean-k-mooney | are you lookig at the correct exchange? | |
| 17:52:06 | gmaan | if you have devstack running up, can you try if you see the comoute queues? | |
| 17:52:20 | gmaan | yes, nove and even i checked all exhange | |
| 17:52:34 | sean-k-mooney | sure i have one | |
| 17:52:49 | gmaan | before i boot new VM and install devstack i just want to cofirm if something in my machine | |
| 17:52:51 | gmaan | thanks | |
| 17:53:21 | sean-k-mooney | i have one on a host internally i can add you ssh key to it and you can ssh in and take a look if you want | |
| 17:54:16 | gmaan | its fine, if you can just check if any queue name with 'compute*'or paste the outphut of sudo rabiitmqctl list_queues | |
| 17:55:50 | sean-k-mooney | https://termbin.com/rzms | |
| 17:56:35 | sean-k-mooney | im not seeing any | |
| 17:56:36 | gmaan | yeah, same issue, no compute queue there | |
| 17:57:27 | sean-k-mooney | ah | |
| 17:57:37 | sean-k-mooney | so we use vhosts in devstack | |
| 17:57:42 | gmaan | instance booting successfully make sure no issue in queue creation but somehow it is not listed in rabbitmq command | |
| 17:57:43 | sean-k-mooney | and the compute are in nova_cell1 | |
| 17:58:08 | gmaan | yeah vhosts we use | |
| 17:58:22 | sean-k-mooney | https://termbin.com/01tb | |
| 17:58:37 | sean-k-mooney | that the output form "rabbitmqctl list_queues --vhost nova_cell1 | nc termbin.com 9999" | |
| 17:58:53 | sean-k-mooney | compute.hibernal01 | |
| 17:58:59 | sean-k-mooney | is the compute queue | |
| 17:59:54 | gmaan | ah yeah, it is listed now. thanks sean-k-mooney ++ | |
| 18:00:13 | sean-k-mooney | no worries. | |
| #openstack-nova - 2025-11-14 | |||
| 02:01:59 | opendevreview | melanie witt proposed openstack/nova master: Fix usage count when user-scoped quota is set https://review.opendev.org/c/openstack/nova/+/967148 | |
| 02:01:59 | opendevreview | melanie witt proposed openstack/nova master: Reproducer for bug 2131272 https://review.opendev.org/c/openstack/nova/+/967147 | |
| 05:10:25 | opendevreview | Rajesh Tailor proposed openstack/nova master: Make GroupAffinityViolation inherit from RescheduledException https://review.opendev.org/c/openstack/nova/+/967156 | |
| 10:38:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 10:38:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP:Move eventlet specific libvirt event handling https://review.opendev.org/c/openstack/nova/+/967178 | |
| 14:49:28 | opendevreview | Dan Smith proposed openstack/nova-specs master: WIP: Spec for async volume attach https://review.opendev.org/c/openstack/nova-specs/+/958900 | |
| 14:51:28 | opendevreview | Dan Smith proposed openstack/nova-specs master: WIP: Spec for async volume attach https://review.opendev.org/c/openstack/nova-specs/+/958900 | |
| 15:09:21 | dansmith | bauzas: I can haz? https://review.opendev.org/c/openstack/nova/+/966447 | |
| 15:09:49 | bauzas | +Wd | |
| 15:10:00 | dansmith | thanks | |
| 15:37:37 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 15:37:38 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 15:37:38 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 15:37:39 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 15:39:49 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 17:27:21 | opendevreview | Merged openstack/nova master: Test nova-next with >1 parallel migrations https://review.opendev.org/c/openstack/nova/+/966447 | |
| 19:53:40 | melwitt | dansmith: sorry if you already saw but just in case you didn't, I replied to your latest comments on the vtpm stack | |
| 22:20:57 | opendevreview | Merged openstack/nova master: add functional repoducer for bug 2048837 https://review.opendev.org/c/openstack/nova/+/919979 | |
| 22:26:17 | dansmith | melwitt: ack, sorry I haven't even looked to see if you had | |
| 22:27:18 | melwitt | oh ok | |
| #openstack-nova - 2025-11-15 | |||
| 10:54:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP:Move eventlet specific libvirt event handling https://review.opendev.org/c/openstack/nova/+/967178 | |
| #openstack-nova - 2025-11-16 | |||
| 05:13:19 | opendevreview | Ghanshyam proposed openstack/nova master: WIP: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| #openstack-nova - 2025-11-17 | |||
| 08:16:44 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 08:45:13 | gibi | dansmith: thanks for the review on https://review.opendev.org/c/openstack/nova/+/966446/1#message-d56cbd036e036b8a6962e6023c1c4213fbadf7ab I replied inline | |
| 10:33:05 | opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 10:34:51 | opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 10:54:47 | opendevreview | sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322 | |
| 11:29:54 | LarsErikP | tobias-urdin: do you know anything about the status of merging this? https://review.opendev.org/c/openstack/nova/+/916409 | |
| 13:27:41 | opendevreview | sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322 | |
| 13:28:38 | sean-k-mooney | gibi: good catch i missed that i reordered it ^ shoudl adress your comments | |
| 13:35:32 | opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 13:37:23 | opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 13:38:59 | gibi | sean-k-mooney: thanks. +2 | |
| 13:47:20 | sean-k-mooney | long term ther is a more generic version fo the FairLockGuard concept that i may eventually upstream to oslo. if i do write that ill eventurally refactort nova to use it but i want thsi simpler version in nova so we can backport the fix. gibi the more generic version would take to sets of lock names to aquire as readers and writers, this would allow me to create a more | |
| 13:47:22 | sean-k-mooney | generic form of the @serial decorator that we use for tempest so that we can declare annotate test that need exclusive access to a subset of resouces with a request for the wright lock for that resouce, the use case is watcher senario tests have some tests that exclsilve operatate on voluem and ohter that exlucsitlvy operato on instnaces and some that work on both | |
| 13:47:42 | opendevreview | Rajesh Tailor proposed openstack/nova master: Clear image_ref for boot-from-volume instances https://review.opendev.org/c/openstack/nova/+/967369 | |
| 13:47:56 | sean-k-mooney | today we run the warcher tempest test with concurrancy 1 so that they dont interfer | |
| 13:48:20 | sean-k-mooney | evecutnally i think we can do better with an enhanced version fo this but that is over kill for what nova needs today | |
| 14:27:51 | gibi | sean-k-mooney: could you check this small hacking rule https://review.opendev.org/c/openstack/nova/+/958408 ? | |
| 14:33:56 | sean-k-mooney | sure | |
| 14:36:07 | sean-k-mooney | ok so we are trying to block import or direct use via fully qulaifed name of eventlets semaphore timeout or event | |
| 14:37:01 | sean-k-mooney | :) and it found one incorrect usage in a test | |
| 14:37:55 | sean-k-mooney | i need to check but we have already converted LibvirtDriver._handle_conn_event to actully use ThreadingEventWithResult() right? | |
| 14:39:30 | gibi | https://github.com/openstack/nova/blob/8a993d583ffedb608a67acac9fce0c3d065e2dd2/nova/compute/manager.py#L268 | |
| 14:39:33 | gibi | yepp | |
| 14:41:45 | sean-k-mooney | _InstanceEvents = ty.Dict[ty.Tuple[str, str], ThreadingEventWithResult] ah ok then yes i think this all looks fine to me | |
| 14:42:22 | sean-k-mooney | gibi: i didnt have a chance to look last week but you mentioend the first copule of patches in the eventlet serise are ready to review | |
| 14:43:27 | sean-k-mooney | ah found the base https://review.opendev.org/c/openstack/nova/+/966443/1 looks like stephen has alrady taken a look at that one at least | |
| 14:44:11 | sean-k-mooney | i have maybe 30-40 mins before my next meeting ill see if i can take a look at one or two of them | |
| 15:08:46 | gibi | thanks | |
| 15:27:41 | dansmith | gibi: I replied... I'm concerned we have such differing views of what init_host() is for | |
| 15:28:04 | dansmith | but, I +W'd as I don't think I'm going to convince you and a fixed bug is better of course | |
| 15:28:53 | dansmith | I would offer to tighten up the definition of init_host() but I'd do so in a way incompatible with your definition, so probably better for me to not | |
| 15:34:27 | gibi | dansmith: thanks. "Initialize anything that is necessary for the driver to function" for me means do not use the driver until this is done. I understand that technically we could use parts of the driver already after host.initialize() like querying the uuids. But what you suggest would actually require the better part of the current libvirt/driver.init_host() to be move to __init__ not just | |
| 15:34:33 | gibi | host.initialize() like the libvirt version check. I really don't want to query anything about libvirt before we are sure we are talking about a libvirt we support | |
| 15:36:32 | gibi | and really changing the meaning of init_host during a bugfix seems dangerous | |
| 15:36:36 | dansmith | except we call host.initialize() before we check the libvirt version :) | |
| 15:36:59 | gibi | we do but we don't return from init_host until we check the libvirt version | |
| 15:37:00 | dansmith | well, as I noted, I don't think the current meaning agrees with what you think it is | |
| 15:37:30 | gibi | the current libvirt implementation agrees with the meaning I hold here and that is our biggest and most used impl | |
| 15:37:53 | dansmith | hence me asking you to change the actual definitions to match, for the future | |