paasta-tools 1.30.7__tar.gz → 1.35.4__tar.gz

{paasta_tools-1.30.7/paasta_tools.egg-info → paasta_tools-1.35.4}/PKG-INFO

@@ -1,11 +1,11 @@
 Metadata-Version: 2.4
 Name: paasta-tools
-Version: 1.30.7
+Version: 1.35.4
 Summary: Tools for Yelps SOA infrastructure
 Author: Compute Infrastructure @ Yelp
 Author-email: compute-infra@yelp.com
 Provides: paasta_tools
-Requires-Python: >=3.8.0
+Requires-Python: >=3.9.0
 License-File: LICENSE
 Requires-Dist: a-sync>=0.5.0
 Requires-Dist: aiohttp>=3.5.4
@@ -34,7 +34,7 @@ Requires-Dist: ipaddress>=1.0.22
 Requires-Dist: isodate>=0.7.2
 Requires-Dist: jsonschema[format]
 Requires-Dist: kazoo>=2.0.0
-Requires-Dist: kubernetes<26.0.0,>=18.20.0
+Requires-Dist: kubernetes<35.0.0,>=29.0.0
 Requires-Dist: ldap3
 Requires-Dist: manhole
 Requires-Dist: mypy-extensions>=0.3.0

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/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.30.7"
+__version__ = "1.35.4"

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/api/api_docs/swagger.json

@@ -1788,6 +1788,11 @@
         "KubernetesVersion": {
             "type": "object",
             "properties": {
+                "container_port": {
+                    "description": "Port the container is expecting to receive traffic on",
+                    "type": "integer",
+                    "format": "int32"
+                },
                 "type": {
                     "description": "Type of version (ReplicaSet or ControllerRevision)",
                     "type": "string"

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/api/views/instance.py

@@ -26,6 +26,7 @@ from typing import Mapping
 from typing import Optional
 
 import a_sync
+from pyramid.request import Request
 from pyramid.response import Response
 from pyramid.view import view_config
 
@@ -139,7 +140,10 @@ def no_configuration_for_service_message(cluster, service, instance):
 @view_config(
     route_name="service.instance.status", request_method="GET", renderer="json"
 )
-def instance_status(request):
+def instance_status(
+    request: Request,
+) -> dict[str, Any]:  # godspeed to anyone typing the retval here
+    # NOTE: swagger_data is populated by pyramid_swagger
     service = request.swagger_data.get("service")
     instance = request.swagger_data.get("instance")
     verbose = request.swagger_data.get("verbose") or 0
@@ -353,7 +357,10 @@ def get_deployment_version(
     request_method="GET",
     renderer="json",
 )
-def instance_mesh_status(request):
+def instance_mesh_status(
+    request: Request,
+) -> dict[str, Any]:  # godspeed to anyone typing the retval here
+    # NOTE: swagger_data is populated by pyramid_swagger
     service = request.swagger_data.get("service")
     instance = request.swagger_data.get("instance")
     include_envoy = request.swagger_data.get("include_envoy")

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/async_utils.py

@@ -3,9 +3,11 @@ import functools
 import time
 import weakref
 from collections import defaultdict
+from typing import Any
 from typing import AsyncIterable
 from typing import Awaitable
 from typing import Callable
+from typing import Coroutine
 from typing import Dict
 from typing import List
 from typing import Optional
@@ -97,7 +99,8 @@ async def aiter_to_list(
 def async_timeout(
     seconds: int = 10,
 ) -> Callable[
-    [Callable[..., Awaitable[T]]], Callable[..., Awaitable[T]]  # wrapped  # inner
+    [Callable[..., Coroutine[Any, Any, T]]],
+    Callable[..., Coroutine[Any, Any, T]],  # wrapped  # inner
 ]:
     def outer(wrapped):
         @functools.wraps(wrapped)

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/bounce_lib.py

@@ -42,11 +42,14 @@ BounceMethodResult = TypedDict(
 
 BounceMethod = Callable[
     [
-        Arg(BounceMethodConfigDict, "new_config"),
-        Arg(bool, "new_app_running"),
-        Arg(Collection, "happy_new_tasks"),
-        Arg(Sequence, "old_non_draining_tasks"),
-        DefaultArg(float, "margin_factor"),
+        Arg(
+            BounceMethodConfigDict,
+            "new_config",  # noqa: F821  # flake8 false-positive, these are not var references
+        ),
+        Arg(bool, "new_app_running"),  # noqa: F821  # flake8 false-positive
+        Arg(Collection, "happy_new_tasks"),  # noqa: F821  # flake8 false-positive
+        Arg(Sequence, "old_non_draining_tasks"),  # noqa: F821  # flake8 false-positive
+        DefaultArg(float, "margin_factor"),  # noqa: F821  # flake8 false-positive
     ],
     BounceMethodResult,
 ]

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/check_services_replication_tools.py

@@ -56,10 +56,16 @@ log = logging.getLogger(__name__)
 
 CheckServiceReplication = Callable[
     [
-        Arg(InstanceConfig_T, "instance_config"),
-        Arg(Dict[str, Dict[str, List[V1Pod]]], "pods_by_service_instance"),
-        Arg(Any, "replication_checker"),
-        NamedArg(bool, "dry_run"),
+        Arg(
+            InstanceConfig_T,
+            "instance_config",  # noqa: F821  # flake8 false-positive, these are not var references
+        ),
+        Arg(
+            Dict[str, Dict[str, List[V1Pod]]],
+            "pods_by_service_instance",  # noqa: F821  # flake8 false-positive
+        ),
+        Arg(Any, "replication_checker"),  # noqa: F821  # flake8 false-positive
+        NamedArg(bool, "dry_run"),  # noqa: F821  # flake8 false-positive
     ],
     Optional[bool],
 ]

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/check_spark_jobs.py

@@ -124,7 +124,7 @@ def format_framework(info):
 def format_message_for_service(service, frameworks):
     output = f"Found the following long-running Spark frameworks associated with service {service}.\n"
     output += (
-        f"Please check why they are still running and terminate if appropriate.\n\n"
+        "Please check why they are still running and terminate if appropriate.\n\n"
     )
     output += "\n".join(format_framework(f) for f in frameworks)
     return output

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cli.py

@@ -68,7 +68,7 @@ class PrintsHelpOnErrorArgumentParser(argparse.ArgumentParser):
 def list_external_commands():
     p = subprocess.check_output(["/bin/bash", "-p", "-c", "compgen -A command paasta-"])
     lines = p.decode("utf-8").strip().split("\n")
-    return {l.replace("paasta-", "", 1) for l in lines}
+    return {line.replace("paasta-", "", 1) for line in lines}
 
 
 def calling_external_command():
@@ -132,10 +132,10 @@ PAASTA_SUBCOMMANDS = {
 }
 
 
-def get_argparser(commands=None):
+def get_argparser(commands: list[str] | None = None) -> argparse.ArgumentParser:
     """Create and return argument parser for a set of subcommands.
 
-    :param commands: Union[None, List[str]] If `commands` argument is `None`,
+    :param commands: list[str] | None: If `commands` argument is `None`,
     add full parsers for all subcommands, if `commands` is empty list -
     add thin parsers for all subcommands, otherwise - add full parsers for
     subcommands in the argument.
@@ -170,7 +170,7 @@ def get_argparser(commands=None):
 
     # Adding a separate help subparser allows us to respond to "help" without --help
     help_parser = subparsers.add_parser(
-        "help", help=f"run `paasta <subcommand> -h` for help"
+        "help", help="run `paasta <subcommand> -h` for help"
     )
     help_parser.set_defaults(command=None)
 

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/local_run.py

@@ -12,6 +12,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+import base64
 import datetime
 import json
 import os
@@ -31,7 +32,9 @@ from urllib.parse import urlparse
 import boto3
 import requests
 from docker import errors
+from docker.api.client import APIClient
 from mypy_extensions import TypedDict
+from service_configuration_lib import read_service_configuration
 
 from paasta_tools.adhoc_tools import get_default_interactive_config
 from paasta_tools.cli.authentication import get_service_auth_token
@@ -612,26 +615,83 @@ class LostContainerException(Exception):
     pass
 
 
-def docker_pull_image(docker_url):
-    """Pull an image via ``docker pull``. Uses the actual pull command instead of the python
-    bindings due to the docker auth/registry transition. Once we are past Docker 1.6
-    we can use better credential management, but for now this function assumes the
-    user running the command has already been authorized for the registry"""
+class DockerAuthConfig(TypedDict):
+    username: str
+    password: str
+
+
+def get_readonly_docker_registry_auth_config(
+    docker_url: str,
+) -> DockerAuthConfig | None:
+    system_paasta_config = load_system_paasta_config()
+    config_path = system_paasta_config.get_readonly_docker_registry_auth_file()
+
+    try:
+        with open(config_path) as f:
+            docker_config = json.load(f)
+    except Exception:
+        print(
+            PaastaColors.yellow(
+                "Warning: unable to load read-only docker registry credentials."
+            ),
+            file=sys.stderr,
+        )
+        # the best we can do is try to pull with whatever auth the user has configured locally
+        # i.e., root-owned docker config in /root/.docker/config.json
+        return None
+    registry = docker_url.split("/")[0]
+
+    # find matching auth config - our usual ro config will have at least two entries
+    # at the time this comment was written
+    auths = docker_config
+    for auth_url, auth_data in auths.items():
+        if registry in auth_url:
+            # Decode the base64 auth string if present
+            if "auth" in auth_data:
+                auth_string = base64.b64decode(auth_data["auth"]).decode("utf-8")
+                username, password = auth_string.split(":", 1)
+                return {"username": username, "password": password}
+
+    # we'll hit this for registries like docker-dev or extra-private internal registries
+    return None
+
+
+def docker_pull_image(docker_client: APIClient, docker_url: str) -> None:
+    """Pull an image using the docker-py library with read-only registry credentials"""
     print(
-        "Please wait while the image (%s) is pulled (times out after 30m)..."
-        % docker_url,
+        f"Please wait while the image ({docker_url}) is pulled (times out after 30m)...",
         file=sys.stderr,
     )
-    with Timeout(
-        seconds=1800, error_message=f"Timed out pulling docker image from {docker_url}"
-    ), open(os.devnull, mode="wb") as DEVNULL:
-        ret, _ = _run("docker pull %s" % docker_url, stream=True, stdin=DEVNULL)
-        if ret != 0:
-            print(
-                "\nPull failed. Are you authorized to run docker commands?",
-                file=sys.stderr,
-            )
-            sys.exit(ret)
+
+    auth_config = get_readonly_docker_registry_auth_config(docker_url)
+    if not auth_config:
+        print(
+            PaastaColors.yellow(
+                "Warning: No read-only docker registry credentials found, attempting to pull without them."
+            ),
+            file=sys.stderr,
+        )
+
+    try:
+        with Timeout(
+            seconds=1800,
+            error_message=f"Timed out pulling docker image from {docker_url}",
+        ):
+            # this is slightly funky since pull() returns the output line-by-line, but as a dict
+            # ...that we then need to format back to the usual `docker pull` output
+            # :p
+            for line in docker_client.pull(
+                docker_url, auth_config=auth_config, stream=True, decode=True
+            ):
+                # not all lines have an 'id' key :(
+                id_prefix = f"{line['id']}: " if "id" in line else ""
+                print(f"{id_prefix}{line['status']}", file=sys.stderr)
+    except Exception as e:
+        print(
+            f"\nPull failed. Error: {e}",
+            file=sys.stderr,
+        )
+        sys.exit(1)
 
 
 def get_container_id(docker_client, container_name):
@@ -1226,7 +1286,7 @@ def configure_and_run_docker_container(
             return 1
 
     if pull_image:
-        docker_pull_image(docker_url)
+        docker_pull_image(docker_client, docker_url)
 
     for volume in instance_config.get_volumes(
         system_paasta_config.get_volumes(),
@@ -1302,10 +1362,40 @@ def docker_config_available():
     )
 
 
+def should_reexec_as_root(
+    service: str, skip_secrets: bool, action: str, soa_dir: str = DEFAULT_SOA_DIR
+) -> bool:
+    # local-run can't pull secrets from Vault in prod without a root-owned token
+    need_vault_token = not skip_secrets and action == "pull"
+
+    # there are some special teams with their own private docker registries and no ro creds
+    # however, we don't know what registry is to be used without loading the service config
+    service_info = read_service_configuration(service, soa_dir)
+    # technically folks can set the standard registry as a value here, but atm no one is doing that :p
+    registry_override = service_info.get("docker_registry")
+    # note: we could also have a list of registries that have ro creds, but this seems fine for now
+    uses_private_registry = (
+        registry_override
+        and registry_override
+        in load_system_paasta_config().get_private_docker_registries()
+    )
+    need_docker_config = uses_private_registry and action == "pull"
+
+    return (need_vault_token or need_docker_config) and os.geteuid() != 0
+
+
 def paasta_local_run(args):
-    if args.action == "pull" and os.geteuid() != 0 and not docker_config_available():
-        print("Re-executing paasta local-run --pull with sudo..")
-        os.execvp("sudo", ["sudo", "-H"] + sys.argv)
+    service = figure_out_service_name(args, soa_dir=args.yelpsoa_config_root)
+    if should_reexec_as_root(
+        service, args.skip_secrets, args.action, args.yelpsoa_config_root
+    ):
+        # 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/Docker registry 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",
@@ -1332,8 +1422,6 @@ def paasta_local_run(args):
 
     local_run_config = system_paasta_config.get_local_run_config()
 
-    service = figure_out_service_name(args, soa_dir=args.yelpsoa_config_root)
-
     if args.cluster:
         cluster = args.cluster
     else:

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/logs.py

@@ -30,6 +30,7 @@ from typing import Callable
 from typing import ContextManager
 from typing import Dict
 from typing import Iterable
+from typing import Iterator
 from typing import List
 from typing import Mapping
 from typing import MutableSequence
@@ -52,10 +53,31 @@ try:
 except ImportError:
     scribereader = None
 
+# NOTE: this is an internal-only package, so we won't be able to typecheck against it with mypy
+# without these hacky inlined stubs
 try:
     from logreader.readers import S3LogsReader
+
+    s3reader_available = True
 except ImportError:
-    S3LogsReader = None
+    s3reader_available = False
+
+    class S3LogsReader:  # type: ignore[no-redef]  # stub class for internal-only package
+        def __init__(self, superregion: str) -> None:
+            raise ImportError(
+                "logreader (internal Yelp package) is not available - unable to display logs."
+            )
+
+        def get_log_reader(
+            self,
+            log_name: str,
+            start_datetime: datetime.datetime,
+            end_datetime: datetime.datetime,
+        ) -> Iterator[str]:
+            raise NotImplementedError(
+                "logreader (internal Yelp package) is not available - unable to display logs."
+            )
+
 
 from pytimeparse.timeparse import timeparse
 
@@ -156,7 +178,7 @@ def add_subparser(subparsers) -> None:
         dest="soa_dir",
         metavar="SOA_DIR",
         default=DEFAULT_SOA_DIR,
-        help=f"Define a different soa config directory. Defaults to %(default)s.",
+        help="Define a different soa config directory. Defaults to %(default)s.",
     )
 
     status_parser.add_argument(
@@ -1174,7 +1196,7 @@ class VectorLogsReader(LogReader):
     ) -> None:
         super().__init__()
 
-        if S3LogsReader is None:
+        if not s3reader_available:
             raise Exception("yelp_clog package must be available to use S3LogsReader")
 
         self.cluster_map = cluster_map
@@ -1226,16 +1248,16 @@ class VectorLogsReader(LogReader):
                 except ValueError:
                     timestamp = pytz.utc.localize(datetime.datetime.min)
 
-                line = {"raw_line": line, "sort_key": timestamp}
-                aggregated_logs.append(line)
+                formatted_line = {"raw_line": line, "sort_key": timestamp}
+                aggregated_logs.append(formatted_line)
 
         aggregated_logs = list(
             {line["raw_line"]: line for line in aggregated_logs}.values()
         )
         aggregated_logs.sort(key=lambda log_line: log_line["sort_key"])
 
-        for line in aggregated_logs:
-            print_log(line["raw_line"], levels, raw_mode, strip_headers)
+        for formatted_line in aggregated_logs:
+            print_log(formatted_line["raw_line"], levels, raw_mode, strip_headers)
 
     def tail_logs(
         self,

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/mark_for_deployment.py

@@ -1157,7 +1157,7 @@ class MarkForDeploymentProcess(RollbackSlackDeploymentProcess):
 
     def on_enter_deploy_errored(self) -> None:
         report_waiting_aborted(self.service, self.deploy_group)
-        self.update_slack_status(f"Deploy aborted, but it will still try to converge.")
+        self.update_slack_status("Deploy aborted, but it will still try to converge.")
         self.send_manual_rollback_instructions()
         if self.deploy_group_is_set_to_notify("notify_after_abort"):
             self.ping_authors("Deploy errored")
@@ -1264,7 +1264,15 @@ class MarkForDeploymentProcess(RollbackSlackDeploymentProcess):
             self.ping_authors()
 
     def send_manual_rollback_instructions(self) -> None:
-        if self.deployment_version != self.old_deployment_version:
+        # NOTE: new deploy groups are not exactly a particularly frequent occurrence, but
+        # we want to prevent sending messages that look like
+        # `If you need to roll back manually, run: paasta rollback --service $S --deploy-group $G --commit None`
+        # since that's not actually valid/actionable.
+        # thus the seemingly out-of-nowhere old_git_sha check: new deploy groups won't have value set there.
+        if (
+            self.deployment_version != self.old_deployment_version
+            and self.old_git_sha is not None
+        ):
             extra_rollback_args = ""
             if self.old_deployment_version.image_version:
                 extra_rollback_args = (
@@ -1317,7 +1325,7 @@ class MarkForDeploymentProcess(RollbackSlackDeploymentProcess):
         inactive_button_texts = {
             "forward": f"Continue Forward to {version_short_str} :arrow_forward:",
             "complete": f"Complete deploy to {version_short_str} :white_check_mark:",
-            "snooze": f"Reset countdown",
+            "snooze": "Reset countdown",
             "enable_auto_rollbacks": "Enable auto rollbacks :eyes:",
             "disable_auto_rollbacks": "Disable auto rollbacks :close_eyes_monkey:",
         }

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/mesh_status.py

@@ -124,7 +124,7 @@ def paasta_mesh_status_on_api_endpoint(
             [PaastaColors.red(f"Could not connect to API: {exc.__class__.__name__}")],
         )
     except Exception as e:
-        output = [PaastaColors.red(f"Exception when talking to the API:")]
+        output = [PaastaColors.red("Exception when talking to the API:")]
         output.extend(str(e).split("\n"))
         return 1, output
 

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/remote_run.py

@@ -116,7 +116,7 @@ def paasta_remote_run_copy(
     )
     if poll_response.status != 200:
         print(
-            f"Unable to find running remote-run pod: have you started one with `paasta remote-run start`?"
+            "Unable to find running remote-run pod: have you started one with `paasta remote-run start`?"
         )
         return 1
 
@@ -295,11 +295,19 @@ def paasta_remote_run_stop(
 
 
 def add_common_args_to_parser(parser: argparse.ArgumentParser):
+    def _validate_service_name(name: str) -> str:
+        if name not in _list_services_and_toolboxes():
+            raise ValueError(f"{name} is not a known service name")
+        return name
+
     service_arg = parser.add_argument(
         "-s",
         "--service",
         help="The name of the service you wish to inspect. Required.",
         required=True,
+        # not using `choices` for validation to avoid the help text
+        # for the command being incredibly large
+        type=_validate_service_name,
     )
     service_arg.completer = lazy_choices_completer(_list_services_and_toolboxes)  # type: ignore
     instance_or_toolbox = parser.add_mutually_exclusive_group()
@@ -323,6 +331,7 @@ def add_common_args_to_parser(parser: argparse.ArgumentParser):
         "--cluster",
         help="The name of the cluster you wish to run your task on. Required.",
         required=True,
+        choices=list_clusters(),
     )
     cluster_arg.completer = lazy_choices_completer(list_clusters)  # type: ignore
 

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/rollback.py

@@ -340,7 +340,7 @@ def paasta_rollback(args: argparse.Namespace) -> int:
         )
         print(
             PaastaColors.yellow(
-                f"WARNING: Failing to do so means that Jenkins will redeploy the latest code on the next scheduled build!"
+                "WARNING: Failing to do so means that Jenkins will redeploy the latest code on the next scheduled build!"
             )
         )
 

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/secret.py

@@ -212,7 +212,7 @@ def _add_vault_auth_args(parser: argparse.ArgumentParser):
         dest="vault_auth_method",
         required=False,
         default="token",
-        choices=["token", "ldap"],
+        choices=["token", "okta"],
     )
     parser.add_argument(
         "--vault-token-file",
@@ -456,7 +456,7 @@ def paasta_secret(args):
                 "vault_token_file": args.vault_token_file,
                 # best solution so far is to change the below string to "token",
                 # so that token file is picked up from argparse
-                "vault_auth_method": "ldap",  # must have LDAP to get 2FA push for prod
+                "vault_auth_method": "okta",  # must use Okta to get 2FA push
             },
         )
         secret_provider.write_secret(

{paasta_tools-1.30.7 → paasta_tools-1.35.4}/paasta_tools/cli/cmds/spark_run.py

@@ -897,20 +897,22 @@ def configure_and_run_docker_container(
         )
     )  # type:ignore
     environment.update(extra_driver_envs)
+    if "jupyter-lab" == args.cmd:
+        environment["SPARK_DRIVER_TYPE"] = "jupyter"
 
     if args.use_service_auth_token:
         environment["YELP_SVC_AUTHZ_TOKEN"] = get_service_auth_token()
 
     webui_url = get_webui_url(spark_conf["spark.ui.port"])
-    webui_url_msg = PaastaColors.green(f"\nSpark monitoring URL: ") + f"{webui_url}\n"
+    webui_url_msg = PaastaColors.green("\nSpark monitoring URL: ") + f"{webui_url}\n"
 
     docker_cmd = get_docker_cmd(args, instance_config, spark_conf_str)
     if "history-server" in docker_cmd:
-        print(PaastaColors.green(f"\nSpark history server URL: ") + f"{webui_url}\n")
+        print(PaastaColors.green("\nSpark history server URL: ") + f"{webui_url}\n")
     elif any(c in docker_cmd for c in ["pyspark", "spark-shell", "spark-submit"]):
         grafana_url = get_grafana_url(spark_conf)
         dashboard_url_msg = (
-            PaastaColors.green(f"\nGrafana dashboard: ") + f"{grafana_url}\n"
+            PaastaColors.green("\nGrafana dashboard: ") + f"{grafana_url}\n"
         )
         print(webui_url_msg)
         print(dashboard_url_msg)