feat: add rule CheckLocalAction

This commit is contained in:
Robert Kaussow 2021-01-31 18:20:24 +01:00
parent 3bcfb4852e
commit 273b32a200
No known key found for this signature in database
GPG Key ID: 65362AE74AF98B61
2 changed files with 23 additions and 0 deletions

View File

@ -0,0 +1,22 @@
# Copyright (c) 2016, Tsukinowa Inc. <info@tsukinowa.jp>
# Copyright (c) 2018, Ansible Project
from ansiblelater.standard import StandardBase
class CheckLocalAction(StandardBase):
sid = "ANSIBLE0024"
description = "Don't use local_action"
helptext = ("`delegate_to: localhost` should be used instead of `local_action`")
version = "0.2"
types = ["playbook", "task", "handler"]
def check(self, candidate, settings):
yamllines, errors = self.get_normalized_yaml(candidate, settings)
if not errors:
for i, line in yamllines:
if 'local_action' in line:
errors.append(self.Error(i, self.helptext))
return self.Result(candidate.path, errors)

View File

@ -38,3 +38,4 @@ Reviews are nothing without some rules or standards against which to review. ans
| CheckMetaChangeFromDefault | ANSIBLE0021 | Roles meta/main.yml default values should be changed. | | | CheckMetaChangeFromDefault | ANSIBLE0021 | Roles meta/main.yml default values should be changed. | |
| CheckWhenFormat | ANSIBLE0022 | Don't use Jinja2 in `when`. | | | CheckWhenFormat | ANSIBLE0022 | Don't use Jinja2 in `when`. | |
| CheckNestedJinja | ANSIBLE0023 | Don't use nested Jinja2 pattern. | | | CheckNestedJinja | ANSIBLE0023 | Don't use nested Jinja2 pattern. | |
| CheckLocalAction | ANSIBLE0024 | Don't use local_action. | |