remove debug by default, switch to infolevel and verify -d works
Signed-off-by: Nico Schottelius <nico@kr.ethz.ch>
This commit is contained in:
parent
5702706adf
commit
c78ce344e7
1 changed files with 10 additions and 5 deletions
11
bin/cdist
11
bin/cdist
|
@ -81,7 +81,7 @@ VERSION = "2.0.0"
|
||||||
#
|
#
|
||||||
|
|
||||||
|
|
||||||
logging.basicConfig(level=logging.DEBUG, format='%(levelname)s: %(message)s')
|
logging.basicConfig(level=logging.INFO, format='%(levelname)s: %(message)s')
|
||||||
log = logging.getLogger()
|
log = logging.getLogger()
|
||||||
|
|
||||||
# List types
|
# List types
|
||||||
|
@ -406,6 +406,9 @@ class Cdist:
|
||||||
remote_cmd = cmd + [os.path.join(self.remote_type_explorer_dir(type), explorer)]
|
remote_cmd = cmd + [os.path.join(self.remote_type_explorer_dir(type), explorer)]
|
||||||
output = os.path.join(self.type_explorer_output_dir(cdist_object), explorer)
|
output = os.path.join(self.type_explorer_output_dir(cdist_object), explorer)
|
||||||
output_fd = open(output, mode='w')
|
output_fd = open(output, mode='w')
|
||||||
|
log.debug("%s exploring %s using %s storing to %s",
|
||||||
|
cdist_object, explorer, remote_cmd, output)
|
||||||
|
|
||||||
self.run_or_fail(remote_cmd, stdout=output_fd, remote=True)
|
self.run_or_fail(remote_cmd, stdout=output_fd, remote=True)
|
||||||
output_fd.close()
|
output_fd.close()
|
||||||
|
|
||||||
|
@ -431,6 +434,7 @@ class Cdist:
|
||||||
type = self.get_type_from_object(cdist_object)
|
type = self.get_type_from_object(cdist_object)
|
||||||
manifest = self.type_manifest_path(type)
|
manifest = self.type_manifest_path(type)
|
||||||
|
|
||||||
|
log.debug("%s: Running %s", cdist_object, manifest)
|
||||||
# FIXME: add more sensible checks for manifest
|
# FIXME: add more sensible checks for manifest
|
||||||
if os.path.exists(manifest):
|
if os.path.exists(manifest):
|
||||||
env = { "__object" : self.object_dir(cdist_object),
|
env = { "__object" : self.object_dir(cdist_object),
|
||||||
|
@ -442,7 +446,7 @@ class Cdist:
|
||||||
|
|
||||||
def run_manifest(self, manifest, extra_env=None):
|
def run_manifest(self, manifest, extra_env=None):
|
||||||
"""Run a manifest"""
|
"""Run a manifest"""
|
||||||
log.info("Running manifest %s, env=%s", manifest, extra_env)
|
log.debug("Running manifest %s, env=%s", manifest, extra_env)
|
||||||
env = os.environ.copy()
|
env = os.environ.copy()
|
||||||
env['PATH'] = self.bin_dir + ":" + env['PATH']
|
env['PATH'] = self.bin_dir + ":" + env['PATH']
|
||||||
|
|
||||||
|
@ -502,7 +506,6 @@ class Cdist:
|
||||||
paths = self.type_gencode_paths(self.get_type_from_object(cdist_object))
|
paths = self.type_gencode_paths(self.get_type_from_object(cdist_object))
|
||||||
for bin in paths:
|
for bin in paths:
|
||||||
if os.path.isfile(bin):
|
if os.path.isfile(bin):
|
||||||
print(bin)
|
|
||||||
self.shell_run_or_debug_fail(bin, [bin], env=env)
|
self.shell_run_or_debug_fail(bin, [bin], env=env)
|
||||||
|
|
||||||
if mode == "code":
|
if mode == "code":
|
||||||
|
@ -546,6 +549,8 @@ class Cdist:
|
||||||
self.object_run(cdist_object, mode="gencode")
|
self.object_run(cdist_object, mode="gencode")
|
||||||
self.object_run(cdist_object, mode="code")
|
self.object_run(cdist_object, mode="code")
|
||||||
|
|
||||||
|
log.info("Finished run of %s", self.target_host)
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
parser = argparse.ArgumentParser(description='cdist ' + VERSION)
|
parser = argparse.ArgumentParser(description='cdist ' + VERSION)
|
||||||
parser.add_argument('host', nargs='*', help='one or more hosts to operate on')
|
parser.add_argument('host', nargs='*', help='one or more hosts to operate on')
|
||||||
|
|
Loading…
Reference in a new issue