Skip to content

Doc(eos_designs): seperate management settings tables#6305

Merged
gmuloc merged 12 commits intoaristanetworks:develfrom
joelbreton2:Doc(eos_designs)--seperate-management-settings-tables
Jan 9, 2026
Merged

Doc(eos_designs): seperate management settings tables#6305
gmuloc merged 12 commits intoaristanetworks:develfrom
joelbreton2:Doc(eos_designs)--seperate-management-settings-tables

Conversation

@joelbreton2
Copy link
Contributor

@joelbreton2 joelbreton2 commented Jan 5, 2026

Change Summary

Seperate different management setting in input variable table to allign with changes in AVD 6.0

Related Issue(s)

Fixes #5756

Component(s) name

arista.avd.eos_designs

Proposed changes

Create seperate table for

  • aaa_settings
  • dns_settings
  • event_handlers
  • logging_settings
  • management_eapi
  • ssh_settings
  • inband_ztp_bootstrap_file

Bundle following table

  • Management interface: ipv6_mgmt_destination_networks, ipv6_mgmt_gateway
  • Time configuration: ntp_settings, timezone

How to test

Review documentation site
Run pre-commit

Checklist

Repository Checklist

  • My code has been rebased from devel before I start
  • I have read the CONTRIBUTING document.
  • My change requires a change to the documentation and documentation have been updated accordingly.
  • I have updated molecule CI testing accordingly. (check the box if not applicable)

@joelbreton2 joelbreton2 requested review from a team as code owners January 5, 2026 19:12
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

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-6305
# Activate the virtual environment
source test-avd-pr-6305/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/joelbreton2/avd.git@Doc(eos_designs)--seperate-management-settings-tables#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/joelbreton2/avd.git#/ansible_collections/arista/avd/,Doc(eos_designs)--seperate-management-settings-tables --force
# Optional: Install AVD examples
cd test-avd-pr-6305
ansible-playbook arista.avd.install_examples

@github-actions github-actions bot added the state: conflict PR with conflict label Jan 7, 2026
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@ankudinov ankudinov self-assigned this Jan 7, 2026
@JulioPDX JulioPDX force-pushed the Doc(eos_designs)--seperate-management-settings-tables branch from c15d26f to 9e09007 Compare January 7, 2026 19:47
@github-actions github-actions bot removed the state: conflict PR with conflict label Jan 7, 2026
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Conflicts have been resolved. A maintainer will review the pull request shortly.

Copy link
Contributor

@JulioPDX JulioPDX left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ankudinov ankudinov left a comment

Choose a reason for hiding this comment

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

Looks good, happy to approve except I don't see ansible_collections/arista/avd/roles/eos_designs/docs/tables/removed-keys.md anywhere.

@gmuloc gmuloc enabled auto-merge January 9, 2026 11:56
@github-actions github-actions bot added the state: conflict PR with conflict label Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@joelbreton2 joelbreton2 force-pushed the Doc(eos_designs)--seperate-management-settings-tables branch from c3c8f43 to b8c8b80 Compare January 9, 2026 18:51
@github-actions github-actions bot removed the state: conflict PR with conflict label Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

Conflicts have been resolved. A maintainer will review the pull request shortly.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

@gmuloc gmuloc added this pull request to the merge queue Jan 9, 2026
Merged via the queue into aristanetworks:devel with commit 112cbe5 Jan 9, 2026
65 checks passed
JulioPDX pushed a commit to JulioPDX/avd that referenced this pull request Feb 17, 2026
…#6305)

Co-authored-by: Carl Buchmann <carl.buchmann@arista.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: gmuloc <gmulocher@arista.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Doc: Update the management-settings documentation

6 participants

Comments