Monday, 2020-06-08

*** eandersson has quit IRC00:35
*** eandersson has joined #openstack-oslo00:36
*** rcernin has quit IRC02:28
*** rcernin has joined #openstack-oslo02:58
*** redrobot has quit IRC04:36
*** redrobot has joined #openstack-oslo05:20
*** iurygregory has joined #openstack-oslo07:10
*** rpittau|afk is now known as rpittau07:14
*** ralonsoh has joined #openstack-oslo07:15
*** rcernin has quit IRC07:21
*** tosky has joined #openstack-oslo07:39
*** tkajinam has quit IRC08:07
AJaegerhberaud, stephenfin, https://review.opendev.org/#/c/732517 is a docstheme update for mox3, could you review, please?08:17
AJaegerstephenfin: could you review the placement ones as well, please? https://review.opendev.org/#/c/731512/ https://review.opendev.org/#/c/729979/ https://review.opendev.org/#/c/730473/ https://review.opendev.org/#/c/732355/08:18
*** ttx has quit IRC08:19
*** ttx has joined #openstack-oslo08:19
openstackgerritMerged openstack/oslo.messaging master: Fix pygments style  https://review.opendev.org/73346009:05
openstackgerritMerged openstack/oslotest master: Revert "Revert "Switch to unittest.mock from mock""  https://review.opendev.org/71731109:06
*** Dmitrii-Sh has joined #openstack-oslo09:35
Dmitrii-ShHi everybody, if anybody has time, I would appreciate another code review on https://review.opendev.org/#/c/731218/. Thanks in advance!09:39
openstackgerritDaniel Bengtsson proposed openstack/oslo.serialization master: Remove the yamlutils module.  https://review.opendev.org/70736609:44
*** adriant has quit IRC09:56
*** adriant has joined #openstack-oslo09:57
*** rpittau is now known as rpittau|bbl10:15
openstackgerritMerged openstack/mox3 master: Switch to newer openstackdocstheme versions  https://review.opendev.org/73251710:36
stephenfinAJaeger: did all those reviews, btw11:14
stephenfinAJaeger: Do we want to look at removing the mox3 project now though? fwict, there are no projects left under openstack governance using it so we don't really need to keep maintaining it11:14
stephenfinDo need to clean up some lower-constraints.txt files, but those won't result in mox3 being installed11:14
AJaegerstephenfin: thanks11:15
AJaegerregarding mox3: I agree to retire it11:18
AJaegerstephenfin: removing mox3 was a rocky goal11:19
stephenfinYup. We finally removed it in nova and horizon during Ussuri, which allowed us to remove it from oslotest11:20
AJaegerso, time to remove in victoria ;)11:21
*** Luzi has joined #openstack-oslo11:53
*** raildo has joined #openstack-oslo11:57
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy master: Reload files in policy_dirs on primary file change  https://review.opendev.org/73121812:07
Dmitrii-Shstephenfin: thanks for the review, uploaded https://review.opendev.org/#/c/731218/312:09
*** rpittau|bbl is now known as rpittau12:13
*** kgiusti has joined #openstack-oslo12:33
openstackgerritMerged openstack/oslo.serialization master: Remove the yamlutils module.  https://review.opendev.org/70736612:42
openstackgerritSean McGinnis proposed openstack/stevedore master: Remove Travis CI config  https://review.opendev.org/73407913:17
*** iurygregory is now known as iurygregory|afk13:20
stephenfinDmitrii-Sh: Two nits on the reno13:36
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy master: Reload files in policy_dirs on primary file change  https://review.opendev.org/73121813:39
Dmitrii-Shstephenfin: ty, uploaded one more with fixes.13:39
stephenfinthanks, +213:40
*** Luzi has quit IRC13:41
*** moguimar has joined #openstack-oslo13:46
hberaudbeekneemech, smcginnis: o/ do you think this is something that could help us to maintains stable branches and reduce the possibilities to forget some backports? https://review.opendev.org/#/c/734096/14:26
smcginnishberaud: I know some projects have been using that.14:27
smcginnisI suppose it could help so we can flag things if they look like they are important to backport.14:28
hberaudsmcginnis: IMO I think it could be useful for us14:28
smcginnisIt would be interesting to hear from the teams that have been using this to see if they think it has helped them.14:28
hberaudsmcginnis: it's just a system to flag important things, right? I mean it don't automatically cherry-pick the tagged things on stable branches, right?14:30
smcginnisRight, just a way to make it easy to flag something in case someone else wants to follow through and propose backports.14:31
hberaudack thanks14:33
hberaudanyway it can't hurt14:33
hberaudsmcginnis: feel free to leave your feedback on the review :)14:34
*** iurygregory|afk is now known as iurygregory14:36
openstackgerritMerged openstack/stevedore master: Replace external mock with built-in unittest.mock  https://review.opendev.org/73374714:51
*** beekneemech is now known as bnemec14:54
Dmitrii-Shstephenfin, hberaud: thanks a lot for the code reviews!14:58
hberaudDmitrii-Sh: you're welcome14:58
moguimaro/15:01
bnemec#startmeeting oslo15:03
bnemecCourtesy ping for bnemec, jungleboyj, moguimar, hberaud, stephenfin, kgiusti, johnsom, e0ne, redrobot, bcafarel, smcginnis15:03
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:03
openstackMeeting started Mon Jun  8 15:03:01 2020 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:03
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:03
*** openstack changes topic to " (Meeting topic: oslo)"15:03
openstackThe meeting name has been set to 'oslo'15:03
smcginniso/15:03
johnsomo/15:03
hberaudo/15:03
moguimarwelcome to the Victoria cycle =D15:03
kgiustio/15:03
bcafarelo/15:03
bnemecHave we not had a meeting since Ussuri shipped?15:04
moguimarI mean post PTG15:04
* hberaud have an appointement soon so I will surely eject before the end of the meeting15:04
bnemecAh, fair enough. I think we had one meeting since release.15:04
bnemechberaud: No problem. It'll probably be a quick one since we just had the PTG.15:05
moguimarso anything we should focus on before you leave hberaud ?15:05
hberaudmoguimar: just this one => https://review.opendev.org/#/c/734096/15:05
hberaudI would love have all of our feedbacks15:06
hberaudbut could be done through the review15:07
moguimarack15:07
moguimarI'd like to see more in the commit message15:07
bnemecI'm fine with that. It doesn't hurt to have it available.15:08
moguimarlike reason and link for the discussion15:08
johnsomWe have really liked having that for Octavia15:08
bnemecFor it to accomplish much someone has to actually look at it, but that can't happen until it exists.15:08
smcginnisjohnsom: Has it worked out well in Octavia?15:08
bnemec#topic Adding Backport-Candidate label to Oslo projects15:08
*** openstack changes topic to "Adding Backport-Candidate label to Oslo projects (Meeting topic: oslo)"15:08
bnemec#link https://review.opendev.org/#/c/734096/15:08
bnemecJust to make it official.15:09
hberaudbnemec: thx15:09
johnsomYes. We came up with it because we were struggling to keep up with backports. It's a very handy flag to make a list.15:09
moguimarjohnsom++15:09
smcginnisI think it can't hurt. :)15:09
hberaud+115:09
bnemecWe should probably create an Oslo dashboard for it: https://wiki.openstack.org/wiki/Oslo#Review_Links15:10
hberaudbnemec: good idea15:10
smcginnisjohnsom: Does ocatavia has a process to remove the flag once it's actually been backported?15:10
smcginnisThat seems like the only issue to track it through.15:11
hberaudah right15:11
johnsomNot today. We have talked about it. You can flip it over to -2 once they are all proposed15:11
bnemecWouldn't we just make the dashboard ignore merged patches?15:11
bnemecOh, no that doesn't work, does it?15:11
hberaudbnemec: could be a good shortcut15:11
moguimarI guess not15:11
smcginnisThe dashboard would have to show patches that have the flag set. It wouldn't know if it has been backport or not.15:11
moguimarcause you see in the list the one which needs backport, not the backport itself15:12
bnemecYou still need to see the flagged reviews even after they merged.15:12
bnemecRight.15:12
smcginnisAnd if we don't remove the flag, that dashboard gets less useful over time because it will fill with irrelevant things.15:12
bnemecWe should write a tool that can do that, if one doesn't exist already.15:12
smcginnisSeems quite possible.15:12
* bnemec is not volunteering, mind you ;-)15:13
* smcginnis neither15:13
* hberaud need to dash, see you soon15:14
bnemeco/ hberaud15:14
bnemecFWIW, I wrote a tool that interacted with Gerrit a while back: https://github.com/cybertron/tripleo-auto-abandon15:14
bnemecYou would probably do something similar, except instead of looking at how old the patch is, you look to see if the backport has merged.15:15
bnemecThe potentially tricky part would be figuring out which review is the backport so you can check its status.15:15
bnemecI don't know how Gerrit would represent that.15:15
smcginnisIf it's backported right, it should have the cherry-pick footer in the commit message.15:16
bnemecI'm hoping maybe there's a cherry-picks section in the Gerrit API data that would point you directly to the right review.15:16
bnemecBut that would be too easy, so it probably doesn't exist. ;-)15:17
bnemec#agreed Add Backport-Candidate label to Oslo projects15:18
bnemec#idea Write a tool to automatically remove Backport-Candidate label once the backport has been proposed15:18
bnemecWe don't actually need to wait for the backport to merge. Once it's been proposed the candidate label has done its job.15:19
bnemec#topic Red flags for/from liaisons15:19
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:19
bnemecSorry for the delay liaisons. Hope you're still with us. :-)15:19
bnemecThe main thing we've been dealing with lately is the bug in oslo.log that broke a number of projects.15:20
bnemecI think backports to all supported branches are merged. Stein still needs to be released, but I believe the others are all in the wild now.15:21
johnsomNothing to mention from Octavia team.15:21
*** larainema has joined #openstack-oslo15:22
bnemec#topic Releases15:23
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:23
bnemecMaster and all the stable branches except Stein got released late last week.15:23
bnemecUnless there's an urgent need I'll probably skip release proposals this week. I doubt much has merged in the past couple of days anyway.15:24
bnemecWill try to get the Stein oslo.log out there today though.15:24
bnemec#action bnemec to release Stein backport(s)15:24
bnemec#topic Action items from last meeting15:25
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:25
bnemec"bnemec to look at hberaud's patch for per-project contributing.rst"15:25
bnemecI believe I did that. I think I had concerns that we were duplicating something already covered by the template.15:25
bnemecWe don't have hberaud right now so we'll have to table that discussion for later.15:26
bnemec"bnemec to create new ping list on meeting agenda"15:26
bnemecDone15:26
bnemec"Pull babel alias list into oslo.i18n for backward compatibility."15:26
bnemecDone15:26
bnemechberaud to review comments on https://review.opendev.org/#/c/671309/15:26
bnemecLooks like that was done too.15:26
bnemecJust needs review now.15:27
bnemec"Oslo team to review series starting at https://review.opendev.org/712479"15:27
bnemecDone, thanks moguimar and smcginnis15:27
bnemecThat's all of our action items.15:27
moguimarcool, are we making new action items out of PTG discussions?15:28
moguimarfor tracking purpose15:28
*** iurygregory has quit IRC15:29
bnemecI guess we could. I have a todo to write a followup blog post summarizing what we discussed.15:29
*** iurygregory has joined #openstack-oslo15:29
bnemecIt might be better to open lp bugs for anything longer term so we don't keep repeating the same action items from week to week.15:29
bnemecAlso, if there's anything we want to revisit in future meetings it should be added to the agenda explicitly. That might be better than using action items.15:31
bnemecSo I guess my preference for PTG followups would be lp bugs or agenda items.15:32
bnemecI won't throw a tantrum if someone adds an action item too though. :-)15:32
bnemec#topic Ping list update15:32
*** openstack changes topic to "Ping list update (Meeting topic: oslo)"15:32
bnemecPer the action item above, the new ping list is now available on the agenda wiki page.15:33
bnemecIf you want to keep receiving courtesy pings for this meeting please add your name to the new list.15:33
bnemecI'll keep using the old list next week, but after that I'll switch over to the new one.15:33
bnemec#action Anyone who wants to continue receiving courtesy pings should add their name to the Victoria list on the agenda wiki15:33
bnemecKind of a wordy action item.15:34
bnemecI'll send an email reminder too now that the PTG is over.15:34
bnemec#action bnemec send email about Victoria ping list15:34
bnemec#topic Oslo core contact details15:35
*** openstack changes topic to "Oslo core contact details (Meeting topic: oslo)"15:35
bnemecThis was actually something that came out of the contributor documentation goal.15:35
openstackgerritMerged openstack/stevedore master: Remove Travis CI config  https://review.opendev.org/73407915:35
bnemecOne of this things we're supposed to do for that is create a contact list for Oslo cores.15:35
bnemecTo that end, I've created some tables on https://wiki.openstack.org/wiki/Oslo#The_Oslo_Team15:35
bnemecI'd like to ask all of the Oslo cores to fill in their details there, if they don't mind.15:36
bnemecNote that there's also one for the project-specific cores.15:36
moguimarack15:36
bnemecI've filled in the names of some of our more active cores, but if you're not there please add yourself.15:37
johnsomDone15:37
bnemec#action Oslo cores to fill out contact details on https://wiki.openstack.org/wiki/Oslo#The_Oslo_Team15:37
bnemecThanks!15:37
bnemecThat's the one-off agenda.15:39
bnemec#topic Weekly Wayward Review15:39
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:39
bnemec#link https://review.opendev.org/68948215:39
bnemecGorka replied to the comments on that so it would be nice to get it in.15:39
bnemecAlright, +2A. It sounds like everyone is okay with the current version.15:41
bnemecIf someone wants a debug log for the other cases they can propose a followup and we'll discuss it there.15:41
bnemec#topic Open discussion15:41
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:41
bnemecThat's it for our regularly scheduled topics. Anything else before we close?15:42
smcginnis*crickets*15:44
moguimarnot on my end15:45
bnemecOkay, thanks for a great PTG last week everyone.15:45
bnemecCatch you all next week.15:45
bnemec#endmeeting15:45
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:45
openstackMeeting ended Mon Jun  8 15:45:40 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:45
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.html15:45
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.txt15:45
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.log.html15:45
openstackgerritMerged openstack/oslo.serialization master: Fix pygments style  https://review.opendev.org/73344715:56
openstackgerritMerged openstack/oslo.context master: Fix pygments style  https://review.opendev.org/73344115:57
openstackgerritMerged openstack/oslo.privsep master: Fix pygments style  https://review.opendev.org/73344515:57
openstackgerritMerged openstack/oslo.db master: Fix pygments style  https://review.opendev.org/73345615:57
openstackgerritMerged openstack/oslo.middleware master: Fix pygments style  https://review.opendev.org/73345815:57
openstackgerritMerged openstack/oslo.service master: Fix pygments style  https://review.opendev.org/73345215:57
openstackgerritMerged openstack/oslo.config master: Fix pygments style  https://review.opendev.org/73347415:57
openstackgerritMerged openstack/oslo.tools master: Fix pygments style  https://review.opendev.org/73345515:58
openstackgerritMerged openstack/oslo.utils master: Fix pygments style  https://review.opendev.org/73343516:01
openstackgerritMerged openstack/oslo.cache master: Fix pygments style  https://review.opendev.org/73344016:01
*** rpittau is now known as rpittau|afk16:08
openstackgerritMerged openstack/oslo.policy master: Reload files in policy_dirs on primary file change  https://review.opendev.org/73121816:23
openstackgerritMerged openstack/oslo.policy master: Fix pygments style  https://review.opendev.org/73346316:23
openstackgerritVladyslav Drok proposed openstack/oslo.utils master: Release greenthread when computing checksum  https://review.opendev.org/73414516:42
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy stable/ussuri: Reload files in policy_dirs on primary file change  https://review.opendev.org/73415116:51
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy stable/train: Reload files in policy_dirs on primary file change  https://review.opendev.org/73415416:57
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy stable/stein: Reload files in policy_dirs on primary file change  https://review.opendev.org/73415516:58
*** AJaeger has left #openstack-oslo16:58
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy stable/rocky: Reload files in policy_dirs on primary file change  https://review.opendev.org/73415617:00
openstackgerritDmitrii Shcherbakov proposed openstack/oslo.policy stable/queens: Reload files in policy_dirs on primary file change  https://review.opendev.org/73415717:00
openstackgerritMerged openstack/oslo.concurrency master: Don't warn on lock removal if file doesn't exist  https://review.opendev.org/68948217:05
openstackgerritMerged openstack/oslo.concurrency master: Fix pygments style  https://review.opendev.org/73347317:05
openstackgerritMerged openstack/taskflow master: Cap jsonschema 3.2.0 as the minimal version  https://review.opendev.org/73097717:14
openstackgerritMerged openstack/oslo.config master: generator: Handle 'cfg.Opt'-type options  https://review.opendev.org/73113317:20
openstackgerritMerged openstack/oslo.versionedobjects master: Cap jsonschema 3.2.0 as the minimal version  https://review.opendev.org/73096517:53
*** hamalq has joined #openstack-oslo18:13
*** ralonsoh has quit IRC18:15
openstackgerritMerged openstack/oslo.versionedobjects master: Remove translation sections from setup.cfg  https://review.opendev.org/72835419:34
openstackgerritHervĂ© Beraud proposed openstack/automaton master: drop mock from lower-constraints  https://review.opendev.org/73421820:22
*** kgiusti has left #openstack-oslo20:36
openstackgerritSean McGinnis proposed openstack/automaton master: Update lower-constraints list  https://review.opendev.org/73424720:39
*** raildo has quit IRC21:18
openstackgerritHervĂ© Beraud proposed openstack/futurist master: drop mock from lower-constraints  https://review.opendev.org/73433622:09
*** threestrands has joined #openstack-oslo22:15
*** threestrands has quit IRC22:16
*** threestrands has joined #openstack-oslo22:17
*** rcernin has joined #openstack-oslo22:17
*** threestrands has quit IRC22:18
*** threestrands has joined #openstack-oslo22:18
*** threestrands has quit IRC22:19
*** threestrands has joined #openstack-oslo22:20
*** threestrands has quit IRC22:21
*** threestrands has joined #openstack-oslo22:21
*** threestrands has quit IRC22:22
*** threestrands has joined #openstack-oslo22:23
*** threestrands has quit IRC22:24
*** threestrands has joined #openstack-oslo22:24
*** threestrands has quit IRC22:25
*** threestrands has joined #openstack-oslo22:26
*** threestrands has quit IRC22:27
*** threestrands has joined #openstack-oslo22:27
*** threestrands has quit IRC22:28
*** threestrands has joined #openstack-oslo22:29
*** threestrands has quit IRC22:30
*** threestrands has joined #openstack-oslo22:30
*** tkajinam has joined #openstack-oslo22:49
*** dmellado_ has joined #openstack-oslo23:10
*** dmellado has quit IRC23:11
*** dmellado_ is now known as dmellado23:11
*** tosky has quit IRC23:13
*** dawzon has left #openstack-oslo23:29
openstackgerritMerged openstack/tooz master: Add python 3.8 to classifiers  https://review.opendev.org/73076723:36

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