Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set IPAddress parameter, fixing Rancher 2.0 issue #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fabiorauber
Copy link

When the VM is created and its IP Address is collected, the XenServer machine driver was not setting the IPAddress parameter, expected by Rancher 2.0. This parameter comes from the base machine driver, so it should be set.

@jonathon2nd
Copy link

@facioraber

Have you used this or your fork in Rancher UI?

@fabiorauber
Copy link
Author

@jonathon2nd, i'm using my fork within Rancher, to create VMs in Rancher node pools:
https://github.com/interlegis/docker-machine-driver-xcp

@jonathon2nd
Copy link

jonathon2nd commented Nov 10, 2020

@facioraber is there a guide which I can refer to for using your fork in Rancher? I have added the node driver from here and I am looking at "XCP URL for boot2docker image" now. Is that referring to this: https://github.com/boot2docker/boot2docker/releases/tag/v19.03.12?

I won't actually be able to use this the way I intended. I was going to use this driver as part of a hybrid cluster, with the Openstack driver for public cloud. But I found out you currently can not mix them rancher/rancher#26330. I may use this later for purely 'on prem'.
Thank you for the assistance.

@fabiorauber
Copy link
Author

fabiorauber commented Nov 10, 2020

Answering your question @jonathon2nd, it refers to a valid boot2docker ISO image file URL. If you specify a different OS Template name with --xcp-os-template variable, then the boot2docker URL is not used. I'm currently using it for purely on-prem clusters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants