Skip to content

Commit

Permalink
fixing TIME_WAIT on the ext-server socket
Browse files Browse the repository at this point in the history
Signed-off-by: Martin Volf <[email protected]>
  • Loading branch information
martin-volf committed Dec 17, 2023
1 parent b10df28 commit eb2a7ee
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/confd_gnmi_api_adapter.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import threading
import json
from enum import Enum
from socket import socket
from socket import create_server, socket
import gnmi_pb2
from confd_gnmi_adapter import GnmiServerAdapter
from confd_gnmi_api_adapter_defaults import ApiAdapterDefaults
Expand Down Expand Up @@ -265,6 +265,7 @@ def process_external_change(self, ext_sock):
self._append_changes(EXT_SPOINT, list(self._external_changes(data)))
self.put_event(self.SubscriptionEvent.SEND_CHANGES)
log.debug("data=%s", data)
connection.recv(1) # make sure the client side closes the socket first
log.info("<==")

def subscribe_monitored_paths_cdb(self, sub_sock):
Expand Down Expand Up @@ -317,10 +318,9 @@ def start_external_change_server():
"""
log.debug("==>")
log.info("Starting external change server!")
ext_server_sock = socket()
# TODO port (host) as const or command line option
ext_server_sock.bind(("localhost",
GnmiConfDApiServerAdapter.external_port))
ext_server_sock = create_server(("localhost", GnmiConfDApiServerAdapter.external_port),
reuse_port=True)
ext_server_sock.listen(5)
log.debug("<== ext_server_sock=%s", ext_server_sock)
return ext_server_sock
Expand Down

0 comments on commit eb2a7ee

Please sign in to comment.