[grpc-io] Call for Speakers! 10 Days left to submit a gRPConf 2024 proposal

2024-05-09 Thread 'Terry Wilson' via grpc.io
Hello gRPC Community, We still have slots available for speakers to join us at gRPConf 2024. If you have a gRPC story to tell, an innovative use case to share, or a burning question to answer, we invite you to submit a proposal by May 19th! Submit a Proposal

[ovs-dev] [PATCH v2 2/2] python: ovsdb-idl: Use monitor_cond for _Server DB.

2024-05-06 Thread Terry Wilson
L did not have support for retrieving the schema automatically and did not have a state for doing so, instead of transitioning on an error response from retrieving the _Server schema to requesting the "data" schema, this moves directly to monitoring the "data" DB. Signed-off-by: Ter

[ovs-dev] [PATCH v2 1/2] ovsdb-idl: Add C IDL test for "monitor" fallback.

2024-05-06 Thread Terry Wilson
There was a Python-only test for ensuring that the library would work when connecting to an older ovsdb-server that did not support monitor_cond. This adds a C IDL version of that test. Signed-off-by: Terry Wilson --- tests/ovsdb-idl.at | 16 +++- 1 file changed, 15 insertions(+), 1

[ovs-dev] [PATCH 0/2] python: ovsdb-idl Use monitor_cond for _Server DB.

2024-05-06 Thread Terry Wilson
nitoring the data DB (since we already were passed a schema at instantiation). It would be good to, in the future, add support for not having to specify the schema at Idl creation, but that kind of API change should be done on its own (hopefully in a backward-compatible way). Terry Wilson (2): ovsdb-i

[ovs-dev] [PATCH 2/2] python: ovsdb-idl: Use monitor_cond for _Server DB

2024-05-06 Thread Terry Wilson
L did not have support for retrieving the schema automatically and did not have a state for doing so, instead of transitioning on an error response from retrieving the _Server schema to requesting the "data" schema, this moves directly to monitoring the "data" DB. Signed-off-by: Ter

[ovs-dev] [PATCH 1/2] ovsdb-idl: Add C IDL test for "monitor" fallback

2024-05-06 Thread Terry Wilson
There was a Python-only test for ensuring that the library would work when connecting to an older ovsdb-server that did not support monitor_cond. This adds a C IDL version of that test. Signed-off-by: Terry Wilson --- tests/ovsdb-idl.at | 16 +++- 1 file changed, 15 insertions(+), 1

[ovs-dev] [PATCH 0/2] python: ovsdb-idl Use monitor_cond for _Server DB

2024-05-06 Thread Terry Wilson
nitoring the data DB (since we already were passed a schema at instantiation). It would be good to, in the future, add support for not having to specify the schema at Idl creation, but that kind of API change should be done on its own (hopefully in a backward-compatible way). Terry Wilson (2): ovsdb-i

[ovs-dev] [PATCH v2] python: ovsdb-idl: Add custom transaction operations.

2024-04-15 Thread Terry Wilson
It can be useful to be able to send raw transaction operations through the Idl's connection. For example, to clean up MAC_Binding entries for floating IPs without having to monitor the MAC_Binding table which can be quite large. Signed-off-by: Terry Wilson --- NEWS | 2

Re: [ovs-dev] [PATCH 2/3] python: ovsdb-idl: Make IndexedRows mirror hmap.

2024-04-11 Thread Terry Wilson
w/o creating a race condition with multiple IDL clients. Terry On Wed, Apr 10, 2024 at 4:39 PM Terry Wilson wrote: > > The Python IDL code very closely mirrors the C IDL code, which uses > an hmap to store table rows. hmap code allows duplicate keys, while > IndexedRows, whi

[ovs-dev] [PATCH 3/3] python: ovsdb-idl: Convert new_uuid insert() arg to UUID.

2024-04-10 Thread Terry Wilson
Signed-off-by: Terry Wilson --- python/ovs/db/idl.py | 2 +- tests/test-ovsdb.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/python/ovs/db/idl.py b/python/ovs/db/idl.py index a80da84e7..0e201366b 100644 --- a/python/ovs/db/idl.py +++ b/python/ovs/db/idl.py @@ -1854,7 +1854,

[ovs-dev] [PATCH 2/3] python: ovsdb-idl: Make IndexedRows mirror hmap.

2024-04-10 Thread Terry Wilson
IndexedRows is modified to behave similarly to the C IDL's hmap implementation. Fixes: 55b9507e6824 ("ovsdb-idl: Add the support to specify the uuid for row insert.") Signed-off-by: Terry Wilson --- python/ovs/db/custom_index.py | 13 ++--- 1 file changed, 10 insertions(+), 3

[ovs-dev] [PATCH 1/3] ovsdb-idl: Add python keyword to persistent UUID test.

2024-04-10 Thread Terry Wilson
The Python persistent UUID tests should have the keyword "python" added so that TESTSUITEFLAGS="-k python" will not miss testing them. Fixes: 55b9507e6824 ("ovsdb-idl: Add the support to specify the uuid for row insert.") Signed-off-by: Terry Wilson --- tes

[ovs-dev] ovsdb-idl: potential issues with the persistent UUID implementation

2024-04-08 Thread Terry Wilson
There was a patch in OVS 3.1 that added support to the IDL code for specifying the permanent UUID of a row when inserting [1]. There are both C and Python implementations. Initially, I was adding support to ovsdbapp for this feature and noticed that the Python tests for this feature passed

[Yahoo-eng-team] [Bug 2056366] [NEW] Neutron ml2/ovn does not exit when killed with SIGTERM

2024-03-06 Thread Terry Wilson
Public bug reported: When Neutron is killed with SIGTERM (like via systemctl), when using ML2/OVN neutron workers do not exit and instead are eventually killed with SIGKILL when the graceful timeout is reached (often around 1 minute). This is happening due to the signal handlers for SIGTERM.

[grpc-io] Save the Date - gRPConf 2024!

2024-03-05 Thread 'Terry Wilson' via grpc.io
Hello gRPC Community, Mark your calendars for the premier gRPC event of the year! gRPConf 2024 will land on the Google Cloud Campus in Sunnyvale, California, on August 27th, 2024. Get ready to: - Dive into the latest gRPC developments - Network with fellow gRPC users

[grpc-io] Re: Is hedgingPolicy supported in C++

2024-01-23 Thread 'Terry Wilson' via grpc.io
Sorry, I don't believe C++ has implemented hedging. On Monday, January 22, 2024 at 6:29:21 AM UTC-8 Leonid Lazarev wrote: > Hello, > > Does "hedgingPolicy" is suported for C++ grpc or not? > > I see description for hedging in proposal, https:// >

Re: [ovs-dev] [PATCH 20/22] ovsdb-server: Allow user-provided config files.

2024-01-10 Thread Terry Wilson
On Mon, Jan 8, 2024 at 8:17 AM Ilya Maximets wrote: > > On 1/5/24 21:26, Terry Wilson wrote: > > On Wed, Dec 13, 2023 at 7:05 PM Ilya Maximets wrote: > > > >> -/* Clears and replaces 'remotes' and 'dbnames' by a configuration read > >> from > >

Re: [ovs-dev] [PATCH 20/22] ovsdb-server: Allow user-provided config files.

2024-01-05 Thread Terry Wilson
On Wed, Dec 13, 2023 at 7:05 PM Ilya Maximets wrote: > -/* Clears and replaces 'remotes' and 'dbnames' by a configuration read from > - * 'config_file', which must have been previously written by save_config(). > */ > -static void > +/* Clears and replaces 'remotes' and 'db_conf' by a

Re: [ovs-dev] [PATCH v2] python: idl: Handle monitor_canceled

2024-01-05 Thread Terry Wilson
On Fri, Jan 5, 2024 at 9:56 AM Simon Horman wrote: > > On Mon, Dec 18, 2023 at 05:31:24PM -0600, Terry Wilson wrote: > > Currently python-ovs claims to be "db change aware" but does not > > parse the "monitor_canceled" notification. Transactions can co

[ovs-dev] [PATCH v2] python: idl: Handle monitor_canceled

2023-12-18 Thread Terry Wilson
Currently python-ovs claims to be "db change aware" but does not parse the "monitor_canceled" notification. Transactions can continue being made, but the monitor updates will not be sent. This handles monitor_cancel similarly to how ovsdb-cs currently does. Signed-

[ovs-dev] [PATCH] python: idl: Handle monitor_canceled

2023-11-16 Thread Terry Wilson
solves this issue. Signed-off-by: Terry Wilson --- python/ovs/db/idl.py | 4 1 file changed, 4 insertions(+) diff --git a/python/ovs/db/idl.py b/python/ovs/db/idl.py index 16ece0334..cfd81a1ec 100644 --- a/python/ovs/db/idl.py +++ b/python/ovs/db/idl.py @@ -481

Re: [ovs-dev] [OVN RFC 0/7] OVN IC bugfixes & proposals/questions

2023-11-16 Thread Terry Wilson
On Tue, Jan 24, 2023 at 8:00 AM Ilya Maximets wrote: > > On 1/24/23 14:12, Vladislav Odintsov wrote: > > Hi Ilya, > > > > could you please take a look on this? > > Maybe you can advice any direction how to investigate this issue? > > > > Thanks in advance. > > > > Regards, > > Vladislav Odintsov

Re: [grpc-io] gRPC client fallback from load-balancer to round robin DNS endpoint on LB outage

2023-11-06 Thread 'Terry Wilson' via grpc.io
Krishna, I'll mention that we also have a short guide on custom LBs that can help you with the fundamentals. On Monday, November 6, 2023 at 3:53:01 PM UTC-8 Larry Safran wrote: > Yes, you will need a custom NameResolver. > You could have

[grpc-io] gRPC-Java v1.59.0 Released

2023-10-20 Thread 'Terry Wilson' via grpc.io
You can now grab the v1.59.0 release. PLANNED ABI BREAKAGE! This breaks the ABI of the @ExperimentalApi classes listed below. This does not impact source code (API); it only impacts code compiled with a different version of gRPC than it

[grpc-io] Re: StreamObservers in multi-thread Java application

2023-09-15 Thread 'Terry Wilson' via grpc.io
Yes, you should not have two threads calling onNext() on the same StreamObserver. If you do, you can get messages interleaved with each other and most likely just produce garbage. You need to make sure that your application acquires a lock before calling any of the methods on StreamObserver.

[grpc-io] 1 Week Until gRPConf 2023 - Almost Sold Out!

2023-09-13 Thread 'Terry Wilson' via grpc.io
e Schedule <https://events.linuxfoundation.org/grpc-conf/program/schedule/> Thank you, Terry Wilson and the gRPC Team gRPConf 2023 Sunnyvale, CA | Register Now <https://youtu.be/pvI9S1O3Mk0> -- You received this message because you are subscribed to the Google Groups "grpc.io" group. To unsubs

[grpc-io] Last Call: gRPConf 2023 Early Bird Registration Ends Today 

2023-09-06 Thread 'Terry Wilson' via grpc.io
row. Thank you, Terry Wilson and the gRPC Team gRPConf 2023 Sunnyvale, CA | Register Now <https://youtu.be/pvI9S1O3Mk0> -- You received this message because you are subscribed to the Google Groups "grpc.io" group. To unsubscribe from this group and stop receiving emails from i

[Yahoo-eng-team] [Bug 2033932] [NEW] Add support for OVN MAC_Binding aging

2023-09-01 Thread Terry Wilson
Public bug reported: VN added support for aging out MAC_Binding entries [1][2]. Without this feature, the MAC_Bindings table can grow indefinitely. [1] https://github.com/ovn-org/ovn/commit/1a947dd3073628d2f2655f46ee7d3db62ed15b55 [2]

Re: [ovs-discuss] ovs pypi package

2023-08-01 Thread Terry Wilson via discuss
On Wed, Jul 26, 2023 at 8:51 AM Terry Wilson wrote: > > On Fri, Jul 21, 2023 at 3:45 AM Ilya Maximets wrote: > > > > On 7/19/23 09:52, Felix Huettner via discuss wrote: > > > Hi everyone, > > > > > > i noticed that the latest release of the ovs l

Re: [ovs-discuss] ovs pypi package

2023-07-26 Thread Terry Wilson via discuss
On Fri, Jul 21, 2023 at 3:45 AM Ilya Maximets wrote: > > On 7/19/23 09:52, Felix Huettner via discuss wrote: > > Hi everyone, > > > > i noticed that the latest release of the ovs library on pypi is for > > 2.17.1 [1]. Would it be possible to pushlish newer versions of the ovs > > python lib there

[ovs-dev] [PATCH v5] python: Add async DNS support

2023-07-11 Thread Terry Wilson
to bring socket_util.py up to equivalency to the C version. Signed-off-by: Terry Wilson --- .github/workflows/build-and-test.yml| 4 +- Documentation/intro/install/general.rst | 4 +- Documentation/intro/install/rhel.rst| 2 +- Documentation/intro/install/windows.rst | 2 +- NEWS

[ovs-dev] [PATCH v4] python: Add async DNS support

2023-07-11 Thread Terry Wilson
to bring socket_util.py up to equivalency to the C version. Signed-off-by: Terry Wilson --- .github/workflows/build-and-test.yml| 4 +- Documentation/intro/install/general.rst | 4 +- Documentation/intro/install/rhel.rst| 2 +- Documentation/intro/install/windows.rst | 2 +- NEWS

Re: [ovs-dev] [PATCH v3] python: Add async DNS support

2023-07-10 Thread Terry Wilson
On Mon, Jul 10, 2023 at 10:32 AM Terry Wilson wrote: > > I accidentally forgot to click reply-to-all. > > On Fri, Jun 30, 2023 at 10:27 AM Ilya Maximets wrote: > > > > On 6/30/23 16:54, Adrian Moreno wrote: > > > > > > > > > On 6/30/23 14:35, I

Re: [ovs-dev] [PATCH v3] python: Add async DNS support

2023-07-10 Thread Terry Wilson
>>> > >>> On 6/30/23 13:40, Ilya Maximets wrote: > >>>> On 6/30/23 12:39, Adrian Moreno wrote: > >>>>> > >>>>> > >>>>> On 6/14/23 23:07, Terry Wilson wrote: > >>>>>> This adds a Python versio

[grpc-io] [Last Call] Come Speak at gRPConf 2023!

2023-07-07 Thread 'Terry Wilson' via grpc.io
Hello gRPC Community! This is the last call to get your proposals in to speak at gRPConf 2023. The deadline is July 9th (this Sunday). Submit your proposals and find more information and tips HERE ! Those not interested in

[grpc-io] 5 Days Left to Submit at Talk for gRPConf 2023

2023-07-05 Thread 'Terry Wilson' via grpc.io
Hello gRPC Community! Time is running out to submit a proposal to speak at gRPConf 2023. The deadline is July 9th (this Sunday). Submit your proposals and find more information HERE ! Spots are still available and topics might

Re: [ovs-discuss] Scaling OVN/Southbound

2023-07-05 Thread Terry Wilson via discuss
On Wed, Jul 5, 2023 at 9:59 AM Terry Wilson wrote: > > On Fri, Jun 30, 2023 at 7:09 PM Han Zhou via discuss > wrote: > > > > > > > > On Wed, May 24, 2023 at 12:26 AM Felix Huettner via discuss > > wrote: > > > > > > Hi Ilya, > > &

Re: [ovs-discuss] Scaling OVN/Southbound

2023-07-05 Thread Terry Wilson via discuss
On Fri, Jun 30, 2023 at 7:09 PM Han Zhou via discuss wrote: > > > > On Wed, May 24, 2023 at 12:26 AM Felix Huettner via discuss > wrote: > > > > Hi Ilya, > > > > thank you for the detailed reply > > > > On Tue, May 23, 2023 at 05:25:49PM +0200, Ilya Maximets wrote: > > > On 5/23/23 15:59, Felix

Re: [ovs-dev] Scale testing OVN with ovn-heater for OpenStack use cases

2023-06-30 Thread Terry Wilson
On Fri, Jun 30, 2023 at 2:26 AM Frode Nordahl wrote: > > Hello all, > > On Tue, May 30, 2023 at 5:16 PM Felix Huettner > wrote: > > > > Hi Dumitru, > > > > On Fri, May 26, 2023 at 01:30:54PM +0200, Dumitru Ceara wrote: > > > On 5/24/23 09:37, Felix Huettner wrote: > > > > Hi everyone, > > > > >

[grpc-io] gRPConf 2023 talk proposal submission deadline now July 9th

2023-06-28 Thread 'Terry Wilson' via grpc.io
Hi everyone, we have extended the deadline for submitting gRPConf 2023 talk proposals to *July 9th*. If you want to reach out to the gRPC community this is a great opportunity! You can submit your talk here

Re: [ovs-dev] [PATCH v3] python: Add async DNS support

2023-06-27 Thread Terry Wilson
On Fri, Jun 23, 2023 at 11:20 AM Ilya Maximets wrote: > > On 6/23/23 17:15, Terry Wilson wrote: > > Just checking in on this. I see a single test failure > > https://github.com/ovsrobot/ovs/actions/runs/5272193494 'linux clang > > test asan' but it looks like some

Re: [ovs-dev] [PATCH v3] python: Add async DNS support

2023-06-23 Thread Terry Wilson
Just checking in on this. I see a single test failure https://github.com/ovsrobot/ovs/actions/runs/5272193494 'linux clang test asan' but it looks like something related to bfd, so definitely not the python patch here. Anything I need to do? On Wed, Jun 14, 2023 at 4:07 PM Terry Wilson wrote

[ovs-dev] [PATCH v3] python: Add async DNS support

2023-06-14 Thread Terry Wilson
to bring socket_util.py up to equivalency to the C version. Signed-off-by: Terry Wilson --- .github/workflows/build-and-test.yml| 4 +- Documentation/intro/install/general.rst | 4 +- Documentation/intro/install/rhel.rst| 2 +- Documentation/intro/install/windows.rst | 2 +- NEWS

[grpc-io] Re: Please join us for gRPConf 2023!

2023-06-12 Thread 'Terry Wilson' via grpc.io
A quick follow-up on the pricing for the event: *Early Bird*: $50 USD (through September 6) *Standard*: $99 USD (September 7 - Event) Hope to see you there! On Monday, June 12, 2023 at 2:23:12 PM UTC-7 Terry Wilson wrote: > gRPConf 2023 is happening! > > Join us on the Google Clo

[grpc-io] Please join us for gRPConf 2023!

2023-06-12 Thread 'Terry Wilson' via grpc.io
gRPConf 2023 is happening! Join us on the Google Cloud Campus for a full day of talks, demos, case studies, and code labs. Experts will discuss real-world implementations of gRPC, best practices for developers, and deep dives into specific topics. This is a must-attend event for anyone using gRPC

[ovs-dev] [PATCH v2] python: Add aync DNS support

2023-05-17 Thread Terry Wilson
to bring socket_util.py up to equivalency to the C version. Signed-off-by: Terry Wilson --- .github/workflows/build-and-test.yml | 4 +- debian/control.in| 1 + python/TODO.rst | 7 + python/automake.mk | 2 + python/ovs

Re: [ovs-dev] [PATCH] python: Add aync DNS support

2023-05-16 Thread Terry Wilson
Signed-off-by: Terry Wilson I can re-send if needed for the sign-off. I've tried to list the caveats in the commit message/TODO entry. Before I got too far into digging around into the socket_util and jsonrpc/reconnect code, I figured I'd put this out there to see if any major changes needed

[ovs-dev] [PATCH] python: Add aync DNS support

2023-05-16 Thread Terry Wilson
This adds a Python version of the async DNS support added in: 771680d96 DNS: Add basic support for asynchronous DNS resolving The above version uses the unbound C library, and this implimentation uses the SWIG-wrapped Python version of that. In the event that the Python unbound library is not

[ovs-discuss] python-ovs and OVSDB set types

2023-04-24 Thread Terry Wilson via discuss
A performance issue that has always bothered me: OVSDB has a set data type that matches up with Python's set data type (an unordered collection of unique items). The in-tree Python library represents this set type as a list. Not only does it do that, but every time you call Row.__getattr__()

[grpc-io] Re: Any suggestion on how to perform load testing of gRPC client & streaming service

2023-03-29 Thread 'Terry Wilson' via grpc.io
I'll also mention that there is a gRPC plugin for Gatling <https://github.com/phiSgr/gatling-grpc> and that K6 also supports gRPC <https://k6.io/blog/performance-testing-grpc-services/>. On Wednesday, March 29, 2023 at 4:27:54 PM UTC-7 Terry Wilson wrote: > I don't have any pers

[grpc-io] Re: Any suggestion on how to perform load testing of gRPC client & streaming service

2023-03-29 Thread 'Terry Wilson' via grpc.io
I don't have any personal experience in this tool, but I've seen community use of ghz.sh. Maybe start by looking at what it provides. On Sunday, March 19, 2023 at 12:15:24 PM UTC-7 Shailendra Omkar wrote: > Hi Team, > > I need to perform load testing of my application which uses gRPC client &

[grpc-io] gRPC-Java 1.54.0 Released

2023-03-24 Thread 'Terry Wilson' via grpc.io
The v1.54.0 release is now available. *New Features* - xds: Add weightedRoundRobin LB policy. The WRR policy allows picking the subchannel by weight based on the metrics feedback from the backend using ORCA API. See gRFC A58: Weighted

[Yahoo-eng-team] [Bug 2011590] [NEW] Startup times for large OVN dbs is greatly increased by frozen_row() calls

2023-03-14 Thread Terry Wilson
Public bug reported: On a large OVN DB, frozen_row() calls for the event notification processing can add 10ms up to 40ms or more depending on the size of the row (as compared to 4usec for just passing the Row object). This can easily make startup times w/ large DBs increase by a factor of 3 or

Re: [grpc-io] grpc vs rest benchmark

2023-02-16 Thread 'Terry Wilson' via grpc.io
In case you are not already aware of them, you might be interested in the benchmarks the gRPC team maintains: https://grpc.io/docs/guides/benchmarking/ You will find a link there to a dashboard with performance history as well as to the repo

[grpc-io] gRPC-Java 1.45.4 released

2023-01-24 Thread 'Terry Wilson' via grpc.io
The v1.45.4 release is now available. *Bug Fixes* - core: Free unused MessageProducer in RetriableStream (#9853 ) -- You received this message because you are subscribed to the Google Groups "grpc.io" group. To unsubscribe from this

Re: [ovs-dev] [PATCH v8] ovsdb-idl: Add the support to specify the uuid for row insert.

2022-11-28 Thread Terry Wilson
print_and_log("%03d: change conditions", step++); > } else if (arg[0] != '[') { > -idl_set(idl, arg, step++); > +if (!idl_set(idl, arg, step++)) { > +/* If idl_set() returns false, then no transaction > +

[grpc-io] Re: Question about proto message - breaking change behavior

2022-10-19 Thread 'Terry Wilson' via grpc.io
I can't comment on the C# specifics, but when dealing with protobuf changes, one should never reuse the ordinal numbers of deleted fields. If you don't need a field anymore, you should *reserve* the field number. You can find some documentation on this over here

[grpc-io] Last chance to sign up for gRPConf 2022

2022-10-19 Thread 'Terry Wilson' via grpc.io
Hi folks, this is your last chance to sign up for next week's KubeCon 2022 in Detroit and more specifically gRPConf 2022 happening on Monday 10/24! Come learn about the latest developments from several talks we have prepared for you and bring your questions to the gRPC team. We are also very

[grpc-io] gRPC-Java 1.43.3 Released

2022-10-07 Thread 'Terry Wilson' via grpc.io
The v1.43.3 release is now available. *Bugfixes* - android: fix for app coming to foreground #8850 - xds: fix the validation code to accept new-style CertificateProviderPluginInstance wherever used *Dependencies* - Bump

[grpc-io] gRPC-Java v1.42.3 Released

2022-10-07 Thread 'Terry Wilson' via grpc.io
The v1.42.3 release is now available *Dependencies* - Bump protobuf to 3.19.6 -- You received this message because you are subscribed to the Google Groups "grpc.io" group. To unsubscribe from this group and stop receiving emails from it, send an email to

[grpc-io] gRPC-Java 1.49.2 Released

2022-10-04 Thread 'Terry Wilson' via grpc.io
The v1.49.2 release is now available. *Dependencies* - Bump protobuf to 3.21.7 -- You received this message because you are subscribed to the Google Groups "grpc.io" group. To unsubscribe from this group and stop receiving emails

[grpc-io] gRPC-Java 1.48.2 Released

2022-10-04 Thread 'Terry Wilson' via grpc.io
The v1.48.2 release is now available. *Bug Fixes* - xds: Fix a bug in ring-hash load balancing policy that, during TRANSIENT_FAILURE state, it might cause unnecessary internal connection requests on subchannels. (#9537

[Yahoo-eng-team] [Bug 1989646] [NEW] neutron's override of ovsdbapp's idlutils.wait_for_change is busted

2022-09-14 Thread Terry Wilson
Public bug reported: The method doesn't actually do anything unless seqno=None due to an indentation issue. ** Affects: neutron Importance: Undecided Status: In Progress -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed

[grpc-io] gRPC Java v1.49.0 Release

2022-08-24 Thread 'Terry Wilson' via grpc.io
Release v1.49.0 is now available on Maven Central. https://github.com/grpc/grpc-java/releases/tag/v1.49.0 *New Features* - okhttp: Add `OkHttpServerBuilder`. The server can be used directly, but is not yet available via `ServerBuilder.forPort()` and `Grpc.newServerBuilderForPort()`.

Re: [ovs-discuss] Unreasonably often OVS DB compaction

2022-08-03 Thread Terry Wilson
You'll need to use ovs 2.17 (at least python-ovs 2.17) to use raft effectively. Leader transfer on compaction will cause disconnected clients, and before python-ovs 2.17, monitor-cond-since/update3 support was not there to make the reconnect happen using a snapshot instead of a full dump of the db

[ovs-dev] [PATCH ovsdb v1] Add CLI remote confiuration parity with DB

2022-07-22 Thread Terry Wilson
Several remote config options are configurable via the DB, e.g. inactivity_probe and max_backoff. This patch adds the ability to configure these options via the CLI as well, e.g.: --remote="pssl:6640;inactivity_probe=1,max_backoff=4000" Signed-off-by: Terry Wilson --- ovsdb/ov

[ovs-dev] [PATCH v4 ovsdb 1/1] Add ovs-confctl utility for Local_Config db

2022-07-12 Thread Terry Wilson
This adds ovs-confctl to configure DBs running the Local_Config schema added in 6f24c2bc7. Signed-off-by: Terry Wilson --- debian/openvswitch-common.install | 1 + debian/openvswitch-common.manpages | 1 + lib/.gitignore | 3 + lib/automake.mk| 8

[ovs-dev] [PATCH v3 ovsdb 1/1] Add ovs-confctl utility for Local_Config db

2022-07-12 Thread Terry Wilson
This adds ovs-confctl to configure DBs running the Local_Config schema added in 6f24c2bc7. Signed-off-by: Terry Wilson --- debian/openvswitch-common.install | 1 + debian/openvswitch-common.manpages | 1 + lib/.gitignore | 3 + lib/automake.mk| 8

[ovs-dev] [PATCH v2 ovsdb 1/1] Add ovs-confctl utility for Local_Config db

2022-07-12 Thread Terry Wilson
This adds ovs-confctl to configure DBs running the Local_Config schema added in 6f24c2bc7. Signed-off-by: Terry Wilson --- debian/openvswitch-common.install | 1 + debian/openvswitch-common.manpages | 1 + lib/.gitignore | 3 + lib/automake.mk| 8

[ovs-dev] [PATCH ovsdb 1/1] Add ovs-confctl utility for Local_Config db

2022-07-12 Thread Terry Wilson
This adds ovs-confctl to configure DBs running the Local_Config schema added in 6f24c2bc7. Signed-off-by: Terry Wilson --- debian/openvswitch-common.install | 1 + debian/openvswitch-common.manpages | 1 + lib/.gitignore | 3 + lib/automake.mk| 8

Re: [ovs-dev] [PATCH ovn v3 1/1] Allow arbitrary args to be passed to called binary

2022-06-29 Thread Terry Wilson
On Wed, Jun 29, 2022 at 2:40 AM Dumitru Ceara wrote: > > On 6/28/22 17:22, Terry Wilson wrote: > > On Tue, Jun 28, 2022 at 4:39 AM Dumitru Ceara wrote: > >> > >> On 6/27/22 18:00, Terry Wilson wrote: > >>> It is common to pass ovn-ctl options via an /et

Re: [ovs-dev] [PATCH ovn v3 1/1] Allow arbitrary args to be passed to called binary

2022-06-28 Thread Terry Wilson
On Tue, Jun 28, 2022 at 4:39 AM Dumitru Ceara wrote: > > On 6/27/22 18:00, Terry Wilson wrote: > > It is common to pass ovn-ctl options via an /etc/sysconfig file. > > It would be useful to be able to pass custom --remote options or > > additional DBs to listen to via th

[ovs-dev] [PATCH ovsdb v3 1/1] Add Local_Config schema

2022-06-28 Thread Terry Wilson
a local "config" database in addition to the main database we are servering (Open_vSwitch, OVN_Southbound, etc.) and this patch adds a Local_Config schema that currently just mirrors the Connection table and a Config table with a 'connections' row that stores each Connection. Signed-off-by: Te

[ovs-dev] [PATCH ovn v3 1/1] Allow arbitrary args to be passed to called binary

2022-06-27 Thread Terry Wilson
-by: Terry Wilson --- utilities/ovn-ctl | 22 ++ utilities/ovn-ctl.8.xml | 14 +- 2 files changed, 35 insertions(+), 1 deletion(-) diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl index 23d4d7f8c..93be9b84b 100755 --- a/utilities/ovn-ctl +++ b/utilities/ovn-ctl

Re: [ovs-dev] [PATCH] python: Add Python bindings TODO file.

2022-06-27 Thread Terry Wilson
cbb06ae 100644 > --- a/python/automake.mk > +++ b/python/automake.mk > @@ -117,3 +117,5 @@ $(srcdir)/python/ovs/dirs.py: python/ovs/dirs.py.template > mv $@.tmp $@ > EXTRA_DIST += python/ovs/dirs.py.template > CLEANFILES += pyt

[ovs-dev] [PATCH ovsdb v2 1/1] Add Local_Config schema

2022-06-27 Thread Terry Wilson
a local "config" database in addition to the main database we are servering (Open_vSwitch, OVN_Southbound, etc.) and this patch adds a Local_Config schema that currently just mirrors the Connection table and a Config table with a 'connections' row that stores each Connection. Signed-off-by: Te

[ovs-dev] [PATCH ovsdb 1/1] Add Local_Config schema

2022-06-24 Thread Terry Wilson
a local "config" database in addition to the main database we are servering (Open_vSwitch, OVN_Southbound, etc.) and this patch adds a Local_Config schema that currently just mirrors the Connection table and a Config table with a 'connections' row that stores each Connection. Signed-off-by: Te

[ovs-dev] [PATCH ovn v2 1/1] Allow arbitrary args to be passed to called binary

2022-06-23 Thread Terry Wilson
-by: Terry Wilson --- utilities/ovn-ctl | 22 ++ utilities/ovn-ctl.8.xml | 10 +- 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl index 23d4d7f8c..93be9b84b 100755 --- a/utilities/ovn-ctl +++ b/utilities/ovn-ctl

[ovs-dev] [PATCH ovn 1/1] Allow arbitrary args to be passed to called binary

2022-06-23 Thread Terry Wilson
-by: Terry Wilson --- utilities/ovn-ctl | 22 ++ utilities/ovn-ctl.8.xml | 10 +- 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl index 23d4d7f8c..955ddd9a3 100755 --- a/utilities/ovn-ctl +++ b/utilities/ovn-ctl

[ovs-dev] [PATCH v5 ovn 2/2] Ensure pid belongs to ovsdb-server in ovn-ctl

2022-06-08 Thread Terry Wilson
When checking if ovsdb-server is running, ensure that the binary we are going to run matches the one actually running with the the pid that was in our pidfile. Signed-off-by: Terry Wilson --- utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovn-ctl

[ovs-dev] [PATCH v5 ovn 1/2] Handle re-used pids in pidfile_is_running

2022-06-08 Thread Terry Wilson
Since pids can be re-used, it is necessary to check that the process that is running with a pid matches the one that we expect. This adds the ability to optionally pass a 'binary' argument to pidfile_is_running, and if it is passed to match the binary against /proc/$pid/exe. Signed-off-by: Terry

[ovs-dev] [PATCH v4 2/2] Ensure pid belongs to ovsdb-server in ovn-ctl

2022-06-07 Thread Terry Wilson
When checking if ovsdb-server is running, ensure that the binary we are going to run matches the one actually running with the the pid that was in our pidfile. Signed-off-by: Terry Wilson --- utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovn-ctl

[ovs-dev] [PATCH v4 1/2] Handle re-used pids in pidfile_is_running

2022-06-07 Thread Terry Wilson
Since pids can be re-used, it is necessary to check that the process that is running with a pid matches the one that we expect. This adds the ability to optionally pass a 'binary' argument to pidfile_is_running, and if it is passed to match the binary against /proc/$pid/exe. Signed-off-by: Terry

[ovs-dev] [PATCH v3 1/2] Handle re-used pids in pidfile_is_running

2022-06-07 Thread Terry Wilson
Since pids can be re-used, it is necessary to check that the process that is running with a pid matches the one that we expect. This adds the ability to optionally pass a 'binary' argument to pidfile_is_running, and if it is passed to match the binary against /proc/$pid/exe. Signed-off-by: Terry

[ovs-dev] [PATCH v3 2/2] Ensure pid belongs to ovsdb-server in ovn-ctl

2022-06-07 Thread Terry Wilson
When checking if ovsdb-server is running, ensure that the binary we are going to run matches the one actually running with the the pid that was in our pidfile. Signed-off-by: Terry Wilson --- utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovn-ctl

[ovs-dev] [PATCH v2 ovn 1/2] Handle re-used pids in pidfile_is_running

2022-06-02 Thread Terry Wilson
Since pids can be re-used, it is necessary to check that the process that is running with a pid matches the one that we expect. This adds the ability to optionally pass a 'binary' argument to pidfile_is_running, and if it is passed to match the binary against /proc/$pid/exe. Signed-off-by: Terry

[ovs-dev] [PATCH v2 ovn 2/2] Ensure pid belongs to ovsdb-server in ovn-ctl

2022-06-02 Thread Terry Wilson
When checking if ovsdb-server is running, ensure that the binary we are going to run matches the one actually running with the the pid that was in our pidfile. Signed-off-by: Terry Wilson --- utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovn-ctl

Re: [ovs-dev] [PATCH ovn 1/2] Handle re-used pids in pidfile_is_running

2022-06-02 Thread Terry Wilson
On Wed, Jun 1, 2022 at 10:18 PM Ihar Hrachyshka wrote: > > On Wed, Jun 1, 2022 at 1:26 PM Terry Wilson wrote: > > > > Since pids can be re-used, it is necessary to check that the > > process that is running with a pid matches the one that we expect. > > > >

Re: [ovs-dev] [PATCH ovn 1/2] Handle re-used pids in pidfile_is_running

2022-06-01 Thread Terry Wilson
On Wed, Jun 1, 2022 at 1:30 PM 0-day Robot wrote: > checkpatch: > WARNING: Line is 124 characters long (recommended limit is 79) > #44 FILE: utilities/ovn-ctl:52: > test -e "$pidfile" && [ -s "$pidfile" ] && pid=`cat "$pidfile"` && > pid_exists "$pid" && pid_exe_matches "$pid" "$binary" > >

[ovs-dev] [PATCH ovn 1/2] Handle re-used pids in pidfile_is_running

2022-06-01 Thread Terry Wilson
Since pids can be re-used, it is necessary to check that the process that is running with a pid matches the one that we expect. This adds the ability to optionally pass a 'binary' argument to pidfile_is_running, and if it is passed to match the binary against /proc/$pid/exe. Signed-off-by: Terry

[ovs-dev] [PATCH ovn 2/2] Ensure pid belongs to ovsdb-server in ovn-ctl

2022-06-01 Thread Terry Wilson
When checking if ovsdb-server is running, ensure that the binary we are going to run matches the one actually running with the the pid that was in our pidfile. Signed-off-by: Terry Wilson --- utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovn-ctl

[ovs-dev] [PATCH ovn 0/2] Add check for for process in pidfile_is_running

2022-06-01 Thread Terry Wilson
the ability to add an optional process name check to avoid this situation. Terry Wilson (2): Handle re-used pids in pidfile_is_running Ensure pid belongs to ovsdb-server in ovn-ctl ovs | 2 +- utilities/ovn-ctl | 11 +-- 2 files changed, 10 insertions(+), 3 deletions

Re: [ovs-dev] [PATCH v3 01/18] python: add generic Key-Value parser

2022-05-02 Thread Terry Wilson
LGTM Acked-by: Terry Wilson On Fri, Mar 11, 2022 at 9:22 AM Adrian Moreno wrote: > > Most of ofproto and dpif flows are based on key-value pairs. These > key-value pairs can be represented in several ways, eg: key:value, > key=value, key(value). > > Add the following classes

[ovs-dev] [PATCH] python: Raise AttributeError from uuid_to_row

2022-05-02 Thread Terry Wilson
, which is not expected from a getattr() or hasattr() call, which could break existing code. Fixes: 4e3966e64 (python: Politely handle misuse of table.condition.) Signed-off-by: Terry Wilson --- python/ovs/db/idl.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/python

Re: [ovs-dev] [PATCH] python: Raise AttributeError from uuid_to_row

2022-04-30 Thread Terry Wilson
On Sat, Apr 30, 2022 at 3:10 PM Terry Wilson wrote: > > Prior to 4e3966e64, when calling _uuid_to_row, it would raise an > AttributeError when trying to access base.ref_table.rows if the > referenced table was not registered. When called from > Row.__getattr__(), this would appr

[ovs-dev] [PATCH] python: Raise AttributeError from uuid_to_row

2022-04-30 Thread Terry Wilson
, which is not expected from a getattr() or hasattr() call, which could break existing code. Fixes: 4e3966e64 (python: Politely handle misuse of table.condition.) Signed-off-by: Terry Wilson --- python/ovs/db/idl.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/python

[grpc-io] gRFC A52: gRPC xDS Custom Load Balancer Configuration

2022-04-26 Thread 'Terry Wilson' via grpc.io
Hi folks, I have a PR out for a new gRFC at: https://github.com/grpc/proposal/pull/298 This one covers custom load balancer policy configurations in the control plane and their delivery to gRPC via xDS. Feel free to leave any comments! Terry -- You received this message because you are

Re: [ovs-discuss] High latencies due to southdb leadership changes?

2022-04-20 Thread Terry Wilson
On Fri, Apr 8, 2022 at 4:55 PM Frode Nordahl wrote: > > > > fre. 8. apr. 2022, 14:41 skrev Christian Stelter : >> >> Hi! >> >> We are experiencing high latencies from time to time in one of our >> openstack environments. While looking into the issue we found that >> the southdb has frequent

[ovs-dev] [PATCH v4] python: Politely handle misuse of table.condition

2022-04-13 Thread Terry Wilson
. Fixes: 46d44cf3b ("python: idl: Add monitor_cond_since support") Signed-off-by: Terry Wilson --- python/ovs/db/idl.py | 102 --- 1 file changed, 67 insertions(+), 35 deletions(-) diff --git a/python/ovs/db/idl.py b/python/ovs/db/idl.py index

Re: [ovs-dev] [PATCH v3] python: Politely handle misuse of table.condition

2022-04-13 Thread Terry Wilson
On Wed, Apr 13, 2022 at 3:06 AM Dumitru Ceara wrote: > > On 4/12/22 21:52, Terry Wilson wrote: > > Before 46d44cf3b, it was technically possible to assign a monitor > > condition directly to Idl.tables[table_name].condition. If done > > before the connection was establishe

  1   2   3   4   5   >