PR#148: fixed merge-standard-inventory script code using inspekt - standard-test-roles

standard-test-roles

#148 fixed merge-standard-inventory script code using inspekt

Merged 6 years ago by astepano. Opened 6 years ago by bgoncalv.

bgoncalv/standard-test-roles inspekt-merge-standard-inventory into master

fixed merge-standard-inventory script code using inspekt

Bruno Goncalves • 6 years ago

c9e333b

scripts/merge-standard-inventory

file modified

+2 -1

		`@@ -92,7 +92,7 @@`

		`try:`
		`inv_out = subprocess.check_output(cmd, stdin=None, close_fds=True)`
		`- except subprocess.CalledProcessError as ex:`
		`+ except subprocess.CalledProcessError:`
		`raise RuntimeError("Could not run: {0}".format(str(cmd)))`

		`merged.merge(inv_out.decode('utf-8'))`
		`@@ -203,5 +203,6 @@`
		`"hosts": self.hosts, "vars": {}}, "_meta": {"hostvars": self.variables}}`
		`return json.dumps(data, indent=4, separators=(',', ': '))`

		`+`
		`if __name__ == '__main__':`
		`sys.exit(main(sys.argv))`

bgoncalv commented 6 years ago

results:

# inspekt indent scripts/merge-standard-inventory
Indentation check PASS

# inspekt lint --enable W0611,W0612,W0622 scripts/merge-standard-inventory
Pylint disabled: W,R,C,E1002,E1101,E1103,E1120,F0401,I0011
Pylint enabled : W0611,W0612,W0622
Using config file /dev/null
Syntax check PASS

# pycodestyle --max-line-length=120 scripts/merge-standard-inventory
#