Applying review comments

Author: petyaslavova

tests/test_scenario/test_hitless_upgrade.py

@@ -30,16 +30,16 @@
 )
 
 logging.basicConfig(
-    level=logging.DEBUG,
+    level=logging.INFO,
     format="%(asctime)s %(levelname)s %(message)s",
-    filemode="w",
-    filename="./test_hitless_upgrade.log",
 )
 
 BIND_TIMEOUT = 30
 MIGRATE_TIMEOUT = 60
 FAILOVER_TIMEOUT = 15
 
+DEFAULT_BIND_TTL = 15
+
 
 class TestPushNotifications:
     """
@@ -767,7 +767,7 @@ def execute_commands(moving_event: threading.Event, errors: Queue):
         assert errors.empty(), f"Errors occurred in threads: {errors.queue}"
 
         logging.info("Waiting for moving ttl to expire")
-        time.sleep(15)
+        time.sleep(DEFAULT_BIND_TTL)
         bind_thread.join()
 
     @pytest.mark.timeout(300)  # 5 minutes timeout
@@ -1021,7 +1021,7 @@ def test_disabled_handling_during_migrating_and_moving(
         )
 
         logging.info("Waiting for moving ttl to expire")
-        time.sleep(20)
+        time.sleep(DEFAULT_BIND_TTL)
 
         logging.info("Validating connection states after MOVING has expired ...")
         self._validate_default_notif_disabled_state(