| *** ansmith has joined #openstack-oslo | 00:15 | |
| *** ansmith has quit IRC | 00:33 | |
| *** threestrands has joined #openstack-oslo | 01:10 | |
| *** dave-mccowan has joined #openstack-oslo | 01:29 | |
| *** dave-mccowan has quit IRC | 01:35 | |
| *** threestrands has quit IRC | 01:35 | |
| *** dave-mccowan has joined #openstack-oslo | 02:24 | |
| *** dave-mccowan has quit IRC | 02:53 | |
| openstackgerrit | Merged openstack/oslo.messaging stable/pike: Add release note for amqp library TLS/SSL error https://review.openstack.org/638735 | 03:42 |
|---|---|---|
| openstackgerrit | Merged openstack/oslo.service master: Update oslo.service to require yappi 1.0 or newer https://review.openstack.org/638489 | 05:27 |
| *** larainema has quit IRC | 06:02 | |
| *** Luzi has joined #openstack-oslo | 06:51 | |
| *** aojea has joined #openstack-oslo | 07:20 | |
| *** takamatsu_ has quit IRC | 08:19 | |
| *** takamatsu_ has joined #openstack-oslo | 08:21 | |
| *** takamatsu_ has quit IRC | 08:32 | |
| *** takamatsu_ has joined #openstack-oslo | 08:38 | |
| *** hberaud|gone has quit IRC | 08:44 | |
| *** e0ne has joined #openstack-oslo | 08:51 | |
| *** tosky has joined #openstack-oslo | 08:55 | |
| *** giblet is now known as gibi | 08:55 | |
| *** rcernin has quit IRC | 09:00 | |
| *** hberaud has joined #openstack-oslo | 09:00 | |
| *** hberaud has quit IRC | 09:06 | |
| *** hberaud has joined #openstack-oslo | 09:06 | |
| *** shardy has joined #openstack-oslo | 09:08 | |
| *** shardy has quit IRC | 09:23 | |
| *** shardy has joined #openstack-oslo | 09:23 | |
| *** a-pugachev has joined #openstack-oslo | 09:40 | |
| *** sambetts_ has quit IRC | 10:28 | |
| *** sambetts_ has joined #openstack-oslo | 10:30 | |
| *** jaosorior has joined #openstack-oslo | 10:37 | |
| *** cdent has joined #openstack-oslo | 10:43 | |
| *** trident has quit IRC | 10:47 | |
| *** trident has joined #openstack-oslo | 10:50 | |
| openstackgerrit | Colleen Murphy proposed openstack/oslo.policy master: Add py36 and py37 tox envs https://review.openstack.org/639060 | 11:11 |
| openstackgerrit | Colleen Murphy proposed openstack/oslo.limit master: Add py36 and py37 tox envs https://review.openstack.org/639063 | 11:14 |
| *** snapiri has quit IRC | 11:23 | |
| *** ansmith has joined #openstack-oslo | 11:34 | |
| *** finucannot is now known as stephenfin | 11:40 | |
| *** ansmith has quit IRC | 11:46 | |
| *** hberaud is now known as hberaud|lunch | 12:01 | |
| *** raildo has joined #openstack-oslo | 12:07 | |
| *** moguimar has joined #openstack-oslo | 12:13 | |
| *** cdent has quit IRC | 12:17 | |
| *** njohnston has joined #openstack-oslo | 12:31 | |
| *** cdent has joined #openstack-oslo | 12:34 | |
| *** e0ne has quit IRC | 12:47 | |
| *** hberaud|lunch is now known as hberaud | 13:07 | |
| *** ansmith has joined #openstack-oslo | 13:15 | |
| *** e0ne has joined #openstack-oslo | 13:35 | |
| *** snapiri has joined #openstack-oslo | 13:37 | |
| *** elbragstad has joined #openstack-oslo | 14:01 | |
| *** mmethot has joined #openstack-oslo | 14:03 | |
| *** dave-mccowan has joined #openstack-oslo | 14:07 | |
| moguimar | beekneemech: around? | 14:12 |
| moguimar | aren't the items in today's agenda from last week? | 14:12 |
| *** a-pugachev has quit IRC | 14:33 | |
| *** kgiusti has joined #openstack-oslo | 14:40 | |
| *** cdent has quit IRC | 14:53 | |
| sean-k-mooney | o/ | 15:01 |
| sean-k-mooney | dhellmann: can i get your input on a posible future oslo.preivsep change it want to do in train | 15:02 |
| sean-k-mooney | in os-vif we do not allow eventlet's to be a runtime depency. | 15:02 |
| sean-k-mooney | but in our test code we have to cat eventlet.Timeout exceptions | 15:03 |
| sean-k-mooney | https://github.com/openstack/os-vif/blob/master/os_vif/tests/functional/base.py#L44-L51 | 15:03 |
| sean-k-mooney | from privsep | 15:03 |
| *** e0ne has quit IRC | 15:03 | |
| sean-k-mooney | do you think we could intoduce a privsep timeout instead in a future privsep version? | 15:03 |
| openstackgerrit | Andy Smith proposed openstack/oslo.messaging master: Update messaging intermediaries for amqp1 tests https://review.openstack.org/625077 | 15:04 |
| sean-k-mooney | i belive the hope is we can swap to asio in U when we drop py27 support so having a privsep.Timeout excpetion class woudl help smooth that transition also | 15:04 |
| dhellmann | sean-k-mooney: that makes sense to me, although I'm not sure how we'd implement it safely without subclassing it from the eventlet class. maybe that's good enough? | 15:05 |
| *** e0ne has joined #openstack-oslo | 15:05 | |
| sean-k-mooney | effectivly i think this is a leaky abstration. am i could play with subclassing and see | 15:05 |
| dhellmann | in order to maintain backwards compatibility someone catching the existing exception would also have to catch the new one | 15:05 |
| sean-k-mooney | well if we did it in two cycles we could subclass in train and try and get people to move over to catching the privsep class | 15:07 |
| sean-k-mooney | at that point we can decouple the privsep class if and when we choose | 15:07 |
| sean-k-mooney | it would solve the os-vif usecase in any case. | 15:08 |
| sean-k-mooney | dhellmann: one final question then. should i file this as a bug or blueprint/spec? | 15:08 |
| dhellmann | sure, but we need that 2 step approach | 15:08 |
| dhellmann | the second step is a lot bigger than the first, so... | 15:08 |
| dhellmann | tracking is up to beekneemech | 15:09 |
| dhellmann | we're usually pretty lightweight | 15:09 |
| dhellmann | I'd start a mailing list thread, at least, though | 15:09 |
| sean-k-mooney | well if we do step 1 i can remove eventlet form the test depencies | 15:09 |
| sean-k-mooney | so that will make me happy | 15:09 |
| dhellmann | privsep will still pull it in, right? | 15:10 |
| sean-k-mooney | sure i can post to the ml | 15:10 |
| sean-k-mooney | yep | 15:10 |
| sean-k-mooney | os-vif is also invoked form nova and soon neutron from a deamon that uses eventlets | 15:10 |
| sean-k-mooney | i just dont want an explict depency in the os-vif code | 15:10 |
| sean-k-mooney | if i could i would remove eventlest form the nova compute agent and neutron l2 agents but im not going to open that can of worms untill at least U | 15:12 |
| *** zaneb has joined #openstack-oslo | 15:12 | |
| dhellmann | maybe that can be part of the "let's just have one agent" project | 15:13 |
| *** Luzi has quit IRC | 15:13 | |
| sean-k-mooney | maybe. if that does every become a thing i think its an opertunity to remove some depencies like eventlets that have standard lib alternitives in python 3 | 15:14 |
| sean-k-mooney | anyway thanks for sanity checking this idea/request :) | 15:15 |
| dhellmann | any time! | 15:16 |
| openstackgerrit | Ken Giusti proposed openstack/oslo.messaging stable/queens: Mark telemetry tests nv and remove from gate https://review.openstack.org/639126 | 15:16 |
| *** beekneemech is now known as bnemec | 15:21 | |
| bnemec | #startmeeting oslo | 15:22 |
| openstack | Meeting started Mon Feb 25 15:22:45 2019 UTC and is due to finish in 60 minutes. The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:22 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:22 |
| *** openstack changes topic to " (Meeting topic: oslo)" | 15:22 | |
| openstack | The meeting name has been set to 'oslo' | 15:22 |
| bnemec | courtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims | 15:23 |
| bnemec | courtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo | 15:23 |
| bnemec | courtesy ping for hberaud, jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_ | 15:23 |
| bnemec | courtesy ping for moguimar, njohnston, raildo, redrobot, sileht, sreshetnyak, stephenfin | 15:23 |
| bnemec | courtesy ping for stevemar, therve, thinrichs, toabctl, zhiyan, zxy, zzzeek | 15:23 |
| moguimar | o/ | 15:23 |
| bnemec | #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting | 15:23 |
| hberaud | o/ | 15:23 |
| kgiusti | o/ | 15:23 |
| bnemec | Apologies for being late. I forgot that my appointment this morning overlapped this meeting. | 15:23 |
| ansmith | o/ | 15:23 |
| jungleboyj | o/ | 15:23 |
| dhellmann | o/ | 15:23 |
| jungleboyj | bnemec: Were you able to get to your appointment? | 15:23 |
| bnemec | jungleboyj: Days like today are why I have a 4x4 truck. :-) | 15:24 |
| bnemec | But yeah, it's post-snowpocalyptic out there. | 15:24 |
| stephenfin | o/ | 15:24 |
| bnemec | Abandoned cars everywhere and stuff. | 15:24 |
| jungleboyj | :-) I have just decided not to tempt fate and am hiding at home. | 15:24 |
| jungleboyj | bnemec: I heard that. | 15:24 |
| bnemec | Yeah, it figures that the one day I have to go out is today. :-) | 15:25 |
| bnemec | #topic Red flags for/from liaisons | 15:25 |
| *** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:25 | |
| bnemec | Other than red flag wind warnings. :-) | 15:25 |
| jungleboyj | Nothing from Cinder. | 15:26 |
| bnemec | I guess from our side, there's the amqp 2.4.1 bump. | 15:26 |
| bnemec | If you're running oslo.messaging with ssl to rabbit, you very much want amqp 2.4.1. | 15:27 |
| bnemec | That goes back to pike, but we can't backport the requirements change. | 15:27 |
| bnemec | We'll be release noting it too, but I'm trying to publicize the change as much as possible. | 15:27 |
| bnemec | Also, oslo.cache is currently broken, but I think it's unit tests only. | 15:28 |
| bnemec | I'll be taking a look at hberaud's change to fix that today. | 15:28 |
| hberaud | yeah due to https://github.com/sqlalchemy/dogpile.cache/commit/7e8d9b0a2eb552adb6734c8570c63eb6ea2e3a7a | 15:28 |
| bnemec | #action bnemec to look at hberaud's oslo.cache fix | 15:28 |
| bnemec | Anyone else is welcome to also. | 15:28 |
| hberaud | my fix is under WIP yet | 15:28 |
| hberaud | but I know why the problem occure | 15:29 |
| bnemec | Okay, so more to do there, and keep that in mind if you're trying to get a patch in to oslo.cache. | 15:29 |
| bnemec | Although we're past feature freeze, so that's less likely. | 15:30 |
| bnemec | I know we had the python 3 bug in that library too, which we'll need to discuss once we unblock the gate. | 15:30 |
| bnemec | Okay, lots of red flags this week, actually. :-/ | 15:31 |
| bnemec | #topic Releases | 15:32 |
| *** openstack changes topic to "Releases (Meeting topic: oslo)" | 15:32 | |
| bnemec | Will do the normal releases today. | 15:32 |
| bnemec | Ideally this will be the last set with any feature releases this cycle. | 15:32 |
| bnemec | Although we may have to make an exception for the oslo.cache bug, but we'll cross that bridge when we come to it. | 15:32 |
| bnemec | moguimar: I believe the new castellan should be released now. | 15:33 |
| moguimar | ok, thanks bnemec | 15:33 |
| bnemec | I know you the barbican team was asking about that last week. | 15:33 |
| bnemec | s/you// | 15:33 |
| *** zaneb has quit IRC | 15:33 | |
| bnemec | #topic Action items from last meeting | 15:34 |
| *** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)" | 15:34 | |
| bnemec | Speaking of which... | 15:34 |
| bnemec | "bnemec to release castellan" | 15:34 |
| bnemec | Done | 15:34 |
| bnemec | "bnemec send email to openstack-discuss about Oslo feature freeze" | 15:34 |
| bnemec | Done | 15:34 |
| bnemec | "Oslo team perform technical vision evaluation" | 15:34 |
| bnemec | Not done. :-( | 15:34 |
| bnemec | I'll leave it on my todo list though. | 15:35 |
| bnemec | "kgiusti to investigate connection explosion related to get_transport_url" | 15:35 |
| *** elbragstad is now known as lbragstad | 15:35 | |
| kgiusti | bnemec: it was recommended to adopt the singleton pattern used by other projects | 15:35 |
| bnemec | kgiusti: Okay, did we have documentation of that? | 15:36 |
| kgiusti | bnemec: nope. | 15:36 |
| kgiusti | bnemec: #action kgiusti document singleton pattern | 15:36 |
| kgiusti | #action kgiusti document singleton pattern | 15:37 |
| kgiusti | ? | 15:37 |
| bnemec | That should do it. | 15:37 |
| bnemec | Not sure whether the bot will have picked up the one addressed to me. | 15:37 |
| bnemec | kgiusti: Thanks | 15:37 |
| kgiusti | bnemec: we should consider integrating that pattern into the library itself | 15:37 |
| kgiusti | bnemec: for train | 15:37 |
| bnemec | Yep, that would be a good followup. | 15:38 |
| bnemec | That was it for action items from last week. | 15:38 |
| bnemec | #topic Oslo Feature Freeze | 15:39 |
| *** openstack changes topic to "Oslo Feature Freeze (Meeting topic: oslo)" | 15:39 | |
| bnemec | We're now officially in feature freeze for Oslo. | 15:39 |
| bnemec | As I mentioned, hopefully the releases I submit today will be the last feature ones for the cycle. | 15:39 |
| bnemec | But we can discuss FFE's as needed. | 15:40 |
| *** agopi has quit IRC | 15:40 | |
| bnemec | So in general, cores please don't merge anything feature-y without discussing it with me first. | 15:40 |
| bnemec | If we merge a feature without an FFE it will just block releasing that library. | 15:41 |
| bnemec | I think we have an experienced enough core team that we shouldn't need to procedural -2 everything. | 15:42 |
| bnemec | At least that's my hope because I'd rather not spend the time to do that. :-) | 15:42 |
| jungleboyj | You get a -2 and you get a -2! | 15:42 |
| bnemec | Everyone gets a -2! | 15:42 |
| jungleboyj | Yay! | 15:42 |
| bnemec | The other topics on the agenda were addressed last week, so I think that's it for those. | 15:43 |
| bnemec | #topic Weekly Wayward Review | 15:43 |
| *** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)" | 15:43 | |
| moguimar | I think this is the first feature freeze I'm seeing from a closer distance | 15:43 |
| moguimar | so we should hold +2 and +w? | 15:44 |
| bnemec | moguimar: +2 is okay, but don't +w anything that would trigger a feature release. | 15:45 |
| * bnemec grabs a link to the release criteria... | 15:45 | |
| moguimar | ok | 15:45 |
| bnemec | #link https://releases.openstack.org/reference/using.html#using-new-release-command | 15:46 |
| *** a-pugachev has joined #openstack-oslo | 15:46 | |
| bnemec | In general we follow semver, but that provides a short description of each release type that I find useful. | 15:46 |
| bnemec | For wayward reviews, we obviously have a more limited set because of feature freeze. | 15:46 |
| bnemec | But I think we can probably do this one: | 15:46 |
| bnemec | #link https://releases.openstack.org/reference/using.html#using-new-release-command | 15:47 |
| bnemec | Bleh. | 15:47 |
| bnemec | #link https://review.openstack.org/#/c/625372 | 15:47 |
| bnemec | Oh, I see stephenfin asked for a reno on that. | 15:48 |
| bnemec | hberaud: Since you already kind of adopted that one, would you be able to write a reno? | 15:49 |
| hberaud | bnemec: yeah | 15:49 |
| bnemec | hberaud: Great, thanks! | 15:49 |
| moguimar | does it needs a release note? | 15:49 |
| * moguimar wrote his first release note on openstack last week =D | 15:50 | |
| moguimar | castellan folks made me do it | 15:50 |
| bnemec | I would think that either a reno or lp bug would be good for discoverability. | 15:50 |
| bnemec | I don't know that our git commit messages are SEO'd especially well. :-) | 15:51 |
| bnemec | I personally wouldn't require both. | 15:51 |
| moguimar | +1 for the lp | 15:51 |
| bnemec | stephenfin was the one who asked, so maybe he can weigh in too. | 15:52 |
| * stephenfin checks | 15:52 | |
| hberaud | I'll create the lp | 15:53 |
| stephenfin | It doesn't need a reno, but for a user facing tool like this it is mighty helpful | 15:53 |
| hberaud | stephenfin: +1 | 15:53 |
| stephenfin | Helps with discoverability of these issues (user: "why is X happening", user -> Google) | 15:54 |
| bnemec | Yeah | 15:54 |
| bnemec | The reno could be pretty brief since it can reference the lp for details. | 15:55 |
| bnemec | Okay, I marked that one WIP. That should get cleared when a new patch set is pushed so we'll know to look again. | 15:57 |
| bnemec | #topic Open discussion | 15:57 |
| *** openstack changes topic to "Open discussion (Meeting topic: oslo)" | 15:57 | |
| bnemec | Anything else for this week? | 15:57 |
| moguimar | yep | 15:58 |
| moguimar | I was checking open changes and found this one | 15:58 |
| moguimar | https://review.openstack.org/#/c/610111 | 15:58 |
| moguimar | so it basically updates .zuul.d/project.yaml and tox.ini | 15:58 |
| moguimar | the current state of project.yaml tests py35 py36 and py37 | 15:59 |
| moguimar | do we have any best practices for that cross projects? | 15:59 |
| johnsom | Sorry I missed the liaison section, but wanted to follow up on the oslo.service /yappi issue we had. | 15:59 |
| moguimar | like, when are we suposed to add py38 and frop 35? | 16:00 |
| bnemec | moguimar: We may need to discuss this on the list again. I looked when I approved all of the py37 changes last week, and the previous conclusion was that we would drop py35 and add py37 this cycle. | 16:00 |
| bnemec | But I'm not clear that we ever actually dropped py35. | 16:00 |
| moguimar | and is it better to keep the tox.ini one in the lower one (3.5) until it gets deprecated to make it fail fast during dev? | 16:00 |
| johnsom | I think it is all resolved now, but that oslo.service release broke all of our live test gates. Luckily I was able to reach the yappi developer and he was willing to cut a new release of yappi with the fix. | 16:00 |
| bnemec | And I'm idly tempted to say we should wait to drop things until next cycle. | 16:01 |
| moguimar | I like having the last 3 versions as it is right now | 16:01 |
| bnemec | I guess dropping py35 is pretty safe since it isn't going to break the gate, but I think we should clarify our support stance. | 16:01 |
| bnemec | johnsom: Yeah, that's weird. I would have expected a problem like that to get caught in the gate since it happened on installation of the lib. | 16:02 |
| moguimar | ok, I'll create some WIP on that and we can push it forward | 16:02 |
| johnsom | Yeah, me too. Not sure what the magic combination was that triggered it for DIB installs | 16:02 |
| gsantomaggio | I am looking how Open Stack does use RabbitMQ, | 16:04 |
| gsantomaggio | I found this HA policy: | 16:04 |
| gsantomaggio | https://git.openstack.org/cgit/openstack/openstack-ansible-rabbitmq_server/tree/defaults/main.yml#n154 | 16:04 |
| gsantomaggio | If I am not wrong, open stack does not use the autodelete queues anymore. | 16:04 |
| gsantomaggio | It works using queues with TTL, right? | 16:04 |
| gsantomaggio | So, this policy should be changed from '^(?!amq\.).*' to '^(?!reply_\.).*' | 16:04 |
| gsantomaggio | What do you think? | 16:04 |
| bnemec | moguimar: Sounds good. Let me find the mailing list thread where this was first discussed. I think we should probably reopen that. | 16:04 |
| moguimar | ok | 16:05 |
| kgiusti | gsantomaggio: quite possibly - let me take a look | 16:06 |
| bnemec | johnsom: Weirdly, it never broke our gate. I see passing test jobs that used .99: http://logs.openstack.org/88/637688/1/check/openstack-tox-py36/0b4045f/job-output.txt.gz#_2019-02-20_00_50_27_988954 | 16:06 |
| gsantomaggio | @kgiusti thanks | 16:06 |
| bnemec | Maybe dib does something different with the encoding that tripped up the pip install? | 16:07 |
| *** zaneb has joined #openstack-oslo | 16:07 | |
| johnsom | bnemec Agreed. They had a bug and patch for it already. No idea how/what caused it to fail to install. It was a xenial image, nothing special. | 16:07 |
| bnemec | Oh, that's right. Someone else actually reported it first. | 16:08 |
| bnemec | moguimar: Okay, this actually goes back to the openstack-dev days: http://lists.openstack.org/pipermail/openstack-dev/2018-October/135626.html | 16:08 |
| bnemec | #action moguimar to follow up on status of py35 in the gate | 16:09 |
| moguimar | thanks | 16:10 |
| bnemec | johnsom: Do you have any proposed actions to take to avoid this in the future? | 16:10 |
| openstackgerrit | Hervé Beraud proposed openstack-dev/pbr master: Resolve ``ValueError`` when mapping value contains a literal ``=``. https://review.openstack.org/625372 | 16:11 |
| johnsom | bnemec I would if I knew what the trigger was, but I don't. They py2 and 3 gates should have caught it. | 16:11 |
| bnemec | johnsom: Well, at least now we have a known-working version so we can add version exclusions if it happens again. | 16:12 |
| johnsom | Yeah, it's fixed and released on the yappi side. I have added the blacklist on yappi in g-r, so we should be good. | 16:12 |
| bnemec | #action kgiusti and gsantomaggio to investigate OSA rabbitmq_policies | 16:13 |
| bnemec | I think that covers all of the topics so far in open discussion. | 16:14 |
| bnemec | I'll take this opportunity to note that anyone can add items to the agenda, so if you have something to discuss please feel free. :-) | 16:14 |
| bnemec | Anything else before we end? | 16:15 |
| moguimar | not on my end | 16:15 |
| gsantomaggio | Nope :)! | 16:15 |
| bnemec | Thanks for joining everyone! | 16:17 |
| bnemec | Should be back to normal schedule next week. | 16:17 |
| bnemec | #endmeeting | 16:17 |
| *** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 16:17 | |
| openstack | Meeting ended Mon Feb 25 16:17:11 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:17 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-25-15.22.html | 16:17 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-25-15.22.txt | 16:17 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2019/oslo.2019-02-25-15.22.log.html | 16:17 |
| kgiusti | gsantomaggio: I think you are correct in that those 'reply_' queues should probably be configured with a ttl | 16:21 |
| gsantomaggio | I think that also 'reply_' queues should not be mirrored | 16:22 |
| kgiusti | gsantomaggio: I'm no rabbitmq HA expert - is there any advantage to mirroring single consumer queues? | 16:25 |
| *** e0ne has quit IRC | 16:26 | |
| gsantomaggio | Nothing, it is only a waste of resources | 16:26 |
| gsantomaggio | this is why I think we should exclude reply_ queues from the mirror | 16:26 |
| kgiusti | gsantomaggio: In general I think RPC is a best effort message QOS. Mirroring/persisting such messages seems overkill to me. | 16:27 |
| gsantomaggio | > Mirroring/persisting such messages seems overkill to me. | 16:27 |
| gsantomaggio | We are in the same path :) | 16:27 |
| gsantomaggio | this is why I am suggesting to *remove* the reply_ queues fro mirror policies | 16:28 |
| *** agopi has joined #openstack-oslo | 16:28 | |
| kgiusti | Then we are in agreement! | 16:28 |
| gsantomaggio | I will prepare a PR. | 16:29 |
| kgiusti | gsantomaggio: +1 great! | 16:30 |
| *** e0ne has joined #openstack-oslo | 16:31 | |
| *** takamatsu_ has quit IRC | 16:39 | |
| *** aojea has quit IRC | 16:39 | |
| *** takamatsu_ has joined #openstack-oslo | 16:41 | |
| openstackgerrit | Ken Giusti proposed openstack/oslo.messaging master: Explain why Listener connections cannot be pooled https://review.openstack.org/639152 | 16:41 |
| *** takamatsu_ has quit IRC | 16:45 | |
| *** takamatsu_ has joined #openstack-oslo | 16:51 | |
| gsantomaggio | > So, a connection cannot be shared between thread/greenthread and | 16:52 |
| gsantomaggio | By default the connections are thread safe, the channel is not thread safe in RabbitMQ | 16:52 |
| gsantomaggio | ( using the standard clients you can use the same connection to send and receive messages using different channels ) | 16:52 |
| gsantomaggio | I think this a limitation from the kombu, right? | 16:53 |
| kgiusti | gsantomaggio: I never understood the need to create a dedicated connection for every consumer. | 16:54 |
| gsantomaggio | I am reading https://review.openstack.org/#/c/639152/1/oslo_messaging/_drivers/common.py | 16:54 |
| kgiusti | gsantomaggio: it may be due to the threading of oslo.messaging itself. | 16:55 |
| gsantomaggio | >I never understood the need to create a dedicated connection for every consumer. | 16:55 |
| gsantomaggio | in general is not the best practice | 16:55 |
| kgiusti | gsantomaggio: agreed - it's a waste of resources. | 16:55 |
| gsantomaggio | especially because, open stack creates and destroy connections continuously, so we should create and destroy channels istead of connection. | 16:59 |
| gsantomaggio | it is a cheap operation and very fast, I think we could drastically drop the number of the connections :) | 16:59 |
| kgiusti | gsantomaggio: that would help scaling up clients significantly | 17:00 |
| gsantomaggio | and it would avoid problems as TIME_WAIT sockets, file descriptors problems etc | 17:01 |
| gsantomaggio | I have to study Kombu and py-amqp | 17:02 |
| gsantomaggio | never used before :) | 17:02 |
| kgiusti | gsantomaggio: be careful - you may become our resident rabbitmq expert :) | 17:04 |
| *** takamatsu_ has quit IRC | 17:06 | |
| *** takamatsu_ has joined #openstack-oslo | 17:07 | |
| gsantomaggio | :) | 17:11 |
| *** e0ne has quit IRC | 17:18 | |
| *** takamatsu_ has quit IRC | 17:21 | |
| *** takamatsu_ has joined #openstack-oslo | 17:22 | |
| *** takamatsu_ has quit IRC | 17:25 | |
| openstackgerrit | Merged openstack/oslo.versionedobjects master: Allow lists to be generated from any non-string iterable https://review.openstack.org/637389 | 17:29 |
| *** a-pugachev has quit IRC | 17:29 | |
| *** takamatsu_ has joined #openstack-oslo | 17:31 | |
| openstackgerrit | Hervé Beraud proposed openstack/oslo.cache master: fix dogpile.cache issue due to the new usage of decorate in the lib https://review.openstack.org/638788 | 17:37 |
| openstackgerrit | Merged openstack/oslo.messaging stable/rocky: Add release note for amqp library TLS/SSL error https://review.openstack.org/638461 | 17:40 |
| openstackgerrit | Hervé Beraud proposed openstack/oslo.cache master: fix dogpile.cache issue due to the new usage of decorate in the lib https://review.openstack.org/638788 | 17:44 |
| openstackgerrit | Hervé Beraud proposed openstack/oslo.cache master: fix dogpile.cache issue due to the new usage of decorate in the lib https://review.openstack.org/638788 | 17:46 |
| *** takamatsu_ has quit IRC | 17:46 | |
| *** takamatsu_ has joined #openstack-oslo | 17:48 | |
| *** hberaud is now known as hberaud|gone | 17:49 | |
| *** takamatsu_ has quit IRC | 18:05 | |
| *** takamatsu_ has joined #openstack-oslo | 18:10 | |
| *** njohnston has quit IRC | 18:13 | |
| *** takamatsu_ has quit IRC | 18:14 | |
| *** shardy has quit IRC | 18:18 | |
| *** takamatsu_ has joined #openstack-oslo | 18:20 | |
| *** bnemec has quit IRC | 18:40 | |
| *** e0ne has joined #openstack-oslo | 18:44 | |
| *** bnemec has joined #openstack-oslo | 18:45 | |
| *** takamatsu_ has quit IRC | 18:50 | |
| *** takamatsu_ has joined #openstack-oslo | 18:56 | |
| *** njohnston has joined #openstack-oslo | 19:00 | |
| *** ansmith has quit IRC | 19:07 | |
| openstackgerrit | Merged openstack/oslo.limit master: Add python3.7 job on Stein+ https://review.openstack.org/610641 | 19:16 |
| openstackgerrit | Hervé Beraud proposed openstack/oslo.cache master: Fix memcache issue due to the __new__ attribute reassign. https://review.openstack.org/634457 | 20:27 |
| * bnemec shakes his fist at the zmq unit tests | 20:55 | |
| *** dkehn has joined #openstack-oslo | 20:58 | |
| *** agopi has quit IRC | 20:58 | |
| *** agopi has joined #openstack-oslo | 20:59 | |
| *** agopi_ has joined #openstack-oslo | 21:07 | |
| *** raildo has quit IRC | 21:09 | |
| *** agopi has quit IRC | 21:10 | |
| *** agopi__ has joined #openstack-oslo | 21:13 | |
| *** agopi_ has quit IRC | 21:16 | |
| *** agopi__ is now known as agopi | 21:16 | |
| *** e0ne has quit IRC | 21:21 | |
| *** kgiusti has left #openstack-oslo | 21:32 | |
| *** rcernin has joined #openstack-oslo | 21:46 | |
| *** e0ne has joined #openstack-oslo | 21:54 | |
| *** e0ne has quit IRC | 21:55 | |
| *** agopi has quit IRC | 22:00 | |
| *** e0ne has joined #openstack-oslo | 22:05 | |
| *** e0ne has quit IRC | 22:08 | |
| *** agopi has joined #openstack-oslo | 22:19 | |
| openstackgerrit | Merged openstack-dev/pbr master: Resolve ``ValueError`` when mapping value contains a literal ``=``. https://review.openstack.org/625372 | 22:43 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!