From 40e7ed9518c0003de47e648db0c240d079f090fa Mon Sep 17 00:00:00 2001 From: stephane Date: Thu, 29 Oct 2015 18:58:53 -0700 Subject: [PATCH] Add support for pip force-reinstall The pip command allows a user to force reinstallation, but the module doesn't currently support it. Add "force-reinstall" as a possible state. --- lib/ansible/modules/packaging/language/pip.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/ansible/modules/packaging/language/pip.py b/lib/ansible/modules/packaging/language/pip.py index 6d325282770..8cb54e32732 100755 --- a/lib/ansible/modules/packaging/language/pip.py +++ b/lib/ansible/modules/packaging/language/pip.py @@ -84,7 +84,7 @@ options: - The state of module required: false default: present - choices: [ "present", "absent", "latest" ] + choices: [ "present", "absent", "latest", "forcereinstall" ] extra_args: description: - Extra arguments passed to pip. @@ -154,6 +154,9 @@ EXAMPLES = ''' # Install (Bottle) for Python 3.3 specifically,using the 'pip-3.3' executable. - pip: name=bottle executable=pip-3.3 + +# Install (Bottle), forcing reinstallation if it's already installed +- pip: name=bottle state=forcereinstall ''' def _get_cmd_options(module, cmd): @@ -235,6 +238,7 @@ def main(): present='install', absent='uninstall -y', latest='install -U', + forcereinstall='install -U --force-reinstall', ) module = AnsibleModule( @@ -358,7 +362,7 @@ def main(): is_present = _is_present(name, version, out.split()) - changed = (state == 'present' and not is_present) or (state == 'absent' and is_present) + changed = (state == 'present' and not is_present) or (state == 'absent' and is_present) or (state == 'forcereinstall' and is_present) module.exit_json(changed=changed, cmd=freeze_cmd, stdout=out, stderr=err) if requirements or has_vcs: