| *** maysamacedos has joined #openstack-kuryr | 01:02 | |
| *** hongbin has joined #openstack-kuryr | 01:28 | |
| *** maysamacedos has quit IRC | 02:54 | |
| *** hongbin has quit IRC | 03:11 | |
| *** hongbin has joined #openstack-kuryr | 03:37 | |
| *** hongbin has quit IRC | 04:24 | |
| *** janki has joined #openstack-kuryr | 05:24 | |
| *** threestrands has quit IRC | 06:19 | |
| *** gcheresh has joined #openstack-kuryr | 06:28 | |
| *** dmellado has joined #openstack-kuryr | 06:29 | |
| *** yboaron has joined #openstack-kuryr | 06:31 | |
| *** yboaron has quit IRC | 06:38 | |
| *** yboaron has joined #openstack-kuryr | 06:38 | |
| *** pcaruana has joined #openstack-kuryr | 06:44 | |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Remove redundant line from dragonflow config https://review.openstack.org/577656 | 06:50 |
|---|---|---|
| snapiri | dmellado: good morning :) | 07:00 |
| dmellado | morning shachar! | 07:01 |
| snapiri | dmellado: any plans to handle https://review.openstack.org/#/c/576486/ ? | 07:01 |
| snapiri | I want to push the splitting of the zuul files that is based on that one | 07:01 |
| dmellado | snapiri: we've been having a rough time | 07:01 |
| dmellado | with the gates | 07:01 |
| snapiri | I know | 07:01 |
| dmellado | as of now I even had to go and drop the lbaas tests | 07:02 |
| dmellado | so for now I'm waiting to https://review.openstack.org/#/c/577760/ | 07:02 |
| dmellado | to get merged | 07:02 |
| dmellado | then I'll do that | 07:02 |
| *** pcaruana has quit IRC | 07:02 | |
| dmellado | I'll recheck that patch after that | 07:03 |
| snapiri | dmellado: I was following it through the past two days. I saw the gates broke for various reasons. | 07:03 |
| dmellado | after trying to see what's going on with the lbaas | 07:03 |
| snapiri | Please ping me when you think I can rework the splitting patch so I can push it forward | 07:03 |
| dmellado | snapiri: sure, hopefully today ;) | 07:03 |
| * snapiri keeps his fingers crossed | 07:04 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-kubernetes master: Services LBaaS driver: fix ensure LB for existing LB. https://review.openstack.org/578298 | 07:20 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Explicitly set threaded=False when starting Flask https://review.openstack.org/577760 | 07:32 |
| *** phuoc has quit IRC | 07:48 | |
| *** phuoc has joined #openstack-kuryr | 07:49 | |
| *** pcaruana has joined #openstack-kuryr | 07:57 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-tempest-plugin master: Restore service tests https://review.openstack.org/578304 | 08:00 |
| dmellado | yboaron: ^^ | 08:01 |
| dmellado | let's see how does that behave | 08:01 |
| yboaron | dmellado, OK | 08:02 |
| yboaron | dmellado, 10x! | 08:02 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support OCP routes https://review.openstack.org/578310 | 08:13 |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Remove redundant line from dragonflow config https://review.openstack.org/577656 | 08:16 |
| *** garyloug has joined #openstack-kuryr | 08:26 | |
| ltomasbo | dmellado, why https://review.openstack.org/#/c/578304/ instead of reverting the patch? | 08:43 |
| openstackgerrit | Genadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Test restart of CNI and Controller kuryr pods https://review.openstack.org/552498 | 08:50 |
| dmellado | ltomasbo: because I'd have to revert 2 patches | 08:51 |
| dmellado | another one for another class that we left out | 08:51 |
| dmellado | https://review.openstack.org/#/c/578195/ | 08:51 |
| ltomasbo | dmellado, and it is not time to revert both? | 08:56 |
| dmellado | ltomasbo: nope, let's do it this way as it allows us | 09:02 |
| dmellado | to test patches against it before doing anything | 09:02 |
| dmellado | I'm seeing some odd things on yboaron queue | 09:03 |
| dmellado | ... | 09:03 |
| dmellado | so looks like the issue it's not really sorted out | 09:03 |
| yboaron | dmellado, what's Q? | 09:03 |
| yboaron | what Q | 09:03 |
| yboaron | dmellado, the tempest plugin one ? | 09:04 |
| dmellado | yboaron: zuul queue | 09:04 |
| dmellado | look for your patch on zuul.openstack.org | 09:04 |
| dmellado | seems like daemon gates are being affected | 09:05 |
| dmellado | :\ | 09:05 |
| ltomasbo | seem normal to me, I cannot even deploy a devstack env with daemon + octavia... | 09:09 |
| ltomasbo | so, we have something broken, not just the gates | 09:09 |
| dulek | I think I've found the reason for healthchecks failure. I'm testing my hypothesis at the moment. | 09:10 |
| dulek | (most likely it's due to Flask upgrade) | 09:10 |
| dulek | dmellado: Wasn't yboaron's patch tested before 577760 was merged? | 09:12 |
| dulek | dmellado: That would fit daemon gates being affected. | 09:12 |
| dulek | There are just too many bugs at the moment. | 09:12 |
| dulek | We have 3 of them, all affecting the gate. :) | 09:12 |
| dmellado | dulek: let's hit recheck, but 2nd PS should have been after it | 09:12 |
| dmellado | in any case for yboaron patch to succeed | 09:12 |
| dmellado | I also need to send another one to requirements | 09:13 |
| dulek | dmellado: Ah, you're right,. | 09:13 |
| dmellado | adding openshift | 09:13 |
| dmellado | https://review.openstack.org/#/c/577656/ | 09:15 |
| dmellado | we also have miracles | 09:15 |
| dmellado | xD | 09:15 |
| dmellado | dulek: https://review.openstack.org/#/c/578304/ | 09:16 |
| dmellado | in any case | 09:16 |
| dmellado | it seems that there's something odd with lbaasv2 as well | 09:17 |
| dmellado | even w/o daemon | 09:17 |
| dmellado | http://logs.openstack.org/04/578304/1/check/kuryr-kubernetes-tempest-openshift-lbaasv2/b78cba9/controller/logs/screen-kuryr-kubernetes.txt.gz#_Jun_27_08_46_30_843509 | 09:18 |
| dmellado | seems to be related to lbaas timeout | 09:18 |
| dulek | dmellado: Please don't exceed my prescribed daily dose of gate bugs. | 09:20 |
| dmellado | lol | 09:20 |
| dmellado | dmellado: don't exceed my weekly one | 09:20 |
| dmellado | I'm tempted to go work on SaaS | 09:21 |
| dmellado | xD | 09:21 |
| dulek | dmellado: The LBaaSv2 issue seems to be just a timeout. | 09:23 |
| dulek | Ah, crap, not really: http://logs.openstack.org/04/578304/1/check/kuryr-kubernetes-tempest-openshift-lbaasv2/b78cba9/controller/logs/screen-kuryr-kubernetes.txt.gz#_Jun_27_08_44_33_768706 | 09:23 |
| dulek | FIP conflict. | 09:24 |
| dulek | For some reason. | 09:24 |
| dmellado | huh, that's quite weird | 09:31 |
| dmellado | looks like an infra issue | 09:31 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix health server binding address https://review.openstack.org/578344 | 09:35 |
| dulek | dmellado, irenab: This fixes the healthchecks, therefore fixing the failing containerized+Octavia gate. ^ | 09:36 |
| dmellado | dulek: ack | 09:37 |
| dmellado | in any case I'll do it the other way around | 09:37 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix health server binding address https://review.openstack.org/578344 | 09:37 |
| dmellado | I'll make yossi's patch depend on the one enabling services | 09:37 |
| dmellado | and yours | 09:37 |
| dulek | dmellado: Whatever, we can check if my patch is working without Services tests - just check the number of restarts of Kuryr pods. | 09:38 |
| dulek | Should be 0. | 09:38 |
| dmellado | sure, let's see | 09:38 |
| dmellado | dulek: we need a beer after all this gate thing | 09:38 |
| dmellado | couldn't do another thing this week | 09:38 |
| dmellado | damn Flask | 09:39 |
| dulek | It's all because of Flask changing default values of arguments. :) | 09:39 |
| dmellado | oh, I see | 09:40 |
| dmellado | so it was binding localhost all the time | 09:40 |
| dmellado | awesome | 09:40 |
| *** threestrands has joined #openstack-kuryr | 09:45 | |
| *** threestrands has quit IRC | 09:45 | |
| *** threestrands has joined #openstack-kuryr | 09:45 | |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Add contrib folder to irrelevant files https://review.openstack.org/577427 | 09:46 |
| *** threestrands has quit IRC | 09:46 | |
| *** threestrands has joined #openstack-kuryr | 09:47 | |
| *** threestrands has joined #openstack-kuryr | 09:47 | |
| *** yboaron_ has joined #openstack-kuryr | 10:01 | |
| *** yboaron has quit IRC | 10:04 | |
| dulek | yboaron_: https://docs.openstack.org/requirements/latest/#proposing-changes - might be useful, though I don't understand this completely. :P | 10:10 |
| dulek | yboaron_: Okay, this one's more comprehensive: https://docs.openstack.org/project-team-guide/dependency-management.html | 10:11 |
| yboaron_ | dulek, 10X! | 10:11 |
| openstackgerrit | Genadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Test restart of CNI and Controller kuryr pods https://review.openstack.org/552498 | 10:24 |
| *** janonymous has joined #openstack-kuryr | 10:25 | |
| *** pcaruana has quit IRC | 10:31 | |
| *** yboaron has joined #openstack-kuryr | 11:05 | |
| *** yboaron_ has quit IRC | 11:07 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-tempest-plugin master: Restore service tests https://review.openstack.org/578304 | 11:20 |
| dmellado | irenab: celebdor could you please +2+A https://review.openstack.org/#/c/578344/ | 11:27 |
| irenab | dmellado, done | 11:27 |
| dmellado | irenab: thanks! | 11:28 |
| snapiri | dmellado: I would appreciate a review for https://review.openstack.org/#/c/577656/ as it fixes a problem with DF deployment. It does not fix the gate, but at least allows the devstack to complete. | 11:44 |
| dmellado | snapiri: sure, I'm just holding on some patches on the gate to merge | 11:45 |
| dmellado | and I'll have this in there after it | 11:45 |
| dmellado | ;) | 11:45 |
| dmellado | snapiri: oh | 11:45 |
| dmellado | quick change | 11:45 |
| dmellado | I'll approve it | 11:45 |
| snapiri | dmellado: thanks | 11:49 |
| *** atoth has joined #openstack-kuryr | 11:55 | |
| *** rh-jelabarre has joined #openstack-kuryr | 12:11 | |
| *** ispp has quit IRC | 12:20 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-kubernetes master: OCP-Router: Add Zuul experimental gates for Openshift Route https://review.openstack.org/565060 | 12:29 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-tempest-plugin master: Restore service tests https://review.openstack.org/578304 | 12:43 |
| *** maysamacedos has joined #openstack-kuryr | 12:47 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-kubernetes master: Services LBaaS driver: fix ensure LB for existing LB. https://review.openstack.org/578298 | 12:48 |
| *** janonymous has quit IRC | 12:54 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Fix health server binding address https://review.openstack.org/578344 | 12:56 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Remove redundant line from dragonflow config https://review.openstack.org/577656 | 12:56 |
| *** yboaron has quit IRC | 13:28 | |
| *** garyloug has quit IRC | 14:25 | |
| *** itzikb_ has joined #openstack-kuryr | 14:28 | |
| *** threestrands has quit IRC | 14:30 | |
| *** maysamacedos has quit IRC | 14:35 | |
| *** janki has quit IRC | 14:35 | |
| *** rh-jelabarre has quit IRC | 14:37 | |
| *** garyloug has joined #openstack-kuryr | 14:44 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix the experimental multinode jobs https://review.openstack.org/558762 | 14:52 |
| *** hongbin has joined #openstack-kuryr | 14:52 | |
| *** gcheresh has quit IRC | 14:58 | |
| *** itzikb_ has quit IRC | 14:59 | |
| *** janki has joined #openstack-kuryr | 15:08 | |
| *** maysamacedos has joined #openstack-kuryr | 15:25 | |
| *** itzikb_ has joined #openstack-kuryr | 15:30 | |
| *** maysamacedos has quit IRC | 15:43 | |
| openstackgerrit | Gary Loughnane proposed openstack/kuryr-kubernetes master: Add DPDK support for nested pods https://review.openstack.org/559363 | 16:28 |
| *** garyloug has quit IRC | 16:56 | |
| *** atoth has quit IRC | 18:00 | |
| *** janki has quit IRC | 18:06 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Services LBaaS driver: fix ensure LB for existing LB. https://review.openstack.org/578298 | 18:13 |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Restore service tests https://review.openstack.org/578304 | 18:16 |
| *** rh-jelabarre has joined #openstack-kuryr | 18:22 | |
| *** atoth has joined #openstack-kuryr | 18:54 | |
| *** itzikb_ has quit IRC | 18:55 | |
| *** rh-jelabarre has quit IRC | 19:34 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Add contrib folder to irrelevant files https://review.openstack.org/577427 | 20:33 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: fix typos https://review.openstack.org/577099 | 20:33 |
| *** hongbin has quit IRC | 22:27 | |
| *** kzaitsev1pi has joined #openstack-kuryr | 22:50 | |
| *** kzaitsev_pi has quit IRC | 22:52 | |
| *** pmannidi has joined #openstack-kuryr | 22:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!