Monday, 2020-10-12

*** zzzeek has quit IRC00:20
*** zzzeek has joined #openstack-oslo00:23
*** rcernin has quit IRC00:26
*** rcernin has joined #openstack-oslo00:54
*** cognifloyd has quit IRC01:15
*** rcernin has quit IRC04:47
*** rcernin has joined #openstack-oslo05:18
*** sboyron_ has joined #openstack-oslo05:39
*** tosky has joined #openstack-oslo07:03
*** rcernin has quit IRC07:26
*** brinzhang has joined #openstack-oslo07:32
*** rpittau|afk is now known as rpittau07:35
*** rcernin has joined #openstack-oslo08:10
hberaudmoguimar, smcginnis: please can you push +W (if everything seems ok for you) we have an urgent bug downstream related to that08:15
hberaudmoguimar: FYI normally with backports on stable branches a single approval from core is enough on oslo08:16
*** rcernin has quit IRC08:33
*** brinzhang_ has joined #openstack-oslo08:33
*** brinzhang has quit IRC08:35
*** elod is now known as elod_pto08:41
*** dtantsur|afk is now known as dtantsur09:03
*** openstackgerrit has joined #openstack-oslo09:35
openstackgerritWenping Song proposed openstack/oslo.config master: Use TOX_CONSTRAINTS_FILE instead of obsolete UPPER_CONSTRAINTS_FILE  https://review.opendev.org/75751509:35
*** ralonsoh has joined #openstack-oslo09:39
*** iurygregory has quit IRC09:57
*** iurygregory has joined #openstack-oslo09:58
moguimarhberaud, which patch?10:44
hberaudmoguimar: lol sorry I forgot the link https://review.opendev.org/#/c/757033/10:47
moguimarok, +W10:48
hberaudmoguimar: thanks10:48
*** Luzi has joined #openstack-oslo10:52
openstackgerritWenping Song proposed openstack/oslo.config master: Remove Babel requirement  https://review.opendev.org/75753411:26
*** hberaud has quit IRC11:30
*** hberaud has joined #openstack-oslo11:35
*** brinzhang_ has quit IRC11:37
*** brinzhang_ has joined #openstack-oslo11:37
openstackgerritZihao Wang proposed openstack/osprofiler master: Remove six.moves  https://review.opendev.org/75754011:40
openstackgerritMerged openstack/tooz master: Blacklist etcd3gw 0.2.6  https://review.opendev.org/75508612:02
moguimarhberaud, stephenfin wanna push this one?12:05
hberaudmoguimar:  https://review.opendev.org/755086 ?12:07
moguimarhttps://review.opendev.org/#/c/74215512:07
moguimarhberaud ^12:08
moguimarforgot the link too xD12:08
hberaudmoguimar: ahaha np, I'll validate this one, but I prefer to let stephenfin push the +W button to avoid conflicts of interest12:09
moguimarhberaud, I think there was another one you +W12:10
moguimarand after 35 patches on the same topic, I think we're quite safe12:10
moguimarpbr is going to be the tough one though12:11
hberaudmoguimar: yep but I wasn't the publisher of the last PS12:11
hberaudyep I agree I think we are safe12:11
moguimarI mean this one12:12
moguimarhttps://review.opendev.org/#/c/742152/12:12
moguimarah, the PS12:12
openstackgerritMerged openstack/oslo.config stable/victoria: sphinxext: Add 'merge_domaindata'  https://review.opendev.org/75337912:14
hberaudmoguimar: so I think you're right we can go ahead I fast approved this one https://review.opendev.org/#/c/742155/512:14
moguimarhberaud, can you check Clark Boylan's comment on https://review.opendev.org/#/c/742160/12:15
moguimarand https://review.opendev.org/#/c/757309/212:15
hberaudomw12:15
hberaudmoguimar: any opinion to bring there? => https://review.opendev.org/#/c/742160/9/tox.ini I'll drop these changes12:20
moguimarhberaud, removing that line defeats the purpose of using pre-commit12:22
openstackgerritMerged openstack/oslo.privsep stable/train: Make compatible with msgpack 1.0.0  https://review.opendev.org/75703312:22
hberaudmoguimar: it depends... we still have it available here, but our CI wouldn't relying on it12:30
hberaudmoguimar: my main concern is more in the formalization of this part through our entire scope12:31
*** lbragstad has joined #openstack-oslo12:39
*** lbragstad has quit IRC12:40
*** lbragstad has joined #openstack-oslo12:46
*** kgiusti has joined #openstack-oslo12:59
*** Luzi has quit IRC13:26
*** iurygregory has quit IRC13:27
*** iurygregory has joined #openstack-oslo13:28
openstackgerritMerged openstack/tooz master: Adding pre-commit  https://review.opendev.org/74215513:42
moguimarhberaud, sorry I was out for lunch13:43
moguimarand now pbr is the last one of them =D13:44
moguimarI think we can bring this patch for the meeting13:44
hberaudmoguimar: good idea13:46
*** dave-mccowan has joined #openstack-oslo14:27
hberaud#startmeeting oslo15:00
openstackMeeting started Mon Oct 12 15:00:09 2020 UTC and is due to finish in 60 minutes.  The chair is hberaud. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: oslo)"15:00
openstackThe meeting name has been set to 'oslo'15:00
hberaudCourtesy ping for bnemec, smcginnis, moguimar, johnsom, stephenfin, bcafarel, kgiusti, jungleboyj, sboyron15:00
beekneemecho/15:00
*** beekneemech is now known as bnemec15:00
hberaud#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:00
stephenfino/15:00
bcafarelo/15:00
johnsomo/15:00
kgiustio/15:01
moguimaro/15:01
sboyron_o/15:01
hberaud#topic Red flags for/from liaisons15:03
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:03
*** sboyron_ is now known as sboyron15:03
bcafarelall good for victoria on neutron side15:04
moguimarnothing from Barbican15:04
johnsomNothing from Octavia15:04
hberaud I don't expect a whole lot of activity on this right now, final release is really near now.15:05
hberaud#topic Releases15:06
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:06
hberaudAs said we are now on the final sprint for victoria15:06
hberaud#topic Action items from last meeting15:07
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:07
hberaudNothing new to bring here either15:08
hberaud#topic pre-commit on PBR15:08
*** openstack changes topic to "pre-commit on PBR (Meeting topic: oslo)"15:08
hberaudas moguimar suggested it could be worth to speak a bit about this https://review.opendev.org/#/c/742160/9/tox.ini15:09
moguimar1st thing is pbr still tests tempest-full, and according to frickler, we can drop it15:10
hberaud99% of the pre-commit patches are now merged but Clark suggested to avoid to call pre-commit with tox15:10
hberaudmoguimar: good point15:10
moguimarwhich I created a patch to drop it here: https://review.opendev.org/#/c/757309/215:11
bnemecI would assume the reason for running tempest is to verify that the things installed with pbr are actually working.15:11
bnemecUnless we're verifying that some other way.15:11
bnemecOh, I see.15:12
bnemecIt's a python 2 thing, not dropping tempest completely.15:12
moguimaryeah, tempest-full-py3 is still there15:12
hberaudyep15:12
sboyronyes15:12
bnemecHmm, that's a bit problematic though. pbr is branchless so we can't just drop py2 on older releases.15:13
hberaudmoguimar: so if I understand it correctly it will allow us to continue with our stuff and on the other hand we will keep some testing for py2.7, exact?15:14
moguimarthis second patch is just to drop tempest-full for python215:15
moguimaras suggested by frickler15:15
bnemecAh, I see Clark suggested a solution to that in his comment.15:15
moguimarafter I fixed requirements.txt to only install pre-commit for python3+ we don't depend on that patch anymore15:16
hberaudI see15:16
hberaudI remember this part now15:17
moguimarthe pre-commit patch was failing cause there is no pre-commit 2.6.0 in py2715:17
moguimarand it is in test-requirements15:17
moguimarbut is only used in the pep8 env15:17
moguimarwhich is py3+15:18
hberaudyep15:18
hberaudconcerning the cache part I asked some feedback from Clark especially on the uniformity of these patches15:19
hberaudbut I don't expect a positive response15:19
hberaudI mean I think he will give the priority to the cache aspects15:20
openstackgerritMoisés Guimarães proposed openstack/pbr master: Adding pre-commit  https://review.opendev.org/74216015:21
moguimarI'm rebasing it without the tempest-full drop15:21
hberaudnice15:21
bnemecPulling from github in ci is pretty bad.15:22
moguimarso now we have 35 bad patches15:22
hberaudlol15:22
bnemecIs there any chance of getting pre-commit to install from pypi packages instead of source?15:23
moguimarcould we work with upstream pre-commit to also accept a package?15:23
hberaudbnemec: we chosen this way as some persons asked for more security by avoiding to pull unchecked versions15:25
hberaudbnemec: so we proposed to pull a specific commit rather than a specific version15:25
hberaudmoguimar: which package?15:26
bnemecHow is installing from source more secure than installing from a release?15:26
moguimarwe would need flake8 and the pre-commit hoops15:26
moguimarhooks15:26
hberaudmoguimar: yep15:27
moguimarso we would need https://github.com/pre-commit/pre-commit-hooks15:27
moguimarI'm not sure if it is relased as a package15:27
moguimarthing is pre-commit was always from source15:28
moguimarthen we pinned the versions15:28
moguimarand someone complained that tags could be relabeled15:28
moguimarso we pinned to commit hash15:28
hberaudpre-commit and flake8 are mainstream projects so if a security hole appear I think it will appear on pypi too15:30
moguimarI think we are losing the point15:30
hberaudyep15:30
moguimarso, there is no install from pypi now15:31
moguimarwe only have source install15:31
hberaudmoguimar: do you want to lead some related actions this week?15:31
moguimarhow do we go on from here?15:31
moguimarI can reach out to pre-commit folks upsream15:31
hberaudawesome15:31
bnemecThat would be a good first step.15:31
bnemecThen depending on how that discussion goes we can figure out what to do.15:32
hberaud#action moguimar to reach pre-commit people15:32
hberaud+115:32
hberaudanything else about this?15:32
moguimarI guess we can go on15:32
hberaud#topic Weekly Wayward Wallaby Review15:33
*** openstack changes topic to "Weekly Wayward Wallaby Review (Meeting topic: oslo)"15:33
hberaud#link https://review.opendev.org/#/c/640057/15:33
hberaudwe are now branched so I assume that we could continue with that15:34
bnemecIt's only in the generator, so I think it's unlikely we would have an 11th hour bug fix touching this code.15:35
hberaud#topic Open discussion15:35
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:35
hberaudanything else?15:35
bnemecYep, a couple of things.15:35
bnemecFirst, the PTG.15:35
bnemecCurrently the only topic on the etherpad is the retrospective, and a cross-project for the policy popup which will happen elsewhere.15:36
bnemecSo at this point I'm thinking we don't need to meet at all, if there's nothing to discuss.15:36
hberaudagreed15:37
moguimarnot even to see each other on video instead of text =P15:37
hberaud:)15:37
bnemecI mean, it's during the regular meeting time so we could also jump on a meetpad. :-)15:37
moguimarwe can also discuss the DPL15:37
bnemecThat was actually my next topic.15:37
hberaudmoguimar: I planed to bring this point here15:38
bnemecWe'll need to at least assign official liaisons for the three required positions.15:38
hberaud#link https://governance.openstack.org/tc/resolutions/20200803-distributed-project-leadership.html#process-for-opting-in-to-distributed-leadership15:38
hberaudas said I'm volunteer for the release liaison but at least we need more than that, any volunteer?15:39
hberaud(and I'm volunteer on the meeting facilitator role too)15:40
bnemecI guess I can stay on as the security liaison. It's a pretty low time commitment and I'm already admin of the oslo-coresec team. ;-)15:40
hberaudbnemec: awesome, thanks15:41
moguimarso what is the 3rd missing?15:41
bnemectact-sig (essentially, infra)15:41
hberaudany volunteer?15:42
sboyronI'm not a core, so I won't be official tag-sig but I can help15:44
damanijust one question it's necesarry be core to be liaison ?15:44
bnemecI'm probably not a good choice for that one since I can't commit to dealing with ci fire drills in a timely fashion anymore.15:44
bnemecNot necessarily, but it's helpful in case there are patches that need to be pushed through.15:45
johnsomYeah, sorry, but I am already over extended.15:45
bnemecHowever, as long as there are cores available that you can ping it would probably be okay to have a non-core as the liaison here.15:45
hberaudsboyron, damani: I've no idea about if we need to be core for this job, I guess yes, but we could ask to TC members about this point15:46
bnemecAnd, frankly, if no cores volunteer then it's clearly better to have a non-core than no liaison at all. :-)15:46
hberaudI agreed with bnemec15:46
bnemecI could also see it being a good path to becoming a core, if that's a goal you have.15:47
damanisboyron, are you volunteer ?15:47
hberaudsboyron: so you're interested by the tact-sig?15:47
bnemecFighting ci fires will teach you a lot about how the projects work and interact.15:47
hberauddamani: are you interesting by the release liaison job?15:47
damanihberaud, yes15:48
hberauds/interesting/interested15:48
damanii'm really interested15:48
hberauddamani: nice15:48
moguimardamani++15:48
sboyronhberaud I am really interrested to help15:48
bnemecIt would be good to have more than one release liaison, for the same reason we always had two people who could ack release requests in the past.15:48
bnemec(PTL + liaison)15:48
hberaudsboyron: awesome15:48
moguimarI can help hberaud with the releases15:49
hberaudmoguimar: damani is volunteer too15:49
moguimarah, I see15:49
sboyronI'll need some help since I do not know all process yet, but tact-sig or release liaison are some interresting subject15:49
bnemecHow do we want to keep track of this? Maybe a policy spec doc so we can review proposed liaisons in the normal way?15:49
moguimarI thought he was volunteering for tact-sig15:49
hberaudmoguimar: maybe you can create a pair tact-sig with sboyron15:50
moguimarsounds good, although I have no idea at all of tact-sig responsibilities15:50
hberauda pair of core/non-core and mentoring sboyron and damani15:50
sboyronit's ok for me to work in a pair, sound goods15:50
hberaudand help them to become core developer on oslo with us15:51
bnemecmoguimar: Basically be the point of contact for infra if something in our ci jobs breaks.15:51
bnemecIt doesn't necessarily mean you have to fix all the ci breakages, but it helps if you know who to pull in to fix them.15:51
moguimarsboyron, what is your timezone?15:51
sboyronCEST15:51
damanihberaud, moguimar, sounds really good15:51
sboyronwhat about yours moguimar ?15:51
moguimarsame I guess15:51
moguimarCzech Republic15:52
hberaudexcellent! I think we reached a great step15:52
sboyronsame yes, great15:52
bnemecThinking some more on this, maybe it's better to keep track of our liaisons on the wiki.15:52
bnemecWe already have a contact list here: https://wiki.openstack.org/wiki/Oslo#The_Oslo_Team15:52
hberaudjohnsom: are you interested for some of these roles?15:53
bnemecWe could add a liaisons section to that.15:53
hberaudbnemec: good idea15:53
hberaudbnemec: do you want to lead that?15:53
johnsomhberaud I can continue as a TaskFlow core and the security group, but I cannot take on new responsibilities at this time.15:53
bnemecYeah, I can do that quick.15:54
hberaudjohnsom: ack np15:54
hberaudbnemec: awesome thanks15:54
hberaud#action bnemec to keep track of our liaisons on the wiki.15:54
hberaudanything else?15:54
bnemecThat's all I had.15:56
hberaudOk then I think we are done15:56
hberaudThanks for joining!15:56
sboyronhberaud, Thanks15:56
hberaud#endmeeting15:56
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:56
openstackMeeting ended Mon Oct 12 15:56:27 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:56
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-10-12-15.00.html15:56
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-10-12-15.00.txt15:56
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-10-12-15.00.log.html15:56
damanibnemec, i have refactor the patch, can you please review https://review.opendev.org/#/c/746723/ ?15:57
*** rpittau is now known as rpittau|afk16:01
bnemecOkay, I created https://wiki.openstack.org/wiki/Oslo#Project_Leadership_Liaisons16:02
bnemechberaud moguimar sboyron damani: please check that I put everyone in the right place and fill in any details I didn't have.16:03
moguimarhberaud at utc+2?16:03
bnemecThat's what was in the other table.16:03
hberaudbnemec, moguimar, bnemec, moguimar, johnsom, stephenfin, kgiusti, smcginnis, damani, sboyron: FYI I missed this point, I'm volunteer to manage the DPL process https://governance.openstack.org/tc/resolutions/20200803-distributed-project-leadership.html#process-for-opting-in-to-distributed-leadership to move our team under this kind of leadership, I'll submit a patch soon to update our status16:03
hberaudmoguimar: yep UTC+216:03
moguimarwhere are you based?16:03
hberaudfrance16:04
moguimarI forgot that France spans 14 timezones16:04
moguimarok, no jokes now. But CZ is UTC+116:04
moguimarmaybe you're counting daylight saving?16:04
sboyronmoguimar, France is UTC+1 too16:05
*** ralonsoh has quit IRC16:05
hberaudmoguimar: we abandonned city timezone only 1 century ago16:05
damanibnemec, thanks a lot for the wiki16:05
sboyronUTC+1 in winter and UTC+2 in summer;16:05
moguimarah, so I'm at UTC+216:05
moguimarand not +1 =P16:06
sboyron6pm right now ;)16:06
moguimarfixed16:07
moguimarthe wiki16:07
moguimarhberaud, the timezone joke is on the overseas part of france16:08
moguimarlike Brazil borders with Frace at French Guiana16:08
hberaudmoguimar: yeah I understood your joke :)16:09
sboyronbnemec, I updated the wiki with all my info; thanks16:09
hberaudmoguimar: french polynesia help us too16:10
hberaud(to cover all these TZ)16:10
stephenfinhberaud: great! :)16:11
hberaudbut a century ago all french city also had their own TZ too16:11
hberaudcity/cities16:11
hberaudbnemec: awesome thanks for the wiki16:11
hberauddamani: feel free to update our informations https://wiki.openstack.org/wiki/Oslo#Project_Leadership_Liaisons16:12
hberauds/our/your16:12
damanihberaud, yes16:13
*** tosky has quit IRC16:21
*** dtantsur is now known as dtantsur|afk16:35
*** hamalq has joined #openstack-oslo16:49
openstackgerritMerged openstack/oslo.config master: Convert rst to plaintext for oslo.config output  https://review.opendev.org/64005717:38
*** dciabrin_ has quit IRC18:29
hamalqhi is there any documentation about how the reply message work on for rabbitmq driver?18:34
*** dciabrin_ has joined #openstack-oslo18:36
hamalqis the rabbitmq feature used for that https://www.rabbitmq.com/direct-reply-to.html18:39
hamalqis this ^^?18:39
*** sboyron has quit IRC20:23
*** tosky has joined #openstack-oslo20:26
*** kgiusti has left #openstack-oslo21:00
*** sboyron has joined #openstack-oslo21:04
*** sboyron has quit IRC21:30
hamalq: hberaud hi how are u? can u please help me with ^^21:30
openstackgerritsean mooney proposed openstack/oslo.messaging master: add min of 1 to rpc_conn_pool_size  https://review.opendev.org/75765421:47
*** rcernin has joined #openstack-oslo21:47
*** rcernin has quit IRC22:05
*** rcernin has joined #openstack-oslo22:30
*** tosky has quit IRC23:15
*** hamalq has quit IRC23:52

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