fix(pytest): Add master restart delay (#909)

- Add restart delay to master in crashing master test
- Add process check after force shutdown
- Increase time limit to 45 min

.github/workflows/regression-tests.yml

@@ -14,7 +14,7 @@ jobs:
         # Test of these containers
         container: ["ubuntu-dev:20"]
         build-type: [Debug, Release]
-    timeout-minutes: 30
+    timeout-minutes: 45
 
     container:
       image: ghcr.io/romange/${{ matrix.container }}

tests/dragonfly/__init__.py

@@ -52,6 +52,7 @@ class DflyInstance:
         except subprocess.TimeoutExpired:
             print("Unable to terminate DragonflyDB gracefully, it was killed")
             proc.kill()
+            proc.communicate()
 
     def _start(self):
         base_args = [f"--{v}" for v in self.params.args]

tests/dragonfly/replication_test.py

@@ -297,6 +297,7 @@ async def test_disconnect_master(df_local_factory, df_seeder_factory, t_master,
         master.stop(kill=True)
 
     async def start_master():
+        await asyncio.sleep(0.2)
         master.start()
         c_master = aioredis.Redis(port=master.port)
         assert await c_master.ping()