| opendevreview | liuyulong proposed openstack/python-neutronclient master: Add CURD actions for packet rate limit rule https://review.opendev.org/c/openstack/python-neutronclient/+/818717 | 00:23 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/ussuri: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818348 | 01:41 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818437 | 02:53 |
| opendevreview | Merged openstack/neutron stable/victoria: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818438 | 02:53 |
| zhangtongjian | Hi where can i submit a bug to openvswitch | 02:56 |
| zhangtongjian | anyone knows that ? | 02:57 |
| zhangtongjian | Cnfn:hi | 03:09 |
| zhangtongjian | ihrachys:hello | 03:17 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Properly clean up ovn-northd in functional tests https://review.opendev.org/c/openstack/neutron/+/819049 | 03:29 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Properly clean up ovn-northd in functional tests https://review.opendev.org/c/openstack/neutron/+/819049 | 04:21 |
| opendevreview | Terry Wilson proposed openstack/neutron master: WIP Use neutron db for ovn agents https://review.opendev.org/c/openstack/neutron/+/818850 | 04:21 |
| frickler | slaweq: ovn people: could you please have a look at this devstack patch? https://review.opendev.org/c/openstack/devstack/+/806858 | 05:20 |
| opendevreview | Merged openstack/neutron stable/victoria: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818342 | 05:32 |
| opendevreview | Merged openstack/neutron stable/ussuri: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818343 | 05:42 |
| frickler | also quite a bunch of gate failures in case you haven't seen those yet https://zuul.opendev.org/t/openstack/builds?pipeline=gate&result=FAILURE | 06:26 |
| opendevreview | Merged openstack/neutron stable/xena: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818471 | 06:35 |
| opendevreview | Merged openstack/neutron stable/wallaby: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818472 | 06:35 |
| slaweq | frickler: hi, sure, I will check it | 07:50 |
| slaweq | frickler: I also added jlibosva and lucasagomes to reviewers of that patch | 07:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Rehome QoS floating IP extension https://review.opendev.org/c/openstack/neutron-lib/+/817797 | 08:53 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Increase openstack-tox-lower-constraints timeout https://review.opendev.org/c/openstack/neutron/+/819071 | 08:54 |
| lajoskatona | slaweq, ralonsoh: ^ l-c job timeout patch | 08:57 |
| lajoskatona | slaweq, ralonsoh, frickler: I see unit test timeouts on stable branches as well (stable/xena, wallaby) so perhaps we can increase timeout on those branches as well, what do you think ? | 08:57 |
| ralonsoh | lajoskatona, +2 | 09:00 |
| ralonsoh | yes, I think we should backport this change to stable branches | 09:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add network QoS inheritance to floating IP https://review.opendev.org/c/openstack/neutron/+/817734 | 09:01 |
| *** ykarel is now known as ykarel|lunch | 09:03 | |
| frickler | lajoskatona: are you talking about l-c or general tox jobs? l-c for stable branches one might consider dropping instead | 09:09 |
| frickler | and doing even more than 1h for l-c on master is ... wow | 09:10 |
| lajoskatona | frickler: for stable branches tox-pyx jobs: https://zuul.openstack.org/builds?project=openstack%2Fneutron&branch=stable%2Fxena&branch=stable%2Fwallaby&result=TIMED_OUT | 09:10 |
| lajoskatona | frickler: for l-c I suppose we can win minutes if we check l-c.txt regularly to make pip resolvers life easier, but not sure if that is really worth it and even possible | 09:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add upgrade check for floating IP QoS inheritance from network https://review.opendev.org/c/openstack/neutron/+/818399 | 09:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement floating IP network QoS inheritance https://review.opendev.org/c/openstack/neutron/+/817790 | 09:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: "L3AgentExtensionsManager" check loaded extensions https://review.opendev.org/c/openstack/neutron/+/818540 | 09:27 |
| frickler | lajoskatona: seems wallaby is much slower than ussuri, any idea what changed there? number of tests hasn't changed much, so likely some slowdown within neutron itself? https://zuul.openstack.org/builds?job_name=openstack-tox-py38&project=openstack%2Fneutron&branch=stable%2Fussuri&branch=stable%2Fwallaby&result=SUCCESS&result=TIMED_OUT | 09:32 |
| lajoskatona | frickler: good question | 09:49 |
| *** ykarel|lunch is now known as ykarel | 10:05 | |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron-tempest-plugin master: QoS min pps API tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/806689 | 10:32 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron-tempest-plugin master: Fix test_port_resource_request_inherited_policy test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817356 | 10:32 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron-tempest-plugin master: Update PortTestCasesResourceRequest for new Neutron API ext https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817357 | 10:32 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron-tempest-plugin master: Test port.resource_request format with min bw and min pps rules https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817358 | 10:32 |
| zhangtongjian | Hi where can i submit a bug to openvswitch? | 10:36 |
| zhangtongjian | hello,has anyone can help me ? | 10:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "target_tenant" with "target_project" in RBAC OVOs and models https://review.opendev.org/c/openstack/neutron/+/815855 | 10:49 |
| ralonsoh | zhangtongjian, if the bug is related to Neutron OVS plugin, you have https://bugs.launchpad.net/neutron/ | 10:51 |
| ralonsoh | if this is an OVS related bug, you have https://docs.openvswitch.org/en/latest/internals/bugs/ | 10:52 |
| zhangtongjian | ok,tks | 10:53 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Execute neutron-db-manage only if q-svc is enabled https://review.opendev.org/c/openstack/tap-as-a-service/+/819094 | 11:13 |
| opendevreview | Merged openstack/neutron stable/xena: Enhanced set of warnings if an answers file or a templates directory is used. https://review.opendev.org/c/openstack/neutron/+/818589 | 11:24 |
| opendevreview | Merged openstack/neutron stable/wallaby: Enhanced set of warnings if an answers file or a templates directory is used. https://review.opendev.org/c/openstack/neutron/+/818730 | 11:26 |
| opendevreview | yatin proposed openstack/neutron master: Bump lower-constraints for eventlet to 0.26.1 https://review.opendev.org/c/openstack/neutron/+/819101 | 11:40 |
| ykarel | lajoskatona, ralonsoh let's see if ^ helps with lower-constraints timeouts | 11:41 |
| ykarel | atleast i see the improvents in my local runs | 11:41 |
| ralonsoh | ykarel, thanks | 11:42 |
| ralonsoh | ykarel, please, remove one on the Change-ID of this patch | 11:42 |
| ykarel | ahh i removed, don't know why it reinstered | 11:43 |
| ralonsoh | the git hooks you have in your local git repo | 11:43 |
| opendevreview | yatin proposed openstack/neutron master: Bump lower-constraints for eventlet to 0.26.1 https://review.opendev.org/c/openstack/neutron/+/819101 | 11:44 |
| ykarel | will check with test patch what's the issue | 11:44 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test fix for lower-constraints timeout https://review.opendev.org/c/openstack/neutron/+/819102 | 11:48 |
| ykarel | this time worked fine | 11:49 |
| opendevreview | Merged openstack/neutron stable/victoria: Enhanced set of warnings if an answers file or a templates directory is used. https://review.opendev.org/c/openstack/neutron/+/818731 | 12:00 |
| opendevreview | Merged openstack/neutron stable/ussuri: Enhanced set of warnings if an answers file or a templates directory is used. https://review.opendev.org/c/openstack/neutron/+/818732 | 12:00 |
| lajoskatona | ykarel: thanks, for proposal | 12:32 |
| ykarel | lajoskatona, ralonsoh results looks good https://review.opendev.org/c/openstack/neutron/+/819102/ | 12:42 |
| ykarel | all three runs less than 50 minutes | 12:42 |
| ykarel | in parent patch also took <40 minutes https://zuul.opendev.org/t/openstack/buildset/37d9438bbb864343a534dcaeab6f86e8 | 12:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add upgrade check for floating IP QoS inheritance from network https://review.opendev.org/c/openstack/neutron/+/818399 | 12:44 |
| lajoskatona | ykarel: \o/ | 12:45 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/rocky: Blacklist bandit, bump neutron-lib and retire neutron-lbaas https://review.opendev.org/c/openstack/networking-odl/+/818384 | 12:54 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron-tempest-plugin master: [stable/rocky] Regroup tests exclusion list and add new failing ones https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/819109 | 13:20 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron stable/rocky: [Functional] Wait for the initial state of ha router before test https://review.opendev.org/c/openstack/neutron/+/808502 | 13:20 |
| opendevreview | Merged openstack/neutron master: Fix tunnel_types in ml2 ovs sample config https://review.opendev.org/c/openstack/neutron/+/818911 | 13:27 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/818152 | 13:27 |
| erlon | ralonsoh: hey, so, I'm still trying to parse what you told about the FIP. I don't understand why the FIP i created is not 'elegible' to be QoSed. The FIP is attached to a fixed port, see: https://gist.github.com/sombrafam/da770f4d9cd96cccd05500e71d01d484 | 13:52 |
| ralonsoh | erlon, are you setting the QoS on the port with device_owner=network:floatingip ? | 13:53 |
| erlon | And I can ping the VM. Also, inthe tests I did in the OVS deployment, I created the ports and assing the QoS rules in the very same way, and the FIP ports were working there | 13:54 |
| erlon | ralonsoh: yes | 13:54 |
| ralonsoh | no, the QoS is set in the FIP | 13:54 |
| ralonsoh | openstack floating ip set --qos-policy <qos> <fip> | 13:54 |
| erlon | ow | 13:55 |
| erlon | I didnt know the fip had the field to store the policy | 13:55 |
| erlon | ralonsoh++ bingo | 13:57 |
| *** whoami-rajat__ is now known as whoami-rajat | 13:57 | |
| erlon | so, its currently possible to set QoS in: | 14:00 |
| erlon | - router: external/internal port | 14:00 |
| erlon | - network: internal/external | 14:00 |
| erlon | - fip (fip object)/fip port | 14:00 |
| erlon | it would be nice if fip (fip object) <-> fip port were equivalent | 14:00 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Update placement allocation of bound ports when network QoS policy is changed https://review.opendev.org/c/openstack/neutron/+/818889 | 14:24 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 14:28 |
| opendevreview | Merged openstack/neutron-lib master: Rehome QoS floating IP extension https://review.opendev.org/c/openstack/neutron-lib/+/817797 | 14:29 |
| opendevreview | Terry Wilson proposed openstack/neutron master: WIP Use neutron db for ovn agents https://review.opendev.org/c/openstack/neutron/+/818850 | 14:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add floating IP network QoS field to floating IP definition https://review.opendev.org/c/openstack/neutron-lib/+/817936 | 14:33 |
| opendevreview | yatin proposed openstack/neutron master: Set report_interval to 0 for ovs agent unit tests https://review.opendev.org/c/openstack/neutron/+/819117 | 14:38 |
| ykarel | ralonsoh, lajoskatona slaweq ^ should fix timeouts in unit tests(tox-py*) jobs | 14:39 |
| ykarel | locally i see good results, let's see how it goes in CI | 14:39 |
| ralonsoh | ykarel, we do this locally in some tests. But yes, this could be a good idea | 14:40 |
| lajoskatona | ykarel: +1 | 14:46 |
| *** ykarel is now known as ykarel|away | 14:51 | |
| *** weechat1 is now known as amorin | 15:01 | |
| lajoskatona | it seems that we are hit with some glance/ g-api issue: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Job%20for%20devstack%40g-api.service%20failed%20because%20the%20control%20process%20exited%20with%20error%20code.%5C%22 | 15:06 |
| lajoskatona | strange is that by logstash it failed only for some networking projects, and devstack. | 15:07 |
| ralonsoh | sorry, I see nothing in this search | 15:09 |
| lajoskatona | ralonsoh: sorry, the interval is set to few minutes | 15:18 |
| ralonsoh | but even increasing it | 15:18 |
| ralonsoh | ok, that was my browser, it was stuck refreshing... | 15:19 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 15:25 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 15:40 |
| opendevreview | Merged openstack/neutron stable/ussuri: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818439 | 15:53 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix gateway_mtu option should not always be set https://review.opendev.org/c/openstack/neutron/+/818443 | 16:10 |
| ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/807354 | 16:49 |
| ralonsoh | If you want to update it, cool. If not, I'll +2 too | 16:49 |
| ralonsoh | (the comment is just a nit) | 16:50 |
| ralonsoh | I was checking the n-lib patches | 16:50 |
| ralonsoh | I think we'll be able to release a new version next week | 16:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][L3] QoS inherit network https://review.opendev.org/c/openstack/neutron/+/819147 | 16:55 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Remove ussuri jobs from the project's CI queues https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/818688 | 16:55 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 17:02 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 19:00 |
| frickler | haleyb: so AWS has defined a ULA address for metadata over IPv6 it seems (fd00:ec2::254). would it make sense to adopt that in neutron? https://aws.amazon.com/blogs/networking-and-content-delivery/introducing-ipv6-only-subnets-and-ec2-instances/ | 19:17 |
| frickler | oh, they also do similar things to provide a standard DNS and NTP service, nice | 19:19 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/818153 | 21:20 |
| opendevreview | Merged openstack/neutron stable/victoria: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/818151 | 21:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove functions which enable Neutron's QoS service https://review.opendev.org/c/openstack/neutron/+/815687 | 21:25 |
| slaweq | gmann: lbragstad hi, regrarding oslo_policy fix for the scopes in BaseRules (https://review.opendev.org/c/openstack/oslo.policy/+/804980), it seems that this timeout in the cross-neutron-tox-py38 job wasn't related to that patch | 21:29 |
| slaweq | I tried recently some dnm patch https://review.opendev.org/c/openstack/oslo.policy/+/818798 which also failed the same way | 21:30 |
| slaweq | so I proposed to increase timeout in that job https://review.opendev.org/c/openstack/oslo.policy/+/819025 | 21:30 |
| slaweq | please review it when You will have a moment | 21:30 |
| slaweq | I'm now going afk but please left comments in gerrit if You have any. I can check them in the morning :) | 21:30 |
| lbragstad | slaweq ack - thanks | 21:43 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 21:48 |
| gmann | lbragstad: thanks, will check | 22:33 |
| gmann | slaweq: ^^ | 22:33 |
| gmann | slaweq: lbragstad lgtm time for neutron job is now same as what neutron gate has. | 22:41 |
| opendevreview | Merged openstack/neutron stable/xena: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818340 | 23:01 |
| opendevreview | Merged openstack/neutron stable/victoria: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818347 | 23:01 |
| opendevreview | Merged openstack/neutron stable/train: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818349 | 23:02 |
| opendevreview | Merged openstack/neutron stable/wallaby: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818346 | 23:03 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Check WaitEvents for match after other events https://review.opendev.org/c/openstack/ovsdbapp/+/819116 | 23:12 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Capture test run logging https://review.opendev.org/c/openstack/ovsdbapp/+/819196 | 23:24 |
| opendevreview | Ghanshyam proposed openstack/neutron master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/neutron/+/819201 | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!