Skip to content

Release 3.8.1 #458

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

Merged
merged 10 commits into from
Jul 2, 2025
Merged

Release 3.8.1 #458

merged 10 commits into from
Jul 2, 2025

Conversation

mikewiebe
Copy link
Collaborator

No description provided.

AKDRG and others added 10 commits June 17, 2025 15:53
…ng invalid fabrics on NDFC" (#429)

* Fix for Issue #428 "dcnm_fabric returns incorrect results when querying invalid fabrics on NDFC

* Fix Test Fabric Query Unit Tests - 0030 and 0031

* "Fix Assertion Cases for DCNM Test Fabric Query Unit TC 00030, 00031"

---------

Co-authored-by: = <=>
* Refactored SANITY test cases, top level playbook and added pydantic schema and helper functions to compare actual and expected data

* Refactored DELETED tests, added extension to handle deleted network checks in action plug in, updated schema, updated tags

* fixed sanity CI checks

* Fixes sanity CI checks

* fixed ignores error, added pydantic and deepdiff in the github actions workflow

* refactored tests so that variables are defined only once, restructured action plugins to make it more readable, finished query and overridden

* fixed ignores error

* rearranged argument assignment and checking in action plugin

* refactored merged and replaced. Added ignore_fields to process_deepdiff. moved templates into separate folders

* fixed issue with check_deleted when config_path is empty

* only check changed attribute for idempotency as response may not be of 0 length

* improved pydantic parser

* refactored self contained tests for DCNM network

* fixed minor issues and merged invalid parameters

* changed parse_obj to model_validate for pydantic
* "Workflow Sanity - Ignore Files Sorting"

* Remove comments

---------

Co-authored-by: = <=>
* Update dcnm_network.py

add condition if fabric is Multisite, remove mcastGroup key

* Update dcnm_network.py

fix pylint issue

* Fix unit tests

---------

Co-authored-by: mwiebe <mwiebe@cisco.com>
* Fix device not reachable scenarios

* Fix comments

* Handle remove case

* Fix github actions issue

* Add clarifying comments

* Add clarifying comments
* Update dcnm_interface.py

refactor breakout

* fix trainling whitespace

* fix pep8 issue

* Update dcnm_interface.py

* Update dcnm_interface.py

* Fix unit tests

* Update integtation tests

* Add ansible.cfg file

* Fix orphan port bug

* Add missing docs

---------

Co-authored-by: mwiebe <mwiebe@cisco.com>
* Update galaxy.yml

* Update CHANGELOG.rst

* Update README.md
@AKDRG AKDRG self-requested a review July 2, 2025 14:31
Copy link
Collaborator

@AKDRG AKDRG 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!

@mikewiebe mikewiebe merged commit a73ca38 into main Jul 2, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants