Revert "Fix EADDRINUSE errors in distributed tests with port manager and retry logic"

[bhimrazy]

What does this PR do?
Reverts #21309

Preparation for: #21335

This revert removes the port management logic and switches to a simpler approach for addressing port-related issues, as this is specific to standalone test environments.
The initial step for this change has been introduced in Lightning-AI/utilities — reference: Lightning-AI/utilities#447

cc: @deependujha

---

📚 Documentation preview 📚: https://pytorch-lightning--21341.org.readthedocs.build/en/21341/

[github-actions]

⛈️ Required checks status: Has failure 🔴

Warning
This job will need to be re-run to merge your PR. If you do not have write access to the repository, you can ask Lightning-AI/lai-frameworks to re-run it. If you push a new commit, all of CI will re-trigger.

Groups summary

🟢 pytorch_lightning: Tests workflow

Check ID	Status
pl-cpu-guardian	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py, tests/tests_pytorch/conftest.py.

🟢 pytorch_lightning: lit GPU

Check ID	Status
pytorch.yml / Lit Job (nvidia/cuda:12.1.1-devel-ubuntu22.04, pytorch, 3.10)	success	✅
pytorch.yml / Lit Job (lightning, 3.12)	success	✅
pytorch.yml / Lit Job (pytorch, 3.12)	success	✅

These checks are required after the changes to tests/tests_pytorch/conftest.py, src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py.

🟢 Benchmarks

Check ID	Status
benchmark.yml / Lit Job (fabric)	success	✅
benchmark.yml / Lit Job (pytorch)	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py.

🟢 fabric: Docs

Check ID	Status
docs-make (fabric, doctest)	success	✅
docs-make (fabric, html)	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py.

🟢 lightning_fabric: CPU workflow

Check ID	Status
fabric-cpu-guardian	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py, tests/tests_fabric/conftest.py, tests/tests_fabric/plugins/environments/test_lightning.py, tests/tests_fabric/utilities/test_port_manager.py.

🔴 lightning_fabric: lit GPU

Check ID	Status
fabric.yml / Lit Job (nvidia/cuda:12.1.1-devel-ubuntu22.04, fabric, 3.10)	success	✅
fabric.yml / Lit Job (fabric, 3.12)	failure	❌
fabric.yml / Lit Job (lightning, 3.12)	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py, tests/tests_fabric/conftest.py, tests/tests_fabric/plugins/environments/test_lightning.py, tests/tests_fabric/utilities/test_port_manager.py.

🟢 mypy

Check ID	Status
mypy	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py.

🟢 install

Check ID	Status
install-pkg-guardian	success	✅

These checks are required after the changes to src/lightning/fabric/plugins/environments/lightning.py, src/lightning/fabric/utilities/port_manager.py.

---

Thank you for your contribution! 💜

Note
This comment is automatically generated and updates for 70 minutes every 180 seconds. If you have any other questions, contact carmocca for help.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87%. Comparing base (19912d0) to head (4efe570).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #21341   +/-   ##
=======================================
- Coverage      87%      87%   -0%     
=======================================
  Files         270      269    -1     
  Lines       23799    23705   -94     
=======================================
- Hits        20629    20535   -94     
  Misses       3170     3170           

[deependujha]

Rather than a separate PR, I believe, we should merge the two PRs.

[deependujha]

pr merged in #21335. closing this.