Refactor(eos_designs): Move default platform_settings. network_services_keys and connected_endpoints_keys to schema

[gmuloc]

Change Summary

Currently default platform_settingss are maintained both in schema default values and in the shared_utils python code.
This PR removes the python code.

It also leverages the same mechanism to retrieve the default value for connected_endpoint_keys and network_services_keys from the schema

Component(s) name

arista.avd.eos_designs

Proposed changes

Read default values from schema

How to test

Molecule tests are still all passing without changes

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)

[github-actions]

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-4395
# Activate the virtual environment
source test-avd-pr-4395/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/gmuloc/avd.git@remove-duplicate-platform-default-values#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/gmuloc/avd.git#/ansible_collections/arista/avd/,remove-duplicate-platform-default-values --force
# Optional: Install AVD examples
cd test-avd-pr-4395
ansible-playbook arista.avd.install_examples

[ClausHolbechArista]

LGTM. As agreed we will remove the campus comment again.

This should help keep things in sync. We should look into the same for other defaults like network_services_keys and connected_endpoints_keys.

[JulioPDX]

Great stuff, tested with built-in examples. Configs generated correctly when specifying platforms and connected endpoints keys. LGTM

[github-actions]

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

[github-actions]

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

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud