Monday, 2018-09-17

*** openstackgerrit has quit IRC00:09
*** ducnv has joined #openstack-oslo01:00
*** bobh has joined #openstack-oslo01:17
*** lbragstad has joined #openstack-oslo02:09
*** e0ne has joined #openstack-oslo02:33
*** lbragstad has quit IRC02:36
*** lbragstad has joined #openstack-oslo02:42
*** e0ne has quit IRC03:21
*** shachar has quit IRC03:44
*** oanson has quit IRC03:44
*** e0ne has joined #openstack-oslo03:54
*** pooja_jadhav has joined #openstack-oslo04:00
*** bobh has quit IRC04:41
*** ducnv_ has joined #openstack-oslo04:51
*** Phuongnh has joined #openstack-oslo04:52
*** ducnv_ has quit IRC04:53
*** e0ne has quit IRC04:55
*** Phuongnh has quit IRC05:07
*** snapiri has joined #openstack-oslo05:10
*** snapiri has quit IRC05:18
*** jbadiapa has joined #openstack-oslo05:25
*** e0ne has joined #openstack-oslo05:42
*** mattgo has joined #openstack-oslo06:38
*** e0ne_ has joined #openstack-oslo06:50
*** e0ne has quit IRC06:53
*** e0ne_ has quit IRC06:54
*** giblet is now known as gibi07:02
*** openstackgerrit has joined #openstack-oslo07:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.cache master: Imported Translations from Zanata  https://review.openstack.org/60302907:08
*** rcernin has quit IRC07:11
*** shardy has joined #openstack-oslo07:15
*** herve4383 has joined #openstack-oslo07:28
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.concurrency master: Imported Translations from Zanata  https://review.openstack.org/60303307:30
*** snapiri has joined #openstack-oslo07:37
*** herve4383_ has joined #openstack-oslo07:39
*** herve4383_ has quit IRC07:40
*** herve4383 has quit IRC07:40
*** herve4383 has joined #openstack-oslo07:41
*** sambetts_ has quit IRC07:42
*** sambetts_ has joined #openstack-oslo07:45
openstackgerritMerged openstack/oslo.concurrency master: Imported Translations from Zanata  https://review.openstack.org/60303307:49
openstackgerritTommyLike proposed openstack/oslo.privsep master: Use threads to process target function  https://review.openstack.org/59355607:55
*** rpittau has joined #openstack-oslo08:23
openstackgerritOleg Bondarev proposed openstack/oslo.messaging master: Use default exchange for direct messaging  https://review.openstack.org/59666108:36
*** tosky has joined #openstack-oslo08:41
openstackgerritlei zhang proposed openstack/stevedore master: Update doc/conf.py to avoid warnings with sphinx 1.8  https://review.openstack.org/60305108:42
*** moguimar has joined #openstack-oslo08:47
*** moguimar has quit IRC08:50
openstackgerritIlya Shakhat proposed openstack/osprofiler master: [WIP] Collect traces from Tempest job  https://review.openstack.org/60258609:14
*** moguimar has joined #openstack-oslo09:16
openstackgerritDuc Nguyen proposed openstack/oslo.config master: Follow up patch 'Handle config mapping changes'.  https://review.openstack.org/60306009:44
openstackgerritDuc Nguyen proposed openstack/oslo.config master: Follow up patch 'Handle config mapping changes'.  https://review.openstack.org/60306009:45
openstackgerritIlya Shakhat proposed openstack/osprofiler master: [WIP] Collect traces from Tempest job  https://review.openstack.org/60258610:02
*** oanson has joined #openstack-oslo10:05
*** finucannot is now known as stephenfin10:34
*** njohnston has joined #openstack-oslo11:00
*** pbourke has quit IRC11:18
*** pbourke has joined #openstack-oslo11:20
*** ansmith has quit IRC11:53
*** dave-mccowan has joined #openstack-oslo11:54
*** raildo has joined #openstack-oslo12:03
*** bobh has joined #openstack-oslo12:07
*** bobh has quit IRC12:12
*** raildo has quit IRC12:45
*** raildo has joined #openstack-oslo12:45
*** kgiusti has joined #openstack-oslo12:57
*** bobh has joined #openstack-oslo13:02
*** ansmith has joined #openstack-oslo13:07
*** mattgo has quit IRC13:21
openstackgerritIlya Shakhat proposed openstack/osprofiler master: Collect traces from Tempest job  https://review.openstack.org/60258613:45
*** ducnv_ has joined #openstack-oslo13:47
openstackgerritIlya Shakhat proposed openstack/osprofiler master: Make tracing of SQL statements configurable in DevStack plugin  https://review.openstack.org/60313213:53
*** bobh has quit IRC13:55
openstackgerritAndy Smith proposed openstack/devstack-plugin-kafka master: Update to add confluent-kafka-python library for driver  https://review.openstack.org/60313413:57
*** bobh has joined #openstack-oslo14:13
*** bobh has quit IRC14:18
*** e0ne has joined #openstack-oslo14:30
*** bobh has joined #openstack-oslo14:35
*** e0ne has quit IRC14:38
openstackgerritAndy Smith proposed openstack/devstack-plugin-kafka master: Update to add confluent-kafka-python library for driver  https://review.openstack.org/60313414:40
*** _d34dh0r53_ is now known as d34dh0r5314:54
*** Phuongnh has joined #openstack-oslo14:57
*** bnemec has joined #openstack-oslo14:59
bnemec#startmeeting oslo15:01
openstackMeeting started Mon Sep 17 15:01:12 2018 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
*** openstack changes topic to " (Meeting topic: oslo)"15:01
openstackThe meeting name has been set to 'oslo'15:01
bnemeccourtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims15:01
bnemeccourtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo15:01
bnemeccourtesy ping for jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_, njohnston15:01
bnemeccourtesy ping for raildo, redrobot, sileht, sreshetnyak, stephenfin, stevemar, therve15:01
bnemeccourtesy ping for thinrichs, toabctl, zhiyan, zxy, zzzeek15:01
dhellmanno/15:01
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:01
ansmitho/15:01
Phuongnho/15:01
ducnv_o/15:02
raildoo/15:02
kgiustiStevenK: ping - oslo meeting...15:02
kgiustio/15:02
njohnstono/15:03
*** Phuongnh_ has joined #openstack-oslo15:03
bnemec#topic Red flags for/from liaisons15:03
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:03
bnemecNothing much from our side. I'm planning to start the regular weekly releases again now that stein is open.15:04
bnemec#topic Releases15:05
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:05
*** Phuongnh has quit IRC15:06
bnemecSee above. I expect to release all the things this week, possibly with the exception of olso.messaging if all of the removal patches haven't merged yet.15:06
bnemecIf you're aware of anything else I should hold off on please let me know asap.15:07
bnemec#topic Action items from last meeting15:07
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:07
bnemec"bnemec check if we can release stein libraries"15:08
bnemecWe can. :-)15:08
bnemec"review patches listed in http://paste.openstack.org/show/728832/"15:08
bnemecI think we finished that.15:09
bnemec"bnemec to check with VMT about Oslo libraries"15:09
bnemecI briefly discussed this at the PTG, but never got a chance to have the full discussion. Still on the todo list.15:10
bnemec"dhellmann to respond to uuidsentinel thread"15:10
bnemecThat one is done.15:10
bnemec"bnemec cancel oslo meeting for next week"15:10
bnemecAnd that one too.15:10
bnemec#topic Config migrator15:11
*** openstack changes topic to "Config migrator (Meeting topic: oslo)"15:11
bnemecPhuongnh_: ducnv_: I can't remember, was there anything new to talk about here or are you just waiting on reviews?15:12
Phuongnh_We have added release note as your request, now waiting for your review15:13
ducnv_hi Ben, I just push follow up patch, it ready for review15:14
ducnv_https://review.openstack.org/#/c/603060/15:14
*** jungleboyj has joined #openstack-oslo15:15
bnemecGreat, thanks!15:15
Phuongnh_:)15:15
* jungleboyj sneaks in late15:15
bnemec#topic  oslo.upgradecheck15:16
*** openstack changes topic to "oslo.upgradecheck (Meeting topic: oslo)"15:16
bnemecIf you haven't been following this, it's a new library to support the upgrade check community goal for this cycle.15:17
bnemecBasically it will house the common code for the checks and hopefully make it a little easier for projects to implement them.15:17
Phuongnh_I see15:18
bnemecI kind of did an end run around the new library proposal process in Oslo though, so if you have any concerns let me know.15:18
bnemechttps://github.com/cybertron/oslo.upgradecheck15:19
bnemecI guess the one thought I had about it since initially proposing it is whether we should make it a more generic oslo.cli library or something like that.15:19
Phuongnh_Thanks, there is nothing for now15:19
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Render API reference documentation  https://review.openstack.org/60026415:19
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Add a conceptual overview to docs  https://review.openstack.org/60026515:19
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Allow ProjectClaims to support multiple resources  https://review.openstack.org/60026615:19
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Ignore documentation builds  https://review.openstack.org/60316715:19
bnemecI don't know whether there is more common cli code that we might want to pull in later.15:20
Phuongnh_Yes, I will check it later15:21
bnemecThe change to add it to governance is here: https://review.openstack.org/#/c/602483/15:23
bnemecI don't have anything else to say about it. This was mostly just an FYI since it was done in a bit of a hurry.15:23
Phuongnh_I see, thanks15:24
bnemec#topic Weekly Wayward Review15:24
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:24
bnemec#link https://review.openstack.org/#/c/559484/15:25
bnemecThis is a review to allow pbr to bootstrap the configuration for a new project.15:28
bnemecIt probably isn't all that relevant for OpenStack since we use cookiecutter for that, but for external users I could see it being useful.15:28
bnemecI don't see any major problem with this and it's been out there long enough for anyone else to object.15:31
bnemecThere is one typo, but we can fix that in a followup.15:31
bnemecOkay, need to look at that one a little closer after the meeting, but I think it should be about ready to go.15:35
bnemec#topic Open discussion15:35
stephenfinbnemec: FWIW, I'm not overly gone on the idea, though I haven't been able to articulate "why" enough to warrant a minus vote15:35
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:35
kgiustibnemec: IIUC it can overwrite any existing setup.py/cfg - is that an issue15:35
stephenfinSomething about pbr already having too much stuff bolted on15:35
bnemeckgiusti: Yeah, I was wondering about that. It should probably be documented.15:36
stephenfinI'll go review15:36
*** herve4383 has quit IRC15:36
bnemecstephenfin: Thanks.15:36
*** sean-k-mooney has joined #openstack-oslo15:36
dhellmannkgiusti : did you see the ML thread about python 3 upgrade testing in which we discussed the idea of a functional test for oslo.messaging to show messages passing between python 2 and 3?15:37
dhellmann#link http://lists.openstack.org/pipermail/openstack-dev/2018-September/134592.html15:37
kgiustibnemec: re: oslo.messaging removals - there's only one patch outstanding (and one for releasenotes) but we've held it up because there are a couple of projects that still reference the removed code:15:37
kgiustihttps://review.openstack.org/#/q/topic:bug/1712399+status:open15:37
dhellmann#link http://lists.openstack.org/pipermail/openstack-dev/2018-September/134663.html15:38
kgiustiStevenK has patched these projects and reached out, but to no avail...15:38
kgiustidhellmann: /me reads...15:38
kgiustidhellmann: that's a fairly large hole in oslo.messaging testing15:39
dhellmannoh, I was hoping you'd say "we could modify the $x job to do that..."15:39
kgiustidhellmann: perhaps, I'd need to throw some grey matter at it for a bit15:40
dhellmannyeah, for now I just wanted to make sure you'd seen it15:40
kgiustidhellmann: thanks.  I'll look at it further.15:40
kgiustidhellmann: perhaps the system functional tests are the best place to put that.15:41
dhellmannthat seems reasonable15:41
dhellmannit doesn't have to be fancy15:41
dhellmannstand up a client on 2 and a server on 3 and makes some cast and call calls, then do the reverse15:42
*** Phuongnh_ has quit IRC15:42
kgiustidhellmann: I've got to dig around there a bit - I also need to remove the blocking executor so that's going to require some work on the tests as well.15:42
njohnstonI have a quick question about oslo.config... one of the proposals discussed for Neutron in the PTG had to do with taking the Provider Networks configuration that is currently in config files (and which Neutron can't function without) and creating CRUD API methods for it, migrating the values to a database.  The question was raised if oslo.config allows config to be read from a database, possibly in combination15:43
njohnston with a file.15:43
bnemecnjohnston: Not at the moment, but with the new driver functionality that could be added.15:43
bnemec#link http://specs.openstack.org/openstack/oslo-specs/specs/queens/oslo-config-drivers.html15:44
njohnstonbnemec: That's what I thought would be the most likely answer.  Thanks!15:44
bnemecHowever, that would require everyone to configure the database driver since it can't be added by default.15:44
dhellmannnjohnston : it doesn't today. I'm not sure the config data model would be flexible enough to support that. it might with the drivers, as bnemec proposes, but you might also want those values to come naturally from the data objects15:44
dhellmannfor example, adding a new one with oslo.config would require HUPping the service at least15:45
dhellmannwhereas doing it with the data models would just require re-reading the database in the application code15:45
njohnstondhellmann: Thanks, I will definitely note that as well15:46
njohnstonIt is an idea fraught with peril on many fronts, but if I can establish what a theoretical path would be we can take the discussion to gerrit review comments.15:46
njohnstonthanks bnemec dhellmann15:47
bnemecYeah, adding CRUD for config opts seems sketchy, but in theory it could be done.15:47
dhellmannif you're going to put them in the db, I think I'd just write a migration tool that does that and then rewrite any application code that uses the existing config values to use the db models15:47
dhellmannthat might be a good thing for the upgrade checker to report on, too15:47
njohnstonif we go that path, definitely15:48
bnemec+1. Maybe for one cycle both code paths exist to allow migration, and then in the next one you remove the config version.15:48
bnemecWe had a similar discussion around oslo.limit and upgrades.15:49
bnemecOkay, keep us in the loop on that.15:50
bnemecAnything else before we close?15:51
kgiustiyes15:51
kgiustiI'd like to get steve's final code patch to remove rpc_backend: https://review.openstack.org/#/c/580910/15:51
kgiustibut there are a couple of projects that have not accepted his patches: https://review.openstack.org/#/q/topic:bug/1712399+status:open15:52
kgiustiWe've given them enough time to adopt the changes15:52
kgiustiSo I'm in favor of moving forward - does anyone disagree?15:52
bnemecSo two of those do appear to be dead. Nothing has merged in months for either. Not even a core review.15:52
kgiusti+1 yep15:53
bnemecTacker does seem to be active still though. Can we send something to the list to get the attention of their cores?15:53
bnemecOr maybe add them to the review?15:53
bnemecOh, weird. One of the +1's is from a core. :-/15:54
bnemecThe incorrect -1 might be preventing them from looking at it though.15:54
kgiustibnemec: I can do that, but I don't want to wait too long into the cycle for this change to land15:55
bnemeckgiusti: Understood, and I don't expect us to wait indefinitely.15:56
bnemecIt seems like we should be able to land the patch in tacker though, so unless we get completely stonewalled there I'd prefer to wait for that.15:56
bnemecThe other two we can send a message to the list, but I don't anticipate a response based on the lack of activity so I don't think we should wait.15:57
kgiustiI'll do both - thanks15:57
bnemecSounds good.15:57
bnemec#action kgiusti to contact tacker team about removing transport_url15:57
bnemec#action kgiusti to send message to openstack-dev for meteos and daisycloud about transport_url15:58
stephenfinAnother open. Since we discussed one of his other patches, I think this needs another look at https://review.openstack.org/#/c/561731/15:58
stephenfindhellmann had reviewed it and there were some opens around removing pbr's automatic versioning machinery. I'm looking at it as we speak but I'd appreciate other eyes15:59
bnemecAh, I had looked at that review too at one point.16:01
dhellmannkgiusti, bnemec : daisycloud and meteos are not official, so if they aren't responding on the mailing list or reviews I don't think we need to wait for them16:02
openstackgerritIlya Shakhat proposed openstack/osprofiler master: Add a job to run full Tempest with enabled profiler  https://review.openstack.org/60299216:02
kgiustidhellmann: thanks - I'll follow up on the mailing list and we can give them a few days before we pull the trigger (next week perhaps)16:03
kgiustidhellmann: also just fyi: https://bugs.launchpad.net/oslo.messaging/+bug/179297716:03
openstackLaunchpad bug 1792977 in oslo.messaging "Need a functional test to verify py2 <-> py3 interoperability" [High,Triaged]16:03
dhellmannkgiusti : thanks * 216:03
bnemecOkay, we're over time.16:06
bnemecI'll add a16:06
bnemec#action review https://review.openstack.org/#/c/56173116:06
bnemecso we follow up on that next week.16:06
bnemecThanks for joining!16:07
bnemec#endmeeting16:07
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"16:07
openstackMeeting ended Mon Sep 17 16:07:15 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:07
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-09-17-15.01.html16:07
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-09-17-15.01.txt16:07
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-09-17-15.01.log.html16:07
openstackgerritIlya Shakhat proposed openstack/osprofiler master: Collect traces from Tempest job  https://review.openstack.org/60258616:07
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Use openstackdocstheme for documentation  https://review.openstack.org/60086616:09
*** ducnv_ has quit IRC16:17
openstackgerritAndy Smith proposed openstack/oslo.messaging master: Switch driver to confluent-kafka client library  https://review.openstack.org/60319017:23
*** harlowja has joined #openstack-oslo17:26
*** AJaeger has joined #openstack-oslo17:29
AJaegerbnemec: what do you propose to do with https://review.openstack.org/#/c/600707/ ? I see the cover job failing for a long time, see http://zuul.openstack.org/builds.html?job_name=openstack-tox-cover&project=openstack%2Ftooz17:30
bnemecAJaeger: Let me poke at it a bit and see if I can get it working. If it's going to be too much trouble I'm okay with just removing it from that repo.17:34
bnemecIt's doing some non-standard stuff when running tests so it may be tricky to do coverage.17:35
AJaegerbnemec: please leave a comment on 600707 and tell me what to do - and I'l update. Thanks!17:38
bnemecAJaeger: Will do.17:38
AJaegercool17:38
*** tbarron_ is now known as tbarron17:41
*** shardy has quit IRC17:46
*** openstackgerrit has quit IRC18:10
*** aspiers has quit IRC18:13
*** openstackgerrit has joined #openstack-oslo18:15
openstackgerritBen Nemec proposed openstack/tooz master: Fix coverage tests  https://review.openstack.org/60320318:15
openstackgerritBen Nemec proposed openstack/tooz master: Use templates for cover  https://review.openstack.org/60070718:18
sileht/2218:18
AJaegerbnemec: thanks!18:18
bnemecnp18:19
AJaegerbnemec: if that works, I'll make the cover job voting again...18:19
* bnemec is slightly terrified by what coverage is reporting on that repo18:19
bnemecAJaeger: Sounds good18:19
*** aspiers has joined #openstack-oslo18:55
*** aspiers has quit IRC19:48
*** njohnston has quit IRC19:51
*** aspiers has joined #openstack-oslo20:03
*** lbragstad has quit IRC20:26
*** lbragstad has joined #openstack-oslo20:27
*** ansmith has quit IRC20:38
*** kgiusti has left #openstack-oslo20:42
*** bobh has quit IRC20:49
*** bobh has joined #openstack-oslo21:25
openstackgerritBen Nemec proposed openstack/oslo.upgradecheck master: Use python3 by default  https://review.openstack.org/60323221:28
*** raildo has quit IRC21:28
*** bobh has quit IRC21:30
openstackgerritBen Nemec proposed openstack/oslo.upgradecheck master: Add proper documentation  https://review.openstack.org/60323822:08
*** rcernin has joined #openstack-oslo22:08
openstackgerritGaetan Semet proposed openstack-dev/pbr master: cmd: 'pbr boostrap' create setup.[cfg,py] automatically  https://review.openstack.org/55948422:30
openstackgerritGaetan Semet proposed openstack-dev/pbr master: option to print only the version of a package  https://review.openstack.org/57761422:40
openstackgerritBen Nemec proposed openstack/tooz master: Fix coverage tests  https://review.openstack.org/60320322:44
openstackgerritBen Nemec proposed openstack/tooz master: Migrate to stestr  https://review.openstack.org/60324422:44
openstackgerritBen Nemec proposed openstack/tooz master: Migrate to stestr  https://review.openstack.org/60324422:45
openstackgerritBen Nemec proposed openstack/tooz master: Use templates for cover  https://review.openstack.org/60070722:46
bnemecBleh, forgot to add stestr as a test dep too. I wonder how that worked?22:50
*** spilla has joined #openstack-oslo22:50
*** spilla has quit IRC22:51
openstackgerritBen Nemec proposed openstack/tooz master: Migrate to stestr  https://review.openstack.org/60324422:53
* bnemec crosses fingers that this one is right22:53
*** tosky has quit IRC23:23

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!