Updates to scripts after refactor

- Run black to reformat all the scripts
- Update makefile test targets, pylint, and fix some of the issues found
- Update pxeconfig script for refactored nbhelper
- Add start of inventory script

Change-Id: I5f426ac2da840dc72f07f8a6844e199e47d49135
diff --git a/cookiecutters/role/hooks/post_gen_project.py b/cookiecutters/role/hooks/post_gen_project.py
index 49667ae..c915d1f 100644
--- a/cookiecutters/role/hooks/post_gen_project.py
+++ b/cookiecutters/role/hooks/post_gen_project.py
@@ -34,7 +34,7 @@
 
 # script is rendered as a template, so this will be filled in with the
 # cookiecutter dict, which is why noqa is needed.
-CONTEXT = {{cookiecutter | jsonify}}  # noqa: F821, E227
+CONTEXT = {{cookiecutter | jsonify}}  # noqa: F821, E227  pylint: disable=E0602
 
 
 def delete_file(filepath):
@@ -59,7 +59,7 @@
         outfile_tmpl = env.from_string(infile)
         delete_file(outfile_tmpl.render({"cookiecutter": CONTEXT}))
 
-    platforms = list(CONTEXT["platforms"].keys())
+    platforms = list(CONTEXT["platforms"].keys())  # pylint: disable=E1136
 
     # Combine Ubuntu and Debian as they're the same ansible_os_family
     if "Ubuntu" in platforms and "Debian" not in platforms:
@@ -98,7 +98,7 @@
 
     # delete any files that don't start with the license identifier
     for licfile in license_files:
-        if not licfile.startswith(CONTEXT["license"]):
+        if not licfile.startswith(CONTEXT["license"]):  # pylint: disable=E1136
             os.remove(os.path.join(os.path.curdir, "LICENSES", licfile))