| opendevreview | Michael Johnson proposed openstack/octavia master: Enable nftables rules for SR-IOV VIPs https://review.opendev.org/c/openstack/octavia/+/910101 | 00:56 |
|---|---|---|
| johnsom | Hmm, looks like pylint updated as I'm getting pep8 failures on code I didn't change. | 01:34 |
| johnsom | https://zuul.opendev.org/t/openstack/build/15ac76c24c3e4b06972dab2e6f5da35c | 01:35 |
| johnsom | I'm pretty sure I reviewed a patch that changed that on Friday, but it must not have merged | 01:35 |
| johnsom | Yeah, the pyupgrade patch fixed those issues | 01:46 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Cap pylint https://review.opendev.org/c/openstack/octavia/+/910133 | 04:02 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Bump pylint https://review.opendev.org/c/openstack/octavia/+/910134 | 04:06 |
| opendevreview | Takashi Kajinami proposed openstack/octavia-lib master: Cap pylint https://review.opendev.org/c/openstack/octavia-lib/+/910135 | 04:07 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Fix pylint error https://review.opendev.org/c/openstack/octavia/+/910133 | 04:16 |
| skraynev_ | Hello, could someone please help with review couple backports: https://review.opendev.org/c/openstack/octavia/+/909824, https://review.opendev.org/c/openstack/octavia/+/909825 | 06:35 |
| skraynev_ | and one new patch with graph optimizations: https://review.opendev.org/c/openstack/octavia/+/883063 | 06:35 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fixed new pep8 warnings https://review.opendev.org/c/openstack/octavia/+/910142 | 07:41 |
| tkajinam | gthiemonge, I've already submitted the same fix but can abandon it if you want https://review.opendev.org/c/openstack/octavia/+/910133 | 07:52 |
| tkajinam | we probably have to pin pylint in stable branches instead of backprting it. | 07:52 |
| gthiemonge | tkajinam: oh sorry I didn't see it, I'll abandon my patch | 07:53 |
| tkajinam | gthiemonge, np ! | 07:54 |
| gthiemonge | I have to check how my gerrit notifications are configured, I didn't get any email about it | 07:55 |
| gthiemonge | tkajinam: thanks for the patch! | 07:56 |
| gthiemonge | tkajinam: why do you pin pylint? (<3.2.0) | 07:57 |
| tkajinam | gthiemonge, oh, let me remove it now | 07:58 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Fix pylint error https://review.opendev.org/c/openstack/octavia/+/910133 | 07:58 |
| tkajinam | though we may want to pin it eventually to avoid updating stable branches later | 07:58 |
| tkajinam | we already cap hacking for example for the same purpose | 07:59 |
| gthiemonge | I think it's ok to not cap it, we'll probably forget to uncap it for new pylint releases | 08:03 |
| gthiemonge | tkajinam: ok, I'm going to approve your patch right now, it may not merge quickly because of the other CI issues (but I'm working on fixes for the other stuff) | 08:04 |
| tkajinam | gthiemonge, I'm ok with keeping it uncapped though we may want to revisit if after releases | 08:27 |
| tkajinam | gthiemonge, do you want me to backport it to stable branches ? I personally prefer cap for stable branches to fixing these new lint errors there but want to hear your thoughts on it before pushing a change. | 08:28 |
| tkajinam | this may not be super urgent as we may have to wait until that fix lands in master, so can wait for any feedback from other cores. | 08:28 |
| gthiemonge | tkajinam: we usually cap it on stable branches | 08:28 |
| tkajinam | yeah | 08:29 |
| tkajinam | ok | 08:29 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Check haproxy status on reload https://review.opendev.org/c/openstack/octavia/+/909849 | 08:38 |
| opendevreview | Bodo Petermann proposed openstack/octavia master: Health sender: support to send via TCP https://review.opendev.org/c/openstack/octavia/+/852269 | 15:58 |
| gthiemonge | The CI should be in a better shape when https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/861743 merges | 16:37 |
| spatel | johnsom hey! | 18:37 |
| johnsom | Hi | 18:38 |
| spatel | I am running kolla-ansible and I have LBaaS nic bond0.15 | 18:39 |
| spatel | I have create lb-mgmt-net using VLAN 15 | 18:40 |
| spatel | I saw it created netns namespace and assign IP for DHCP agent | 18:40 |
| spatel | I noticed dhcp-agent namespace IP not able to ping or communicate (so amphora not getting IP) | 18:41 |
| spatel | As soon as I remove bond0.15 on host machine dhcp namespace start pinging... | 18:41 |
| spatel | This is not a octavia issue but what is the solution here :( | 18:41 |
| spatel | This is kernel namespace apr issue I believe.... | 18:42 |
| johnsom | I don’t know how kolla wires up those networks. | 18:43 |
| spatel | Hmm! | 18:43 |
| spatel | Let me try one more solution.. | 18:43 |
| spatel | all my other deployment using OVN so its totally different beast without namespace :) | 18:44 |
| spatel | In this deployment I am using OVS which use namespace | 18:44 |
| opendevreview | Merged openstack/octavia-tempest-plugin master: Add missing cleanups after octavia resources creation https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/861743 | 18:54 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org will be restarted to perform a minor upgrade to the service. | 22:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!