Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2026-07-13
16:42:22 gibi antia: feel free to push even a half working stuff so that we can start giving guidance
16:42:23 antia One problem I faced was when rebasing the patches and uploading to Gerrit
16:43:10 antia Is it normal to have the changes be pushed to the other series as well?
16:43:37 sean-k-mooney it depends on your local git tree
16:43:54 gibi antia: you can place your stuff on other people's changes just be careful not to change their commits
16:44:18 sean-k-mooney when you do that its best to do "git review -R"
16:44:18 antia so as long as the commits remain unchanged it should be fine?
16:44:47 gibi antia: yes
16:44:56 sean-k-mooney yes although normlalyits best to work off master when possible unless your cordianting with the other person
16:45:11 sean-k-mooney we stack changes if they have a natual depency
16:45:13 gibi sean-k-mooney: here we coordinating as the two series are in conflict
16:45:20 sean-k-mooney ack
16:45:26 gibi touching the same enum, etc
16:45:47 antia then it looks correctly, Gerrit just gives a scary warning :)
16:45:59 gibi antia: what warning? :)
16:46:24 tkajinam I guess it warns multiple commits are being proposed
16:47:00 antia yes exactly
16:47:02 tkajinam nothing (except for topic, which we can fix easily) should be changed unless you edit the existing commits
16:47:54 tkajinam or we could even revert the series back to the previous patch sets
16:48:52 gibi antia: double check that the hash the last commit from tkajinam in your local tree is the same as the latest from gerrit. I.e. to see if tkajinam pushed a new version of his patch in the meantime
16:49:16 antia Yes will do that before uploading
16:49:29 gibi if you are unsure then just drop the git log --oneline output to some 18:42 < antia> Hi! Sorry for being late. Nothing mayor, had quite the busy downstream week but made some progress on parts of the series
16:49:33 gibi sorry
16:49:40 gibi wrong copy-paste buffer
16:50:01 gibi so git log --oneline output to https://paste.openstack.org/ and we can help
16:50:29 gibi anyhow running out of time and tkajinam had another topic
16:50:34 gibi tkajinam: go ahead
16:51:01 tkajinam https://review.opendev.org/c/openstack/nova/+/995620
16:51:36 tkajinam I happened to find that in review queue and was involved in its review. it seems we likely need a new image prop.
16:52:01 gibi is this a new feature?
16:52:14 tkajinam I'm wondering which process should we follow now.(spec or specless bp)
16:52:29 gibi we are late for either at this point
16:52:30 tkajinam the new image prop to enable a new hyper-v enlightenment feature
16:52:54 tkajinam yeah I thought so, but just wanted to hear opinion from others for double-check
16:53:30 tkajinam ok I'll leave such comment in review. thanks (I'm unsure if the onwer is in irc)
16:53:55 gibi I think it might qualify for specless in Indri but we have to wait
16:54:18 gibi anything else before we close the meeting?
16:54:29 tkajinam nothing else from me !
16:55:01 opendevmeet Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.log.html
16:55:01 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.txt
16:55:01 opendevmeet Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.html
16:55:01 opendevmeet Meeting ended Mon Jul 13 16:55:01 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
16:55:01 gibi #endmeeting
16:55:05 gibi thanks folks
16:55:11 gibi antia: we can continue
16:55:19 elodilles thanks too o/
16:55:21 tkajinam thanks !
16:59:10 antia gibi: here is gitlog and gerrit warning https://paste.openstack.org/show/b3hGRYq7qtxpuY2DRUpk/
16:59:27 sean-k-mooney gibi: tkajinam so speaking of conflict im inclinet too say since i need to rebase my change anyway lets just land https://review.opendev.org/c/openstack/nova/+/994342
16:59:42 sean-k-mooney so im going to go review that properly now with that in mind
16:59:58 sean-k-mooney well
17:00:26 sean-k-mooney im still not sure about the new is_supported_mem_encryption_model fucntion and if its ok to remvoe that again
17:01:23 tkajinam checking
17:02:25 sean-k-mooney antia: so if you did "git review -R" that disables rebasing in git review
17:02:35 gibi eb455009ac this is an old version of tkajinam's series
17:02:43 sean-k-mooney antia: so provided you didnt already rebase the inital commits in the series they wont be updated
17:03:03 antia I need to update this based on the changes
17:03:51 gibi (in pure git without git review, this means pulling the ps 13 from https://review.opendev.org/c/openstack/nova/+/994764 and rebasing your commit on top)
17:04:12 tkajinam sean-k-mooney, hmm a problem is that memory encryption support is not in driver capabilities yet. we could add it but that'd be removed later once start up check is delegated to drivers.
17:05:43 antia yes I rebased in pure git before so it is just about rebasing with the new ps
17:05:50 tkajinam sean-k-mooney, are you ok if I propose a follow-up to add that capability ? though I'd abandon that once your validation delegation is merged.
17:06:28 sean-k-mooney adding and removing a cpablity is cheaper then a dirver function
17:06:51 sean-k-mooney but once we add and remove either i think and dont do a release in between we are sort of ok
17:07:19 sean-k-mooney there wont have been a release for any one to depend on it
17:07:30 sean-k-mooney so no need for deprecation of your new methond
17:07:52 sean-k-mooney i just want to make sure other agree that i can jsut delete is as part fo the refactor
17:15:19 gibi antia: yepp one more rebase :)
17:16:16 antia I think it is clear then :)
17:16:44 antia I'll ask again if I run into something else
17:18:08 gibi antia: sure
17:36:40 tkajinam https://zuul.opendev.org/t/openstack/builds?job_name=nova-alt-configurations&project=openstack/nova
17:37:09 tkajinam it seems nova-alt-configurations POST_FAILURE is now broken now due to missing cinder service
17:42:47 tkajinam https://bugs.launchpad.net/python-openstackclient/+bug/2160553
17:45:06 melwitt OSC server create requires cinder? that seems ... wrong
17:47:59 tkajinam yeah
17:48:37 tkajinam I roughly suspect it is an regression caused by https://review.opendev.org/c/openstack/python-openstackclient/+/994996
17:54:42 opendevreview Takashi Kajinami proposed openstack/nova master: DNM: Testing https://review.opendev.org/c/openstack/nova/+/997104
17:55:24 tkajinam https://review.opendev.org/c/openstack/python-openstackclient/+/996626
17:55:30 tkajinam oh slawek already proposed a fix
17:56:33 melwitt ahh nice
18:03:03 opendevreview sean mooney proposed openstack/nova stable/2026.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997027
18:03:19 sean-k-mooney melwitt: elodilles if that work for you ill update the rest
18:03:46 sean-k-mooney although ill probly to the rest tomorrow
18:03:53 sean-k-mooney so no rush
18:05:12 melwitt cool thanks sean-k-mooney, will review
18:31:24 opendevreview Merged openstack/nova master: Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500
20:05:10 opendevreview sean mooney proposed openstack/nova master: Delegate startup instance processing to drivers https://review.opendev.org/c/openstack/nova/+/990552
20:07:43 opendevreview sean mooney proposed openstack/nova master: Add vTPM startup validation reproducer https://review.opendev.org/c/openstack/nova/+/990551
20:07:44 opendevreview sean mooney proposed openstack/nova master: Reduce ProviderTree copy cost for Ironic https://review.opendev.org/c/openstack/nova/+/980676
20:07:44 opendevreview sean mooney proposed openstack/nova master: Delegate startup instance processing to drivers https://review.opendev.org/c/openstack/nova/+/990552
20:08:29 opendevreview sean mooney proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679
20:08:59 opendevreview sean mooney proposed openstack/nova master: perf: add ironic startup performance simulator https://review.opendev.org/c/openstack/nova/+/980641
20:09:00 opendevreview sean mooney proposed openstack/nova master: docs: add Ironic startup performance measurements https://review.opendev.org/c/openstack/nova/+/980680
20:25:59 JayF dansmith: if you get an extra moment; https://review.opendev.org/c/openstack/ironic/+/997105 is the corollary fix in Ironic for OSSN-0101 if you wanna take a look. Just wanted to let you know since you authored the Nova one.
22:51:15 gmaan https://review.opendev.org/c/openstack/nova/+/996757
22:51:15 gmaan gibi: sean-k-mooney: thanks for the more details in bug and I am good on the attachment_id 404 case and it does not need microverison. The only use case I was thinking was if user monitoring it to know if the attachment is ready but that is not valid or guranteed case. So returning attachment_id=None is not actually a success case even return code is 200. -1 on that change only for the test cases
23:01:53 opendevreview Ghanshyam Maan proposed openstack/nova master: Add doc and release notes for the graceful shutdown task tracking https://review.opendev.org/c/openstack/nova/+/997141
23:34:47 opendevreview KWAK JOONGKEE proposed openstack/nova master: policy: initialize enforcer before publishing it https://review.opendev.org/c/openstack/nova/+/997142
#openstack-nova - 2026-07-14
00:42:51 opendevreview Ghanshyam Maan proposed openstack/nova master: Add doc and release notes for the graceful shutdown task tracking https://review.opendev.org/c/openstack/nova/+/997141
00:42:56 gmaan dansmith: melwitt: graceful shutdown changes are ready for review https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown-part2-task-tracking%22

Earlier   Later