update

Author: deependujha

src/lightning/fabric/plugins/environments/lightning.py

@@ -112,8 +112,6 @@ def find_free_network_port() -> int:
     `MASTER_PORT` environment variable.
 
     """
-    # If an external launcher already specified a MASTER_PORT (for example, torch.distributed.spawn or
-    # multiprocessing helpers), reserve it through the port manager so no other test reuses the same number.
     s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
     s.bind(("", 0))
     port = s.getsockname()[1]

tests/tests_fabric/conftest.py

@@ -80,8 +80,6 @@ def teardown_process_group():
     yield
     _destroy_dist_connection()
 
-    os.environ.pop("MASTER_PORT", None)
-
 
 @pytest.fixture(autouse=True)
 def thread_police_duuu_daaa_duuu_daaa():

tests/tests_pytorch/conftest.py

@@ -130,9 +130,6 @@ def teardown_process_group():
     yield
     _destroy_dist_connection()
 
-    # clean MASTER_PORT environment variable, to avoid leaking between tests
-    os.environ.pop("MASTER_PORT", None)
-
 
 @pytest.fixture(autouse=True)
 def reset_deterministic_algorithm():