Tuesday, 2023-09-19

opendevreviewNobuhiro MIKI proposed openstack/nova master: libvirt: Add config option to support maxphysaddr  https://review.opendev.org/c/openstack/nova/+/89575903:50
opendevreviewNobuhiro MIKI proposed openstack/nova master: scheduler: Request COMPUTE_ADDRESS_SPACE_* traits based on flavor  https://review.opendev.org/c/openstack/nova/+/89576103:52
opendevreviewTakashi Kajinami proposed openstack/nova master: Fix missing oslo.versionedobjects library option  https://review.opendev.org/c/openstack/nova/+/89071804:43
opendevreviewDr. Jens Harbott proposed openstack/nova master: Fix URLs in status check results  https://review.opendev.org/c/openstack/nova/+/89577008:34
frickler^^ does this prove that nobody reads upgrade check results? ;)08:34
bauzasfrickler: operators tend to run it automatically by tools :)09:02
bauzasand just look at the status code09:02
fricklerbauzas: yes, I also noticed that kolla-ansible only fails and doesn't even show the result. need to fix that, too09:11
opendevreviewJorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry  https://review.opendev.org/c/openstack/nova/+/89489209:16
opendevreviewSylvain Bauza proposed openstack/nova master: Add a Bobcat prelude section  https://review.opendev.org/c/openstack/nova/+/89494009:32
bauzasdansmith: melwitt: gmann: this is way early for you, but could you guys swing quickly https://review.opendev.org/c/openstack/nova/+/894940 so we could have a late-but-hopefully-soon RC1 ?09:36
*** ralonsoh_ is now known as ralonsoh11:07
opendevreviewJorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry  https://review.opendev.org/c/openstack/nova/+/89489211:47
opendevreviewJorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry  https://review.opendev.org/c/openstack/nova/+/89489211:54
sean-k-mooneybauzas: dansmith  this is the patch i submitted to disabel the neutron trunk migration test12:39
sean-k-mooneyhttps://review.opendev.org/c/openstack/nova/+/89565512:39
sean-k-mooneylooks like it pased ci12:39
Ugglasean-k-mooney, bauzas, I wrote what I have found regarding the issue with service token. If you have a look --> https://paste.opendev.org/show/blcYEIcNRW24u8lVOqIH/12:45
sean-k-mooneycan you point me at the ptach wehre you are trying to use them12:46
bauzassean-k-mooney: okidoki12:46
bauzassean-k-mooney: I was wondering when your PTO days were starting12:46
sean-k-mooneymonday to firday next week12:47
bauzasif you are still on, can you review https://review.opendev.org/c/openstack/nova/+/894940 ?12:47
bauzasprelude ^12:47
bauzaswe need it for the RC112:47
sean-k-mooneyso i +2'd that this morning12:47
bauzasoh sorry, I haven't seen on the emails12:47
bauzasfor Gerrit12:47
sean-k-mooneyno worries12:47
Ugglasean-k-mooney, yep give me a sec12:59
bauzasUggla: lemme look too13:04
Ugglasean-k-mooney, bauzas changes as simple as possible here: https://github.com/uggla/nova/blob/5341a726baaa54289838655d6b231a4716872556/nova/utils.py#L91313:07
UgglaL913 --> 916 which is basically passing an auth with service token.13:07
bauzasI do remember this13:08
bauzasUggla: interesting thoughts you made13:08
UgglaI think I can adapt the method in plugin.py, based on the one in identity to get the proper endpoint. But I don't know if that's a correct approach 13:09
bauzascan we somehow force the endpoint discovery with the wrapper ?13:09
Ugglabauzas, I think instead of returning None in the plugin.py method, maybe I can get add the code to get the endpoint.13:10
bauzasin keystoneauth ?13:11
UgglaI mean here: https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/plugin.py#L13613:11
Ugglayep13:11
bauzasI was just wondering whether endpoint discovery was a public API argument13:11
sean-k-mooneyUggla  i was asking for your gerrit review13:13
sean-k-mooneyso i coudl see what you were changing13:13
sean-k-mooneybut also im not conviced you shoudl be changing it there13:13
Ugglasean-k-mooney, so far I have not pushed to gerrit because it was not working. But I gave you the changes in my github.13:14
sean-k-mooneyso im not really expecting you to modify nova/utils.py13:16
bauzassean-k-mooney: afaik, Uggla did this on a testing purpose13:17
bauzasbecause he was stuck with this error13:17
bauzasso the code is damn simple and shouldn't probably be automatically creating a service token for *all* calls but for the moment, Uggla just wonders *why* keystoneauth gets an exception13:18
bauzasand if you see the problem, it's due to the endpoint13:18
sean-k-mooneyso we currently have service token support in https://github.com/openstack/nova/blob/master/nova/service_auth.py13:22
sean-k-mooneyso we shoudl be reusing that as a start13:24
sean-k-mooneyand im expectign this to be used form the manilaclient function13:26
dansmithbauzas: done13:29
bauzasdansmith: <313:29
* bauzas grabs popcorn and glasses for looking at Zuul13:30
sean-k-mooneyi am going to redpeloy my devstack vm and Uggla  do you have a local.conf for manilla13:31
bauzasdansmith: you may be interested in unblocking our placement gate https://review.opendev.org/c/openstack/nova-specs/+/89568313:33
Ugglasean-k-mooney, local.conf  --> https://paste.opendev.org/show/bfVUU744TgKbsFarvc97/   with ceph13:35
dansmithbauzas: mind if I fix all the typos in the commit message first?13:36
bauzasdansmith: shoot, I saw them but meh'd them13:36
sean-k-mooneyUggla: cool im going to update all the packages in the vm reboot and do a vaniall devstack isntall first then ill restack with ceph and manilla13:37
opendevreviewDan Smith proposed openstack/nova-specs master: nova-specs should use upper-constraints  https://review.opendev.org/c/openstack/nova-specs/+/89568313:37
opendevreviewSylvain Bauza proposed openstack/nova-specs master: Move Bobcat implemented specs  https://review.opendev.org/c/openstack/nova-specs/+/89581513:45
bauzasgibi: I'll work on the agenda for today's meeting but I won't be able to attend it13:47
bauzastl;dr: RC1 is still on flight as we need to wait for the prelude to be merged before respinning the rc1 patch13:48
bauzasand I'll also provide an etherpad for PTG planning and I'll propose to book 4 days x 4 hours like every cycle13:48
bauzaswe could unbook the slots if we have a short list of PTG topics13:49
opendevreviewMerged openstack/nova master: Add a Bobcat prelude section  https://review.opendev.org/c/openstack/nova/+/89494013:50
bauzaswow, that was fast \o/13:53
sean-k-mooneyquite the ci gods will here you13:53
bauzaswell, we only ran a very few subset of jobs so it was an easy peasy :p13:56
opendevreviewMerged openstack/nova-specs master: nova-specs should use upper-constraints  https://review.opendev.org/c/openstack/nova-specs/+/89568314:00
gibibauzas: ack, I will drive the meeting14:23
bauzasgibi: just updated the wiki https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting14:25
gibithanks14:27
*** mtreinish_ is now known as mtreinish14:43
Ugglagibi, bauzas , just to tell I could not join today's meeting.15:05
bauzasUggla: ditto here, I need to leave now15:05
bauzasyet another parent-teacher meeting, now for Clémence (middle-school)15:06
gibiack15:30
gibinova meeting will start here in 8 minutes15:52
gibi#startmeeting nova16:02
opendevmeetMeeting started Tue Sep 19 16:02:27 2023 UTC and is due to finish in 60 minutes.  The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot.16:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:02
opendevmeetThe meeting name has been set to 'nova'16:02
gibio/16:02
gibiI will be your chair today as bauzas is not available16:02
gibido have anybody here today?16:04
gibiif nobody is around then I won't go through the agenda16:07
sean-k-mooneyim here but....16:08
sean-k-mooneyi only had one topic16:08
sean-k-mooneybasically do we ant to skip the trunk migration test in the  ovn jobs 16:09
sean-k-mooneyhttps://review.opendev.org/c/openstack/nova/+/89565516:09
gibiI'm OK to skip it now and re-enable it once it is fixed16:10
sean-k-mooneyonce its merged i can propsoe the revert16:10
gibiyeah that is good idea16:10
sean-k-mooneyand then we can hold it until neutron tells us to merge the revert16:10
sean-k-mooneyso thats all i had16:10
gibithe RC1 patch is already approved by bauzas https://review.opendev.org/c/openstack/releases/+/894693 16:11
gibiso I'm not sure we want to land that on master now16:11
gibimaybe after RC1 released16:11
gibiI don't want to mix up the RC1 rel process16:11
sean-k-mooneyya we can wait16:12
sean-k-mooneyi can backport it to stable if need or we can include it in rc216:12
gibiack16:13
gibianybody has anything else?16:13
gibiI guess we can wrap this up16:15
gibi#endmeeting16:15
opendevmeetMeeting ended Tue Sep 19 16:15:40 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:15
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.html16:15
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.txt16:15
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.log.html16:15

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!