Monday, 2024-08-05

rafaelweingartner#startmeeting cloudkitty14:00
opendevmeetMeeting started Mon Aug  5 14:00:40 2024 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'cloudkitty'14:00
rafaelweingartnerHello guys!14:00
rafaelweingartnerRoll count14:00
rafaelweingartner\O14:00
priteauo/14:00
rafaelweingartner#topic Target reviews14:03
rafaelweingartnerWe merged #link https://review.opendev.org/c/openstack/cloudkitty/+/876643, and now, I guess we should merge #link https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/892382. 14:03
rafaelweingartnercan we proceed?14:03
priteauIf it passes tests now.14:04
rafaelweingartnerthe other code was not merged14:05
rafaelweingartnerhow does it work?14:05
opendevreviewPierre Riteau proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238214:05
priteauI've rebased it14:05
mkarpiarzOK14:05
rafaelweingartnerok, thanks14:06
rafaelweingartnerbecause the other patch was not merged, as it depends on this one14:06
rafaelweingartnerI do not know how it works in Zuul14:06
priteauWe had these issues with another patch set before14:07
priteauI am not sure we are doing it correctly14:07
rafaelweingartnerme neither14:08
rafaelweingartnerI have never worked with a circular patch as this one14:08
rafaelweingartneranyways, moving on while that one is executing14:10
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/915992, is waiting for the author response now14:10
priteauBut remember that the tempest repo is branchless14:10
rafaelweingartneralso waiting the author response #link https://review.opendev.org/c/openstack/cloudkitty/+/91547914:11
priteauSo the master branch of cloudkitty-tempest-plugin is being used to test stable branches of cloudkitty14:11
priteauYou can't just start depending on a new behaviour in master14:11
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/921339 is waiting for the author fixes/amends 14:11
rafaelweingartnerpriteau: I think that I did not get it14:14
rafaelweingartnerso, what would be the process to merge a feature/process such as this one that we have14:14
priteauProbably with some configuration variable that tells tempest which behaviour to use14:15
rafaelweingartnerbesides those patches I listed, and this one that we are discussing that is all from my side14:20
rafaelweingartnerlet's see if we can merge this one14:20
mattcreeso/ 14:21
priteaurafaelweingartner: Do you think Pedro or yourself can look at updating the tempest patch to support multiple branches?14:26
rafaelweingartnerdo we need it?14:27
priteauDid you see my messages just above?14:28
priteauThe master branch of cloudkitty-tempest-plugin is being used to test stable branches of cloudkitty14:28
priteauSo we need test code that supports all maintained versions14:28
rafaelweingartneryes, I did14:36
rafaelweingartnerbut that is what i do not understand14:36
priteauI am not sure how to explain it in other terms. Which part is unclera?14:38
priteauunclear14:38
rafaelweingartner=) openstack design sometimes make me very confusing14:38
rafaelweingartnerI see, the guys are reusing the same set of tests, and applying it to all code branches14:38
rafaelweingartnertherefore, the functional tests are appart, and when we merge something in upstream, and not in the other branches (code branches), then the functional tests do not pass14:39
rafaelweingartnertherefore, then one needs to maitain more functional tests14:39
rafaelweingartnerthen, this also needs to be propagate to other configurations I guess14:39
rafaelweingartnerso much effort to so little gain14:39
rafaelweingartnerI will discuss with Pedro, and see what we can do14:41
rafaelweingartnerhave no idea where to change things, and what to do, but we will see what needs to be done14:41
rafaelweingartnerit would be easier to version all of them together, if they depend on each other =), but go figure right... there is always a good reason to make things more complicated14:44
rafaelweingartnerI already added this here as well to us14:45
rafaelweingartnerbesides these, and the others thare are kind of stuck, I do not have anything else for today's meeting14:45
mkarpiarzI have an annoucement to make.14:45
mkarpiarzI can no longer devote time to the project on a regular basis and so I've decided I'll be stepping down as a core reviewer by the end of this month.14:46
mkarpiarzI'll still be keeping an eye on changes from time to time but won't be joining IRC meetings nor PTGs.14:46
rafaelweingartnerIt is said to see you go mkarpiarz14:46
rafaelweingartnerbut it has been a pleasure to work with you so far14:46
mkarpiarzThanks! I'm really glad to hear this and hopefully, we can find someone who will take my place.14:47
priteauSorry to have to say good bye Mariusz14:47
mkarpiarzIt's been a pleaure to work with you guys. :)14:48
mkarpiarzPlease let me know what you want to do when it comes to announcing these changes on the mailing list. Should I send a message there?14:48
mkarpiarzOr wait until we find someone who can step in?14:48
priteauSome people announce it on the list, some just go away quietly14:48
priteauTo be honest CloudKitty is a small project within OpenStack, I am not sure there is a need to send email to everyone14:49
priteauIn terms of replacement, my colleague mattcrees has been working with CloudKitty for a while. He doesn't have many reviews on this project specifically but he has good experience from other projects 14:50
priteauI would be happy to nominate him as a core reviewer14:50
rafaelweingartnerI think that he would be a great fit14:50
mattcreesThanks, I would be happy to step up 14:51
mkarpiarzThanks, Matt!14:51
mkarpiarzIn this case maybe we can send one message nominating Matt to a core reviewer and in it we mention I'll be removed from the core team.14:52
mkarpiarzAnyway, that's all from my side.14:54
rafaelweingartnerok, that might works14:55
rafaelweingartnerthanks Mariusz14:55
mkarpiarzNo, thank you all! :)14:57
rafaelweingartnerif we do not have anything else, we can close for today14:58
rafaelweingartnerI will take a look into that merge, and send/talk to you guys via e-mail then14:58
mkarpiarzOK14:59
rafaelweingartnerThank you guys for participating. Have a nice week.15:04
rafaelweingartner#endmeeting15:04
opendevmeetMeeting ended Mon Aug  5 15:04:59 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-08-05-14.00.html15:04
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-08-05-14.00.txt15:04
opendevmeetLog:            https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-08-05-14.00.log.html15:04
mattcreesCheers 15:05
opendevreviewRafael Weingartner proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238215:06
opendevreviewRafael Weingartner proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238215:07
opendevreviewPedro Henrique Pereira Martins proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238219:11
opendevreviewPedro Henrique Pereira Martins proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238219:24
opendevreviewPedro Henrique Pereira Martins proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API  https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/89238220:25

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