fix: docker wrapper refactoring - Githubissues

tari-project / tari-launchpad

A node, wallet and miner for Tari focusing on ease of use

BSD 3-Clause "New" or "Revised" License

8 stars 15 forks source link

fix: docker wrapper refactoring #175

Closed therustmonk closed 1 year ago

therustmonk commented 1 year ago

Description

The PR improves DockerWrapper.

Changes:

Used a shared AppState instead of the shared global DOCKER_INSTANCE
Used a DockerWrapper instead of Docker to have extended methods (later DockerWrapper will be joined with Workspaces)
Used a reference to a wrapper
Removed unnecessary lock from the AppState that protects a Docker instance

Motivation and Context

Related to #177 The step towards the goal to have a single entity to manage docker.

How Has This Been Tested?

CI + manual smoke test