| *** livelace has quit IRC | 00:44 | |
| *** livelace has joined #openstack-kuryr | 00:44 | |
| *** hongbin has joined #openstack-kuryr | 01:57 | |
| *** hongbin_ has joined #openstack-kuryr | 03:21 | |
| *** hongbin has quit IRC | 03:24 | |
| openstackgerrit | Filippo Inzaghi proposed openstack/fuxi master: import zuul job settings from project-config https://review.openstack.org/595425 | 03:31 |
|---|---|---|
| openstackgerrit | Filippo Inzaghi proposed openstack/fuxi master: switch documentation job to new PTI https://review.openstack.org/595426 | 03:31 |
| openstackgerrit | Filippo Inzaghi proposed openstack/fuxi master: add python 3.6 unit test job https://review.openstack.org/595427 | 03:31 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr master: import zuul job settings from project-config https://review.openstack.org/595431 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr master: switch documentation job to new PTI https://review.openstack.org/595432 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr master: add python 3.6 unit test job https://review.openstack.org/595433 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-kubernetes master: import zuul job settings from project-config https://review.openstack.org/595434 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-kubernetes master: switch documentation job to new PTI https://review.openstack.org/595435 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-kubernetes master: add python 3.6 unit test job https://review.openstack.org/595436 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-libnetwork master: import zuul job settings from project-config https://review.openstack.org/595437 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-libnetwork master: switch documentation job to new PTI https://review.openstack.org/595438 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-libnetwork master: add python 3.6 unit test job https://review.openstack.org/595439 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-tempest-plugin master: import zuul job settings from project-config https://review.openstack.org/595440 | 03:32 |
| openstackgerrit | Filippo Inzaghi proposed openstack/kuryr-tempest-plugin master: switch documentation job to new PTI https://review.openstack.org/595441 | 03:32 |
| *** itzikb has joined #openstack-kuryr | 04:09 | |
| *** yboaron_ has joined #openstack-kuryr | 04:43 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 04:49 |
| *** itzikb has quit IRC | 05:13 | |
| *** hongbin_ has quit IRC | 05:19 | |
| *** dmellado has joined #openstack-kuryr | 05:27 | |
| *** janki has joined #openstack-kuryr | 05:40 | |
| *** jchhatbar has joined #openstack-kuryr | 06:31 | |
| *** janki has quit IRC | 06:31 | |
| *** pcaruana has joined #openstack-kuryr | 06:33 | |
| *** gcheresh has joined #openstack-kuryr | 06:45 | |
| *** itzikb has joined #openstack-kuryr | 06:46 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 06:57 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 07:01 |
| *** yboaron_ has quit IRC | 07:22 | |
| *** jchhatba_ has joined #openstack-kuryr | 07:24 | |
| *** jchhatbar has quit IRC | 07:25 | |
| openstackgerrit | Merged openstack/kuryr-libnetwork master: install-guide: download source from stable branch https://review.openstack.org/594770 | 07:29 |
| openstackgerrit | Merged openstack/kuryr-libnetwork master: Update reno for stable/rocky https://review.openstack.org/593158 | 07:29 |
| *** celebdor has joined #openstack-kuryr | 07:38 | |
| *** gkadam has joined #openstack-kuryr | 07:44 | |
| *** jchhatba_ has quit IRC | 07:46 | |
| *** jchhatba_ has joined #openstack-kuryr | 07:48 | |
| *** jchhatba_ has quit IRC | 07:50 | |
| celebdor | dmellado: I just saw on my email backlog that they are removing openshift start commands from origin master | 08:00 |
| celebdor | this means we'll need to redo the devstack openshift support | 08:00 |
| dmellado | celebdor: hmm | 08:01 |
| dmellado | exactly | 08:01 |
| dmellado | damn! | 08:01 |
| celebdor | I love it when I have good news after holidays | 08:01 |
| celebdor | :/ | 08:01 |
| dmellado | just go and create a launchpad for that | 08:01 |
| dmellado | we'll also need to tackle the python3 migration start | 08:01 |
| celebdor | on the flip side, it will just use kubelet for the node instead | 08:01 |
| dmellado | I've got quite a big chunk of patches | 08:01 |
| celebdor | dmellado: don't make me sad | 08:01 |
| dmellado | which I'll start assigning soon to you folks | 08:01 |
| celebdor | now you make me remember my dropped python3 original kuryr controller code | 08:01 |
| dmellado | also celebdor | 08:01 |
| celebdor | which did not depend on any openstack libraries | 08:02 |
| dmellado | you owe me because while you were away | 08:02 |
| celebdor | and that I miss so much | 08:02 |
| dmellado | I had to go and endure *all* the meetings | 08:02 |
| dmellado | xD | 08:02 |
| dmellado | now we can safely split them back xD | 08:02 |
| celebdor | dmellado: I'll get you a fuet for that | 08:02 |
| dmellado | there was a meeting I had to endure 'again' | 08:02 |
| dmellado | and re: python3 yeah | 08:04 |
| dmellado | it all looks absurd now that I look back there | 08:04 |
| dmellado | but anyways | 08:04 |
| dmellado | that's a TC goal for the S cycle | 08:04 |
| celebdor | dmellado: can we drop python2? | 08:04 |
| dmellado | celebdor: we *will* drop python2 | 08:04 |
| dmellado | at least for the trunk and so | 08:05 |
| celebdor | ltomasbo: how did the move to python-requests break your namespace tests? | 08:11 |
| ltomasbo | you forgot to add the protocol (http) | 08:12 |
| ltomasbo | it got already fixed | 08:12 |
| ltomasbo | celebdor, ^^ | 08:12 |
| *** gkadam is now known as gkadam-afk | 08:14 | |
| celebdor | ltomasbo: damn... I probably forgot it in git stash | 08:15 |
| celebdor | as usual | 08:15 |
| celebdor | :/ | 08:15 |
| celebdor | or worse, lost it on a rebase | 08:15 |
| ltomasbo | don't worry, it was easy to catch | 08:16 |
| celebdor | thanks | 08:16 |
| ltomasbo | celebdor, and I found a race between pools and namespaces... | 08:16 |
| ltomasbo | celebdor, and currently working on pools + HA | 08:16 |
| celebdor | what's the nature of the race? | 08:18 |
| ltomasbo | celebdor, when deleting the namespace, the pods are getting deleted too | 08:19 |
| ltomasbo | but then the vif handler was triggering the release_vif after the namespace obj was deleted | 08:19 |
| ltomasbo | therefore, when trying to get the security_group asociated t othe namespace, it was failing due to missing namespace object | 08:19 |
| ltomasbo | so, once the pod obj was deleted, the namespace object is deleted, and that (sometimes) happens before the vif handler on_deleted function gets executed | 08:20 |
| celebdor | I see | 08:22 |
| celebdor | dmellado: shouldn't I change the Dockerfiles we use for upstream containerized gates to python3? | 08:28 |
| dmellado | that too, ofc | 08:28 |
| dmellado | feel free to propose a patch | 08:28 |
| celebdor | cause, infra's change to python3 will not affect us at all | 08:28 |
| celebdor | dmellado: when? | 08:28 |
| dmellado | but hold into that | 08:28 |
| dmellado | did you go through the mails on dev-list | 08:29 |
| dmellado | ? | 08:29 |
| dmellado | [goal][python3]? | 08:29 |
| *** yboaron_ has joined #openstack-kuryr | 08:30 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure ports pool are not duplicated with kuryr-controller HA https://review.openstack.org/593679 | 08:32 |
| celebdor | dmellado: dev-list upstream? | 08:33 |
| dmellado | celebdor: yep, just look at our launchpad | 08:33 |
| celebdor | I don't check things withouth the [kuryr] tag often | 08:33 |
| celebdor | xD | 08:33 |
| dmellado | I've summarized that into an issue | 08:33 |
| dmellado | for your laziness | 08:33 |
| dmellado | xD | 08:33 |
| dmellado | FYL, instead of FYI | 08:33 |
| dmellado | xD | 08:34 |
| celebdor | thanks | 08:37 |
| ltomasbo | celebdor, one question about the HA | 08:49 |
| ltomasbo | celebdor, once a controller that is active, lost the leadership, (due to for instance transient networking issues_ | 08:50 |
| celebdor | yes | 08:50 |
| ltomasbo | celebdor, it is expected that it can get back to active at some point | 08:50 |
| ltomasbo | for intance, if the network is up again and hte new active controller goes down for wahtever reason | 08:51 |
| celebdor | ltomasbo: right, there would be two leader elections | 08:51 |
| celebdor | and it could end up being leader again | 08:51 |
| ltomasbo | ok | 08:52 |
| ltomasbo | in that case I think we have a problem | 08:52 |
| ltomasbo | as the watchers will not be restarted | 08:52 |
| *** garyloug has joined #openstack-kuryr | 08:52 | |
| ltomasbo | celebdor, https://github.com/openstack/kuryr-kubernetes/blob/master/kuryr_kubernetes/controller/service.py#L86 | 08:53 |
| celebdor | ltomasbo: most likely, if it is due to network issues, the health checks will fail and the pod will be restarted | 08:53 |
| celebdor | but if there is a way in which it happens where the health checks still pass | 08:54 |
| celebdor | then yes | 08:54 |
| celebdor | it will be a problem | 08:54 |
| ltomasbo | yep | 08:54 |
| ltomasbo | I was just blocking traffic on port 443 | 08:54 |
| ltomasbo | to test the swtichover | 08:54 |
| ltomasbo | so, that is independent of the HA + Pool support | 08:55 |
| celebdor | that should fail the "connected to k8s health check" | 08:55 |
| ltomasbo | I tested it and this seems to work: https://review.openstack.org/#/c/593679/4 | 08:56 |
| ltomasbo | so celebdor, dmellado ^^ reviews are welcome! xD | 08:56 |
| dmellado | ltomasbo: I'll assign you a few patches as well | 08:56 |
| ltomasbo | xD | 08:56 |
| dmellado | you look like you'll enjoy them | 08:56 |
| dmellado | xD | 08:56 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 09:56 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Ensure OpenShift gate uses the namespace subnet/sg drivers https://review.openstack.org/580680 | 10:06 |
| *** rh-jelabarre has joined #openstack-kuryr | 11:35 | |
| *** yboaron_ has quit IRC | 11:50 | |
| *** yboaron_ has joined #openstack-kuryr | 11:51 | |
| celebdor | dmellado: yboaron_: can we get https://review.openstack.org/#/c/591394/12 in? | 11:52 |
| celebdor | yboaron_: do you know what's the issue with https://review.openstack.org/#/c/578310/40 ? | 11:52 |
| dmellado | celebdor: looks like some weird incompatibility with the openshift client | 11:59 |
| dmellado | check it there and if you've some time lend yboaron_ a hand | 11:59 |
| celebdor | what?! | 11:59 |
| dmellado | I'll be looking into it as well | 11:59 |
| * dmellado off to a meeting, brb | 11:59 | |
| celebdor | yboaron_: do you have a link on that? | 12:00 |
| dmellado | celebdor: I was just having a quick meeting with yboaron_ re that | 12:00 |
| dmellado | check also his patch here https://review.openstack.org/#/c/595569/1 | 12:01 |
| *** yboaron_ has quit IRC | 12:02 | |
| *** pcaruana has quit IRC | 12:16 | |
| *** pcaruana has joined #openstack-kuryr | 12:16 | |
| *** yboaron_ has joined #openstack-kuryr | 12:27 | |
| *** rh-jelabarre has quit IRC | 12:32 | |
| *** garyloug_ has joined #openstack-kuryr | 12:34 | |
| *** gcheresh has quit IRC | 12:36 | |
| *** itzikb has quit IRC | 12:37 | |
| *** dims_ is now known as dims | 12:37 | |
| *** garyloug has quit IRC | 12:37 | |
| *** yboaron_ has quit IRC | 12:38 | |
| *** rh-jelabarre has joined #openstack-kuryr | 13:34 | |
| *** itzikb has joined #openstack-kuryr | 13:38 | |
| *** gcheresh has joined #openstack-kuryr | 13:38 | |
| *** yboaron has joined #openstack-kuryr | 13:44 | |
| *** garyloug_ has quit IRC | 13:45 | |
| *** celebdor has quit IRC | 13:51 | |
| yboaron | ltomasbo, dmellado : could you please take a look at : https://review.openstack.org/#/c/591394/ | 13:52 |
| ltomasbo | sure! | 13:53 |
| *** celebdor has joined #openstack-kuryr | 14:01 | |
| *** garyloug has joined #openstack-kuryr | 14:06 | |
| *** gcheresh has quit IRC | 14:18 | |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 14:33 |
| *** itzikb has quit IRC | 15:06 | |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Use ThreadPool and requests instead of subprocess https://review.openstack.org/591394 | 15:28 |
| celebdor | yay! | 15:29 |
| celebdor | yboaron: ping | 15:29 |
| yboaron | celebdor, pong | 15:29 |
| celebdor | yboaron: did you find out the problem with the requirements bump? | 15:29 |
| yboaron | celebdor, I think so, Kubernetes was raised to 7.0.0 and Openshift clinet doesn't work with k8s 7.0.0 | 15:30 |
| celebdor | yboaron: do you know why? | 15:30 |
| yboaron | celebdor, give me a sec I"ll paste the link from openshift github | 15:31 |
| celebdor | ok | 15:31 |
| celebdor | thanks | 15:31 |
| yboaron | celebdor, https://github.com/openshift/openshift-restclient-python/commit/666f77817300f2d0095551d519b3413865b83ddc | 15:32 |
| yboaron | celebdor, this PR pinned openshift to K8S 6.0.0 | 15:32 |
| celebdor | yboaron: can we set kuryr-kubernetes upper constraints to 6.0.0? | 15:34 |
| yboaron | celebdor, I think so, I changed kuryr-tempest plugin req to be 6.0.0, building it .. | 15:35 |
| yboaron | celebdor, hope it will work this time | 15:36 |
| celebdor | very well | 15:36 |
| *** pcaruana has quit IRC | 15:36 | |
| yboaron | celebdor, this is the PR that changed K8S to 7.0.0 , was merged this morning | 15:36 |
| yboaron | celebdor, https://review.openstack.org/#/c/594495/3 | 15:37 |
| yboaron | celebdor, I mean upper constraints | 15:37 |
| celebdor | thanks | 15:40 |
| yboaron | celebdor, to set kubernetes to 6.0.0. I just need change to 'kubernetes==6.0.0' in requirements.txt @ kuryr-tempest right? | 15:55 |
| celebdor | yboaron: not sure | 15:55 |
| celebdor | try and see what the gate says | 15:55 |
| yboaron | celebdor, will do | 15:56 |
| openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: OCP-Router - add support for OCP routes https://review.openstack.org/578310 | 16:01 |
| *** gkadam-afk has quit IRC | 16:07 | |
| *** yboaron has quit IRC | 16:56 | |
| *** garyloug has quit IRC | 16:57 | |
| *** pcaruana has joined #openstack-kuryr | 18:52 | |
| *** pcaruana has quit IRC | 19:09 | |
| *** rh-jelabarre has quit IRC | 21:09 | |
| *** threestrands has joined #openstack-kuryr | 22:53 | |
| *** celebdor has quit IRC | 22:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!