| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-08 | |||
| 15:06:51 | sean-k-mooney | wwe also had requests=[InstancePCIRequest,InstancePCIRequest]) | |
| 15:07:06 | sean-k-mooney | so it does not recursivly print the nested objects | |
| 15:08:08 | dansmith | yeah, which it could do... | |
| 15:08:48 | sean-k-mooney | so i guess i wasnt expecting much push back as this to me is a significant readbility gain | |
| 15:09:17 | sean-k-mooney | we could adress it other wasy but this to mean was a clean an consitent way to make it more readable and support debuggin form logs | |
| 15:09:28 | dansmith | it's a significant readability regression to me :) | |
| 15:09:28 | sean-k-mooney | for the specific objects where the current logging is lacking | |
| 15:11:38 | sean-k-mooney | i guess you would prefer if i modified the list object ot print recurrsivly instead | |
| 15:11:43 | sean-k-mooney | keeping the same format | |
| 15:12:02 | sean-k-mooney | do you know if there is an atribute we would need to apply to do that | |
| 15:12:44 | dansmith | I mean, the reason it doesn't recurse now is just because the dump could end up being super massive and so the decision not to recurse was intentional | |
| 15:13:00 | dansmith | that was at a time where we had many fewer object types and less nesting I'm sure | |
| 15:13:10 | sean-k-mooney | but that often means i cant debug custoemr issues form the logs | |
| 15:13:11 | dansmith | but things like a list of instances would be a massive dump in the logs | |
| 15:13:25 | dansmith | sure, I know | |
| 15:13:33 | sean-k-mooney | i think we shoudl recuse for debug mode | |
| 15:13:40 | sean-k-mooney | but perhaps not for anythign else | |
| 15:13:42 | dansmith | I'm not saying don't make it recursive, I'm just saying it's pretty heavy | |
| 15:13:46 | sean-k-mooney | do you know if we have dont that before? | |
| 15:13:53 | dansmith | not that I know of | |
| 15:14:07 | sean-k-mooney | ok ill pause the patch there then and see if that is possibel | |
| 15:14:15 | sean-k-mooney | if thats an approch you and bauzas prefer | |
| 15:14:39 | sean-k-mooney | dansmith: for context i have othen patch in this dumping in the past | |
| 15:14:49 | bauzas | I'm just testing what we get | |
| 15:14:51 | sean-k-mooney | when doing testing locally to repoduce an issue | |
| 15:15:09 | bauzas | so we just jsondump the primitive | |
| 15:15:19 | bauzas | which means for unset attrs that we won't show them | |
| 15:16:29 | bauzas | so yeah I agree with dansmith, we should show all the fields regardless, as it will be easier for ops to know whether we provided the fields or not | |
| 15:31:20 | kashyap | tobias-urdin: Hi, for later - I think this removal of "tunnelled migration" should be merged this cycle (also wrote it on the review): https://review.opendev.org/c/openstack/nova/+/879021 | |
| 15:31:39 | kashyap | tobias-urdin: If you're respinning it for other reasons, please add the deprecation patch URL to the commit message :) | |
| 18:34:47 | tobias-urdin | kashyap: ack, thanks! | |
| #openstack-nova - 2024-04-09 | |||
| 06:18:02 | zigo | My first Caracal VM pings ! :) | |
| 06:33:07 | zigo | Tempest is running ... :P | |
| 06:42:44 | opendevreview | Haidong Pang proposed openstack/nova master: Catch MessagingTimeout during evacuation https://review.opendev.org/c/openstack/nova/+/915309 | |
| 06:47:34 | opendevreview | Haidong Pang proposed openstack/nova master: Catch MessagingTimeout during evacuation https://review.opendev.org/c/openstack/nova/+/915309 | |
| 06:59:01 | opendevreview | Haidong Pang proposed openstack/nova master: Catch MessagingTimeout during evacuation https://review.opendev.org/c/openstack/nova/+/915309 | |
| 07:13:40 | mikal | sean-k-mooney: hey Sean... Unfortunately 11pm my time is well past my old man bed time so I missed your message. I'm mostly good, just pottering away on things I think are interesting because I'm not good at being fun. | |
| 07:50:44 | zigo | bauzas: There's no operator feedback session in the Nova PTG schedule this year? | |
| 07:50:58 | zigo | We're in the "public cloud SIG session", and we wonder when we could all join. | |
| 07:51:15 | bauzas | zigo: nope, the TC didn't discussed about it afaik | |
| 07:51:23 | bauzas | I can join if people wany | |
| 07:51:35 | zigo | It's the end of the session ... :/ | |
| 07:52:05 | zigo | We have another session on Thurdsay 14:00 UTC. | |
| 12:52:14 | bauzas | nova PTG sessions start at the top of the hour, ie. in 8 mins :) | |
| 12:55:53 | SvenKieske | I'll join, I guess mostly as a spectator for now :) | |
| 13:43:39 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix trivial doc issues https://review.opendev.org/c/openstack/nova/+/878779 | |
| 14:30:24 | kashyap | gibi: When your topic starts, can you ping here, please? :) | |
| 14:32:45 | bauzas | kashyap: you can look at https://ptg.opendev.org/ptg.html | |
| 14:33:29 | kashyap | Thanks! | |
| 14:43:31 | bauzas | fwiesel : we start discussing "nova on top of k8s" | |
| 14:44:15 | sean-k-mooney | ill joing the ptg soon fyi | |
| 14:44:33 | sean-k-mooney | i need to grab a drink before i do | |
| 15:02:08 | bauzas | melwitt: are you around ? | |
| 15:02:17 | bauzas | could you join us for unified limits ? | |
| 15:04:57 | melwitt | bauzas: yeah joining in a few | |
| 15:24:45 | opendevreview | Rafael Weingartner proposed openstack/nova master: Add logs for PCI passthrough device selection https://review.opendev.org/c/openstack/nova/+/915361 | |
| 15:30:57 | bauzas | fwiesel: want to discuss "Compute node (instead host) for all scheduling operations and APIs" just after the unified limits discussion ? | |
| 15:35:16 | bauzas | nevermind, we'll discuss this tomorrow | |
| 15:42:59 | dansmith | bauzas: is that your keyboard being so loud? | |
| 15:43:10 | bauzas | not mine | |
| 15:43:17 | dansmith | okay | |
| 15:50:18 | melwitt | kashyap: thanks for taking notes on the etherpad about unified limits :) | |
| 15:51:27 | kashyap | melwitt: Happy to be a scribe :) Now I'm out of steam, so I stopped taking any notes on the current topics. (I've done one for an earlier, longer topic too) | |
| 16:02:57 | bauzas | haleyb: so, what about a nova-neutron session ? do you have specific bits in your agenda to share with us ? | |
| 16:40:17 | haleyb | bauzas: i think the only thing on my list was a continuation from last ptg as we never worked on the multiple port bindings items, that was all i had | |
| 16:41:18 | bauzas | haleyb: okay, do you think you need a nova audience for this then ? | |
| 16:44:43 | haleyb | bauzas: unclear, it's more of a reminder to pick up the work. it would be good to have a blueprint/rfe on what extensions need to be mandatory on the neutron side | |
| 16:45:11 | bauzas | haleyb: then, let's do a quick wrap on it on tomorrow if you want | |
| 16:48:08 | haleyb | bauzas: sure. i had created a time slot from 15:30-16:00 to talk about it if that works | |
| 16:48:21 | haleyb | or we can move things around a little | |
| 17:04:41 | bauzas | haleyb: we may have a conflicting session for discussing with the glance folks at the same time actually | |
| 17:04:58 | bauzas | haleyb: any possibility to move that 30-min chat ? | |
| 17:05:39 | haleyb | bauzas: can move to 15:00, does that work? | |
| 17:12:06 | bauzas | haleyb: yup, thanks ! | |
| 17:12:15 | bauzas | I just added it to our etherpad | |
| 17:12:28 | bauzas | haleyb: we'll then go to your room | |
| 17:12:40 | bauzas | wfy ? | |
| 17:15:16 | haleyb | yes, works for me | |
| 17:15:18 | haleyb | thanks | |
| 17:16:51 | bauzas | haleyb: thanks | |
| 17:23:16 | opendevreview | ribaudr proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | |
| 17:23:16 | opendevreview | ribaudr proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (object part) https://review.opendev.org/c/openstack/nova/+/903706 | |
| 17:23:17 | opendevreview | ribaudr proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 18:06:27 | opendevreview | Rafael Weingartner proposed openstack/nova master: Add logs for PCI passthrough device selection https://review.opendev.org/c/openstack/nova/+/915361 | |
| 18:06:40 | opendevreview | Merged openstack/nova stable/2023.1: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/913224 | |
| #openstack-nova - 2024-04-10 | |||
| 02:38:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use native DictOpt for disk_cachemode https://review.opendev.org/c/openstack/nova/+/915390 | |
| 02:47:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Validate valid cache mode at config input layer https://review.opendev.org/c/openstack/nova/+/915391 | |
| 09:51:21 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: [ironic] Partition & use cache for list_instance* https://review.opendev.org/c/openstack/nova/+/906155 | |
| 11:03:45 | opendevreview | Merged openstack/nova stable/2024.1: Fix nova-manage image_property show unexpected keyword https://review.opendev.org/c/openstack/nova/+/914671 | |
| 12:38:12 | opendevreview | Merged openstack/nova stable/zed: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/908692 | |
| 12:52:46 | bauzas | fwiesel: tkajinam: hey, which one of you both would want to start to discuss about your topics ? | |
| 12:53:03 | bauzas | Extending memory encryption support or Nova Metadata Optimization ? | |
| 12:57:01 | fwiesel | bauzas: No strong preference. It would just be good to be finished with the metadata optimization before the neutron cross project call. | |
| 12:57:37 | tkajinam | the other 3 topis are mine it seems so I'm fine with metadata discussion going first | |
| 12:58:00 | bauzas | fwiesel: tkajinam: cool, we'll start with the metadata, thanks | |
| 13:50:54 | opendevreview | Merged openstack/nova stable/2023.1: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/913225 | |
| 14:11:02 | opendevreview | Merged openstack/nova stable/2023.2: Fix nova-manage image_property show unexpected keyword https://review.opendev.org/c/openstack/nova/+/914765 | |
| 14:33:12 | zigo | Can nova use something else than genisoimage? If not, it'd be a good idea to make it support xorriso ... | |
| 14:33:12 | zigo | I still have this bug open, unaddress since 2021: https://bugs.debian.org/982241 | |
| 14:34:29 | tkajinam | zigo, ROD switched to xorriso. probably you can follow the same patch. There is a patch to update the default but has been stall. if other distros follow that switch then we can move the defualt switch forward | |