Open jongio opened 2 years ago
We could consider making CheckInstalled()
responsible for this. If we did that, we may want to change the ExternalTool
interface slightly so that instead of having an InstallUrl() we return an error that would have some description of how to solve the problem. We could aggregate it with one of the mutli error packages.
CheckRunning()
would make sense as an optional method.
This came up a bunch in our UXR for simplified init. We should investigate this as polish during Germanium
I am learning Go and wanted to take a look at a fully fledged app, so I came across this help-wanted issue. Looking at this it seems like the EnsureInstalled()
function needs to be updated. The function checks to see if a tool is installed, and if it is, it passes the check. This won't work for Docker as it can be installed but not running. Here is the line in question that needs to be updated.
for _, tool := range tools {
_, ok := confirmedTools[tool.Name()]
if ok {
log.Printf("Skipping install check for '%s'. It was previously confirmed.", tool.Name())
continue
}
Ideally there needs to be a check in this area to check if docker is running (maybe docker info). One thing that should be added to the ExternalTool
interface to flag tools that need to be checked and that calls an alternate workflow (possibly an EnsureRunning()
function)
@jongio what sample did you get this error with? I am not able to replicate using Dev Containers. Not sure if that is expected however
azd up
Got this:
Turns out docker wasn't running
I would like azd to tell me it isn't running before it tries to execute the command.