| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-15 | |||
| 11:34:26 | jlejeune | for other tests | |
| 11:38:16 | opendevreview | sean mooney proposed openstack/nova master: [DNM] Test bumping guest ram https://review.opendev.org/c/openstack/nova/+/924142 | |
| 11:43:21 | sean-k-mooney | jlejeune: this was a preexisting depency | |
| 11:43:32 | sean-k-mooney | oh in the unit test | |
| 11:43:58 | sean-k-mooney | am we added some skip patches | |
| 11:44:09 | sean-k-mooney | to skip if qemu was not there | |
| 11:44:21 | sean-k-mooney | its posible that we have some paths that are still not skipped | |
| 11:44:52 | jlejeune | indeed, in self._create_allocated_vmdk() method in unit test | |
| 11:45:09 | jlejeune | patching if you want | |
| 11:45:37 | sean-k-mooney | you can submit a patch for that if you have time | |
| 11:45:45 | jlejeune | sure, doing | |
| 11:45:47 | sean-k-mooney | its not strictly speaking incorrect | |
| 11:46:07 | sean-k-mooney | it just nova tened to try and have depencies on external tools in our unit tests | |
| 11:46:18 | sean-k-mooney | glance however does not have that concept | |
| 11:46:39 | sean-k-mooney | so when we import the unit tests form glance we gained this depency and we have been trying to relax it to make it optional | |
| 11:46:58 | sean-k-mooney | eventurlaly these tests will be remove form nova and moved to oslo when the image inspector is moved | |
| 11:47:31 | jlejeune | ok get it | |
| 11:54:18 | amorin | sean-k-mooney, that's perfect, it will fit my needs | |
| 12:47:48 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | |
| 13:42:59 | amorin | hey again, another question regarding BDM, when booting an instance with a BDM, I cannot find a way to specify the availability-zone field, while if I create the volume before using cinder API, it's available, is it missing on nova side on purpose? | |
| 14:53:37 | ralonsoh | sean-k-mooney, hello! If you have some mins, please check https://review.opendev.org/c/openstack/neutron/+/923962 | |
| 14:54:10 | ralonsoh | is that something you (nova team) discussed with the author? | |
| 14:56:34 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 15:04:53 | sean-k-mooney | so OVS_HYBRID_PLUG is not intented to be used with ovn | |
| 15:05:53 | sean-k-mooney | ralonsoh: i have not read the patch properly but i wound not normally expect that to be set when using ovn | |
| 15:06:22 | sean-k-mooney | we wait for port binind events dirfferencly for ovn | |
| 15:32:58 | opendevreview | Julien Le Jeune proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | |
| 17:59:15 | sean-k-mooney | amorin: yes its not in the api bdm because if the volume is created by nova it will have the same az as the instance and if its a prexisting volume form cinder then it will already be recored on the volume | |
| 20:31:09 | chris218 | Hi, does placement track usage of vcpu/pcpu cores on given node or it's done by nova-compute? I was wondering if it would be possible to run two nova-compute instances on one node with different drivers configured on them | |
| 20:33:29 | noonedeadpunk | nova-compute does report consumption data to the placement iirc | |
| 20:34:22 | noonedeadpunk | well, the tricky thing might be to make nova-compute service to register independently | |
| 20:36:35 | noonedeadpunk | as it relies on hostnames. while in nova.conf you can set different `my_ip` and `host` vars per service, I'm not 100% sure you will be able to do the same trick for hypervisors... As there table is populated by what driver would report as a hostname... | |
| 20:37:15 | noonedeadpunk | and then this will be pushed towards placement | |
| 21:07:55 | chris218 | so... yes but it's complicated | |
| 21:47:08 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Rebuild server: Add offset for iSCSI volumes https://review.opendev.org/c/openstack/nova/+/924184 | |
| 21:53:34 | melwitt | amorin: if nova is creating the volume, it will create it in the same AZ as you specify you want to create the instance. it's mentioned here (and maybe also in other docs) https://docs.openstack.org/nova/latest/configuration/config.html#cinder.cross_az_attach | |
| #openstack-nova - 2024-07-16 | |||
| 04:39:37 | opendevreview | Merged openstack/nova master: hardware: Fix image_meta.id within get_mem_encryption_constraint https://review.opendev.org/c/openstack/nova/+/904197 | |
| 05:19:14 | whoami-rajat | hey #openstack-nova , can i have review on this small patch? it's sitting for quite a while https://review.opendev.org/c/openstack/nova/+/889257 | |
| 07:34:34 | amorin | melwitt, sean-k-mooney so maybe I misconfigured something because my volume endup in a different AZ when doing my tests, I will double check, thanks for your answers | |
| 09:11:32 | bauzas_ | morning folks | |
| 11:25:29 | sean-k-mooney | amorin: check if you have the cross az attach config option configured to prevent cross az attach | |
| 12:47:28 | sean-k-mooney | bauzas: this is a very short spec form melwitt which we have discussed at previous ptgs https://review.opendev.org/c/openstack/nova-specs/+/923807 will you have time to review that today | |
| 12:47:49 | bauzas | yes, I'm slowing starting back | |
| 12:48:03 | sean-k-mooney | ack melwitt also has a poc implemation up for it | |
| 12:52:50 | frickler | bauzas: Uggla: auniyal: could you please check https://review.opendev.org/c/openstack/releases/+/923118 once again? (osc-placement release) | |
| 13:16:23 | weanew | Hello! I've submitted a bug fix: https://review.opendev.org/c/openstack/nova/+/923395 Could someone review it? | |
| 13:21:52 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for 2025.1 Epoxy https://review.opendev.org/c/openstack/nova-specs/+/924227 | |
| 13:36:46 | jlejeune | hello, can I have some review for my fix in unit tests please: https://review.opendev.org/c/openstack/nova/+/924148 ? | |
| 13:39:42 | bauzas | folks, can we try to not use this channel for asking for some fixes ? | |
| 13:40:24 | bauzas | jlejeune: weanew: if you want, you can add your changes in the Dalmatian etherpad : https://etherpad.opendev.org/p/nova-dalmatian-status | |
| 13:49:48 | jlejeune | bauzas: ok, I didn't create a bug in launchpad for that because it just affects unit tests, is that ok ? | |
| 13:57:03 | bauzas | jlejeune: yup, for internal changes like that, indeed you don't need to add a new Launchpad bug report | |
| 13:58:45 | jlejeune | bauzas: perfect, thanks | |
| 14:25:24 | bauzas | reminder (it was long due) : next nova meeting here in 1.5 hours (1600UTC) | |
| 14:55:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test back and forth migration https://review.opendev.org/c/openstack/nova/+/924234 | |
| 15:00:05 | elodilles | dansmith sean-k-mooney : i know it's unmaintained, but could you please have a quick look that my thinking was right when resolving conflict in this backport? https://review.opendev.org/c/openstack/nova/+/924083 | |
| 15:01:26 | elodilles | conflicts were due to patch to consolidate image creation functions (I111cfc8a5eae27b15c6312957255fcf973038ddf) was added only in zed | |
| 15:02:26 | elodilles | and as far as i understood, now i only had to touch the create_cow_image function, and no need for to change anything in create_image function (nova/virt/libvirt/utils.py) | |
| 15:03:23 | elodilles | as create_cow_image functions handles only the case when there is a backing_file | |
| 15:08:32 | sean-k-mooney | elodilles: a very light reivew, it looks correct but havented dnoe anything more then compare the 2023.1 patch to the yoga version | |
| 15:09:42 | elodilles | sean-k-mooney: ACK, thanks, i know that this is unmaintained, so i appreciate it! o:) | |
| 15:10:10 | sean-k-mooney | the ci results are relivily clean too | |
| 15:10:46 | opendevreview | Merged openstack/nova-specs master: Propose config option to control behavior of unset unified limits https://review.opendev.org/c/openstack/nova-specs/+/923807 | |
| 15:10:49 | elodilles | yepp, hence i dared to ask o:) | |
| 15:11:44 | sean-k-mooney | this obviously still has the iso regression | |
| 15:12:48 | sean-k-mooney | and im not sure if you have alredy fixed the ami format | |
| 15:13:53 | sean-k-mooney | ok the ami regression is adressed in the backport of a previous file as it was in master | |
| 15:13:55 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/924082/1/nova/virt/images.py#163 | |
| 15:14:30 | sean-k-mooney | so the main regression is the breakgae of iso supprot form rescue, volumes and boot form iso | |
| 15:19:32 | elodilles | so you say something is still missing? | |
| 15:22:40 | sean-k-mooney | elodilles: not for the cve mitigation but the cve broke seveeral features | |
| 15:22:57 | sean-k-mooney | the one you are missing is the fixes for ISO files | |
| 15:23:32 | sean-k-mooney | elodilles: this series https://review.opendev.org/q/topic:%22format-inspector%22 | |
| 15:26:02 | elodilles | sean-k-mooney: oh, that, those are still on my TODO to backport o:) ACK | |
| 15:26:10 | elodilles | thx o/ | |
| 15:27:05 | sean-k-mooney | we backported them downstream to our trian equivelent branch with very little issues | |
| 15:27:25 | sean-k-mooney | all of thhis code is basically new so there is almost no opertunity for there to be a conflict | |
| 15:29:03 | elodilles | oh, good to hear that :] | |
| 16:00:02 | bauzas | #startmeeting nova | |
| 16:00:02 | opendevmeet | Meeting started Tue Jul 16 16:00:02 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:02 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:02 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:12 | fwiesel | o/ | |
| 16:00:14 | tkajinam | o/ | |
| 16:00:19 | bauzas | hey everyone, welcome back, it was long overdue | |
| 16:00:39 | elodilles | o/ | |
| 16:01:21 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:42 | auniyal | o/ | |
| 16:02:24 | bauzas | I guess we can softly start | |
| 16:02:42 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:49 | bauzas | #info No Critical bug | |
| 16:02:59 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:03:14 | bauzas | do people want to discuss about some specific bugs, like some now public ones :) | |
| 16:04:04 | sean-k-mooney | i think you can take that as a no | |
| 16:04:51 | bauzas | thanks btw. dansmith and sean-k-mooney for https://bugs.launchpad.net/nova/+bug/2059809 | |
| 16:05:27 | bauzas | moving on | |
| 16:05:28 | gibi | o./ | |
| 16:05:30 | sean-k-mooney | *for fixing https://bugs.launchpad.net/nova/+bug/2059809 :) elodilles did a lot fo the hevay lifting on the backports as did gibi | |
| 16:05:45 | bauzas | :) | |
| 16:05:46 | elodilles | o:) | |