Merge pull request #62 from cisagov/bugfix/fix_parallelism_issues_in_validate

Force the `packer_validate` hook to be run in serial by `pre-commit`

Author: mcdonnnj

.pre-commit-hooks.yaml

@@ -13,3 +13,11 @@
   id: packer_validate
   language: script
   name: Packer Validate
+  # This is necessary because Packer's plugin cache is not safe for concurrent access
+  # per https://github.com/hashicorp/terraform/issues/32915#issuecomment-1485061999. If
+  # this is not set to true then pre-commit itself will perform some level of
+  # parallelization across the files to run this hook against. Since we only care about
+  # unique directory paths it should not impact performance and it will help prevent
+  # issues arising from concurrent attempts to perform `packer init` on a particular
+  # directory path.
+  require_serial: true

README.md

@@ -34,7 +34,7 @@ against any directory that houses `.pkr.hcl` files.
 ```yaml
 repos:
   - repo: https://github.com/cisagov/pre-commit-packer
-    rev: v0.3.0
+    rev: v0.3.1
     hooks:
       - id: packer_fmt
       - id: packer_validate

config/version.txt

@@ -1 +1 @@
-0.3.0
+0.3.1