| *** janki has joined #openstack-kuryr | 00:55 | |
| *** chenyb4 has joined #openstack-kuryr | 00:55 | |
| *** janki has quit IRC | 00:55 | |
| *** janki has joined #openstack-kuryr | 00:56 | |
| *** hongbin_ has joined #openstack-kuryr | 01:26 | |
| *** salv-orl_ has joined #openstack-kuryr | 01:58 | |
| *** salv-orl_ has quit IRC | 02:00 | |
| *** jchhatbar has joined #openstack-kuryr | 02:01 | |
| *** salv-orlando has quit IRC | 02:01 | |
| *** janki has quit IRC | 02:03 | |
| *** threestrands has joined #openstack-kuryr | 02:59 | |
| *** premsankar has quit IRC | 03:05 | |
| *** maysamacedos has quit IRC | 03:40 | |
| *** hongbin_ has quit IRC | 04:03 | |
| *** salv-orlando has joined #openstack-kuryr | 05:11 | |
| *** premsankar has joined #openstack-kuryr | 05:33 | |
| *** pcaruana has joined #openstack-kuryr | 06:43 | |
| *** yamamoto has quit IRC | 06:50 | |
| *** threestrands has quit IRC | 07:00 | |
| *** celebdor1 has joined #openstack-kuryr | 07:11 | |
| dulek | dmellado: Do we release anything for R-1 milestone? | 07:28 |
|---|---|---|
| dmellado | dulek: hey, morning | 07:29 |
| dulek | o/ | 07:29 |
| dmellado | well, no that we're forced to cope with the milestones | 07:29 |
| dulek | Ah, the deadline was yesterday, so nevermind. :D | 07:29 |
| dmellado | but that said I wanted to check with you and apuimedo about cutting a stable/queens release anyway | 07:29 |
| dmellado | I was about to check what was going on with your patches from yesterday | 07:30 |
| dmellado | good find! | 07:30 |
| dulek | dmellado: First we need apuimedo's patch, right? | 07:30 |
| dulek | dmellado: They're passing! :D | 07:30 |
| dulek | It's kind of embarrassing that we were blocked because of Octavia timeouts, but here we are. | 07:31 |
| dmellado | I'm still suspecting on non-nested-kvm slowness... | 07:32 |
| dmellado | mnasser, from vexxhost was pushing to enable this | 07:32 |
| dmellado | dulek: lol https://review.openstack.org/#/c/549945/ | 07:33 |
| dmellado | let's check what failed on the merge gate | 07:34 |
| dmellado | could be even *more* slowness | 07:34 |
| dulek | Hm! | 07:35 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:785 : [[ PENDING_CREATE != \A\C\T\I\V\E ]] | 07:36 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:786 : echo -n 'Waiting till LB=default/kubernetes is ACTIVE.' | 07:36 |
| dmellado | Waiting till LB=default/kubernetes is ACTIVE.+++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:787 : date +%s | 07:36 |
| dulek | dmellado: :D | 07:36 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:787 : curr_time=1524174130 | 07:36 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:788 : time_diff=608 | 07:36 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:789 : [[ 608 -le 600 ]] | 07:36 |
| dmellado | ++ /opt/stack/kuryr-kubernetes/devstack/lib/kuryr_kubernetes:wait_for_lb:789 : die 'Timed out waiting for default/kubernetes' | 07:36 |
| dmellado | dammit | 07:36 |
| * dmellado sighs | 07:36 | |
| dmellado | could you amend your patch to increase that even further, dulek ? | 07:36 |
| dulek | dmellado: DevStack has timeout of 600 seconds as yboaron set it. | 07:36 |
| dmellado | dulek: let's run next CI runs on my good ol 486DX4 | 07:37 |
| dulek | dmellado: Yep, in a moment. This is just another occurrence of hardcoded timeout, the fix is correct. | 07:37 |
| celebdor1 | dulek: nah.... | 07:37 |
| dulek | ? | 07:37 |
| celebdor1 | about R-1 | 07:37 |
| celebdor1 | I don't believe in milestones | 07:37 |
| dmellado | celebdor1: how are your allergies today? | 07:38 |
| dmellado | mine went back wild | 07:38 |
| dmellado | :\ | 07:38 |
| celebdor1 | dmellado: better than yesterday | 07:38 |
| celebdor1 | I didn't even take the pill today yet | 07:38 |
| celebdor1 | and the last two days I had been forced to | 07:38 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Make Neutron LBaaS Activation Timeout configurable https://review.openstack.org/549945 | 07:42 |
| dulek | That should make it better. ^ | 07:42 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Switch amphora tarball to ubuntu xenial https://review.openstack.org/562969 | 07:48 |
| *** yamamoto has joined #openstack-kuryr | 07:51 | |
| dulek | dmellado: We're running almost all the tests on Ubuntu, why do we even want centos amphorae? | 07:57 |
| celebdor1 | ltomasbo: re https://review.openstack.org/#/c/562322/ | 07:59 |
| celebdor1 | what you want there? | 07:59 |
| celebdor1 | Isn't the l2 problem unrelated? | 08:00 |
| dulek | dmellado: Sorry, got disconnected. Had you answered? | 08:00 |
| celebdor1 | dulek: that's a very good point | 08:00 |
| celebdor1 | dmellado: answer! | 08:00 |
| celebdor1 | xD | 08:00 |
| dmellado | lol | 08:00 |
| celebdor1 | dmellado: did bcafarelli bribe you? | 08:01 |
| dmellado | hey dulek, well basically I wanted to try that out | 08:01 |
| dmellado | yeah, he told me it was better! xD | 08:01 |
| *** yamamoto has quit IRC | 08:01 | |
| dmellado | and I was so naive of trusting him | 08:01 |
| dmellado | my plan was also to switch to rdo's hosted amphora, based on centos in the end | 08:01 |
| dulek | dmellado: Okay, so I won't bug you with adding #FIXME comment to switch it back once bug's fixed. | 08:02 |
| dulek | dmellado: But still the URL in the playbook seems malformed to me. | 08:02 |
| dmellado | and also I'm dumb xD | 08:02 |
| dmellado | yeah dulek | 08:02 |
| dmellado | fridays and no coffe aren't good to me | 08:02 |
| *** celebdor1 is now known as apuimedo | 08:02 | |
| dmellado | I still recall what happened last week xD | 08:02 |
| dulek | :) | 08:02 |
| dmellado | will amend now | 08:02 |
| apuimedo | dulek: dmellado: ltomasbo: guys, we should make a release today | 08:03 |
| apuimedo | gather up your bugs and backports | 08:03 |
| dulek | apuimedo: First thing to do is to make Octavia gates non-voting, right? | 08:04 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Switch amphora tarball to ubuntu xenial https://review.openstack.org/562969 | 08:04 |
| dmellado | dulek: that already has happened | 08:04 |
| apuimedo | dulek: didn't we do that already? | 08:04 |
| dulek | dmellado: Oh, nice. | 08:04 |
| dmellado | for stable/queens | 08:04 |
| dulek | Haven't noticed. | 08:05 |
| apuimedo | dulek: https://review.openstack.org/#/c/562744/ | 08:05 |
| apuimedo | dulek: you were in the pub | 08:05 |
| apuimedo | xD | 08:05 |
| dmellado | dulek: how were the beers? xD | 08:05 |
| dulek | apuimedo: Good to have so nice colleagues that keep working while I'm out. | 08:06 |
| dulek | dmellado: My team got 3rd place, which means winning a bottle of vodka. Which means I've just left quietly because I know those guys and I'm sure it haven't ended with just one bottle. :P | 08:07 |
| dmellado | xDDD | 08:07 |
| *** garyloug has joined #openstack-kuryr | 08:08 | |
| dulek | Okay, 561974 is rechecking. | 08:09 |
| dulek | I'll fix up 562736 - I think we need it as backport. | 08:09 |
| dulek | I don't think we're in a hurry to do 562146. | 08:09 |
| dulek | And I don't care about 562501. | 08:10 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: devstack: Fix octavia api lb kubelet iface access https://review.openstack.org/562322 | 08:10 |
| dulek | dmellado, apuimedo, ltomasbo: And there's only one question left. Do we backport LB activation timeout patch? | 08:11 |
| dmellado | dulek: +1 to dhat | 08:11 |
| dmellado | dulek: let's go for backporting that as it could hurt the stable gates at any random pace | 08:11 |
| dmellado | I mean, I'd like to revert the non-voting octavia | 08:11 |
| dulek | dmellado: What's that in "backporting that"? | 08:12 |
| dulek | dmellado: UC? | 08:12 |
| dmellado | the timeout patch | 08:12 |
| dulek | Ah, okay, sorry. :P | 08:12 |
| dmellado | np! If you won the vodka bottle it's totally understandable xDDD | 08:13 |
| dulek | dmellado: So there's a small problem there. It's backporting a new configuration option. | 08:13 |
| dmellado | hmm, that's true | 08:13 |
| *** yamamoto has joined #openstack-kuryr | 08:13 | |
| dulek | dmellado: I haven't had a single shot of it! :P | 08:13 |
| dmellado | maybe what we could do is just hardcode the new timeout | 08:13 |
| dulek | dmellado: I'll review stable guidelines. | 08:13 |
| dmellado | awesome, thanks dulek | 08:13 |
| dulek | (on the other hand - do we even have a tag that says Kuryr follows stable guidelines? :D) | 08:14 |
| dulek | dmellado: https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes doesn't say anything about configuration changes. | 08:16 |
| dulek | And we're just adding a new option that defaults to old behavior. | 08:16 |
| dulek | So I'm in to backport it. | 08:16 |
| dmellado | +1 | 08:19 |
| dmellado | go for it | 08:19 |
| dmellado | and LOL on the tag xDDDD | 08:19 |
| apuimedo | dulek: yes. I want the LB activation patch | 08:19 |
| apuimedo | it is an important bug | 08:19 |
| apuimedo | it may make or break deployments | 08:19 |
| apuimedo | sadly | 08:19 |
| apuimedo | AMPHORAE! Y U SO SLOW! | 08:20 |
| dmellado | apuimedo: I'm having nightmares about 2012 cpus' | 08:20 |
| dulek | apuimedo: It definitely was breaking if Eunsoo proposed the patch. :P | 08:20 |
| apuimedo | dulek: exactly | 08:20 |
| dulek | This guy is a step ahead of us! :D | 08:20 |
| apuimedo | dulek: dmellado: https://review.openstack.org/#/c/562322/3 | 08:21 |
| apuimedo | dulek: he probably has hw more powerful than potatos to run it on | 08:22 |
| dulek | https://review.openstack.org/#/c/562736 - HTTPS for kubelet is ready on stable/queens. | 08:22 |
| apuimedo | xD | 08:22 |
| apuimedo | dulek: won't be ready until v+1 | 08:22 |
| apuimedo | :-) | 08:22 |
| dulek | Okay, so let's wait for v+1 on LB activation timeout patch as well and once it's merged I'll backport it. | 08:23 |
| apuimedo | ok | 08:24 |
| *** premsankar has quit IRC | 08:30 | |
| apuimedo | ltomasbo: we need your review on https://review.openstack.org/#/c/562969/ | 08:34 |
| apuimedo | specially whether this worked for you or not | 08:34 |
| apuimedo | since l2 is not gated | 08:34 |
| ltomasbo | apuimedo, let me take a look | 08:34 |
| ltomasbo | apuimedo, I'll be adding a gate for in asap (and ovs-firewall | 08:34 |
| apuimedo | ltomasbo: infra will finally kill us with so many gates | 08:34 |
| apuimedo | put the l2 one in experimental at least | 08:35 |
| ltomasbo | apuimedo, I thought about that, l2 on experimental | 08:35 |
| ltomasbo | apuimedo, and maybe not even have a new one for ovs-firewall, but moving one of the existing ones, what do you thinkg? | 08:35 |
| ltomasbo | apuimedo, and regarding the patch, yes, that how I make it work with L2 (and L3) | 08:38 |
| ltomasbo | so I +1 it! | 08:38 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 08:39 |
| apuimedo | ltomasbo: move the existing ones | 08:41 |
| apuimedo | definitely | 08:41 |
| *** garyloug has quit IRC | 08:41 | |
| ltomasbo | apuimedo, great! | 08:41 |
| *** garyloug has joined #openstack-kuryr | 08:41 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 08:43 |
| *** yamamoto has quit IRC | 08:44 | |
| dulek | dmellado: Hey, you need to return the infra VM! | 08:45 |
| dmellado | dulek: oh, you're right | 08:45 |
| dulek | dmellado: Or they will never let us do such debugging. :P | 08:45 |
| dmellado | let me ping them | 08:45 |
| dmellado | LOL | 08:45 |
| ltomasbo | dmellado, dulek, apuimedo, we don't have a single deployment with containerized option at the gate for standar checks and gate checks | 09:04 |
| ltomasbo | only one for experimentals | 09:04 |
| dulek | ltomasbo: Yup, it's high time to move those from experimental. | 09:05 |
| ltomasbo | should we add the experimental one to the gate/check? | 09:05 |
| dulek | ltomasbo: I think they're stable enough. | 09:05 |
| ltomasbo | dulek, ok, I'm sending a patch adding a couple and moving that to normal check | 09:05 |
| dulek | I can make a patch. Do we want OpenShift in the gate as well? | 09:05 |
| dulek | ltomasbo: Ok! | 09:05 |
| phuoc_ | hi you guys | 09:12 |
| phuoc_ | I am installing kuryr-kubernetes manually | 09:12 |
| phuoc_ | https://github.com/openstack/kuryr-kubernetes/blob/master/doc/source/installation/services.rst#making-the-pods-be-able-to-reach-the-kubernetes-api | 09:12 |
| phuoc_ | when I create "openstack loadbalancer member create --address 192.168.1.2", I don't know why address is `192.168.1.2` | 09:13 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add better octavia coverage at the gate https://review.openstack.org/562998 | 09:14 |
| phuoc_ | it does not belongs to any subnets | 09:14 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Make Neutron LBaaS Activation Timeout configurable https://review.openstack.org/549945 | 09:15 |
| phuoc_ | ltomasbo, dulek hi | 09:17 |
| dulek | phuoc_: That seems to be local address. | 09:18 |
| dulek | phuoc_: If that's a LB member it's supposed to be K8s API address. | 09:18 |
| phuoc_ | dulek, you mean it is HOST IP | 09:19 |
| dulek | phuoc_: If your K8s is deployed on that host, then yes. | 09:19 |
| phuoc_ | should I specify it to Kubernetes master IP? | 09:21 |
| phuoc_ | dulek, thanks | 09:21 |
| phuoc_ | I will try to do it :D | 09:22 |
| dulek | :) | 09:22 |
| dulek | apuimedo, dmellado: Backport of LB timeout: https://review.openstack.org/#/c/563006/ | 09:28 |
| *** mfedosin has quit IRC | 09:29 | |
| dulek | apuimedo, dmellado: And backport of apuimedo's fix is green as well: https://review.openstack.org/#/c/561974/ | 09:29 |
| dulek | BTW - it's green on Octavia as well! xD | 09:29 |
| dulek | They must have fixed stuff overnight. | 09:29 |
| dulek | This really seems to be a good day… And it's Friday. I'm both happy and scared of what's going to happen in the evening. | 09:30 |
| *** mfedosin has joined #openstack-kuryr | 09:31 | |
| dmellado | \o/ | 09:34 |
| dmellado | lol | 09:34 |
| ltomasbo | lol | 09:35 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add ovs-firewall coverage https://review.openstack.org/563008 | 09:36 |
| apuimedo | dmellado: any idea why https://review.openstack.org/#/c/561364/ is not merging? | 09:39 |
| apuimedo | ltomasbo: -1 | 09:41 |
| ltomasbo | apuimedo, where? xD | 09:41 |
| apuimedo | ltomasbo: "improve octavia coverage" | 09:41 |
| apuimedo | that is the laziest commit message I've seen after "Initial commit" | 09:41 |
| ltomasbo | let me see | 09:41 |
| dmellado | LOL | 09:41 |
| ltomasbo | lol | 09:41 |
| ltomasbo | apuimedo, I'll update that | 09:42 |
| ltomasbo | apuimedo, besides that, happy with the move/addition? | 09:42 |
| dmellado | apuimedo: let's see now | 09:42 |
| dmellado | ltomasbo: in any case we'd need to split this | 09:42 |
| dmellado | I had a patch for that which I'll recover as it got lost in the way of life | 09:42 |
| dmellado | .zuul.yaml is too big right now and unless we split it'll keep growing and growing | 09:43 |
| dmellado | ltomasbo: also pls address my comments in that lazy commit message patch | 09:43 |
| *** salv-orlando has quit IRC | 09:43 | |
| ltomasbo | xD | 09:44 |
| ltomasbo | going to! | 09:44 |
| *** yamamoto has joined #openstack-kuryr | 09:44 | |
| apuimedo | ltomasbo: yes | 09:44 |
| apuimedo | dmellado: you can split after the release | 09:45 |
| dmellado | apuimedo: do you recall your old command too? | 09:45 |
| dmellado | apudo release | 09:45 |
| dmellado | xD | 09:45 |
| dmellado | let's get this sorted out now | 09:45 |
| dmellado | apuimedo: let's take a look at zuul queue | 09:45 |
| dmellado | it might've gotten stuck | 09:45 |
| *** salv-orlando has joined #openstack-kuryr | 09:46 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add better octavia coverage at the gate https://review.openstack.org/562998 | 09:50 |
| apuimedo | ltomasbo: this is good now | 09:50 |
| dmellado | apuimedo: | 09:50 |
| dmellado | pls approve this one | 09:50 |
| dmellado | https://review.openstack.org/#/c/563006/ | 09:50 |
| dmellado | it's blocking the another one from merging | 09:50 |
| ltomasbo | apuimedo, xD | 09:51 |
| apuimedo | dmellado: done | 09:51 |
| dmellado | allright | 09:53 |
| dmellado | so after all this goes thru queue | 09:53 |
| dmellado | we ought to be fine for release | 09:53 |
| dmellado | what a productive friday | 09:53 |
| dmellado | xD | 09:53 |
| *** yamamoto has quit IRC | 09:54 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 10:10 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 10:10 |
| ltomasbo | dmellado, apuimedo dulek: https://review.openstack.org/#/c/562146/ | 10:11 |
| dmellado | ltomasbo: about to put -1 due to the tests | 10:12 |
| dmellado | xD | 10:12 |
| dmellado | don't be lazy, man! | 10:12 |
| ltomasbo | dmellado, it was your patch! I just cherry-pick it! | 10:12 |
| ltomasbo | dmellado, so, it is your fault not to have written tests! | 10:12 |
| dmellado | ltomasbo: oh, it wasn't that one | 10:13 |
| dmellado | but https://review.openstack.org/#/c/562247/6 | 10:13 |
| dmellado | xD | 10:13 |
| ltomasbo | dmellado, men, I have the to do to add the tests (in that patch) | 10:14 |
| ltomasbo | dmellado, just review the other section! once we are ok with the structure I'll add unit test! | 10:14 |
| dmellado | ltomasbo: heh, alll right | 10:15 |
| dmellado | but don't think I'll forget | 10:15 |
| ltomasbo | dmellado, I'll add the tests in that patch! don't give the +1 or +2, just review the rest (with the corresponding -1!) | 10:15 |
| ltomasbo | once I get some reviews, I'll add the unittests | 10:16 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: devstack: Fix octavia api lb kubelet iface access https://review.openstack.org/562322 | 10:17 |
| dmellado | ltomasbo: ok with that, just some minor nits there | 10:18 |
| ltomasbo | dmellado, there will be more than a few minor nits, as nobody has yet reviewed it! | 10:20 |
| dmellado | ltomasbo: heh, for now | 10:21 |
| ltomasbo | dmellado, and to be clear, my intention is to add the unittest in the same patch set, not in a follow up patch! | 10:21 |
| dmellado | :P | 10:21 |
| ltomasbo | xD | 10:21 |
| dmellado | eeeven better! | 10:21 |
| dmellado | if this goes on I might even start sounding like LeChuck | 10:21 |
| dmellado | http://pirate.monkeyness.com/online_pirate_translator | 10:21 |
| ltomasbo | xD | 10:22 |
| dulek | dmellado: Hey, I'm surprised that backport blocked a patch on master. | 10:23 |
| dulek | dmellado: Ah, it's cross-project, so I've probably stopped it by doing the backport. | 10:23 |
| apuimedo | dmellado: please, make sure you go over https://groups.google.com/forum/#!topic/kubernetes-sig-network/A8oyMrWpJ8Q since you said you'd take over NetworkPolicy | 10:25 |
| dmellado | apuimedo: let me see | 10:27 |
| dmellado | sure, will reply | 10:27 |
| apuimedo | dmellado: you don't need to reply | 10:28 |
| apuimedo | it's about seeing how other people are coping with "are all the NetworkPolicy applied?" | 10:28 |
| apuimedo | And whether it is fine to do it after the pod is created | 10:29 |
| *** chenyb4 has quit IRC | 10:29 | |
| dmellado | I still need to go over patches | 10:29 |
| dmellado | I'll sync with irenab next week | 10:29 |
| *** chenyb4 has joined #openstack-kuryr | 10:29 | |
| dulek | This is probably the last backport that hasn't +W yet: https://review.openstack.org/#/c/562736/ | 10:30 |
| dmellado | and probably with you too, will setup a meeting | 10:30 |
| dmellado | dulek: ltomasbo I can add you if you're interested too but I assume you might have too much to cover | 10:30 |
| dmellado | dulek: and done! | 10:30 |
| *** pcaruana has quit IRC | 10:31 | |
| ltomasbo | dmellado, ok! | 10:31 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Kuryr controller - pluggable handlers documentation https://review.openstack.org/561447 | 10:32 |
| dulek | dmellado: Please add me. | 10:36 |
| dmellado | awesome then! I'll take the chance to gather all of you for the slides preparation as well xD | 10:36 |
| *** dulek has quit IRC | 10:46 | |
| *** chenyb4 has quit IRC | 10:48 | |
| *** chenyb4 has joined #openstack-kuryr | 10:48 | |
| *** dulek has joined #openstack-kuryr | 10:50 | |
| apuimedo | ltomasbo: I am not too convinced with your namespace_subnet driver | 11:13 |
| apuimedo | why are you not annotating the namespace with the reference to the CRD to avoid the lookup of all the kuryrnets? | 11:14 |
| ltomasbo | apuimedo, umm, that is actually a nice improvement! | 11:15 |
| ltomasbo | apuimedo, I'll add that! | 11:16 |
| apuimedo | thanks ltomasbo | 11:17 |
| ltomasbo | apuimedo, thank you for the tip! | 11:18 |
| apuimedo | ltomasbo: you're welcome | 11:19 |
| dulek | Okay, seems like we have everything we need on stable! | 11:20 |
| dulek | There are 3 more patches by ltomasbo, but those are only affecting DevStack. | 11:21 |
| dulek | So I guess we can release. I have the commit ready, please just ACK that we don't wait for anything else. | 11:21 |
| apuimedo | dulek: links? | 11:21 |
| dulek | apuimedo: https://review.openstack.org/#/q/project:openstack/kuryr-kubernetes+branch:stable/queens+status:open | 11:22 |
| dulek | Commit ready means commit ready for submission. ;) | 11:22 |
| ltomasbo | dulek, this is not just for devstack, right? https://review.openstack.org/#/c/563079/ | 11:23 |
| dulek | ltomasbo: It's not, but it only makes developer life easier, doesn't really affect end user. | 11:24 |
| dulek | I don't even think we should backport it. | 11:24 |
| apuimedo | dulek: what do you think about getting those in before cutting the release (and still make it to today) | 11:25 |
| apuimedo | it's not a big deal not to do it | 11:25 |
| apuimedo | since we can merge them after the release is cut | 11:25 |
| apuimedo | and we'll benefit the same from it | 11:26 |
| apuimedo | you know what | 11:26 |
| dulek | apuimedo: I don't really see a point in that, those patches are not affecting end user nor RPMs. | 11:26 |
| apuimedo | belay that | 11:26 |
| apuimedo | let's cut a release right now | 11:26 |
| dulek | https://review.openstack.org/#/q/project:openstack/kuryr-kubernetes+branch:stable/queens+status:open | 11:27 |
| dulek | Ah, not this. | 11:27 |
| dulek | https://review.openstack.org/#/c/563084/ - this will need dmellado ACK. Someone please double check the commit hash. | 11:28 |
| apuimedo | dulek: I just confirmed the ahsh | 11:29 |
| dmellado | dulek: all clear | 11:29 |
| apuimedo | *hash | 11:29 |
| dmellado | hash and ack there | 11:29 |
| dmellado | and so folks | 11:30 |
| dmellado | we've been able to go through another release! xD | 11:30 |
| dmellado | congrats everyone! | 11:30 |
| apuimedo | dmellado: I need this https://review.openstack.org/#/c/561364/ merged fucksakes | 11:30 |
| dmellado | hmm this is weird | 11:31 |
| dmellado | dependant commits are merged | 11:31 |
| *** yamamoto has joined #openstack-kuryr | 11:31 | |
| ltomasbo | dmellado, apuimedo, dulek: do we want to wait for octavia l2 support? | 11:32 |
| dmellado | ltomasbo: that won't be affecting queens release at all | 11:32 |
| dulek | ltomasbo: It's DevStack-only as well? | 11:32 |
| ltomasbo | true | 11:32 |
| apuimedo | it's devstack only | 11:33 |
| apuimedo | so fuck it | 11:33 |
| ltomasbo | dulek, yep | 11:33 |
| apuimedo | dmellado: man... Work your infra muscle | 11:33 |
| dmellado | dulek: that's the equivalent of hitting the car | 11:33 |
| dulek | apuimedo: It's going now. I've flipped +W. | 11:33 |
| dmellado | when it doesn't start | 11:33 |
| dmellado | xD | 11:33 |
| dmellado | LOL | 11:33 |
| dmellado | the sad thing is that I was going to do just the same xD | 11:33 |
| apuimedo | SAD | 11:34 |
| apuimedo | xD | 11:34 |
| dmellado | apuimedo: and now is when our friend doug comes and says -1 because of the weather | 11:36 |
| dmellado | xD | 11:36 |
| *** yamamoto_ has joined #openstack-kuryr | 11:44 | |
| *** pcaruana has joined #openstack-kuryr | 11:46 | |
| *** yamamoto has quit IRC | 11:47 | |
| *** chenyb4 has quit IRC | 11:49 | |
| *** maysamacedos has joined #openstack-kuryr | 11:52 | |
| *** garyloug has quit IRC | 11:57 | |
| *** atoth has joined #openstack-kuryr | 12:07 | |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Revert "Skip service test" https://review.openstack.org/561364 | 12:19 |
| *** yamamoto_ has quit IRC | 12:46 | |
| *** yamamoto has joined #openstack-kuryr | 12:47 | |
| *** yamamoto has quit IRC | 12:52 | |
| apuimedo | dmellado: it is expected of course | 12:53 |
| apuimedo | dmellado: you here? | 12:53 |
| *** maysamacedos has quit IRC | 13:00 | |
| *** kiennt26_ has joined #openstack-kuryr | 13:05 | |
| *** garyloug has joined #openstack-kuryr | 13:08 | |
| *** yamamoto has joined #openstack-kuryr | 13:21 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add better octavia coverage at the gate https://review.openstack.org/562998 | 13:21 |
| *** maysamacedos has joined #openstack-kuryr | 13:26 | |
| *** fkautz has quit IRC | 13:35 | |
| *** fkautz has joined #openstack-kuryr | 13:35 | |
| *** fkautz has quit IRC | 13:36 | |
| *** fkautz has joined #openstack-kuryr | 13:37 | |
| *** fkautz has quit IRC | 13:37 | |
| *** fkautz has joined #openstack-kuryr | 13:38 | |
| *** atoth has quit IRC | 13:38 | |
| *** fkautz has quit IRC | 13:38 | |
| *** fkautz has joined #openstack-kuryr | 13:39 | |
| *** fkautz has quit IRC | 13:39 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-tempest-plugin master: Testing curl succeeds from pod to the service https://review.openstack.org/558482 | 13:40 |
| *** fkautz has joined #openstack-kuryr | 13:40 | |
| *** fkautz has quit IRC | 13:41 | |
| *** fkautz has joined #openstack-kuryr | 13:41 | |
| *** fkautz has quit IRC | 13:42 | |
| *** fkautz has joined #openstack-kuryr | 13:43 | |
| *** fkautz has quit IRC | 13:44 | |
| *** fkautz has joined #openstack-kuryr | 13:44 | |
| *** fkautz has quit IRC | 13:45 | |
| dmellado | apuimedo: was having lunch | 13:45 |
| dmellado | what's up? | 13:46 |
| *** fkautz has joined #openstack-kuryr | 13:46 | |
| *** fkautz has quit IRC | 13:47 | |
| apuimedo | dmellado: I forgoet | 13:47 |
| apuimedo | *forgot | 13:47 |
| *** fkautz has joined #openstack-kuryr | 13:47 | |
| *** fkautz has quit IRC | 13:48 | |
| *** fkautz has joined #openstack-kuryr | 13:49 | |
| *** fkautz has quit IRC | 13:49 | |
| *** fkautz has joined #openstack-kuryr | 13:55 | |
| *** fkautz has quit IRC | 13:56 | |
| dmellado | apuimedo: lol | 13:56 |
| apuimedo | dmellado: your fault for not being around XD | 13:56 |
| *** atoth has joined #openstack-kuryr | 13:57 | |
| apuimedo | maybe it was good news or something | 13:57 |
| apuimedo | and now you'll never get them | 13:57 |
| apuimedo | but most likely it was about the rdo rpm build | 13:57 |
| apuimedo | xD | 13:57 |
| dmellado | xD | 13:57 |
| *** dims has quit IRC | 13:58 | |
| *** fkautz has joined #openstack-kuryr | 14:01 | |
| *** fkautz has quit IRC | 14:01 | |
| *** dims has joined #openstack-kuryr | 14:04 | |
| *** fkautz has joined #openstack-kuryr | 14:07 | |
| *** fkautz has quit IRC | 14:08 | |
| *** chenyb4 has joined #openstack-kuryr | 14:10 | |
| *** fkautz has joined #openstack-kuryr | 14:12 | |
| *** fkautz has quit IRC | 14:13 | |
| *** hongbin_ has joined #openstack-kuryr | 14:14 | |
| *** fkautz has joined #openstack-kuryr | 14:19 | |
| *** fkautz has quit IRC | 14:19 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 14:22 |
| *** fkautz has joined #openstack-kuryr | 14:24 | |
| *** fkautz has quit IRC | 14:25 | |
| *** chenyb4 has quit IRC | 14:27 | |
| *** fkautz has joined #openstack-kuryr | 14:31 | |
| *** fkautz has quit IRC | 14:31 | |
| apuimedo | ltomasbo: it improved | 14:34 |
| apuimedo | but it's still a strong -1 | 14:34 |
| *** fkautz has joined #openstack-kuryr | 14:36 | |
| ltomasbo | apuimedo, damn, you are fast! | 14:36 |
| ltomasbo | apuimedo, as long as it is not a strong -2! | 14:36 |
| apuimedo | ltomasbo: you are one strike away | 14:36 |
| *** fkautz has quit IRC | 14:37 | |
| ltomasbo | ups... | 14:37 |
| apuimedo | xD | 14:37 |
| apuimedo | I like where this is going thgouh | 14:37 |
| ltomasbo | apuimedo, thanks for the feedback, yep, it is cleaner the way you propose, I guess I was still in the 'saving calls' mode | 14:41 |
| apuimedo | ltomasbo: that single extra call you saved was evil | 14:42 |
| *** fkautz has joined #openstack-kuryr | 14:43 | |
| *** fkautz has quit IRC | 14:43 | |
| ltomasbo | xD | 14:44 |
| ltomasbo | apuimedo, I know, agree with you... | 14:44 |
| apuimedo | :-) | 14:45 |
| *** fkautz has joined #openstack-kuryr | 14:48 | |
| *** fkautz has quit IRC | 14:49 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Switch amphora tarball to ubuntu xenial https://review.openstack.org/562969 | 14:50 |
| *** kiennt26_ has quit IRC | 15:06 | |
| dmellado | Folks, I sent out an email related to the PTG attendance, could you please fill out the linked form? | 15:14 |
| dmellado | thanks! | 15:14 |
| apuimedo | dmellado: thanks | 15:15 |
| *** livelace-link has joined #openstack-kuryr | 15:16 | |
| apuimedo | dulek: it passed experimental | 15:16 |
| apuimedo | https://review.openstack.org/#/c/562322/ | 15:16 |
| apuimedo | please review and merge if you see fit | 15:16 |
| *** kiennt26_ has joined #openstack-kuryr | 15:16 | |
| dulek | apuimedo: Looking. | 15:17 |
| apuimedo | ltomasbo: you got some failures at https://review.openstack.org/#/c/562998/ | 15:17 |
| ltomasbo | apuimedo, yes | 15:18 |
| ltomasbo | apuimedo, but I rebase it on top of the other https://review.openstack.org/#/c/562322/ | 15:18 |
| *** kiennt26_ has quit IRC | 15:18 | |
| ltomasbo | ohh, check experimental already finished | 15:19 |
| ltomasbo | need to check then... | 15:19 |
| *** livelace-link has quit IRC | 15:19 | |
| apuimedo | indeed | 15:20 |
| *** livelace-link has joined #openstack-kuryr | 15:22 | |
| ltomasbo | apuimedo, seems to be a timeout... | 15:22 |
| apuimedo | dmellado: https://review.openstack.org/#/c/563137/ is taking long, eh? | 15:24 |
| *** fkautz has joined #openstack-kuryr | 15:26 | |
| dmellado | apuimedo: still ongoing | 15:26 |
| dmellado | check http://zuul.openstack.org/ | 15:26 |
| *** fkautz has quit IRC | 15:27 | |
| dmellado | apuimedo: it failed xD http://logs.openstack.org/37/563137/3/check/kuryr-kubernetes-tempest-lbaasv2-daemon/34e96ce/ | 15:33 |
| dmellado | apuimedo: http://logs.openstack.org/37/563137/3/check/kuryr-kubernetes-tempest-lbaasv2-daemon/34e96ce/logs/testr_results.html.gz | 15:34 |
| dmellado | somehow this is not respecting constraints | 15:34 |
| apuimedo | dmellado: meaning? | 15:35 |
| dmellado | see that issue about the service | 15:38 |
| dmellado | that's related to python-kubernetes version | 15:39 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add ovs-firewall coverage https://review.openstack.org/563008 | 15:39 |
| dmellado | I'd say we had a patch for dealing with this | 15:39 |
| dmellado | dulek: did you backport the UC one? | 15:39 |
| dulek | dmellado: ? | 15:40 |
| dulek | dmellado: Which one? | 15:40 |
| dmellado | oh, no, that was on devstack actually | 15:40 |
| dmellado | for the tempest plugins | 15:40 |
| dmellado | dammit | 15:40 |
| dmellado | dulek: do you recall the issue about python-kubernetes version on the tempest plugin going haywire? | 15:40 |
| dmellado | what devstack version runs with stable/queens? | 15:40 |
| dmellado | apuimedo: dulek: https://review.openstack.org/#/c/560946/ | 15:41 |
| dulek | apuimedo: This doesn't need to be backported, right? | 15:42 |
| *** maysamacedos has quit IRC | 15:43 | |
| dmellado | dulek: that's what I thought but check the logs | 15:43 |
| dmellado | it's installing the wrong version again | 15:43 |
| dulek | Ghhh. | 15:43 |
| dmellado | https://review.openstack.org/#/c/563137/ | 15:43 |
| dulek | Hm… | 15:44 |
| dulek | Wait. Okay, we haven't backported DevStack patches, okay. | 15:44 |
| dmellado | dulek: yeah | 15:44 |
| dmellado | I'd propose that as a backport | 15:44 |
| dmellado | let's see if I don't get complains | 15:44 |
| dulek | But how the heck did we merged rest of the patches then?! | 15:44 |
| dmellado | dulek: becasue the revert of the skip the service | 15:45 |
| dmellado | wasn't updated | 15:45 |
| dmellado | merged | 15:45 |
| dmellado | that's my suspect | 15:46 |
| dulek | dmellado: Damn, you're right. | 15:47 |
| * dulek knew something will break on Friday. | 15:47 | |
| *** fkautz has joined #openstack-kuryr | 15:50 | |
| *** fkautz has quit IRC | 15:50 | |
| *** dims has quit IRC | 16:06 | |
| *** dims has joined #openstack-kuryr | 16:11 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: devstack: Fix octavia api lb kubelet iface access https://review.openstack.org/562322 | 16:20 |
| *** fkautz has joined #openstack-kuryr | 16:28 | |
| *** fkautz has quit IRC | 16:29 | |
| apuimedo | what doesn't? | 16:29 |
| *** yamamoto has quit IRC | 16:31 | |
| *** maysamacedos has joined #openstack-kuryr | 16:37 | |
| *** fkautz has joined #openstack-kuryr | 16:37 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 16:40 |
| apuimedo | ltomasbo: did you try that already? | 16:44 |
| ltomasbo | the namespace creation? | 16:45 |
| ltomasbo | apuimedo, ^^ | 16:45 |
| apuimedo | yes | 16:45 |
| ltomasbo | yes, I was rebasing the deletion and adapting it to the modifications for the creation | 16:45 |
| ltomasbo | I tested it with lbaasv2 though | 16:45 |
| ltomasbo | but it should not matter | 16:45 |
| apuimedo | cool | 16:46 |
| apuimedo | sometimes the separation between handler and driver feels clunky to me xD | 16:46 |
| ltomasbo | apuimedo, still, once the design is ok, I need to add testing | 16:46 |
| apuimedo | like it could be just a single module | 16:46 |
| ltomasbo | ohh, yes | 16:46 |
| apuimedo | ltomasbo: dulek: dmellado: please read https://docs.google.com/presentation/d/1C3r91ev0tWnFFUjiV4W84Hp965YGR1D9lChZo73Jwq0/edit#slide=id.g37632803a9_34_365 | 16:48 |
| apuimedo | it is quite interesting | 16:48 |
| ltomasbo | apuimedo, thanks! I'll read it later | 16:49 |
| apuimedo | ltomasbo: it's about services mesh AND multidevice | 16:50 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 17:07 |
| *** pcaruana has quit IRC | 17:22 | |
| *** yamamoto has joined #openstack-kuryr | 17:32 | |
| *** pcaruana has joined #openstack-kuryr | 17:33 | |
| *** yamamoto has quit IRC | 17:38 | |
| *** pcaruana has quit IRC | 18:29 | |
| *** yamamoto has joined #openstack-kuryr | 18:34 | |
| *** yamamoto has quit IRC | 18:38 | |
| *** apuimedo has quit IRC | 18:40 | |
| *** jchhatbar has quit IRC | 18:41 | |
| *** apuimedo has joined #openstack-kuryr | 18:44 | |
| *** yamamoto has joined #openstack-kuryr | 19:34 | |
| *** yamamoto has quit IRC | 19:39 | |
| *** atoth has quit IRC | 19:58 | |
| *** maysamacedos has quit IRC | 20:31 | |
| *** yamamoto has joined #openstack-kuryr | 20:36 | |
| *** yamamoto has quit IRC | 20:41 | |
| *** yamamoto has joined #openstack-kuryr | 20:57 | |
| *** premsankar has joined #openstack-kuryr | 20:58 | |
| *** yamamoto has quit IRC | 21:04 | |
| *** yamamoto has joined #openstack-kuryr | 21:20 | |
| *** yamamoto has quit IRC | 21:25 | |
| *** maysamacedos has joined #openstack-kuryr | 21:27 | |
| *** yamamoto has joined #openstack-kuryr | 21:41 | |
| *** yamamoto has quit IRC | 21:47 | |
| *** yamamoto has joined #openstack-kuryr | 22:04 | |
| *** yamamoto has quit IRC | 22:08 | |
| *** apuimedo has quit IRC | 22:12 | |
| *** yamamoto has joined #openstack-kuryr | 22:24 | |
| *** yamamoto has quit IRC | 22:29 | |
| *** yamamoto has joined #openstack-kuryr | 22:45 | |
| *** yamamoto has quit IRC | 22:51 | |
| *** yamamoto has joined #openstack-kuryr | 23:08 | |
| *** yamamoto has quit IRC | 23:12 | |
| *** yamamoto has joined #openstack-kuryr | 23:20 | |
| *** hongbin_ has quit IRC | 23:23 | |
| *** yamamoto has quit IRC | 23:32 | |
| *** yamamoto has joined #openstack-kuryr | 23:33 | |
| *** salv-orl_ has joined #openstack-kuryr | 23:45 | |
| *** salv-orlando has quit IRC | 23:48 | |
| openstackgerrit | melissaml proposed openstack/kuryr master: Trivial: Update pypi url to new url https://review.openstack.org/563340 | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!