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

First version of openstack_hyper_v to align with stackforge/puppet-openstack #48

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

Conversation

luisfdez
Copy link
Contributor

Hi,

This PR does the refactor of the module to follow the new service per module approach. Again, it follows stackforge style and structure to make a future merge easier.

Now, it only provides a compute.pp class to configure nova compute on hyperv. This calls nova_hyper_v components.

I didn't check intensively if the interface is backwards compatible with previous version of the module.

I have to clean a bit more, testing, documentation, but the main content is this one.

Example:

    class {'openstack_hyper_v::compute':
      # Required Network
      internal_address       => $fqdn,
      # Rabbit (*)
      rabbit_host            => 'rabbit-host',
      rabbit_password        => hiera('rabbit_password'),
      rabbit_user            => 'nova',
      rabbit_virtual_host    => '/',
      # Glance
      glance_api_servers     => 'glance-server:9393',
      # General
      verbose                => true,
      debug                  => true,
      enabled                => true,
      nova_version           => "2013.2.1",
      migration_support      => true,
      virtual_switch_address => inline_template("<%= require 'resolv'; Resolv.new.getaddress(@fqdn)%>"),
      hyperv_service_user    => 'DOMAIN\service-user',
      hyperv_service_pass    => hiera('hyperv_service_pass', ''),
    }

@ppouliot
Copy link
Owner

This should be merged sooner than later. We need to validate that we're not going to break any of our existing bits for the CI. Which I do not think we will. That being said, let me verify and we can move forward asap.

p

@benh57
Copy link

benh57 commented Oct 17, 2014

Would love to see this integrated.

@luisfdez luisfdez force-pushed the refactoring-align-stackforge branch from 8e7386f to ff8bbcf Compare October 20, 2014 09:18
@luisfdez
Copy link
Contributor Author

@benh57, if you're interested in this PR, you can use it from my fork until it's merged in the master repo.

Feedback & issues are welcome :)

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.

3 participants