Friday, 2018-03-02

*** dulek__ has quit IRC00:01
*** dulek__ has joined #openstack-kuryr00:03
*** dulek__ has quit IRC00:03
*** dulek__ has joined #openstack-kuryr00:05
*** kiennt26 has joined #openstack-kuryr00:41
*** jlabarre has quit IRC00:42
*** jdavis has quit IRC01:01
*** maysamacedos has joined #openstack-kuryr01:17
*** dulek__ has quit IRC01:30
*** yamamoto has joined #openstack-kuryr01:35
*** yamamoto_ has quit IRC01:38
*** dulek__ has joined #openstack-kuryr01:41
*** dulek__ has quit IRC02:11
*** dulek__ has joined #openstack-kuryr02:14
*** caowei has joined #openstack-kuryr02:15
*** salv-orlando has joined #openstack-kuryr02:18
*** salv-orl_ has quit IRC02:21
*** maysamacedos has quit IRC02:35
*** dulek__ has quit IRC02:53
*** dulek__ has joined #openstack-kuryr02:59
*** dulek__ has quit IRC03:04
*** dulek__ has joined #openstack-kuryr03:29
*** dulek__ has quit IRC03:30
*** dulek__ has joined #openstack-kuryr03:31
*** janonymous has joined #openstack-kuryr03:41
*** dangtrinhnt has joined #openstack-kuryr03:47
*** ajo has quit IRC04:10
*** kong has quit IRC04:10
*** janonymous has quit IRC04:10
*** portdirect has quit IRC04:11
*** fkautz has quit IRC04:11
*** kong has joined #openstack-kuryr04:12
*** ajo has joined #openstack-kuryr04:12
*** janonymous has joined #openstack-kuryr04:12
*** fkautz has joined #openstack-kuryr04:12
*** portdirect has joined #openstack-kuryr04:17
*** dulek__ has quit IRC04:24
*** dulek__ has joined #openstack-kuryr04:24
*** yamamoto has quit IRC04:40
*** yamamoto has joined #openstack-kuryr04:40
*** gcheresh has joined #openstack-kuryr05:02
*** gcheresh has quit IRC05:08
*** janki has joined #openstack-kuryr05:20
*** gcheresh has joined #openstack-kuryr05:42
*** dulek__ has quit IRC05:45
*** dulek__ has joined #openstack-kuryr05:45
*** dulek__ has quit IRC05:47
*** dulek__ has joined #openstack-kuryr05:49
*** gcheresh has quit IRC06:04
*** gcheresh has joined #openstack-kuryr06:27
*** juriarte has joined #openstack-kuryr07:07
*** jchhatbar has joined #openstack-kuryr07:13
*** janki has quit IRC07:14
*** gcheresh has quit IRC07:25
*** dulek__ has quit IRC07:25
*** pcaruana has joined #openstack-kuryr07:34
*** pmannidi has quit IRC07:35
*** slaweq has joined #openstack-kuryr07:36
dmelladofolks, as noone will be surprised about07:56
dmelladotoday's sessions are cancelled07:56
dmelladowe'll try to get some space at the hotel and start open discussions and code sprint07:56
dmelladobut timing is nothing that we could predict in advance07:56
dmelladoltomasbo: juriarte ^^07:57
ltomasbothanks for the update dmellado07:57
ltomasbodmellado, if you are going to have any live (BJN) session, just ping us here07:57
juriartesure, thanks dmellado07:58
ltomasbodmellado, btw, can we talk about the issue on kuryr-kubernetes vs kuryr-tempest-plugin testing problem?07:58
ltomasbodmellado, I guess that is something you can talk about there, my only idea is to do the ñapa...07:58
dmelladoltomasbo: I was checking that and that's you fault xD08:04
dmelladoplease do make it in a way that doesn't break compatibility08:04
dmelladosame would had happened with former settings08:05
dmelladoI mean08:05
dmelladorepo A => change that depends on B08:05
dmelladorepo B => change that depends on A08:05
dmelladoBOOOM08:05
ltomasboyep, I know,08:05
dmelladoI'll comment on your patch later on, but let's rewrite it in a way so that this doesn't happen08:06
ltomasbobut, if there is a bug in repo A, and B relies on that bug (it is not this case)08:06
ltomasboso we need to maintain backwards compatibility to the bug?08:06
ltomasbowe will hit this issue sooner or later08:06
ltomasbodmellado, I'll do the quick fix for patch on B, that is just adding and 'or' at the if08:07
*** livelace has joined #openstack-kuryr08:07
dmelladoltomasbo: do the 'ñapa' for now08:07
ltomasboby the way, dmellado, is apuimedo with you?08:07
dmelladothe kuryr refugees on snowpenstack ptg will appreciate it08:07
dmelladoyeah, we're all having breakfast08:07
dmelladoxD08:08
ltomasbotell him to read this thread: ovn: Support port groups in ACLs (specially email from Mark Michelson)08:08
ltomasbohi has tested Numan Siddique's patch08:08
ltomasbo*he08:08
dmelladowe'll take a look08:09
ltomasboand results are, for creating 100 ports: from 2 min and 17 seconds ---> to 6 seconds08:09
dmelladosounds awesome08:09
ltomasboand for 200 ports from 9min and 29 seconds ---> to 16 seconds!08:09
ltomasboapuimedo will love this!08:09
dmelladoar yu suar?08:09
dmelladoxD08:09
dmelladosounds awesome08:10
ltomasboI didn't try it, but sounds good08:10
dmelladoa-ha, so you didn't try it!08:10
dmelladoheh08:10
ltomasboI just read the email!08:10
ltomasboxD08:10
ltomasboI'll wait until is fully reviewed and merged in ovn upstream08:11
ltomasbothen we will test it in our gate! xD08:11
*** slaweq has quit IRC08:15
dmelladosounds like a plan!08:16
dmelladonext week, if I'm back in Madrid and not in Ireland or around the world08:16
*** slaweq has joined #openstack-kuryr08:16
dmelladowhich I do hope08:19
dmelladoI'll sit with you and explain a few things about THE ZUUL08:19
dmelladoxD08:19
ltomasboxD08:22
*** yamamoto has quit IRC08:26
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Ensure port name includes namespace name  https://review.openstack.org/54851808:28
*** aojeagarcia has joined #openstack-kuryr08:34
*** dulek_ has joined #openstack-kuryr08:41
dulek_We're still having breakfast, so it'll take a moment for us to move to the hotel. Sorry about that!08:45
openstackgerritMerged openstack/kuryr-tempest-plugin master: Check connectivity from Pod to VM  https://review.openstack.org/54895408:48
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add network id to pools keys  https://review.openstack.org/54867308:49
*** dulek_ has quit IRC09:04
*** yamamoto has joined #openstack-kuryr09:15
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Ensure port name includes namespace name  https://review.openstack.org/54851809:22
*** yamamoto has quit IRC09:29
*** janonymous has quit IRC09:40
*** celebdor has joined #openstack-kuryr09:52
*** shadower has joined #openstack-kuryr09:53
*** kiennt26 has quit IRC10:03
dmelladoltomasbo Internet connectivity is not really working here...10:12
*** olivierbourdon38 has joined #openstack-kuryr10:12
dmelladoFyi10:12
dmelladoExpect outages from our side...10:12
ltomasbodmellado, ok, so no BJ today XD10:12
ltomasbodmellado, by the way, I fixed the kuryr-tempest-plugin thing, in case you want to take a look at the patch10:13
*** gcheresh has joined #openstack-kuryr10:17
*** celebdor has quit IRC10:21
*** caowei has quit IRC10:24
*** gcheresh has quit IRC10:29
*** gcheresh has joined #openstack-kuryr10:30
dmelladoI got internet!10:50
dmelladoxD10:50
*** gcheresh has quit IRC10:54
*** celebdor has joined #openstack-kuryr10:59
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: WiP: Revert "Use virtualenv to build kuryr-cni Docker image"  https://review.openstack.org/54900811:32
*** maysamacedos has joined #openstack-kuryr11:36
*** yamamoto has joined #openstack-kuryr12:06
dmelladoInternet is blazing fast here12:15
*** celebdor has quit IRC12:19
*** olivierbourdon38 has quit IRC12:20
*** yamamoto has quit IRC12:29
*** olivierbourdon38 has joined #openstack-kuryr12:35
*** jlabarre has joined #openstack-kuryr12:40
openstackgerritMerged openstack/kuryr-kubernetes master: Add readiness and liveness checks to CNI.  https://review.openstack.org/53772112:50
*** yamamoto has joined #openstack-kuryr13:02
*** celebdor has joined #openstack-kuryr13:20
celebdorltomasbo: ping13:22
ltomasbocelebdor, pong13:22
celebdorltomasbo: I forgot what I wanted to ask13:24
celebdorah, yes13:25
celebdorltomasbo: you need to rebase your cni stuff13:25
ltomasboman, I was not that slow answering the ping!13:25
*** jchhatbar has quit IRC13:25
celebdorltomasbo: it took you longer than my volatile memory holds stuff13:25
ltomasboxD13:25
ltomasbocelebdor, which one?13:26
celebdorthe one where you are moving stuff around13:27
celebdorin dirs and crap13:27
celebdorpackages even IIRC13:27
ltomasbook, what do you want me to add?13:27
ltomasbohttps://review.openstack.org/#/c/548228/13:27
ltomasbocelebdor, ^^13:27
ltomasbocelebdor, not sure how to read: "in dirs and crap" xD13:29
ltomasbocelebdor, does it mean you don't like the re-shape?13:29
celebdorltomasbo: it means that I didn't check if it does anything else13:30
celebdorso "crap" stands for whatever else it does13:30
celebdor:P13:30
ltomasboit does not13:30
celebdorsorry for the language13:30
celebdorI only have one thing against the patch13:30
*** janki has joined #openstack-kuryr13:30
ltomasbojust moving all the k8s-cni-plugins to a common folder13:30
celebdorwhich is that it may make backports a bit more complicated13:30
ltomasboinstead of being at main, api and service13:30
celebdorbut it's okay I suppose13:30
ltomasbothat is true13:31
ltomasbobut if you change X at plugin Y, it will go where it went before, as I moved the complete block13:32
ltomasboso, it should not be that bad13:32
celebdorokey dokey13:32
celebdorso just rebase it to solve the conflict and we're good13:33
*** yamamoto has quit IRC13:33
ltomasboumm, it does not show me any conflict, strange13:33
ltomasbo(on gerrit UI)13:33
celebdorlet me check again13:35
celebdorthat's weird, it shows it to me from the patch list13:36
ltomasbocelebdor, weird enough, mozilla says it is ok, google chrome says it is not13:36
celebdorxD13:36
ltomasboI'll fix it13:36
celebdorltomasbo: just rebase to make sure13:36
ltomasboyep13:37
*** yamamoto has joined #openstack-kuryr13:39
openstackgerritMerged openstack/kuryr-kubernetes master: Add experimental containerized jobs  https://review.openstack.org/53482714:12
*** olivierbourdon38 has quit IRC14:15
*** yamamoto has quit IRC14:16
*** maysamacedos has quit IRC14:20
ltomasbocelebdor, problem is that the commit does not appears yet on the github: https://github.com/openstack/kuryr-kubernetes/commits/master14:36
ltomasboI'll do the rebase asap14:36
*** garyloug has joined #openstack-kuryr14:56
*** jdavis has joined #openstack-kuryr14:57
celebdorthanks15:05
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Move cni plugins to a common folder  https://review.openstack.org/54822815:10
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Make CNI Registry Plugin namespace aware  https://review.openstack.org/54831315:10
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes master: Updated from global requirements  https://review.openstack.org/54925315:10
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add network id to pools keys  https://review.openstack.org/54867315:12
*** slaweq has quit IRC15:16
*** slaweq has joined #openstack-kuryr15:17
*** yamamoto has joined #openstack-kuryr15:20
*** slaweq has quit IRC15:22
*** livelace-link has joined #openstack-kuryr15:29
*** livelace-link has quit IRC15:31
ltomasboping dulek15:33
*** livelace-link has joined #openstack-kuryr15:34
dulekltomasbo: What's up?15:34
ltomasbodulek, I see you set the os-vif to not be a bug anymore15:34
ltomasbodulek, did you proof your point with the patch you submitted yesterday?15:34
ltomasbowere you able to reproduce it?15:34
dulekltomasbo: Yup, take a look on kubelet logs here: https://review.openstack.org/#/c/534827/15:35
ltomasboohh, I just see you were waiting for the containerized...15:35
dulekltomasbo: I mean - in experimental containerized jobs.15:35
dulekltomasbo: I've made it depend from containerized.15:35
dulekltomasbo: So it was reproduced.15:35
*** yamamoto has quit IRC15:35
*** dougbtv has joined #openstack-kuryr15:36
ltomasbogreat! thanks!15:36
ltomasboso are we safe then? with both venv and rpm base, right?15:36
dulekltomasbo: Yup! And current upstream container is fine as well.15:36
dulekThough I think I'll try to implement the `docker run` version upstream as well.15:37
dulekJust to be consistent.15:37
ltomasbosounds good15:38
*** maysamacedos has joined #openstack-kuryr15:45
*** slaweq has joined #openstack-kuryr15:47
*** jdavis has quit IRC15:51
*** yamamoto has joined #openstack-kuryr15:51
*** slaweq has quit IRC15:52
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: cni health: Avoid capsh dependency  https://review.openstack.org/54927616:03
*** yamamoto has quit IRC16:17
*** yamamoto has joined #openstack-kuryr16:21
*** dougbtv is now known as dougbtv|laptop16:24
*** yamamoto has quit IRC16:26
*** yamamoto has joined #openstack-kuryr16:27
*** yamamoto has quit IRC16:35
*** yamamoto has joined #openstack-kuryr16:42
*** yamamoto has quit IRC16:43
*** yamamoto has joined #openstack-kuryr16:51
*** janki has quit IRC16:55
dulekdmellado: Can you get the workflow flag from celebdor's patch?16:58
dulekdmellado: I'd like to run containerized gates with that before merging.16:58
dmelladoSure thing16:58
dulekdmellado: If you get +W from it it'll have +2 from Zuul, but will not merge.16:58
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928916:58
dmelladoLet's check16:59
dmelladoShuuu flight cancelled again....16:59
dulekdmellado: Uhhh…17:00
dulekdmellado: What's the time of your flight?17:01
dmelladoTomorrow morning17:01
dmelladoRed alert got extended to 9 am17:02
dmelladoLet's see17:02
dmelladoHow about yours?17:02
*** aojeagarcia has quit IRC17:03
dulekdmellado: So you've got the info on cancellation or not? Mine's fine, I guess.17:04
dmelladoI got a message, yeah17:04
dmelladoGood luck in any case...17:04
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928917:05
*** aojeagarcia has joined #openstack-kuryr17:07
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928917:10
celebdormaysamacedos: I filed a bug https://bugs.launchpad.net/kuryr-kubernetes/+bug/175293917:19
openstackLaunchpad bug 1752939 in kuryr-kubernetes "cni health memory accounting does not count with the children" [Undecided,New]17:19
celebdorlet me know what you think about it, you don't need to take it. I just want your opinion17:20
*** maysamacedos has quit IRC17:24
dmelladohttps://www.dropbox.com/sh/dtei3ovfi7z74vo/AAD9R3JUnqs3CHurSiaGDVEHa/Kuryr?dl=0&preview=DSC_4331.JPG17:25
dmelladothanks for attending folks, it was sunny back there!17:25
*** slaweq_ has joined #openstack-kuryr17:39
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Split .zuul.yaml into smaller files  https://review.openstack.org/54830917:42
*** slaweq_ has quit IRC17:43
*** jdavis has joined #openstack-kuryr17:44
*** celebdor has quit IRC17:52
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928917:57
*** dims has quit IRC18:51
*** dangtrinhnt has quit IRC18:54
*** juriarte has quit IRC18:57
*** dims has joined #openstack-kuryr19:01
*** dulek has quit IRC19:01
*** dangtrinhnt has joined #openstack-kuryr19:03
*** dulek has joined #openstack-kuryr19:06
*** gcheresh has joined #openstack-kuryr19:09
*** garyloug has quit IRC19:09
*** aojeagarcia has quit IRC19:12
*** jdavis has quit IRC19:14
*** yboaron_ has joined #openstack-kuryr19:31
*** maysamacedos has joined #openstack-kuryr19:34
*** mfedosin has quit IRC19:43
*** celebdor has joined #openstack-kuryr19:43
*** yboaron_ has quit IRC19:53
maysamacedoscelebdor: I took a look on the bug you reported, and since CNI uses cotyledon to create different process for (watcher/server/health)19:55
maysamacedosthe current implementation that checks memory leaks, only takes into account the health process19:57
maysamacedoswhich will check the leak existent in IPDB19:57
celebdormaysamacedos: that's what I feared :P19:57
maysamacedos:P19:58
celebdorI am thinking an easier solution than what I suggested in the bug19:58
celebdor:-019:58
celebdor:-)19:58
celebdorit just occurred to me19:58
celebdorbut I need to verify it19:59
celebdorthat maybe wit can just consider the container whole memory consumption19:59
celebdorsince only cni and its subprocesses run19:59
celebdormaybe that will make it easier19:59
maysamacedosyes, maybe19:59
celebdorif I manage to do it somehow I'll update hte bug19:59
maysamacedosok, I will also take a look on this20:00
celebdor:-)20:01
celebdoralso, instead of doing a tree walk like in option 220:02
maysamacedoscelebdor: please ping me if you found out something :D20:02
celebdorwe can just sum the consumption for all the processes20:02
celebdorsince the container only sees its own processes20:02
celebdor(and that's the safe and probably good solution :P )20:02
maysamacedosyes..20:03
celebdorit's much easier :-)20:04
celebdororiginally I was not taking into account that we only see the container procs20:04
celebdor:P20:04
maysamacedosyes, it's indeed easier20:06
maysamacedosI can work on that, if you won't20:06
celebdorI prefer you work on the plugin if you don't mind20:07
maysamacedosI don't :)20:07
celebdorI'll investigate if we can get the memory all at once from cgroups somehow20:07
maysamacedosok20:07
openstackgerritDaniel Mellado proposed openstack/kuryr-tempest-plugin master: Run OVN job containerized and daemonized  https://review.openstack.org/54931420:08
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928920:09
openstackgerritDaniel Mellado proposed openstack/kuryr-tempest-plugin master: Run OVN job containerized and daemonized  https://review.openstack.org/54931420:12
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928920:22
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928920:29
celebdormaysamacedos: I think I found it20:37
celebdorit should be the content of this file /sys/fs/cgroup/memory/memory.memsw.usage_in_bytes20:38
maysamacedosmmm20:39
maysamacedosgreat o/20:41
*** mfedosin has joined #openstack-kuryr20:43
*** slaweq_ has joined #openstack-kuryr20:57
*** slaweq_ has quit IRC21:02
celebdorP:-)21:08
*** atoth has quit IRC21:17
*** jdavis has joined #openstack-kuryr21:19
*** maysamacedos has quit IRC21:19
*** dougbtv|laptop has quit IRC21:22
*** jdavis has quit IRC21:23
*** jdavis has joined #openstack-kuryr21:24
*** dougbtv_ has quit IRC21:26
*** dougbtv_ has joined #openstack-kuryr21:28
*** jdavis has quit IRC21:29
*** gcheresh has quit IRC21:47
*** jdavis has joined #openstack-kuryr21:57
*** livelace has quit IRC22:00
*** jdavis_ has joined #openstack-kuryr22:01
*** jdavis has quit IRC22:01
*** pcaruana has quit IRC22:08
*** slaweq_ has joined #openstack-kuryr22:08
*** slaweq_ has quit IRC22:13
*** dims has quit IRC22:17
*** maysamacedos has joined #openstack-kuryr22:49
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: cni health: track all cgroup memory usage  https://review.openstack.org/54933022:57
*** jdavis_ has quit IRC22:58
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: cni health: track all cgroup memory usage  https://review.openstack.org/54933023:12
*** maysamacedos has quit IRC23:23
*** slaweq_ has joined #openstack-kuryr23:26
*** slaweq_ has quit IRC23:30

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