| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-07 | |||
| 18:04:19 | luvn | okay, so with the instance running and the volume attached: | |
| 18:04:20 | sean-k-mooney | that reminds me that we still have not completed https://review.opendev.org/c/openstack/nova/+/873560 | |
| 18:04:24 | sean-k-mooney | https://blueprints.launchpad.net/nova/+spec/assisted-volume-extend | |
| 18:04:28 | luvn | resize: Failed to set volume size: Volume is in in-use state, it must be available before size can be extended | |
| 18:04:41 | luvn | retype: Failed to set volume type: Invalid volume: Retype needs volume to be in available or in-use state, not be part of an active migration or a consistency group, requested type has to be different that the one from the volume, and for in-use volumes front-end qos specs cannot change. | |
| 18:05:18 | luvn | retyping appears to be more "loose" than resize, but still, can't really perform it | |
| 18:05:38 | sean-k-mooney | what storage backend are you using out of interest | |
| 18:06:07 | luvn | i'm using ceph | |
| 18:06:19 | sean-k-mooney | ok ya that should be the most flexiable | |
| 18:08:02 | sean-k-mooney | im kind fo torn, i guess this would be a new feature either on the cinder or nova side | |
| 18:08:17 | sean-k-mooney | for nova to enabel it we would need a new instance action | |
| 18:08:56 | sean-k-mooney | im not sure what woudl eb required on the cidner side to make it work if we did it form cinder insetead or if that is even possible | |
| 18:11:09 | sean-k-mooney | there is a complex back and forth between nova and cidner and the storage backend ot orchestrated depending on how the volume is attached and if the isntance is runnign so it wouldnt be a 1 liner to enabel unfrotuently | |
| 18:11:34 | luvn | oh, i see | |
| 18:11:56 | sean-k-mooney | for example for nfs if the vm is runnign nova need to grow the backign file on the nfs share (it cant be done by cidner) | |
| 18:12:13 | sean-k-mooney | for ceph that pretty easy you just need to ask ceph to grow it | |
| 18:12:54 | sean-k-mooney | for iscsi/fiber channel cinder woudl have to ask the storage backend to extend it and that may or may not be visabel to the guest before the iscisi export is remonted | |
| 18:14:03 | sean-k-mooney | the thing is we do supprot extnedign cidner volume when they are not the root volume (i dont recall in which casese it offline vs online) | |
| 18:14:06 | luvn | now i see why ceph is by far the most flexible heh | |
| 18:14:38 | luvn | in any case, i sent a message on cinder's channel; let's see if we can get a better insight into why these limitations exist | |
| 18:14:42 | luvn | i'm legit curious | |
| 18:15:22 | dansmith | likely because three connection mechanisms require three different paths, among other things :) | |
| 18:15:32 | dansmith | although in reserved state that should be not a problem I think | |
| 18:16:19 | sean-k-mooney | right reserved means for one that the vm is not running so cidner can alwasy do the opertion itself | |
| 18:16:42 | sean-k-mooney | the onely reason nova has to grow it for nfs when the vm is runnign is qemu has locked the file on the share | |
| 18:18:29 | sean-k-mooney | https://github.com/openstack/nova/blob/master/releasenotes/notes/nova-support-attached-volume-extend-88ce16ce41aa6d41.yaml | |
| 18:19:02 | luvn | so i guess there's a world where it's a matter of cinder allowing these operations to be performed when the volume is reserved | |
| 18:19:11 | luvn | (of course, if there are no impediments, etc) | |
| 18:19:32 | sean-k-mooney | that release note might be out of date and does nto mention BFV but it say "only the libvirt compute driver with iSCSI and FC volumes | |
| 18:19:34 | sean-k-mooney | supports the online volume size change." | |
| 18:21:18 | sean-k-mooney | we added supprot for extendign in use rbd voluem later | |
| 18:21:20 | sean-k-mooney | https://github.com/openstack/nova/commit/eab58069ea6afba875fb0a7d7ac68c7e83ebf14a | |
| 18:21:33 | sean-k-mooney | that was stien | |
| 18:27:30 | sean-k-mooney | so everythign is in place for ceph for addtional ciner data volumes the vm is running | |
| 18:28:22 | sean-k-mooney | there must just be somethign missign for BFV but im not sure whty that could be | |
| 19:56:05 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Try vTPM live migration with non-private, non-eph secret https://review.opendev.org/c/openstack/nova/+/938613 | |
| 20:34:12 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Try vTPM live migration with non-private, non-eph secret https://review.opendev.org/c/openstack/nova/+/938613 | |
| #openstack-nova - 2025-01-08 | |||
| 05:02:30 | opendevreview | Merged openstack/nova master: Drop unused decoding of xml content https://review.opendev.org/c/openstack/nova/+/937506 | |
| 11:52:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Update note on flavor disabled property https://review.opendev.org/c/openstack/nova/+/937356 | |
| 12:00:32 | opendevreview | Bence Romsics proposed openstack/os-vif master: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/923036 | |
| 12:29:39 | opendevreview | Merged openstack/nova master: api-ref: Update note on flavor disabled property https://review.opendev.org/c/openstack/nova/+/937356 | |
| 14:14:40 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:22:07 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:35:13 | dansmith | sean-k-mooney: reminder, this iso+gpt is ready: https://review.opendev.org/c/openstack/nova/+/931833/4 | |
| 14:36:09 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:37:24 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:41:29 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:44:16 | sean-k-mooney | dansmith: i think thats patch 5 in my queue for today but ill see if i can get to it today thanks for the reminder | |
| 14:44:36 | dansmith | sean-k-mooney: ack, just wanted to make sure it was in the queue somewhere :D | |
| 14:44:39 | sean-k-mooney | i have one watcher review then 3 placment reviews and then that | |
| 15:09:50 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 15:29:01 | opendevreview | ribaudr proposed openstack/nova-specs master: Enable VFIO devices with kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/936407 | |
| 15:45:03 | opendevreview | Merged openstack/os-vif master: Integrate pre-commit https://review.opendev.org/c/openstack/os-vif/+/934240 | |
| 16:50:01 | opendevreview | Ivan Anfimov proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 17:00:06 | opendevreview | Ivan Anfimov proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 17:04:59 | opendevreview | ribaudr proposed openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 17:09:10 | opendevreview | Ivan Anfimov proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 17:15:46 | opendevreview | Ivan Anfimov proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 17:39:16 | opendevreview | Douglas Viroel proposed openstack/nova master: WIP - Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 19:39:37 | opendevreview | Douglas Viroel proposed openstack/nova master: WIP - Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 19:41:53 | opendevreview | Alan Bishop proposed openstack/nova-specs master: Show swap_progress in attachment details https://review.opendev.org/c/openstack/nova-specs/+/937485 | |
| 21:21:18 | opendevreview | Merged openstack/os-vif master: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/923036 | |
| 21:43:26 | opendevreview | Merged openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 21:44:29 | opendevreview | Merged openstack/placement master: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/937273 | |
| 22:59:47 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| #openstack-nova - 2025-01-09 | |||
| 02:54:16 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 08:29:52 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 08:29:53 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | |
| 11:39:42 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 14:14:38 | s3rj1k | hi team, to iterate quickly on https://review.opendev.org/c/openstack/nova-specs/+/937185 maybe we should go with some very generic proposal for solving this issue? at least start drafting something up | |
| 15:09:50 | opendevreview | Merged openstack/nova-specs master: Enable VFIO devices with kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/936407 | |
| 16:11:06 | opendevreview | Masahito Muroi proposed openstack/nova-specs master: Add spec for the network id query in server list and server details https://review.opendev.org/c/openstack/nova-specs/+/938823 | |
| 16:22:05 | masahito | Hi folks, if you have time please take a look https://review.opendev.org/c/openstack/nova-specs/+/938823. I know the deadline is today so my first proposal is late and the spec may require some iteration, though. | |
| 16:32:22 | opendevreview | Masahito Muroi proposed openstack/nova-specs master: Add spec for the network id query in server list and server details https://review.opendev.org/c/openstack/nova-specs/+/938823 | |
| 16:56:37 | gibi | melwitt: I've pushed a new revision for https://review.opendev.org/c/openstack/placement/+/936832 to fix sean-k-mooney's comments. So when you have time please re-review | |
| 16:58:49 | opendevreview | ribaudr proposed openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 17:03:19 | sean-k-mooney | masahito: why not do a port list for the given network and just get the device-id form the port | |
| 17:04:15 | sean-k-mooney | oh becase you want more then just the uuids | |
| 17:04:24 | sean-k-mooney | you also want the other server details. | |
| 17:21:47 | melwitt | gibi: sure will do | |
| 17:22:00 | opendevreview | ribaudr proposed openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 17:25:17 | masahito | sean-k-mooney: yes. we want to fetch more server info not only the uuids. The device-id is just a list of ids. | |
| 17:27:19 | sean-k-mooney | masahito: im not -2 on this but im kind of -1.5 | |
| 17:27:58 | sean-k-mooney | i think we woudl want the neutron team to weigh in on it too but there are a lot of other critira you could filter nova isntnace on even if we just consider neutorn | |
| 17:28:52 | sean-k-mooney | and im not sure it a good idea ot extend the nova api to support all or really any of them | |
| 17:30:43 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | |
| 17:31:13 | sean-k-mooney | masahito: at present we do not supprot filterign server by the external identify or any resouce form any other service with the sole expction beign the keystone project and glance image uuid which are directly assocated with the instnace object | |
| 17:31:28 | artom | dansmith, melwitt, gibi, sean-k-mooney ^^ took longer than I wanted (had to manage car troubles), and there's still a couple of todo open questions, but I think it's close enough to push up | |
| 17:31:56 | sean-k-mooney | can you revert the tox.ini change | |
| 17:32:03 | artom | Doh, yeah. | |
| 17:32:12 | artom | It doesn't run locally with it, and I keep forgetting to clean it up. | |
| 17:32:24 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | |
| 17:32:49 | sean-k-mooney | what you can do instead for future reference is you can create a py3.10 virtual env | |
| 17:32:55 | sean-k-mooney | activate that and then run tox form there | |
| 17:33:12 | sean-k-mooney | then it will use that version of python instaead fo your default system python | |
| 17:36:50 | masahito | Thank for the quick review. I see your point. The netowrk_id is not an server's attributes. I had same concern so that I needs Nova team's idea. | |
| 17:38:28 | sean-k-mooney | ya im not sure. im inclidne to say we shoudl not adress this pain point unless we see many opertoar bring it up | |
| 17:38:29 | masahito | One quick question, is there reason the list server API doesn't have id query param? like id=id1,id2,id3. If it's okay, the query can solve the current problem as I mentioned in the alternative. | |
| 17:39:06 | sean-k-mooney | hum ok that could work in some cases | |
| 17:39:18 | sean-k-mooney | so i belive the max url lenght is 256 | |