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

Added VM.host_name and VM.rename #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

simonjohansson
Copy link

Howdie.

Added VM.host_name.

Added VM.rename.

@maintux
Copy link
Owner

maintux commented Feb 20, 2014

hi @simonjohansson, this is interesting but:

  • Don't update version in gemspec
  • This works with "free license mode"?

@simonjohansson
Copy link
Author

@maintux
Sorry about the version update, just a habit of mine..

And yes, this works with free license mode. Was a couple of months ago i made the changes so I cant remember all the details. But I believe I was using 4.1. I did not test with either payed licence or 5.x

@maintux
Copy link
Owner

maintux commented Feb 20, 2014

mmm I'm not sure for the free license mode. Usually when you have a free license, you cannot use write api for example 'power_on'. The rename api seems to be exactly a write api so I'm sceptical about it.

Have you tried it with the hipervisor in trial period (with all features enabled)?

@simonjohansson
Copy link
Author

Hi, yes, this was how I used it. In the 60 day trial.

Now I only have licensed hosts so I cant try it again, woe is me. :(

On Thu, Feb 20, 2014 at 10:26 AM, Massimo Maino [email protected]:

mmm I'm not sure for the free license mode. Usually when you have a free
license, you cannot use write api for example 'power_on'. The rename api
seems to be exactly a write api so I'm sceptical about it.

Have you tried it with the hipervisor in trial period (with all features
enabled)?

Reply to this email directly or view it on GitHubhttps://github.com//pull/2#issuecomment-35606507
.

@maintux
Copy link
Owner

maintux commented Feb 20, 2014

Ok I leave this open because I'm very busy right now. I'll fix it asap

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