Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Addition of HSRP modules under ACI L3out#505

Merged
lhercot merged 10 commits intoCiscoDevNet:masterfrom
shrsr:hsrp
Nov 17, 2023
Merged

Addition of HSRP modules under ACI L3out#505
lhercot merged 10 commits intoCiscoDevNet:masterfrom
shrsr:hsrp

Conversation

@shrsr
Copy link
Collaborator

@shrsr shrsr commented Oct 23, 2023

No description provided.

@shrsr shrsr self-assigned this Oct 23, 2023
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Attention: 138 lines in your changes are missing coverage. Please review.

Comparison is base (96dddfe) 96.52% compared to head (e50ad92) 35.31%.
Report is 45 commits behind head on master.

Files Patch % Lines
plugins/module_utils/aci.py 1.75% 56 Missing ⚠️
plugins/modules/aci_l3out_hsrp_group.py 40.00% 24 Missing ⚠️
plugins/modules/aci_hsrp_interface_policy.py 43.24% 21 Missing ⚠️
...lugins/modules/aci_l3out_hsrp_interface_profile.py 44.44% 20 Missing ⚠️
plugins/modules/aci_l3out_hsrp_secondary_vip.py 48.48% 17 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #505       +/-   ##
===========================================
- Coverage   96.52%   35.31%   -61.21%     
===========================================
  Files         183      210       +27     
  Lines        8479     9720     +1241     
  Branches     1265     1447      +182     
===========================================
- Hits         8184     3433     -4751     
- Misses        222     6287     +6065     
+ Partials       73        0       -73     
Flag Coverage Δ
integration ?
sanity 35.31% <32.01%> (+0.11%) ⬆️

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.

@shrsr shrsr requested a review from akinross October 25, 2023 21:13
@shrsr shrsr requested a review from akinross October 27, 2023 18:45
@lhercot lhercot requested a review from akinross November 3, 2023 17:02
akinross
akinross previously approved these changes Nov 3, 2023
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

akinross
akinross previously approved these changes Nov 7, 2023
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

@shrsr shrsr requested a review from sajagana November 8, 2023 18:39
akinross
akinross previously approved these changes Nov 9, 2023
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.

Have you tried it on every APIC version we have at our disposals (especially the old 4.2 version where a lot of class objects/attributes do not exist) ?

sajagana
sajagana previously approved these changes Nov 10, 2023
Copy link
Collaborator

@sajagana sajagana left a comment

Choose a reason for hiding this comment

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

LGTM!

gmicol
gmicol previously approved these changes Nov 13, 2023
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

akinross
akinross previously approved these changes Nov 17, 2023
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

@shrsr shrsr dismissed stale reviews from akinross, gmicol, and sajagana via a51aa4f November 17, 2023 19:20
@shrsr shrsr requested a review from lhercot November 17, 2023 19:38
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 910e70d into CiscoDevNet:master Nov 17, 2023
@gmicol
Copy link
Collaborator

gmicol commented Dec 21, 2023

is part of issue #126

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.

5 participants