Skip to content

Conversation

@samiib
Copy link
Collaborator

@samiib samiib commented Feb 1, 2024

  1. Added modules and tests for 1-N Relationships for BD.
  • bd_to_netflow_monitor_policy
  • bd_rogue_exception_mac
  1. Added, missing clean-up in bd_to_l3out tests.
  2. Please note this PR is reliant on the merging of PR Add Netflow Policies as new modules. #582

Closes #534

@samiib samiib self-assigned this Feb 1, 2024
@samiib samiib added the enhancement New feature or request label Feb 1, 2024
1. Added modules and tests for 1-N   Relationships for BD.
- bd_to_netflow_monitor_policy
- bd_rogue_exception_mac
2. Fixed missing clean-up  in bd_to_l3out tests.
@samiib samiib force-pushed the 534_bd_child_modules branch from 94bf795 to 460e912 Compare February 1, 2024 10:36
@codecov
Copy link

codecov bot commented Feb 1, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 30 lines in your changes are missing coverage. Please review.

Project coverage is 35.61%. Comparing base (f3b017f) to head (a7c0d97).

Files Patch % Lines
plugins/modules/aci_bd_rogue_exception_mac.py 50.00% 15 Missing ⚠️
...lugins/modules/aci_bd_to_netflow_monitor_policy.py 50.00% 15 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #600       +/-   ##
===========================================
- Coverage   96.30%   35.61%   -60.69%     
===========================================
  Files         241      243        +2     
  Lines       11024    11084       +60     
  Branches     1658     1664        +6     
===========================================
- Hits        10617     3948     -6669     
- Misses        308     7136     +6828     
+ Partials       99        0       -99     
Flag Coverage Δ
integration ?
sanity 35.61% <50.00%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@samiib samiib force-pushed the 534_bd_child_modules branch from c0401d8 to a339ba2 Compare February 20, 2024 23:34
@samiib samiib requested a review from akinross February 20, 2024 23:48
@samiib samiib force-pushed the 534_bd_child_modules branch from a339ba2 to 7dc4f24 Compare February 21, 2024 06:51
@samiib samiib requested a review from akinross February 21, 2024 06:52
akinross
akinross previously approved these changes Feb 21, 2024
Copy link
Collaborator

@akinross akinross left a comment

Choose a reason for hiding this comment

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

LGTM

@samiib samiib force-pushed the 534_bd_child_modules branch from 674a190 to a7c0d97 Compare February 26, 2024 22:13
Copy link
Collaborator

@akinross akinross 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
Collaborator

@gmicol gmicol 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
Member

@lhercot lhercot left a comment

Choose a reason for hiding this comment

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

LGTM

@lhercot lhercot merged commit 86aa9cb into CiscoDevNet:master Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cisco.aci.aci_bd module - add - advertise host routes

4 participants