Thursday, 2018-07-05

*** eck` is now known as eck`gone00:04
*** salv-orlando has joined #openstack-oslo00:20
*** salv-orlando has quit IRC00:25
*** larainema has quit IRC01:04
*** edmondsw has joined #openstack-oslo01:13
*** edmondsw has quit IRC01:18
openstackgerritVu Cong Tuan proposed openstack/oslo.middleware master: Removing oslo.serialization in test-requirements.txt  https://review.openstack.org/58024001:19
*** salv-orlando has joined #openstack-oslo01:21
*** larainema has joined #openstack-oslo01:25
*** salv-orlando has quit IRC01:26
*** bobh has joined #openstack-oslo01:26
openstackgerritVu Cong Tuan proposed openstack/oslo.messaging master: Moving stestr to correct package order in test-requirements.txt  https://review.openstack.org/58024201:32
*** salv-orlando has joined #openstack-oslo02:22
*** salv-orlando has quit IRC02:26
*** bobh has quit IRC02:27
openstackgerritVu Cong Tuan proposed openstack/stevedore master: Switch to stestr  https://review.openstack.org/58025002:31
*** edmondsw has joined #openstack-oslo03:02
*** edmondsw has quit IRC03:06
*** salv-orlando has joined #openstack-oslo03:23
*** salv-orlando has quit IRC03:27
*** lpetrut has joined #openstack-oslo03:44
*** dkehn has quit IRC04:02
*** dkehn has joined #openstack-oslo04:19
*** salv-orlando has joined #openstack-oslo04:23
*** lpetrut has quit IRC04:24
*** salv-orlando has quit IRC04:28
*** njohnston has quit IRC04:28
*** njohnston has joined #openstack-oslo04:29
*** lpetrut has joined #openstack-oslo04:33
*** zzzeek has quit IRC04:40
*** zzzeek has joined #openstack-oslo04:43
*** edmondsw has joined #openstack-oslo04:51
*** lpetrut has quit IRC04:53
*** edmondsw has quit IRC04:55
*** zzzeek has quit IRC05:10
*** zzzeek has joined #openstack-oslo05:11
*** e0ne has joined #openstack-oslo05:18
*** e0ne has quit IRC05:21
*** salv-orlando has joined #openstack-oslo05:24
*** jbadiapa has quit IRC05:24
*** lpetrut has joined #openstack-oslo05:27
*** salv-orlando has quit IRC05:28
*** lpetrut has quit IRC05:33
*** ianychoi has joined #openstack-oslo05:34
*** nicolasbock has joined #openstack-oslo05:49
*** Guest38444 has quit IRC05:57
*** salv-orlando has joined #openstack-oslo06:11
openstackgerritVu Cong Tuan proposed openstack/stevedore master: Switch to stestr  https://review.openstack.org/58025006:13
openstackgerritSteve Kowalik proposed openstack/oslo.messaging master: Remove transport aliases support  https://review.openstack.org/58029406:16
*** lpetrut has joined #openstack-oslo06:20
*** salv-orlando has quit IRC06:24
*** salv-orlando has joined #openstack-oslo06:24
*** salv-orlando has quit IRC06:25
*** jbadiapa has joined #openstack-oslo06:43
*** lpetrut has quit IRC07:12
*** yamamoto has quit IRC07:18
*** lpetrut has joined #openstack-oslo07:21
*** peereb has joined #openstack-oslo07:22
*** salv-orlando has joined #openstack-oslo07:26
*** yamamoto has joined #openstack-oslo07:26
*** salv-orlando has quit IRC07:31
*** Guest38444 has joined #openstack-oslo07:38
*** yamamoto has quit IRC07:42
*** e0ne has joined #openstack-oslo07:46
*** rcernin has quit IRC07:54
*** mago_ has joined #openstack-oslo07:58
*** yamamoto has joined #openstack-oslo08:10
*** salv-orlando has joined #openstack-oslo08:27
*** edmondsw has joined #openstack-oslo08:28
*** salv-orlando has quit IRC08:31
*** edmondsw has quit IRC08:32
*** pbourke has quit IRC09:13
*** pbourke has joined #openstack-oslo09:15
*** sambetts|afk is now known as sambetts09:19
*** e0ne has quit IRC09:21
*** e0ne has joined #openstack-oslo09:23
*** salv-orlando has joined #openstack-oslo09:28
*** salv-orlando has quit IRC09:34
*** jbadiapa has quit IRC10:15
openstackgerritMerged openstack/oslo.messaging master: Moving stestr to correct package order in test-requirements.txt  https://review.openstack.org/58024210:20
*** e0ne has quit IRC10:28
*** salv-orlando has joined #openstack-oslo10:30
*** salv-orlando has quit IRC10:34
*** lpetrut_ has joined #openstack-oslo10:39
*** lpetrut has quit IRC10:42
*** gcb has joined #openstack-oslo10:58
openstackgerritVu Cong Tuan proposed openstack/stevedore master: Switch to stestr  https://review.openstack.org/58025011:02
openstackgerritJames Page proposed openstack/oslo.messaging master: py37: drop use of 'async' as parameter name  https://review.openstack.org/58034311:05
openstackgerritJames Page proposed openstack/oslo.messaging master: py37: drop use of 'async' as parameter name  https://review.openstack.org/58034311:10
*** gcb has quit IRC11:13
*** edmondsw has joined #openstack-oslo11:29
*** salv-orlando has joined #openstack-oslo11:31
*** salv-orlando has quit IRC11:36
openstackgerritJames Page proposed openstack/oslo.messaging master: py37: deal with Exception repr changes  https://review.openstack.org/58034911:39
*** lifeless has quit IRC11:45
*** e0ne has joined #openstack-oslo12:07
*** edmondsw has quit IRC12:07
*** edmondsw has joined #openstack-oslo12:13
*** edmondsw_ has joined #openstack-oslo12:16
*** edmondsw has quit IRC12:19
*** jbadiapa has joined #openstack-oslo12:20
*** salv-orlando has joined #openstack-oslo12:31
*** raildo has joined #openstack-oslo12:32
*** salv-orlando has quit IRC12:36
*** e0ne has quit IRC12:48
*** salv-orlando has joined #openstack-oslo12:53
*** salv-orl_ has joined #openstack-oslo12:54
*** salv-orlando has quit IRC12:56
*** salv-orlando has joined #openstack-oslo12:57
*** eck`gone is now known as eck`12:57
*** salv-orl_ has quit IRC12:59
*** kgiusti has joined #openstack-oslo13:03
*** cdent has joined #openstack-oslo13:15
*** salv-orlando has quit IRC13:15
*** salv-orlando has joined #openstack-oslo13:16
*** ansmith has joined #openstack-oslo13:32
*** salv-orlando has quit IRC13:35
*** rmascena has joined #openstack-oslo13:57
*** e0ne has joined #openstack-oslo13:59
*** raildo has quit IRC13:59
openstackgerritRaildo Mascena proposed openstack/oslo.config master: New cache layer for external sources  https://review.openstack.org/57814414:11
*** rmascena is now known as raildo14:11
*** rmascena has joined #openstack-oslo14:13
*** raildo has quit IRC14:16
openstackgerritMerged openstack/stevedore master: Switch to stestr  https://review.openstack.org/58025014:24
*** rmascena is now known as raildo14:24
*** lpetrut_ has quit IRC14:24
openstackgerritMichal Arbet proposed openstack/taskflow master: Fix code to support networkx > 1.0  https://review.openstack.org/57783314:33
*** salv-orlando has joined #openstack-oslo14:36
cdentbnemec: do you know if there's been any further work or discussion on https://bugs.launchpad.net/cinder/+bug/1432387 ? It's come up in downstream discussions and I'm trying to track it, and maybe even fix it14:40
openstackLaunchpad bug 1432387 in oslo.concurrency "abandoned lock files" [Wishlist,Confirmed]14:40
*** salv-orlando has quit IRC14:40
smcginnisI haven't heard anything on that in a long time.14:41
openstackgerritRaildo Mascena proposed openstack/oslo.config master: add detail to driver options in config generator  https://review.openstack.org/57846114:42
smcginniscdent: If it's a downstream concern, maybe switch to using something other than the file locks? https://bugs.launchpad.net/cinder/+bug/1432387/comments/1814:43
openstackLaunchpad bug 1432387 in oslo.concurrency "abandoned lock files" [Wishlist,Confirmed]14:43
bnemeccdent: Wow, I'm surprised I didn't nack that when I triaged it.14:43
bnemecI'm pretty strongly opposed to further attempts to "fix" this "problem".14:43
bnemecWhich is cosmetic at worst.14:44
openstackgerritRaildo Mascena proposed openstack/oslo.config master: User guide documentation for backend drivers for oslo.config  https://review.openstack.org/57694714:44
bnemecAnd past attempts have had some fairly ugly side effects.14:44
openstackgerritRaildo Mascena proposed openstack/oslo.config master: New cache layer for external sources  https://review.openstack.org/57814414:44
cdentsmcginnis: yeah, that's kind of what I was thinking too, but apparently there's some past limitation, not sure what14:44
cdentI'm digging in both directions14:45
smcginnisOK, good.14:45
bnemeccdent: Is it actually causing a problem?14:45
cdentapparently in large enough and old enough setups it can results in either more files than a directory can handle or too many inodes14:46
bnemeccdent: Are they cleaning up the lock files on reboot?  Or are these setups being rebooted rarely enough that they still pile up?14:46
cdentstill digging in the direction, so don't have much data yet14:47
cdent(i'm on the long end of a telephone game)14:47
bnemecIIRC the number of files in a directory that is supported by all major Linux filesystems is something absurdly large.14:47
cdentit _may_ be that it is just making people feel aesthetically uncomfy14:48
*** ansmith has quit IRC14:48
cdentlike I said: telephone game14:48
bnemecThat's always been the case in the past. :-)14:48
bnemecUnfortunately it's impossible to safely remove the lock files while a service is still running.14:48
cdentbut knowing that you both think it is not going to change and that using a different style of locking is good data, so thank you14:49
smcginnisJust whack them and cross your fingers. :)14:49
smcginnisBut yeah, now that etcd is considered a base service, I think tooz can just be configured to use that and the file problem goes away.14:49
bnemeccdent: Okay.  If it does turn out that there is a legitimate functional issue feel free to follow up though.14:49
bnemecIt _is_ possible to clean up lock files manually if the service knows for certain that it's done with one.14:50
cdentsmcginnis: I first read that as "whack them across the fingers" and wondered if you were secretly a catholic nun running a school14:50
bnemechttps://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/lockutils.py#L15114:50
bnemecFor cinder it might be okay to call that after a volume is deleted and you're sure the id isn't going to be locked anymore.14:51
smcginnisOoh, can we get some rulers at the next Summit? That could be interesting.14:51
smcginnisI'm not sure if we have visibility to that with how things are abstracted through tooz.14:52
smcginnisBut if it was a real issue I'm sure we could find a way around that.14:52
cdenti've tried to gin up some proper feedback14:56
* smcginnis is failing to come up with a good tonic response pun to make cdent groan14:58
cdentI'll take it on faith14:58
smcginnis:)14:58
* cdent can see some gin and tonic in his future this evening14:58
smcginnisWhoo hoo14:58
cdentthis stuff. http://www.tarquinsgin.com/14:59
*** salv-orlando has joined #openstack-oslo15:01
*** salv-orlando has quit IRC15:01
*** namnh has joined #openstack-oslo15:01
cdentbnemec, smcginnis: looks like most of the lock files are coming from neutron (this is vmware nsx-v install), not cinder, and for the time being file locks are where things are. 900K files in the same directory. short term workaround is to shut down neutron, kill some files, start neutron back up15:04
smcginnisWow, 900k!15:06
bnemecThat is quite a lot.  Is it causing failures?15:07
cdentyes, sometimes that can't create files15:11
bnemecHmm, okay.15:13
bnemecWould be interesting to see why there are so many and if we can reduce that number or clean some of them up.15:13
bnemecI need to duck out and take my car in to the shop right now though.15:14
cdentthanks for the engagement15:14
cdentI'm trying to locate the offending code so I can see why so many are happening15:14
* bnemec biab15:21
*** peereb has quit IRC15:22
smcginniscdent: I can ping mlavalle if you need some help there.15:25
*** bnemec has quit IRC15:25
cdentsmcginnis: It's nsx code being bad: https://github.com/openstack/vmware-nsx/blob/master/vmware_nsx/common/locking.py#L3615:26
cdentI'm not sure why locking_coordinator_url is never set. If it were this goes away, presumably.15:26
smcginniscdent: Well, I guess at least you can probably get to the right people to fix that.15:26
cdentone would hope so15:26
cdentit looks like a configuration problem to me, but what do I know15:27
smcginnisYeah, hard to tell just looking at that snippet.15:27
smcginniscdent: I wonder if they can just look at moving to tooz to follow what is being done elsewhere.15:32
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Add docstring from LocalManifestMaker.add_defaults  https://review.openstack.org/58041215:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Switch to stestr  https://review.openstack.org/58041315:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: trivial: Remove 'tools/releasenotes_tox.sh'  https://review.openstack.org/58041415:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: trivial: Fix file permissions  https://review.openstack.org/58041515:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: builddoc: Remove support for Sphinx < 1.6  https://review.openstack.org/58041615:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: packaging: Remove support for pyN requirement files  https://review.openstack.org/58041715:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: doc: Fix formatting of packagers guide  https://review.openstack.org/58041815:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: doc: Add documentation for missing '[pbr]' options  https://review.openstack.org/58041915:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Add Sphinx extension  https://review.openstack.org/58042015:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Deprecate 'build_sphinx' integration  https://review.openstack.org/58042115:36
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Deprecate 'test' integration  https://review.openstack.org/58042215:36
*** e0ne has quit IRC15:42
*** lpetrut_ has joined #openstack-oslo15:46
*** dtruong has joined #openstack-oslo15:55
*** bnemec has joined #openstack-oslo15:56
openstackgerritwu.chunyang proposed openstack/debtcollector master: Add release note link in README  https://review.openstack.org/58043716:02
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Switch to stestr  https://review.openstack.org/58041316:03
openstackgerritStephen Finucane proposed openstack-dev/pbr master: trivial: Remove 'tools/releasenotes_tox.sh'  https://review.openstack.org/58041416:03
openstackgerritStephen Finucane proposed openstack-dev/pbr master: trivial: Fix file permissions  https://review.openstack.org/58041516:03
openstackgerritStephen Finucane proposed openstack-dev/pbr master: builddoc: Remove support for Sphinx < 1.6  https://review.openstack.org/58041616:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: packaging: Remove support for pyN requirement files  https://review.openstack.org/58041716:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: doc: Fix formatting of packagers guide  https://review.openstack.org/58041816:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: doc: Add documentation for missing '[pbr]' options  https://review.openstack.org/58041916:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Add Sphinx extension  https://review.openstack.org/58042016:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Deprecate 'build_sphinx' integration  https://review.openstack.org/58042116:04
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Deprecate 'test' integration  https://review.openstack.org/58042216:04
*** mago_ has quit IRC16:11
*** tosky has joined #openstack-oslo16:11
*** Guest38444 has quit IRC16:13
*** bobh has joined #openstack-oslo16:17
namnhbnemec: Long time did not get your comment. If you have free time, can you help me to review this patch: https://review.openstack.org/#/c/52631416:25
bnemecnamnh: Yeah, sorry.  I need to get back to that ASAP.16:25
namnhbnemec: thanks in advance16:26
*** gcb has joined #openstack-oslo16:28
*** gcb has quit IRC16:36
*** bobh has quit IRC16:41
*** bobh has joined #openstack-oslo16:43
*** bobh has quit IRC16:47
*** bnemec has quit IRC16:51
*** e0ne has joined #openstack-oslo17:03
openstackgerritMichal Arbet proposed openstack/taskflow master: Fix code to support networkx > 1.0  https://review.openstack.org/57783317:09
*** lpetrut_ has quit IRC17:10
*** bnemec has joined #openstack-oslo17:18
*** namnh has quit IRC17:22
kgiustibnemec: hey ben - I approved this https://review.openstack.org/#/c/580294/  - this contains an API change however...17:23
kgiustibnemec: and it looks like you just did a major bump...  :(17:23
bnemeckgiusti: Yeah, I saw that.17:24
bnemecIt's not the end of the world if we have to bump two major versions in a week.17:24
kgiustibnemec: maybe I should revert for now17:24
bnemecUnfortunate timing, but what can you do?17:24
kgiustibnemec: should this wait until beginning of stein instead?17:24
bnemeckgiusti: Unless you think there's significant risk to the change, I wouldn't bother.17:25
kgiustibnemec: the change itself isn't risky17:25
kgiustibnemec: folks may not be happy with api breakage so late in the dev cycle - that's my only concern.17:27
*** ansmith has joined #openstack-oslo17:28
kgiustiansmith: hey, some vacation you must be having if you're here...17:28
bnemeckgiusti: Is this blocking anything that needs to land in Rocky?  If not, maybe it would be better to wait for Stein.17:29
kgiustibnemec: not aware of anything - StevenK might17:31
kgiustibnemec: I'll drop a comment in the review.17:32
bnemeckgiusti: Sounds good.17:33
*** lpetrut_ has joined #openstack-oslo17:33
*** e0ne has quit IRC17:36
*** lpetrut_ has quit IRC17:41
*** sambetts is now known as sambetts|afk17:47
*** lpetrut_ has joined #openstack-oslo17:52
*** e0ne has joined #openstack-oslo18:01
*** e0ne_ has joined #openstack-oslo18:03
*** e0ne__ has joined #openstack-oslo18:05
*** e0ne has quit IRC18:06
*** e0ne_ has quit IRC18:08
openstackgerritMerged openstack/oslo.messaging master: Remove transport aliases support  https://review.openstack.org/58029418:26
*** lpetrut_ has quit IRC18:27
*** lpetrut_ has joined #openstack-oslo18:27
*** purplerbot has quit IRC18:48
*** purplerbot has joined #openstack-oslo18:48
*** cdent has quit IRC18:53
*** lpetrut_ has quit IRC19:21
*** lpetrut_ has joined #openstack-oslo19:22
*** lifeless has joined #openstack-oslo19:39
openstackgerritMonty Taylor proposed openstack/futurist master: Only install monotonic on python2  https://review.openstack.org/57437219:52
openstackgerritMonty Taylor proposed openstack/futurist master: Restrict contextlib2 to only python2  https://review.openstack.org/57437119:52
openstackgerritMonty Taylor proposed openstack/futurist master: Restrict contextlib2 to only python2  https://review.openstack.org/57437119:53
openstackgerritMonty Taylor proposed openstack/taskflow master: Restrict contextlib2 to only python2  https://review.openstack.org/58048219:54
*** lpetrut_ has quit IRC20:12
*** kgiusti has left #openstack-oslo20:24
*** dmellado has quit IRC20:28
*** raildo has quit IRC20:40
*** cdent has joined #openstack-oslo20:58
*** cdent has quit IRC21:33
*** e0ne__ has quit IRC21:41
*** e0ne has joined #openstack-oslo22:00
*** rcernin has joined #openstack-oslo22:00
*** nicolasbock has quit IRC22:17
*** edmondsw_ has quit IRC22:40
*** edmondsw has joined #openstack-oslo22:41
*** edmondsw has quit IRC22:45
*** e0ne has quit IRC22:50
*** e0ne has joined #openstack-oslo22:51
*** e0ne has quit IRC22:55
*** rcernin has quit IRC22:58
openstackgerritMichal Arbet proposed openstack/taskflow master: Fix code to support networkx > 1.0  https://review.openstack.org/57783323:01
*** rcernin has joined #openstack-oslo23:01
*** tosky has quit IRC23:01
*** ianychoi_ has joined #openstack-oslo23:01
*** ianychoi has quit IRC23:04

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