-
Notifications
You must be signed in to change notification settings - Fork 201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor(eos_validate_state): Add warning for ansible test mode #4173
Refactor(eos_validate_state): Add warning for ansible test mode #4173
Conversation
Review docs on Read the Docs To test this pull request: # Create virtual environment for this testing below the current directory
python -m venv test-avd-pr-4173
# Activate the virtual environment
source test-avd-pr-4173/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/gmuloc/avd.git@deprecate-legacy-validate-state#subdirectory=python-avd" --force
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/gmuloc/avd.git#/ansible_collections/arista/avd/,deprecate-legacy-validate-state --force
# Optional: Install AVD examples
cd test-avd-pr-4173
ansible-playbook arista.avd.install_examples |
ansible_collections/arista/avd/roles/eos_validate_state/README.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_validate_state/tasks/ansible_tests.yml
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/plugins/modules/_emit_warning.py
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/plugins/Modules_and_action_plugins/_emit_warning.md
Show resolved
Hide resolved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change Summary
Indicate that only
use_anta: true
behavior will be kept in 5.0.0Component(s) name
arista.avd.eos_validate_state
Proposed changes
After too long searching, added a temporary action plugin to emit a warning. We will remove it when we remove the ansible-test before 5.0.0. This plugin is meant to be used only for this warning
How to test
In the molecule scenario for validate_state, set
use_anta
tofalse
and run. The scenario will fail because it is not meant for this mode, but you will see the emitted warningChecklist
Repository Checklist