| *** maysams has joined #openstack-kuryr | 01:23 | |
| *** hongbin has joined #openstack-kuryr | 01:55 | |
| *** maysams has quit IRC | 02:34 | |
| *** hongbin has quit IRC | 03:27 | |
| *** yboaron has joined #openstack-kuryr | 04:23 | |
| *** yboaron has quit IRC | 04:27 | |
| *** yboaron has joined #openstack-kuryr | 06:03 | |
| *** ccamposr has joined #openstack-kuryr | 06:40 | |
| *** pcaruana has joined #openstack-kuryr | 06:40 | |
| *** shachar has joined #openstack-kuryr | 06:54 | |
| *** shachar has quit IRC | 06:54 | |
| *** shachar has joined #openstack-kuryr | 07:15 | |
| *** aperevalov has joined #openstack-kuryr | 07:15 | |
| *** shachar has quit IRC | 07:16 | |
| dmellado | dulek: I now checked your patch xD | 07:18 |
|---|---|---|
| dmellado | I'll take a look at it | 07:18 |
| dulek | dmellado: Thanks. | 07:18 |
| dulek | dmellado: I see total gate unstability lately. It's hard to merge patches as well as get them tested. | 07:19 |
| dmellado | yeah, noticed it as well | 07:19 |
| dmellado | specially on octavia's | 07:19 |
| dmellado | I was going to try taking a look at it today | 07:19 |
| dmellado | but we should make sure our gates are stable if they're voting | 07:19 |
| dulek | dmellado: That would be great, at the moment I'm a bit swamped with what celebdor assigned to me. | 07:20 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-tempest-plugin master: Remove raising exceptions to indicate failure https://review.openstack.org/607648 | 07:25 |
| dmellado | dulek no worries, I'll take a look | 07:29 |
| celebdor | total gate unstability is a bit exaggerated. I prefer to call it semi-deterministic gate actuation | 07:41 |
| *** snapiri has quit IRC | 07:54 | |
| *** pcaruana has quit IRC | 07:55 | |
| *** pcaruana has joined #openstack-kuryr | 07:57 | |
| *** snapiri has joined #openstack-kuryr | 08:01 | |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Limit the scope of try-except block https://review.openstack.org/607606 | 08:13 |
| *** garyloug has joined #openstack-kuryr | 08:18 | |
| *** yboaron_ has joined #openstack-kuryr | 08:22 | |
| *** gkadam has joined #openstack-kuryr | 08:30 | |
| *** gkadam has quit IRC | 08:31 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Add kuryr-status utility for upgrade-checkers https://review.openstack.org/607311 | 08:41 |
| celebdor | ltomasbo: dulek: I guess we can merge https://review.openstack.org/#/c/606044/ , right? | 08:57 |
| * ltomasbo checking | 08:57 | |
| ltomasbo | fine with me, did you see yboaron's comment? | 08:57 |
| celebdor | ltomasbo: that's fit for a follow-up patch | 08:57 |
| celebdor | :-) | 08:57 |
| dulek | ltomasbo: I agree with celebdor here. ;) | 08:58 |
| ltomasbo | sure! | 08:58 |
| dulek | celebdor: That decision would be much easier with more deterministic gates. :D | 08:58 |
| yboaron | celebdor, dulek : I'm fine with that :-) | 08:58 |
| celebdor | dulek: I know :'( | 08:58 |
| celebdor | Right now I'm mad about https://review.openstack.org/#/c/605620/ | 08:58 |
| celebdor | damned over complicated unit tests | 08:59 |
| celebdor | xD | 08:59 |
| celebdor | they will be a pain to get right and useful | 08:59 |
| garyloug | Hi folks, our nested-dpdk patch has been merged in os-vif. Now we need to refactor our nested-dpdk patch in kuryr to use it. The issue is the tox tests. These won’t use the latest os-vif master but will use the latest stable version. This does not include our patch and will cause the tests to fail… I’m sure this is a common problem. Any suggestion how we work around it? | 09:18 |
| celebdor | great | 09:19 |
| celebdor | I don't remember | 09:20 |
| celebdor | for the gates depends on should be enough | 09:20 |
| celebdor | or maybe libs from git | 09:21 |
| celebdor | garyloug: but we may have had to force a mid-cycle release sometimes | 09:21 |
| dulek | garyloug: Well, you just need to wait for the release and then raise the requirements (first global, then Kuryr's). | 09:31 |
| dulek | garyloug: Not really anything else can be done, it's normal procedure. | 09:31 |
| dmellado | dulek: hmmm I didn't get to review your kuryr-status patch | 09:32 |
| dmellado | next time I'll have you fix the spec to rdo at the same time | 09:32 |
| * dmellado fixing it... | 09:32 | |
| dulek | garyloug: You can try LIBS_FROM_GIT, but I'll be against putting it into any gates - doing so stops us from testing a python package and its requirements. | 09:32 |
| dulek | dmellado: Crap, forgot about it. | 09:32 |
| *** garyloug_ has joined #openstack-kuryr | 09:33 | |
| *** garyloug has quit IRC | 09:36 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: [WIP] Support network policy update https://review.openstack.org/605150 | 09:51 |
| *** garyloug__ has joined #openstack-kuryr | 10:04 | |
| *** garyloug_ has quit IRC | 10:09 | |
| garyloug__ | thanks guys | 10:10 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: devstack: Add compatibility for newer k8s https://review.openstack.org/606044 | 10:24 |
| dmellado | garyloug__: in any case for the gates you can even force on one specific version hash | 10:54 |
| dmellado | if you create your gate on experimental I won't be against that | 10:55 |
| *** pcaruana has quit IRC | 11:02 | |
| *** pcaruana has joined #openstack-kuryr | 11:02 | |
| *** janki has joined #openstack-kuryr | 11:06 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-kubernetes master: K8S-services: add support for UDP ports https://review.openstack.org/607938 | 11:55 |
| *** rh-jelabarre has joined #openstack-kuryr | 12:04 | |
| *** celebdor has quit IRC | 12:24 | |
| *** garyloug_ has joined #openstack-kuryr | 12:29 | |
| *** celebdor has joined #openstack-kuryr | 12:30 | |
| *** garyloug__ has quit IRC | 12:33 | |
| *** yboaron has quit IRC | 12:33 | |
| *** yboaron_ has quit IRC | 12:34 | |
| *** yboaron has joined #openstack-kuryr | 12:34 | |
| *** maysams has joined #openstack-kuryr | 13:22 | |
| *** yboaron has quit IRC | 13:29 | |
| *** yboaron has joined #openstack-kuryr | 13:29 | |
| *** garyloug__ has joined #openstack-kuryr | 13:32 | |
| *** garyloug_ has quit IRC | 13:35 | |
| *** pcaruana has quit IRC | 13:50 | |
| *** oanson has quit IRC | 13:51 | |
| dulek | Hey, yboaron, had you ever tested ingress + containerized deployments? | 13:58 |
| dulek | yboaron: Or rather Routers + Routes, not ingress. | 13:58 |
| dulek | yboaron: Just noticed that we haven't added those resources to cluster role: https://github.com/openstack/kuryr-kubernetes/blob/master/devstack/lib/kuryr_kubernetes#L490 | 13:59 |
| dulek | Actually same goes with NetworkPolicies, dmellado. | 13:59 |
| *** gkadam has joined #openstack-kuryr | 14:00 | |
| yboaron | dulek, Hmmm, don't think so, but as far as I remember our ingress gates used kuryr containerized, and I saw it green before | 14:00 |
| dulek | yboaron: ingress is there, just no routes or routers. | 14:00 |
| yboaron | dulek, ingress is not supported currently in Kuryr just ocp-routes | 14:01 |
| dulek | yboaron: Hm, okay, checking the gates then. Thanks! | 14:01 |
| yboaron | dulek, seems that Ingress gate in non-containerized : https://github.com/openstack/kuryr-kubernetes/blob/master/.zuul.d/octavia.yaml#L190 | 14:06 |
| dulek | yboaron: Yup, so I guess containerized version wouldn't work too well. :) | 14:07 |
| dulek | yboaron: I'm rewriting the ServiceAccount definition at the moment, so I'll see if I can fix this as well. | 14:08 |
| yboaron | dulek, thanks for letting me know, I"ll create extra ingress gate for containerized mode and make it depend on your patch | 14:09 |
| yboaron | dulek, 10x! | 14:09 |
| yboaron | dulek, BTW, The Routes resource should be added to ServiceAccount only for the Openshift case, I assume that this is our first conditional resource | 14:11 |
| dulek | yboaron: Yup, though I don't think having more hurts - we add kuryrnets unconditionally and nothing breaks. | 14:16 |
| dmellado | dulek: network policies goes namespaced | 14:24 |
| dulek | dmellado: But you're watching for NetworkPolicy resource, aren't you? Or you need to get it. | 14:25 |
| dmellado | but I'm watching actually for network policy + namespace | 14:25 |
| dulek | dmellado: Suuure. It's just matter of permissions. When Kuryr runs in pods it gets token of the kuryr-controller ServiceAccount. | 14:26 |
| dulek | And it has those permissions only: https://github.com/openstack/kuryr-kubernetes/blob/master/devstack/lib/kuryr_kubernetes#L490 | 14:27 |
| *** shachar has joined #openstack-kuryr | 14:27 | |
| *** snapiri has quit IRC | 14:29 | |
| dmellado | dulek: lemme check, I'm a little bit dizzy today so cope with me | 14:30 |
| dmellado | dulek: got it | 14:30 |
| dmellado | I'll send a patch to add the resource | 14:31 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: [WIP] Support network policy update https://review.openstack.org/605150 | 14:35 |
| *** jchhatbar has joined #openstack-kuryr | 14:50 | |
| *** janki has quit IRC | 14:53 | |
| *** shachar is now known as snapiri | 14:57 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-kubernetes master: Add containerized gate for openshift Routes https://review.openstack.org/607992 | 15:30 |
| yboaron | dulek, let me know you have the patch with update service account, I would like to set dependency @ https://review.openstack.org/#/c/607992/ | 15:35 |
| dulek | yboaron: Sure, I can try to write it now… | 15:39 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: Add experimental gate for Octavia OVN provider https://review.openstack.org/604041 | 15:39 |
| *** ccamposr has quit IRC | 15:45 | |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Update kuryr-controller ClusterRole RBAC https://review.openstack.org/607994 | 15:46 |
| *** jchhatbar has quit IRC | 15:46 | |
| dulek | yboaron: ^ | 15:46 |
| yboaron | dulek, 10x | 15:47 |
| dulek | yboaron: Do we have Routes support in stable/rocky? | 15:47 |
| yboaron | dulek, yes | 15:47 |
| dulek | yboaron: So I'll better split this to make backporting easier. Just a second. | 15:48 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: Add experimental gate for Routes kuryr containerized https://review.openstack.org/608010 | 15:50 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Add routes and multi-vif resources to Kuryr's CR https://review.openstack.org/608011 | 15:54 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Add NetworkPolicies resources to kuryr's CR https://review.openstack.org/608012 | 15:54 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: [WIP] Support network policy update https://review.openstack.org/605150 | 15:54 |
| *** yboaron has quit IRC | 16:02 | |
| *** aperevalov has quit IRC | 16:26 | |
| *** maysams has quit IRC | 16:33 | |
| *** garyloug__ has quit IRC | 16:46 | |
| *** maysams has joined #openstack-kuryr | 16:48 | |
| *** gkadam has quit IRC | 17:08 | |
| *** pcaruana has joined #openstack-kuryr | 17:17 | |
| *** pcaruana has quit IRC | 18:01 | |
| *** aojea has joined #openstack-kuryr | 18:36 | |
| *** maysams has quit IRC | 19:03 | |
| *** maysams has joined #openstack-kuryr | 20:23 | |
| *** celebdor has quit IRC | 20:46 | |
| *** aojea has quit IRC | 22:46 | |
| *** phuoc_ has quit IRC | 23:26 | |
| *** phuoc_ has joined #openstack-kuryr | 23:27 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!