Skip to content

Commit

Permalink
enable redis debug again
Browse files Browse the repository at this point in the history
  • Loading branch information
Richard Schonberg authored and Richard Schonberg committed Sep 15, 2024
1 parent 19d88d2 commit 78b9707
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion improv/nexus.py
Original file line number Diff line number Diff line change
Expand Up @@ -776,7 +776,7 @@ def _start_store_interface(self, size, attempts=20):
)
# try with failure, incrementing port number
self.p_StoreInterface = self.start_redis(size)
time.sleep(1)
time.sleep(2)
if self.p_StoreInterface.poll(): # Redis could not start
logger.info("Could not connect to port {}".format(self.store_port))
self.store_port = str(int(self.store_port) + 1)
Expand Down
6 changes: 4 additions & 2 deletions test/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,12 @@ def setup_store(server_port_num):
"--maxmemory",
str(10000000),
],
stdout=subprocess.DEVNULL,
stderr=subprocess.DEVNULL,
)

time.sleep(3)
if p.poll() is not None:
raise Exception("redis-server failed to start")

yield p

# kill the subprocess when the caller is done with it
Expand Down
1 change: 0 additions & 1 deletion test/test_store_with_errors.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ def test_connect(setup_store, server_port_num):

def test_redis_connect(setup_store, server_port_num):
store = RedisStoreInterface(server_port_num=server_port_num)
assert isinstance(store.client, redis.Redis)
assert store.client.ping()


Expand Down

0 comments on commit 78b9707

Please sign in to comment.