opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add config option to support maxphysaddr https://review.opendev.org/c/openstack/nova/+/895759 | 03:50 |
---|---|---|
opendevreview | Nobuhiro MIKI proposed openstack/nova master: scheduler: Request COMPUTE_ADDRESS_SPACE_* traits based on flavor https://review.opendev.org/c/openstack/nova/+/895761 | 03:52 |
opendevreview | Takashi Kajinami proposed openstack/nova master: Fix missing oslo.versionedobjects library option https://review.opendev.org/c/openstack/nova/+/890718 | 04:43 |
opendevreview | Dr. Jens Harbott proposed openstack/nova master: Fix URLs in status check results https://review.opendev.org/c/openstack/nova/+/895770 | 08:34 |
frickler | ^^ does this prove that nobody reads upgrade check results? ;) | 08:34 |
bauzas | frickler: operators tend to run it automatically by tools :) | 09:02 |
bauzas | and just look at the status code | 09:02 |
frickler | bauzas: yes, I also noticed that kolla-ansible only fails and doesn't even show the result. need to fix that, too | 09:11 |
opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry https://review.opendev.org/c/openstack/nova/+/894892 | 09:16 |
opendevreview | Sylvain Bauza proposed openstack/nova master: Add a Bobcat prelude section https://review.opendev.org/c/openstack/nova/+/894940 | 09:32 |
bauzas | dansmith: 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 ralonsoh | 11:07 | |
opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry https://review.opendev.org/c/openstack/nova/+/894892 | 11:47 |
opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry https://review.opendev.org/c/openstack/nova/+/894892 | 11:54 |
sean-k-mooney | bauzas: dansmith this is the patch i submitted to disabel the neutron trunk migration test | 12:39 |
sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/895655 | 12:39 |
sean-k-mooney | looks like it pased ci | 12:39 |
Uggla | sean-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-mooney | can you point me at the ptach wehre you are trying to use them | 12:46 |
bauzas | sean-k-mooney: okidoki | 12:46 |
bauzas | sean-k-mooney: I was wondering when your PTO days were starting | 12:46 |
sean-k-mooney | monday to firday next week | 12:47 |
bauzas | if you are still on, can you review https://review.opendev.org/c/openstack/nova/+/894940 ? | 12:47 |
bauzas | prelude ^ | 12:47 |
bauzas | we need it for the RC1 | 12:47 |
sean-k-mooney | so i +2'd that this morning | 12:47 |
bauzas | oh sorry, I haven't seen on the emails | 12:47 |
bauzas | for Gerrit | 12:47 |
sean-k-mooney | no worries | 12:47 |
Uggla | sean-k-mooney, yep give me a sec | 12:59 |
bauzas | Uggla: lemme look too | 13:04 |
Uggla | sean-k-mooney, bauzas changes as simple as possible here: https://github.com/uggla/nova/blob/5341a726baaa54289838655d6b231a4716872556/nova/utils.py#L913 | 13:07 |
Uggla | L913 --> 916 which is basically passing an auth with service token. | 13:07 |
bauzas | I do remember this | 13:08 |
bauzas | Uggla: interesting thoughts you made | 13:08 |
Uggla | I 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 |
bauzas | can we somehow force the endpoint discovery with the wrapper ? | 13:09 |
Uggla | bauzas, I think instead of returning None in the plugin.py method, maybe I can get add the code to get the endpoint. | 13:10 |
bauzas | in keystoneauth ? | 13:11 |
Uggla | I mean here: https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/plugin.py#L136 | 13:11 |
Uggla | yep | 13:11 |
bauzas | I was just wondering whether endpoint discovery was a public API argument | 13:11 |
sean-k-mooney | Uggla i was asking for your gerrit review | 13:13 |
sean-k-mooney | so i coudl see what you were changing | 13:13 |
sean-k-mooney | but also im not conviced you shoudl be changing it there | 13:13 |
Uggla | sean-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-mooney | so im not really expecting you to modify nova/utils.py | 13:16 |
bauzas | sean-k-mooney: afaik, Uggla did this on a testing purpose | 13:17 |
bauzas | because he was stuck with this error | 13:17 |
bauzas | so 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 exception | 13:18 |
bauzas | and if you see the problem, it's due to the endpoint | 13:18 |
sean-k-mooney | so we currently have service token support in https://github.com/openstack/nova/blob/master/nova/service_auth.py | 13:22 |
sean-k-mooney | so we shoudl be reusing that as a start | 13:24 |
sean-k-mooney | and im expectign this to be used form the manilaclient function | 13:26 |
dansmith | bauzas: done | 13:29 |
bauzas | dansmith: <3 | 13:29 |
* bauzas grabs popcorn and glasses for looking at Zuul | 13:30 | |
sean-k-mooney | i am going to redpeloy my devstack vm and Uggla do you have a local.conf for manilla | 13:31 |
bauzas | dansmith: you may be interested in unblocking our placement gate https://review.opendev.org/c/openstack/nova-specs/+/895683 | 13:33 |
Uggla | sean-k-mooney, local.conf --> https://paste.opendev.org/show/bfVUU744TgKbsFarvc97/ with ceph | 13:35 |
dansmith | bauzas: mind if I fix all the typos in the commit message first? | 13:36 |
bauzas | dansmith: shoot, I saw them but meh'd them | 13:36 |
sean-k-mooney | Uggla: 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 manilla | 13:37 |
opendevreview | Dan Smith proposed openstack/nova-specs master: nova-specs should use upper-constraints https://review.opendev.org/c/openstack/nova-specs/+/895683 | 13:37 |
opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move Bobcat implemented specs https://review.opendev.org/c/openstack/nova-specs/+/895815 | 13:45 |
bauzas | gibi: I'll work on the agenda for today's meeting but I won't be able to attend it | 13:47 |
bauzas | tl;dr: RC1 is still on flight as we need to wait for the prelude to be merged before respinning the rc1 patch | 13:48 |
bauzas | and I'll also provide an etherpad for PTG planning and I'll propose to book 4 days x 4 hours like every cycle | 13:48 |
bauzas | we could unbook the slots if we have a short list of PTG topics | 13:49 |
opendevreview | Merged openstack/nova master: Add a Bobcat prelude section https://review.opendev.org/c/openstack/nova/+/894940 | 13:50 |
bauzas | wow, that was fast \o/ | 13:53 |
sean-k-mooney | quite the ci gods will here you | 13:53 |
bauzas | well, we only ran a very few subset of jobs so it was an easy peasy :p | 13:56 |
opendevreview | Merged openstack/nova-specs master: nova-specs should use upper-constraints https://review.opendev.org/c/openstack/nova-specs/+/895683 | 14:00 |
gibi | bauzas: ack, I will drive the meeting | 14:23 |
bauzas | gibi: just updated the wiki https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 14:25 |
gibi | thanks | 14:27 |
*** mtreinish_ is now known as mtreinish | 14:43 | |
Uggla | gibi, bauzas , just to tell I could not join today's meeting. | 15:05 |
bauzas | Uggla: ditto here, I need to leave now | 15:05 |
bauzas | yet another parent-teacher meeting, now for Clémence (middle-school) | 15:06 |
gibi | ack | 15:30 |
gibi | nova meeting will start here in 8 minutes | 15:52 |
gibi | #startmeeting nova | 16:02 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:02 |
opendevmeet | The meeting name has been set to 'nova' | 16:02 |
gibi | o/ | 16:02 |
gibi | I will be your chair today as bauzas is not available | 16:02 |
gibi | do have anybody here today? | 16:04 |
gibi | if nobody is around then I won't go through the agenda | 16:07 |
sean-k-mooney | im here but.... | 16:08 |
sean-k-mooney | i only had one topic | 16:08 |
sean-k-mooney | basically do we ant to skip the trunk migration test in the ovn jobs | 16:09 |
sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/895655 | 16:09 |
gibi | I'm OK to skip it now and re-enable it once it is fixed | 16:10 |
sean-k-mooney | once its merged i can propsoe the revert | 16:10 |
gibi | yeah that is good idea | 16:10 |
sean-k-mooney | and then we can hold it until neutron tells us to merge the revert | 16:10 |
sean-k-mooney | so thats all i had | 16:10 |
gibi | the RC1 patch is already approved by bauzas https://review.opendev.org/c/openstack/releases/+/894693 | 16:11 |
gibi | so I'm not sure we want to land that on master now | 16:11 |
gibi | maybe after RC1 released | 16:11 |
gibi | I don't want to mix up the RC1 rel process | 16:11 |
sean-k-mooney | ya we can wait | 16:12 |
sean-k-mooney | i can backport it to stable if need or we can include it in rc2 | 16:12 |
gibi | ack | 16:13 |
gibi | anybody has anything else? | 16:13 |
gibi | I guess we can wrap this up | 16:15 |
gibi | #endmeeting | 16:15 |
opendevmeet | Meeting ended Tue Sep 19 16:15:40 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:15 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.html | 16:15 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.txt | 16:15 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.log.html | 16:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!