*** eandersson has quit IRC | 00:35 | |
*** eandersson has joined #openstack-oslo | 00:36 | |
*** rcernin has quit IRC | 02:28 | |
*** rcernin has joined #openstack-oslo | 02:58 | |
*** redrobot has quit IRC | 04:36 | |
*** redrobot has joined #openstack-oslo | 05:20 | |
*** iurygregory has joined #openstack-oslo | 07:10 | |
*** rpittau|afk is now known as rpittau | 07:14 | |
*** ralonsoh has joined #openstack-oslo | 07:15 | |
*** rcernin has quit IRC | 07:21 | |
*** tosky has joined #openstack-oslo | 07:39 | |
*** tkajinam has quit IRC | 08:07 | |
AJaeger | hberaud, stephenfin, https://review.opendev.org/#/c/732517 is a docstheme update for mox3, could you review, please? | 08:17 |
---|---|---|
AJaeger | stephenfin: 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 IRC | 08:19 | |
*** ttx has joined #openstack-oslo | 08:19 | |
openstackgerrit | Merged openstack/oslo.messaging master: Fix pygments style https://review.opendev.org/733460 | 09:05 |
openstackgerrit | Merged openstack/oslotest master: Revert "Revert "Switch to unittest.mock from mock"" https://review.opendev.org/717311 | 09:06 |
*** Dmitrii-Sh has joined #openstack-oslo | 09:35 | |
Dmitrii-Sh | Hi everybody, if anybody has time, I would appreciate another code review on https://review.opendev.org/#/c/731218/. Thanks in advance! | 09:39 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.serialization master: Remove the yamlutils module. https://review.opendev.org/707366 | 09:44 |
*** adriant has quit IRC | 09:56 | |
*** adriant has joined #openstack-oslo | 09:57 | |
*** rpittau is now known as rpittau|bbl | 10:15 | |
openstackgerrit | Merged openstack/mox3 master: Switch to newer openstackdocstheme versions https://review.opendev.org/732517 | 10:36 |
stephenfin | AJaeger: did all those reviews, btw | 11:14 |
stephenfin | AJaeger: 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 it | 11:14 |
stephenfin | Do need to clean up some lower-constraints.txt files, but those won't result in mox3 being installed | 11:14 |
AJaeger | stephenfin: thanks | 11:15 |
AJaeger | regarding mox3: I agree to retire it | 11:18 |
AJaeger | stephenfin: removing mox3 was a rocky goal | 11:19 |
stephenfin | Yup. We finally removed it in nova and horizon during Ussuri, which allowed us to remove it from oslotest | 11:20 |
AJaeger | so, time to remove in victoria ;) | 11:21 |
*** Luzi has joined #openstack-oslo | 11:53 | |
*** raildo has joined #openstack-oslo | 11:57 | |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy master: Reload files in policy_dirs on primary file change https://review.opendev.org/731218 | 12:07 |
Dmitrii-Sh | stephenfin: thanks for the review, uploaded https://review.opendev.org/#/c/731218/3 | 12:09 |
*** rpittau|bbl is now known as rpittau | 12:13 | |
*** kgiusti has joined #openstack-oslo | 12:33 | |
openstackgerrit | Merged openstack/oslo.serialization master: Remove the yamlutils module. https://review.opendev.org/707366 | 12:42 |
openstackgerrit | Sean McGinnis proposed openstack/stevedore master: Remove Travis CI config https://review.opendev.org/734079 | 13:17 |
*** iurygregory is now known as iurygregory|afk | 13:20 | |
stephenfin | Dmitrii-Sh: Two nits on the reno | 13:36 |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy master: Reload files in policy_dirs on primary file change https://review.opendev.org/731218 | 13:39 |
Dmitrii-Sh | stephenfin: ty, uploaded one more with fixes. | 13:39 |
stephenfin | thanks, +2 | 13:40 |
*** Luzi has quit IRC | 13:41 | |
*** moguimar has joined #openstack-oslo | 13:46 | |
hberaud | beekneemech, 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 |
smcginnis | hberaud: I know some projects have been using that. | 14:27 |
smcginnis | I suppose it could help so we can flag things if they look like they are important to backport. | 14:28 |
hberaud | smcginnis: IMO I think it could be useful for us | 14:28 |
smcginnis | It would be interesting to hear from the teams that have been using this to see if they think it has helped them. | 14:28 |
hberaud | smcginnis: 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 |
smcginnis | Right, just a way to make it easy to flag something in case someone else wants to follow through and propose backports. | 14:31 |
hberaud | ack thanks | 14:33 |
hberaud | anyway it can't hurt | 14:33 |
hberaud | smcginnis: feel free to leave your feedback on the review :) | 14:34 |
*** iurygregory|afk is now known as iurygregory | 14:36 | |
openstackgerrit | Merged openstack/stevedore master: Replace external mock with built-in unittest.mock https://review.opendev.org/733747 | 14:51 |
*** beekneemech is now known as bnemec | 14:54 | |
Dmitrii-Sh | stephenfin, hberaud: thanks a lot for the code reviews! | 14:58 |
hberaud | Dmitrii-Sh: you're welcome | 14:58 |
moguimar | o/ | 15:01 |
bnemec | #startmeeting oslo | 15:03 |
bnemec | Courtesy ping for bnemec, jungleboyj, moguimar, hberaud, stephenfin, kgiusti, johnsom, e0ne, redrobot, bcafarel, smcginnis | 15:03 |
bnemec | #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting | 15:03 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
*** openstack changes topic to " (Meeting topic: oslo)" | 15:03 | |
openstack | The meeting name has been set to 'oslo' | 15:03 |
smcginnis | o/ | 15:03 |
johnsom | o/ | 15:03 |
hberaud | o/ | 15:03 |
moguimar | welcome to the Victoria cycle =D | 15:03 |
kgiusti | o/ | 15:03 |
bcafarel | o/ | 15:03 |
bnemec | Have we not had a meeting since Ussuri shipped? | 15:04 |
moguimar | I mean post PTG | 15:04 |
* hberaud have an appointement soon so I will surely eject before the end of the meeting | 15:04 | |
bnemec | Ah, fair enough. I think we had one meeting since release. | 15:04 |
bnemec | hberaud: No problem. It'll probably be a quick one since we just had the PTG. | 15:05 |
moguimar | so anything we should focus on before you leave hberaud ? | 15:05 |
hberaud | moguimar: just this one => https://review.opendev.org/#/c/734096/ | 15:05 |
hberaud | I would love have all of our feedbacks | 15:06 |
hberaud | but could be done through the review | 15:07 |
moguimar | ack | 15:07 |
moguimar | I'd like to see more in the commit message | 15:07 |
bnemec | I'm fine with that. It doesn't hurt to have it available. | 15:08 |
moguimar | like reason and link for the discussion | 15:08 |
johnsom | We have really liked having that for Octavia | 15:08 |
bnemec | For it to accomplish much someone has to actually look at it, but that can't happen until it exists. | 15:08 |
smcginnis | johnsom: Has it worked out well in Octavia? | 15:08 |
bnemec | #topic Adding Backport-Candidate label to Oslo projects | 15: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 |
bnemec | Just to make it official. | 15:09 |
hberaud | bnemec: thx | 15:09 |
johnsom | Yes. 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 |
moguimar | johnsom++ | 15:09 |
smcginnis | I think it can't hurt. :) | 15:09 |
hberaud | +1 | 15:09 |
bnemec | We should probably create an Oslo dashboard for it: https://wiki.openstack.org/wiki/Oslo#Review_Links | 15:10 |
hberaud | bnemec: good idea | 15:10 |
smcginnis | johnsom: Does ocatavia has a process to remove the flag once it's actually been backported? | 15:10 |
smcginnis | That seems like the only issue to track it through. | 15:11 |
hberaud | ah right | 15:11 |
johnsom | Not today. We have talked about it. You can flip it over to -2 once they are all proposed | 15:11 |
bnemec | Wouldn't we just make the dashboard ignore merged patches? | 15:11 |
bnemec | Oh, no that doesn't work, does it? | 15:11 |
hberaud | bnemec: could be a good shortcut | 15:11 |
moguimar | I guess not | 15:11 |
smcginnis | The dashboard would have to show patches that have the flag set. It wouldn't know if it has been backport or not. | 15:11 |
moguimar | cause you see in the list the one which needs backport, not the backport itself | 15:12 |
bnemec | You still need to see the flagged reviews even after they merged. | 15:12 |
bnemec | Right. | 15:12 |
smcginnis | And if we don't remove the flag, that dashboard gets less useful over time because it will fill with irrelevant things. | 15:12 |
bnemec | We should write a tool that can do that, if one doesn't exist already. | 15:12 |
smcginnis | Seems quite possible. | 15:12 |
* bnemec is not volunteering, mind you ;-) | 15:13 | |
* smcginnis neither | 15:13 | |
* hberaud need to dash, see you soon | 15:14 | |
bnemec | o/ hberaud | 15:14 |
bnemec | FWIW, I wrote a tool that interacted with Gerrit a while back: https://github.com/cybertron/tripleo-auto-abandon | 15:14 |
bnemec | You 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 |
bnemec | The potentially tricky part would be figuring out which review is the backport so you can check its status. | 15:15 |
bnemec | I don't know how Gerrit would represent that. | 15:15 |
smcginnis | If it's backported right, it should have the cherry-pick footer in the commit message. | 15:16 |
bnemec | I'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 |
bnemec | But that would be too easy, so it probably doesn't exist. ;-) | 15:17 |
bnemec | #agreed Add Backport-Candidate label to Oslo projects | 15:18 |
bnemec | #idea Write a tool to automatically remove Backport-Candidate label once the backport has been proposed | 15:18 |
bnemec | We 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 liaisons | 15:19 |
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:19 | |
bnemec | Sorry for the delay liaisons. Hope you're still with us. :-) | 15:19 |
bnemec | The main thing we've been dealing with lately is the bug in oslo.log that broke a number of projects. | 15:20 |
bnemec | I 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 |
johnsom | Nothing to mention from Octavia team. | 15:21 |
*** larainema has joined #openstack-oslo | 15:22 | |
bnemec | #topic Releases | 15:23 |
*** openstack changes topic to "Releases (Meeting topic: oslo)" | 15:23 | |
bnemec | Master and all the stable branches except Stein got released late last week. | 15:23 |
bnemec | Unless 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 |
bnemec | Will 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 meeting | 15: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 |
bnemec | I believe I did that. I think I had concerns that we were duplicating something already covered by the template. | 15:25 |
bnemec | We 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 |
bnemec | Done | 15:26 |
bnemec | "Pull babel alias list into oslo.i18n for backward compatibility." | 15:26 |
bnemec | Done | 15:26 |
bnemec | hberaud to review comments on https://review.opendev.org/#/c/671309/ | 15:26 |
bnemec | Looks like that was done too. | 15:26 |
bnemec | Just needs review now. | 15:27 |
bnemec | "Oslo team to review series starting at https://review.opendev.org/712479" | 15:27 |
bnemec | Done, thanks moguimar and smcginnis | 15:27 |
bnemec | That's all of our action items. | 15:27 |
moguimar | cool, are we making new action items out of PTG discussions? | 15:28 |
moguimar | for tracking purpose | 15:28 |
*** iurygregory has quit IRC | 15:29 | |
bnemec | I guess we could. I have a todo to write a followup blog post summarizing what we discussed. | 15:29 |
*** iurygregory has joined #openstack-oslo | 15:29 | |
bnemec | It 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 |
bnemec | Also, 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 |
bnemec | So I guess my preference for PTG followups would be lp bugs or agenda items. | 15:32 |
bnemec | I won't throw a tantrum if someone adds an action item too though. :-) | 15:32 |
bnemec | #topic Ping list update | 15:32 |
*** openstack changes topic to "Ping list update (Meeting topic: oslo)" | 15:32 | |
bnemec | Per the action item above, the new ping list is now available on the agenda wiki page. | 15:33 |
bnemec | If you want to keep receiving courtesy pings for this meeting please add your name to the new list. | 15:33 |
bnemec | I'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 wiki | 15:33 |
bnemec | Kind of a wordy action item. | 15:34 |
bnemec | I'll send an email reminder too now that the PTG is over. | 15:34 |
bnemec | #action bnemec send email about Victoria ping list | 15:34 |
bnemec | #topic Oslo core contact details | 15:35 |
*** openstack changes topic to "Oslo core contact details (Meeting topic: oslo)" | 15:35 | |
bnemec | This was actually something that came out of the contributor documentation goal. | 15:35 |
openstackgerrit | Merged openstack/stevedore master: Remove Travis CI config https://review.opendev.org/734079 | 15:35 |
bnemec | One of this things we're supposed to do for that is create a contact list for Oslo cores. | 15:35 |
bnemec | To that end, I've created some tables on https://wiki.openstack.org/wiki/Oslo#The_Oslo_Team | 15:35 |
bnemec | I'd like to ask all of the Oslo cores to fill in their details there, if they don't mind. | 15:36 |
bnemec | Note that there's also one for the project-specific cores. | 15:36 |
moguimar | ack | 15:36 |
bnemec | I've filled in the names of some of our more active cores, but if you're not there please add yourself. | 15:37 |
johnsom | Done | 15:37 |
bnemec | #action Oslo cores to fill out contact details on https://wiki.openstack.org/wiki/Oslo#The_Oslo_Team | 15:37 |
bnemec | Thanks! | 15:37 |
bnemec | That's the one-off agenda. | 15:39 |
bnemec | #topic Weekly Wayward Review | 15:39 |
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)" | 15:39 | |
bnemec | #link https://review.opendev.org/689482 | 15:39 |
bnemec | Gorka replied to the comments on that so it would be nice to get it in. | 15:39 |
bnemec | Alright, +2A. It sounds like everyone is okay with the current version. | 15:41 |
bnemec | If 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 discussion | 15:41 |
*** openstack changes topic to "Open discussion (Meeting topic: oslo)" | 15:41 | |
bnemec | That's it for our regularly scheduled topics. Anything else before we close? | 15:42 |
smcginnis | *crickets* | 15:44 |
moguimar | not on my end | 15:45 |
bnemec | Okay, thanks for a great PTG last week everyone. | 15:45 |
bnemec | Catch you all next week. | 15:45 |
bnemec | #endmeeting | 15:45 |
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 15:45 | |
openstack | Meeting ended Mon Jun 8 15:45:40 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:45 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.html | 15:45 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.txt | 15:45 |
openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-06-08-15.03.log.html | 15:45 |
openstackgerrit | Merged openstack/oslo.serialization master: Fix pygments style https://review.opendev.org/733447 | 15:56 |
openstackgerrit | Merged openstack/oslo.context master: Fix pygments style https://review.opendev.org/733441 | 15:57 |
openstackgerrit | Merged openstack/oslo.privsep master: Fix pygments style https://review.opendev.org/733445 | 15:57 |
openstackgerrit | Merged openstack/oslo.db master: Fix pygments style https://review.opendev.org/733456 | 15:57 |
openstackgerrit | Merged openstack/oslo.middleware master: Fix pygments style https://review.opendev.org/733458 | 15:57 |
openstackgerrit | Merged openstack/oslo.service master: Fix pygments style https://review.opendev.org/733452 | 15:57 |
openstackgerrit | Merged openstack/oslo.config master: Fix pygments style https://review.opendev.org/733474 | 15:57 |
openstackgerrit | Merged openstack/oslo.tools master: Fix pygments style https://review.opendev.org/733455 | 15:58 |
openstackgerrit | Merged openstack/oslo.utils master: Fix pygments style https://review.opendev.org/733435 | 16:01 |
openstackgerrit | Merged openstack/oslo.cache master: Fix pygments style https://review.opendev.org/733440 | 16:01 |
*** rpittau is now known as rpittau|afk | 16:08 | |
openstackgerrit | Merged openstack/oslo.policy master: Reload files in policy_dirs on primary file change https://review.opendev.org/731218 | 16:23 |
openstackgerrit | Merged openstack/oslo.policy master: Fix pygments style https://review.opendev.org/733463 | 16:23 |
openstackgerrit | Vladyslav Drok proposed openstack/oslo.utils master: Release greenthread when computing checksum https://review.opendev.org/734145 | 16:42 |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy stable/ussuri: Reload files in policy_dirs on primary file change https://review.opendev.org/734151 | 16:51 |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy stable/train: Reload files in policy_dirs on primary file change https://review.opendev.org/734154 | 16:57 |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy stable/stein: Reload files in policy_dirs on primary file change https://review.opendev.org/734155 | 16:58 |
*** AJaeger has left #openstack-oslo | 16:58 | |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy stable/rocky: Reload files in policy_dirs on primary file change https://review.opendev.org/734156 | 17:00 |
openstackgerrit | Dmitrii Shcherbakov proposed openstack/oslo.policy stable/queens: Reload files in policy_dirs on primary file change https://review.opendev.org/734157 | 17:00 |
openstackgerrit | Merged openstack/oslo.concurrency master: Don't warn on lock removal if file doesn't exist https://review.opendev.org/689482 | 17:05 |
openstackgerrit | Merged openstack/oslo.concurrency master: Fix pygments style https://review.opendev.org/733473 | 17:05 |
openstackgerrit | Merged openstack/taskflow master: Cap jsonschema 3.2.0 as the minimal version https://review.opendev.org/730977 | 17:14 |
openstackgerrit | Merged openstack/oslo.config master: generator: Handle 'cfg.Opt'-type options https://review.opendev.org/731133 | 17:20 |
openstackgerrit | Merged openstack/oslo.versionedobjects master: Cap jsonschema 3.2.0 as the minimal version https://review.opendev.org/730965 | 17:53 |
*** hamalq has joined #openstack-oslo | 18:13 | |
*** ralonsoh has quit IRC | 18:15 | |
openstackgerrit | Merged openstack/oslo.versionedobjects master: Remove translation sections from setup.cfg https://review.opendev.org/728354 | 19:34 |
openstackgerrit | Hervé Beraud proposed openstack/automaton master: drop mock from lower-constraints https://review.opendev.org/734218 | 20:22 |
*** kgiusti has left #openstack-oslo | 20:36 | |
openstackgerrit | Sean McGinnis proposed openstack/automaton master: Update lower-constraints list https://review.opendev.org/734247 | 20:39 |
*** raildo has quit IRC | 21:18 | |
openstackgerrit | Hervé Beraud proposed openstack/futurist master: drop mock from lower-constraints https://review.opendev.org/734336 | 22:09 |
*** threestrands has joined #openstack-oslo | 22:15 | |
*** threestrands has quit IRC | 22:16 | |
*** threestrands has joined #openstack-oslo | 22:17 | |
*** rcernin has joined #openstack-oslo | 22:17 | |
*** threestrands has quit IRC | 22:18 | |
*** threestrands has joined #openstack-oslo | 22:18 | |
*** threestrands has quit IRC | 22:19 | |
*** threestrands has joined #openstack-oslo | 22:20 | |
*** threestrands has quit IRC | 22:21 | |
*** threestrands has joined #openstack-oslo | 22:21 | |
*** threestrands has quit IRC | 22:22 | |
*** threestrands has joined #openstack-oslo | 22:23 | |
*** threestrands has quit IRC | 22:24 | |
*** threestrands has joined #openstack-oslo | 22:24 | |
*** threestrands has quit IRC | 22:25 | |
*** threestrands has joined #openstack-oslo | 22:26 | |
*** threestrands has quit IRC | 22:27 | |
*** threestrands has joined #openstack-oslo | 22:27 | |
*** threestrands has quit IRC | 22:28 | |
*** threestrands has joined #openstack-oslo | 22:29 | |
*** threestrands has quit IRC | 22:30 | |
*** threestrands has joined #openstack-oslo | 22:30 | |
*** tkajinam has joined #openstack-oslo | 22:49 | |
*** dmellado_ has joined #openstack-oslo | 23:10 | |
*** dmellado has quit IRC | 23:11 | |
*** dmellado_ is now known as dmellado | 23:11 | |
*** tosky has quit IRC | 23:13 | |
*** dawzon has left #openstack-oslo | 23:29 | |
openstackgerrit | Merged openstack/tooz master: Add python 3.8 to classifiers https://review.opendev.org/730767 | 23:36 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!