```
salt/_modules/cri.py:174: [W0120(useless-else-on-loop), wait_container] Else clause on loop without a break statement
```

```
salt/_modules/metalk8s_cordon.py:84: [E8303(too-many-blank-lines), ] PEP8 E303: too many blank lines (2)
```

Disable some "raise-missing-from" as it's known exception and we do not
want to have upper exceptions raised
```
salt/_modules/metalk8s_drain.py:388: [W0707(raise-missing-from), Drain.run_drain] Consider explicitly re-raising using the 'from' keyword
salt/_modules/metalk8s_drain.py:394: [W0707(raise-missing-from), Drain.run_drain] Consider explicitly re-raising using the 'from' keyword
```

Use `from` in exception instead of formatting with the exception
message
```
salt/_modules/metalk8s_drain.py:360: [W0707(raise-missing-from), Drain.run_drain] Consider explicitly re-raising using the 'from' keyword
salt/_modules/metalk8s_drain.py:547: [W0707(raise-missing-from), evict_pod] Consider explicitly re-raising using the 'from' keyword
```

Remove "unecessary-pass"
```
salt/_modules/metalk8s_drain.py:59: [W0107(unnecessary-pass), DrainTimeoutException] Unnecessary pass statement
```

Remove the unused variable
```
salt/_modules/metalk8s_drain.py:516: [W0612(unused-variable), evict_pod] Unused variable 'result'
```

Use six urllib parse module from salt instead of six directly
```
salt/_modules/metalk8s_etcd.py:6: [E9402(blacklisted-module), ] Uses of a blacklisted module 'six.moves.urllib.parse': Please report this error to SaltStack so we can fix it: Trying to import urlparse from six.moves.urllib.parse
```

Fix "bare-except" and disable "broad-except" as we want to catch all
exceptions
```
salt/_modules/metalk8s_etcd.py:198: [W0702(bare-except), get_etcd_member_list] No exception type(s) specified
```

Disable "broad-except" as we want to catch all exceptions
```
salt/_modules/metalk8s_kubeconfig.py:43: [W0703(broad-except), validate] Catching too general exception Exception
```

Re-format to not use `backslash` inside brackets
```
salt/_utils/volume_utils.py:280: [E8502(the-backslash-is-redundant-between-brackets), ] PEP8 E502: the backslash is redundant between brackets
```

Disable "unused-argument" for `ext_pillar` as we need a specific
function signature
```
salt/_pillar/metalk8s_endpoints.py:20: [W0613(unused-argument), ext_pillar] Unused argument 'minion_id'
salt/_pillar/metalk8s_endpoints.py:20: [W0613(unused-argument), ext_pillar] Unused argument 'pillar'
```

Disable "broad-except" as we want to catch all exceptions
```
salt/_pillar/metalk8s_etcd.py:18: [W0703(broad-except), _load_members] Catching too general exception Exception
```

Disable "unused-argument" from `ext_pillar` as we need a specific
function signature
```
salt/_pillar/metalk8s_etcd.py:32: [W0613(unused-argument), ext_pillar] Unused argument 'minion_id'
```

Use `fopen` from salt instead of classic `open`
```
salt/_pillar/metalk8s_private.py:19: [W8470(resource-leakage), _read_private_key] Resource leakage detected. Please use 'with salt.utils.files.fopen()' instead of 'with open()'. It assures salt does not leak file handles.
```

Remove uneeded comma
```
salt/_pillar/metalk8s.py:187: [E8231(missing-whitespace-after-comma), ] PEP8 E231: missing whitespace after ','
```

Disable "broad-except" as we want to catch all exceptions
```
salt/_pillar/metalk8s.py:28: [W0703(broad-except), _load_config] Catching too general exception Exception
```

Disable "unused-argument" for `ext_pillar` as we need a specific
function signature
```
salt/_pillar/metalk8s.py:156: [W0613(unused-argument), ext_pillar] Unused argument 'minion_id'
salt/_pillar/metalk8s.py:156: [W0613(unused-argument), ext_pillar] Unused argument 'pillar'
```

Disable "broad-except" as we want to catch all exceptions
```
salt/_pillar/metalk8s_solutions.py:43: [W0703(broad-except), _load_solutions] Catching too general exception Exception
salt/_pillar/metalk8s_solutions.py:50: [W0703(broad-except), _load_solutions] Catching too general exception Exception
salt/_pillar/metalk8s_solutions.py:70: [W0703(broad-except), _load_solutions] Catching too general exception Exception
```

Disable "unused-argument" for `ext_pillar` as we need a specific
function signature
```
salt/_pillar/metalk8s_solutions.py:81: [W0613(unused-argument), ext_pillar] Unused argument 'minion_id'
```

Add missing import for `SaltInvocationError` exception
```
salt/_runners/metalk8s_saltutil.py:136: [E0602(undefined-variable), orchestrate_show_sls] Undefined variable 'SaltInvocationError'
```

Add one more blank line
```
salt/_auth/kubernetes_rbac.py:24: [E8302(expected-2-blank-lines-found-0), ] PEP8 E302: expected 2 blank lines, found 1
```

Remove unused import
```
salt/_auth/kubernetes_rbac.py:1: [W0611(unused-import), ] Unused import base64
salt/_auth/kubernetes_rbac.py:10: [W0611(unused-import), ] Unused ApiException imported from kubernetes.client.rest
```

Disable "unused-import" as we import to check for virtual
```
salt/_auth/kubernetes_rbac.py:15: [W0611(unused-import), ] Unused import requests
```

Add missing import for `CommandExecutionError` exception
```
salt/_auth/kubernetes_rbac.py:57: [E0602(undefined-variable), _check_auth_args.wrapped] Undefined variable 'CommandExecutionError'
```

Disable "unused-argument" for `auth` and `groups` as we need a specific
function signature
```
salt/_auth/kubernetes_rbac.py:198: [W0613(unused-argument), groups] Unused argument 'password'
```

Discard `kwargs` argument as it does not change the function signature
```
salt/_auth/kubernetes_rbac.py:180: [W0613(unused-argument), auth] Unused argument 'kwargs'
salt/_auth/kubernetes_rbac.py:198: [W0613(unused-argument), groups] Unused argument 'kwargs'
```

Use six module from salt instead of six directly
```
salt/_beacons/metalk8s_kubeconfig_info.py:4: [E9405(blacklisted-external-import), ] Uses of an external blacklisted import 'six': Please use 'import salt.ext.six as six'
```

Use `from` in exception instead of formatting with the exception message
```
salt/_renderers/metalk8s_kubernetes.py:41: [W0707(raise-missing-from), _step_name] Consider explicitly re-raising using the 'from' keyword
```

Disable "unused-argument" for `render` as we need a specific function
signature
```
salt/_renderers/metalk8s_kubernetes.py:73: [W0613(unused-argument), render] Unused argument 'saltenv'
salt/_renderers/metalk8s_kubernetes.py:73: [W0613(unused-argument), render] Unused argument 'sls'
```

Discard `kwargs` argument as it does not change the function signature
```
salt/_renderers/metalk8s_kubernetes.py:73: [W0613(unused-argument), render] Unused argument 'kwargs'
```

Discard `kwargs` argument as it does not change the function signature
```
salt/_roster/kubernetes_nodes.py:14: [W0613(unused-argument), targets] Unused argument 'kwargs'
```

Add a dedicated `pylintrc` file for all Salt Python modules and run a
pylint on all thoses files with tox `lint-python` as well as
`./doit.sh lint:python` so that we do some linting check in the CI

Fixes: #2649