mirror of
https://github.com/kubernetes-sigs/kubespray.git
synced 2025-12-13 21:34:40 +03:00
9b122fb5a8
* Use alternate self-sufficient shellcheck precommit This pre-commit does not require prerequisite on the host, making it easier to run in CI workflows. * Switch to upstream ansible-lint pre-commit hook This way, the hook is self contained and does not depend on a previous virtualenv installation. * pre-commit: fix hooks dependencies - ansible-syntax-check - tox-inventory-builder - jinja-syntax-check * Fix ci-matrix pre-commit hook - Remove dependency of pydblite which fails to setup on recent pythons - Discard shell script and put everything into pre-commit * pre-commit: apply autofixes hooks and fix the rest manually - markdownlint (manual fix) - end-of-file-fixer - requirements-txt-fixer - trailing-whitespace * Convert check_typo to pre-commit + use maintained version client9/misspell is unmaintained, and has been forked by the golangci team, see https://github.com/client9/misspell/issues/197#issuecomment-1596318684. 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. * collection-build-install convert to pre-commit * Run pre-commit hooks in dynamic pipeline 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. * Remove gitlab-ci job done in pre-commit * pre-commit: adjust mardownlint default, md fixes 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 https://github.com/markdownlint/markdownlint/pull/373) * Update pre-commit hooks --------- Co-authored-by: Max Gautier <mg@max.gautier.name>
110 lines
3.1 KiB
YAML
110 lines
3.1 KiB
YAML
---
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: v4.6.0
|
|
hooks:
|
|
- id: check-added-large-files
|
|
- id: check-case-conflict
|
|
- id: check-executables-have-shebangs
|
|
- id: check-xml
|
|
- id: check-merge-conflict
|
|
- id: detect-private-key
|
|
- id: end-of-file-fixer
|
|
- id: forbid-new-submodules
|
|
- id: requirements-txt-fixer
|
|
- id: trailing-whitespace
|
|
|
|
- repo: https://github.com/adrienverge/yamllint.git
|
|
rev: v1.35.1
|
|
hooks:
|
|
- id: yamllint
|
|
args: [--strict]
|
|
|
|
- repo: https://github.com/markdownlint/markdownlint
|
|
rev: v0.12.0
|
|
hooks:
|
|
- id: markdownlint
|
|
exclude: "^.github|(^docs/_sidebar\\.md$)"
|
|
|
|
- repo: https://github.com/shellcheck-py/shellcheck-py
|
|
rev: v0.10.0.1
|
|
hooks:
|
|
- id: shellcheck
|
|
args: ["--severity=error"]
|
|
exclude: "^.git"
|
|
files: "\\.sh$"
|
|
|
|
- repo: https://github.com/ansible/ansible-lint
|
|
rev: v24.5.0
|
|
hooks:
|
|
- id: ansible-lint
|
|
additional_dependencies:
|
|
- ansible==9.5.1
|
|
- jsonschema==4.22.0
|
|
- jmespath==1.0.1
|
|
- netaddr==1.2.1
|
|
|
|
- repo: https://github.com/VannTen/misspell
|
|
# Waiting on https://github.com/golangci/misspell/pull/19 to get merged
|
|
rev: 8592a4e
|
|
hooks:
|
|
- id: misspell
|
|
exclude: "OWNERS_ALIASES$"
|
|
|
|
- repo: local
|
|
hooks:
|
|
- id: ansible-syntax-check
|
|
name: ansible-syntax-check
|
|
entry: env ANSIBLE_INVENTORY=inventory/local-tests.cfg ANSIBLE_REMOTE_USER=root ANSIBLE_BECOME="true" ANSIBLE_BECOME_USER=root ANSIBLE_VERBOSITY="3" ansible-playbook --syntax-check
|
|
language: python
|
|
files: "^cluster.yml|^upgrade-cluster.yml|^reset.yml|^extra_playbooks/upgrade-only-k8s.yml"
|
|
additional_dependencies:
|
|
- ansible==9.5.1
|
|
|
|
- id: tox-inventory-builder
|
|
name: tox-inventory-builder
|
|
entry: bash -c "cd contrib/inventory_builder && tox"
|
|
language: python
|
|
pass_filenames: false
|
|
additional_dependencies:
|
|
- tox==4.15.0
|
|
|
|
- id: check-readme-versions
|
|
name: check-readme-versions
|
|
entry: tests/scripts/check_readme_versions.sh
|
|
language: script
|
|
pass_filenames: false
|
|
|
|
- id: collection-build-install
|
|
name: Build and install kubernetes-sigs.kubespray Ansible collection
|
|
language: python
|
|
additional_dependencies:
|
|
- ansible-core>=2.16.4
|
|
entry: tests/scripts/collection-build-install.sh
|
|
pass_filenames: false
|
|
|
|
- id: generate-docs-sidebar
|
|
name: generate-docs-sidebar
|
|
entry: scripts/gen_docs_sidebar.sh
|
|
language: script
|
|
pass_filenames: false
|
|
|
|
- id: ci-matrix
|
|
name: ci-matrix
|
|
entry: tests/scripts/md-table/main.py
|
|
language: python
|
|
pass_filenames: false
|
|
additional_dependencies:
|
|
- jinja2
|
|
- pathlib
|
|
- pyaml
|
|
|
|
- id: jinja-syntax-check
|
|
name: jinja-syntax-check
|
|
entry: tests/scripts/check-templates.py
|
|
language: python
|
|
types:
|
|
- jinja
|
|
additional_dependencies:
|
|
- jinja2
|