Fix vault_password positional args error
Fixing variable name bug introduced in ac8950f

Fixes #8617
Fix host pattern matching and enhance error detection

Fixes #8614
Use match() instead of search() for pattern matching

Also related to f48fa3, fix for #8614
Load group_vars for new groups loaded via add_host

Fixes #8605
Catch error in atomic_move when creating temporary file

Fixes #8480
Make code more PEP8 compliant (if(x))

Remove useless parentheses, `if (x):` should be `if x:`.
Make code more PEP8 compliant (whitespace)

> Always surround these binary operators with a single space on either
> side: assignment (`=`), augmented assignment (`+=`, `-=`
> etc.), comparisons (`==`, `<`, `>`, ``!=``, `<>`, `<=`,
> `>=`, `in`, `not in`, `is`, `is not`), Booleans (`and`,
> `or`, `not`).
Make docker inventory plugin more Pythonic

There's no need to check if kwargs is empty before iterating over it.
Refactor CronTab.is_empty
Make code more PEP8 compliant (if len(seq))

> For sequences, (strings, lists, tuples), use the fact that empty
> sequences are false.
>
>     Yes: if not seq:
>          if seq:
>
>     No: if len(seq)
>         if not len(seq)