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