paasta-tools 1.33.0__tar.gz → 1.33.2__tar.gz

{paasta_tools-1.33.0/paasta_tools.egg-info → paasta_tools-1.33.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: paasta-tools
-Version: 1.33.0
+Version: 1.33.2
 Summary: Tools for Yelps SOA infrastructure
 Author: Compute Infrastructure @ Yelp
 Author-email: compute-infra@yelp.com

{paasta_tools-1.33.0 → paasta_tools-1.33.2}/paasta_tools/__init__.py

@@ -17,4 +17,4 @@
 # setup phase, the dependencies may not exist on disk yet.
 #
 # Don't bump version manually. See `make release` docs in ./Makefile
-__version__ = "1.33.0"
+__version__ = "1.33.2"

{paasta_tools-1.33.0 → paasta_tools-1.33.2}/paasta_tools/cli/cmds/local_run.py

@@ -75,6 +75,12 @@ from paasta_tools.utils import Timeout
 from paasta_tools.utils import TimeoutError
 from paasta_tools.utils import validate_service_instance
 
+STANDARD_PAASTA_CLI_PATHS = {
+    "/bin/paasta",
+    "/opt/venvs/paasta-tools/bin/paasta",
+    "/usr/bin/paasta",
+}
+
 
 class AWSSessionCreds(TypedDict):
     AWS_ACCESS_KEY_ID: str
@@ -1349,6 +1355,26 @@ def docker_config_available():
 
 
 def paasta_local_run(args):
+    if args.action == "pull" and os.geteuid() != 0 and not args.skip_secrets:
+        # we unfortunately sometimes install paasta inside the virtualenv of repos where
+        # folks might be running commands such as `paasta local-run`.
+        # this tends to cause some confusion as this results folks being denied access
+        # to sudo since the virtualenv paasta is not in our sudoers :)
+        # NOTE: in the unlikely event that someone does need to run this from a venv,
+        # there's an opt-out env var
+        if (
+            sys.argv[0] not in STANDARD_PAASTA_CLI_PATHS
+            and "PAASTA_ALLOW_VENV" not in os.environ
+        ):
+            print(
+                "You are running the PaaSTA CLI from a virtualenv - this is unexpected. Re-executing `paasta` from the standard location.."
+            )
+            os.execvp("/usr/bin/paasta", ["/usr/bin/paasta"] + sys.argv[1:])
+        # XXX: we should re-architect this to not need sudo, but for now,
+        # re-exec ourselves with sudo to get access to the paasta vault token
+        # NOTE: once we do that, we can also remove the venv check above :)
+        print("Re-executing paasta local-run --pull with sudo for Vault access...")
+        os.execvp("sudo", ["sudo", "-H", "/usr/bin/paasta"] + sys.argv[1:])
     if args.action == "build" and not makefile_responds_to("cook-image"):
         print(
             "A local Makefile with a 'cook-image' target is required for --build",

{paasta_tools-1.33.0 → paasta_tools-1.33.2}/paasta_tools/kubernetes_tools.py

@@ -2272,6 +2272,7 @@ class KubernetesDeploymentConfig(LongRunningServiceConfig):
             or self.get_prometheus_port() is not None
             or self.should_use_metrics_provider(METRICS_PROVIDER_UWSGI)
             or self.should_use_metrics_provider(METRICS_PROVIDER_GUNICORN)
+            or self.should_use_metrics_provider(METRICS_PROVIDER_WORKER_LOAD)
         ):
             return "true"
         return "false"

{paasta_tools-1.33.0 → paasta_tools-1.33.2/paasta_tools.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: paasta-tools
-Version: 1.33.0
+Version: 1.33.2
 Summary: Tools for Yelps SOA infrastructure
 Author: Compute Infrastructure @ Yelp
 Author-email: compute-infra@yelp.com

{paasta_tools-1.33.0 → paasta_tools-1.33.2}/tests/test_kubernetes_tools.py

@@ -1885,6 +1885,7 @@ class TestKubernetesDeploymentConfig:
         if autoscaling_metric_provider in (
             METRICS_PROVIDER_UWSGI,
             METRICS_PROVIDER_GUNICORN,
+            METRICS_PROVIDER_WORKER_LOAD,
         ):
             routable_ip = "true"