From fdd01d84a8cd653511648ef86c2bd0370ca6cfb1 Mon Sep 17 00:00:00 2001
From: "James E. Blair" <jeblair@hp.com>
Date: Thu, 14 Aug 2014 15:28:47 -0700
Subject: [PATCH] Nodepool dependens indirectly on lxml

Nodepool now depends indirectly on lxml via keystoneclient.  To pip
install lxml you need libxml and libxslt development packages on the
system to provide headers so that the library bindings can be
compiled, so it's easier to just use the python-lxml system packages
instead.

Co-Authored-By: Jeremy Stanley <fungi@yuggoth.org>
Change-Id: Id5c2070250cd9474c83edaa6ed1119a5325ea913
---
 modules/nodepool/manifests/init.pp | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/modules/nodepool/manifests/init.pp b/modules/nodepool/manifests/init.pp
index d0b81ac84f..c05dae5a52 100644
--- a/modules/nodepool/manifests/init.pp
+++ b/modules/nodepool/manifests/init.pp
@@ -30,6 +30,14 @@ class nodepool (
   $environment = {},
 ) {
 
+  $packages = [
+    'python-lxml', # needed by python-keystoneclient, has system bindings
+  ]
+
+  package { $packages:
+    ensure => present,
+  }
+
   class { 'mysql::server':
     config_hash => {
       'root_password'  => $mysql_root_password,
@@ -87,7 +95,10 @@ class nodepool (
     path        => '/usr/local/bin:/usr/bin:/bin/',
     refreshonly => true,
     subscribe   => Vcsrepo['/opt/nodepool'],
-    require     => Class['pip'],
+    require     => [
+      Class['pip'],
+      Package['python-lxml'],
+    ],
   }
 
   file { '/etc/nodepool':