| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-13 | |||
| 19:04:44 | bauzas | ok | |
| 19:05:27 | bauzas | OK then let's go for specifying them | |
| 19:05:31 | bauzas | thanks | |
| 19:06:01 | sean-k-mooney | the other reason they have to be seperated is | |
| 19:06:32 | sean-k-mooney | with the new weither if the new config option is not defied we effectivly weigh each option as 1 | |
| 19:06:35 | sean-k-mooney | i.e. evenly | |
| 19:07:03 | sean-k-mooney | but when its defiend we woudl only apply the weigts to the specified values and ignore the rest | |
| 19:07:30 | sean-k-mooney | where as for the metric weigher the cofnig option is required for it to work | |
| 19:07:40 | sean-k-mooney | instead of being optional | |
| 19:07:46 | sean-k-mooney | so we dont want to share it | |
| 19:26:01 | mikal | Morning | |
| 19:29:09 | mikal | It seems like its been fairly quiet on the spice patches overnight, apart from gmann approving the temptest test (thanks gmann!). Are there any actions items for me right now that I should know about? | |
| 19:29:20 | bauzas | sean-k-mooney: fine, then I'll just provide *one* additional config option, by default we'll stick to evenly weigh the props | |
| 19:30:25 | sean-k-mooney | +1 | |
| 19:30:50 | sean-k-mooney | mikal: i honestly didnt have time to look at the current state | |
| 19:31:03 | sean-k-mooney | but you got it to the point of workign and testign in a ci job | |
| 19:31:19 | sean-k-mooney | so i think we could move forward with merging some/all of these? | |
| 19:31:24 | mikal | sean-k-mooney: yeah, no worries. | |
| 19:31:41 | sean-k-mooney | alos we should move forward iwth os-trait patch if not allready done? | |
| 19:31:46 | mikal | sean-k-mooney: yes. The tempest test passes in the ovs-plug job, assuming that the entire rube goldberg machine of changes is landed | |
| 19:31:52 | gmann | mikal: pinged frickler for devstack change 2nd review review in qa channel | |
| 19:32:14 | mikal | sean-k-mooney: yeah, once the spice-direct series is landed the os-traits change will be the main thing blocking me. It blocks the sound / usb changes. | |
| 19:32:18 | mikal | gmann: thank you. | |
| 19:32:46 | sean-k-mooney | we need a second +2 on https://review.opendev.org/c/openstack/os-traits/+/940418 and we need to a release before the non clinet lib freeze next week | |
| 19:32:54 | mikal | sean-k-mooney: I am a little unsure if we want to compress your zuul changes into a single change with the spice API changes or keep it like this. I definitely don't want to take credit for your work by stealing your changes. | |
| 19:33:21 | sean-k-mooney | steal all the things | |
| 19:33:27 | mikal | sean-k-mooney: that freeze is a week before feature freeze, or at the same time? | |
| 19:33:29 | sean-k-mooney | im fine with squashing them if you want | |
| 19:33:38 | gmann | bauzas: not sure if you see my message early this week. but let me know what I need to do to get this spec merged? I added it in Nova meeting agenda for spec exception approval (but i would not be able to attend meeting due to time conflict). I started code change in parallel if that help, manager role is almost ready and will work on service role change https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 19:33:49 | sean-k-mooney | mikal: the non client freeze is a week before feature freeze | |
| 19:33:52 | mikal | sean-k-mooney: I'd prefer to not squash them unless there's a really solid reason why we should. I think the history is useful. | |
| 19:34:16 | sean-k-mooney | then i think we can jsut try and move them forward as is | |
| 19:34:22 | bauzas | gmann: this cycle, I unfortunately don't have a lot of time upstream | |
| 19:34:26 | bauzas | so I saw your ping | |
| 19:35:01 | sean-k-mooney | gmann: is the code written and tested? | |
| 19:35:13 | sean-k-mooney | becasue otherwise i think this need to go to next cycle | |
| 19:35:14 | bauzas | yeah, that's my question | |
| 19:35:43 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/941056 | |
| 19:35:48 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/941347 | |
| 19:35:49 | gmann | sean-k-mooney: I have started that, manager role changes is almost ready (fixing some unit and tempest test) | |
| 19:35:51 | sean-k-mooney | are teh nova patches | |
| 19:36:02 | gmann | service role change I will work soon | |
| 19:36:03 | gmann | https://review.opendev.org/q/topic:%22bp/policy-service-and-manager-role-default%22 | |
| 19:36:18 | gmann | ^^ this is all series where service role changes i need to rebase/fix | |
| 19:36:37 | sean-k-mooney | oh there are others ok | |
| 19:37:21 | sean-k-mooney | we kind of have a lot of patches that need review so i kind of feel like this should move to next cycle | |
| 19:37:34 | sean-k-mooney | on the ohter had i dont really want to be the one to punt it so.. | |
| 19:39:06 | gmann | well, thing is I am not sure if I can spend more time on upstream activities in next cycle (depend on various factor) so I thought of getting it done in nova this cycle | |
| 19:40:50 | gmann | anyways I added it in meeting agenda and will be ok based on how team decide. unfortunately I will not be able to attend meeting but you knows the current situation on this so I will check meeting log/decision. | |
| 19:44:37 | bauzas | gmann: noted, we'll discuss this on next tues | |
| 19:44:45 | gmann | bauzas: thanks | |
| 19:45:58 | bauzas | sean-k-mooney: one last question about the weigher, do we agree on the fact that props not set on the pre-prop setting won't be weighed ? | |
| 19:46:21 | bauzas | we'll just skip them if so | |
| 19:46:58 | bauzas | ie. default of non-set props will be 0.0 | |
| 19:49:08 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 19:49:08 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 19:56:48 | sean-k-mooney | bauzas: i think that make the most sense | |
| 19:57:14 | sean-k-mooney | bauzas: so sure you can use wights.get(name, 0.0) | |
| 19:57:28 | bauzas | that's litterally what I wrote :D | |
| 19:57:43 | bauzas | sending the WIP for the FUP in a very sec | |
| 19:58:07 | mikal | sean-k-mooney: I need to wander off and get ready for work. I'll check in again this evening but y'all look busy on other things today which is fine. | |
| 20:00:16 | sean-k-mooney | im on pto tomorroow but ill try and make time on monday | |
| 20:00:49 | mikal | No worries | |
| 20:22:21 | birbilakos_ | hello | |
| 20:22:26 | birbilakos_ | After some network changes in my 2023.2 deployment which broke network connectivity to one controller node (and reverting that back) I get bombarded by these type of errors: https://paste.opendev.org/show/b8QRG6bxWiKh9CLM79oR/ | |
| 20:22:40 | birbilakos_ | i also see that several nova actions either timeout or are very slow to go through. any ideas how to recover this? | |
| 20:23:26 | birbilakos_ | rabbitmq containers seem to be running fine, also checked the management web interface of rabbitmq and didn't notice anything strange | |
| 20:24:21 | sean-k-mooney | that just looks like issue with rabbit mq | |
| 20:24:53 | sean-k-mooney | it does not look like its really nova specific beyond tat | |
| 20:25:42 | sean-k-mooney | the messagign timeout may be because you restart nova conductor for example | |
| 20:25:47 | sean-k-mooney | and the requestrs were lost | |
| 20:25:57 | sean-k-mooney | or restart rabbitmq | |
| 20:28:25 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/941347 | |
| 20:46:47 | opendevreview | Sylvain Bauza proposed openstack/nova master: per-property ImageMetaPropsWeigher https://review.opendev.org/c/openstack/nova/+/941601 | |
| 20:46:47 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a new ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/940642 | |
| 20:52:06 | birbilakos_ | thanks sean | |
| 20:52:32 | birbilakos_ | how do i proceed then? currently nova is reporting missing messages and also fails to do actions | |
| 20:53:22 | birbilakos_ | e.g. instance creation and deletion is a hit and miss... sometimes it works, sometimes it doesnt | |
| #openstack-nova - 2025-02-14 | |||
| 01:56:58 | opendevreview | Antony Messerli proposed openstack/nova master: docs: Corrects a typo on a placement audit command https://review.opendev.org/c/openstack/nova/+/922850 | |
| 02:14:28 | tkajinam | Hi I wonder if https://review.opendev.org/c/openstack/nova/+/931582 and https://review.opendev.org/c/openstack/nova/+/936287 can get 2nd +2 so that we can get rid of netifaces ? | |
| 02:15:46 | tkajinam | gibi bauzas melwitt ^^^ in case you have time for reviewing these | |
| 04:06:43 | MengyangZhang[m] | https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-01-15-14.00.log.html#l-57:~:text=%3Cjbernard%3E%20thanks-,14%3A15%3A34,-%3Cjbernard%3E%20%23topic). In addition, neutron has already supported similar features of live updating network QoS to ports and per-project control of QoS policies. | |
| 04:06:43 | MengyangZhang[m] | Hi folks, I was wondering if the community is interested in this feature: Enhanced Granularity and Live Application of Volume QoS (https://blueprints.launchpad.net/nova/+spec/enhanced-granularity-and-live-application-of-volume-qos). Since this would be a cross-project efforts, we first proposed it to Cinder folks and received positive feedback. (See discussion here: | |
| 04:07:52 | MengyangZhang[m] | https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-01-15-14.00.log.html#l-57:~:text=%3Cjbernard%3E%20thanks-,14%3A15%3A34,-%3Cjbernard%3E%20%23topic). So we would like to hear the feedback from the nova community. | |
| 04:07:52 | MengyangZhang[m] | * Hi folks, I was wondering if the community is interested in this feature: Enhanced Granularity and Live Application of Volume QoS (https://blueprints.launchpad.net/nova/+spec/enhanced-granularity-and-live-application-of-volume-qos). Since this would be a cross-project efforts, we first proposed it to Cinder folks and received positive feedback. (See discussion here: | |
| 08:02:40 | gibi | tkajinam: approved the series. Thanks for improving nova's health | |
| 08:19:23 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 08:19:23 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 09:22:35 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP:Reproducer for cold migration on shared storage https://review.opendev.org/c/openstack/nova/+/940304 | |
| 10:36:53 | tkajinam | gibi, thanks ! | |
| 10:38:34 | tkajinam | ugh it seems we have to fix unit tests now... | |
| 10:47:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| 10:48:04 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add unit test coverage of get_machine_ips https://review.opendev.org/c/openstack/nova/+/936287 | |
| 11:12:59 | gibi | tkajinam: thanks for fixing it | |
| 14:53:28 | MengyangZhang[m] | do we have a nova meeting today? | |
| 15:21:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/941673 | |
| 15:23:25 | dansmith | sean-k-mooney: can you +2 this so we can get this fix landed? https://review.opendev.org/c/openstack/nova/+/940604/8 | |
| 15:24:37 | bauzas | dansmith: sean said he was on PTO but I can take a look | |
| 15:29:50 | dansmith | oh right | |
| 15:30:22 | dansmith | bauzas: well, it was specifically because he had comments already, but I think they're addressed and just wanted that ack so we could move on | |