Skip to content
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

Doc: Deprecate the eos_designs and eos_cli_config_gen jsonschema.json… #4195

Merged

Conversation

gmuloc
Copy link
Contributor

@gmuloc gmuloc commented Jul 9, 2024

Change Summary

Announce in the release notes that the eos_cli_config_gen.jsonschema.json and eos_designs.jsonschema.json files won't be generated starting AVD 5.0.0

Component(s) name

arista.avd.eos_designs
arista.avd.eos_cli_config_gen

Proposed changes

deprecation message in release_notes 4.x.x

How to test

no test

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)

@gmuloc gmuloc requested review from a team as code owners July 9, 2024 14:50
Copy link

github-actions bot commented Jul 9, 2024

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-4195
# Activate the virtual environment
source test-avd-pr-4195/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/gmuloc/avd.git@chore-deprecate-avd-json-schemas#subdirectory=python-avd" --force
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/gmuloc/avd.git#/ansible_collections/arista/avd/,chore-deprecate-avd-json-schemas --force
# Optional: Install AVD examples
cd test-avd-pr-4195
ansible-playbook arista.avd.install_examples

@gmuloc gmuloc requested a review from carlbuchmann July 9, 2024 14:50
@github-actions github-actions bot added the type: documentation Improvements or additions to documentation label Jul 9, 2024
@@ -42,6 +42,8 @@ Pull request:
- AVD version 5.0.0 will drop support for Python version 3.9. The decision has been taken to remove Python version 3.9 support in AVD collection to anticipate its removal in `ansible-core`.
`ansible-core` version 2.15 End-Of-Life is scheduled for November 2024 and it will be the last `ansible-core` version supporting Python version 3.9 as documented here: https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#ansible-core-support-matrix.

- In AVD version 5.0.0, the `eos_designs.jsonschema.json` and `eos_cli_config_gen.jsonschema.json` will not be generated anymore. These schemas are not being used and have never been complete.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- In AVD version 5.0.0, the `eos_designs.jsonschema.json` and `eos_cli_config_gen.jsonschema.json` will not be generated anymore. These schemas are not being used and have never been complete.
- In AVD version 5.0.0, the `eos_designs.jsonschema.json` and `eos_cli_config_gen.jsonschema.json` will no longer be generated. These schemas are not being used and have never been completed.

Copy link
Member

Choose a reason for hiding this comment

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

This was added under 4.9.0, I'm thinking perhaps we need to move the Deprecations section at the top outside of a specific release with H2 heading?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm indeed need to prepare for 4.10.x
good point

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would keep complete instead of completed

@gmuloc gmuloc requested a review from carlbuchmann July 10, 2024 09:14
Copy link
Member

@carlbuchmann carlbuchmann left a comment

Choose a reason for hiding this comment

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

LGTM

@carlbuchmann carlbuchmann merged commit 72b2d27 into aristanetworks:devel Jul 16, 2024
39 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn: Doc type: documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants