| *** yamamoto has joined #openstack-neutron | 00:08 | |
| *** hamalq has quit IRC | 00:09 | |
| openstackgerrit | lkk proposed openstack/neutron-tempest-plugin master: case for update qos rule, rule or policy non-existent https://review.opendev.org/739906 | 00:56 |
|---|---|---|
| *** gouthamr_ has quit IRC | 00:59 | |
| *** gouthamr_ has joined #openstack-neutron | 01:05 | |
| *** gouthamr_ has quit IRC | 01:25 | |
| *** gouthamr_ has joined #openstack-neutron | 01:37 | |
| *** Yumeng has joined #openstack-neutron | 02:01 | |
| *** armax has quit IRC | 02:01 | |
| openstackgerrit | Li YaJie proposed openstack/neutron master: Fix delete-accepted-egress-direct-flow issue on deleting VM https://review.opendev.org/731446 | 02:02 |
| *** yangyapeng has joined #openstack-neutron | 02:33 | |
| *** gouthamr_ has quit IRC | 02:52 | |
| *** gouthamr_ has joined #openstack-neutron | 02:56 | |
| *** rcernin has quit IRC | 03:00 | |
| *** jmlowe has quit IRC | 03:01 | |
| *** rcernin has joined #openstack-neutron | 03:03 | |
| *** HenryG has quit IRC | 03:06 | |
| *** HenryG has joined #openstack-neutron | 03:07 | |
| *** gouthamr_ has quit IRC | 03:26 | |
| *** gouthamr_ has joined #openstack-neutron | 03:32 | |
| *** yangyapeng_ has joined #openstack-neutron | 03:46 | |
| *** ociuhandu has joined #openstack-neutron | 03:56 | |
| *** gouthamr_ has quit IRC | 04:00 | |
| *** ociuhandu has quit IRC | 04:01 | |
| *** markvoelker has joined #openstack-neutron | 04:08 | |
| *** gouthamr_ has joined #openstack-neutron | 04:11 | |
| *** ratailor has joined #openstack-neutron | 04:22 | |
| *** gouthamr_ has quit IRC | 04:29 | |
| *** gouthamr_ has joined #openstack-neutron | 04:30 | |
| *** gouthamr_ has quit IRC | 04:35 | |
| *** gouthamr_ has joined #openstack-neutron | 04:40 | |
| openstackgerrit | Li YaJie proposed openstack/neutron master: Fix delete-accepted-egress-direct-flow issue on deleting VM https://review.opendev.org/731446 | 04:50 |
| openstackgerrit | melissaml proposed openstack/networking-bagpipe master: Switch from unittest2 compat methods to Python 3.x methods https://review.opendev.org/740155 | 04:59 |
| *** abdysn has joined #openstack-neutron | 05:02 | |
| openstackgerrit | melissaml proposed openstack/networking-bgpvpn master: Switch from unittest2 compat methods to Python 3.x methods https://review.opendev.org/740157 | 05:02 |
| *** Luzi has joined #openstack-neutron | 05:12 | |
| *** markvoelker has quit IRC | 05:13 | |
| *** markvoelker has joined #openstack-neutron | 05:14 | |
| openstackgerrit | melissaml proposed openstack/networking-sfc master: Switch from unittest2 compat methods to Python 3.x methods https://review.opendev.org/740158 | 05:17 |
| *** markvoelker has quit IRC | 05:18 | |
| *** Luzi has quit IRC | 05:19 | |
| *** zzzeek has quit IRC | 05:19 | |
| *** Luzi has joined #openstack-neutron | 05:19 | |
| *** slaweq has joined #openstack-neutron | 05:20 | |
| *** zzzeek has joined #openstack-neutron | 05:23 | |
| *** slaweq has quit IRC | 05:24 | |
| *** lxkong has joined #openstack-neutron | 05:27 | |
| *** gcheresh has joined #openstack-neutron | 05:28 | |
| *** lajoskatona has joined #openstack-neutron | 05:36 | |
| *** psahoo has joined #openstack-neutron | 05:56 | |
| *** ralonsoh has joined #openstack-neutron | 06:10 | |
| *** lajoskatona has quit IRC | 06:32 | |
| *** slaweq has joined #openstack-neutron | 06:39 | |
| *** lajoskatona has joined #openstack-neutron | 06:44 | |
| *** CeeMac has joined #openstack-neutron | 06:44 | |
| *** yamamoto has quit IRC | 06:47 | |
| *** yamamoto has joined #openstack-neutron | 06:48 | |
| *** gouthamr_ has quit IRC | 06:50 | |
| *** gouthamr_ has joined #openstack-neutron | 06:52 | |
| *** knikolla has quit IRC | 06:54 | |
| *** lajoskatona has quit IRC | 06:54 | |
| *** slaweq has quit IRC | 06:55 | |
| *** ildikov has quit IRC | 06:55 | |
| *** slaweq has joined #openstack-neutron | 06:55 | |
| *** arjunbaindur has quit IRC | 06:56 | |
| *** knikolla has joined #openstack-neutron | 06:56 | |
| *** slaweq has quit IRC | 06:56 | |
| *** ildikov has joined #openstack-neutron | 06:58 | |
| *** gouthamr_ has quit IRC | 07:04 | |
| *** gouthamr_ has joined #openstack-neutron | 07:05 | |
| *** lajoskatona has joined #openstack-neutron | 07:07 | |
| frickler | jamespage: I guess there's your regression again, I give up fighting windmills https://review.opendev.org/731624 | 07:15 |
| *** gouthamr_ has quit IRC | 07:15 | |
| *** jlibosva has joined #openstack-neutron | 07:17 | |
| *** tesseract has joined #openstack-neutron | 07:18 | |
| *** gouthamr_ has joined #openstack-neutron | 07:22 | |
| *** tmorin has joined #openstack-neutron | 07:27 | |
| *** bengates has joined #openstack-neutron | 07:32 | |
| *** bengates has quit IRC | 07:37 | |
| *** bengates has joined #openstack-neutron | 07:38 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron master: Optionally use admin powers when deleting DNS records https://review.opendev.org/728385 | 07:38 |
| *** bengates_ has joined #openstack-neutron | 07:39 | |
| *** bengates has quit IRC | 07:43 | |
| *** markvoelker has joined #openstack-neutron | 07:43 | |
| *** gouthamr_ has quit IRC | 07:46 | |
| *** slaweq has joined #openstack-neutron | 07:47 | |
| *** markvoelker has quit IRC | 07:48 | |
| *** gouthamr_ has joined #openstack-neutron | 07:52 | |
| *** tmorin has quit IRC | 07:54 | |
| *** tmorin has joined #openstack-neutron | 07:54 | |
| *** gouthamr_ has quit IRC | 07:57 | |
| *** gmann has quit IRC | 08:03 | |
| *** fyx has quit IRC | 08:04 | |
| frickler | slaweq: I updated 728385 and now I have a question about the tempest test for it: should I wait until the fix is backported to stable releases (not sure if it's actually backportable) or skip the test on stable branches? | 08:04 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Convert neutron-grenade-ovn job to be zuulv3 https://review.opendev.org/729591 | 08:04 |
| *** fyx has joined #openstack-neutron | 08:04 | |
| *** v1k0d3n has quit IRC | 08:05 | |
| *** gmann has joined #openstack-neutron | 08:06 | |
| *** v1k0d3n has joined #openstack-neutron | 08:07 | |
| *** gouthamr_ has joined #openstack-neutron | 08:09 | |
| *** iurygregory has quit IRC | 08:10 | |
| openstackgerrit | Bence Romsics proposed openstack/neutron-lib master: Metadata addresses https://review.opendev.org/738205 | 08:12 |
| *** lucasagomes has joined #openstack-neutron | 08:15 | |
| *** iurygregory has joined #openstack-neutron | 08:24 | |
| ralonsoh | frickler, https://review.opendev.org/#/c/728385/2..4/neutron/services/externaldns/drivers/designate/driver.py | 08:25 |
| ralonsoh | one question | 08:25 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: Test internal dns with external dns-integration https://review.opendev.org/740186 | 08:25 |
| ralonsoh | I'm concerned about this double query, first with the normal client and then with "all_projects" | 08:26 |
| ralonsoh | why don't we use the admin_context there? | 08:26 |
| frickler | ralonsoh: because the admin_context doesn't have admin privileges | 08:26 |
| *** yangyapeng__ has joined #openstack-neutron | 08:27 | |
| frickler | it is a normal user account | 08:27 |
| frickler | at least in current deployments | 08:27 |
| ralonsoh | frickler, and who is admin? | 08:28 |
| frickler | in that scenarion, an admin is the API user submitting the delete | 08:28 |
| frickler | -n | 08:28 |
| *** maciejjozefczyk has joined #openstack-neutron | 08:28 | |
| frickler | so in a devstack deployment, you have a resource created by the demo user, and then e.g. do "openstack --os-cloud devstack-admin port delete xy" | 08:29 |
| frickler | and then neutron essentially does "openstack zone list" with those credentials, which cannot see the zone belonging to the demo project | 08:30 |
| ralonsoh | frickler, sorry but I still don't see how it works | 08:32 |
| frickler | o.k., so let's start from the start. user demo creates zone test.com, sets --dns-domain test.com. on private network and --dns-publish-fixed-ip on the private subnets | 08:34 |
| frickler | then user creates port p1 in that network. neutron dns-integration creates records p1.test.com in test.com domain as that user | 08:34 |
| ralonsoh | ok | 08:35 |
| frickler | if the user deletes p1, neutron uses that user's credentials and all is good | 08:35 |
| frickler | if instead an admin deletes p1, neutron uses the admin's credentials with which the API request is made to talk to designate and cannot see the test.com. zone | 08:36 |
| ralonsoh | ok | 08:37 |
| frickler | if neutron would try to use the admin_context instead as you propose, it would still need to set the "all_projects" flag in order to see the zone, but that would still fail in current deployments, as the admin_context isn't really admin, it's just the set of credentials intended to manage PTR records | 08:38 |
| frickler | so it doesn't really have to be a global admin, just a user+project intended to host PTR records | 08:38 |
| ralonsoh | frickler, but that's why I don't understand your patch | 08:39 |
| ralonsoh | when is this going to be useful? | 08:39 |
| frickler | so the solution in my patch is to use the normal context we get from the API request, and when we cannot find the zone in the first try:, retry with adding all_projects flag to it | 08:39 |
| ralonsoh | that's the point | 08:39 |
| ralonsoh | 1) if the user calls, the zone will be present | 08:39 |
| ralonsoh | 2) if an admin calls this (deletes the port), the zone won't be there | 08:40 |
| ralonsoh | using "get_clients" or "get_all_projects_client" | 08:40 |
| ralonsoh | as you said | 08:40 |
| frickler | did you see the test in https://review.opendev.org/#/c/728409/1/neutron_tempest_plugin/scenario/test_dns_integration.py ? | 08:40 |
| ralonsoh | this admin user is just a regular user for designate | 08:40 |
| ralonsoh | yes | 08:40 |
| frickler | no, if an user is admin for neutron, it is also admin for designate | 08:41 |
| *** benj_ has quit IRC | 08:41 | |
| frickler | it's just that what is called admin_context in dns-integration isn't an admin user. might want to rename it ptr_record_managing_context instead | 08:41 |
| ralonsoh | ok, understood | 08:42 |
| frickler | in the scenario test you can see how it works for master and still fails for the stable branches that don't have the fix | 08:43 |
| ralonsoh | yeah, the context do not have the admin user token, just the normal user one | 08:43 |
| ralonsoh | thanks! | 08:43 |
| frickler | yw, did you see my question to slaweq earlier? maybe you have some opinion on that, too? | 08:45 |
| ralonsoh | let me read | 08:45 |
| ralonsoh | frickler, I think this is a bug and should be backported | 08:45 |
| frickler | ralonsoh: o.k., so I'll leave the tempest patch as is until the neutron fix is merged and backported, thx | 08:47 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Uses the OVN DevStack module https://review.opendev.org/734622 | 08:50 |
| *** benj_ has joined #openstack-neutron | 08:58 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Convert neutron-grenade-ovn job to be zuulv3 https://review.opendev.org/729591 | 09:03 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: WIP: Test internal dns resolution with external dns-integration https://review.opendev.org/740195 | 09:06 |
| openstackgerrit | Flavio Fernandes proposed openstack/neutron master: WIP DNM: please ignore https://review.opendev.org/729877 | 09:06 |
| openstackgerrit | Flavio Fernandes proposed openstack/neutron master: wip https://review.opendev.org/740196 | 09:06 |
| * frickler triggered a wip flood :-o | 09:07 | |
| *** tmorin has quit IRC | 09:09 | |
| openstackgerrit | Merged openstack/neutron master: Better document router requirements for IPv6 https://review.opendev.org/739848 | 09:11 |
| openstackgerrit | LIU Yulong proposed openstack/neutron master: Support gateway which is not in subnet CIDR in ha_router https://review.opendev.org/705441 | 09:11 |
| *** tmorin has joined #openstack-neutron | 09:16 | |
| openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: [DNM] testing neutron jobs with Ubuntu Focal https://review.opendev.org/738163 | 09:17 |
| *** rcernin has quit IRC | 09:23 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: [WIP] Install python-openvswitch from PyPI using requirements https://review.opendev.org/740206 | 09:28 |
| *** dtantsur|afk is now known as dtantsur | 09:29 | |
| *** markvoelker has joined #openstack-neutron | 09:44 | |
| maciejjozefczyk | bcafarel, \o do you have any idea what to do with this ? https://review.opendev.org/#/c/739912/ | 09:47 |
| *** markvoelker has quit IRC | 09:48 | |
| bcafarel | maciejjozefczyk: not many right now apart from "sigh loudly and go have a beer to forget it" | 09:51 |
| bcafarel | maciejjozefczyk: I am curious if it will pass for new patchets submitted a few min ago https://review.opendev.org/#/q/status:open+project:openstack/requirements+branch:stable/ussuri | 09:51 |
| maciejjozefczyk | bcafarel, ;( | 09:53 |
| maciejjozefczyk | bcafarel, lets wait then | 09:54 |
| slaweq | frickler: sorry, I was on meeting | 09:59 |
| frickler | seems 2020 is finally at full pace in our CI now, too | 09:59 |
| slaweq | frickler: regarding https://review.opendev.org/#/c/728385/ I don't see any reason why we should backport it to stable branches | 09:59 |
| slaweq | IMO should be fine | 09:59 |
| frickler | slaweq: so you are saying it shouldn't be backported? or was your answer missing another negation? | 10:01 |
| slaweq | frickler: sorry, I wanted to say "I don't see any reason why we should NOT backport it" | 10:02 |
| slaweq | I'm ok with backporting this patch | 10:02 |
| slaweq | sorry for this mistake :) | 10:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Uses the OVN DevStack module https://review.opendev.org/734622 | 10:03 |
| slaweq | maciejjozefczyk: I asked on openstack-requirements about Your patch | 10:03 |
| *** jmlowe has joined #openstack-neutron | 10:03 | |
| slaweq | lets see what they will say | 10:03 |
| maciejjozefczyk | ok slaweq thanks | 10:04 |
| openstackgerrit | Flavio Fernandes proposed openstack/neutron master: port_forwarding: validate args before invoking db update https://review.opendev.org/738145 | 10:06 |
| openstackgerrit | Merged openstack/ovsdbapp master: drop mock from lower-constraints https://review.opendev.org/734495 | 10:10 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Add config option ``http_retries`` https://review.opendev.org/740214 | 10:11 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/train: Add config option ``http_retries`` https://review.opendev.org/740215 | 10:11 |
| *** hjensas is now known as hjensas|afk | 10:12 | |
| *** maciejjozefczyk has quit IRC | 10:13 | |
| *** maciejjozefczyk has joined #openstack-neutron | 10:14 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/stein: Add config option ``http_retries`` https://review.opendev.org/740216 | 10:14 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron master: Stop disabling peakmem_tracker https://review.opendev.org/740218 | 10:15 |
| *** yamamoto has quit IRC | 10:15 | |
| *** ociuhandu has joined #openstack-neutron | 10:16 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/rocky: Add config option ``http_retries`` https://review.opendev.org/740220 | 10:18 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/queens: Add config option ``http_retries`` https://review.opendev.org/740222 | 10:18 |
| *** jmlowe has quit IRC | 10:24 | |
| *** jmlowe has joined #openstack-neutron | 10:25 | |
| openstackgerrit | Lajos Katona proposed openstack/neutron-tempest-plugin master: Make dynamic routing tests use clients as others https://review.opendev.org/740235 | 10:37 |
| openstackgerrit | Stephen Finucane proposed openstack/python-neutronclient master: lower-constraints: Drop os-testr, mox3 https://review.opendev.org/740239 | 10:42 |
| *** yamamoto has joined #openstack-neutron | 10:48 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: Test internal dns with external dns-integration https://review.opendev.org/740186 | 10:49 |
| *** psahoo has quit IRC | 10:52 | |
| *** yamamoto has quit IRC | 10:53 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron-lib master: New api-def: port-numa-affinity-policy https://review.opendev.org/740058 | 10:54 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: WIP: Test internal dns resolution with external dns-integration https://review.opendev.org/740195 | 10:55 |
| slaweq | jlibosva: lucasagomes: hi, I'm working on neutron-grenade-ovn job to move it to the zuul v3 syntax in https://review.opendev.org/#/c/729591/ - job seems to be running now, but it's failing - can You maybe take a look if I maybe missed something in the job's config or maybe this failure isn't really related to the job definition? | 10:57 |
| slaweq | thx in advance for any help with that | 10:57 |
| jlibosva | slaweq: sure | 10:58 |
| jlibosva | oh, ovn | 10:58 |
| jlibosva | nice | 10:58 |
| jlibosva | slaweq: I think what you need is to use grenade_devstack_localrc instead of devstack_localrc | 11:00 |
| jlibosva | hm, I see it works without in for ml2/ovs | 11:02 |
| jlibosva | but that could be related to the fact it's the default | 11:02 |
| *** tkajinam has quit IRC | 11:02 | |
| openstackgerrit | Bence Romsics proposed openstack/neutron master: metadata-ipv6: DHCP namespace https://review.opendev.org/715482 | 11:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron master: metadata-ipv6: Router namespace https://review.opendev.org/715483 | 11:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron master: metadata-ipv6: Accept link local address in X-Forwarded-For https://review.opendev.org/718729 | 11:04 |
| jlibosva | slaweq: The /opt/stack/old/octavia project was not found; if this is a gate job, add the project to the $PROJECTS variable in the job definition. | 11:05 |
| jlibosva | slaweq: also I think you will need to use the grenade_devstack_localrc, I *think* if you use devstack, it's gonna deploy old version with OVN and then try to deploy OVS after the upgrade | 11:06 |
| openstackgerrit | Merged openstack/neutron master: Use network.external DB model parameter when creating a floating IP https://review.opendev.org/737047 | 11:09 |
| openstackgerrit | Merged openstack/neutron-tempest-plugin master: Fix neutron-dynamic-routing tests when DVR is disabled https://review.opendev.org/738861 | 11:10 |
| *** psahoo has joined #openstack-neutron | 11:12 | |
| bcafarel | slaweq: maciejjozefczyk https://review.opendev.org/#/c/740202/ (another ussuri requirements backport) failed with similar errors | 11:13 |
| bcafarel | so it is generic requirements issue in stable branches | 11:13 |
| *** dave-mccowan has joined #openstack-neutron | 11:17 | |
| *** markvoelker has joined #openstack-neutron | 11:19 | |
| *** markvoelker has quit IRC | 11:24 | |
| *** yamamoto has joined #openstack-neutron | 11:42 | |
| *** jungleboyj_ is now known as jungleboyj | 11:46 | |
| openstackgerrit | Rafael Weingärtner proposed openstack/neutron master: Granular metering data in neutron-metering-agent https://review.opendev.org/735605 | 11:48 |
| maciejjozefczyk | bcafarel, ok, thanks for the information, so we need to wait ;/ | 11:49 |
| openstackgerrit | Maciej Józefczyk proposed openstack/neutron stable/ussuri: [OVN] Unify OVN/OVS compilation https://review.opendev.org/740255 | 12:04 |
| *** raildo has joined #openstack-neutron | 12:23 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Uses the OVN DevStack module https://review.opendev.org/734622 | 12:24 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Install python-openvswitch from PyPI using requirements https://review.opendev.org/740206 | 12:30 |
| *** psahoo_ has joined #openstack-neutron | 12:31 | |
| *** yamamoto has quit IRC | 12:32 | |
| *** psahoo has quit IRC | 12:35 | |
| openstackgerrit | Merged openstack/networking-ovn stable/queens: Fix broken gate because of OVS module compilation https://review.opendev.org/737817 | 12:38 |
| *** stingrayza has joined #openstack-neutron | 12:39 | |
| *** also_stingrayza has quit IRC | 12:40 | |
| *** raildo has quit IRC | 12:47 | |
| *** nweinber has joined #openstack-neutron | 12:48 | |
| *** raildo has joined #openstack-neutron | 12:50 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Create a Port resource request DB mixin class https://review.opendev.org/740269 | 12:55 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Create a Port resource request helper method https://review.opendev.org/740269 | 13:04 |
| *** hjensas|afk is now known as hjensas | 13:09 | |
| openstackgerrit | Andrey Kurilin proposed openstack/neutron stable/ussuri: [ci] Fix several rally task arguments https://review.opendev.org/740276 | 13:12 |
| openstackgerrit | Maciej Józefczyk proposed openstack/neutron stable/ussuri: Stop installing Octavia in OVN jobs https://review.opendev.org/740279 | 13:15 |
| openstackgerrit | Maciej Józefczyk proposed openstack/neutron stable/ussuri: [OVN] Stop using neutron_tempest_plugin in OVN singlenode job https://review.opendev.org/740281 | 13:19 |
| openstackgerrit | Lajos Katona proposed openstack/neutron master: New fullstack test for subnet port delete race condition https://review.opendev.org/728904 | 13:20 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [Functional] Add logging to the check test file function https://review.opendev.org/740283 | 13:22 |
| openstackgerrit | Lajos Katona proposed openstack/networking-odl stable/ussuri: Fix pep8 job https://review.opendev.org/740284 | 13:22 |
| openstackgerrit | Lajos Katona proposed openstack/networking-bagpipe master: Fix pep8 job https://review.opendev.org/740286 | 13:26 |
| openstackgerrit | Lajos Katona proposed openstack/networking-bgpvpn master: Fix pep8 job https://review.opendev.org/740287 | 13:29 |
| openstackgerrit | Merged openstack/neutron master: Update cirros image to cirros-0.5.1-x86_64 globally https://review.opendev.org/711425 | 13:30 |
| *** gcheresh has quit IRC | 13:33 | |
| openstackgerrit | Lajos Katona proposed openstack/networking-odl master: Migrate legacy grenade job to be native Zuul v3 https://review.opendev.org/725647 | 13:34 |
| *** yangyapeng__ has quit IRC | 13:43 | |
| *** yangyapeng has quit IRC | 13:43 | |
| *** yangyapeng_ has quit IRC | 13:43 | |
| slaweq | jlibosva: dalvarez: hi, I need one more review of https://review.opendev.org/#/c/736386/ - can You take a look if You will have some time? | 13:43 |
| slaweq | thx in advance | 13:43 |
| jlibosva | slaweq: o/ did you get my messages about my thoughts about the grenade you sent earlier today? or you want me to write it down to gerrit? | 13:44 |
| slaweq | jlibosva: yes, thx | 13:44 |
| slaweq | I will check that grenade_devstack_localrc in few minutes | 13:44 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron-lib master: New api-def: port-numa-affinity-policy https://review.opendev.org/740058 | 13:44 |
| slaweq | thx a lot for help | 13:44 |
| haleyb | slaweq: i can't remember, was the intention to deprecated the metering agent? just got a bug on updating it | 13:44 |
| openstackgerrit | Elod Illes proposed openstack/neutron stable/stein: Revert "Make greande jobs n-v for EM and oldest stable" https://review.opendev.org/740294 | 13:48 |
| maciejjozefczyk | https://review.opendev.org/#/c/711425/ \o/ | 13:50 |
| slaweq | haleyb, let me check that | 13:58 |
| slaweq | haleyb: https://bugs.launchpad.net/neutron/+bug/1817881 | 13:59 |
| openstack | Launchpad bug 1817881 in neutron " [RFE] L3 IPs monitor/metering via current QoS functionality (tc filters)" [Wishlist,In progress] - Assigned to LIU Yulong (dragon889) | 13:59 |
| slaweq | see last commenent there | 13:59 |
| slaweq | we discussed that maybe we will deprecate it in the future | 14:00 |
| slaweq | but not now for sure | 14:00 |
| lajoskatona | slaweq, ralonsoh: could you please take a look at this: https://review.opendev.org/729250 | 14:00 |
| ralonsoh | lajoskatona, sure | 14:00 |
| lajoskatona | ralonsoh: thanks | 14:00 |
| haleyb | slaweq: ack, will ping you in a minute about the new bug, meetings... | 14:00 |
| slaweq | sure, I'm in the meeting now too | 14:01 |
| slaweq | lajoskatona: I will take a look today | 14:02 |
| *** gcheresh has joined #openstack-neutron | 14:04 | |
| *** ratailor has quit IRC | 14:04 | |
| maciejjozefczyk | slaweq, ralonsoh heya guys please take a look https://review.opendev.org/#/c/729250/ | 14:05 |
| maciejjozefczyk | lucasagomes, ^ | 14:05 |
| ralonsoh | maciejjozefczyk, sure | 14:05 |
| maciejjozefczyk | thanks ralonsoh \o | 14:05 |
| lucasagomes | maciejjozefczyk, will take a look | 14:05 |
| lucasagomes | maciejjozefczyk, btw, does it conflict with that patch of yours introducing the distributed type ? | 14:06 |
| hjensas | lucasagomes: not sure you are correct to ask, but I tried to deploy TripleO with Ironic and OVN. | 14:07 |
| hjensas | lucasagomes: We include the NeutronDhcpAgent in the environment enabling ironic, so that DHCP with extra_opts for network booting work. I.e not using the DHCP in OVN. | 14:08 |
| hjensas | lucasagomes: Now there seem to be a safety raising an error when extra_dhcp_opts not supported by ovn is added on a port, so things no longer work. | 14:09 |
| lucasagomes | hjensas, oh can you paste me the error message ? Cause I think it may be a problem with some recent patch around extra dhcp options that I've proposed | 14:10 |
| lucasagomes | that does some sanity checks between the dhcp options and what's supported in core OVN | 14:10 |
| lucasagomes | hjensas, https://review.opendev.org/#/c/722025/ | 14:11 |
| lucasagomes | I'm 99% sure this patch might be the problem | 14:11 |
| hjensas | lucasagomes: yup, I think thats the case. I gave up on ovn, and tried ovs .. which does'nt deploy either. So I don't have the exact error now, will have to reproduce again. | 14:11 |
| lucasagomes | hjensas, no worries, does the error looks like this: https://review.opendev.org/#/c/722025/8/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py@496 ? | 14:11 |
| hjensas | lucasagomes: as I recall it, it was exactly that. | 14:12 |
| lucasagomes | hjensas, :-( ok | 14:12 |
| dalvarez | slaweq: i left some comments at https://review.opendev.org/#/c/736386/4 | 14:12 |
| lucasagomes | hjensas, I can try to send a patch fixing it, perhaps instead of raising an exception we should just log the problem | 14:12 |
| lucasagomes | hjensas, cause the problem before was that, not all DHCP options are supported by core OVN, so ppl would input something and it showed up in the neutron api | 14:13 |
| lucasagomes | but it didn't work | 14:13 |
| hjensas | lucasagomes: a warning log? Or if we can somehow see that a non-OVN dhcp agent is available that could handle it? (I dunno.) | 14:13 |
| lucasagomes | so the patch is an attempt to do some sanity checks on it | 14:13 |
| hjensas | lucasagomes: yes, the patch makes sense. | 14:14 |
| lucasagomes | hjensas, yeah that's a good point, perhaps if neutron dhcp is enable (idk yet how we can check it) we shouldn't raise that exception | 14:14 |
| slaweq | dalvarez: thx | 14:15 |
| slaweq | ralonsoh: njohnston: please take a look at https://review.opendev.org/#/c/737286/ if You will have few minutes :) | 14:16 |
| ralonsoh | slaweq, of course | 14:17 |
| maciejjozefczyk | lucasagomes, its is not in conflict, I rely of on this patchy :) | 14:17 |
| lucasagomes | maciejjozefczyk, ack | 14:17 |
| njohnston | Yes, I tried running this on my home setup late last night; everything seemed to work but it's a little hard to say for sure that it is good when all you have is a single node allinone. | 14:17 |
| njohnston | slaweq ^^ | 14:17 |
| lucasagomes | hjensas, I will try to propose a fix for it between today or tomorrow. Unless you are on it already | 14:18 |
| njohnston | slaweq: So I will give it a +2 but wait to see if ralonsoh can give a better verification for the +w | 14:19 |
| slaweq | njohnston: yes, I was testing that on multinode setup with test https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/scenario/test_connectivity.py#L72 | 14:19 |
| hjensas | lucasagomes: thanks, that would be great! | 14:20 |
| ralonsoh | njohnston, is already approved | 14:20 |
| slaweq | it was failing without this patch as "related" routers wasn't created | 14:20 |
| ralonsoh | njohnston, but I'm still reviewing the code | 14:20 |
| slaweq | You can't really check it on single node env | 14:20 |
| haleyb | slaweq: so the metering bug is https://bugs.launchpad.net/neutron/+bug/1886949 and this was also on the ML. | 14:24 |
| openstack | Launchpad bug 1886949 in neutron "[RFE] Granular metering data in neutron-metering-agent" [Undecided,In progress] - Assigned to Rafael Weingartner (rafaelweingartner) | 14:24 |
| *** abdysn has quit IRC | 14:24 | |
| openstackgerrit | vinay harsha mitta proposed openstack/neutron-lib master: Neutron agent-list API added with sort_key keyword https://review.opendev.org/738471 | 14:27 |
| *** armax has joined #openstack-neutron | 14:28 | |
| *** nplanel_ has quit IRC | 14:30 | |
| *** nplanel has joined #openstack-neutron | 14:30 | |
| openstackgerrit | vinay harsha mitta proposed openstack/neutron master: Supports Pagination and Sorting for agent List https://review.opendev.org/738475 | 14:31 |
| slaweq | thx haleyb, I will check it later tonight | 14:31 |
| openstackgerrit | Lajos Katona proposed openstack/networking-odl master: Workaround mock._is_started https://review.opendev.org/740315 | 14:43 |
| *** yamamoto has joined #openstack-neutron | 14:45 | |
| *** yamamoto has quit IRC | 14:47 | |
| *** mlavalle has joined #openstack-neutron | 14:50 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | 14:56 |
| *** Luzi has quit IRC | 14:56 | |
| *** mnasiadka has joined #openstack-neutron | 14:58 | |
| openstackgerrit | Merged openstack/networking-ovn stable/rocky: [OVN] Don't set virtual port type on ports with similar addresses https://review.opendev.org/733629 | 15:00 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: Stop disabling TLS for tests https://review.opendev.org/739019 | 15:10 |
| openstackgerrit | Merged openstack/neutron master: Support gateway which is not in subnet CIDR in ha_router https://review.opendev.org/705441 | 15:11 |
| *** dviroel_ is now known as dviroel | 15:15 | |
| *** yamamoto has joined #openstack-neutron | 15:23 | |
| openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: [DNM] testing neutron jobs with Ubuntu Focal https://review.opendev.org/738163 | 15:28 |
| *** yamamoto has quit IRC | 15:29 | |
| openstackgerrit | Merged openstack/networking-ovn stable/queens: [OVN] Don't set virtual port type on ports with similar addresses https://review.opendev.org/733632 | 15:48 |
| *** jlibosva has quit IRC | 15:55 | |
| *** bengates_ has quit IRC | 16:00 | |
| *** gcheresh has quit IRC | 16:10 | |
| *** armax has quit IRC | 16:17 | |
| *** hamalq has joined #openstack-neutron | 16:18 | |
| *** hamalq_ has joined #openstack-neutron | 16:19 | |
| *** armax has joined #openstack-neutron | 16:21 | |
| *** hamalq has quit IRC | 16:22 | |
| *** ociuhandu_ has joined #openstack-neutron | 16:23 | |
| *** ociuhandu has quit IRC | 16:27 | |
| *** ociuhandu_ has quit IRC | 16:28 | |
| *** armax has quit IRC | 16:37 | |
| *** lucasagomes has quit IRC | 16:39 | |
| *** thedac_ has joined #openstack-neutron | 16:47 | |
| *** andreaf has quit IRC | 16:48 | |
| *** mnasiadka_ has joined #openstack-neutron | 16:48 | |
| *** dasp_ has joined #openstack-neutron | 16:49 | |
| *** coreycb has quit IRC | 16:55 | |
| *** thedac has quit IRC | 16:55 | |
| *** mnasiadka has quit IRC | 16:55 | |
| *** rm_work has quit IRC | 16:55 | |
| *** nicolasbock has quit IRC | 16:55 | |
| *** dasp has quit IRC | 16:55 | |
| *** thedac_ is now known as thedac | 16:55 | |
| *** mnasiadka_ is now known as mnasiadka | 16:55 | |
| *** rouk has quit IRC | 16:56 | |
| *** masterpe has quit IRC | 16:56 | |
| *** coreycb has joined #openstack-neutron | 16:58 | |
| *** rm_work has joined #openstack-neutron | 16:58 | |
| *** nicolasbock has joined #openstack-neutron | 16:58 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron-lib master: New api-def: port-numa-affinity-policy https://review.opendev.org/740058 | 17:11 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron-lib master: New api-def: port-numa-affinity-policy https://review.opendev.org/740058 | 17:22 |
| *** masterpe has joined #openstack-neutron | 17:25 | |
| *** slaweq has quit IRC | 17:28 | |
| *** psahoo_ has quit IRC | 17:30 | |
| *** slaweq has joined #openstack-neutron | 17:31 | |
| *** slaweq has quit IRC | 17:36 | |
| *** dtantsur is now known as dtantsur|afk | 17:45 | |
| *** ralonsoh has quit IRC | 17:57 | |
| *** JamesBenson has joined #openstack-neutron | 18:18 | |
| *** lajoskatona has quit IRC | 18:22 | |
| *** tesseract has quit IRC | 18:27 | |
| *** armax has joined #openstack-neutron | 18:52 | |
| *** yamamoto has joined #openstack-neutron | 19:27 | |
| *** yamamoto has quit IRC | 19:31 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-dynamic-routing master: Add mock to test-requirements https://review.opendev.org/740354 | 19:47 |
| frickler | bcafarel: amotoki: slaweq: ^^ not sure what you decided on an intermediate neutron release, trying with this workaround now | 19:48 |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/neutron-tempest-plugin master: WIP: Test internal dns resolution with external dns-integration https://review.opendev.org/740195 | 20:04 |
| *** JamesBenson has quit IRC | 20:06 | |
| *** ociuhandu has joined #openstack-neutron | 20:12 | |
| *** Yumeng has quit IRC | 20:13 | |
| openstackgerrit | Merged openstack/neutron master: Auto-delete dhcp ports on segment delete https://review.opendev.org/729250 | 20:19 |
| *** ociuhandu has quit IRC | 20:31 | |
| *** armax has quit IRC | 20:31 | |
| *** slaweq has joined #openstack-neutron | 20:33 | |
| *** ociuhandu has joined #openstack-neutron | 20:47 | |
| *** nweinber has quit IRC | 20:56 | |
| *** ociuhandu has quit IRC | 20:59 | |
| *** maciejjozefczyk has quit IRC | 21:12 | |
| *** raildo has quit IRC | 21:22 | |
| *** tmorin has quit IRC | 21:35 | |
| *** yamamoto has joined #openstack-neutron | 22:07 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Allow to parse keywords in dns labels https://review.opendev.org/686343 | 22:11 |
| *** yamamoto has quit IRC | 22:12 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add test for default DNS zone per tenant https://review.opendev.org/733994 | 22:12 |
| *** slaweq has quit IRC | 22:21 | |
| *** slaweq has joined #openstack-neutron | 22:38 | |
| *** jamesdenton has quit IRC | 22:40 | |
| *** tkajinam has joined #openstack-neutron | 23:00 | |
| *** mlavalle has quit IRC | 23:06 | |
| *** dmellado has quit IRC | 23:39 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!