Remove drain_lib and all references, since this is not used for Kuber…

…netes.
Yelp · May 18, 2024 · cbe056b · cbe056b
1 parent 2700f45
commit cbe056b
Show file tree

Hide file tree

Showing 12 changed files with 0 additions and 849 deletions.
diff --git a/docs/source/generated/paasta_tools.drain_lib.rst b/docs/source/generated/paasta_tools.drain_lib.rst
diff --git a/docs/source/generated/paasta_tools.rst b/docs/source/generated/paasta_tools.rst
@@ -47,7 +47,6 @@ Submodules
    paasta_tools.deployment_utils
    paasta_tools.docker_wrapper
    paasta_tools.docker_wrapper_imports
-   paasta_tools.drain_lib
    paasta_tools.dump_locally_running_services
    paasta_tools.eks_tools
    paasta_tools.envoy_tools

diff --git a/docs/source/installation/getting_started.rst b/docs/source/installation/getting_started.rst
@@ -157,8 +157,6 @@ Hacheck provides several behaviors that are useful for Paasta:
 
   * It can preemptively return error codes for health checks, allowing us to remove a task from load balancers before
     shutting it down.
-    (This is implemented in the
-    `HacheckDrainMethod <../generated/paasta_tools.drain_lib.html#paasta_tools.drain_lib.HacheckDrainMethod>`_.)
 
 Sensu
 -----

diff --git a/docs/source/yelpsoa_configs.rst b/docs/source/yelpsoa_configs.rst
@@ -368,14 +368,6 @@ instance MAY have:
     This setting only affects the first time deployd processes an instance after a change --
       instances that need to be reprocessed will be reenqueued normally.
 
-  * ``drain_method``: Controls the drain method; see `drain_lib
-    <generated/paasta_tools.drain_lib.html>`_. Defaults to ``noop`` for
-    instances that are not in Smartstack, or ``hacheck`` if they are.
-
-  * ``drain_method_params``: A dictionary of parameters for the specified
-    drain_method. Valid parameters are any of the kwargs defined for the
-    specified bounce_method in `drain_lib <generated/paasta_tools.drain_lib.html>`_.
-
   * ``cmd``: The command that is executed. If a string, will be wrapped in ``/bin/sh -c``.
     If a list, will be executed directly as is with no shell parsing.
 
@@ -621,14 +613,6 @@ instance MAY have:
     This setting only affects the first time deployd processes an instance after a change --
       instances that need to be reprocessed will be reenqueued normally.
 
-  * ``drain_method``: Controls the drain method; see `drain_lib
-    <generated/paasta_tools.drain_lib.html>`_. Defaults to ``noop`` for
-    instances that are not in Smartstack, or ``hacheck`` if they are.
-
-  * ``drain_method_params``: A dictionary of parameters for the specified
-    drain_method. Valid parameters are any of the kwargs defined for the
-    specified bounce_method in `drain_lib <generated/paasta_tools.drain_lib.html>`_.
-
   * ``cmd``: The command that is executed. Can be used as an alternative to
     args for containers without an `entrypoint
     <https://docs.docker.com/reference/builder/#entrypoint>`_. This value is

diff --git a/general_itests/steps/http_drain_method_steps.py b/general_itests/steps/http_drain_method_steps.py
diff --git a/mypy.ini b/mypy.ini
@@ -64,9 +64,6 @@ disallow_untyped_defs = True
 [mypy-paasta_tools.generate_deployments_for_service]
 disallow_untyped_defs = True
 
-[mypy-paasta_tools.drain_lib]
-disallow_untyped_defs = True
-
 [mypy-paasta_tools.smartstack_tools]
 disallow_untyped_defs = True
 

diff --git a/paasta_tools/cli/schemas/kubernetes_schema.json b/paasta_tools/cli/schemas/kubernetes_schema.json
@@ -43,56 +43,6 @@
                             ]
                         }
                     ]
-                },
-                {
-                    "oneOf": [
-                        {
-                            "properties": {
-                                "drain_method": {
-                                    "enum": [
-                                        "noop",
-                                        "hacheck",
-                                        "test"
-                                    ]
-                                }
-                            }
-                        },
-                        {
-                            "properties": {
-                                "drain_method": {
-                                    "enum": [
-                                        "http"
-                                    ]
-                                },
-                                "drain_method_params": {
-                                    "type": "object",
-                                    "properties": {
-                                        "drain": {
-                                            "type": "object"
-                                        },
-                                        "stop_draining": {
-                                            "type": "object"
-                                        },
-                                        "is_draining": {
-                                            "type": "object"
-                                        },
-                                        "is_safe_to_kill": {
-                                            "type": "object"
-                                        }
-                                    },
-                                    "required": [
-                                        "drain",
-                                        "stop_draining",
-                                        "is_draining",
-                                        "is_safe_to_kill"
-                                    ]
-                                }
-                            },
-                            "required": [
-                                "drain_method_params"
-                            ]
-                        }
-                    ]
                 }
             ],
             "properties": {
@@ -205,18 +155,6 @@
                 "service_account_name": {
                     "type": "string"
                 },
-                "drain_method": {
-                    "enum": [
-                        "noop",
-                        "hacheck",
-                        "http",
-                        "test"
-                    ],
-                    "default": "noop"
-                },
-                "drain_method_params": {
-                    "type": "object"
-                },
                 "constraints": {
                     "type": "array",
                     "items": {