-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
returned diskname is not correct #16
Comments
Hi and thank you for using the driver with kubernetes! It's still on my todo list :-D I recently introduced a check that should try to detect any errors with the disk creation, so that is to blame for that. Unfortunately, it sends it's expected and actual data in the second line of the error message, that is not displayed in your output (maybe the software expects only one line of error). Anyway, I have a possible workaround but no machine I can actually test it. Would you be so kind? https://github.com/lnxbil/docker-machine-driver-proxmox-ve/releases/tag/possible-fix-for-16 |
Thank you, now with a better error message I can deal with. brb |
So, another prerelease |
Yes, this is a new and external error. Strange that it is found in the first place to create the VM. In which directory did you copy the driver file? |
It think i have a possible solution : |
In rancher you upload the driver from the interface... and then he store it somewhere... I think it's important to work on your repo because it is pointed by rancher. |
Just to be clear, changing this works? diff --git a/cmd/docker-machine-driver-proxmox-ve/main.go b/cmd/docker-machine-driver-proxmox-ve/main.go
index 5ca2279..7a2df1f 100644
--- a/cmd/docker-machine-driver-proxmox-ve/main.go
+++ b/cmd/docker-machine-driver-proxmox-ve/main.go
@@ -2,9 +2,9 @@ package main
import (
"github.com/docker/machine/libmachine/drivers/plugin"
- proxmox "github.com/lnxbil/docker-machine-driver-proxmox-ve"
+ proxmoxve "github.com/lnxbil/docker-machine-driver-proxmox-ve"
)
func main() {
- plugin.RegisterDriver(proxmox.NewDriver("default", ""))
+ plugin.RegisterDriver(proxmoxve.NewDriver("default", ""))
} |
Ah, nevermind. Found your fork. |
So, changed it in 5014644 and forgot to reference this issue. |
I've not uploaded a new release yet, you're capable of building your own as you revealed above :-D |
i encourage you to build and create the pre-release 3... to avoid problem with our possible different golang version |
Sure thing ... online. |
Hello,
I try to use your driver (v2) as a
node-driver
in Rancher to provide a Kubernetes cluster.But i have a issue, the logs return me this error :
Error creating machine: Error in driver during machine creation: returned diskname is not correct.
However, as you can see in the screen shot, the disk is correctly created but the VM isn't created:
Here is my proxmox
node template
in Rancher:Any idea to resolve this problem?
The text was updated successfully, but these errors were encountered: