Skip to content

Conversation

@axel7083
Copy link
Contributor

@axel7083 axel7083 commented Sep 30, 2024

What does this PR do?

Following #1813 the isModelUploaded and deleteRemoteModel are moved inside the ModelManager class, and uses the PodmanConnection, executeSSH method.

Screenshot / video of UI

What issues does this PR fix or reference?

Fixes #1809

How to test this PR?

  • unit tests has been provided
@axel7083 axel7083 force-pushed the refactor/move-model-utils-logic-podman branch from 6ee9c3b to 963bad8 Compare September 30, 2024 17:08
@axel7083
Copy link
Contributor Author

Need rebase when the following will be merged

Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
@axel7083 axel7083 force-pushed the refactor/move-model-utils-logic-podman branch from 963bad8 to 1f0b6f5 Compare October 1, 2024 08:16
@axel7083 axel7083 changed the title feat: improve podman cli execution feat: improve podman cli execution for models Oct 1, 2024
@axel7083 axel7083 marked this pull request as ready for review October 1, 2024 08:19
@axel7083 axel7083 requested review from a team, benoitf and jeffmaury as code owners October 1, 2024 08:19
@axel7083 axel7083 requested review from dgolovin and feloy October 1, 2024 08:19
@axel7083 axel7083 marked this pull request as draft October 7, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant