Earlier  
Posted Nick Remark
#openstack-nova - 2025-07-24
17:12:02 sp-bmilanov gibi: dansmith: based on your suggestion https://review.opendev.org/c/openstack/nova-specs/+/955812 I have to go now, but I hope we discuss it again on the next Nova weekly
17:39:15 sean-k-mooney ...
17:39:22 sean-k-mooney how is https://review.opendev.org/c/openstack/nova/+/944054 not merged already....
17:39:43 sean-k-mooney this was ment to be in epoxy
17:39:51 sean-k-mooney before we released ...
17:57:55 opendevreview sean mooney proposed openstack/nova master: [tests] Add printing of sample and template paths https://review.opendev.org/c/openstack/nova/+/944054
17:57:55 opendevreview sean mooney proposed openstack/nova master: only show standard image properties in server show. https://review.opendev.org/c/openstack/nova/+/942413
17:59:24 sean-k-mooney dansmith: can you readd your +2 to those... that was ment to be included before we released epoxy. i cosidered that to be a release blocker so sin tthat aprpently didnt happen im keen to get that fixed in stable as soon as possible
18:00:47 sean-k-mooney i only noticed that was not there because i went to debug a test an notice my follow patch to improve the test loging was missing
18:04:28 gibi sean-k-mooney: fast approved both
18:04:42 sean-k-mooney thansk
18:05:07 sean-k-mooney i think we just missed this in the scamble to fix all the tempest schemam issues that those micoverions caused
18:05:13 sean-k-mooney i.e. missed that it failed to merge
18:05:33 sean-k-mooney i was 100% certian this was in
18:14:18 gmaan tempest sche3ma check key-value pattern not exact values so we are good there https://github.com/openstack/tempest/blob/master/tempest/lib/api_schema/response/compute/v2_98/servers.py#L32
18:15:13 sean-k-mooney gmaan: ya the tempest side is fine
18:15:35 sean-k-mooney it was more that we discoverd the issue when trying to merge the speice stuff and hadn several patches in flight
18:15:43 sean-k-mooney this just fell through the cracks
18:16:45 sean-k-mooney gmaan: by the way i didnt find time to look at the testing of the manager role stuff today but ill try and do that tomorrow
18:17:22 gmaan sean-k-mooney: no issue. thanks
20:19:31 opendevreview Merged openstack/nova master: api: Add response body schemas for networks API https://review.opendev.org/c/openstack/nova/+/952283
#openstack-nova - 2025-07-25
02:13:56 opendevreview melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197
02:13:56 opendevreview melanie witt proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196
02:13:57 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502
02:13:57 opendevreview melanie witt proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501
02:13:58 opendevreview melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628
02:13:58 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795
02:13:59 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021
02:13:59 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483
02:14:00 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771
02:14:01 opendevreview melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630
02:14:01 opendevreview melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629
02:14:03 opendevreview melanie witt proposed openstack/nova master: TPM: confirm secret security via hard reboot https://review.opendev.org/c/openstack/nova/+/955847
05:13:25 opendevreview melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197
05:13:25 opendevreview melanie witt proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196
05:13:26 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502
05:13:26 opendevreview melanie witt proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501
05:13:27 opendevreview melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628
05:13:27 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795
05:13:28 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483
05:13:29 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771
05:13:29 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021
05:13:31 opendevreview melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630
05:13:31 opendevreview melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629
05:13:33 opendevreview melanie witt proposed openstack/nova master: TPM: confirm secret security via hard reboot https://review.opendev.org/c/openstack/nova/+/955847
05:17:37 jayaanand_ we are addressing NFS volume extension issue we want to address in this resale and Nova dependency addressed in PR https://review.opendev.org/c/openstack/nova/+/680648. i am not able to reach out to author. please suggest way out
06:37:06 noonedeadpunk jayaanand_: so, do you consider taking over the patch and addressing the review point, or wanna original author to do that?
06:37:38 noonedeadpunk As with gerrit you can quite easily pull in the patch and update it
06:37:56 noonedeadpunk the only thing to remember now - is adding DSO to it...
06:58:38 rpittau gibi: thanks for the answer and apologies if I haven't replied, I was distracted downstream, we found the root cause, it does not look related to nova, it was a red herring, here's the issue if you're curious https://551b5d3d5ab1e9429cee-63329dcd236a98c48b3784d0d458e269.ssl.cf5.rackcdn.com/openstack/a4a4e91242434212aaaf067deecc5292/controller/logs/screen-q-l3.txt
07:05:46 gibi rpittau: no worries. I'm glad you found the root
07:20:51 opendevreview Lajos Katona proposed openstack/nova master: DNM: Test pyroute2 bump to greate than 0.9.x https://review.opendev.org/c/openstack/nova/+/955858
07:36:40 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502
07:36:41 opendevreview melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628
07:36:41 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795
07:36:42 opendevreview melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021
07:36:42 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483
07:36:43 opendevreview melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629
07:36:43 opendevreview melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771
07:36:44 opendevreview melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630
07:36:45 opendevreview melanie witt proposed openstack/nova master: TPM: confirm secret security via hard reboot https://review.opendev.org/c/openstack/nova/+/955847
08:43:40 opendevreview Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754
10:51:44 sean-k-mooney noonedeadpunk: jayaanand_ so volume extened for voluem attach to a nova instnace is tecnhially not supported
10:51:51 sean-k-mooney the feature depend on https://review.opendev.org/c/openstack/nova-specs/+/949504
10:52:01 sean-k-mooney https://blueprints.launchpad.net/openstack/?searchtext=assisted-volume-extend
10:52:28 sean-k-mooney basicaly there is a spec to properly supprot it but at present its not expected to fully work
10:52:28 noonedeadpunk frankly, I think this topic even boils down to be able to detach root volume
10:52:41 noonedeadpunk which was rejected due to absent usecase, if I'm not mistaken
10:52:53 sean-k-mooney noonedeadpunk: well no that slithgly idfferten
10:53:15 noonedeadpunk it would solve it though?
10:53:43 noonedeadpunk like a usecase of restoring volume from snapshot and from backup
10:53:45 sean-k-mooney only for root voluems
10:53:56 sean-k-mooney you can already do that
10:54:00 noonedeadpunk but rest you can already detach and resize...
10:54:03 sean-k-mooney you can rebuild BFV guests
10:54:25 noonedeadpunk I'm side-=tracking now the topic a bit....
10:54:33 noonedeadpunk But how does rebuild helps?
10:54:44 sean-k-mooney that how you restore form snapshot
10:54:52 noonedeadpunk and cinder just rejects to restore to volume that is attached?
10:55:02 sean-k-mooney no
10:55:28 sean-k-mooney if you have a nova vm booted form a cidner volume
10:55:47 sean-k-mooney you can rebuild to restore form snapshot isnce antelope ish maybe longer
10:56:05 noonedeadpunk huh
10:56:17 noonedeadpunk Yeah, I know that rebuild does work
10:56:20 sean-k-mooney https://specs.openstack.org/openstack/nova-specs/specs/zed/implemented/volume-backed-server-rebuild.html
10:56:48 sean-k-mooney the comment i made about is, nova/cidner does not offically supprot extendeing nfs volumes that are in use
10:57:07 noonedeadpunk Though I still don't get a bit about snapshots
10:57:09 sean-k-mooney that what https://blueprints.launchpad.net/nova/+spec/assisted-volume-extend is intended to resolve
10:57:27 noonedeadpunk yeah, ok ,right
10:58:02 sean-k-mooney for any other backend however that perfectly valid to do so fixing bugs in that area is not a bad thing
10:58:21 noonedeadpunk or well, you mean instance snapshot, which is effectively a glance image, not cinder snapshots?
10:58:27 sean-k-mooney im just saying that even if we merged https://review.opendev.org/c/openstack/nova/+/680648 extending nfs would still not be supproted
10:58:45 noonedeadpunk ++ fair enough
10:58:47 sean-k-mooney noonedeadpunk: nova has logic to do a cinder volume snapshot if it BFV
10:58:59 noonedeadpunk oh...
10:59:03 noonedeadpunk I missed that entirely
10:59:11 sean-k-mooney we still upload an empty glance iamge to hold metadata
10:59:22 sean-k-mooney but it just point to the cinder snapshot
11:00:10 sean-k-mooney we also have an addtion gem
11:00:26 noonedeadpunk but it's kinda... not very user frioendly? in terms of - you need to spawn snapshot in one specific way rather then other to be able to restore later?

Earlier   Later