| *** mhen_ is now known as mhen | 01:33 | |
| jayaanand | Hi All, good morning. i am planning to co-author and own patch https://review.opendev.org/c/openstack/nova/+/680648 as part of our NFS extend feature. I will address any pending review comments. please complete this review. we need this change for This relase | 04:47 |
|---|---|---|
| jssfr | jayaanand, what do you mean by "NFS extend feature"? | 05:26 |
| jayaanand | i am trying to fix BUG https://bugs.launchpad.net/cinder/+bug/1878913 from ONTAP | 06:44 |
| jayaanand | to improve ERROR reporting. we need fix https://review.opendev.org/c/openstack/nova/+/680648 from NOVA side | 06:45 |
| jssfr | uhuh | 07:03 |
| sthoffmann[m] | Thanks for the feedback. Should I add this to the POC or will we include it to the spec for later implementation? | 07:59 |
| jssfr | jayaanand, we had submitted specs for that feature: https://review.opendev.org/c/openstack/nova-specs/+/949504 and https://review.opendev.org/c/openstack/cinder-specs/+/866718 but we didn't get around to re-submit them for Flamingo. | 08:36 |
| jssfr | we plan on resuming that work in the G cycle | 08:37 |
| jssfr | *get it merged for Flamingo | 08:40 |
| sean-k-mooney | Uggla: https://blueprints.launchpad.net/nova/+spec/xml-image-meta is now merged if you want to mark it as implemnted | 09:44 |
| jayaanand | can we consider https://bugs.launchpad.net/cinder/+bug/1878913 and https://review.opendev.org/c/openstack/nova/+/680648 as stop gap arrangement. right now, customer is not able extend NFS volumes attached to Nova VM | 09:53 |
| sean-k-mooney | jayaanand: if i recall correctly the spec for that was nto accpated this cycle but ill check | 09:59 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/949504 | 10:00 |
| sean-k-mooney | jayaanand: ^ | 10:00 |
| sean-k-mooney | jayaanand: extended for nfs i think is techinally not supproted today | 10:01 |
| sean-k-mooney | """ | 10:02 |
| sean-k-mooney | Many remotefs-based volume drivers in Cinder use the ``qemu-img resize`` | 10:02 |
| sean-k-mooney | command to extend volume files. | 10:02 |
| sean-k-mooney | However, when the volume is attached to a guest, QEMU will lock the file and | 10:02 |
| sean-k-mooney | ``qemu-img`` will be unable to resize it. | 10:02 |
| sean-k-mooney | In this case, only the QEMU process holding the lock can resize the volume, | 10:02 |
| sean-k-mooney | which can be triggered through the QEMU monitor command ``block-resize``. | 10:02 |
| sean-k-mooney | There is currently no adequate way for Cinder to use this feature, so the NFS, | 10:02 |
| sean-k-mooney | NetApp NFS, Powerstore NFS, and Quobyte volume drivers all disable extending | 10:02 |
| sean-k-mooney | attached volumes.""" | 10:02 |
| sean-k-mooney | from teh spec | 10:02 |
| sean-k-mooney | jayaanand: we dont consdier that to be bug becasue it has never work and there is a proposal to add it as a feature the problem is that the folks workign on it have done so very intermitently | 10:03 |
| sean-k-mooney | jayaanand: https://review.opendev.org/c/openstack/nova/+/873560 that is the poc | 10:04 |
| sean-k-mooney | jayaanand: part of the problem is the cider side is not done yet https://review.opendev.org/q/topic:%22bp/extend-volume-completion-action%22 | 10:05 |
| sean-k-mooney | although part of it merged in epocy i belive so we may be at the poitn where the nova side can progress but that will be 2026.1 at this rate | 10:06 |
| jayaanand | we tested https://review.opendev.org/c/openstack/nova/+/680648 with customer NFS volumes attached to Nova VM. except encrypted volumes other scenarios NFS volumes expanded. This is ok with most of the customers. Without this changes. user need expand volume in NetApp filer and reboot VM to reflect size change. | 10:08 |
| kgube | sean-k-mooney, the cinder dependency is actually merged | 10:09 |
| jayaanand | this changes https://review.opendev.org/c/openstack/nova/+/680648 has extra error scenarios no logic changers will help customer to get information about errors. | 10:09 |
| kgube | sean-k-mooney, at least the first part, that the nova changes depend on | 10:09 |
| sean-k-mooney | kgube: the new api exits but is not hooked up right | 10:10 |
| sean-k-mooney | i.e. cinder is not actully waiting for the completion callback correct? | 10:10 |
| kgube | well, it has to be enabled in the drivers, but that depends on nova support again | 10:11 |
| sean-k-mooney | jayaanand: we can consdier improving the error handeling but from a nova perspective extending an nfs voluem wont be supproted until kgube's feateure is complete | 10:12 |
| kgube | sean-k-mooney, it's a cross-dependency, so we have to do a sort of a ping-pong of merges | 10:12 |
| sean-k-mooney | kgube: ack, i assume your going to reporpose the nova sepc for 2026.1 | 10:13 |
| kgube | currently the ball is on the nova side | 10:13 |
| kgube | yes | 10:13 |
| jayaanand | thank you! sean-k-mooney for considering error handling. is it possible to merge https://review.opendev.org/c/openstack/nova/+/680648 for this release. | 10:14 |
| sean-k-mooney | kgube: have the cidner folks reviewd the cinder parts? if we are tat the critical poing we proably need to corrdiate the revew on both side to get it finally landed | 10:14 |
| sean-k-mooney | jayaanand: i have not looked at it in detail but maybe. it likely will need a release note and moer testing. ideally a functional test to repoduce the incorrect event state in a preceedign patch then this patch woudl update the functional test to show that the instance action event was properly set to error | 10:16 |
| sean-k-mooney | the curret unit test does not actully show it fixes the reporte bug in reporting | 10:17 |
| kgube | sean-k-mooney, cinder folks have merged the initial patch: https://review.opendev.org/c/openstack/cinder-specs/+/866718, the next cinder patch is a minor change that will depend on a new nova microversion to enable the new behavior: https://review.opendev.org/c/openstack/cinder/+/891602 | 10:23 |
| kgube | so this will have to be merged after the nova change | 10:23 |
| kgube | I can ask them to already review it, though | 10:24 |
| jayaanand | thank you! sean-k-mooney and kgube. i really appreciate your attenction | 10:27 |
| opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948276 | 10:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate v2 API https://review.opendev.org/c/openstack/nova/+/954102 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Remove version controller split https://review.opendev.org/c/openstack/nova/+/954103 | 11:12 |
| opendevreview | Merged openstack/nova master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/nova/+/953751 | 13:28 |
| opendevreview | Merged openstack/nova master: Enable E711 consistently https://review.opendev.org/c/openstack/nova/+/956436 | 13:28 |
| opendevreview | Merged openstack/nova master: db: Fix api_db_opts reading main_db_opts https://review.opendev.org/c/openstack/nova/+/956869 | 13:28 |
| *** haleyb is now known as haleyb|out | 13:32 | |
| opendevreview | Lajos Katona proposed openstack/nova master: WIP Use SDK for Neutron https://review.opendev.org/c/openstack/nova/+/928022 | 14:10 |
| opendevreview | Merged openstack/nova master: Fix live migration error logging https://review.opendev.org/c/openstack/nova/+/951009 | 14:35 |
| opendevreview | sean mooney proposed openstack/nova master: Remove Linux bridge VIF support from Nova https://review.opendev.org/c/openstack/nova/+/957040 | 18:10 |
| opendevreview | sean mooney proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | 18:15 |
| opendevreview | Merged openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | 20:14 |
| Callum027 | sean-k-mooney: Screwed up the upgrade note for the image meta patch, so a quick fix: https://review.opendev.org/c/openstack/nova/+/956988 | 22:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!