Monday, 2018-11-19

*** janki has joined #openstack-kuryr04:24
*** aperevalov has joined #openstack-kuryr06:57
*** ccamposr has joined #openstack-kuryr07:08
*** pcaruana has joined #openstack-kuryr07:24
*** maysams has joined #openstack-kuryr07:42
*** gkadam has joined #openstack-kuryr07:58
*** yboaron has quit IRC08:27
*** garyloug has joined #openstack-kuryr08:58
*** yboaron has joined #openstack-kuryr09:09
*** celebdor has joined #openstack-kuryr09:23
*** f0o has joined #openstack-kuryr09:55
openstackgerritMerged openstack/kuryr-tempest-plugin master: Add timeout parameter to 'connect_get_namespaced_pod_exec'  https://review.openstack.org/61863510:33
*** janki has quit IRC11:18
*** janki has joined #openstack-kuryr11:49
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: devstack: use https to access the k8s cluster  https://review.openstack.org/61755212:33
*** rh-jelabarre has joined #openstack-kuryr12:45
openstackgerritGenadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Testing that LB is created/deleted for created/deleted service  https://review.openstack.org/61389913:03
*** yboaron has quit IRC13:26
*** yboaron has joined #openstack-kuryr13:28
*** aojea_ has joined #openstack-kuryr13:29
*** aojea_ has quit IRC13:30
*** aojea_ has joined #openstack-kuryr13:31
*** aojea_ has quit IRC13:35
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Enhance svc namespace isolation with LoadBalancer type svc  https://review.openstack.org/61875113:42
*** aojea_ has joined #openstack-kuryr13:58
*** yboaron_ has joined #openstack-kuryr14:01
*** yboaron has quit IRC14:04
*** aperevalov_ has joined #openstack-kuryr14:10
*** aperevalov has quit IRC14:11
openstackgerritMerged openstack/kuryr-kubernetes master: Remove hooks directory  https://review.openstack.org/61763314:11
*** aojea_ has quit IRC14:31
aperevalov_please review: https://review.openstack.org/#/c/616504/ it add forgotten unit test into sriov binding driver.14:31
*** aojea_ has joined #openstack-kuryr14:32
*** aojea_ has quit IRC14:36
*** yboaron_ has quit IRC14:58
*** yboaron_ has joined #openstack-kuryr15:04
*** gkadam has quit IRC15:31
*** celebdor has quit IRC16:01
*** pcaruana has quit IRC16:07
dulekmaysams: Hey, any reason you're probing the API directly instead of listing the crds in https://review.openstack.org/#/c/617557 ?16:15
* dulek is catching up with reviews.16:15
dulek(at least trying to :P)16:15
dulekSeems like /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions can return a CustomResourceDefinitionList.16:16
maysamsdulek: oh, this way is better16:21
maysamsthe one you suggested, I mean16:22
dulekmaysams: Yup, just one call and you can also check if the CRD definition is okay if you want.16:23
maysamsdulek: yes, thank you :-)16:23
dulekmaysams: I was just wondering if there isn't some hidden caveat that someone pointed out earlier in the review process.16:23
maysamsno16:23
dulekmaysams: But if not it may be worth to update that code. :)16:23
maysamsdulek: nobody pointed something regarding this. I will update it later16:25
maysamsThank you :-)16:26
*** janki has quit IRC16:26
dulekmaysams: Hm, there is one thing though… Listing CRDs will require new privilege for kuryr ServiceAccount.16:26
maysamsdulek: I believe is okay to update it, no?16:27
dulekmaysams: That's the place: https://github.com/openstack/kuryr-kubernetes/blob/b0b170f7a8b79a92d0bcf5b911f6d5a1b89a8f3b/devstack/lib/kuryr_kubernetes#L455-L49216:27
dulekmaysams: I think that if we add list access it will be fine.16:28
maysamsdulek: cool16:28
maysamsI will update it16:28
dulekmaysams: Okay, thanks!16:28
maysamsdulek: Thank you!16:28
*** celebdor has joined #openstack-kuryr16:45
*** yboaron_ has quit IRC16:52
*** maysams has quit IRC17:06
*** ccamposr has quit IRC17:08
*** garyloug has quit IRC17:22
*** aperevalov_ has quit IRC17:29
*** pcaruana has joined #openstack-kuryr17:44
*** celebdor has quit IRC17:57
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Revert "Make LBaaS v2 gates non-voting"  https://review.openstack.org/61879518:35
*** premsankar has joined #openstack-kuryr19:18
*** phuoc has quit IRC19:36
*** gianpietro has joined #openstack-kuryr19:54
*** aojea has joined #openstack-kuryr19:56
*** aojea has quit IRC20:01
*** gianpietro has quit IRC20:19
*** gianpietro has joined #openstack-kuryr20:19
*** gianpietro has quit IRC20:22
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: [WIP] Add quota readiness check to controller  https://review.openstack.org/61865520:22
*** gianpietro has joined #openstack-kuryr20:22
*** maysams has joined #openstack-kuryr20:23
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: [WIP] Add quota readiness check to controller  https://review.openstack.org/61865520:29
*** aojea has joined #openstack-kuryr20:32
*** pcaruana has quit IRC21:51
*** gianpietro has quit IRC21:52
*** gianpietro has joined #openstack-kuryr21:53
openstackgerritMerged openstack/kuryr master: Add kuryr-status upgrade check command framework  https://review.openstack.org/61266722:07
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: Add quota readiness check to controller  https://review.openstack.org/61865522:49
*** maysams has quit IRC22:59
*** rh-jelabarre has quit IRC23:32
*** aojea has quit IRC23:39
*** gianpietro has quit IRC23:39
*** gianpietro has joined #openstack-kuryr23:40
*** gianpietro has quit IRC23:45

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