| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-17 | |||
| 14:41:29 | bauzas | sean-k-mooney: i wonder if operators will scream about it | |
| 14:41:59 | bauzas | of course we can't provide different input validation based on a config option | |
| 14:42:16 | bauzas | but still, they'll have to change something probably | |
| 14:42:43 | sahid | stephenfin: o/ | |
| 14:43:03 | sahid | I can see that you are involving on osprofiler | |
| 14:43:12 | sahid | I have question for you :-) | |
| 14:43:47 | sean-k-mooney | bauzas: we are not changing the exting behavior | |
| 14:44:07 | sean-k-mooney | so if they were not doing this before we blocked FQDNs they will get the same behavior | |
| 14:44:25 | sahid | ut's related to the driver Jaeger, we would like to add an option | |
| 14:44:27 | sahid | https://github.com/openstack/osprofiler/blob/master/osprofiler/drivers/jaeger.py#L56 | |
| 14:44:40 | sean-k-mooney | bauzas: that is why we orginally did not mention it in the sepc | |
| 14:44:47 | sean-k-mooney | we did dicuss this option in hte past | |
| 14:45:02 | sahid | basically the point will be to have a prefix for the service_name, so we can make a difference between for example services that are running on different region | |
| 14:45:38 | sahid | does that mae sense for you if I add an option like service_prefix | |
| 14:45:59 | sean-k-mooney | bauzas: so i dont think operators will be upset that we are mainting the behvior they expect | |
| 14:46:43 | sahid | from my understanding that one will be only useful for jaeger, so I'm considering adding a section [jaager] | |
| 14:46:45 | bauzas | sean-k-mooneytechnically I agree | |
| 14:46:52 | bauzas | sean-k-mooney: technically I agree | |
| 14:46:57 | bauzas | we never supported FQDNs | |
| 14:47:20 | sean-k-mooney | and those that used dispaly name with an fqdn hand it modifed by the config option | |
| 14:47:29 | bauzas | so when passing a hostname, cloud-init was getting a FQDN based on the hostname + the default domain name from the option | |
| 14:47:35 | sean-k-mooney | so if they wanted it to not be modifed they already had to set the config option to the empty string | |
| 14:47:37 | bauzas | which was consistent | |
| 14:47:41 | artom | bauzas, fixed | |
| 14:47:45 | kashyap | Unrelated: Is the "nova-tox-functional-py38" job passing reliably for everyone? - it's still failing with "TypeError: getresponse() got an unexpected keyword argument 'buffering' | |
| 14:47:49 | kashyap | " | |
| 14:48:28 | sean-k-mooney | bauzas: when stephen added --hostname we also added the dhcp_doamin to the dns name in neutron | |
| 14:48:37 | sean-k-mooney | we did nto modify what we put in the metadtaa | |
| 14:48:48 | sean-k-mooney | and we are not going to with artoms code because he is not changing that | |
| 14:48:59 | sean-k-mooney | it was only the value in the neutron port that was changed | |
| 14:49:43 | bauzas | you know what ? I'll play the ostrich about any kind of FQDN questions | |
| 14:50:01 | bauzas | once artom uploads his change, I'll review it and I'm done | |
| 14:50:04 | sean-k-mooney | bauzas: dont feel like you cant ask them | |
| 14:50:24 | sean-k-mooney | im just saying we are intentiolly not changing the behvior to not break anyone | |
| 14:50:37 | sean-k-mooney | and because we dont want cofnig dirven api behavior | |
| 14:50:43 | bauzas | the ship has sailed | |
| 14:50:47 | sean-k-mooney | so without removign the option entirly which would impact everyone | |
| 14:50:55 | sean-k-mooney | we cant really do much else | |
| 14:51:07 | bauzas | but honestly, I liked the fact that we were saying domain names was something unrelated to nova :) | |
| 14:51:26 | sean-k-mooney | yep so did i | |
| 14:51:38 | sean-k-mooney | this is the compromsie so that we dont have to care about them again once done | |
| 14:52:11 | sean-k-mooney | like all good compromises it does not make anyone happy but we can all live with it | |
| 14:52:26 | bauzas | the ostrich theory applied to me. | |
| 14:53:20 | sean-k-mooney | i prefer magpie psychology. distract people with other shiny things that matter | |
| 14:55:34 | bauzas | I need to learn the three-card monte | |
| 14:55:58 | bauzas | 'follow your card' | |
| 14:56:56 | kashyap | bauzas: sean-k-mooney: There's a thing called "Belgian compromise", which loosely means: | |
| 14:56:57 | sean-k-mooney | bauzas: added https://blueprints.launchpad.net/nova/+spec/default-ephemeral-format-unformated to meeting adjenda | |
| 14:57:00 | kashyap | (quote) | |
| 14:57:01 | kashyap | complex issues are settled by conceding something to every party concerned, through an agreement that is usually so complicated that nobody completely understands all its implications. | |
| 14:57:05 | kashyap | (/quote) | |
| 14:57:28 | sean-k-mooney | hehe ya sounds like a typical eu threaty | |
| 14:58:17 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up skip compareCPU() with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 14:59:14 | sahid | stephenfin: https://bugs.launchpad.net/osprofiler/+bug/2003092 | |
| 15:24:50 | artom | bauzas, so I should just remove the ( ) about i18n? | |
| 15:32:35 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/845748 | |
| 15:38:00 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up allow skiping compareCPU() with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 15:49:15 | bauzas | artom: yup IMHO | |
| 15:50:16 | bauzas | as a reminder nova meeting here in 10 mins | |
| 15:59:44 | opendevreview | Artom Lifshitz proposed openstack/nova master: Microversion 2.94: FQDN in hostname https://review.opendev.org/c/openstack/nova/+/869812 | |
| 16:00:19 | bauzas | #startmeeting nova | |
| 16:00:19 | opendevmeet | Meeting started Tue Jan 17 16:00:19 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:19 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:19 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:24 | bauzas | gdi, just in time | |
| 16:00:32 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:36 | bauzas | hi everyone | |
| 16:00:40 | dansmith | o/ | |
| 16:01:11 | elodilles | o/ | |
| 16:01:59 | gibi | o/ | |
| 16:02:00 | bauzas | okay let's start | |
| 16:02:06 | gibi | (I'm a bit distracted) | |
| 16:02:15 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:30 | bauzas | #info One critical bug | |
| 16:02:33 | bauzas | #info One critical bug | |
| 16:02:38 | Uggla | o/ | |
| 16:03:08 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2002951 | |
| 16:03:20 | bauzas | gibi: I marked this one as critical for the sake of the discussion | |
| 16:03:22 | gmann | o/ | |
| 16:03:29 | bauzas | but we can put it back to High | |
| 16:03:49 | bauzas | in general, I tend to triage CI bugs to Critical until we agree this is not holding the gate | |
| 16:04:04 | bauzas | do we want to discuss about it now or no ? | |
| 16:04:38 | gibi | sure | |
| 16:04:57 | gibi | I updated the bug | |
| 16:05:01 | bauzas | ok, so, gibi (mostly) and I looked at this one today | |
| 16:05:05 | gibi | I think it is tempest.api.compute.admin.test_volume.AttachSCSIVolumeTestJSON.test_attach_scsi_disk_with_config_drive test case that tirggers the OOM | |
| 16:05:11 | bauzas | yeah | |
| 16:05:17 | bauzas | and like I said, I tried to find wherer | |
| 16:06:17 | bauzas | but I wasn't able to see | |
| 16:06:34 | bauzas | context : https://github.com/openstack/tempest/blob/7c8b49becef78a257e2515970a552c84982f59cd/tempest/api/compute/admin/test_volume.py#L84-L120 | |
| 16:06:46 | bauzas | we try to create an image | |
| 16:06:52 | bauzas | then we create an instance | |
| 16:07:06 | bauzas | and then a volume which we attach to the instance | |
| 16:07:44 | gibi | I haven't had time to look into the actual tc yet | |
| 16:07:45 | sean-k-mooney | p/ | |
| 16:08:08 | gibi | also it would be nice to see how the python interpreter rss size grows during the test execution | |
| 16:08:11 | dansmith | yeah surely seems like a benign test case | |
| 16:08:38 | sean-k-mooney | we unfortuently dont have the memtacker stuff form devstack | |
| 16:08:50 | sean-k-mooney | btu it would be nice if we coudl get that and also dmsg in the tox based tests | |
| 16:08:52 | bauzas | I tried to grep the testname in n-api | |