From f12c1fd2ebc19234b061d7f03e8c954e72f8d48c Mon Sep 17 00:00:00 2001
From: Evgeni Golov <evgeni@golov.de>
Date: Thu, 7 Mar 2024 10:09:30 +0100
Subject: [PATCH] drop unused get_value_from_yaml_in_tarball module

---
 .ansible-lint                                 |  1 -
 .../library/get_value_from_yaml_in_tarball.py | 90 -------------------
 2 files changed, 91 deletions(-)
 delete mode 100644 roles/satellite-clone/library/get_value_from_yaml_in_tarball.py

diff --git a/.ansible-lint b/.ansible-lint
index 4253c0b3..2adc7d4d 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -11,5 +11,4 @@ warn_list:
 enable_list:
   - only-builtins
 only_builtins_allow_modules:
-  - get_value_from_yaml_in_tarball
   - parse_backup_metadata
diff --git a/roles/satellite-clone/library/get_value_from_yaml_in_tarball.py b/roles/satellite-clone/library/get_value_from_yaml_in_tarball.py
deleted file mode 100644
index 078583c1..00000000
--- a/roles/satellite-clone/library/get_value_from_yaml_in_tarball.py
+++ /dev/null
@@ -1,90 +0,0 @@
-import yaml
-import tarfile
-from ansible.module_utils.basic import *
-from functools import reduce
-import operator
-
-DOCUMENTATION = '''
----
-module: get_value_from_yaml_in_tarball
-description:
-    - Return value from a YAML file that is in a tarball
-options:
-    tarball:
-        description:
-          - path to tarball that contains the target yaml file.
-        required: true
-    target_file:
-        description:
-          - Path of target YAML file
-        required: true
-    keys:
-        description:
-          - List of key(s) to find value in file. It can take multiple values for nested keys.
-            For example: ["foreman_proxy", "dhcp"] or ["satellite_version"]
-        required: true
-    allow_null:
-        description:
-        - Allows the parameter to be null, defaulting to an empty string. Defaults to false.
-        required: false
-'''
-
-EXAMPLES = '''
-- name: Get puppet user from Satellite answers
-  get_value_from_yaml_in_tarball:
-    tarball: "/path/to/config_files.tar.gz"
-    target_file: "etc/foreman-installer/scenarios.d/satellite-answers.yaml"
-    keys: ["puppet", "user"]
-  register: puppet_user
-
-  then you can use puppet_user.value
-'''
-
-BACKUP_CONFIG = "config_files.tar.gz"
-
-
-# Takes keys in the format ['foo', 'bar']
-# when the dict looks like
-# { "foo": { "bar": "hi" }}
-# will return None if it a key doesn't exist
-def get_value(data, keys):
-    try:
-        return reduce(operator.getitem, keys, data)
-    except KeyError:
-        return None
-
-
-def get_value_from_tarball(params):
-    config_tar = tarfile.open(params["tarball"])
-    answers_file = config_tar.extractfile(params["target_file"])
-    allow_null = params.get("allow_null", False)
-    answers = yaml.load(answers_file.read())
-    found_value = get_value(answers, params['keys'])
-    # The value itself can be false, so only check for NoneType
-    if found_value is not None:
-        return True, dict(msg="Value found!", changed=False, value=found_value)
-    elif allow_null:
-        return True, dict(msg="Defaulting to \"\"", changed=False, value="")
-    else:
-        msg = "Keys {} were not found in {}!".format(
-            params["keys"], params["target_file"])
-        return False, dict(msg=msg)
-
-
-def main():
-    fields = {
-        "tarball": {"required": True, "type": "str"},
-        "target_file": {"required": True, "type": "str"},
-        "keys": {"required": True, "type": "list"},
-        "allow_null": {"required": False, "type": "bool", "default": False}
-    }
-    module = AnsibleModule(argument_spec=fields)
-    success, result = get_value_from_tarball(module.params)
-    if success:
-        module.exit_json(**result)
-    else:
-        module.fail_json(**result)
-
-
-if __name__ == '__main__':
-    main()