Pep8 fixes for django_manage (#24198)

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
This commit is contained in:
Abhijeet Kasurde 2017-05-02 22:03:36 +05:30 committed by Matt Martz
commit df5be2b8ea
2 changed files with 27 additions and 20 deletions

View file

@ -141,6 +141,7 @@ EXAMPLES = """
import os import os
def _fail(module, cmd, out, err, **kwargs): def _fail(module, cmd, out, err, **kwargs):
msg = '' msg = ''
if out: if out:
@ -170,24 +171,31 @@ def _ensure_virtualenv(module):
os.environ["PATH"] = "%s:%s" % (vbin, os.environ["PATH"]) os.environ["PATH"] = "%s:%s" % (vbin, os.environ["PATH"])
os.environ["VIRTUAL_ENV"] = venv_param os.environ["VIRTUAL_ENV"] = venv_param
def createcachetable_filter_output(line): def createcachetable_filter_output(line):
return "Already exists" not in line return "Already exists" not in line
def flush_filter_output(line): def flush_filter_output(line):
return "Installed" in line and "Installed 0 object" not in line return "Installed" in line and "Installed 0 object" not in line
def loaddata_filter_output(line): def loaddata_filter_output(line):
return "Installed" in line and "Installed 0 object" not in line return "Installed" in line and "Installed 0 object" not in line
def syncdb_filter_output(line): def syncdb_filter_output(line):
return ("Creating table " in line) or ("Installed" in line and "Installed 0 object" not in line) return ("Creating table " in line) or ("Installed" in line and "Installed 0 object" not in line)
def migrate_filter_output(line): def migrate_filter_output(line):
return ("Migrating forwards " in line) or ("Installed" in line and "Installed 0 object" not in line) or ("Applying" in line) return ("Migrating forwards " in line) or ("Installed" in line and "Installed 0 object" not in line) or ("Applying" in line)
def collectstatic_filter_output(line): def collectstatic_filter_output(line):
return line and "0 static files" not in line return line and "0 static files" not in line
def main(): def main():
command_allowed_param_map = dict( command_allowed_param_map = dict(
cleanup=(), cleanup=(),
@ -199,11 +207,11 @@ def main():
validate=(), validate=(),
migrate=('apps', 'skip', 'merge', 'database',), migrate=('apps', 'skip', 'merge', 'database',),
collectstatic=('clear', 'link', ), collectstatic=('clear', 'link', ),
) )
command_required_param_map = dict( command_required_param_map = dict(
loaddata=('fixtures', ), loaddata=('fixtures', ),
) )
# forces --noinput on every command that needs it # forces --noinput on every command that needs it
noinput_commands = ( noinput_commands = (
@ -212,7 +220,7 @@ def main():
'migrate', 'migrate',
'test', 'test',
'collectstatic', 'collectstatic',
) )
# These params are allowed for certain commands only # These params are allowed for certain commands only
specific_params = ('apps', 'clear', 'database', 'failfast', 'fixtures', 'liveserver', 'testrunner') specific_params = ('apps', 'clear', 'database', 'failfast', 'fixtures', 'liveserver', 'testrunner')
@ -224,23 +232,23 @@ def main():
module = AnsibleModule( module = AnsibleModule(
argument_spec=dict( argument_spec=dict(
command = dict(default=None, required=True), command=dict(default=None, required=True),
app_path = dict(default=None, required=True), app_path=dict(default=None, required=True),
settings = dict(default=None, required=False), settings=dict(default=None, required=False),
pythonpath = dict(default=None, required=False, aliases=['python_path']), pythonpath=dict(default=None, required=False, aliases=['python_path']),
virtualenv = dict(default=None, required=False, aliases=['virtual_env']), virtualenv=dict(default=None, required=False, aliases=['virtual_env']),
apps = dict(default=None, required=False), apps=dict(default=None, required=False),
cache_table = dict(default=None, required=False), cache_table=dict(default=None, required=False),
clear = dict(default=None, required=False, type='bool'), clear=dict(default=None, required=False, type='bool'),
database = dict(default=None, required=False), database=dict(default=None, required=False),
failfast = dict(default='no', required=False, type='bool', aliases=['fail_fast']), failfast=dict(default='no', required=False, type='bool', aliases=['fail_fast']),
fixtures = dict(default=None, required=False), fixtures=dict(default=None, required=False),
liveserver = dict(default=None, required=False, aliases=['live_server']), liveserver=dict(default=None, required=False, aliases=['live_server']),
testrunner = dict(default=None, required=False, aliases=['test_runner']), testrunner=dict(default=None, required=False, aliases=['test_runner']),
skip = dict(default=None, required=False, type='bool'), skip=dict(default=None, required=False, type='bool'),
merge = dict(default=None, required=False, type='bool'), merge=dict(default=None, required=False, type='bool'),
link = dict(default=None, required=False, type='bool'), link=dict(default=None, required=False, type='bool'),
), ),
) )

View file

@ -768,7 +768,6 @@ lib/ansible/modules/web_infrastructure/ansible_tower/tower_role.py
lib/ansible/modules/web_infrastructure/ansible_tower/tower_team.py lib/ansible/modules/web_infrastructure/ansible_tower/tower_team.py
lib/ansible/modules/web_infrastructure/ansible_tower/tower_user.py lib/ansible/modules/web_infrastructure/ansible_tower/tower_user.py
lib/ansible/modules/web_infrastructure/deploy_helper.py lib/ansible/modules/web_infrastructure/deploy_helper.py
lib/ansible/modules/web_infrastructure/django_manage.py
lib/ansible/modules/web_infrastructure/ejabberd_user.py lib/ansible/modules/web_infrastructure/ejabberd_user.py
lib/ansible/modules/web_infrastructure/htpasswd.py lib/ansible/modules/web_infrastructure/htpasswd.py
lib/ansible/modules/web_infrastructure/jboss.py lib/ansible/modules/web_infrastructure/jboss.py