Friday, 2018-03-23

*** hongbin has joined #openstack-kuryr01:15
*** caowei has joined #openstack-kuryr01:42
*** dangtrinhnt has joined #openstack-kuryr01:56
*** caowei has quit IRC02:09
*** maysamacedos has joined #openstack-kuryr02:11
*** caowei has joined #openstack-kuryr02:30
*** dougbtv__ has joined #openstack-kuryr02:34
*** dougbtv_ has quit IRC02:34
*** livelace-link has quit IRC02:45
*** livelace-link has joined #openstack-kuryr02:46
*** salv-orl_ has joined #openstack-kuryr02:47
*** livelace-link has quit IRC02:48
*** salv-orlando has quit IRC02:50
*** hongbin has quit IRC03:47
*** maysamacedos has quit IRC04:59
*** gcheresh has joined #openstack-kuryr05:25
*** dangtrinhnt has quit IRC05:34
*** gcheresh has quit IRC05:40
*** janonymous has joined #openstack-kuryr06:04
*** gcheresh has joined #openstack-kuryr06:11
*** celebdor has joined #openstack-kuryr06:12
celebdordulek: https://streamable.com/qyiof06:21
celebdor:-)06:21
*** aojea has joined #openstack-kuryr06:52
*** lihi has quit IRC07:00
*** lihi has joined #openstack-kuryr07:01
*** kiennt26 has quit IRC07:05
*** gcheresh has quit IRC07:10
openstackgerritjihoon proposed openstack/kuryr-kubernetes master: remove unnecessary checking activated neutron port when using macvlan  https://review.openstack.org/55566707:16
*** pcaruana has joined #openstack-kuryr07:21
*** aojea has quit IRC07:28
dmelladocelebdor: soccer? xD08:22
celebdoryeah08:24
celebdoralmost08:24
dulekcelebdor: Oh, this just got international? I've seen the original post with that somewhere. :D08:25
celebdor:-)08:25
dmelladohttps://www.youtube.com/watch?v=MXrRNf7myX408:32
dmelladoxD08:32
*** celebdor has quit IRC08:33
*** phuoc_ has quit IRC08:34
*** celebdor has joined #openstack-kuryr08:49
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Add CA certificates Secret and mount it  https://review.openstack.org/55550209:01
*** aojea has joined #openstack-kuryr09:10
celebdordulek: why are we encoding the bundle in base64?10:04
dulekcelebdor: Don't think I wanted to encrypt it. :D10:05
dulekcelebdor: https://kubernetes.io/docs/concepts/configuration/secret/10:05
dulekcelebdor: It's required by Secret API in K8s. Otherwise it will not accept it.10:05
celebdorI suspected as much10:05
celebdorthanks for the confirmation10:05
dulekcelebdor: I guess that's one difference between ConfigMap. :D10:05
*** yamamoto has quit IRC10:05
celebdordulek: I suppose then when mounting it gets decoded, right?10:06
*** yamamoto has joined #openstack-kuryr10:06
dulekcelebdor: Yes, in the file it's decoded.10:06
celebdorright10:06
celebdordulek: have to -1 due to indentation https://review.openstack.org/#/c/555502/210:07
dulekcelebdor: Ha, you're right. I shouldn't code after midnight. ;)10:08
dulekcelebdor: Okay, I've addressed the comments and moved resources creation into post-config. Will upload new version once it's verified on my env.10:11
*** yamamoto has quit IRC10:11
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Add CA certificates Secret and mount it  https://review.openstack.org/55550210:15
celebdordulek: I'm getting rid of the OBJECT_KIND contract as well10:26
dulekcelebdor: I'm not opposed if you believe it's worth it.10:27
celebdordulek: I generally dislike magic :P10:27
celebdorI'm a muggle10:27
dulekcelebdor: You're stripping magic from the code. That's more like a task for member of Department of Magical Accidents and Catastrophes.10:30
celebdordulek: Yay! I'm like Arthur Weasley10:31
* dulek is amazed how diligent is Harry Potter fandom wikia.10:31
celebdordulek: I just bought the trilogy illustrated10:31
celebdorhttps://www.amazon.co.uk/Harry-Potter-Philosophers-Stone-Illustrated/dp/1408845644/ref=sr_1_1?ie=UTF8&qid=1521801111&sr=8-1&keywords=harry+potter+illustrated10:31
celebdorit is brilliant10:32
celebdorI hope they won't stop at those three10:32
*** gcheresh has joined #openstack-kuryr10:33
dulekcelebdor: Beautiful illustrations. Seems like they're a bit inspired by the movies?10:34
celebdornot the characters10:34
celebdorat least10:34
dulekcelebdor: There is one issue with HP books. I've read first 4 multiple times, so I remember the plot in general.10:35
dulekcelebdor: Then the next ones I've read like over a single night. So I don't remember anything from them now. :D10:35
celebdorxD10:36
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Add Kuryr pods logs to gate results  https://review.openstack.org/55525410:48
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504010:48
dmelladolol10:51
*** caowei has quit IRC10:51
dmelladothey were showing an exhibition from HP just until a while ago here in Madrid10:52
dmelladoand you folks wouldn't believe the amount of cosplayers around it10:52
dmelladosome of those were really creepy, just like Madrid's wax museum10:52
celebdordmellado: for a moment I thought you meant the printer maker10:53
dmelladolol10:58
dmelladoHPe cosplayers? xD10:59
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: [WIP] Make handlers pluggable  https://review.openstack.org/55573510:59
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: [WIP] Simplify handlers interface  https://review.openstack.org/55573610:59
celebdordulek: let me know what you think of those two patches10:59
celebdormostly the interface and configurabilityy10:59
celebdordmellado: same thing11:00
dulekSure.11:00
*** gcheresh has quit IRC11:00
dmelladosure11:01
dmelladocelebdor: https://ssl-product-images.www8-hp.com/digmedialib/prodimg/lowres/c02920449.png11:01
dmelladowould you cosplay as this?11:01
celebdordmellado: of course. It is much easier11:01
celebdorjust a box painted white11:01
*** phuoc has joined #openstack-kuryr11:01
celebdordulek: regarding https://review.openstack.org/#/c/555040/3/cni_ds_init11:01
dulekcelebdor: Uhm?11:02
celebdordo you think it would be hard to to swap the sleep for something that checks for the status?11:02
celebdorsimilar to wait_for11:02
dulekcelebdor: Hm, to be honest it shouldn't. I can just loop on $CONTAINERID contents. I can try that.11:04
celebdordulek: that's another option11:04
celebdora loop for that with a 1 second sleep should be better11:04
dulekcelebdor: Yup, but with some maximum tries limit.11:05
celebdorsure11:05
celebdorit can be 10 :P11:05
dulekcelebdor: :D11:05
dmelladoxD11:06
*** yamamoto has joined #openstack-kuryr11:07
*** phuoc has quit IRC11:12
*** phuoc has joined #openstack-kuryr11:12
*** phuoc_ has joined #openstack-kuryr11:12
*** phuoc_ has quit IRC11:12
*** yamamoto has quit IRC11:13
*** pcaruana has quit IRC11:27
*** dangtrinhnt has joined #openstack-kuryr11:37
*** dangtrinhnt has quit IRC11:53
*** yamamoto has joined #openstack-kuryr11:56
*** yamamoto has quit IRC11:56
*** jlabarre has joined #openstack-kuryr12:03
*** yamamoto has joined #openstack-kuryr12:10
*** janonymous has quit IRC12:24
*** yamamoto has quit IRC12:25
*** maysamacedos has joined #openstack-kuryr12:26
* celebdor -> lunch12:27
openstackgerritDoug Hellmann proposed openstack/kuryr master: add lower-constraints job  https://review.openstack.org/55575412:28
openstackgerritDoug Hellmann proposed openstack/kuryr-kubernetes master: add lower-constraints job  https://review.openstack.org/55575512:28
openstackgerritDoug Hellmann proposed openstack/kuryr-libnetwork master: add lower-constraints job  https://review.openstack.org/55575612:29
*** pcaruana has joined #openstack-kuryr12:30
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504012:31
*** caowei has joined #openstack-kuryr12:42
dmelladolower-constraints? xD12:44
dmelladohttp://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html fuuuu12:44
dmelladoso long xD12:44
*** maysamacedos has quit IRC12:49
dulekdmellado: Basically idea is to stop syncing global-requirements and just do lower bound.12:54
dmelladodulek: yeah, I went thru all the email now12:54
dulekdmellado: To give services more flexibility on which they depend.12:54
dulekdmellado: :)12:54
dmelladoI'll be adding our stable liaison to these reviews xD12:57
dmelladopoor luis xD12:57
dmelladothis just happened to be on his pto12:57
*** maysamacedos has joined #openstack-kuryr13:04
*** caowei has quit IRC13:07
*** yamamoto has joined #openstack-kuryr13:08
*** yamamoto has quit IRC13:08
*** yamamoto has joined #openstack-kuryr13:08
*** yamamoto has quit IRC13:08
*** yamamoto has joined #openstack-kuryr13:15
*** yamamoto has quit IRC13:19
dulekhttps://review.openstack.org/#/c/555502/ - this commit unblocks containerized gates.13:21
*** yamamoto has joined #openstack-kuryr13:30
*** yamamoto has quit IRC13:35
dmelladodulek: oh, well13:38
dmelladoI was thinking about turning tls-proxy off, tbh xD13:38
dmelladobut let's get your patch in13:38
dulekdmellado: Well, it's a coincidence, but we've hit similar issue with openshift-ansible.13:39
dulekdmellado: So it's great we'll fix it upstream first.13:40
dmelladodulek: yep13:40
dmelladojust +2+1 it13:40
celebdorthanks for the merge dmellado13:41
celebdordulek: thanks for the patch13:42
*** yamamoto has joined #openstack-kuryr13:45
*** yamamoto has quit IRC13:50
*** yamamoto has joined #openstack-kuryr13:51
*** yamamoto has quit IRC13:56
*** irenab has quit IRC14:18
*** irenab has joined #openstack-kuryr14:19
*** gcheresh has joined #openstack-kuryr14:40
*** yamamoto has joined #openstack-kuryr14:52
*** yamamoto has quit IRC14:57
*** hongbin has joined #openstack-kuryr15:14
openstackgerritMerged openstack/kuryr-kubernetes master: Add CA certificates Secret and mount it  https://review.openstack.org/55550215:45
*** yamamoto has joined #openstack-kuryr15:53
*** celebdor has quit IRC15:59
*** yamamoto has quit IRC15:59
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: add lower-constraints job  https://review.openstack.org/55575516:10
dmelladodulek: could you also handle the regeneration from the lower reqs and amend it to the patch?16:12
dulekdmellado: Sorry, I don't understand.16:12
*** gcheresh has quit IRC16:12
dmelladoyeah, if you read dough email at http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html16:13
dmelladothose lower reqs should be renegerated per project16:13
dulekdmellado: Oh, let's see.16:13
dmellado3. Run ".tox/lower-constraints/bin/pip freeze > lower-constraints.txt"16:14
dmellado   to create the initial version of the lower-constraints.txt file for16:14
dmellado   the current repo.16:14
dmelladothat was what I was expecting from dough patch, and what I was replying in my comment16:14
dulekdmellado: I think dhellmann's patch already did that, but I'm checking.16:15
dulekdmellado: Hm, some versions are actually different.16:16
dulekI wonder if that's correct.16:16
dmelladodulek: that's my concern16:16
dulekdmellado: I'll check…16:17
dulekdmellado: Okay, so for some reason there are differences.16:19
dmelladolet me double check my diffs as well16:21
dmelladowill get you a pastebin16:21
dulekdmellado: Okay, I know what's wrong. At least I think.16:22
dmelladohttp://paste.openstack.org/show/710227/16:25
dmelladosome minor updates, I do wonder whether it'd make sense to update those as well16:25
dulekdmellado: Same here, though I don't have pyinotify.16:25
dmelladodulek: yeah, pynotify went away16:26
dulekdmellado: And I would be totally fine with updates. But there's one downgrade!16:26
dmelladothe docstheme?16:27
dulekYEah.16:27
dulekBut okay, I see that commit updating it got reverted.16:27
dulekBut where? :P16:28
dulekOkay, f7d93e57e1661bbe26212e3313d9e0f7074cec09 reverted it.16:29
dulekSo 1.19 is is the correct one. I'm updating the file.16:29
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: add lower-constraints job  https://review.openstack.org/55575516:30
dmelladoheh, you were faster than me on updating it16:31
dmelladolet's see if we don't hit any issue16:31
dulekdmellado: No way, we rarely depend on versions.16:33
dulekThough an os.vif upgrade sneaked in… ;)16:34
dmelladodulek: I mean, there seems to be an issue on shade which is breaking this new job xD16:34
dmelladohoorray zuulv316:34
dulekdmellado: Infra's fixing it.16:36
* dulek lurks their channel.16:36
dmelladothen I guess I'll be calling it a day, will check later at night when I'm back from beers16:36
dmelladoI encourage you to get one as well, dulek16:36
dmelladoxD16:36
dulekdmellado: Be responsible, don't drink and merge!16:36
dmelladoxD16:36
dmelladoI can do both! xD16:36
dulek:)16:37
*** celebdor has joined #openstack-kuryr16:53
*** yamamoto has joined #openstack-kuryr16:55
*** aojea has quit IRC16:56
*** yamamoto has quit IRC17:00
*** pcaruana has quit IRC17:23
*** celebdor has quit IRC17:37
*** aojea has joined #openstack-kuryr17:51
*** yamamoto has joined #openstack-kuryr17:56
*** yamamoto has quit IRC18:02
*** aojea has quit IRC18:09
*** aojea has joined #openstack-kuryr18:12
*** aojea has quit IRC18:30
*** yamamoto has joined #openstack-kuryr18:58
*** yamamoto has quit IRC19:02
*** vikasc has quit IRC19:06
*** vikasc has joined #openstack-kuryr19:18
*** mspreitz has joined #openstack-kuryr19:28
*** openstackgerrit has quit IRC19:34
*** yamamoto has joined #openstack-kuryr19:58
*** yamamoto has quit IRC20:04
*** celebdor has joined #openstack-kuryr20:15
*** mspreitz has quit IRC20:50
*** livelace-link has joined #openstack-kuryr20:58
*** maysamacedos has quit IRC20:59
*** yamamoto has joined #openstack-kuryr21:00
*** yamamoto has quit IRC21:06
*** hongbin has quit IRC21:14
*** jlabarre has quit IRC21:59
*** yamamoto has joined #openstack-kuryr22:02
*** yamamoto has quit IRC22:08
*** aojea has joined #openstack-kuryr22:49
*** yamamoto has joined #openstack-kuryr23:04
*** yamamoto has quit IRC23:10
*** celebdor has quit IRC23:48
*** aojea has quit IRC23:54

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