diff options
author | Russell Teague <rteague@redhat.com> | 2017-07-26 14:26:47 -0400 |
---|---|---|
committer | Russell Teague <rteague@redhat.com> | 2017-07-26 14:26:47 -0400 |
commit | e70ca762c7b523bf4f7f9cb9015731f9d6e1688e (patch) | |
tree | 03fc4b81779fe5fd9ebe80c9ac2dfc9866514755 | |
parent | cff5116a1dae3c6b44afa59ae5fd08b398873945 (diff) | |
download | openshift-e70ca762c7b523bf4f7f9cb9015731f9d6e1688e.tar.gz openshift-e70ca762c7b523bf4f7f9cb9015731f9d6e1688e.tar.bz2 openshift-e70ca762c7b523bf4f7f9cb9015731f9d6e1688e.tar.xz openshift-e70ca762c7b523bf4f7f9cb9015731f9d6e1688e.zip |
Refactor openshift_facts BIOS vendor discovery
In Ansible 2.4, `get_file_content` is moving
from ansible.module_utils.facts
to ansible.module_utils.facts.utils
However, the method is no longer needed as the BIOS vendor fact can be
retrieved
from a standard Ansible fact, `ansible_system_vendor`.
i.e.:
"ansible_system_vendor": "Google",
-rwxr-xr-x | roles/openshift_facts/library/openshift_facts.py | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/roles/openshift_facts/library/openshift_facts.py b/roles/openshift_facts/library/openshift_facts.py index 49cc51b48..42c4945b4 100755 --- a/roles/openshift_facts/library/openshift_facts.py +++ b/roles/openshift_facts/library/openshift_facts.py @@ -2222,14 +2222,10 @@ class OpenShiftFacts(object): product_version = self.system_facts['ansible_product_version'] virt_type = self.system_facts['ansible_virtualization_type'] virt_role = self.system_facts['ansible_virtualization_role'] + bios_vendor = self.system_facts['ansible_system_vendor'] provider = None metadata = None - # TODO: this is not exposed through module_utils/facts.py in ansible, - # need to create PR for ansible to expose it - bios_vendor = get_file_content( # noqa: F405 - '/sys/devices/virtual/dmi/id/bios_vendor' - ) if bios_vendor == 'Google': provider = 'gce' metadata_url = ('http://metadata.google.internal/' |