已合并
test(distributed): fix c10d rendezvous backend upstream patch #34336
Jinfan Liu创建于 4月24日
test(distributed): fix c10d rendezvous backend upstream patch #34336
已合并
共 1 个文件变更+40-0
| @@ -0,0 +1,40 @@ | |||
| 1 | +diff --git a/test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py b/test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py | ||
| 2 | +index 82bc62d..521513e 100644 | ||
| 3 | +--- a/test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py | ||
| 4 | ++++ b/test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py | ||
| 5 | + | ||
| 6 | + port = get_free_port() | ||
| 7 | + self._params = RendezvousParameters( | ||
| 8 | + backend="dummy_backend", | ||
| 9 | +- endpoint=f"localhost:{port}", | ||
| 10 | ++ endpoint=f"127.0.0.1:{port}", | ||
| 11 | + run_id="dummy_run_id", | ||
| 12 | + min_nodes=1, | ||
| 13 | + max_nodes=1, | ||
| 14 | + | ||
| 15 | + self._expected_endpoint_file = tmp_path | ||
| 16 | + self._expected_temp_dir = tempfile.gettempdir() | ||
| 17 | + | ||
| 18 | +- self._expected_endpoint_host = "localhost" | ||
| 19 | ++ self._expected_endpoint_host = "127.0.0.1" | ||
| 20 | + self._expected_endpoint_port = port | ||
| 21 | + self._expected_store_type = TCPStore | ||
| 22 | + self._expected_read_timeout = timedelta(seconds=10) | ||
| 23 | + | ||
| 24 | + ) | ||
| 25 | + | ||
| 26 | + def test_create_backend_returns_backend_if_is_host_is_false(self) -> None: | ||
| 27 | +- TCPStore( # type: ignore[call-arg] | ||
| 28 | ++ self._bootstrap_store = TCPStore( # type: ignore[call-arg] | ||
| 29 | + self._expected_endpoint_host, self._expected_endpoint_port, is_master=True | ||
| 30 | + ) | ||
| 31 | + | ||
| 32 | + | ||
| 33 | + def test_create_backend_returns_backend_if_is_host_is_not_specified_and_store_already_exists( | ||
| 34 | + self, | ||
| 35 | + ) -> None: | ||
| 36 | +- TCPStore( # type: ignore[call-arg] | ||
| 37 | ++ self._bootstrap_store = TCPStore( # type: ignore[call-arg] | ||
| 38 | + self._expected_endpoint_host, self._expected_endpoint_port, is_master=True | ||
| 39 | + ) | ||
| 40 | + | ||