Wrap -api requirements in install flag #809

goodrobots · Mar 27, 2019 · fb9ec47 · fb9ec47
1 parent cf7458a
commit fb9ec47
Showing 1 changed file with 17 additions and 10 deletions.
diff --git a/manifests/maverick-modules/maverick_web/manifests/maverick_api.pp b/manifests/maverick-modules/maverick_web/manifests/maverick_api.pp
@@ -1,16 +1,23 @@
 class maverick_web::maverick_api (
 ) {
 
-    # install python components
-    python::requirements { "/srv/maverick/code/maverick-api/requirements.txt":
-        cwd             => "/srv/maverick/code/maverick-api",
-        pip_provider    => "pip3",
-        environment     => ["PATH=/srv/maverick/software/python/bin:\$PATH"],
-        timeout         => 0,
-        forceupdate     => true,
-        fix_requirements_owner => false,
-        manage_requirements => false,
-    } ->
+    # Install python components
+    # python::requirements isn't idempotent, so only run once on initial install
+    if ! ("install_flag_apirequirements" in $installflags) {
+        python::requirements { "/srv/maverick/code/maverick-api/requirements.txt":
+            cwd             => "/srv/maverick/code/maverick-api",
+            pip_provider    => "pip3",
+            environment     => ["PATH=/srv/maverick/software/python/bin:\$PATH"],
+            timeout         => 0,
+            forceupdate     => true,
+            fix_requirements_owner => false,
+            manage_requirements => false,
+            before          => Oncevcsrepo["git-maverick-api"],
+        } ->
+        file { "/srv/maverick/var/build/.install_flag_apirequirements":
+            ensure      => present,
+        }
+    }
     oncevcsrepo { "git-maverick-api":
         gitsource   => "https://github.com/goodrobots/maverick-api.git",
         dest        => "/srv/maverick/code/maverick-api",