-
Notifications
You must be signed in to change notification settings - Fork 479
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
Feature #1454: OpenNebula to OpenNebula drivers #1743
Conversation
share/etc/oned.conf
Outdated
ARGUMENTS = "-t 15 -r 0 one", | ||
TYPE = "xml", | ||
KEEP_SNAPSHOTS = "no", | ||
IMPORTED_VMS_ACTIONS = "terminate, terminate-hard, hold, release, suspend, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Driver actually accept import, this list of actions needs to be consistent with the actions implemented
@@ -722,7 +722,8 @@ static const map<std::string, unsigned int> MAX_HOST_VAR_SIZES = { | |||
{"EC2_SECRET", 41}, | |||
{"AZ_ID", 41}, | |||
{"AZ_CERT", 3130}, | |||
{"VCENTER_PASSWORD", 22} | |||
{"VCENTER_PASSWORD", 22}, | |||
{"ONE_PASSWORD", 22} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
22 is the password limit, it may be increased for OpenNebula, that does not have this limitation. OpenNebula has a x509 driver so 8192 could be enough for that
#!/usr/bin/env ruby | ||
|
||
# -------------------------------------------------------------------------- # | ||
# Copyright 2002-2016, OpenNebula Project, OpenNebula Systems # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update year
@@ -32,7 +32,7 @@ def create(template_json) | |||
host_hash['cluster_id'].to_i) | |||
delete_values = ['name', 'im_mad', 'vm_mad', 'cluster_id'] | |||
|
|||
template_str = hash_to_str(host_hash, delete_values) | |||
template_str = template_to_str(host_hash, delete_values) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this change needed, does it not break other things?
src/vmm_mad/remotes/one/attach_disk
Outdated
#!/bin/bash | ||
|
||
# -------------------------------------------------------------------------- # | ||
# Copyright 2002-2016, OpenNebula Project, OpenNebula Systems # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
year
src/vmm_mad/remotes/one/cancel
Outdated
#!/usr/bin/env ruby | ||
|
||
# -------------------------------------------------------------------------- # | ||
# Copyright 2002-2016, OpenNebula Project, OpenNebula Systems # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
year
@@ -0,0 +1,643 @@ | |||
#!/usr/bin/env ruby | |||
# -------------------------------------------------------------------------- # | |||
# Copyright 2002-2016, OpenNebula Project, OpenNebula Systems # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
year
|
||
secret = "#{region['user']}:#{region['password']}" | ||
|
||
# This breaks the CLI SSL support for Ruby 1.8.7, but is necessary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get rid of this?
end | ||
|
||
# TODO move this method to a OneGate library. See ec2_driver.rb | ||
def generate_onegate_token(xml) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review this todo
state = "" | ||
|
||
state = case vm.state_str | ||
when "INIT" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe better to use a compact form for the same mapping.
when "INIT" || "PENDING" || "HOLD"
VM_STATE[:active]
Also review with states in VirtualMachine.h that we are not missing any new state
No description provided.