Issue 213 - Make sure kick.py can still parse Puppet files with newer Puppet releases

kick.py

 #!/usr/bin/env python
 # coding: utf-8
 
 import sys
 import os
 import re
 import subprocess
 import getopt
 import yaml
 
@@ skipping 38 matching lines @@
     print >>sys.stderr, 'No user name specified'
     usage()
     sys.exit(1)
 
   return user, mode, args
 
 def readMonitoringConfig():
   # Use Puppet's parser to convert monitoringserver.pp into YAML
   manifest = os.path.join(os.path.dirname(__file__), 'manifests', 'monitoringserver.pp')
   parseScript = '''
+    require 'puppet'
     require 'puppet/parser'
     parser = Puppet::Parser::Parser.new(Puppet[:environment])
+    Puppet.settings[:ignoreimport] = true
     parser.file = ARGV[0]
     print ZAML.dump(parser.parse)
   '''
   data, dummy = subprocess.Popen(['ruby', '', manifest],
                   stdin=subprocess.PIPE,
-                  stdout=subprocess.PIPE,
-                  stderr=subprocess.PIPE).communicate(parseScript)
+                  stdout=subprocess.PIPE).communicate(parseScript)

[Felix Dahlke, 2014/03/26 11:19:55]

Why ignore stderr now?

[Wladimir Palant, 2014/03/26 11:27:41]

Quite the opposite actually - before this change stderr would go into a black
hole, now it will be printed to the default stderr.

 
   # See http://stackoverflow.com/q/8357650/785541 on parsing Puppet's YAML
   yaml.add_multi_constructor(u"!ruby/object:", lambda loader, suffix, node: loader.construct_yaml_map(node))
   yaml.add_constructor(u"!ruby/sym", lambda loader, node: loader.construct_yaml_str(node))
   return yaml.load(data)
 
 def getValidHosts():
   def processNode(node, hosts=None, groups=None):
     if hosts == None:
       hosts = set()
@@ skipping 62 matching lines @@
 if __name__ == "__main__":
   user, mode, args = parseOptions(sys.argv[1:])
   hosts, groups = getValidHosts()
   needKicking = resolveHostList(args, hosts, groups)
   if len(needKicking) == 0:
     print >>sys.stderr, 'No valid hosts or groups specified, nothing to do'
     sys.exit(0)
   updateMaster(user)
   for host in needKicking:
     updateClient(user, host, mode)