| slaweq | ralonsoh hi, I just proposed https://review.opendev.org/c/openstack/releases/+/880267 to release new neutron-lib | 06:48 |
|---|---|---|
| slaweq | please check and tell me if You would like to wait for something else before doing this release | 06:49 |
| slaweq | ralonsoh ykarel lajoskatona hi, please check https://review.opendev.org/q/topic:issue/OSP-20968+status:open when You will have few minutes | 06:59 |
| slaweq | thx in advance | 06:59 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] ovn skip level packages https://review.opendev.org/c/openstack/neutron/+/880269 | 07:15 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: Add context wrapper around periodic task methods https://review.opendev.org/c/openstack/networking-odl/+/879334 | 08:02 |
| ralonsoh | slaweq, I'll check it now | 08:09 |
| ralonsoh | slaweq, I'll push a new PS for the releases patch, if you don't mind, adding the git log message | 08:10 |
| lajoskatona | ralonsoh, slaweq: Hi, I checked the grenade issue (https://bugs.launchpad.net/neutron/+bug/2015065 ) and I updated the lp with comments | 08:25 |
| ralonsoh | let me check | 08:26 |
| lajoskatona | slaweq: ralonsoh: 2 things: even the sg-rule api is for compute and that is failing (the POST calls are not in nova-api.log at least in the log whch I checked yesterday the call was not visible in the logs) | 08:27 |
| lajoskatona | slaweq, ralonsoh: and another hting it is not only sg-rule nova API, but other calls also as I checked this morning, see the examples #4, but the sg-rule call is the most frequent | 08:28 |
| lajoskatona | slaweq, ralonsoh: so I have doubts that this is a Neutron issue, but with my current understanding I don't know exactly what is behind it | 08:29 |
| lajoskatona | slaweq, ralonsoh: and another loud thinking, we should check with nova folks why there is API endpoint still for these calls in Nova as it just proxies these toward Neutron.... | 08:30 |
| ralonsoh | but so far (I've opened 4 logs) all fails are related to create_security_group_rule | 08:30 |
| ralonsoh | lajoskatona, what does it mean " sg-rule nova AP"? does Nova have an API for this? | 08:31 |
| lajoskatona | ralonsoh: yes from the legacy network API they still have the endpoints, but it just proxyes the calls to neutron, ie: https://opendev.org/openstack/nova/src/branch/master/nova/network/security_group_api.py#L382 | 09:02 |
| lajoskatona | ralonsoh: I don't think that with openstackclient you can reach this API, but perhaps with old novaclient, or in this case with tempest we still test it | 09:03 |
| ralonsoh | you are right, we are no longer using that in OSC | 09:04 |
| ralonsoh | but maybe nova client or neutronclient | 09:04 |
| ralonsoh | no, not neutronclient | 09:04 |
| ralonsoh | lajoskatona, ok, I need to finish something this morning | 09:05 |
| ralonsoh | but I'll check how are we making this call | 09:05 |
| ralonsoh | and if we are using novaclient, I'll try to change that | 09:05 |
| lajoskatona | ralonsoh: I check it with QA guys as these calls are coming from tempest | 09:10 |
| ralonsoh | that will be fantastic | 09:10 |
| ralonsoh | and thanks for your time debugging this isue | 09:10 |
| vvr-plv__ | Hello, I would like to implement 3rd party CI for testing Gerrit review. Can I ask a few questions? Maybe someone can answer or tell which channel is better to choose. | 09:31 |
| ralonsoh | vvr-plv, maybe better in #openstack-infra | 09:57 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Add support for localnet_learn_fdb OVN option https://review.opendev.org/c/openstack/neutron/+/877675 | 10:01 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Set unlimited SG rules quota in security groups scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/880029 | 10:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Add reader context to ``get_ports_on_host_by_subnet`` https://review.opendev.org/c/openstack/neutron/+/880298 | 12:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: [sqlalchemy-20] Add reader context to ``get_bgp_peers`` https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/880299 | 12:53 |
| ralonsoh | ^^ folks, trivial patches but needed | 12:53 |
| ralonsoh | we still have some "ORM session" errors | 12:54 |
| ralonsoh | https://opensearch.logs.openstack.org/_dashboards/app/discover/?security_tenant=global#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-30d,to:now))&_a=(columns:!(build_status,build_name),filters:!(('$state':(store:appState),meta:(alias:!n,disabled:!f,index:'94869730-aea8-11ec-9e6a-83741af3fdcd',key:build_status,negate:!f,params:(query:FAILURE),type:phrase),query:(match_phrase:(build_status:FAILUR | 12:54 |
| ralonsoh | E)))),index:'94869730-aea8-11ec-9e6a-83741af3fdcd',interval:auto,query:(language:kuery,query:'message:%22ORM%20session%22'),sort:!()) | 12:54 |
| ralonsoh | arggg let me paste the link | 12:54 |
| ralonsoh | https://paste.opendev.org/show/bpaQxxTbi0dRNFq4XE9i/ | 12:54 |
| opendevreview | Merged openstack/neutron master: Checkout "sqlalchemy/alembic" main branch in sqlalchmey-master jobs https://review.opendev.org/c/openstack/neutron/+/880126 | 13:20 |
| opendevreview | Merged openstack/neutron master: Update the quota guide examples https://review.opendev.org/c/openstack/neutron/+/878720 | 13:34 |
| opendevreview | Merged openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 13:34 |
| opendevreview | Merged openstack/neutron master: Update url and package name https://review.opendev.org/c/openstack/neutron/+/863013 | 13:38 |
| opendevreview | Merged openstack/neutron master: Change external process manager tests to clean temp files https://review.opendev.org/c/openstack/neutron/+/877140 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Mark "ipv6_pd_enabled" as deprecated and experimental. https://review.opendev.org/c/openstack/neutron/+/879030 | 13:45 |
| opendevreview | Brian Haley proposed openstack/neutron master: Do not assume order of get_sync_data_metering response elements https://review.opendev.org/c/openstack/neutron/+/880309 | 14:06 |
| haleyb | ralonsoh: ^^ that should fix the error moving neutron-lib requirements | 14:06 |
| ralonsoh | I was investigating this issue now | 14:07 |
| ralonsoh | I'll check that | 14:07 |
| ralonsoh | next time i'll assign the bug to me | 14:07 |
| haleyb | the elements were correct, just in wrong order. there was a previous patch in that file in 2015 that fixed a similar problem | 14:07 |
| haleyb | saw you were busy so pushed that out | 14:08 |
| ralonsoh | I'll check if requirements can run on top of a dependant patch | 14:08 |
| haleyb | ralonsoh: btw, i will be on pto starting tomorrow, for a week, will miss any drivers meetings and such | 14:09 |
| ralonsoh | won't you be tomorrow then? | 14:09 |
| haleyb | my plane leaves here in 7 hours, so no work tomorrow :) | 14:10 |
| ralonsoh | understood, thanks for the heads-up | 14:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Finish processing the Database table as soon as possible https://review.opendev.org/c/openstack/neutron/+/857010 | 14:40 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Do not dispose local_vlan_hints https://review.opendev.org/c/openstack/neutron/+/880334 | 15:04 |
| ralonsoh | haleyb, before you leave | 15:33 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/879913 | 15:33 |
| ralonsoh | I think the patch is good | 15:33 |
| haleyb | ralonsoh: ack, will try to get to release note | 15:34 |
| ralonsoh | maybe deserves a reno, what do you think? this is changing the current behaviour (we are re-creating a port and assigning an IP) | 15:34 |
| ralonsoh | thanks! | 15:34 |
| ralonsoh | I'll +2 it again | 15:34 |
| haleyb | seems the other patch still fails on that other job | 15:34 |
| ralonsoh | let me check | 15:34 |
| haleyb | UnorderedList() should have worked, i verified the expected/actual are identical except for order | 15:35 |
| ralonsoh | haleyb, I don't think this is using the depends on | 15:36 |
| ralonsoh | it is at the beginning | 15:36 |
| ralonsoh | but then --> | 15:36 |
| ralonsoh | 2023-04-13 14:48:47.933911 | ubuntu-jammy | Processing ./.tox/.tmp/package/1/neutron-22.1.0.dev108-0.editable-py3-none-any.whl | 15:36 |
| ralonsoh | 2023-04-13 14:48:47.941540 | ubuntu-jammy | Installing collected packages: neutron | 15:36 |
| ralonsoh | 2023-04-13 14:48:47.997125 | ubuntu-jammy | Successfully installed neutron-22.1.0.dev108 | 15:36 |
| ralonsoh | let me test that manually | 15:37 |
| haleyb | i did a check experimental on the neutron change to see if use latest neutron-lib code | 15:38 |
| ralonsoh | it is failing in the same way | 15:39 |
| haleyb | wth | 15:43 |
| haleyb | and actually the experimental jobs don't run unit tests | 15:44 |
| ralonsoh | the problem is the context object | 15:45 |
| ralonsoh | and this patch in n-lib | 15:45 |
| ralonsoh | https://review.opendev.org/q/Ibde6acdb99555a6e43ca253523df7cbe4d150787 | 15:45 |
| haleyb | interesting, would never have guessed that. feel free to ignore my change | 15:47 |
| ralonsoh | I'll push an new PS today | 15:49 |
| opendevreview | Brian Haley proposed openstack/neutron master: Do not assume order of get_sync_data_metering response elements https://review.opendev.org/c/openstack/neutron/+/880309 | 16:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Do not check the context object in ``TestMeteringPlugin`` https://review.opendev.org/c/openstack/neutron/+/880338 | 16:01 |
| ralonsoh | haleyb, ^^ | 16:02 |
| ralonsoh | this is passing | 16:02 |
| haleyb | ralonsoh: yes, just looking, never even noticed the context object was different, focused on the dict | 16:02 |
| haleyb | feel free to abandon mine i guess | 16:03 |
| ralonsoh | ok, I'll wait until the CI check in the requirements patch | 16:04 |
| ralonsoh | but I've tested it manually | 16:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Do not check the context object in ``TestMeteringPlugin`` https://review.opendev.org/c/openstack/neutron/+/880340 | 16:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Do not check the context object in ``TestMeteringPlugin`` https://review.opendev.org/c/openstack/neutron/+/880341 | 16:29 |
| opendevreview | Brian Haley proposed openstack/neutron master: OVN: Always try and create a metadata port on subnets https://review.opendev.org/c/openstack/neutron/+/879913 | 16:33 |
| opendevreview | Merged openstack/neutron master: [CI][fullstack/functional] Report slowest tests https://review.opendev.org/c/openstack/neutron/+/880161 | 16:40 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Drop use of OVN_GW_PORT_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877831 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Drop use of LR OVN_GW_NETWORK_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877712 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] OVNClient._get_router_ports: Drop unused parameter https://review.opendev.org/c/openstack/neutron/+/878527 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add a gw_ports fields to the Router object https://review.opendev.org/c/openstack/neutron/+/879462 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 18:52 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 18:52 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 21:09 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 21:11 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 22:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!