Skip to content

[TensorDock] Remove the tensordock from supported backeds #3092#3094

Merged
peterschmidt85 merged 4 commits intomasterfrom
3092-remove-tensordock
Sep 11, 2025
Merged

[TensorDock] Remove the tensordock from supported backeds #3092#3094
peterschmidt85 merged 4 commits intomasterfrom
3092-remove-tensordock

Conversation

@peterschmidt85
Copy link
Copy Markdown
Contributor

No description provided.

@peterschmidt85 peterschmidt85 linked an issue Sep 11, 2025 that may be closed by this pull request
@r4victor
Copy link
Copy Markdown
Collaborator

Ideally, show a warning if tensordock is configured so that it can be dropped completely in the future.

Also leave a comment in models.py that it's to be removed eventually.

@peterschmidt85
Copy link
Copy Markdown
Contributor Author

peterschmidt85 commented Sep 11, 2025

Ideally, show a warning if tensordock is configured so that it can be dropped completely in the future.

I thought the following is enough, no?

WARNING  dstack._internal.server.services.config:151 Backend tensordock not available and won't be configured. Check that backend dependencies are installed.

@peterschmidt85
Copy link
Copy Markdown
Contributor Author

Also leave a comment in models.py that it's to be removed eventually.

Done

@r4victor
Copy link
Copy Markdown
Collaborator

r4victor commented Sep 11, 2025

Ideally, show a warning if tensordock is configured so that it can be dropped completely in the future.

I thought the following is enough, no?

WARNING  dstack._internal.server.services.config:151 Backend tensordock not available and won't be configured. Check that backend dependencies are installed.

A bit confusing recommending "Check that backend dependencies are installed", but ok. We're dropping tensordock support without prior deprecations anyway.

@peterschmidt85 peterschmidt85 merged commit 449f43d into master Sep 11, 2025
26 checks passed
@peterschmidt85 peterschmidt85 deleted the 3092-remove-tensordock branch September 11, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TensorDock] Remove the tensordock from supported backeds

2 participants