| *** yamamoto has joined #openstack-kuryr | 00:20 | |
| *** yamamoto has quit IRC | 00:24 | |
| *** yamamoto has joined #openstack-kuryr | 00:41 | |
| *** fabian has joined #openstack-kuryr | 01:07 | |
| *** salv-orlando has joined #openstack-kuryr | 01:28 | |
| *** salv-orl_ has quit IRC | 01:31 | |
| *** kiennt2609 has joined #openstack-kuryr | 01:48 | |
| *** kiennt2609 is now known as kiennt26_ | 01:48 | |
| *** fabian is now known as chenyb4 | 01:50 | |
| *** kiennt26_ has quit IRC | 01:55 | |
| *** dangtrinhnt has quit IRC | 03:09 | |
| *** hongbin_ has quit IRC | 03:50 | |
| *** janonymous has joined #openstack-kuryr | 04:51 | |
| *** yboaron has joined #openstack-kuryr | 05:03 | |
| *** gcheresh has joined #openstack-kuryr | 05:24 | |
| *** gcheresh_ has joined #openstack-kuryr | 05:31 | |
| *** gcheresh has quit IRC | 05:31 | |
| *** pmannidi has quit IRC | 05:37 | |
| *** chenyb4 has quit IRC | 06:45 | |
| *** chenyb4 has joined #openstack-kuryr | 06:48 | |
| *** yboaron has quit IRC | 06:59 | |
| *** caowei has joined #openstack-kuryr | 07:00 | |
| *** janki has joined #openstack-kuryr | 07:15 | |
| *** jchhatbar has joined #openstack-kuryr | 07:21 | |
| *** janki has quit IRC | 07:23 | |
| *** pcaruana has joined #openstack-kuryr | 07:25 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add better octavia coverage at the gate https://review.openstack.org/562998 | 07:27 |
|---|---|---|
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add ovs-firewall coverage https://review.openstack.org/563008 | 07:40 |
| *** livelace has joined #openstack-kuryr | 07:51 | |
| *** livelace has quit IRC | 07:51 | |
| *** threestrands has quit IRC | 08:17 | |
| *** yboaron has joined #openstack-kuryr | 08:34 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Apply ovs-firewall in openshift gates https://review.openstack.org/563524 | 08:38 |
| *** garyloug has joined #openstack-kuryr | 08:44 | |
| *** apuimedo has joined #openstack-kuryr | 09:02 | |
| ltomasbo | dulek, dmellado: https://review.openstack.org/#/c/562998/4 | 09:05 |
| ltomasbo | dmellado, dulek: it is working with the ubuntu image instead of the centos one | 09:06 |
| dmellado | ltomasbo: no wonder, bcafarel-- xD | 09:06 |
| dmellado | ltomasbo: btw, looks like this might work | 09:09 |
| dmellado | https://review.openstack.org/#/c/563524/1/.zuul.yaml | 09:09 |
| ltomasbo | dmellado, ok, I'll give it a try! | 09:11 |
| dmellado | ltomasbo: hold for the gate to pass, though | 09:11 |
| ltomasbo | dmellado, ahh, you sent a new patch! you should have modified mine! XD | 09:12 |
| dmellado | as I wasn't confident on mine either | 09:12 |
| dmellado | I wanted to test as twice as fast xD | 09:12 |
| dmellado | so let's hold on | 09:12 |
| ltomasbo | xD | 09:12 |
| dmellado | btw, folks, as soon as we get to merge https://review.openstack.org/#/c/563172/ | 09:14 |
| dmellado | we'll release xD | 09:14 |
| ltomasbo | great! | 09:16 |
| dmellado | argh it failed too | 09:39 |
| dulek | ltomasbo, dmellado: I'm a bit scared about adding any new tests before we do the stable release. | 09:42 |
| dmellado | huh, makes sense | 09:44 |
| dmellado | I've withheld my w+1 | 09:44 |
| dmellado | ltomasbo: could you split the test onto another patch? | 09:44 |
| apuimedo | dulek: good point | 09:44 |
| dmellado | dulek: btw, looks like the backport for devstack will merge soon | 09:44 |
| dmellado | we'll release just after | 09:44 |
| dulek | Fingers crossed. ;) | 09:45 |
| apuimedo | dmellado: how soon? | 09:45 |
| dmellado | as soon as some us folks wake up | 09:45 |
| apuimedo | ok | 09:46 |
| dulek | dmellado: Isn't andfreaf available to review that backport? | 09:47 |
| dmellado | dulek: I'm afraid he isn't online :\ | 09:47 |
| dmellado | already pinged him without reply | 09:47 |
| dulek | Ah, right, ~70 hours idel. | 09:48 |
| dulek | idle. | 09:48 |
| ltomasbo | dmellado, split which one? | 09:48 |
| dmellado | ltomasbo: your octavia patch which adds gate + tests | 09:49 |
| ltomasbo | dmellado, why? | 09:49 |
| dmellado | either that or we withhold it | 09:49 |
| dmellado | I'm keen on going with dulek 's suggestion and not adding new tests before the release | 09:50 |
| ltomasbo | dmellado, I'm fine waiting for the release | 09:50 |
| ltomasbo | dmellado, though merge on master branch should not break stable/queens, right? | 09:50 |
| dmellado | shouldn't, but we might want to backport some stuff, so let's hold on | 09:51 |
| ltomasbo | dmellado, yep, no problem on holding after release, and yes, it will be nice to backport the octavia tests | 09:51 |
| dulek | Yup, it seems it is 100% safe, but in the end we've broke stable last Friday because of unforeseen consequences. Let's just wait this little bit. | 09:51 |
| dmellado | dulek: lol, lasst Friday was terrible xD | 09:52 |
| ltomasbo | xD | 09:52 |
| dulek | Waiting will not hurt us, I think. :) | 09:52 |
| ltomasbo | dulek, yep yep, +1 on waiting | 09:52 |
| ltomasbo | there is no hurry at all, specially if we are holding changes | 09:53 |
| dmellado | I was having nightmares with that and also with the 2012 cpus | 09:53 |
| openstackgerrit | Gary Loughnane proposed openstack/kuryr-kubernetes master: Add DPDK support for nested pods https://review.openstack.org/559363 | 09:53 |
| dmellado | apuimedo: I had a dream where we received servers with this micros | 09:53 |
| dmellado | https://upload.wikimedia.org/wikipedia/commons/e/e5/Intel_DX4_microprocessor.jpg | 09:53 |
| apuimedo | :'( | 09:55 |
| dmellado | apuimedo: it could be worse | 09:56 |
| dmellado | https://upload.wikimedia.org/wikipedia/commons/7/78/KL_intel_i486SX2.jpg | 09:56 |
| dmellado | without math co-processor! xD | 09:56 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 10:07 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 10:07 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add namespace subnet driver for namespace creation https://review.openstack.org/562247 | 10:13 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 10:13 |
| *** chenyb4 has quit IRC | 10:13 | |
| apuimedo | ltomasbo: what do we need the base class for https://review.openstack.org/#/c/562159/1 ? | 10:18 |
| * ltomasbo checking | 10:18 | |
| ltomasbo | you mean the K8s_base? | 10:20 |
| ltomasbo | apuimedo, ^^ | 10:20 |
| apuimedo | ltomasbo: I mean the whole patch | 10:20 |
| apuimedo | I don't see why we need this having the other patch | 10:20 |
| ltomasbo | apuimedo, to have a handler to listen for namespace events | 10:21 |
| ltomasbo | apuimedo, having what patch? I tried to split it in different patch sets to make reviews easier | 10:21 |
| ltomasbo | apuimedo, 1 patch adding base handler, 1 patch adding driver (for creating namespaces) and 1 for deleting | 10:22 |
| apuimedo | ltomasbo: just a sec | 10:22 |
| ltomasbo | apuimedo, it could be just in one, but it will end up being more than 1000 new lines, which usually make it harder to review | 10:22 |
| apuimedo | ltomasbo: ok. It's a bit weird but I'll take it | 10:23 |
| apuimedo | from the name of the first patch I thought for a moment that you made a base class and an inheriting handler class taht actually does the work | 10:23 |
| ltomasbo | xD, I always try to split patch sets as much as possible | 10:23 |
| apuimedo | I'm glad it's not the case | 10:23 |
| apuimedo | ltomasbo: that's good | 10:23 |
| ltomasbo | xD | 10:24 |
| ltomasbo | no, it is a simple handler (for now at least), so no need to inheriting and enable different implementations of it | 10:25 |
| ltomasbo | apuimedo, ^^ that could be added on the drivers it uses instead (to keep the same design we have) | 10:25 |
| *** pcaruana has quit IRC | 10:26 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Apply ovs-firewall in openshift gates https://review.openstack.org/563524 | 10:28 |
| *** yamamoto has quit IRC | 10:31 | |
| *** pcaruana has joined #openstack-kuryr | 10:33 | |
| *** mfedosin has quit IRC | 11:08 | |
| dmellado | apuimedo: dulek awesome | 11:22 |
| dmellado | it just merged | 11:22 |
| dmellado | so we can go with the release once it passes the gates | 11:22 |
| dulek | dmellado: Okay, it's going through gate, then we'll recheck https://review.openstack.org/#/c/563137/ and then we can do the release. | 11:23 |
| dmellado | eexactly | 11:23 |
| dmellado | \o/ | 11:23 |
| dmellado | then we'll go get beers and take the afternoon off | 11:24 |
| * dmellado sighs, if only xD | 11:24 | |
| *** caowei has quit IRC | 11:30 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Apply ovs-firewall in openshift gates https://review.openstack.org/563524 | 11:31 |
| *** yamamoto has joined #openstack-kuryr | 11:31 | |
| *** mfedosin has joined #openstack-kuryr | 11:37 | |
| *** yamamoto has quit IRC | 11:37 | |
| irenab | dmellado, hi | 11:54 |
| *** pcaruana has quit IRC | 11:54 | |
| *** atoth has joined #openstack-kuryr | 11:56 | |
| dmellado | hey irenab o/ | 11:58 |
| irenab | dmellado, I won't be able to attend weekly, at least first 20 mins, have to attend school meeting | 12:00 |
| dmellado | irenab: no worries, thanks for noticing me | 12:00 |
| dmellado | irenab: is there any topic you wanted to tackle? | 12:01 |
| irenab | dmellado, nothing specific | 12:01 |
| irenab | maybe just attention to network policies | 12:01 |
| irenab | if we decide to try and merge it | 12:02 |
| dmellado | irenab: actually I wanted to sync with you about that as well | 12:02 |
| dmellado | I intend to take over the patches but didn't have time until now | 12:02 |
| dmellado | so no worries, we'll wait for you | 12:02 |
| dmellado | remaining topics will be mainly release related | 12:02 |
| dmellado | so take it easy | 12:02 |
| apuimedo | dmellado: did you read that thread I sent you on network policies? | 12:04 |
| dmellado | apuimedo: overall yeah, please do put the link again for when we discuss that later | 12:04 |
| dmellado | apuimedo: btw, isn't it a day off in catalonia today? | 12:05 |
| apuimedo | dmellado: it is not. Catalans work even in their Saint Patron day | 12:05 |
| apuimedo | it's tradition | 12:05 |
| *** yamamoto has joined #openstack-kuryr | 12:05 | |
| dmellado | lol | 12:05 |
| apuimedo | dmellado: funnily enough it is festive in Aragon | 12:06 |
| apuimedo | xD | 12:06 |
| *** yamamoto has quit IRC | 12:06 | |
| dmellado | pseudo catalonians? /me flees xD | 12:06 |
| apuimedo | dmellado: no, no. Splitters | 12:06 |
| apuimedo | https://www.youtube.com/watch?v=iS-0Az7dgRY | 12:07 |
| dmellado | catalonians then xD | 12:07 |
| *** pcaruana has joined #openstack-kuryr | 12:10 | |
| irenab | apuimedo, share the link with me as well please | 12:20 |
| apuimedo | irenab: if I find it | 12:21 |
| apuimedo | :-) | 12:21 |
| dmellado | irenab: apuimedo | 12:22 |
| dmellado | https://groups.google.com/forum/#!topic/kubernetes-sig-network/A8oyMrWpJ8Q | 12:22 |
| apuimedo | thanks dmellado | 12:24 |
| irenab | thanks | 12:24 |
| *** yamamoto has joined #openstack-kuryr | 12:37 | |
| *** jchhatbar is now known as janki | 12:37 | |
| *** yamamoto has quit IRC | 12:42 | |
| yboaron | ping ltomasbo | 12:49 |
| ltomasbo | pong yboaron | 12:50 |
| yboaron | ltomasbo, regarding https://review.openstack.org/#/c/562247/ , thanks for the detailed answer! | 12:50 |
| yboaron | ltomasbo, So, the 'Kuryr-net' CRD will serve as the contract between the namespace handler and the new driver, | 12:50 |
| * ltomasbo remembering what I answered... | 12:51 | |
| *** yamamoto has joined #openstack-kuryr | 12:51 | |
| yboaron | ltomasbo, and isn't be part of IM (information model) | 12:51 |
| yboaron | ltomasbo, that make sense | 12:51 |
| ltomasbo | yboaron, yep, the idea about using CRDs is to store the information at the kubernetes side, rather than neutron | 12:52 |
| yboaron | ltomasbo, so at the bottom line - I agree with your approach - I knew that I'm missing something wasn't sure what .. :-) | 12:52 |
| ltomasbo | and yes, I have yet to send an extra patch to manage the ports pool deletion, using the CRDs at the handler to get the right keys | 12:53 |
| yboaron | ltomasbo, cool! | 12:53 |
| ltomasbo | yboaron, thanks for taking a look! and provide feedback! | 12:53 |
| yboaron | ltomasbo, you're welcome! | 12:54 |
| ltomasbo | yboaron, I send those patch before the tests to also have time to discuss and agree on the final way we wnat it | 12:54 |
| ltomasbo | yboaron, and the main idea was to try to have it well isolated (new handler, new driver) | 12:54 |
| ltomasbo | so it will be easier to load istio driver, or community upstream driver without this one if they contradict | 12:55 |
| *** yamamoto has quit IRC | 12:55 | |
| yboaron | ltomasbo, yep , sounds good | 12:56 |
| ltomasbo | great! | 12:56 |
| dmellado | dulek: you were faster than me on the recheck | 13:10 |
| dmellado | awesome | 13:10 |
| dmellado | let's see if this goes smooth this time | 13:10 |
| dulek | dmellado: I was following the patch on zuul.openstack.org. | 13:10 |
| dmellado | :P | 13:10 |
| dmellado | heh, great, I just was about to hit recheck as soon as I got the email from zuul | 13:10 |
| dmellado | nevertheless, let's see if this goes thru and we can finally do the release | 13:11 |
| *** janonymous has quit IRC | 13:27 | |
| *** yboaron has quit IRC | 13:48 | |
| *** yboaron has joined #openstack-kuryr | 13:48 | |
| *** yamamoto has joined #openstack-kuryr | 13:51 | |
| *** fabian has joined #openstack-kuryr | 13:54 | |
| *** rh-jelabarre has quit IRC | 13:55 | |
| *** yamamoto has quit IRC | 13:57 | |
| *** rh-jelabarre has joined #openstack-kuryr | 14:03 | |
| *** hongbin_ has joined #openstack-kuryr | 14:07 | |
| *** fabian has quit IRC | 14:09 | |
| dulek | dmellado: Can you add your +W on https://review.openstack.org/#/c/563137 ? Somehow Zuul is not catching mine. | 14:18 |
| dmellado | heh, let's try | 14:19 |
| dulek | dmellado: Nevermind, it got it. | 14:19 |
| dmellado | I guess it's just slightly overloaded | 14:19 |
| dulek | Yup. | 14:19 |
| *** maysamacedos has joined #openstack-kuryr | 14:24 | |
| dulek | dmellado: https://review.openstack.org/563084 | 14:52 |
| dmellado | dulek: ack there | 14:52 |
| dmellado | and all the appropiate stakeholders added as reviewers | 14:52 |
| dmellado | good work everyone! | 14:52 |
| dmellado | \o/ | 14:52 |
| *** yamamoto has joined #openstack-kuryr | 14:53 | |
| *** yamamoto has quit IRC | 14:58 | |
| *** yboaron has quit IRC | 15:04 | |
| *** janki has quit IRC | 15:17 | |
| *** pcaruana has quit IRC | 15:19 | |
| *** gcheresh_ has quit IRC | 15:20 | |
| *** premsankar has joined #openstack-kuryr | 15:25 | |
| *** caisan has joined #openstack-kuryr | 15:37 | |
| *** maysamacedos has quit IRC | 15:49 | |
| *** yamamoto has joined #openstack-kuryr | 15:54 | |
| *** yamamoto has quit IRC | 16:00 | |
| *** caisan has quit IRC | 16:01 | |
| *** maysamacedos has joined #openstack-kuryr | 16:08 | |
| *** pcaruana has joined #openstack-kuryr | 16:13 | |
| openstackgerrit | Merged openstack/kuryr-libnetwork master: Allow skip processing exposed ports https://review.openstack.org/563442 | 16:37 |
| *** garyloug has quit IRC | 16:40 | |
| openstackgerrit | Merged openstack/kuryr-libnetwork master: Support running subset of unit tests https://review.openstack.org/563434 | 16:44 |
| *** gcheresh_ has joined #openstack-kuryr | 16:53 | |
| *** yamamoto has joined #openstack-kuryr | 16:56 | |
| *** yamamoto has quit IRC | 17:01 | |
| *** gcheresh_ has quit IRC | 17:28 | |
| *** maysamacedos has quit IRC | 17:45 | |
| *** maysamacedos has joined #openstack-kuryr | 17:45 | |
| *** yamamoto has joined #openstack-kuryr | 17:58 | |
| *** yamamoto has quit IRC | 18:02 | |
| *** yamamoto has joined #openstack-kuryr | 18:59 | |
| *** yamamoto has quit IRC | 19:03 | |
| *** atoth has quit IRC | 19:20 | |
| *** yamamoto has joined #openstack-kuryr | 19:59 | |
| *** yamamoto has quit IRC | 20:05 | |
| *** gcheresh_ has joined #openstack-kuryr | 20:09 | |
| *** dougbtv_ has joined #openstack-kuryr | 20:37 | |
| *** gcheresh_ has quit IRC | 20:39 | |
| *** dougbtv has quit IRC | 20:40 | |
| *** yboaron has joined #openstack-kuryr | 20:57 | |
| *** dougbtv__ has joined #openstack-kuryr | 21:01 | |
| *** dougbtv_ has quit IRC | 21:03 | |
| *** maysamacedos has quit IRC | 21:35 | |
| *** yboaron has quit IRC | 21:51 | |
| *** yamamoto has joined #openstack-kuryr | 22:02 | |
| *** yamamoto has quit IRC | 22:07 | |
| *** maysamacedos has joined #openstack-kuryr | 22:17 | |
| *** threestrands has joined #openstack-kuryr | 22:41 | |
| *** threestrands has quit IRC | 22:41 | |
| *** threestrands has joined #openstack-kuryr | 22:41 | |
| *** rh-jelabarre has quit IRC | 22:57 | |
| *** yamamoto has joined #openstack-kuryr | 23:04 | |
| *** yamamoto has quit IRC | 23:10 | |
| *** yamamoto has joined #openstack-kuryr | 23:26 | |
| *** hongbin_ has quit IRC | 23:27 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
