Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-10-07
16:28:22 elodilles (and that is more 'readable' for every participants)
16:28:44 elodilles (at least that is a 'common' timezone everyone understands)
16:28:46 gibi which is fine by me. But you are talking about changing during dayligh saving and that does not make sense to me as UTC is not affected
16:30:01 gibi e.g. if I vote 16:00 UTC, then it is always 16:00 UTC regardless of where am I on earth and what part of the year we are
16:30:26 gibi do you suggest we will update the 16:00 UTC to some other time when we move to daylight saving?
16:30:41 Uggla gibi, after the daylight our meeting which is usually at 18h00 CET, will move to 17h00 CET and will conflict with one internal.
16:30:55 Uggla "do you suggest we will update the 16:00 UTC to some other time when we move to daylight saving?" yes absolutelly
16:31:17 gibi why? :)
16:31:42 Uggla We will probably have to move from 16:00 UTC to 17:00 UTC.
16:32:02 gibi do you want to keep the timing constant in EU?
16:32:20 gibi that will create 4 changes of time in US
16:32:33 gibi probably not something the US folks will like
16:32:49 gibi if we stick to UTC then both EU and US has 2 changes per year to adapt to
16:33:35 Uggla s/want/wants/
16:34:51 Uggla But having internal ones in UTC would help.
16:34:56 bauzas honestly, there is no solution here, let's continue with UTC and just signal when we have conflicts after DST
16:35:07 gibi I think sticking to UTC is the least confusing for the community. Moving twice a year to keep the EU time constant is an optimization for EU but a complication for U
16:35:31 gibi I have to drop now but I will read back...
16:35:34 bauzas that's why it takes time to look at the framedate one given I need to look at both of the times
16:36:19 Uggla bauzas, yep
16:36:51 Uggla Anyway, something else to discuss ?
16:37:37 kgube I just wanted to ask, is there a cross-project session planned with Cinder at the PTG?
16:38:06 Uggla kgube, I have not discussed with cinder yep, but probably yes.
16:38:23 kgube alright, thanks!
16:39:06 Uggla If you want to have a cross discussion with cinder, please write that in our doc, and I'll arrange with Cinder folks to find a timeslot.
16:39:47 kgube The NetApp/NFS online resize topic might fit well there, I have added a comment
16:40:14 Uggla ok, please also add your name around?
16:40:42 Uggla Anything else ?
16:42:31 bauzas -
16:43:23 Uggla ok so thanks for joining this meeting, closing now.
16:43:34 Uggla #endmeeting
16:43:34 opendevmeet Meeting ended Tue Oct 7 16:43:34 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
16:43:34 opendevmeet Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-07-16.00.html
16:43:34 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-07-16.00.txt
16:43:34 opendevmeet Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-07-16.00.log.html
16:43:41 elodilles thanks o/
16:44:06 fwiesel Thanks!
16:44:08 Uggla elodilles, you are welcome
16:46:09 bauzas Uggla: thanks
16:54:18 bauzas dansmith: found why we didn't found the fill_metadata issue with the unittests
16:54:26 dansmith melwitt: question/sanity-check for you
16:54:34 dansmith melwitt: in https://review.opendev.org/c/openstack/nova/+/942502
16:54:41 bauzas but I wonder how to fix the coverage
16:55:23 bauzas anyway, just looking at the time, I need to leave now :(
16:55:23 dansmith bauzas: I still haven't heard a problem in the code I agree with, but .. go on :)
16:55:28 dansmith ack
16:55:33 bauzas very quickly
16:55:41 bauzas inst.system_metadata = utils.instance_sys_meta(
16:55:41 bauzas updated[inst.uuid])
16:55:47 melwitt dansmith: looking
16:55:51 bauzas that's what I fixed
16:56:04 bauzas because we were having a dict of dict
16:56:23 dansmith bauzas: oh the dict thing, right, but I also want to know why it passed our QE...
16:56:33 dansmith bauzas: anyway, put up a full patch and I'll look
16:57:01 bauzas but given the instances you createed in https://github.com/openstack/nova/commit/420050cf33dd58bc6b608540fc8f32b2f65ef274 don't have any metadata recorded in DB, then it always returns an empty dict
16:57:28 bauzas dansmith: ack, will upload a patch and try to write some comment + some pastebin
16:57:43 bauzas I need to leave for tai-chi but hopefully be back around 7pm UTC
16:59:13 bauzas long story short, I need to find a way to persist some metadata for the instances you create in test_fill_metadata() so that when calling db.instances_fill_metadata() we get those values for adding them into the object metadata itself
16:59:42 bauzas my brain is fried and I have to run errand
17:00:55 dansmith okay
17:01:46 dansmith melwitt: I think you're going to say it's expected behavior and it looks like the resize patch has a lot of conversion logic..
17:02:28 dansmith I'm not sure if I think it's problematic behavior, but it's very much not what I expected
17:02:58 melwitt dansmith: yes I am typing that. I think the main issue is that in the 'user' patch the resize is not looking at image_hw_ in sysmeta but it should be (ignoring all of these patches ...) so I'm currently confused about that
17:03:07 dansmith because I don't know how to reconcile when the flavor and image disagree (especially in this case where I haven't set anything on my image)
17:04:02 dansmith melwitt: if it's not looking at image_hw_tpm_security then I don't know why it let me schedule to a user host when the flavor requests a host host
17:04:26 melwitt dansmith: yeah. that's what I was chatting with sean about last week is, is this going to be a problem that we have overloaded the image_hw_tpm_secret_security sysmeta key when it did not necessarily come from any image
17:05:38 melwitt dansmith: it's because there is no trait filtering logic yet at that point in the series. it knows nothing about 'host' at this point so it's treating it as if it doesn't exist. it is still 'user' despite the fact that it lets you set the flavor as 'host'
17:06:23 dansmith melwitt: eh? isn't this the trait filtering? https://review.opendev.org/c/openstack/nova/+/942502/19/nova/scheduler/request_filter.py
17:06:26 melwitt dansmith: so I guess maybe what I need to do is at least make it know about the "will-be-valid" values early on to reject them until support is introduced
17:06:37 dansmith oh,
17:06:47 dansmith you mean because it lets any security it doesn't know about through
17:06:52 melwitt yes
17:07:06 dansmith ack, so that should probably have an else: False or something
17:07:22 dansmith (if false means fail, I don't remember)
17:07:51 melwitt it's confusing because resize is allowed today for TPM, so 'user' is just current behavior so it's always going to work until support for the others like 'host' and 'deployment' are added
17:08:25 dansmith well,
17:08:28 melwitt I agree there is a problem here, just thinking of what the best way to deal with it
17:08:54 dansmith I would think it should be security=flavor.get('security', 'user') and not "security is something I don't recognize, so MEH WFM
17:09:26 dansmith because the extra spec lets you set =host right now, which is fine, but we should not just ignore that I think
17:09:41 melwitt it is, but if you specify 'host', it will set that. it just doesn't know what that is yet
17:10:28 dansmith well, right.. I just mean dropping out the bottom for unsupported-as-of-yet values
17:10:31 melwitt yeah I agree, that's what I was trying to say a minute ago is that I think I need to add awareness of what the valid values are so far and reject anything that is not
17:10:45 dansmith and even when they're all there, I think not allowing a security that it doesn't know about (regardless of how that got in there) is a better plan
17:10:53 melwitt yeah, agree
17:10:54 dansmith yeah
17:11:30 melwitt I'll update the patch to do that
17:12:03 dansmith cool, just commented on the missing else
17:12:10 melwitt thanks
19:02:40 opendevreview sean mooney proposed openstack/nova-specs master: Add ResourceProviderWeigher scheduler spec https://review.opendev.org/c/openstack/nova-specs/+/951222
19:19:56 opendevreview sean mooney proposed openstack/nova-specs master: Add ResourceProviderWeigher scheduler spec https://review.opendev.org/c/openstack/nova-specs/+/951222
20:36:18 opendevreview Merged openstack/nova master: [nova-tox-py312-threading]Ignore failing tests https://review.opendev.org/c/openstack/nova/+/961781
21:36:03 opendevreview John Garbutt proposed openstack/nova master: Fix ironic rebuild with API >=2.93 https://review.opendev.org/c/openstack/nova/+/963298
#openstack-nova - 2025-10-08
02:09:26 opendevreview Sam Morrison proposed openstack/nova master: Filter out deleted instances when looking for build timouts https://review.opendev.org/c/openstack/nova/+/880125
06:13:40 opendevreview melanie witt proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196
06:13:40 opendevreview melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197
06:13:41 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502
06:13:41 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795
06:13:42 opendevreview melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628
06:13:42 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483
06:13:43 opendevreview melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051

Earlier   Later