Thursday, 2018-03-15

*** yamamoto has joined #openstack-kuryr00:07
*** c00281451 has joined #openstack-kuryr01:41
*** c00281451 is now known as zengchen01:41
*** hongbin has joined #openstack-kuryr01:46
*** salv-orl_ has joined #openstack-kuryr02:04
*** salv-orlando has quit IRC02:07
openstackgerritEunsoo Park proposed openstack/kuryr-kubernetes master: Watcher restarts watching resources in failure  https://review.openstack.org/52983202:22
openstackgerritEunsoo Park proposed openstack/kuryr-kubernetes master: Make Neutron LBaaS Activation Timeout configurable  https://review.openstack.org/54994502:24
openstackgerritEunsoo Park proposed openstack/kuryr-kubernetes master: Make Neutron LBaaS Activation Timeout configurable  https://review.openstack.org/54994502:26
*** wangbo has joined #openstack-kuryr02:45
*** janki has joined #openstack-kuryr03:34
*** maysamacedos has quit IRC03:43
*** hongbin has quit IRC03:52
*** janonymous has joined #openstack-kuryr04:08
*** wangbo has quit IRC04:38
*** wangbo has joined #openstack-kuryr05:15
*** wangbo has quit IRC06:02
*** wangbo has joined #openstack-kuryr06:03
*** yboaron has joined #openstack-kuryr06:04
irenabkaliya, regarding https://bugs.launchpad.net/kuryr-kubernetes/+bug/1708173, can be useful06:13
openstackLaunchpad bug 1708173 in kuryr-kubernetes "Update Vagrant to fit Octavia used for LBaaS requirements" [Low,Triaged]06:13
*** wangbo has left #openstack-kuryr06:39
*** premsankar has quit IRC06:51
*** gcheresh_ has joined #openstack-kuryr07:05
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr master: Updated from global requirements  https://review.openstack.org/54953707:05
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes master: Updated from global requirements  https://review.openstack.org/54925307:06
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork master: Updated from global requirements  https://review.openstack.org/54953807:06
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-tempest-plugin master: Updated from global requirements  https://review.openstack.org/55149707:06
*** pcaruana has joined #openstack-kuryr07:48
*** pcaruana has quit IRC07:50
*** pcaruana has joined #openstack-kuryr07:50
*** pcaruana is now known as pcaruana|wfh|07:53
*** yboaron has quit IRC08:17
openstackgerritGenadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Adding pools and containerized jobs to .zuul  https://review.openstack.org/55251208:53
*** ispp is now known as burned09:04
*** celebdor has joined #openstack-kuryr09:22
*** janki has quit IRC09:28
dulekAnyone have a clue why gates fail with POST_FAILURE? Logs look pretty much okay.09:33
dulekOh wait, I see, post fails when copying the logs - I guess.09:33
*** yboaron has joined #openstack-kuryr09:35
dulekOkay, looks like infra folks are still working on fixing this, meanwhile rechecking the patches doesn't make a lot of sense.09:35
celebdorthanks for the info dulek09:44
*** garyloug has joined #openstack-kuryr09:44
dulek"yes, see my discussion with tobiash earlier. reason not clear yet, we will probably need to wait for some US people to look into it" :(10:01
dmelladodulek: post-failure seems to be due to infra side issues10:22
dmelladoseems that we can't get to keep the infra working for more than 1 day10:22
dmelladodamn10:22
dulekdmellado: Yup, it's affecting all DevStack jobs.10:22
openstackgerritYossi Boaron proposed openstack/kuryr-kubernetes master: Services: Add support for K8S service's port edit use case  https://review.openstack.org/55177310:35
*** garyloug_ has joined #openstack-kuryr10:42
*** garyloug_ has quit IRC10:42
openstackgerritYossi Boaron proposed openstack/kuryr-kubernetes master: Services: Add support for K8S service's port edit use case  https://review.openstack.org/55177310:42
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Split .zuul.yaml into smaller files  https://review.openstack.org/54830910:48
*** yamamoto has quit IRC11:04
*** snapiri- has joined #openstack-kuryr11:18
*** caowei_ has joined #openstack-kuryr11:19
dmelladodulek: https://review.openstack.org/#/c/553329/11:21
*** pcaruana|wfh| has quit IRC11:27
*** caowei has quit IRC11:27
*** snapiri has quit IRC11:27
*** kzaitsev_pi has quit IRC11:27
*** caowei_ is now known as caowei11:27
*** pcaruana|wfh| has joined #openstack-kuryr11:28
*** kzaitsev1pi has joined #openstack-kuryr11:28
*** alraddarla has joined #openstack-kuryr11:32
*** dmellado has quit IRC11:50
*** dmellado has joined #openstack-kuryr11:58
*** yamamoto has joined #openstack-kuryr12:04
*** dmellado has quit IRC12:05
*** alraddarla has quit IRC12:08
*** yamamoto has quit IRC12:09
*** pcaruana|wfh| is now known as pcaruana12:14
*** maysamacedos has joined #openstack-kuryr12:15
*** yamamoto has joined #openstack-kuryr12:16
*** maysamacedos has quit IRC12:17
*** dmellado has joined #openstack-kuryr12:23
*** alraddarla_ has joined #openstack-kuryr12:24
*** snapiri- is now known as snapiri12:27
*** snapiri has quit IRC12:27
*** snapiri has joined #openstack-kuryr12:28
snapiridmellado: Isn't there a way around it? Some definition one can add to the job?12:30
*** dmellado has quit IRC12:30
*** snapiri has quit IRC12:41
*** snapiri has joined #openstack-kuryr12:41
*** garyloug has quit IRC12:46
duleksnapiri: Not really, it's a change in ansible: https://review.openstack.org/#/c/553329/1/roles/devstack-project-conf/tasks/main.yaml12:49
snapiridulek: thanks. So we will have to wait for the infra guys to merge their rollback :)12:51
*** janonymous has quit IRC13:02
*** atoth has joined #openstack-kuryr13:06
*** maysamacedos has joined #openstack-kuryr13:07
*** yamamoto has quit IRC13:18
*** yamamoto has joined #openstack-kuryr13:19
*** dmellado has joined #openstack-kuryr13:19
celebdoryboaron: regarding https://review.openstack.org/#/c/547822/13:20
*** dmellado has quit IRC13:20
celebdorI'm not a fan of factories and this approach seems a bit restrictive13:20
*** garyloug has joined #openstack-kuryr13:20
celebdoryboaron: dulek: I'm thinking about merging it to get the functionality in and then changing it so that it works similarly to neutron service plugins13:21
yboaroncelebdor, so what do you suggest? alternative ?13:21
celebdorbased on entrypoints13:21
irenabcelebdor, merging what?13:21
celebdormerging yboaron's patch13:21
celebdorthat introduces the factory13:21
yboaronirenab, https://review.openstack.org/#/c/547822/13:21
celebdorand then swapping that part out13:21
dulekcelebdor: You mean using stevedore. I'm fine with that.13:22
celebdorso that you can just put in config comma separated entrypoints13:22
yboaronirenab, add support for Openshift13:22
dulekcelebdor: That would make it extendable.13:22
celebdor(which can be declared by other python libraries)13:22
celebdordulek: that's exactly what I'm saying13:22
irenabyboaron, taking a look13:22
celebdorthe only missing piece in my plan is to have pre-defined collections13:22
celebdorof entrypoints13:22
celebdorbut I'm sure it can be done somehow13:22
irenabyboaron, celebdor so this patch is just the framework?13:23
*** yamamoto has quit IRC13:23
yboaronyes13:23
irenabThis should be properly defined since itg defines a contract13:24
irenabcelebdor, let me take a look. I prefer not to merge fundamentals when  there is an agreement to change it13:25
celebdorirenab: so you would advocate making a devref and going for the approach I'm saying directly?13:25
irenabgive me a sec, missed this patch before13:25
celebdorok13:25
irenabbut as a general say, I would probably advocate for devref and what you say (you know me pretty well :-))13:26
celebdoralrighty then13:27
celebdoryboaron: let's sync on this after the meeting13:28
yboaroncelebdor, OK13:29
*** snapiri has quit IRC13:29
irenabcelebdor, this looks a bit similar to the direction we discussed about a year ago with Ilya, the profiles13:31
*** snapiri has joined #openstack-kuryr13:32
irenabthen it was more focused on 'south' side (the drivers), but probably should be also for the 'north' (the watchers)13:33
*** janki has joined #openstack-kuryr13:52
*** natanbro has joined #openstack-kuryr14:00
*** neiljerram has joined #openstack-kuryr14:05
*** kiennt26 has joined #openstack-kuryr14:07
*** natanbro has quit IRC14:11
*** yamamoto has joined #openstack-kuryr14:20
*** yamamoto has quit IRC14:25
celebdorirenab: I think since yossi's patch is not changing the fact that they are hardcoded14:26
celebdorwe can merge it, and then I'll make a patch with the devref and the stevedore approach14:27
irenabcelebdor, I am not sure we need platform abstraction14:27
irenabin the code14:28
irenabthis is just template/profile to deploy which mixture of watchers and drivers14:28
irenabDo you think platform is needed?14:29
celebdoryou know me irenab14:35
celebdorirenab: personally I'd just have done a if get_platform == OSP: register_watchers(osp_watchers)14:36
celebdorI despise factories and inheritance14:36
irenabcelebdor, I just not sure Platform is something that should exist in kuryr code, it is more of a deployment environment that kuryr is used in14:38
celebdorirenab: I agree with you14:38
celebdorthat's why I propose to have the handlers config variable14:39
celebdorthat in deployment time you'll just put your handlers comma separated there14:39
irenabyes, this is more aligned with the direction, agreed14:39
celebdorI would only merge it to allow the osp and k8s handlers in the meantime14:39
celebdorbut I would scrap it14:39
celebdorafter we have it deployment config ready14:40
celebdormaybe I'd add default k8s and osp entrypoints that encompass the other ones that they need14:40
irenabMaybe just removing the Factory and follow more lightweight approach you suggest?14:40
celebdorbut that's about it14:40
*** gcheresh has joined #openstack-kuryr14:42
irenaband going forward, we really need to do some steps towards the profiles, since number of options start to grow14:42
irenabcelebdor, entrypoints for watchers?14:43
*** gcheresh_ has quit IRC14:43
celebdoryes, entrypoints for watchers14:44
celebdoryou'll have a config like:14:44
celebdorpipeline_watchers = vif,service,ingress14:45
celebdorfor example14:45
*** dmellado_ has joined #openstack-kuryr14:46
*** phuoc_ has joined #openstack-kuryr14:47
irenabcelebdor, I agree. I prefer kuryr code to be less 'smart' with regards to what environment it lives in14:49
irenabso the more can be provided during the deployment time,  the better14:49
*** phuoc has quit IRC14:51
*** phuoc has joined #openstack-kuryr14:52
*** phuoc_ has quit IRC14:52
*** dmellado_ is now known as dmellado14:54
dmellado+114:56
dmelladomaybe we can just add that as an option in kuryr.conf and have the user handle it14:56
*** gcheresh has quit IRC14:58
*** gcheresh has joined #openstack-kuryr14:58
irenabdmellado, yes, this is the idea. The cluster manager will provide the config based on deployment environment15:01
irenabcelebdor, dmellado need to  go now, ping me if some follow-up is required15:02
irenabdmellado, enjoy your time in BCN15:03
*** gcheresh has quit IRC15:09
celebdorirenab: he already left15:09
celebdorto the center to shop and then to the train15:09
*** yamamoto has joined #openstack-kuryr15:21
*** maysamacedos has quit IRC15:27
*** yamamoto has quit IRC15:27
yboaroncelebdor, another point for having configurable watchers is integration with Istio, where only the pod and NetPolicy will be required in that case15:32
*** garyloug_ has joined #openstack-kuryr15:36
*** maysamacedos has joined #openstack-kuryr15:38
*** garyloug has quit IRC15:38
*** kiennt26 has quit IRC15:44
celebdoralright. I'll draft something up then15:45
*** maysamacedos has quit IRC15:56
openstackgerritMichaƂ Dulko proposed openstack/kuryr-kubernetes master: WiP: Preview of CNI daemon side VIF choice  https://review.openstack.org/52724315:58
dulekcelebdor: ^ This should be rebased and was tested. I'd love to get some feedback on how this works on the lowest level - CNI binding and stuff. Please note that only bridge drivers are currently done.15:59
dulekcelebdor: And I haven't incorporated the switch from marking VIFs reservation in pods instead of CRDs yet.16:00
* dulek needs to go now.16:02
celebdordulek: will check tomorrow16:05
celebdorthanks16:05
dulekcelebdor: I'd be grateful. Meanwhile I'll take a Friday break from this and will work on other stuff from the PTG. ;)16:06
celebdordulek: like?16:07
yboaroncelebdor, we plan to allow configuring not just the resources to watch, but also the handler functions right ?16:17
celebdorno, jus tthe handlers16:18
celebdorjust16:18
celebdorthe handlers decide what they watch16:19
yboaronOK, right!16:20
*** yamamoto has joined #openstack-kuryr16:23
*** hongbin has joined #openstack-kuryr16:25
*** yamamoto has quit IRC16:29
*** garyloug_ has quit IRC16:35
*** yboaron has quit IRC16:38
*** janki has quit IRC16:46
*** garyloug_ has joined #openstack-kuryr16:52
*** celebdor has quit IRC16:56
*** yamamoto has joined #openstack-kuryr17:25
*** yamamoto has quit IRC17:30
*** maysamacedos has joined #openstack-kuryr17:32
*** garyloug_ has quit IRC17:42
*** yamamoto has joined #openstack-kuryr18:27
*** yamamoto has quit IRC18:33
*** hongbin has quit IRC19:25
*** hongbin has joined #openstack-kuryr19:26
*** yamamoto has joined #openstack-kuryr19:29
*** yamamoto has quit IRC19:33
*** yamamoto has joined #openstack-kuryr20:29
*** yamamoto has quit IRC20:35
*** gcheresh has joined #openstack-kuryr20:53
*** s1061123 has quit IRC20:58
*** s1061123 has joined #openstack-kuryr21:00
*** mburrows has quit IRC21:21
*** gcheresh has quit IRC21:26
*** yamamoto has joined #openstack-kuryr21:31
*** yamamoto has quit IRC21:36
*** celebdor has joined #openstack-kuryr21:59
*** celebdor has quit IRC22:19
*** yamamoto has joined #openstack-kuryr22:32
*** yamamoto has quit IRC22:38
*** hongbin has quit IRC22:50
*** salv-orl_ has quit IRC22:53
*** salv-orlando has joined #openstack-kuryr22:54
*** salv-orlando has quit IRC22:59
*** celebdor has joined #openstack-kuryr23:04
*** yamamoto has joined #openstack-kuryr23:27
*** salv-orlando has joined #openstack-kuryr23:54
*** salv-orlando has quit IRC23:59

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