Add config option to disable dns syncing (#151)

- Added config option to disable dns syncing
- Added option to force dns syncing

aladdin/commands/sync_dns.py

@@ -12,18 +12,22 @@ def parse_args(sub_parser):
         "sync-dns", help="Synchronize the dns from the kubernetes services"
     )
     add_namespace_argument(subparser)
-    subparser.set_defaults(func=sync_dns_args)
-
-
-def sync_dns_args(args):
-    sync_dns(args.namespace)
+    subparser.add_argument(
+        "--force",
+        "-f",
+        action="store_true",
+        default=False,
+    )
+    subparser.set_defaults(
+        func=lambda args: sync_dns(args.namespace, force=args.force)
+    )
 
 
 @container_command
-def sync_dns(namespace):
+def sync_dns(namespace: str, force: bool = False):
     cr = ClusterRules(namespace=namespace)
-    if cr.is_local:
-        logging.info("Not syncing DNS because you are on local")
+    if not cr.dns_sync and not force:
+        logging.info("Not syncing DNS because it's disabled for this cluster")
         return
 
     k_utils = KubernetesUtils(Kubernetes(namespace=namespace))

aladdin/lib/cluster_rules.py

@@ -114,6 +114,14 @@ def certificate_lookup(self):
             return False
         return self.rules.get("certificate_lookup", True)
 
+    @property
+    def dns_sync(self):
+        if strtobool(os.getenv("IS_LOCAL", "false")) or self.is_local:
+            return False
+        if strtobool(os.getenv("ALADDIN_DISABLE_DNS_SYNC", "false")):
+            return False
+        return self.rules.get("dns_sync", True)
+
     @cached_property
     def boto(self):
         return boto3.Session(profile_name=self.aws_profile)