Dzahn has submitted this change and it was merged.

Change subject: openstack: lint, fix alignment
......................................................................


openstack: lint, fix alignment

Fixes these files
conductor.pp, network.pp, scheduler.pp and spiceproxy.pp

@Dzahn helped me to to do my first puppet-lint :)

Change-Id: I994ce37253be10fe0ef355f3de24f85025cb78bb
---
M modules/openstack/manifests/nova/conductor.pp
M modules/openstack/manifests/nova/network.pp
M modules/openstack/manifests/nova/scheduler.pp
M modules/openstack/manifests/nova/spiceproxy.pp
4 files changed, 11 insertions(+), 11 deletions(-)

Approvals:
  Dzahn: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/openstack/manifests/nova/conductor.pp 
b/modules/openstack/manifests/nova/conductor.pp
index 6f12395..ec12745 100644
--- a/modules/openstack/manifests/nova/conductor.pp
+++ b/modules/openstack/manifests/nova/conductor.pp
@@ -25,8 +25,8 @@
         }
     } else {
         service { 'nova-conductor':
-            ensure    => stopped,
-            require   => Package['nova-conductor'];
+            ensure  => stopped,
+            require => Package['nova-conductor'];
         }
     }
 }
diff --git a/modules/openstack/manifests/nova/network.pp 
b/modules/openstack/manifests/nova/network.pp
index 03593b2..74024b1 100644
--- a/modules/openstack/manifests/nova/network.pp
+++ b/modules/openstack/manifests/nova/network.pp
@@ -104,8 +104,8 @@
         }
     } else {
         service { 'nova-network':
-            ensure    => stopped,
-            require   => Package['nova-network'];
+            ensure  => stopped,
+            require => Package['nova-network'];
         }
     }
 }
diff --git a/modules/openstack/manifests/nova/scheduler.pp 
b/modules/openstack/manifests/nova/scheduler.pp
index c79fd4c..cfda65c 100644
--- a/modules/openstack/manifests/nova/scheduler.pp
+++ b/modules/openstack/manifests/nova/scheduler.pp
@@ -32,8 +32,8 @@
         }
     } else {
         service { 'nova-scheduler':
-            ensure    => stopped,
-            require   => Package['nova-scheduler'];
+            ensure  => stopped,
+            require => Package['nova-scheduler'];
         }
     }
 }
diff --git a/modules/openstack/manifests/nova/spiceproxy.pp 
b/modules/openstack/manifests/nova/spiceproxy.pp
index d32e2a0..58749a2 100644
--- a/modules/openstack/manifests/nova/spiceproxy.pp
+++ b/modules/openstack/manifests/nova/spiceproxy.pp
@@ -20,7 +20,7 @@
         group   => 'root',
         mode    => '0444',
         source  => 
"puppet:///modules/openstack/${openstack_version}/nova/spice_sec_auto.html",
-        require   => Package['spice-html5'];
+        require => Package['spice-html5'];
     }
 
     if $::fqdn == hiera('labs_nova_controller') {
@@ -39,12 +39,12 @@
         }
     } else {
         service { 'nova-spiceproxy':
-            ensure    => stopped,
-            require   => Package['nova-spiceproxy'];
+            ensure  => stopped,
+            require => Package['nova-spiceproxy'];
         }
         service { 'nova-consoleauth':
-            ensure    => stopped,
-            require   => Package['nova-consoleauth'];
+            ensure  => stopped,
+            require => Package['nova-consoleauth'];
         }
     }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/308235
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I994ce37253be10fe0ef355f3de24f85025cb78bb
Gerrit-PatchSet: 9
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: Andrew Bogott <abog...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Rush <r...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to