GoogleCloudPlatform / cloud-code-intellij

Plugin to support the Google Cloud Platform in IntelliJ IDEA - Docs and Issues Repository
Apache License 2.0
319 stars 59 forks source link

Cloud Run: Docker not detected properly on Windows #3041

Closed ilpersi closed 2 years ago

ilpersi commented 2 years ago

(Please ensure you are running the latest version of Cloud Cloud IntelliJ with Help > Check for Updates.)

Expected Behavior

When I run the "Cloud Run: Run Locally" configuration, my project is deployed and run locally

Actual Behavior

Plugin is complaining that Docker is missing with the following message

It looks like you don't have Docker installed. Please install and start Docker to use this feature.

Below an example of the error. As you can see from the same screenshot, Docker is correctly installed and running.

image

image

Additional Information

etanshaul commented 2 years ago

Kamino cloned this issue to GoogleCloudPlatform/cloud-code-intellij-internal

etanshaul commented 2 years ago

thanks @ilpersi we are looking into a workaround for the next release coming very soon. The issue appears to be that our Docker detection is not reliable on Windows - incorrectly assuming that Docker is missing when it is actually installed. In the short term we might update the logic to simply warn and not block - as your Cloud Run session should work despite this invalid detection warning. update: we've identified the root cause and will fix the docker detection logic

ilpersi commented 2 years ago

@etanshaul I am not sure if this can help. I just tested on Pycharm 2021.2.3 (same machine, parallel installation) and Docker detection is working flawlessly.

etanshaul commented 2 years ago

also windows? edit: you did say "same machine" so I assume so :). This is odd. Though I am confident that our fix will help as I was able to reproduce. I'll let you know when it is out for you to try.

etanshaul commented 2 years ago

@ilpersi please try this out on the just released version 21.12.1-xxx where this issue should be resolved, and let us know if you still have any issues. Closing in the meantime. Thanks.

ilpersi commented 2 years ago

@etanshaul wow, this was fast! 👍🏻 Thank you very much for this, it works like a charm now on 21.12.1-213