Friday, 2018-11-30

*** celebdor has quit IRC00:04
*** threestrands has joined #openstack-kuryr00:18
*** tosky has quit IRC00:27
*** rh-jelabarre has joined #openstack-kuryr00:41
*** gianpietro has joined #openstack-kuryr02:11
*** gianpietro has quit IRC02:18
*** premsankar has quit IRC02:43
*** hongbin has joined #openstack-kuryr03:02
openstackgerritMerged openstack/kuryr-kubernetes master: Support network policy update  https://review.openstack.org/60515003:31
*** hongbin has quit IRC03:56
*** janki has joined #openstack-kuryr04:03
*** threestrands has quit IRC05:20
*** pcaruana has quit IRC05:35
*** ccamposr has joined #openstack-kuryr07:07
*** pcaruana has joined #openstack-kuryr07:22
*** maysams has joined #openstack-kuryr07:51
*** celebdor has joined #openstack-kuryr08:13
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: devstack: Wait for the k8s api to be healthy  https://review.openstack.org/61854508:26
*** tosky has joined #openstack-kuryr08:42
*** celebdor has quit IRC08:49
*** celebdor has joined #openstack-kuryr08:51
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: devstack: parametrize openshift version fetching  https://review.openstack.org/61901809:06
celebdordulek: could it be that openshift doesn't like your https://review.openstack.org/#/c/620045/09:07
celebdor?09:07
celebdorah no. It seems like flakiness09:08
celebdor:/09:08
celebdorhttp://logs.openstack.org/45/620045/4/check/kuryr-kubernetes-tempest-daemon-openshift-octavia/378b9b2/testr_results.html.gz09:09
celebdoror maybe not09:09
celebdorhttp://paste.openstack.org/show/736468/09:10
dulekcelebdor: I'll look in a minute.09:11
celebdor;-)09:11
dulekcelebdor: Ha, how could I break namespace isolation?09:12
dulekltomasbo: ? ^09:12
celebdordulek: yeah09:13
celebdorthat's what I'm wondering09:14
ltomasboI think that test is broken in openshift?09:14
celebdor*taking out pokéball* ltomasbo, I choose you09:14
celebdorltomasbo: damn. Not the best platform to have isolation tests broken09:14
ltomasbocelebdor, not isolated test, just the svc one09:19
ltomasbocelebdor, it is the octavia sg thing...09:19
celebdorltomasbo: this test is pod to pod09:20
celebdornot pod to service09:20
ltomasbolet me see, I may have look at a different one09:20
celebdorltomasbo: test_namespace09:20
celebdorlook for # check no connectivity from NS1 to NS209:20
ltomasboI see this in your paste: in test_namespace_sg_svc_isolation09:20
ltomasbocelebdor, ahh, you meant that the part failing is on the pod to pod part09:22
ltomasbook, but the failure is weird...09:22
ltomasbotesttools.matchers._impl.MismatchError: 'kuryr-pod-615029703: HELLO! I AM ALIVE!!!\n' matches Contains('HELLO! I AM ALIVE!!!')09:22
ltomasbowhy there is a mismatch there?09:22
ltomasbocelebdor, ^^09:22
ltomasboIf I'm seeing it right, it does contain the string, right?09:22
celebdorltomasbo: you are right09:24
celebdorIt is that test09:24
celebdorand it is the octavia issue09:24
celebdorand of course, since we only test it in Octavia09:24
celebdorthat's the only gate that fials09:24
celebdors/Octavia/OpenShift/09:25
ltomasboyes, the line is checking that pod from NS1 cannot reach service on NS2, and that is not blocked at the moment09:25
celebdoryes, yes09:27
celebdoryou are right09:27
*** takamatsu has quit IRC09:41
*** pcaruana has quit IRC10:44
*** pcaruana has joined #openstack-kuryr10:50
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Enhance svc namespace isolation with LoadBalancer type svc  https://review.openstack.org/61875111:19
openstackgerritDanil Golov proposed openstack/kuryr-kubernetes master: Introduce test case document for SRIOV functionality  https://review.openstack.org/60002211:20
*** pcaruana has quit IRC12:11
openstackgerritMerged openstack/kuryr-kubernetes master: Add security groups driver for NP  https://review.openstack.org/60861612:40
openstackgerritMerged openstack/kuryr-kubernetes master: Fix definition of CNIRunner.get_container_id()  https://review.openstack.org/61999612:40
*** gianpietro has joined #openstack-kuryr13:36
*** takamatsu has joined #openstack-kuryr13:55
*** aperevalov has quit IRC14:21
celebdordulek: ltomasbo: dmellado: https://review.openstack.org/#/c/619018/14:24
dulekcelebdor: awk used as format operator? Do I see correctly? That's insane…14:26
dulekNo, I don't understand what's going on there. :P14:27
*** janki has quit IRC14:30
celebdordulek: that's just finding the line in checksum that defines the name of the server tarball14:31
celebdorand when it has it, it constructs the url to download the tarball14:31
dulekcelebdor: Oh…14:31
celebdorcause they have a non predictable commit string there in the filename14:32
celebdorso that's the only way I knew how to find it, so that our devstack users only need to specify the v3.10.0 and not have to lookup the whole url14:32
celebdorI think it is quite convenient14:32
celebdordulek: SonicSong is having some issues on devstack 16.04.3 with cni https://www.irccloud.com/pastebin/0TIEJjlE/14:36
SonicSongSorry, ubuntu 16.04.5 instead14:37
dulekSonicSong: Can you check out kuryr-daemon logs?14:38
SonicSongokay14:38
dulekSonicSong: It's either devstack@kuryr-daemon in systemd or a container on kube-system namespace.14:39
dulekSonicSong: Depending if you're trying to run Kuryr in containers.14:39
SonicSongi'm using systemd14:39
SonicSonghttps://www.irccloud.com/pastebin/es1uA352/14:39
SonicSonghere is a part of it14:40
dulekSonicSong: "Timed out waiting for requested pod to appear in registry" - okay, so seems like kuryr-controller is having troubles with something and doesn't annotate the pod.14:41
dulekSonicSong: It's devstack@kuryr-kubernetes.14:41
dulek(I know, we should rename it one day)14:41
SonicSonghttps://www.irccloud.com/pastebin/JjyxFT82/14:42
SonicSongi think it is the key part of the error14:43
SonicSongit's from kuryr-controller14:43
*** takamatsu has quit IRC14:48
*** gianpiet_ has joined #openstack-kuryr14:52
dulekSonicSong: Hm, no idea at the moment. Seems like annotation is added correctly.14:53
*** gianpie__ has joined #openstack-kuryr14:53
SonicSongdulek: I'm curious about the VIFOpenVSwitch part14:53
dulekSonicSong: But that's for pod svc-vdu1-e493cd-b54fd7f95-6whrz. And previous error was about the same?14:54
dulekYeah, that's the same.14:54
dulekSonicSong: What's wrong with that one?14:54
*** gianpiet_ has quit IRC14:54
SonicSongdulek: Seems that it's inactive?14:54
dulekSonicSong: Yeah, it will be inactive until kuryr-daemon plugs it.14:55
*** gianpietro has quit IRC14:55
SonicSongdulek: oh, I see14:55
dulekSonicSong: You can try restarting kuryr-daemon. Maybe it wasn't watching when pod was created, but that's rather unlikely.14:55
SonicSongI will have a try14:56
SonicSongdulex: it seems the same14:59
SonicSongoops...sorry14:59
SonicSongdulek: it seems the same14:59
dulekSonicSong: Can you mail me full logs? mdulko@redhat.com15:01
SonicSongdulek: sure, which part do you want15:01
SonicSongdulek: both kuryr-controller and daemon?15:02
dulekSonicSong: Yup!15:02
SonicSongokay15:02
dulekSonicSong: Maybe just kuryr-daemon first. :P15:02
SonicSongdulek: okay15:03
dulekSonicSong: Okay, I have a hypothesis, just a second…15:13
dulekSonicSong: Oh my, you've discovered a really sneaky bug. :D15:14
dulekSonicSong: kuryr-daemon watches for pods based on hostname.15:15
dulekAnd it looks like K8s nodeName is lowercased hostname.15:15
dulekWe're not doing lowercasing.15:15
dulekSonicSong: Let me check out if there's a workaround for you.15:15
dulekSonicSong: Yeah, for a quick fix you can set KUBERNETES_NODE_NAME=kuryr-demo-kvm in the env that starts kuryr-daemon, e.g. in systemd servicefile.15:17
dulekSonicSong: I'll fix that on upstream.15:17
*** gianpie__ has quit IRC15:20
*** ccamposr has quit IRC15:27
*** takamatsu has joined #openstack-kuryr15:29
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Lowercase hostname when using it as K8s nodeName  https://review.openstack.org/62118815:36
dulekSonicSong: The fix is above, you can apply it to your repo and restart kuryr-daemon - it should help.15:37
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Enhance svc namespace isolation with LoadBalancer type svc  https://review.openstack.org/61875115:48
*** maysams has quit IRC15:52
*** takamatsu has quit IRC15:53
*** premsankar has joined #openstack-kuryr16:14
*** pcaruana has joined #openstack-kuryr16:26
*** celebdor has quit IRC16:51
*** pcaruana has quit IRC17:04
openstackgerritMerged openstack/kuryr-kubernetes master: Fix quota readiness check  https://review.openstack.org/62088317:35
*** openstackgerrit has quit IRC17:51
*** celebdor has joined #openstack-kuryr19:31
*** rh-jelabarre has quit IRC20:03
*** rh-jelabarre has joined #openstack-kuryr20:05
*** jistr has quit IRC20:11
*** jistr has joined #openstack-kuryr20:13
*** takamatsu has joined #openstack-kuryr20:53
*** celebdor has quit IRC21:23
*** tosky has quit IRC23:42

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