Skip to content

Commit

Permalink
Merge pull request #11606 from VannTen/cleanup/pre_commit_ansible_lint
Browse files Browse the repository at this point in the history
Reduce pre-commit hooks dependencies
  • Loading branch information
k8s-ci-robot authored Oct 8, 2024
2 parents 8aa4c9a + 4f27bc2 commit 9204f60
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 14 deletions.
2 changes: 2 additions & 0 deletions .ansible-lint
Original file line number Diff line number Diff line change
Expand Up @@ -37,3 +37,5 @@ exclude_paths:
- tests/files/custom_cni/cilium.yaml
- venv
- .github
mock_modules:
- gluster.gluster.gluster_volume
13 changes: 2 additions & 11 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.6.0
rev: v5.0.0
hooks:
- id: check-added-large-files
- id: check-case-conflict
Expand Down Expand Up @@ -35,11 +35,10 @@ repos:
files: "\\.sh$"

- repo: https://github.com/ansible/ansible-lint
rev: v24.5.0
rev: v24.9.2
hooks:
- id: ansible-lint
additional_dependencies:
- ansible==9.8.0
- jsonschema==4.22.0
- jmespath==1.0.1
- netaddr==1.3.0
Expand All @@ -53,14 +52,6 @@ repos:

- 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"
Expand Down
3 changes: 0 additions & 3 deletions tests/requirements.yml

This file was deleted.

0 comments on commit 9204f60

Please sign in to comment.