]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Acceptance: use common bits from puppet-openstack-integration
authorEmilien Macchi <emilien@redhat.com>
Wed, 18 Nov 2015 16:47:31 +0000 (17:47 +0100)
committerEmilien Macchi <emilien@redhat.com>
Thu, 19 Nov 2015 10:09:31 +0000 (11:09 +0100)
Change-Id: I6eb1198d7a375c1168f02c6257055c1db070dc12

spec/acceptance/ceilometer_with_mysql_spec.rb
spec/acceptance/ceilometer_wsgi_apache_spec.rb

index 60f7322a374d210f3456d000bf7b1dc66f42afb0..5ef941cf39c5c972aeaa3a37ffd77c881274ab9f 100644 (file)
@@ -6,8 +6,6 @@ describe 'ceilometer with mysql' do
 
     it 'should work with no errors' do
       pp= <<-EOS
-      Exec { logoutput => 'on_failure' }
-
       # make sure apache is stopped before ceilometer-api eventlet
       # in case of wsgi was run before
       class { '::apache':
@@ -15,39 +13,10 @@ describe 'ceilometer with mysql' do
       }
       Service['httpd'] -> Service['keystone']
 
-      # Common resources
-      case $::osfamily {
-        'Debian': {
-          include ::apt
-          class { '::openstack_extras::repo::debian::ubuntu':
-            release         => 'liberty',
-            package_require => true,
-          }
-          $package_provider = 'apt'
-        }
-        'RedHat': {
-          class { '::openstack_extras::repo::redhat::redhat':
-            release => 'liberty',
-          }
-          package { 'openstack-selinux': ensure => 'latest' }
-          $package_provider = 'yum'
-        }
-        default: {
-          fail("Unsupported osfamily (${::osfamily})")
-        }
-      }
-
-      class { '::mysql::server': }
-
-      class { '::rabbitmq':
-        delete_guest_user => true,
-        package_provider  => $package_provider,
-      }
-
-      rabbitmq_vhost { '/':
-        provider => 'rabbitmqctl',
-        require  => Class['rabbitmq'],
-      }
+      include ::openstack_integration
+      include ::openstack_integration::repos
+      include ::openstack_integration::rabbitmq
+      include ::openstack_integration::mysql
 
       rabbitmq_user { 'ceilometer':
         admin    => true,
@@ -64,7 +33,6 @@ describe 'ceilometer with mysql' do
         require              => Class['rabbitmq'],
       }
 
-
       # Keystone resources, needed by Ceilometer to run
       class { '::keystone::db::mysql':
         password => 'keystone',
index f454bcc1dc1688a71828792152c229702bb67c41..19c5914c42f2b666aa651ac43f3e43fa845a78b9 100644 (file)
@@ -6,41 +6,11 @@ describe 'ceilometer with mysql' do
 
     it 'should work with no errors' do
       pp= <<-EOS
-      Exec { logoutput => 'on_failure' }
-
-      # Common resources
-      case $::osfamily {
-        'Debian': {
-          include ::apt
-          class { '::openstack_extras::repo::debian::ubuntu':
-            release         => 'liberty',
-            package_require => true,
-          }
-          $package_provider = 'apt'
-        }
-        'RedHat': {
-          class { '::openstack_extras::repo::redhat::redhat':
-            release => 'liberty',
-          }
-          package { 'openstack-selinux': ensure => 'latest' }
-          $package_provider = 'yum'
-        }
-        default: {
-          fail("Unsupported osfamily (${::osfamily})")
-        }
-      }
-
-      class { '::mysql::server': }
-
-      class { '::rabbitmq':
-        delete_guest_user => true,
-        package_provider  => $package_provider,
-      }
-
-      rabbitmq_vhost { '/':
-        provider => 'rabbitmqctl',
-        require  => Class['rabbitmq'],
-      }
+      include ::openstack_integration
+      include ::openstack_integration::repos
+      include ::openstack_integration::rabbitmq
+      include ::openstack_integration::mysql
+      include ::openstack_integration::keystone
 
       rabbitmq_user { 'ceilometer':
         admin    => true,
@@ -57,27 +27,6 @@ describe 'ceilometer with mysql' do
         require              => Class['rabbitmq'],
       }
 
-
-      # Keystone resources, needed by Ceilometer to run
-      class { '::keystone::db::mysql':
-        password => 'keystone',
-      }
-      class { '::keystone':
-        verbose             => true,
-        debug               => true,
-        database_connection => 'mysql://keystone:keystone@127.0.0.1/keystone',
-        admin_token         => 'admin_token',
-        enabled             => true,
-      }
-      class { '::keystone::roles::admin':
-        email    => 'test@example.tld',
-        password => 'a_big_secret',
-      }
-      class { '::keystone::endpoint':
-        public_url => "https://${::fqdn}:5000/",
-        admin_url  => "https://${::fqdn}:35357/",
-      }
-
       # Ceilometer resources
       class { '::ceilometer':
         metering_secret     => 'secrete',