Skip to content

Commit

Permalink
Rename 'setup_ssh_connection_to_central_server' to 'setup_ssh_connect…
Browse files Browse the repository at this point in the history
…ion'.
  • Loading branch information
JoeZiminski committed Apr 9, 2024
1 parent d768627 commit 1b30458
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions datashuttle/datashuttle.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ class DataShuttle:
cfgs.make_and_get_logging_path().
For transferring data between a central data storage
with SSH, use setup setup_ssh_connection_to_central_server().
with SSH, use setup setup_ssh_connection().
This will allow you to check the server key, add host key to
profile if accepted, and setup ssh key pair.
Expand Down Expand Up @@ -588,7 +588,7 @@ def _transfer_specific_file_or_folder(
# -------------------------------------------------------------------------

@requires_ssh_configs
def setup_ssh_connection_to_central_server(self) -> None:
def setup_ssh_connection(self) -> None:
"""
Setup a connection to the central server using SSH.
Assumes the central_host_id and central_host_username
Expand Down
2 changes: 1 addition & 1 deletion datashuttle/tui/screens/setup_ssh.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
class SetupSshScreen(ModalScreen):
"""
This dialog windows handles the TUI equivalent of API's
setup_connection_to_central_server(). This asks to
setup_ssh_connection(). This asks to
confirm the central hostkey, and takes password to setup
SSH key pair.
Expand Down
2 changes: 1 addition & 1 deletion datashuttle/utils/ssh.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ def connect_client_with_logging(
except Exception:
utils.log_and_raise_error(
f"Could not connect to server. Ensure that \n"
f"1) You have run setup_ssh_connection_to_central_server() \n"
f"1) You have run setup_ssh_connection() \n"
f"2) You are on VPN network if required. \n"
f"3) The central_host_id: {cfg['central_host_id']} is"
f" correct.\n"
Expand Down
2 changes: 1 addition & 1 deletion docs/source/pages/documentation.md
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,7 @@ Next, a one-time command to setup the SSH connection must be run:

:::{tab-item} Python API
```{code-block} python
project.setup_ssh_connection_to_central_server()
project.setup_ssh_connection()
```
:::

Expand Down
2 changes: 1 addition & 1 deletion docs/source/pages/how_tos/make-a-new-project.md
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ project.make_config_file(
Next, a one-time command to set up the SSH connection must be run:

```{code-block} python
project.setup_ssh_connection_to_central_server()
project.setup_ssh_connection()
```

Running `setup-ssh-connection-to-central-server` will require verification
Expand Down
2 changes: 1 addition & 1 deletion docs/source/pages/how_tos/update-configs.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ If changing `SSH` configs, the connection may need to be
[reconfigured](new-project-ssh) with:

```python
project.setup_ssh_connection_to_central_server()
project.setup_ssh_connection()
```

:::
Expand Down
2 changes: 1 addition & 1 deletion tests/tests_integration/test_ssh_setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def test_verify_ssh_central_host_do_not_accept(
"""
orig_builtin = ssh_test_utils.setup_mock_input(input_)

project.setup_ssh_connection_to_central_server()
project.setup_ssh_connection()

ssh_test_utils.restore_mock_input(orig_builtin)

Expand Down

0 comments on commit 1b30458

Please sign in to comment.