| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-19 | |||
| 12:39:19 | sean-k-mooney | bauzas: dansmith this is the patch i submitted to disabel the neutron trunk migration test | |
| 12:39:23 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/895655 | |
| 12:39:30 | sean-k-mooney | looks like it pased ci | |
| 12:45:58 | 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:46:32 | sean-k-mooney | can you point me at the ptach wehre you are trying to use them | |
| 12:46:38 | bauzas | sean-k-mooney: okidoki | |
| 12:46:50 | bauzas | sean-k-mooney: I was wondering when your PTO days were starting | |
| 12:47:01 | sean-k-mooney | monday to firday next week | |
| 12:47:09 | bauzas | if you are still on, can you review https://review.opendev.org/c/openstack/nova/+/894940 ? | |
| 12:47:17 | bauzas | prelude ^ | |
| 12:47:23 | bauzas | we need it for the RC1 | |
| 12:47:23 | sean-k-mooney | so i +2'd that this morning | |
| 12:47:42 | bauzas | oh sorry, I haven't seen on the emails | |
| 12:47:48 | bauzas | for Gerrit | |
| 12:47:52 | sean-k-mooney | no worries | |
| 12:59:37 | Uggla | sean-k-mooney, yep give me a sec | |
| 13:04:03 | bauzas | Uggla: lemme look too | |
| 13:07:11 | Uggla | sean-k-mooney, bauzas changes as simple as possible here: https://github.com/uggla/nova/blob/5341a726baaa54289838655d6b231a4716872556/nova/utils.py#L913 | |
| 13:07:37 | Uggla | L913 --> 916 which is basically passing an auth with service token. | |
| 13:08:33 | bauzas | I do remember this | |
| 13:08:55 | bauzas | Uggla: interesting thoughts you made | |
| 13:09:04 | 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:19 | bauzas | can we somehow force the endpoint discovery with the wrapper ? | |
| 13:10:39 | 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:11:26 | bauzas | in keystoneauth ? | |
| 13:11:39 | Uggla | I mean here: https://github.com/openstack/keystoneauth/blob/e6f3999c6f2f846e3dda505343166ab8c8346c2a/keystoneauth1/plugin.py#L136 | |
| 13:11:43 | Uggla | yep | |
| 13:11:51 | bauzas | I was just wondering whether endpoint discovery was a public API argument | |
| 13:13:05 | sean-k-mooney | Uggla i was asking for your gerrit review | |
| 13:13:17 | sean-k-mooney | so i coudl see what you were changing | |
| 13:13:38 | sean-k-mooney | but also im not conviced you shoudl be changing it there | |
| 13:14:58 | 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:16:14 | sean-k-mooney | so im not really expecting you to modify nova/utils.py | |
| 13:17:36 | bauzas | sean-k-mooney: afaik, Uggla did this on a testing purpose | |
| 13:17:59 | bauzas | because he was stuck with this error | |
| 13:18:42 | 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:54 | bauzas | and if you see the problem, it's due to the endpoint | |
| 13:22:48 | sean-k-mooney | so we currently have service token support in https://github.com/openstack/nova/blob/master/nova/service_auth.py | |
| 13:24:50 | sean-k-mooney | so we shoudl be reusing that as a start | |
| 13:26:31 | sean-k-mooney | and im expectign this to be used form the manilaclient function | |
| 13:29:45 | dansmith | bauzas: done | |
| 13:29:52 | bauzas | dansmith: <3 | |
| 13:31:58 | sean-k-mooney | i am going to redpeloy my devstack vm and Uggla do you have a local.conf for manilla | |
| 13:33:59 | bauzas | dansmith: you may be interested in unblocking our placement gate https://review.opendev.org/c/openstack/nova-specs/+/895683 | |
| 13:35:25 | Uggla | sean-k-mooney, local.conf --> https://paste.opendev.org/show/bfVUU744TgKbsFarvc97/ with ceph | |
| 13:36:33 | dansmith | bauzas: mind if I fix all the typos in the commit message first? | |
| 13:36:47 | bauzas | dansmith: shoot, I saw them but meh'd them | |
| 13:37:23 | 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:32 | opendevreview | Dan Smith proposed openstack/nova-specs master: nova-specs should use upper-constraints https://review.opendev.org/c/openstack/nova-specs/+/895683 | |
| 13:45:56 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move Bobcat implemented specs https://review.opendev.org/c/openstack/nova-specs/+/895815 | |
| 13:47:51 | bauzas | gibi: I'll work on the agenda for today's meeting but I won't be able to attend it | |
| 13:48:20 | 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:59 | 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:49:18 | bauzas | we could unbook the slots if we have a short list of PTG topics | |
| 13:50:24 | opendevreview | Merged openstack/nova master: Add a Bobcat prelude section https://review.opendev.org/c/openstack/nova/+/894940 | |
| 13:53:11 | bauzas | wow, that was fast \o/ | |
| 13:53:49 | sean-k-mooney | quite the ci gods will here you | |
| 13:56:34 | bauzas | well, we only ran a very few subset of jobs so it was an easy peasy :p | |
| 14:00:10 | opendevreview | Merged openstack/nova-specs master: nova-specs should use upper-constraints https://review.opendev.org/c/openstack/nova-specs/+/895683 | |
| 14:23:11 | gibi | bauzas: ack, I will drive the meeting | |
| 14:25:58 | bauzas | gibi: just updated the wiki https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 14:27:48 | gibi | thanks | |
| 15:05:16 | Uggla | gibi, bauzas , just to tell I could not join today's meeting. | |
| 15:05:46 | bauzas | Uggla: ditto here, I need to leave now | |
| 15:06:27 | bauzas | yet another parent-teacher meeting, now for Clémence (middle-school) | |
| 15:06:27 | bauzas | yet another parent-teacher meeting, now for Clémence (middle-school) | |
| 15:30:29 | gibi | ack | |
| 15:52:59 | gibi | nova meeting will start here in 8 minutes | |
| 16:02:27 | gibi | #startmeeting nova | |
| 16:02:27 | 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:27 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:02:27 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:02:29 | gibi | o/ | |
| 16:02:46 | gibi | I will be your chair today as bauzas is not available | |
| 16:04:57 | gibi | do have anybody here today? | |
| 16:07:03 | gibi | if nobody is around then I won't go through the agenda | |
| 16:08:00 | sean-k-mooney | im here but.... | |
| 16:08:04 | sean-k-mooney | i only had one topic | |
| 16:09:32 | sean-k-mooney | basically do we ant to skip the trunk migration test in the ovn jobs | |
| 16:09:48 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/895655 | |
| 16:10:00 | gibi | I'm OK to skip it now and re-enable it once it is fixed | |
| 16:10:15 | sean-k-mooney | once its merged i can propsoe the revert | |
| 16:10:22 | gibi | yeah that is good idea | |
| 16:10:32 | sean-k-mooney | and then we can hold it until neutron tells us to merge the revert | |
| 16:10:50 | sean-k-mooney | so thats all i had | |
| 16:11:20 | gibi | the RC1 patch is already approved by bauzas https://review.opendev.org/c/openstack/releases/+/894693 | |
| 16:11:34 | gibi | so I'm not sure we want to land that on master now | |
| 16:11:47 | gibi | maybe after RC1 released | |
| 16:11:57 | gibi | I don't want to mix up the RC1 rel process | |
| 16:12:33 | sean-k-mooney | ya we can wait | |
| 16:12:54 | sean-k-mooney | i can backport it to stable if need or we can include it in rc2 | |
| 16:13:09 | gibi | ack | |
| 16:13:21 | gibi | anybody has anything else? | |
| 16:15:20 | gibi | I guess we can wrap this up | |
| 16:15:40 | gibi | #endmeeting | |
| 16:15:40 | 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:40 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.html | |
| 16:15:40 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.txt | |
| 16:15:40 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-19-16.02.log.html | |
| #openstack-nova - 2023-09-20 | |||
| 00:19:03 | opendevreview | Jorhson Deng proposed openstack/nova master: Use the VersionedObjectSerializer as parent class in NovaObjectRegistry https://review.opendev.org/c/openstack/nova/+/894892 | |