*** dave-mccowan has joined #openstack-oslo | 01:21 | |
openstackgerrit | weizj proposed openstack/oslo.config master: Update json module to jsonutils https://review.openstack.org/637426 | 02:09 |
---|---|---|
openstackgerrit | weizj proposed openstack/oslo.utils master: Update json module to jsonutils https://review.openstack.org/637427 | 02:40 |
openstackgerrit | weizj proposed openstack/oslo.utils master: Update json module to jsonutils https://review.openstack.org/637427 | 02:42 |
*** dave-mccowan has quit IRC | 04:07 | |
*** agopi has quit IRC | 04:15 | |
*** lpetrut has joined #openstack-oslo | 04:43 | |
*** lpetrut has quit IRC | 05:10 | |
*** Luzi has joined #openstack-oslo | 05:46 | |
*** jbadiapa has joined #openstack-oslo | 06:02 | |
*** e0ne has joined #openstack-oslo | 06:41 | |
*** e0ne has quit IRC | 06:47 | |
*** aojea has joined #openstack-oslo | 07:19 | |
*** e0ne has joined #openstack-oslo | 07:35 | |
*** e0ne has quit IRC | 07:38 | |
*** pcaruana has joined #openstack-oslo | 07:43 | |
openstackgerrit | Merged openstack/oslo.concurrency master: Update hacking version https://review.openstack.org/627649 | 07:45 |
*** tosky has joined #openstack-oslo | 08:33 | |
openstackgerrit | Merged openstack/oslo.cache master: uncap dogpile https://review.openstack.org/636037 | 08:57 |
*** aojea has quit IRC | 08:57 | |
*** pcaruana|afk| has joined #openstack-oslo | 09:01 | |
*** pcaruana has quit IRC | 09:02 | |
*** e0ne has joined #openstack-oslo | 09:08 | |
*** trident has joined #openstack-oslo | 09:38 | |
*** pcaruana|afk| has quit IRC | 10:01 | |
*** pcaruana has joined #openstack-oslo | 10:07 | |
*** moguimar has joined #openstack-oslo | 10:09 | |
*** yan0s has joined #openstack-oslo | 11:33 | |
*** raildo has joined #openstack-oslo | 11:51 | |
*** irclogbot_2 has quit IRC | 13:08 | |
*** irclogbot_2 has joined #openstack-oslo | 13:10 | |
openstackgerrit | weizj proposed openstack/oslo.rootwrap master: Update hacking version https://review.openstack.org/627659 | 13:51 |
*** sum12 has quit IRC | 13:56 | |
*** agopi has joined #openstack-oslo | 13:59 | |
*** sum12 has joined #openstack-oslo | 14:04 | |
*** Luzi has quit IRC | 14:04 | |
*** jroll has quit IRC | 14:13 | |
*** elbragstad has joined #openstack-oslo | 14:14 | |
*** jroll has joined #openstack-oslo | 14:14 | |
*** elbragstad is now known as lbragstad | 14:24 | |
*** ansmith has joined #openstack-oslo | 14:31 | |
*** bnemec-pto is now known as bnemec | 14:37 | |
*** aojea has joined #openstack-oslo | 14:53 | |
*** kgiusti has joined #openstack-oslo | 14:55 | |
*** aojea has quit IRC | 14:57 | |
*** dave-mccowan has joined #openstack-oslo | 14:58 | |
* bnemec in another meeting | 15:00 | |
bnemec | Will start the Oslo meeting shortly. | 15:00 |
*** dave-mccowan has quit IRC | 15:03 | |
bnemec | #startmeeting oslo | 15:06 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:06 |
*** openstack changes topic to " (Meeting topic: oslo)" | 15:06 | |
openstack | The meeting name has been set to 'oslo' | 15:06 |
bnemec | courtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims | 15:06 |
bnemec | courtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo | 15:06 |
bnemec | courtesy ping for hberaud, jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_ | 15:06 |
bnemec | courtesy ping for moguimar, njohnston, raildo, redrobot, sileht, sreshetnyak, stephenfin | 15:06 |
bnemec | courtesy ping for stevemar, therve, thinrichs, toabctl, zhiyan, zxy, zzzeek | 15:06 |
moguimar | o/ | 15:06 |
bnemec | #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting | 15:06 |
*** aojea has joined #openstack-oslo | 15:06 | |
ansmith | o/ | 15:07 |
kgiusti | o/ | 15:07 |
moguimar | Ben, 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 |
bnemec | moguimar: Oslo releases everything with changes weekly. | 15:09 |
bnemec | However, I should double-check that Castellan is in our list. | 15:09 |
bnemec | We 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 |
bnemec | It's possible Castellan is owned by Barbican and doesn't show up in my list. | 15:10 |
bnemec | Nope, it's Oslo: https://github.com/openstack/releases/blob/master/deliverables/stein/castellan.yaml | 15:10 |
bnemec | And was released about a month ago. | 15:10 |
bnemec | I assume there haven't been any substantial changes since then. | 15:11 |
bnemec | #topic Releases | 15:11 |
*** openstack changes topic to "Releases (Meeting topic: oslo)" | 15:11 | |
bnemec | Since we're already discussing this topic. :-) | 15:11 |
bnemec | Oh crud, I missed that castellan had new changes to release because they were showing up as October commits. | 15:13 |
bnemec | The more recent changes were just test and doc changes. | 15:13 |
bnemec | moguimar: Looks like I dropped the ball. I'll get Castellan released this week. | 15:14 |
bnemec | #action bnemec to release castellan | 15:14 |
bnemec | And of course I'll do the regular set of releases too. | 15:15 |
bnemec | Anything else on releases? | 15:15 |
moguimar | not on my end | 15:15 |
bnemec | #topic Red flags for/from liaisons | 15:16 |
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:16 | |
bnemec | Back to your regularly scheduled topic. :-) | 15:16 |
bnemec | Not sure we have any liaisons here today though. | 15:17 |
bnemec | Okay moving on then. | 15:18 |
bnemec | I don't think there was anything to report on our side either. | 15:18 |
bnemec | #topic Action items from last meeting | 15: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 |
bnemec | I haven't had a chance to look into this yet. | 15:19 |
bnemec | "bnemec to update openstack-discuss about privsep/fork issue" | 15:19 |
bnemec | However, this is done, so the plan is on the record. | 15:19 |
bnemec | If 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 |
bnemec | I saw discussion on this, so calling it done. | 15:20 |
moguimar | working on it | 15:20 |
moguimar | not done =P | 15:20 |
bnemec | Cool. | 15:20 |
bnemec | Well, you're working on it, so the action item is done. :-P | 15:20 |
moguimar | ah, ok | 15:20 |
bnemec | I'm all about checking the done boxes in the meeting. :-) | 15:21 |
bnemec | And that was it for action items. | 15:21 |
bnemec | #topic Oslo Feature Freeze Feb. 22 | 15:21 |
*** openstack changes topic to "Oslo Feature Freeze Feb. 22 (Meeting topic: oslo)" | 15:21 | |
bnemec | Per https://releases.openstack.org/stein/schedule.html | 15:21 |
bnemec | Oslo feature freeze is this week. Any features after Friday will require an FFE. | 15:22 |
bnemec | I'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 freeze | 15:23 |
bnemec | I 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 evaluation | 15:24 |
*** openstack changes topic to "Technical vision evaluation (Meeting topic: oslo)" | 15:24 | |
bnemec | We need to do this. | 15:24 |
bnemec | I mean to do more homework on it before this meeting, but unfortunately didn't get a chance. :-/ | 15:24 |
bnemec | For details on this, see: | 15:25 |
bnemec | #link http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001417.html | 15:25 |
bnemec | My 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 |
bnemec | I realize it doesn't quite fit there, but unless someone has a better option I think we should go with it. | 15:26 |
bnemec | I'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 |
bnemec | So unless we're doing something wrong, we should already be in alignment. | 15:27 |
bnemec | If 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 meeting | 15:28 | |
bnemec | #action Oslo team perform technical vision evaluation | 15:29 |
bnemec | We'll see if that gets done by next week, but hopefully. | 15:30 |
moguimar | bnemec: you remind me of the monday.com add on youtube | 15:30 |
bnemec | #topic Messaging singleton for get_transport_url | 15:30 |
*** openstack changes topic to "Messaging singleton for get_transport_url (Meeting topic: oslo)" | 15:30 | |
bnemec | moguimar: lol | 15:30 |
bnemec | I mean, that's pretty much the life of a PTL. | 15:30 |
bnemec | It's basically a project management position. | 15:30 |
bnemec | If you're lucky you also get to write code once in a while. ;-) | 15:31 |
bnemec | kgiusti: ansmith: Hey, we had a question from eandersson about https://review.openstack.org/#/c/636428 | 15:31 |
bnemec | Apparently octavia was having issues with a huge number of rabbit connections piling up. | 15:31 |
bnemec | He was wondering if get_transport_url should be a singleton. | 15:32 |
kgiusti | bnemec: sorry I've been MIA last week (blame europe) | 15:32 |
johnsom | Yes, I don’t have all of the details there, so didn’t raise it earlier. | 15:33 |
bnemec | kgiusti: Yeah, no worries. I was in the same boat a couple of weeks ago. :-) | 15:33 |
bnemec | Not literally in a boat, you understand. I flew there. | 15:33 |
johnsom | The docs for oslo messaging don’t even mention the close method on the transport. | 15:33 |
* bnemec dodges rotten fruit | 15:33 | |
kgiusti | bnemec: several projects do their own singleton-ish wrapper under the covers | 15:33 |
bnemec | Yeah, I think that's what eandersson is proposing in that patch. | 15:34 |
bnemec | Should we document that as a best practice? | 15:34 |
johnsom | What I don’t know is why they didn’t close when they went out of scope | 15:34 |
kgiusti | bnemec: documenting that would be a good idea yes. | 15:35 |
kgiusti | bnemec: there | 15:35 |
bnemec | kgiusti: Okay, sounds good. If we can somehow avoid this bad behavior on the Oslo side that would be even better. | 15:35 |
kgiusti | bnemec: oops - the TCP connection per transport is only implied | 15:35 |
bnemec | Not sure we can cache it on the library side though since it's generated from a conf object and a url. | 15:36 |
kgiusti | bnemec: one sec... | 15:36 |
bnemec | We don't necessarily need to resolve this now. | 15:38 |
kgiusti | bnemec: can't find the specific code but impl_rabbit uses dedicated conns for some reason | 15:38 |
kgiusti | bnemec: which also blows up the # of connections | 15:39 |
bnemec | I just wanted to bring it up since I know next to nothing about messaging and was completely unhelpful on this issue. :-) | 15:39 |
kgiusti | bnemec: np. | 15:39 |
kgiusti | bnemec: I'll look into it | 15:39 |
bnemec | kgiusti: Great, thanks! | 15:39 |
bnemec | #action kgiusti to investigate connection explosion related to get_transport_url | 15:40 |
bnemec | That was it for topics. Let's see which review we can check off this list... | 15:40 |
bnemec | #topic Weekly Wayward Review | 15:40 |
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)" | 15:40 | |
bnemec | #link https://review.openstack.org/#/c/624509/ | 15:41 |
bnemec | Pretty simple one. | 15:41 |
bnemec | This was causing some spurious unit test failures for me at one point. | 15:42 |
*** e0ne has quit IRC | 15:42 | |
bnemec | Oh crud, I never added a test for that. | 15:44 |
bnemec | Okay, never mind. I'll WIP that patch until I can add tests. | 15:44 |
bnemec | Well, that gets it out of the review queue at least. :-) | 15:45 |
bnemec | #topic Open discussion | 15:45 |
*** openstack changes topic to "Open discussion (Meeting topic: oslo)" | 15:45 | |
bnemec | Anything else before we end the meeting for this week? | 15:45 |
moguimar | I accidentally ran into kgiusti last week =P | 15:46 |
bnemec | moguimar: I hope you didn't hurt him. :-P | 15:46 |
moguimar | he was spawning in every room I walked into at the office | 15:47 |
kgiusti | moguimar: don't tell my wife.... | 15:47 |
bnemec | Hax! | 15:47 |
moguimar | xD | 15:47 |
bnemec | lol | 15:47 |
bnemec | kgiusti: went a very different direction with spawning than I did. | 15:47 |
kgiusti | bnemec: sorry | 15:48 |
*** e0ne has joined #openstack-oslo | 15:48 | |
bnemec | kgiusti: Not at all, yours was better anyway. :-) | 15:49 |
moguimar | I was able to introduce him to the best facilities we have at the Brno office | 15:50 |
bnemec | You mean Pegas? ;-) | 15:50 |
kgiusti | moguimar: 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 ribs | 15:51 | |
moguimar | on my second visit to Prague, for PyCon CZ, my friends from Brazil asked me if beer was realy cheaper than water | 15:52 |
moguimar | I said: ¯\_(ツ)_/¯ | 15:52 |
moguimar | I didn't buy any water there | 15:52 |
bnemec | lol | 15:52 |
bnemec | moguimar: 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 |
moguimar | but IIRC there is a law that says there should be a non alcoholic drink in the menu cheaper than the alcoholic ones | 15:54 |
moguimar | I'll let jaosorior know about that | 15:54 |
jaosorior | I didn't get much momentum for it | 15:55 |
jaosorior | so I guess it'll have to wait | 15:55 |
bnemec | Sorry about that. I should have looked at it before last week. | 15:55 |
bnemec | Oh, we had a spec open for this too: https://review.openstack.org/#/c/578719/ | 15:57 |
bnemec | Okay, 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 |
bnemec | And we're just about out of time. | 15:59 |
bnemec | Thanks for joining everyone! | 15:59 |
bnemec | #endmeeting | 15:59 |
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 15:59 | |
openstack | Meeting ended Mon Feb 18 15:59:21 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.html | 15:59 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.txt | 15:59 |
openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-18-15.06.log.html | 15:59 |
bnemec | And I have another meeting on my calendar. | 16:00 |
bnemec | How am I supposed to get any work done?! | 16:00 |
bnemec | ;-) | 16:00 |
*** pcaruana has quit IRC | 16:10 | |
*** cdent has joined #openstack-oslo | 16:10 | |
cdent | bnemec: 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 |
openstackgerrit | Magnus Bergman proposed openstack/oslo.messaging master: Bump amqp requirement version to >=2.4.1 https://review.openstack.org/637576 | 16:21 |
*** e0ne has quit IRC | 16:34 | |
openstackgerrit | Doug Wiegley proposed openstack/oslo.versionedobjects master: Allow lists to be generated from any non-string iterable https://review.openstack.org/637389 | 16:51 |
bnemec | cdent: Yeah, thanks. That was on my list to look at anyway. | 16:51 |
cdent | \o/ thanks! | 16:53 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/oslo.log master: Clarify some config options https://review.openstack.org/637587 | 17:01 |
openstackgerrit | Merged openstack/oslo.upgradecheck master: Add doc params to run and register_cli_opts https://review.openstack.org/636584 | 17:03 |
*** aojea has quit IRC | 17:04 | |
openstackgerrit | Ben Nemec proposed openstack/oslo-specs master: Add co-owned library exception to feature freeze policy https://review.openstack.org/637588 | 17: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/637323 | 17:07 |
bnemec | njohnston_: kgiusti and ansmith are your huckleberries | 17:08 |
njohnston_ | bnemec: Thanks! | 17:08 |
*** moguimar has quit IRC | 17:16 | |
*** yan0s has quit IRC | 17:21 | |
openstackgerrit | Magnus Bergman proposed openstack/oslo.messaging master: Bump amqp requirement version to >=2.4.1 https://review.openstack.org/637576 | 17:30 |
*** e0ne has joined #openstack-oslo | 18:24 | |
*** ansmith_ has joined #openstack-oslo | 18:28 | |
*** ansmith has quit IRC | 18:30 | |
*** raildo has quit IRC | 18:49 | |
*** ansmith_ has quit IRC | 18:50 | |
*** ansmith_ has joined #openstack-oslo | 18:51 | |
*** raildo has joined #openstack-oslo | 19:14 | |
*** e0ne has quit IRC | 19:30 | |
*** ansmith has joined #openstack-oslo | 19:34 | |
*** ansmith_ has quit IRC | 19:35 | |
*** e0ne has joined #openstack-oslo | 19:37 | |
*** zaneb has quit IRC | 19:50 | |
eandersson | bnemec, best practices would be nice | 19:56 |
eandersson | My 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 practices | 19:57 |
*** raildo has quit IRC | 20:04 | |
bnemec | eandersson: 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 |
bnemec | kgiusti: ansmith: Looks like we have a breakage in the messaging integration tests. | 20:05 |
bnemec | Related to gnocchi. | 20:05 |
bnemec | http://logs.openstack.org/76/610676/6/check/oslo.messaging-telemetry-dsvm-integration-rabbit/54660e8/job-output.txt.gz | 20:06 |
bnemec | gnocchiclient.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 |
ansmith | bnemec: also hit this today | 20:08 |
kgiusti | bnemec: I'm inclined to mark those telemetry tests as non-voting | 20:08 |
ansmith | +1 | 20:08 |
kgiusti | bnemec: last time they broke they stayed broken for awhile - then unbroke without any changes on our behalf | 20:09 |
bnemec | I'm okay with that, especially if we think the problem is on their end. | 20:10 |
bnemec | We can always flip the bit back if/when they're working again. | 20:10 |
ansmith | bnemec, kgiusti: i will submit a patch to change to nv | 20:10 |
kgiusti | bnemec: 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 |
kgiusti | ansmith: thanks | 20:12 |
bnemec | Also, 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 |
kgiusti | bnemec: yeah I think the tests in oslo.messaging are a copy of some other project's tests (perhaps gnocchi) | 20:17 |
kgiusti | bnemec: probably our copies are stale | 20:18 |
bnemec | Oh, that's kind of ugly. | 20:18 |
bnemec | But I just read on Twitter that copy-paste development is the new hotness, so I guess it's okay. :-P | 20:18 |
kgiusti | bnemec: +1 I'd rather use a test exported by the projects we're trying to vet against myself. | 20:19 |
openstackgerrit | Andy Smith proposed openstack/oslo.messaging master: Mark telemetry tests nv and remove from gate https://review.openstack.org/637622 | 20:20 |
openstackgerrit | Andy Smith proposed openstack/oslo.messaging master: Update messaging intermediaries for amqp1 tests https://review.openstack.org/625077 | 20:23 |
*** agopi has quit IRC | 20:37 | |
*** cdent has left #openstack-oslo | 20:55 | |
*** njohnston_ has quit IRC | 21:04 | |
*** zaneb has joined #openstack-oslo | 21:12 | |
*** zaneb has quit IRC | 21:37 | |
*** rcernin has joined #openstack-oslo | 21:40 | |
*** e0ne has quit IRC | 22:03 | |
*** e0ne has joined #openstack-oslo | 22:04 | |
*** ansmith has quit IRC | 22:05 | |
*** kgiusti has quit IRC | 22:09 | |
*** dave-mccowan has joined #openstack-oslo | 22:17 | |
*** dave-mccowan has quit IRC | 22:21 | |
*** e0ne has quit IRC | 23:12 | |
*** e0ne has joined #openstack-oslo | 23:23 | |
*** e0ne has quit IRC | 23:30 | |
*** agopi has joined #openstack-oslo | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!