mirror of
https://github.com/thegeeklab/ansible-later.git
synced 2024-11-13 00:30:39 +00:00
Robert Kaussow
43d7edca32
* refactor plugin system to use a class-based approach * disable some docstring linter errors and fix imports * cleanup * fix docs * add metavars to cli arguments for better helptext * add option to disable buildin rules * remove print * remove dead code
54 lines
1.9 KiB
Python
54 lines
1.9 KiB
Python
import os
|
|
|
|
from ansiblelater.standard import StandardBase
|
|
|
|
|
|
class CheckCommandInsteadOfModule(StandardBase):
|
|
|
|
sid = "ANSIBLE0008"
|
|
description = "Commands should not be used in place of modules"
|
|
helptext = "{exec} command used in place of {module} module"
|
|
version = "0.1"
|
|
types = ["playbook", "task", "handler"]
|
|
|
|
def check(self, candidate, settings):
|
|
tasks, errors = self.get_normalized_tasks(candidate, settings)
|
|
commands = ["command", "shell", "raw"]
|
|
modules = {
|
|
"git": "git",
|
|
"hg": "hg",
|
|
"curl": "get_url or uri",
|
|
"wget": "get_url or uri",
|
|
"svn": "subversion",
|
|
"service": "service",
|
|
"mount": "mount",
|
|
"rpm": "yum or rpm_key",
|
|
"yum": "yum",
|
|
"apt-get": "apt-get",
|
|
"unzip": "unarchive",
|
|
"tar": "unarchive",
|
|
"chkconfig": "service",
|
|
"rsync": "synchronize",
|
|
"supervisorctl": "supervisorctl",
|
|
"systemctl": "systemd",
|
|
"sed": "template or lineinfile"
|
|
}
|
|
|
|
if not errors:
|
|
for task in tasks:
|
|
if task["action"]["__ansible_module__"] in commands:
|
|
first_cmd_arg = self.get_first_cmd_arg(task)
|
|
executable = os.path.basename(first_cmd_arg)
|
|
if (
|
|
first_cmd_arg and executable in modules
|
|
and task["action"].get("warn", True) and "register" not in task
|
|
):
|
|
errors.append(
|
|
self.Error(
|
|
task["__line__"],
|
|
self.helptext.format(exec=executable, module=modules[executable])
|
|
)
|
|
)
|
|
|
|
return self.Result(candidate.path, errors)
|