| *** hongbin has joined #openstack-kuryr | 02:41 | |
| *** rh-jelabarre has quit IRC | 03:08 | |
| *** janki has joined #openstack-kuryr | 04:46 | |
| *** celebdor has joined #openstack-kuryr | 05:57 | |
| *** hongbin has quit IRC | 06:32 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-tempest-plugin master: Add exclusive run facility https://review.openstack.org/623408 | 07:07 |
|---|---|---|
| *** JJ-Club has joined #openstack-kuryr | 07:24 | |
| JJ-Club | when I was creating pods, ERROR kuryr_kubernetes.handlers.logging NoMatches: No 'kuryr_kubernetes.vif_translators' driver found, looking for u'binding_failed' , how to fix it | 07:31 |
| *** dims has quit IRC | 07:44 | |
| *** dims has joined #openstack-kuryr | 07:47 | |
| *** maysams has joined #openstack-kuryr | 08:27 | |
| dulek | JJ-Club: Can you paste whole log? This seems like Neutron has issues with port creation/binding? | 08:55 |
| dulek | JJ-Club: By paste I mean paste.openstack.org. ;) | 08:55 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Enhance svc namespace isolation with LoadBalancer type svc https://review.openstack.org/618751 | 08:56 |
| *** garyloug_ has joined #openstack-kuryr | 09:40 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-tempest-plugin master: Add exclusive run facility https://review.openstack.org/623408 | 09:45 |
| dulek | celebdor: Regarding ^. Tempest has oslo.concurrency in the requirements.txt anyway, so I've advised gcheresh to just use lockutils from there. | 10:38 |
| celebdor | dulek: what is the advantage of that? | 10:38 |
| dulek | celebdor: Tested and proved implementation? | 10:39 |
| dulek | celebdor: And won't your implementation use 100% CPU because of lack of any delays between lock acquiring tries? | 10:40 |
| celebdor | dulek: I don't see any decorator in oslo lockutils | 10:40 |
| celebdor | which retries? | 10:41 |
| dulek | celebdor: Oh, you're not looping. Sorry. :P | 10:43 |
| dulek | celebdor: @lockutils.synchronized | 10:44 |
| celebdor | if I don't want timeouts I don't see why I should do non-block and loop with sleep | 10:44 |
| dulek | celebdor: Naaah, this is fine, I've got confused a bit. | 10:45 |
| celebdor | no worries. I was not aware of those oslo facilities | 10:46 |
| celebdor | I may be biased against them | 10:46 |
| celebdor | so I appreciate the outside look | 10:46 |
| dulek | celebdor: It doesn't matter too much in tempest plugin, which needs oslo.concurrency anyway. | 10:46 |
| celebdor | dulek: I'm fine with using this @lockutils.synchronized | 10:47 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Remove non-containerized/daemon gates https://review.openstack.org/623444 | 10:50 |
| dulek | dmellado: Whoa, that was quick, when did we discussed this? ^ | 10:53 |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-tempest-plugin master: Synchronize port pool tests https://review.openstack.org/623408 | 10:57 |
| celebdor | dulek: ^^ | 10:58 |
| celebdor | note that I had to add oslo.concurrency as it was not part of requirements yet | 10:59 |
| dulek | celebdor: Yes, but it's tempest requirement, so plugins have it anyway. | 10:59 |
| dulek | But yup, good to add it to ours as well. | 10:59 |
| celebdor | dulek: but if we use it, we have to put it explicitly | 11:00 |
| dulek | celebdor: Sure, sure. | 11:00 |
| celebdor | ltomasbo: dmellado: dulek: https://review.openstack.org/#/c/621597/ before I need to rebase.. | 11:00 |
| dulek | celebdor: http://logs.openstack.org/97/621597/2/check/kuryr-kubernetes-tempest-daemon-openshift-octavia/a23a18e/job-output.txt.gz#_2018-12-04_16_38_22_160202 | 11:02 |
| dulek | celebdor: This still breaks OpenShift gate. | 11:02 |
| celebdor | ok, thanks | 11:05 |
| celebdor | I'll check later | 11:05 |
| *** danil has quit IRC | 11:10 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Use /var/run instead of /var/run/openvswitch https://review.openstack.org/602621 | 11:13 |
| *** celebdor has quit IRC | 11:23 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Fix labels encode https://review.openstack.org/622965 | 11:31 |
| dmellado | celebdor_: ok diario? | 11:33 |
| dmellado | hey dulek | 11:40 |
| dmellado | didn't see your ping | 11:40 |
| dulek | :) | 11:40 |
| dmellado | so before you complain about the changes | 11:40 |
| dmellado | two things | 11:40 |
| dulek | dmellado: I've already did a bit in the review. ;) | 11:40 |
| dmellado | 1st of all, I'm fine keeping some of the stuff there but I'd like to change for example centos7 version to containerized as well, asap | 11:41 |
| dmellado | 2nd, sow your issue on the another patch with the lower-contraints | 11:41 |
| dmellado | that doesn't apply to tempest plugins | 11:41 |
| dulek | dmellado: I'm okay with such change, sure. | 11:42 |
| dulek | dmellado: I haven't added lower-constraints to Tempest plugins, I think. | 11:42 |
| dulek | dmellado: We need to do another approach here if we want to drop non-containerized from gate. | 11:42 |
| dmellado | maybe you changed it after I reviewed, R-view is fine but doesn't suit long reviews xD | 11:42 |
| dulek | dmellado: First of all we don't advertise that as recommended way of running Kuryr. | 11:42 |
| dmellado | that needs to be changed, deff | 11:42 |
| dulek | dmellado: So we should start to do that. | 11:43 |
| dmellado | +1 | 11:43 |
| dmellado | then after that we'll make the changes, and yeah, I'd keep at least lbaas for now | 11:43 |
| dulek | dmellado: Second - this isn't just adding a new kuryr-daemon service. This is completly changing how you run it. | 11:43 |
| dulek | dmellado: We need some feedback about how people use Kuryr. We need a post to openstack-discuss, feedback from irenab, guys from Samsung. | 11:44 |
| dmellado | well, we already had a lot of containerized/daemonized gates, so it's not like is new | 11:44 |
| dmellado | but it'll take some time | 11:44 |
| dmellado | actually I'll discuss that on the Mon meeting | 11:44 |
| dmellado | alongside with the os-vif stuff | 11:44 |
| dulek | Sure, but it's an option, so operators may not be aware of that. | 11:44 |
| dulek | dmellado: Sure, meeting's designed for such discussions. | 11:44 |
| dmellado | so I'll hold on the patch until then | 11:45 |
| dulek | dmellado: Sure. | 11:45 |
| *** aperevalov has joined #openstack-kuryr | 11:45 | |
| dmellado | dulek: just make sure the 'new party' there don't get you before | 11:45 |
| dmellado | xD | 11:45 |
| dulek | dmellado: Oh come on, I've lived in Poland all my life, I know how to avoid conflicts with right-wing. ;) | 11:47 |
| dmellado | well, you used to have 'twin wings' | 11:47 |
| dmellado | now maybe there's a 'right one' xD | 11:47 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix -D option on build_cni__daemonset_image https://review.openstack.org/623462 | 11:55 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix -D option on build_cni_daemonset_image https://review.openstack.org/623462 | 11:55 |
| dulek | Fixup for the HA gate. ^ | 11:56 |
| *** aperevalov has quit IRC | 12:03 | |
| *** celebdor has joined #openstack-kuryr | 12:06 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Add experimental gate with cri-o https://review.openstack.org/620388 | 12:13 |
| *** garyloug_ has quit IRC | 12:18 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: devstack: move master config to separate dir https://review.openstack.org/621597 | 12:26 |
| *** rh-jelabarre has joined #openstack-kuryr | 12:39 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Allocate service FIP after LB was provisioned https://review.openstack.org/622257 | 12:46 |
| *** janki has quit IRC | 13:24 | |
| celebdor | dulek: https://twitter.com/runc0m/status/1070976898401034241 | 13:42 |
| dulek | celebdor: I think it'll get tested once it's in the PPA. | 13:43 |
| dulek | celebdor: They have it automated so it should happen soon. | 13:44 |
| celebdor | cool | 13:46 |
| *** celebdor has quit IRC | 13:56 | |
| *** celebdor has joined #openstack-kuryr | 13:58 | |
| *** celebdor has quit IRC | 14:00 | |
| *** celebdor has joined #openstack-kuryr | 14:00 | |
| celebdor | ltomasbo: https://review.openstack.org/#/c/623408/ | 14:03 |
| celebdor | dmellado: https://review.openstack.org/#/c/623408/ | 14:03 |
| dmellado | I'll take a look | 14:06 |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-tempest-plugin master: Testing port_pool_min and port_pool_max https://review.openstack.org/620060 | 14:41 |
| *** takamatsu has joined #openstack-kuryr | 14:43 | |
| openstackgerrit | Alexey Perevalov proposed openstack/kuryr-kubernetes master: Add DPDK support for nested pods https://review.openstack.org/559363 | 15:11 |
| *** maysams has quit IRC | 15:36 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: Amend the spelling error of a word https://review.openstack.org/572710 | 15:36 |
| celebdor | dmellado: what do you think of https://review.openstack.org/#/c/585866/ should I close for lack of activity or you want to salvage some of it | 15:39 |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Synchronize port pool tests https://review.openstack.org/623408 | 15:44 |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: spelling error https://review.openstack.org/619861 | 15:47 |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: fix a typo in docstring https://review.openstack.org/617140 | 15:48 |
| *** janki has joined #openstack-kuryr | 16:00 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Fix -D option on build_cni_daemonset_image https://review.openstack.org/623462 | 16:02 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: [Trivialfix]Modify a grammatical error https://review.openstack.org/549979 | 16:02 |
| *** celebdor has quit IRC | 16:18 | |
| *** celebdor has joined #openstack-kuryr | 16:20 | |
| *** hsnuonly has joined #openstack-kuryr | 16:27 | |
| hsnuonly | Hi, I saw @JJ-Club mentioned that he has an issue of "No 'kuryr_kubernetes.vif_translators' driver found" | 16:29 |
| hsnuonly | I just want to say that had the same issue on CentOS 7 with devstack+kubeadm | 16:30 |
| hsnuonly | it seems to be a common issue, do you have any idea how to fix it? | 16:31 |
| *** hsnuonly has quit IRC | 16:39 | |
| dmellado | @celebdor close it | 16:48 |
| *** sean-k-mooney has quit IRC | 16:59 | |
| *** sean-k-mooney has joined #openstack-kuryr | 17:05 | |
| *** dims has quit IRC | 17:21 | |
| *** janki has quit IRC | 17:33 | |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Move job list to project-template https://review.openstack.org/623183 | 17:43 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add support for podSelector https://review.openstack.org/623554 | 17:44 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Set kuryr_daemon_enabled in tempest.conf correctly https://review.openstack.org/622928 | 17:44 |
| *** dims has joined #openstack-kuryr | 18:50 | |
| *** JJ-Club has quit IRC | 19:27 | |
| *** sean-k-mooney has quit IRC | 20:20 | |
| *** sean-k-mooney has joined #openstack-kuryr | 20:28 | |
| *** celebdor has quit IRC | 21:54 | |
| *** dmellado has quit IRC | 22:22 | |
| *** EricAdamsZNC2 has quit IRC | 22:27 | |
| *** EricAdamsZNC has joined #openstack-kuryr | 22:27 | |
| *** dmellado has joined #openstack-kuryr | 23:11 | |
| *** celebdor has joined #openstack-kuryr | 23:34 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!