Skip to content

Commit

Permalink
feat: change download_logs url
Browse files Browse the repository at this point in the history
Signed-off-by: SdgJlbl <sarah.diot-girard@owkin.com>
  • Loading branch information
SdgJlbl committed Feb 14, 2024
1 parent 64d40e3 commit 41527e7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- BREAKING: Renamed `function` field of the Function pydantic model to `archive`([#393](https://github.com/Substra/substra/pull/393))
- BREAKING: Renamed ComputeTask status ([#397](https://github.com/Substra/substra/pull/397))
- `download_logs` uses the new endpoint ([#398](https://github.com/Substra/substra/pull/398))

### Added

Expand Down
2 changes: 1 addition & 1 deletion substra/sdk/backends/remote/backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ def download_logs(self, task_key: str, destination_file: str = None) -> str:
"""Download the logs of a failed task. If destination_file is set, return the full
destination path, otherwise, return the logs as a str.
"""
url = f"{self._client.base_url}/logs/{task_key}/file/"
url = f"{self._client.base_url}/task/{task_key}/logs/"

if destination_file:
return self._download(url, destination_file)
Expand Down

0 comments on commit 41527e7

Please sign in to comment.