aristanetworks · sugetha24 · Jul 10, 2024 · Jun 3, 2024 · Jun 3, 2024 · Jun 3, 2024
diff --git a/.github/workflows/pull-request-management.yml b/.github/workflows/pull-request-management.yml
@@ -270,3 +270,26 @@ jobs:
         run: |
           cd ansible_collections/arista/cvp/
           ansible-test sanity -v --requirements --docker --python ${{ matrix.python_version }}
+
+  ansible_lint:
+    name: Run ansible-lint test case
+    runs-on: ubuntu-latest
+    needs: [ file-changes ]
+    steps:
+      - name: 'Set environment variables'
+        run: |
+          echo "PY_COLORS=1" >> $GITHUB_ENV
+      - uses: actions/checkout@v4
+      - name: Set up Python 3
+        uses: actions/setup-python@v5
+        with:
+          python-version: |
+            3.10
+      - name: 'Install Python & Ansible requirements'
+        run: |
+          pip install -r ansible_collections/arista/cvp/requirements.txt -r ansible_collections/arista/cvp/requirements-dev.txt --upgrade
+          ansible-galaxy collection install -r ansible_collections/arista/cvp/collections.yml
+      - name: 'Run ansible-test integration test cases'
+        run: |
+          cd ansible_collections/arista/cvp/
+          ansible-lint --force-color --strict -v
diff --git a/.github/yamllintrc b/.github/yamllintrc
@@ -24,7 +24,7 @@ rules:
     level: warning
     indent-sequences: consistent
   line-length:
-    max: 140
+    max: 350
     level: warning
     allow-non-breakable-inline-mappings: true
   truthy: disable
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -2,14 +2,14 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v2.4.0
+    rev: v4.6.0
     hooks:
       - id: trailing-whitespace
       - id: end-of-file-fixer
       - id: check-added-large-files
       - id: check-merge-conflict
   - repo: https://github.com/Lucas-C/pre-commit-hooks
-    rev: v1.5.4
+    rev: v1.5.5
     hooks:
       - name: Check and insert license on Python files
         id: insert-license
@@ -62,7 +62,7 @@ repos:
           - '<!--|  ~|  -->'
 
   - repo: https://github.com/pycqa/pylint
-    rev: "v2.16.1"
+    rev: "v3.1.1"
     hooks:
       - id: pylint # Use pylintrc file in repository
         name: Check for Linting errors on Python files
@@ -73,7 +73,7 @@ repos:
           - -d duplicate-code
 
   - repo: https://github.com/adrienverge/yamllint.git
-    rev: 'v1.23.0'
+    rev: 'v1.35.1'
     hooks:
     - id: yamllint
       name: Check for Linting error on YAML files
@@ -83,16 +83,8 @@ repos:
       types: [file, yaml]
       args: [-c=.github/yamllintrc]
 
-  - repo: https://github.com/ansible/ansible-lint.git
-    rev: v4.2.0
-    hooks:
-      - id: ansible-lint
-        name: Check for ansible-lint errors
-        files: \.(yaml|yml)$
-        exclude: ansible_collections/arista/cvp/docs
-
   - repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.32.1
+    rev: v0.41.0
     hooks:
       - id: markdownlint
         args:

diff --git a/README.md b/README.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/.ansible-lint b/ansible_collections/arista/cvp/.ansible-lint
@@ -0,0 +1,14 @@
+---
+profile: production
+
+skip_list:
+  - name[template] # Remove warnings on task names where Jinja template is not at the end
+  - run-once[task] # Remove warnings on using run_once with other strategies
+  - meta-no-info # Ansible-lint does not honor meta-schema with standalone: false
+  - var-naming[no-role-prefix]
+  - var-naming[pattern]
+# kinds:
+#   - yaml: "**/molecule/**/inventory/host_vars/roles.yml"
+exclude_paths:
+  - molecule/
+  - examples/
diff --git a/ansible_collections/arista/cvp/.yamllint b/ansible_collections/arista/cvp/.yamllint
@@ -0,0 +1,33 @@
+---
+# Based on ansible-lint config
+extends: default
+
+rules:
+  braces:
+    max-spaces-inside: 1
+    level: error
+  brackets:
+    max-spaces-inside: 1
+    level: error
+  colons:
+    max-spaces-after: -1
+    level: error
+  commas:
+    max-spaces-after: -1
+    level: error
+  comments: disable
+  comments-indentation: disable
+  document-start: disable
+  empty-lines:
+    max: 3
+    level: error
+  hyphens:
+    level: error
+  indentation: disable
+  key-duplicates: enable
+  line-length: disable
+  new-line-at-end-of-file: disable
+  new-lines:
+    type: unix
+  trailing-spaces: disable
+  truthy: disable
diff --git a/ansible_collections/arista/cvp/README.md b/ansible_collections/arista/cvp/README.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/build-md/doc.py b/ansible_collections/arista/cvp/docs/build-md/doc.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2023 Arista Networks, Inc.
+# Copyright (c) 2023-2024 Arista Networks, Inc.
 # Use of this source code is governed by the Apache License 2.0
 # that can be found in the LICENSE file.
 import os

diff --git a/ansible_collections/arista/cvp/docs/build-md/templates/md.j2 b/ansible_collections/arista/cvp/docs/build-md/templates/md.j2
@@ -1,5 +1,5 @@
 {#
- Copyright (c) 2023 Arista Networks, Inc.
+ Copyright (c) 2023-2024 Arista Networks, Inc.
  Use of this source code is governed by the Apache License 2.0
  that can be found in the LICENSE file.
 #}

diff --git a/ansible_collections/arista/cvp/docs/contributing.md b/ansible_collections/arista/cvp/docs/contributing.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/faq/containers.md b/ansible_collections/arista/cvp/docs/faq/containers.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/faq/errors.md b/ansible_collections/arista/cvp/docs/faq/errors.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/getting-started.md b/ansible_collections/arista/cvp/docs/getting-started.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/how-to/cvp-authentication.md b/ansible_collections/arista/cvp/docs/how-to/cvp-authentication.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/how-to/debug.md b/ansible_collections/arista/cvp/docs/how-to/debug.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/installation/collection-installation.md b/ansible_collections/arista/cvp/docs/installation/collection-installation.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/installation/requirements.md b/ansible_collections/arista/cvp/docs/installation/requirements.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_change_control_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_change_control_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_configlet.md b/ansible_collections/arista/cvp/docs/modules/cv_configlet.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_configlet_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_configlet_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_container.md b/ansible_collections/arista/cvp/docs/modules/cv_container.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_container_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_container_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_device.md b/ansible_collections/arista/cvp/docs/modules/cv_device.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_device_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_device_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_facts.md b/ansible_collections/arista/cvp/docs/modules/cv_facts.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_facts_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_facts_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_image_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_image_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_tag_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_tag_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_task.md b/ansible_collections/arista/cvp/docs/modules/cv_task.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_task_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_task_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/modules/cv_validate_v3.md b/ansible_collections/arista/cvp/docs/modules/cv_validate_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/release-notes/v1.x.md b/ansible_collections/arista/cvp/docs/release-notes/v1.x.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/release-notes/v2.x.md b/ansible_collections/arista/cvp/docs/release-notes/v2.x.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/release-notes/v3.x.md b/ansible_collections/arista/cvp/docs/release-notes/v3.x.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/schema/cv_change_control_v3.md b/ansible_collections/arista/cvp/docs/schema/cv_change_control_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/schema/cv_container_v3.md b/ansible_collections/arista/cvp/docs/schema/cv_container_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/schema/cv_device_v3.md b/ansible_collections/arista/cvp/docs/schema/cv_device_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/schema/cv_tag_v3.md b/ansible_collections/arista/cvp/docs/schema/cv_tag_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/ansible_collections/arista/cvp/docs/schema/cv_validate_v3.md b/ansible_collections/arista/cvp/docs/schema/cv_validate_v3.md
@@ -1,5 +1,5 @@
 <!--
-  ~ Copyright (c) 2023 Arista Networks, Inc.
+  ~ Copyright (c) 2023-2024 Arista Networks, Inc.
   ~ Use of this source code is governed by the Apache License 2.0
   ~ that can be found in the LICENSE file.
   -->

diff --git a/...arista/cvp/examples/cv_change_control_v3/change_control_v3_create_approve_and_execute.yml b/...arista/cvp/examples/cv_change_control_v3/change_control_v3_create_approve_and_execute.yml
@@ -34,13 +34,13 @@
           parent: Leaf Upgrades
 
   tasks:
-    - name: "Create a change control on {{inventory_hostname}}"
+    - name: "Create a change control on {{ inventory_hostname }}"
       arista.cvp.cv_change_control_v3:
         state: set
         change: "{{ change }}"
       register: cv_change_control_test
 
-    - name: "Approve and Execute a change control on {{inventory_hostname}} using change_id field"
+    - name: "Approve and Execute a change control using change_id field on {{ inventory_hostname }}"
       arista.cvp.cv_change_control_v3:
         state: approve_and_execute
         change_id: ["{{ cv_change_control_test.data.id }}"]