Monday, 2019-06-17

*** dave-mccowan has joined #openstack-oslo00:20
*** bobh has quit IRC01:18
*** bobh has joined #openstack-oslo01:24
*** dave-mccowan has quit IRC01:45
*** bobh has quit IRC02:00
*** goldyfruit has joined #openstack-oslo02:23
*** ianychoi has quit IRC02:25
*** ianychoi has joined #openstack-oslo02:26
*** goldyfruit has quit IRC02:42
*** Luzi has joined #openstack-oslo05:56
*** rcernin has quit IRC06:02
*** jbadiapa has joined #openstack-oslo06:07
*** rcernin has joined #openstack-oslo07:00
*** trident has quit IRC07:06
*** trident has joined #openstack-oslo07:08
*** tosky has joined #openstack-oslo07:21
*** tesseract has joined #openstack-oslo07:24
*** iurygregory has joined #openstack-oslo07:26
*** hberaud|gone is now known as hberaud07:34
*** ralonsoh has joined #openstack-oslo08:02
*** jbadiapa has quit IRC08:50
*** dciabrin has quit IRC08:51
*** pcaruana has joined #openstack-oslo08:52
*** jbadiapa has joined #openstack-oslo09:03
gsantomaggio@hberaud  about https://review.opendev.org/#/c/664275/109:07
gsantomaggioI tried the tests locally and it works :((09:07
hberaudgsantomaggio: I'll take a look09:17
hberaudgsantomaggio: ty09:17
*** tosky__ has joined #openstack-oslo09:20
*** tosky is now known as Guest3174409:20
*** tosky__ is now known as tosky09:20
hberaudgsantomaggio: you've tested with or without the patch?09:20
gsantomaggio```09:21
gsantomaggiogit review -d 66427509:21
gsantomaggiogit branch:  review/ben_nemec/kafka-archive09:21
gsantomaggio```09:21
hberaudack09:22
*** Guest31744 has quit IRC09:23
hberaudgsantomaggio: the CI is mostly a random system.... depends on the system load etc... and the stars alignment...09:25
hberaudgsantomaggio: work for me too locally09:25
gsantomaggio@hberaud  recheck ?!09:26
hberaudgsantomaggio: as I can see it's not a blocking test, but why not, it can be useful to see this test became ok in an testing env like CI09:27
hberaudgsantomaggio: the main issue at the origin is that the kafka test break the CI so, yep recheck make sense09:28
gsantomaggio@hberaud  done09:29
hberaudgsantomaggio: ty09:30
*** rcernin has quit IRC09:36
*** finucannot is now known as stephenfin09:36
*** jbadiapa has quit IRC09:54
*** hberaud is now known as hberaud|school-r09:57
*** hberaud|school-r is now known as hberaud10:05
*** jbadiapa has joined #openstack-oslo10:07
*** hberaud is now known as hberaud|lunch10:23
openstackgerritNatal Ngétal proposed openstack/oslo.log master: Modify the constraints url in tox.  https://review.opendev.org/66110010:41
*** moguimar has joined #openstack-oslo11:12
*** ansmith has quit IRC11:23
*** hberaud|lunch is now known as hberaud11:24
*** boden has joined #openstack-oslo11:55
*** starborn has joined #openstack-oslo11:56
*** bobh has joined #openstack-oslo12:33
*** bobh has quit IRC12:37
*** dave-mccowan has joined #openstack-oslo12:42
*** ansmith has joined #openstack-oslo12:46
*** kgiusti has joined #openstack-oslo13:10
*** lbragstad has joined #openstack-oslo13:12
*** raildo has joined #openstack-oslo13:15
*** beekneemech is now known as bnemec13:18
*** goldyfruit has joined #openstack-oslo13:24
*** pcaruana|afk| has joined #openstack-oslo13:26
*** pcaruana has quit IRC13:26
*** iurygregory_ has joined #openstack-oslo13:40
*** iurygregory has quit IRC13:42
*** pcaruana has joined #openstack-oslo13:48
*** pcaruana|afk| has quit IRC13:51
*** Luzi has quit IRC14:03
*** e0ne has joined #openstack-oslo14:21
*** e0ne has quit IRC14:26
*** hberaud is now known as hberaud|school-r14:26
*** e0ne has joined #openstack-oslo14:36
*** pcaruana has quit IRC14:47
*** bobh has joined #openstack-oslo14:50
*** hberaud|school-r is now known as hberaud14:50
bnemec#startmeeting oslo15:00
openstackMeeting started Mon Jun 17 15:00:27 2019 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: oslo)"15:00
openstackThe meeting name has been set to 'oslo'15:00
bnemecCourtesy ping for bnemec, jungleboyj, moguimar, hberaud, kgiusti, redrobot, stephenfin, johnsom15:00
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:00
hberaudo/15:00
johnsomo/15:00
jungleboyjo/15:00
moguimaro/15:00
bnemecFirst week of the new ping list. Hope nobody missed the transition. :-)15:01
ansmitho/15:02
jungleboyj:-)15:02
bnemec#topic Red flags for/from liaisons15:03
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:03
moguimarnone from Barbican15:03
johnsomNothing from Octavia team15:03
kgiustio/15:04
moguimaralthough redrobot asked last week about feature freeze dates15:04
jungleboyjNothing from Cinder.  :-)15:04
bnemecmoguimar: Oslo-specific dates are on the releases page: https://releases.openstack.org/train/schedule.html15:04
moguimarack15:04
bnemecWe only have feature freeze since we don't really do spec freeze or anything like that.15:05
bnemecLately we don't really do specs, period. ;-)15:05
bnemecI have nothing in my notes as far as red flags from the Oslo side either.15:05
bnemec#topic Releases15:06
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:06
bnemecI think the big news here is that pbr with the utf-8 description fix got merged last week.15:06
bnemecI need to reply to that mailing list thread and have everyone who was affected verify that it fixed the problem.15:06
bnemec#action bnemec reply to unicode project description thread15:07
bnemecOtherwise expect the usual set of releases this week.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"liaisons who want to continue receiving courtesy pings need to update the ping list on the meeting agenda wiki page"15:08
bnemecThat was the only one, and hopefully it's done.15:08
johnsomSome of us are here, so success! lol15:08
hberaudlol15:08
bnemec\o/15:08
bnemecI should look for where we document that for liaisons and get it updated.15:09
bnemecAnd also retire the script in oslo.tools.15:09
bnemec#action bnemec update liaison documentation with new ping list details15:09
bnemec#action bnemec retire courtesy ping script in oslo.tools15:09
bnemec#topic Kafka functional tests15:10
*** openstack changes topic to "Kafka functional tests (Meeting topic: oslo)"15:10
bnemecWhile we have ansmith, I wanted to discuss this a bit.15:10
bnemecCurrently the kafka functional tests are blocked because the kafka archive we were downloading got removed.15:11
bnemecI have a patch up to fix the download location without changing the version, but it seems to be a bit flaky: https://review.opendev.org/#/c/664275/15:12
bnemecThere's also ansmith's patch that bumps the version, but that seems to be passing even less: https://review.opendev.org/#/c/653825/15:12
kgiustibnemec: I don't think apache will be cool with that - tried that with qpid-proton awhile ago.  much pain15:13
ansmithbnemec: +1 to changing to pull from archive, the functional tests have been flaky on and off, mostly on15:13
kgiustibnemec: I've also tried pulling from github - even worse15:13
bnemeckgiusti: Where do they want us to pull from?15:14
bnemecansmith: Okay, so this isn't a new thing.15:14
bnemecDo you want to do this in two steps (download from archive, bump version) or just go with your patch?15:14
kgiustibnemec: not that we can't pull from it - but if it happens frequently enuff we may be blocked (that's what happened to me)15:15
ansmithwhy don't we do it in two steps, take your patch for the switch to pull from archive, meanwhile i will focus on getting more instight into func test issues15:15
ansmithkgiusti: switching from dist to archive should unblock15:15
hberaud+1 for 2 steps15:15
bnemecansmith: Okay, sounds good.15:15
kgiustiansmith: https://archive.apache.org/dist/15:16
bnemecMaybe we can run it that way for a while and see if we run into rate-limiting from apache.15:16
kgiustiansmith: +115:16
bnemecOh, so maybe we should pick a different mirror?15:16
bnemecPer the "Please do not download from apache.org! If you are currently at apache.org and would like to browse, please visit a nearby mirror site instead."15:17
bnemecI'll update my patch to point at a mirror instead of apache.org directly.15:18
kgiustibnemec: ansmith: one aspect of a mirror (or any site not under our control) is that we don't control when images are removed/updated15:18
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Remove ProjectClaim object from oslo.limit  https://review.opendev.org/66570815:18
bnemec#action bnemec to point kafka functional test patch at apache mirror15:18
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Remove verification functionality  https://review.opendev.org/66570915:18
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Remove __enter__ and __exit__ methods from Enforcer  https://review.opendev.org/66571015:18
openstackgerritLance Bragstad proposed openstack/oslo.limit master: Add skeleton enforce() method to Enforcer  https://review.opendev.org/66571115:18
bnemecYay for progress on oslo.limit. Now back to your regularly scheduled meeting. :-)15:19
bnemeckgiusti: It looked like they had some pretty old releases in the archive though, and hopefully the mirrors are actually mirrors and not subsets.15:19
kgiustibnemec: + mirror - worth a try15:20
kgiustibnemec: +1 that is15:20
bnemecOkay cool.15:20
*** e0ne has quit IRC15:20
bnemec#agreed switch kafka functional tests to an apache archive mirror15:20
bnemecThat's all I had for topics this week.15:21
bnemec#topic Weekly Wayward Review15:21
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:21
bnemec#link https://review.opendev.org/64872715:22
bnemecThis is a pbr fix that can probably merge now that ci is more stable.15:22
hberaud+115:23
bnemecBut I have to admit I haven't looked at it yet because of the pbr ci instability.15:23
bnemecAnd it's a bunch of regexes so it probably isn't a quick review. ;-)15:23
bnemecBut lots of unit test cases, so ++ for that.15:24
bnemecwow, that's an old bug15:25
hberaudyeah15:26
moguimarwill put it into my list15:26
bnemecYeah, it's going to take me a minute to wrap my head around the regexes. Let's try to get this reviewed this week.15:27
bnemec#action Cores to review https://review.opendev.org/64872715:27
bnemecAnd non-cores of course. :-)15:28
bnemec#topic Open discussion15:28
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:28
bnemecAnything else this week?15:28
*** njohnston has joined #openstack-oslo15:29
hberaudI continue to work on the oslo.messaging rabbit driver heartbeat issue on my side15:29
bnemechberaud: The native thread thing?15:29
bnemecI saw kgiusti had commented on that.15:29
bnemecBut I was distracted by other stuff late last week so I haven't looked closely.15:29
hberaudYeah. I've inspect the complete stack from nova-puppet to the driver initialization15:29
kgiustikgiusti: yep - Mr "Debby Downer" at your service...15:29
hberaudI will publish my analyze and my POC soon15:30
kgiustihberaud: +115:30
bnemecSounds good, thanks15:30
hberaudand after that we will discuss about that15:30
*** iurygregory_ is now known as iurygregory15:30
bnemecFeel free to add it to the meeting agenda once it's ready.15:30
bnemecOr we can discuss it ad-hoc if we can get everyone together some other time.15:31
hberaudack15:31
hberaudgood idea15:31
bnemecOh, I need to sync up with mnaser about the oslo.service reload thing too.15:32
bnemecI'm wondering if we want to just merge the fix and rely on OSA ci to verify that it's working.15:33
bnemecSince the actual functional code changes were fairly minimal, but writing tests was going to be more of a hassle.15:33
bnemecAnd it would be nice to get that in sooner rather than later so we aren't merging a big, potentially breaking change right at the end of the cycle.15:34
bnemec#action bnemec to follow up on oslo.service reload bug15:34
hberaudwhich bug?15:35
bnemec#link https://review.opendev.org/#/c/641907/15:35
bnemechberaud: That patch^15:35
hberaudty15:36
bnemecWow, I included this in the PTG wrapup but provided no useful information about it. :-(15:37
*** tesseract has quit IRC15:38
bnemecFortunately I think the conclusion we came to was basically to address the comments on the review and then go with it.15:38
bnemecAnyway, hopefully more on that next week.15:38
bnemecAnything else for this week?15:38
moguimarnot on my end15:39
*** pcaruana has joined #openstack-oslo15:40
hberaudnot for me15:41
bnemecOkay, thanks for joining everyone!15:41
bnemec#endmeeting15:41
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:41
openstackMeeting ended Mon Jun 17 15:41:37 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:41
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-06-17-15.00.html15:41
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-06-17-15.00.txt15:41
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-06-17-15.00.log.html15:41
*** moguimar is now known as moguimar|gone15:44
*** moguimar|gone is now known as moguimar15:44
*** ianychoi_ has joined #openstack-oslo15:48
*** iurygregory has quit IRC15:49
*** ianychoi has quit IRC15:52
*** trident has quit IRC15:56
gsantomaggiospeaking about the documentation there are few errors, or let's say imprecisions15:57
gsantomaggiohttps://docs.openstack.org/nova/stein/reference/rpc.html15:57
gsantomaggio```15:57
gsantomaggioNova uses Kombu to connect to the RabbitMQ environment. Kombu is a Python library that in turn uses AMQPLib,15:57
gsantomaggio```15:57
gsantomaggioAMQPLib is a nodejs lib, the correct one is `py-amqp`15:57
gsantomaggio```15:57
gsantomaggio, a library that implements the standard AMQP 0.815:57
gsantomaggio```15:57
gsantomaggioAMQP 0.9.1 not AMQP 0.815:57
*** trident has joined #openstack-oslo15:58
gsantomaggio( https://review.opendev.org/#/c/664153/ )15:58
gsantomaggiocc @kgiusti15:58
kgiustigsantomaggio: good find15:58
openstackgerritHervé Beraud proposed openstack/oslo.messaging master: fix a typo  https://review.opendev.org/66571815:58
gsantomaggiolet me post on the bug15:59
kgiustigsantomaggio: I'm not sure that doc even needs to exist15:59
kgiustigsantomaggio: +115:59
gsantomaggio@kgiusti  is this correct? https://docs.openstack.org/nova/stein/reference/rpc.html ?15:59
kgiustigsantomaggio: do you think that information is worth while?15:59
kgiustigsantomaggio: yep that's it16:00
gsantomaggio@kgiusti  We have to give the correct info, else it is better to don't give them :)16:00
bnemeckgiusti: ansmith: Oh bugger, the apache mirrors aren't mirroring the archive: http://apache.mirrors.pair.com/kafka/16:01
bnemecThey've only got releases from the past couple of months.16:01
kgiustigsantomaggio: it probably is helpful for debugging issues, but I'm not sure it's 100% up to date16:01
kgiustiansmith: help me swap in a memory - is there not a kafka package in Our Favorite Distros?16:02
ansmithkgiusti: not that i am aware of16:03
kgiustiansmith: darn.16:04
gsantomaggio@kgiusti16:05
gsantomaggio```16:05
gsantomaggioNova uses Kombu to connect to the RabbitMQ ...16:05
gsantomaggio```16:05
gsantomaggioshould be:16:05
gsantomaggio```16:05
gsantomaggioNova uses olso.messaging ....16:05
gsantomaggio```16:05
gsantomaggioThen the link to olso messaging, else is a redundant information that can cause entropy.16:05
gsantomaggioIt is not correct ( in my opinion ) to report all this info. That are generic info and not specific to nova16:05
kgiustigsantomaggio: agreed.16:06
gsantomaggio@kgiusti  let me update the bug, and the issue.16:07
kgiustigsantomaggio: thanks16:07
bnemecThere's a good chance that's leftover from before oslo.messaging was extracted from Nova. :-)16:13
gsantomaggio@bnemec  I don't know too much nova,but they use oslo.messaging:16:15
gsantomaggiohttps://github.com/openstack/nova/blob/master/requirements.txt#L4816:15
hberaudgsantomaggio: +116:15
gsantomaggioAm I missing something ?16:15
bnemecYeah, but waaaaay back in the day there was just Nova and Swift.16:15
bnemecA lot of the code in Oslo was factored out of Nova originally.16:15
hberaudgsantomaggio: bnemec kgiusti => https://github.com/dciabrin/pyamqp-heartbeat/pull/4/files16:16
bnemecSo at one point it's likely Nova had the messaging code directly in its tree and was in fact using kombu directly.16:16
bnemecIt's not at all relevant now, but that's probably how it happened.16:16
hberaudI just finish to inspect the entire stack from puppet-nova to the oslo.messaging rabbitmq instantiate ^^^, gsantomaggio you can see who use why16:17
kgiustihberaud: nice!16:18
gsantomaggio@hberaud  cool16:18
hberaudit's a fresh paint, maybe I'm missing some points but I think we have a big picture of the stuff here16:19
hberaudI've stop tthe description when I reach the oslo layer, so I don't speak about kombu and pyamqp...16:20
gsantomaggio@hberaud  that's fine, I think that in this case we should jut point to the libraries.16:21
gsantomaggiobecause Kombu for example can use py-ampq or librabbitmq16:21
hberaudyeah16:22
gsantomaggioso, we should not write anything about that, just point the documenation to the libraries documentation.16:22
hberaudgsantomaggio: I agree16:22
gsantomaggio@hberaud  ty16:22
hberaudthe most blur part is around the nova.objects.service and how they use => https://github.com/openstack/nova/blob/013aa1915c79cfcb90c4333ce1e16b3c40f16be8/nova/service.py#L55,L6216:23
*** njohnston is now known as njohnston__16:23
hberaudbut I don't think is really useful here...16:24
hberaudmy goal is to provide a lightweight fake service to emulate nova mechanismes and to help us to debug eventlet things and related stuffs...16:26
*** lbragstad has quit IRC16:26
kgiustihberaud: a small reproducer/experimental example - very good idea.16:27
hberaudso this fake service will use paste.deploy to generate the wsgi part, initiate the rabbitdriver, and serve things over apache mod_wsgi etc...16:27
*** lbragstad has joined #openstack-oslo16:28
gsantomaggio@hberaud  +1 !!16:29
*** njohnston__ is now known as njohnston16:30
stephenfinbnemec: Think you meant to abandon this? https://review.opendev.org/#/c/662320/16:31
openstackgerritStephen Finucane proposed openstack/oslo.utils master: Add py36 testenv  https://review.opendev.org/58198716:31
bnemecstephenfin: Yes I did. The rest of the series should be rebased away from it already though.16:32
stephenfinSweet16:32
hberaudWell, see you tomorrow!16:33
* hberaud eject16:33
*** hberaud is now known as hberaud|gone16:33
openstackgerritMerged openstack/oslo.messaging master: Add transport_options parameter  https://review.opendev.org/66037316:35
gsantomaggioyeaaahhhh ^^^^16:36
kgiustigsantomaggio: nice job!16:43
openstackgerritMerged openstack/oslotest master: Sync Sphinx requirement  https://review.opendev.org/66171916:43
gsantomaggio@kgiusti  ty, I can move to the second step16:43
kgiustigsantomaggio: onward and upward!16:44
openstackgerritMerged openstack/oslo.serialization master: Sync Sphinx requirement  https://review.opendev.org/66267216:44
openstackgerritMerged openstack/oslo.upgradecheck master: Sync Sphinx requirement  https://review.opendev.org/66267516:44
*** trident has quit IRC17:27
*** trident has joined #openstack-oslo17:29
openstackgerritBen Nemec proposed openstack/oslo.config master: Deprecate ConfigFilter  https://review.opendev.org/66573517:51
openstackgerritBen Nemec proposed openstack/oslo.config master: Remove ConfigFilter  https://review.opendev.org/66573617:51
*** ralonsoh has quit IRC18:09
openstackgerritMerged openstack/pbr master: Install more dependencies for integration testing  https://review.opendev.org/66044918:10
openstackgerritMerged openstack/pbr master: Fix Windows support  https://review.opendev.org/66245018:19
*** niceplace has quit IRC18:30
*** niceplace has joined #openstack-oslo18:31
*** pcaruana has quit IRC18:44
*** dave-mccowan has quit IRC18:54
*** ansmith_ has joined #openstack-oslo19:00
*** ansmith has quit IRC19:02
*** dave-mccowan has joined #openstack-oslo19:05
*** dave-mccowan has quit IRC19:14
*** pcaruana has joined #openstack-oslo19:17
*** ansmith_ has quit IRC19:19
*** ansmith_ has joined #openstack-oslo19:20
*** starborn has quit IRC19:24
*** pcaruana has quit IRC19:27
*** trident has quit IRC19:37
*** trident has joined #openstack-oslo19:39
*** tosky has quit IRC19:41
*** zaneb has quit IRC19:52
*** zaneb has joined #openstack-oslo19:52
*** e0ne has joined #openstack-oslo20:07
*** e0ne has quit IRC20:07
*** bobh has quit IRC20:37
*** ansmith_ has quit IRC20:56
*** kgiusti has left #openstack-oslo20:58
*** boden has quit IRC21:10
*** njohnston has quit IRC21:12
*** rcernin has joined #openstack-oslo21:43
*** raildo has quit IRC21:50
*** shachar has quit IRC22:17
*** shachar has joined #openstack-oslo22:20
*** goldyfruit has quit IRC22:27
*** ansmith_ has joined #openstack-oslo22:33
*** goldyfruit has joined #openstack-oslo23:43

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