| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-10-24 | |||
| 12:10:55 | sean-k-mooney | it also kind of messes with the resouce allcoatins if you dont follwo our guidance and use flavor with 0 disk_gb for bfv guests | |
| 12:11:11 | noonedeadpunk | But I guess challange there was also to record what the drive is detached to re-attach it as root lately | |
| 12:11:31 | sean-k-mooney | noonedeadpunk: shelve_offloaded does not work because we cant unshelve if you have detached the volume and attached it to somethign else | |
| 12:11:54 | noonedeadpunk | but it would be a valid failure that VM can't be offloaded | |
| 12:12:22 | sean-k-mooney | ya so it really comes down to the usecase | |
| 12:12:23 | noonedeadpunk | sean-k-mooney: sorry, I didn't get part about "use flavor with 0 disk_gb for bfv guests" - can you elaborate a bit on this part? | |
| 12:12:44 | noonedeadpunk | As I thought that for bfv you're kinda good at having 0 disk_gb? | |
| 12:12:56 | sean-k-mooney | yes that is the recomendation | |
| 12:13:09 | sean-k-mooney | but some peopel dont do that and use a normal flavor | |
| 12:13:13 | bauzas | sean-k-mooney: the fact is, we need a new design discussin | |
| 12:13:22 | bauzas | but if someone wants to work on it, sure I can hel | |
| 12:13:24 | noonedeadpunk | ah, ok, as I read it as "you don't follow recommendation AND use 0 disk_gb" | |
| 12:13:25 | bauzas | help | |
| 12:13:29 | bauzas | that was my point | |
| 12:13:39 | bauzas | noonedeadpunk: ++ I'll also discuss it on the PTG | |
| 12:14:14 | sean-k-mooney | bauzas: well right now i think its a hard cell to add to nova unless we put some restricitons in place | |
| 12:14:23 | sean-k-mooney | ... sell | |
| 12:14:42 | bauzas | sure, see my above comments :) | |
| 12:14:49 | bauzas | anyway | |
| 12:15:03 | bauzas | noonedeadpunk: I'll provide a bugfix hopefully next week | |
| 12:15:13 | sean-k-mooney | noonedeadpunk: i would have less issue with saying "you can only detach the root voluem flavor with 0 disk_gb" | |
| 12:15:15 | bauzas | or tomorrow morning if I have time | |
| 12:15:27 | sean-k-mooney | and then putting the vm in to an "unbootable state" | |
| 12:15:40 | sean-k-mooney | or only supporting this for shleve_offloaded instances | |
| 12:16:22 | sean-k-mooney | noonedeadpunk: the real concern was not with the detach but with reattaching potentially a diffent root volume with image properties that were invlaid for the current host | |
| 12:16:53 | sean-k-mooney | but if its shelve_offloaded after the reattachment then that not a problem as when we unshleve it will go throughthe schduler | |
| 12:17:28 | noonedeadpunk | Yeah, that's why I thought that shelved should be quite appropriate state where you potentially can do that | |
| 12:17:46 | noonedeadpunk | And I would say that having 0 disk_gb for flavor would be also fair requirements | |
| 12:18:03 | noonedeadpunk | for the feature to work | |
| 12:18:27 | sean-k-mooney | the 0 disk_gb thing is partly a personal prefence. we do not allow non bfv guest to use 0 disk flavors | |
| 12:18:39 | sean-k-mooney | but we dont require them too either and i really would prefer to require that | |
| 12:19:37 | sean-k-mooney | we dont actully need to use the flavor size to knwo if its bfv in the api | |
| 12:19:47 | sean-k-mooney | so we could make it work without that restirction | |
| 12:20:06 | sean-k-mooney | noonedeadpunk: what i really want to aovid is needing to hit the schduler on every volume attach | |
| 12:20:16 | noonedeadpunk | well. requiring that might complicate some things actually... | |
| 12:20:34 | sean-k-mooney | just in case its happens to be a root volume | |
| 12:20:47 | noonedeadpunk | I mean prohibit to do bfv with non-0 flavor | |
| 12:21:01 | sean-k-mooney | we didnt do that in the past due to upgrade concerns | |
| 12:21:19 | sean-k-mooney | what we did instead id if its BFV we nolonger include the disk_gb in the allcoation request | |
| 12:21:24 | noonedeadpunk | nah, I think that it indeed should be some offloaded state imo... Or indeed introduce new one, so that resources won't be wasted on offload per sat | |
| 12:21:30 | sean-k-mooney | as of some releivly recent openstack version | |
| 12:21:59 | noonedeadpunk | but which will still go throguh scheduler on boot | |
| 12:22:18 | noonedeadpunk | but yeah, eventually it's still offloaded... | |
| 12:22:55 | sean-k-mooney | noonedeadpunk: we will talk about this later in the ptg but im also wondering about the orginal usecase that is of most interest to people | |
| 12:23:17 | sean-k-mooney | i.e. what fraction of those that want this are using a version of nova that does not supprot rescue or rebuild for bfv guests | |
| 12:23:24 | sean-k-mooney | we added both in the lat 2 years | |
| 12:23:30 | noonedeadpunk | I think most usecase is in fact some rescue.... | |
| 12:23:46 | noonedeadpunk | and yes, that kinda covers my usecase fully | |
| 12:24:25 | noonedeadpunk | sean-k-mooney: is it possible to reset delete on termination flag for the root drive? | |
| 12:24:27 | sean-k-mooney | noonedeadpunk: part of the reason we didnt do this was we decied lets just make the instance action that dont work with bfv work first | |
| 12:24:50 | sean-k-mooney | noonedeadpunk: that im not sure i think you might eb able to do it via cinder | |
| 12:25:07 | sean-k-mooney | but that woudl also be reasonable to support IMO if we dont supprot that already | |
| 12:25:33 | sean-k-mooney | you shoudl always be able to say "hay dont delete this" after the fact i think | |
| 12:25:39 | noonedeadpunk | yeah | |
| 12:26:00 | noonedeadpunk | as this what strikes my mind when I think why I would need to detach the root volume... | |
| 12:26:10 | noonedeadpunk | Just to save it from being deleted | |
| 12:26:18 | noonedeadpunk | (with the VM) | |
| 12:26:36 | noonedeadpunk | and then rescue works nicely indeed | |
| 12:26:44 | sean-k-mooney | ya and you should be able to do the same for ports. the port case is defenitly on our todo list | |
| 12:28:44 | noonedeadpunk | but then indeed I don't know why would you need to detach it | |
| 12:29:17 | noonedeadpunk | Just out of convenience... Like first thing someone would try to do rather using rescue... | |
| 12:29:46 | noonedeadpunk | so I think you;re right here and it's smth that ppl were just historically missing due to not working rescue | |
| 12:30:26 | noonedeadpunk | and that is kinda inertion of thinking | |
| 12:32:27 | sean-k-mooney | ya so thats why im wondering about the usecase | |
| 12:32:43 | sean-k-mooney | if there is a usecase we cant supprot then im happy to hear that and see what we can do to supprot it | |
| 12:33:11 | sean-k-mooney | but if its already covered by rescue or updating the delete on terminate flag | |
| 12:33:24 | sean-k-mooney | then i woudl perfer to use that approch then add root detach | |
| 12:33:38 | sean-k-mooney | so we will see later i guess | |
| 13:00:35 | bauzas | we gonna use Zoom eventually https://www.openstack.org/ptg/rooms/diablo | |
| 13:05:19 | bauzas | sean-k-mooney: are you arriving to the PTg ? | |
| 14:28:53 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Deprecate use_cow_images and 'default' images_type https://review.opendev.org/c/openstack/nova/+/898229 | |
| 14:28:53 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix image conversion check in finish_migration https://review.opendev.org/c/openstack/nova/+/897842 | |
| 15:19:31 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Avoid getCapabilities() to calculate host CPU definition https://review.opendev.org/c/openstack/nova/+/899185 | |
| 15:25:46 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Avoid getCapabilities() to calculate host CPU definition https://review.opendev.org/c/openstack/nova/+/899185 | |
| 16:18:20 | simondodsley | sean-k-mooney: sorry - i can't stay any longer on the PTG today - i have other commitments i have to deal with. I would love some update on the spec I put in the etherpad though | |
| 16:21:30 | opendevreview | Merged openstack/nova stable/2023.2: Install lxml before we need it in post-run https://review.opendev.org/c/openstack/nova/+/899142 | |
| 16:43:14 | opendevreview | Pavlo Shchelokovskyy proposed openstack/placement master: Use new db session when retrying https://review.opendev.org/c/openstack/placement/+/898807 | |
| 16:45:37 | opendevreview | Pavlo Shchelokovskyy proposed openstack/placement master: Use new db session when retrying https://review.opendev.org/c/openstack/placement/+/898807 | |
| 18:13:19 | bauzas | not fun: we had the exact same issue with rolling upgrade in Victoria>Wallaby on rebuild https://review.opendev.org/c/openstack/nova/+/761458/ | |
| 18:13:28 | bauzas | so we really need a grenade test on it | |
| 18:13:37 | bauzas | and perhaps some hacking check, I dunno | |
| 18:14:42 | bauzas | I'll write the fixes tomorrow morning | |
| 18:15:23 | bauzas | and yeah, we don't test rebuild https://github.com/openstack/nova/blob/master/.zuul.yaml#L560 | |
| 21:49:08 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Deprecate use_cow_images and 'default' images_type https://review.opendev.org/c/openstack/nova/+/898229 | |
| 21:49:08 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix image conversion check in finish_migration https://review.opendev.org/c/openstack/nova/+/897842 | |
| 23:01:00 | opendevreview | melanie witt proposed openstack/nova stable/zed: Remove use of removeprefix https://review.opendev.org/c/openstack/nova/+/899148 | |
| 23:54:50 | opendevreview | Merged openstack/nova master: fix sphinx-lint issues in releasenotes https://review.opendev.org/c/openstack/nova/+/897087 | |
| 23:54:58 | opendevreview | Merged openstack/nova master: fix sphinx-lint issues in api guide https://review.opendev.org/c/openstack/nova/+/897088 | |
| #openstack-nova - 2023-10-25 | |||
| 01:41:10 | melwitt | bauzas: I did a little bit of belated bug triage on my turn in the bug triage roster, sorry I didn't get around to it earlier 😟 | |
| 01:41:10 | melwitt | bauzas: I did a little bit of belated bug triage on my turn in the bug triage roster, sorry I didn't get around to it earlier 😟 | |
| 06:02:35 | auniyal | its frequent - https://zuul.openstack.org/builds?job_name=openstacksdk-functional-devstack+&project=openstack%2Fnova&branch=stable%2F2023.1&skip=0 | |
| 06:02:35 | auniyal | openstacksdk-functional-devstack is failing for branch: stable/2023.1, because there is a conflict in openstacksdk version, | |
| 06:06:09 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/899225 | |
| 07:04:03 | gibi | melwitt bauzas: I felt bad after the yesterday sessions so I also did some bug triage :) | |
| 07:12:31 | kashyap | gibi: Morning. I started a crude version of it here, to get the ball rolling: https://review.opendev.org/c/openstack/nova/+/899185 | |
| 07:13:57 | kashyap | gibi: By publicly admitting that, you're implicitly making me feel guilty for not doing enough upstream bug triage | |
| 07:20:33 | bauzas | gibi: melwitt: sending all e-hugs to both of you <3 | |
| 07:28:52 | frickler | auniyal: see https://review.opendev.org/q/topic:gate-fix-unpin-osdk , seems we need that for all stable branches now (except 2023.2) | |
| 08:05:14 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/898945 | |
| 08:05:50 | opendevreview | Amit Uniyal proposed openstack/nova stable/zed: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/899229 | |