Wednesday, 2018-10-31

*** gyee has quit IRC00:36
*** ykarel has joined #openstack-rpm-packaging02:34
*** ykarel has quit IRC05:21
*** ykarel has joined #openstack-rpm-packaging05:21
*** pcaruana|elisa| has joined #openstack-rpm-packaging05:29
*** pcaruana|elisa| has quit IRC05:37
*** ykarel has quit IRC05:51
*** chkumar|off is now known as chandankumar07:13
*** ykarel has joined #openstack-rpm-packaging07:42
*** pcaruana|elisa| has joined #openstack-rpm-packaging07:45
openstackgerritDirk Mueller proposed openstack/rpm-packaging master: update oslo.config to 6.6.2  https://review.openstack.org/61307207:48
*** ykarel has quit IRC07:49
*** ykarel has joined #openstack-rpm-packaging07:50
*** gkadam has joined #openstack-rpm-packaging07:54
*** ykarel has quit IRC07:56
*** ykarel has joined #openstack-rpm-packaging08:02
*** ykarel has quit IRC08:10
*** ykarel has joined #openstack-rpm-packaging08:27
*** jpich has joined #openstack-rpm-packaging08:28
*** ykarel has quit IRC08:28
*** ykarel has joined #openstack-rpm-packaging08:29
*** ykarel_ has joined #openstack-rpm-packaging08:31
*** ykarel has quit IRC08:34
*** ykarel has joined #openstack-rpm-packaging08:34
*** ykarel_ has quit IRC08:38
*** ykarel has quit IRC08:40
*** jpena|off is now known as jpena08:52
openstackgerritMerged openstack/rpm-packaging master: Switch keystone pyldap dependency to python-ldap  https://review.openstack.org/61420909:04
*** pcaruana|elisa| has quit IRC09:13
*** pcaruana|elisa| has joined #openstack-rpm-packaging09:15
*** zul has joined #openstack-rpm-packaging09:57
*** jpena is now known as jpena|lunch12:46
*** gkadam has quit IRC12:57
*** gkadam has joined #openstack-rpm-packaging13:03
*** jpena|lunch is now known as jpena13:48
openstackgerritJohannes Grassler proposed openstack/rpm-packaging master: monasca-agent: switch to systemd supervision  https://review.openstack.org/61391814:02
dirk#startmeeting rpm_packaging14:06
openstackMeeting started Wed Oct 31 14:06:56 2018 UTC and is due to finish in 60 minutes.  The chair is dirk. Information about MeetBot at http://wiki.debian.org/MeetBot.14:06
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:06
*** openstack changes topic to " (Meeting topic: rpm_packaging)"14:06
openstackThe meeting name has been set to 'rpm_packaging'14:07
dirkping toabctl, dirk, apevec, aplanas, IgorYozhikov, jpena, jruzicka, number80, kaslcrof, ykarel14:07
dirk#topic roll call14:07
*** openstack changes topic to "roll call (Meeting topic: rpm_packaging)"14:07
* dirk struggles with etherpad14:08
jpenao/14:11
cmurphyo/14:16
dirk#chair jpena  cmurphy14:18
openstackCurrent chairs: cmurphy dirk jpena14:18
dirkdoes anyone have topics? I can't access etherpad for some reason14:19
jpenanothing special for me14:19
dirkjpena: are you coming to the berlin summit?14:19
jpenadirk: no, not this time14:19
dirkanyone else coming that is worth talking to ?14:22
dirkI mean, sorry, in the context of contributions for the openstack rpm packaging14:22
jpenafrom the Red Hat side I'm not aware of many people going to Berlin14:23
dirkk, thanks14:30
dirk#topic reviews14:30
*** openstack changes topic to "reviews (Meeting topic: rpm_packaging)"14:30
dirkdo we have reviews to talk about?14:30
dirkhttps://review.openstack.org/#/c/613652/ looks like an easy merge14:30
dirkah, I see cmurphy and jpena would need to look at https://review.openstack.org/#/c/610011/ again - I would like to close out this topic14:31
dirkany strong opinions on how to get this merged?14:31
dirkor if it should be merged?14:31
jpenaI don't have a strong opinion on the limit increase. I saw cmurphy didn't like the increase in TasksMax14:32
cmurphyI'm still unclear on why raising both nproc and tasksmax is okay14:32
jpenadirk: how did you come up with those numbers?14:33
dirkjpena: I copied them from ceph-osd.service14:33
cmurphywhen i happened to check this on a customer's production site the number of tasks was at 22 for cinder-volume14:33
dirkthe issue we had was with a user that was running cinder-volume with rbd at scale.14:33
dirkand librbd just asserts() (which crashes all of cinder-volume) when it can't start a new thread14:34
dirkso TasksMax=infinity basicall removes the systemd cgroup pid controller14:34
dirkand the rest is just bumping the limits accross the sanity imho14:34
dirkcmurphy: so you considered tasksmax=infinity the unsafe part? or the NPROC?14:35
cmurphydirk: the combination of raising both is what seems unsafe to me14:38
cmurphybut i'm not an expert14:38
cmurphybut i noted that ubuntu doesn't bother with these limits http://paste.openstack.org/show/732149/14:39
dirkso LimitNPROC=1234\nTasksMax=500 means "this service and its children are allowed to use 500 pids (threads, processes), all of the same uid are allowed 1234"14:39
dirkLimitNPROC=1234\nTasksMax=inifinity means "this service is allowed to use up to the user rlimit (1234)"14:40
jpenaoh, so nproc still affects14:40
dirkyes14:40
dirknproc is number of processes for this *user* (not this process tree)14:40
dirkso e.g. cinder-api and cinder-volume share limits14:41
dirk(as they run as the same user)14:41
dirktasksMax is a systemd feature to limit cinder-volume so that its not "eating" the quota of cinder-api14:41
dirkbut exactly this feature caused the issue for the customer (he was having 8191 threads in ceph and then it asserted)14:41
dirkwhen it tried to spawn 819214:41
dirkgiven the tradeoff "all of cinder volume is down for every user" and "well, ceph goes slightly beyond sanity but if the hardware is powerful enough we allow it" this change is changing it towards "we keep cinder volume up at all cost"14:42
cmurphyis librbd supposed to be using that many threads?14:43
dirkcmurphy: I have to admit I ignore the ubuntu reason because it just means they eithe rhave some other way of overriding the defaults (e.g. via a limits drop in from their orchestration) or they never had the issue so far (for example because they use the older systemd or have a DefaultTasksMax=infinity configured)14:43
dirkso if they don't set it, the global default catches, and if that one is infinity then they never have cgroup pid limits14:43
dirkcmurphy: well.. thats a whole other debate.. :-)14:44
dirkIMHO 8192 is plenty but then again we saw that this limit is being reached14:44
cmurphybut is it reached because it's a runaway, or because it's functioning normally? if it's a runaway and malfunctioning then raising the limits for it is just going to delay the inevitable crash14:45
dirkso, to draw a conclusion here, a simple TasksMax=infinity change without raising LimitNOFILE and limitNPROC would be acceptable?14:45
dirkcmurphy: there wasn't a leak, its just a load spike, it is normal a few seconds later14:46
cmurphyokay14:46
dirke.g. the user is spawning a heat stack with a few dozenn volumes inside14:46
dirkand while I agree librbd could be less agressively spawning threads, still any decent hardware should be capable of handling 8192 threads, so its not that the machine would be crawling death with that many threads14:47
dirk(although it will eat like 64GB of RAM just for stack space)14:48
cmurphyokay, tasksmax=infinity without the other limit changes sounds fine to me14:48
dirk#topic open floor14:53
*** openstack changes topic to "open floor (Meeting topic: rpm_packaging)"14:53
openstackgerritDirk Mueller proposed openstack/rpm-packaging master: cinder-volume: Raise limits and disable cgroup limtis  https://review.openstack.org/61001114:54
dirkcmurphy: done ^^ :)14:54
dirkanything else?14:54
dirkT-2 min before ending ;)14:54
dirk(happy halloween)14:54
* cmurphy nothing14:54
jpenanope14:56
dirkthanks :)14:56
dirk#endmeeting14:57
*** openstack changes topic to "https://etherpad.openstack.org/p/openstack-rpm-packaging - Regular IRC Meeting Wednesdays 1:00 PM UTC in openstack-rpm-packaging"14:57
openstackMeeting ended Wed Oct 31 14:57:00 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/rpm_packaging/2018/rpm_packaging.2018-10-31-14.06.html14:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/rpm_packaging/2018/rpm_packaging.2018-10-31-14.06.txt14:57
openstackLog:            http://eavesdrop.openstack.org/meetings/rpm_packaging/2018/rpm_packaging.2018-10-31-14.06.log.html14:57
*** gyee has joined #openstack-rpm-packaging15:01
*** gkadam has quit IRC16:20
openstackgerritDirk Mueller proposed openstack/rpm-packaging master: WIP Restore monasca-ceilometer package  https://review.openstack.org/61198216:22
*** pcaruana|elisa| has quit IRC16:37
*** gyee has quit IRC16:50
openstackgerritJohannes Grassler proposed openstack/rpm-packaging master: monasca-agent: switch to systemd supervision  https://review.openstack.org/61391816:52
*** chandankumar is now known as chkumar|off16:53
*** jpich has quit IRC16:59
*** gyee has joined #openstack-rpm-packaging17:44
*** jpena is now known as jpena|off18:04
*** nicolasbock has joined #openstack-rpm-packaging19:19
*** gyee has quit IRC22:19

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