Monday, 2019-02-18

*** dave-mccowan has joined #openstack-oslo01:21
openstackgerritweizj proposed openstack/oslo.config master: Update json module to jsonutils  https://review.openstack.org/63742602:09
openstackgerritweizj proposed openstack/oslo.utils master: Update json module to jsonutils  https://review.openstack.org/63742702:40
openstackgerritweizj proposed openstack/oslo.utils master: Update json module to jsonutils  https://review.openstack.org/63742702:42
*** dave-mccowan has quit IRC04:07
*** agopi has quit IRC04:15
*** lpetrut has joined #openstack-oslo04:43
*** lpetrut has quit IRC05:10
*** Luzi has joined #openstack-oslo05:46
*** jbadiapa has joined #openstack-oslo06:02
*** e0ne has joined #openstack-oslo06:41
*** e0ne has quit IRC06:47
*** aojea has joined #openstack-oslo07:19
*** e0ne has joined #openstack-oslo07:35
*** e0ne has quit IRC07:38
*** pcaruana has joined #openstack-oslo07:43
openstackgerritMerged openstack/oslo.concurrency master: Update hacking version  https://review.openstack.org/62764907:45
*** tosky has joined #openstack-oslo08:33
openstackgerritMerged openstack/oslo.cache master: uncap dogpile  https://review.openstack.org/63603708:57
*** aojea has quit IRC08:57
*** pcaruana|afk| has joined #openstack-oslo09:01
*** pcaruana has quit IRC09:02
*** e0ne has joined #openstack-oslo09:08
*** trident has joined #openstack-oslo09:38
*** pcaruana|afk| has quit IRC10:01
*** pcaruana has joined #openstack-oslo10:07
*** moguimar has joined #openstack-oslo10:09
*** yan0s has joined #openstack-oslo11:33
*** raildo has joined #openstack-oslo11:51
*** irclogbot_2 has quit IRC13:08
*** irclogbot_2 has joined #openstack-oslo13:10
openstackgerritweizj proposed openstack/oslo.rootwrap master: Update hacking version  https://review.openstack.org/62765913:51
*** sum12 has quit IRC13:56
*** agopi has joined #openstack-oslo13:59
*** sum12 has joined #openstack-oslo14:04
*** Luzi has quit IRC14:04
*** jroll has quit IRC14:13
*** elbragstad has joined #openstack-oslo14:14
*** jroll has joined #openstack-oslo14:14
*** elbragstad is now known as lbragstad14:24
*** ansmith has joined #openstack-oslo14:31
*** bnemec-pto is now known as bnemec14:37
*** aojea has joined #openstack-oslo14:53
*** kgiusti has joined #openstack-oslo14:55
*** aojea has quit IRC14:57
*** dave-mccowan has joined #openstack-oslo14:58
* bnemec in another meeting15:00
bnemecWill start the Oslo meeting shortly.15:00
*** dave-mccowan has quit IRC15:03
bnemec#startmeeting oslo15:06
openstackMeeting started Mon Feb 18 15:06:05 2019 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:06
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:06
*** openstack changes topic to " (Meeting topic: oslo)"15:06
openstackThe meeting name has been set to 'oslo'15:06
bnemeccourtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims15:06
bnemeccourtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo15:06
bnemeccourtesy ping for hberaud, jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_15:06
bnemeccourtesy ping for moguimar, njohnston, raildo, redrobot, sileht, sreshetnyak, stephenfin15:06
bnemeccourtesy ping for stevemar, therve, thinrichs, toabctl, zhiyan, zxy, zzzeek15:06
moguimaro/15:06
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:06
*** aojea has joined #openstack-oslo15:06
ansmitho/15:07
kgiustio/15:07
moguimarBen, quick question, the Barbican guys asked when is the next castellan comming out. Where can I find info about Oslo releases and schedule?15:08
bnemecmoguimar: Oslo releases everything with changes weekly.15:09
bnemecHowever, I should double-check that Castellan is in our list.15:09
bnemecWe ran into a situation recently with pycadf (I think) where it wasn't in the oslo list because it was owned by the keystone team in the releases repo.15:10
bnemecIt's possible Castellan is owned by Barbican and doesn't show up in my list.15:10
bnemecNope, it's Oslo: https://github.com/openstack/releases/blob/master/deliverables/stein/castellan.yaml15:10
bnemecAnd was released about a month ago.15:10
bnemecI assume there haven't been any substantial changes since then.15:11
bnemec#topic Releases15:11
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:11
bnemecSince we're already discussing this topic. :-)15:11
bnemecOh crud, I missed that castellan had new changes to release because they were showing up as October commits.15:13
bnemecThe more recent changes were just test and doc changes.15:13
bnemecmoguimar: Looks like I dropped the ball. I'll get Castellan released this week.15:14
bnemec#action bnemec to release castellan15:14
bnemecAnd of course I'll do the regular set of releases too.15:15
bnemecAnything else on releases?15:15
moguimarnot on my end15:15
bnemec#topic Red flags for/from liaisons15:16
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:16
bnemecBack to your regularly scheduled topic. :-)15:16
bnemecNot sure we have any liaisons here today though.15:17
bnemecOkay moving on then.15:18
bnemecI don't think there was anything to report on our side either.15:18
bnemec#topic Action items from last meeting15:18
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:18
bnemec"Investigate adding main thread execution to privsep"15:18
bnemecI haven't had a chance to look into this yet.15:19
bnemec"bnemec to update openstack-discuss about privsep/fork issue"15:19
bnemecHowever, this is done, so the plan is on the record.15:19
bnemecIf anyone has free cycles and wants to look into this, please feel free.15:19
bnemec"moguimar to take over https://review.openstack.org/579186"15:20
bnemecI saw discussion on this, so calling it done.15:20
moguimarworking on it15:20
moguimarnot done =P15:20
bnemecCool.15:20
bnemecWell, you're working on it, so the action item is done. :-P15:20
moguimarah, ok15:20
bnemecI'm all about checking the done boxes in the meeting. :-)15:21
bnemecAnd that was it for action items.15:21
bnemec#topic Oslo Feature Freeze Feb. 2215:21
*** openstack changes topic to "Oslo Feature Freeze Feb. 22 (Meeting topic: oslo)"15:21
bnemecPer https://releases.openstack.org/stein/schedule.html15:21
bnemecOslo feature freeze is this week. Any features after Friday will require an FFE.15:22
bnemecI'll send email to the list about this since we're earlier with feature freeze than any other projects.15:22
bnemec#action bnemec send email to openstack-discuss about Oslo feature freeze15:23
bnemecI guess that's it. If you have feature patches open that you want in this cycle, make sure they merge by Friday!15:24
bnemec#topic Technical vision evaluation15:24
*** openstack changes topic to "Technical vision evaluation (Meeting topic: oslo)"15:24
bnemecWe need to do this.15:24
bnemecI mean to do more homework on it before this meeting, but unfortunately didn't get a chance. :-/15:24
bnemecFor details on this, see:15:25
bnemec#link http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001417.html15:25
bnemecMy thought is that since Oslo doesn't have one main source repo, we just put this in our policies section in the specs repo.15:25
bnemecI realize it doesn't quite fit there, but unless someone has a better option I think we should go with it.15:26
bnemecI'm not sure how much we'll have to do since I'm pretty sure they added a bullet point to the vision doc specifically to address Oslo.15:27
bnemecSo unless we're doing something wrong, we should already be in alignment.15:27
bnemecIf that's the case then writing the bullet points should be easy, but I'd like to at least do the evaluation so I can reply "Done" to the ML thread. :-)15:28
* bnemec notices a theme for this week's meeting15:28
bnemec#action Oslo team perform technical vision evaluation15:29
bnemecWe'll see if that gets done by next week, but hopefully.15:30
moguimarbnemec: you remind me of the monday.com add on youtube15:30
bnemec#topic Messaging singleton for get_transport_url15:30
*** openstack changes topic to "Messaging singleton for get_transport_url (Meeting topic: oslo)"15:30
bnemecmoguimar: lol15:30
bnemecI mean, that's pretty much the life of a PTL.15:30
bnemecIt's basically a project management position.15:30
bnemecIf you're lucky you also get to write code once in a while. ;-)15:31
bnemeckgiusti: ansmith: Hey, we had a question from eandersson about https://review.openstack.org/#/c/63642815:31
bnemecApparently octavia was having issues with a huge number of rabbit connections piling up.15:31
bnemecHe was wondering if get_transport_url should be a singleton.15:32
kgiustibnemec: sorry I've been MIA last week (blame europe)15:32
johnsomYes, I don’t have all of the details there, so didn’t raise it earlier.15:33
bnemeckgiusti: Yeah, no worries. I was in the same boat a couple of weeks ago. :-)15:33
bnemecNot literally in a boat, you understand. I flew there.15:33
johnsomThe docs for oslo messaging don’t even mention the close method on the transport.15:33
* bnemec dodges rotten fruit15:33
kgiustibnemec: several projects do their own singleton-ish wrapper under the covers15:33
bnemecYeah, I think that's what eandersson is proposing in that patch.15:34
bnemecShould we document that as a best practice?15:34
johnsomWhat I don’t know is why they didn’t close when they went out of scope15:34
kgiustibnemec: documenting that would be a good idea yes.15:35
kgiustibnemec: there15:35
bnemeckgiusti: Okay, sounds good. If we can somehow avoid this bad behavior on the Oslo side that would be even better.15:35
kgiustibnemec: oops - the TCP connection per transport is only implied15:35
bnemecNot sure we can cache it on the library side though since it's generated from a conf object and a url.15:36
kgiustibnemec: one sec...15:36
bnemecWe don't necessarily need to resolve this now.15:38
kgiustibnemec: can't find the specific code but impl_rabbit uses dedicated conns for some reason15:38
kgiustibnemec: which also blows up the # of connections15:39
bnemecI just wanted to bring it up since I know next to nothing about messaging and was completely unhelpful on this issue. :-)15:39
kgiustibnemec: np.15:39
kgiustibnemec: I'll look into it15:39
bnemeckgiusti: Great, thanks!15:39
bnemec#action kgiusti to investigate connection explosion related to get_transport_url15:40
bnemecThat was it for topics. Let's see which review we can check off this list...15:40
bnemec#topic Weekly Wayward Review15:40
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:40
bnemec#link https://review.openstack.org/#/c/624509/15:41
bnemecPretty simple one.15:41
bnemecThis was causing some spurious unit test failures for me at one point.15:42
*** e0ne has quit IRC15:42
bnemecOh crud, I never added a test for that.15:44
bnemecOkay, never mind. I'll WIP that patch until I can add tests.15:44
bnemecWell, that gets it out of the review queue at least. :-)15:45
bnemec#topic Open discussion15:45
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:45
bnemecAnything else before we end the meeting for this week?15:45
moguimarI accidentally ran into kgiusti last week =P15:46
bnemecmoguimar: I hope you didn't hurt him. :-P15:46
moguimarhe was spawning in every room I walked into at the office15:47
kgiustimoguimar: don't tell my wife....15:47
bnemecHax!15:47
moguimarxD15:47
bnemeclol15:47
bnemeckgiusti: went a very different direction with spawning than I did.15:47
kgiustibnemec: sorry15:48
*** e0ne has joined #openstack-oslo15:48
bnemeckgiusti: Not at all, yours was better anyway. :-)15:49
moguimarI was able to introduce him to the best facilities we have at the Brno office15:50
bnemecYou mean Pegas? ;-)15:50
kgiustimoguimar: the facilities were impressive, but not as impressive as the local brew - it's literally cheaper than water!15:51
* bnemec 's mouth still waters thinking about those ribs15:51
moguimaron my second visit to Prague, for PyCon CZ, my friends from Brazil asked me if beer was realy cheaper than water15:52
moguimarI said:  ¯\_(ツ)_/¯15:52
moguimarI didn't buy any water there15:52
bnemeclol15:52
bnemecmoguimar: BTW, I don't know if you have any interest in https://review.openstack.org/#/c/614224/ but it's going to be affected by feature freeze.15:54
moguimarbut IIRC there is a law that says there should be a non alcoholic drink in the menu cheaper than the alcoholic ones15:54
moguimarI'll let jaosorior know about that15:54
jaosoriorI didn't get much momentum for it15:55
jaosoriorso I guess it'll have to wait15:55
bnemecSorry about that. I should have looked at it before last week.15:55
bnemecOh, we had a spec open for this too: https://review.openstack.org/#/c/578719/15:57
bnemecOkay, there's a bunch of discussion around this that probably still needs to be resolved, so I guess this isn't likely to land this cycle.15:59
bnemecAnd we're just about out of time.15:59
bnemecThanks for joining everyone!15:59
bnemec#endmeeting15:59
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:59
openstackMeeting ended Mon Feb 18 15:59:21 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:59
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.html15:59
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.txt15:59
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.log.html15:59
bnemecAnd I have another meeting on my calendar.16:00
bnemecHow am I supposed to get any work done?!16:00
bnemec;-)16:00
*** pcaruana has quit IRC16:10
*** cdent has joined #openstack-oslo16:10
cdentbnemec: I would be very happy if we could get this (and the feature that has already merged) into this cycle: https://review.openstack.org/#/c/636584/16:11
openstackgerritMagnus Bergman proposed openstack/oslo.messaging master: Bump amqp requirement version to >=2.4.1  https://review.openstack.org/63757616:21
*** e0ne has quit IRC16:34
openstackgerritDoug Wiegley proposed openstack/oslo.versionedobjects master: Allow lists to be generated from any non-string iterable  https://review.openstack.org/63738916:51
bnemeccdent: Yeah, thanks. That was on my list to look at anyway.16:51
cdent\o/ thanks!16:53
openstackgerritPavlo Shchelokovskyy proposed openstack/oslo.log master: Clarify some config options  https://review.openstack.org/63758717:01
openstackgerritMerged openstack/oslo.upgradecheck master: Add doc params to run and register_cli_opts  https://review.openstack.org/63658417:03
*** aojea has quit IRC17:04
openstackgerritBen Nemec proposed openstack/oslo-specs master: Add co-owned library exception to feature freeze policy  https://review.openstack.org/63758817:06
njohnston_bnemec: Who are the best people to ask about oslo.messaging?  I have a change to cleanup an old TODO and I want to make sure I am approaching it the right way.  https://review.openstack.org/63732317:07
bnemecnjohnston_: kgiusti and ansmith are your huckleberries17:08
njohnston_bnemec: Thanks!17:08
*** moguimar has quit IRC17:16
*** yan0s has quit IRC17:21
openstackgerritMagnus Bergman proposed openstack/oslo.messaging master: Bump amqp requirement version to >=2.4.1  https://review.openstack.org/63757617:30
*** e0ne has joined #openstack-oslo18:24
*** ansmith_ has joined #openstack-oslo18:28
*** ansmith has quit IRC18:30
*** raildo has quit IRC18:49
*** ansmith_ has quit IRC18:50
*** ansmith_ has joined #openstack-oslo18:51
*** raildo has joined #openstack-oslo19:14
*** e0ne has quit IRC19:30
*** ansmith has joined #openstack-oslo19:34
*** ansmith_ has quit IRC19:35
*** e0ne has joined #openstack-oslo19:37
*** zaneb has quit IRC19:50
eanderssonbnemec, best practices would be nice19:56
eanderssonMy idea was to fix it by following a common pattern for now, and then work with the oslo team to come up with "modern" best practices19:57
*** raildo has quit IRC20:04
bnemeceandersson: Yeah, that sounds about like what we discussed. Document the best practice for now, and see if there's a way to improve things on the Oslo side to avoid this in the future.20:05
bnemeckgiusti: ansmith: Looks like we have a breakage in the messaging integration tests.20:05
bnemecRelated to gnocchi.20:05
bnemechttp://logs.openstack.org/76/610676/6/check/oslo.messaging-telemetry-dsvm-integration-rabbit/54660e8/job-output.txt.gz20:06
bnemecgnocchiclient.exceptions.NotFound: {u'cause': u'Aggregation method does not exist for this metric', u'detail': {u'aggregation_method': u'rate:mean', u'metric': u'ee512d52-ee9e-4624-b81a-aa7fe41f88cc'}}20:06
ansmithbnemec: also hit this today20:08
kgiustibnemec: I'm inclined to mark those telemetry tests as non-voting20:08
ansmith+120:08
kgiustibnemec: last time they broke they stayed broken for awhile - then unbroke without any changes on our behalf20:09
bnemecI'm okay with that, especially if we think the problem is on their end.20:10
bnemecWe can always flip the bit back if/when they're working again.20:10
ansmithbnemec, kgiusti: i will submit a patch to change to nv20:10
kgiustibnemec: we've got definitions of these tests locally, in our playbooks.  Ideally we should be using a common definition maintained by, well, not us.20:12
kgiustiansmith: thanks20:12
bnemecAlso, given that gnocchi doesn't live in openstack anymore and we can't co-gate, we probably shouldn't be blocking our repo on them.20:16
kgiustibnemec: yeah I think the tests in oslo.messaging are a copy of some other project's tests (perhaps gnocchi)20:17
kgiustibnemec: probably our copies are stale20:18
bnemecOh, that's kind of ugly.20:18
bnemecBut I just read on Twitter that copy-paste development is the new hotness, so I guess it's okay. :-P20:18
kgiustibnemec: +1 I'd rather use a test exported by the projects we're trying to vet against myself.20:19
openstackgerritAndy Smith proposed openstack/oslo.messaging master: Mark telemetry tests nv and remove from gate  https://review.openstack.org/63762220:20
openstackgerritAndy Smith proposed openstack/oslo.messaging master: Update messaging intermediaries for amqp1 tests  https://review.openstack.org/62507720:23
*** agopi has quit IRC20:37
*** cdent has left #openstack-oslo20:55
*** njohnston_ has quit IRC21:04
*** zaneb has joined #openstack-oslo21:12
*** zaneb has quit IRC21:37
*** rcernin has joined #openstack-oslo21:40
*** e0ne has quit IRC22:03
*** e0ne has joined #openstack-oslo22:04
*** ansmith has quit IRC22:05
*** kgiusti has quit IRC22:09
*** dave-mccowan has joined #openstack-oslo22:17
*** dave-mccowan has quit IRC22:21
*** e0ne has quit IRC23:12
*** e0ne has joined #openstack-oslo23:23
*** e0ne has quit IRC23:30
*** agopi has joined #openstack-oslo23:47

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