Check if peers is defined when peering with routers

…#11231)

Signed-off-by: tico88612 <17496418+tico88612@users.noreply.github.com>

Bumps [ansible](https://github.com/ansible-community/ansible-build-data) from 9.5.1 to 9.6.0.
- [Changelog](https://github.com/ansible-community/ansible-build-data/blob/main/docs/release-process.md)
- [Commits](ansible-community/ansible-build-data@9.5.1...9.6.0)

---
updated-dependencies:
- dependency-name: ansible
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

This pre-commit does not require prerequisite on the host, making it
easier to run in CI workflows.

This way, the hook is self contained and does not depend on a previous
virtualenv installation.

- ansible-syntax-check
- tox-inventory-builder
- jinja-syntax-check

- Remove dependency of pydblite which fails to setup on recent pythons
- Discard shell script and put everything into pre-commit

- markdownlint (manual fix)
- end-of-file-fixer
- requirements-txt-fixer
- trailing-whitespace

client9/misspell is unmaintained, and has been forked by the golangci
team, see client9/misspell#197 (comment).

They haven't yet added a pre-commit config, so use my fork with the
pre-commit hook config until the pull request is merged.

Use gitlab dynamic child pipelines feature to have one source of truth
for the pre-commit jobs, the pre-commit config file.

Use one cache per pre-commit. This should reduce the "fetching cache"
time steps in gitlab-ci, since each job will have a separate cache with
only its hook installed.

Use a style file as recommended by upstream. This makes for only one
source of truth.
Conserve previous upstream default for MD007 (upstream default changed
here markdownlint/markdownlint#373)

While uncommon, provisioning only a control plane is a valid use case,
so don't block it.

The default for galaxy. `manifest` works well enough for our case, and
this avoids maintaining a blacklist.

…sitories (kubernetes-sigs#10872)

…netes-sigs#11224)

Fix regression added in 663fcd1 for
flatcar nodes.

See: kubernetes-sigs@663fcd1

Bumps [pytest-testinfra](https://github.com/pytest-dev/pytest-testinfra) from 10.1.0 to 10.1.1.
- [Release notes](https://github.com/pytest-dev/pytest-testinfra/releases)
- [Changelog](https://github.com/pytest-dev/pytest-testinfra/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-testinfra@10.1.0...10.1.1)

---
updated-dependencies:
- dependency-name: pytest-testinfra
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

The inventory file generated by Terraform produces the following warnings:
```
[WARNING]:  * Failed to parse <PATH>/kubespray/contrib/terraform/hetzner/inventory.ini with ini plugin:
<PATH>/kubespray/contrib/terraform/hetzner/inventory.ini:21: Section [k8s_cluster:children] includes undefined group: kube-master
...
[WARNING]: Could not match supplied host pattern, ignoring: kube-master

PLAY [Add kube-master nodes to kube_control_plane] ********************************************************************************************************
skipping: no hosts matched
[WARNING]: Could not match supplied host pattern, ignoring: kube-node

PLAY [Add kube-node nodes to kube_node] *******************************************************************************************************************
skipping: no hosts matched
```

…1251)

- Require a 'lgtm' or 'ok-to-test' label for running CI after the
  moderator stage

Signed-off-by: ant31 <2t.antoine@gmail.com>