Merge pull request #480 from marcusdots/modulesync

workaround: lint check disabled. See description for details

Author: marcusdots

.github/workflows/ci.yml

@@ -23,3 +23,5 @@ jobs:
   puppet:
     name: Puppet
     uses: voxpupuli/gha-puppet/.github/workflows/beaker.yml@v4
+    with:
+      beaker_hypervisor: 'docker'

.sync.yml

@@ -11,3 +11,6 @@
   secure: "iStKD31BM2tH9V3IbVJ0BTekwPWwhZbj4rb+6Xiz44cL4pNTHX3ymiAmbGjP0tlVbkzr1+jgUiTXG7ubLG0lTr2mjbgmBWa8ZcyH933HTZ7pzwIUtBjKuZjHT9vG1qZ3MPLDgb9R/i1qRj9rk6pRNlAyhqk5B/ynmO1uzcPneQHo8wToN/WvFGQIw/9I/FFWQQO/6Ap9LpD7xSF5ZAcg94tMpTkBebEuibZOi3Ze5Ah3zf2vHOOFSRcTTrFynyn1mYkUFU1xaOuoqvyTp7Tv+M1+uB1zIXJSJ4oNNeWsC5mLkKz1UBT5GQN7CFaFLawmGqhLQx29sk483Y5+K1NIX8UST47oZFc+bTQjAWJPlVj16lMApB1AXBzov3Cdz3n7GOfZla4BK/2FLqXXbG2KLMMQS/dG3rnLFM4vCT3ED48ZIXV4mb+wpbiXPV4HlOhEbtMSNwXwgWzq3+9yn2PO0rrmGA8tYeY1+ffHoyYGu4aiXO6xzQE8Xbp4LqDeYzRRM078t5BlYyuaU6ZTYgCVFoS9weQETGD1soF8nRObD9Flk2kB+XRIaQytIkQwOR8B28Z44L4ha5syfqvNbuNcJWHvJJNkhT9WlfdcQdmTdIPhglrCGrPtiVp83Ictps6qL5mJRpWcVQarubOGac2YfWC6CBFvxgo7lMFSoYFR3gc="
 spec/spec_helper.rb:
   hiera_config: "File.expand_path(File.join(__dir__, 'fixtures', 'hiera.yaml'))"
+
+.github/workflows/ci.yml:
+  beaker_hypervisor: 'docker'

manifests/service.pp

@@ -51,13 +51,17 @@
   }
 
   if ($service_manage and $service_provider_restart) {
+    #lint:ignore:exec_idempotency
     exec { 'gitlab_reconfigure':
       notify => Service[$service_name],
       *      => $_reconfigure_attributes,
     }
+    #lint:endignore
   } else {
+    #lint:ignore:exec_idempotency
     exec { 'gitlab_reconfigure':
       * => $_reconfigure_attributes,
     }
+    #lint:endignore
   }
 }