14:00:01 <rafaelweingartner> #startmeeting cloudkitty
14:00:01 <opendevmeet> Meeting started Mon Oct  3 14:00:01 2022 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:01 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:01 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:00:11 <rafaelweingartner> Hello guys!
14:00:12 <rafaelweingartner> Roll count
14:00:32 <rafaelweingartner> \O
14:01:09 <mkarpiarz> Hi!
14:01:17 <priteau> o/
14:01:36 <rafaelweingartner> How are you guys doing today?
14:03:09 <priteau> Fine, and you
14:03:38 <rafaelweingartner> Fine as well :)
14:03:54 <mkarpiarz> :)
14:04:14 <rafaelweingartner> #topic PTG meeting (October 2022 PTG) -- October 17-20, 2022
14:04:45 <rafaelweingartner> We scheduled the PTG for CloudKitty to take place on October 17
14:04:58 <rafaelweingartner> Therefore, guys, do not forget to register at #link https://openinfra-ptg.eventbrite.com
14:05:12 <mkarpiarz> Already done. :)
14:05:15 <rafaelweingartner> Also, do not forget to post your ideas at: #link https://etherpad.opendev.org/p/oct2022-ptg-cloudkitty
14:05:27 <rafaelweingartner> PTG Schedule: https://ptg.opendev.org/ptg.html
14:06:17 <mkarpiarz> Ah, thanks for this reminder!
14:08:18 <rafaelweingartner> Welcome! Moving on to the reviews.
14:08:29 <rafaelweingartner> #topic Target review of the meeting
14:08:45 <rafaelweingartner> We seem to have one open issue from last meeting: #link https://review.opendev.org/c/openstack/cloudkitty/+/858341
14:09:04 <rafaelweingartner> There was a Zuul upgrade which has made queue declarations invalid unless define at the project level
14:09:22 <rafaelweingartner> This is one of the patches that we were not able to merge right away due to a dependency issue
14:09:26 <rafaelweingartner> I tried to work around it
14:09:32 <rafaelweingartner> can you guys check it?
14:09:50 <rafaelweingartner> In main/master everything is fine already with these dependencies
14:12:55 <mkarpiarz> Should we try to merge or will this cause a failure?
14:13:14 <priteau> Left a comment
14:15:21 <rafaelweingartner> I amended the patch
14:15:35 <mkarpiarz> I'm looking at the commit which has been cherry-picked and that one looks slightly different.
14:15:47 <mkarpiarz> Could this be what's causing problems?
14:17:27 <rafaelweingartner> the changes in the cherry-picked commit are the same
14:17:42 <rafaelweingartner> the things is that, in this branch we are already suffering from legacy libraries
14:17:48 <rafaelweingartner> this was already resolved upstream
14:18:02 <rafaelweingartner> the change is basically this one: https://review.opendev.org/c/openstack/cloudkitty/+/842691/2/.zuul.yaml
14:19:59 <mkarpiarz> OK, but the original commit doesn't touch doc/requirements.txt. Is this because you've been trying to dealing with legacy libraries?
14:20:59 <rafaelweingartner> yes, it does not
14:21:09 <rafaelweingartner> because master was already fixed with upstream
14:21:33 <rafaelweingartner> if we do not touch doc/requirements, and try to apply only the commit itself, it does not pass the tests
14:21:46 <rafaelweingartner> this only happens in this specific branch due to library issues
14:23:07 <mkarpiarz> I see. And do we know which library is causing the problem?
14:23:39 <mkarpiarz> Sorry, I feel like I'm far behind you guys and trying to catch up. ;)
14:25:32 <priteau> When I run `tox -e docs` on the wallaby branch locally, it fails with
14:25:33 <priteau> ImportError: cannot import name 'environmentfilter' from 'jinja2'
14:26:13 <rafaelweingartner> it has to do with Flask, and other libraries, but to fix that, we would need to do the same as we did in master. I guess we would not want to go through that trouble, as we already fixed that in upstream branches
14:27:20 <priteau> I thought I remember seeing such issue
14:27:24 <priteau> Seen it in blazar: https://review.opendev.org/c/openstack/blazar/+/835154
14:28:03 <priteau> Note: this is *not* the right fix as it was discovered later that skipping install in docs job produced invalid config files
14:28:56 <priteau> I may have a better fix, hang on
14:32:57 <priteau> Working on it
14:34:25 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: [docs] Install cloudkitty requirements using constraints  https://review.opendev.org/c/openstack/cloudkitty/+/860096
14:34:46 <priteau> This is the proper fix, rather than adding specific requirements to doc/requirements.txt, which could get out of sync
14:35:00 <priteau> We backport this so all branches use the same configuration
14:35:20 <priteau> But for wallaby we will need to merge this patch with the pipeline setting one
14:35:30 <priteau> What do you think?
14:35:53 <rafaelweingartner> That seems to work
14:36:05 <rafaelweingartner> I thought I had tried something similar, and it had not worked
14:36:11 <rafaelweingartner> but maybe I did something wrong
14:37:08 <rafaelweingartner> I guess we can then work on this one offline
14:38:45 <mkarpiarz> Sure, if this works for you guys.
14:40:16 <rafaelweingartner> We do not have other target reviews mapped
14:40:27 <rafaelweingartner> do you guys have anything else to add, besides these two that we have been discussing?
14:41:56 <priteau> I've not had much time to work on CloudKitty recently. Hopefully more after the PTG
14:42:43 <rafaelweingartner> No worries, we have been quite busy with other issues as well
14:43:10 <rafaelweingartner> but we have started a few new implementations for Cloudkitty
14:43:23 <rafaelweingartner> therefore, in a few weeks, we will be sending the new patches
14:43:23 <mkarpiarz> I have nothing else to add.
14:44:39 <rafaelweingartner> Now, I open for general questions and topics that people might have.
14:44:45 <rafaelweingartner> #topic Open floor moment
14:46:48 <mkarpiarz> Nothing from my side.
14:48:25 <priteau> I have updated https://review.opendev.org/c/openstack/cloudkitty/+/858341
14:48:37 <rafaelweingartner> I see
14:48:38 <rafaelweingartner> thanks!
14:48:53 <priteau> Hopefully it works, let's see what Zuul says
14:49:02 <rafaelweingartner> sure
14:49:26 <rafaelweingartner> I guess, we can close the meeting for today then, and sync about the patches in the patches themselves.
14:50:00 <mkarpiarz> Sounds good.
14:50:10 <rafaelweingartner> Thank you guys for participating. Have a nice week.
14:50:14 <rafaelweingartner> #endmeeting