Skip to content

Refactor: Rename avdutils to pyavd-utils#6065

Merged
ClausHolbechArista merged 13 commits intoaristanetworks:develfrom
ClausHolbechArista:refactor/rename-avdutils-to-pyavd-utils
Oct 29, 2025
Merged

Refactor: Rename avdutils to pyavd-utils#6065
ClausHolbechArista merged 13 commits intoaristanetworks:develfrom
ClausHolbechArista:refactor/rename-avdutils-to-pyavd-utils

Conversation

@ClausHolbechArista
Copy link
Copy Markdown
Contributor

Rename/move the WIP avdutils python library to pyavd-utils. Also move the bindings of Rust code into the same folder structure.

@ClausHolbechArista ClausHolbechArista requested review from a team as code owners October 27, 2025 14:42
@github-actions github-actions Bot added the type: code quality CI and development toolset label Oct 27, 2025
@github-actions
Copy link
Copy Markdown

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-6065
# Activate the virtual environment
source test-avd-pr-6065/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/ClausHolbechArista/avd.git@refactor/rename-avdutils-to-pyavd-utils#subdirectory=python-avd" --force
# Point Ansible collections path to the Python virtual environment
export ANSIBLE_COLLECTIONS_PATH=$VIRTUAL_ENV/ansible_collections
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/ClausHolbechArista/avd.git#/ansible_collections/arista/avd/,refactor/rename-avdutils-to-pyavd-utils --force
# Optional: Install AVD examples
cd test-avd-pr-6065
ansible-playbook arista.avd.install_examples

Copy link
Copy Markdown
Contributor

@gmuloc gmuloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall - just two questions

Comment thread pyavd-utils/Makefile
Comment thread pyavd-utils/tests/validation/test_validate_json.py
@gmuloc
Copy link
Copy Markdown
Contributor

gmuloc commented Oct 28, 2025

Should we increase coverage here?

Comment thread rust/validation/Cargo.toml Outdated
Comment thread .github/workflows/pull-request-management.yml Outdated
Copy link
Copy Markdown
Contributor

@gmuloc gmuloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - as discussed on the call, a couple of more tests will be added

@gmuloc gmuloc added the one approval This PR has one approval and is only missing one more. label Oct 28, 2025
@sonarqubecloud
Copy link
Copy Markdown

@ClausHolbechArista ClausHolbechArista merged commit 1b7abcd into aristanetworks:devel Oct 29, 2025
91 checks passed
JulioPDX pushed a commit to JulioPDX/avd that referenced this pull request Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

one approval This PR has one approval and is only missing one more. rn: Refactor type: code quality CI and development toolset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants