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

Add a new module to manage Port Channel Interfaces in Fabric Resource Policies Template. (DCNE-52) #539

Merged
merged 14 commits into from
Dec 17, 2024

Conversation

gmicol
Copy link
Collaborator

@gmicol gmicol commented Sep 27, 2024

resolves #466

New Modules Added:

  • ndo_port_channel_interface

@gmicol gmicol added enhancement New feature or request jira-sync Sync this issue to Jira labels Sep 27, 2024
@gmicol gmicol self-assigned this Sep 27, 2024
@gmicol gmicol force-pushed the mso_port_channel_interface branch from 6a28890 to 5016d36 Compare October 7, 2024 17:07
@gmicol
Copy link
Collaborator Author

gmicol commented Oct 7, 2024

To all reviewers, the test is not complete yet as I am waiting for the ndo_interface_setting module to be merged. Please review accordingly. thanks!

plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
@gmicol gmicol force-pushed the mso_port_channel_interface branch from 8a0ef1a to 7f88d0a Compare October 11, 2024 14:39
@gmicol gmicol requested a review from akinross October 11, 2024 14:39
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
@gmicol gmicol requested a review from akinross October 15, 2024 17:50
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Outdated Show resolved Hide resolved
@gmicol gmicol force-pushed the mso_port_channel_interface branch from 0af57d3 to d5a18f8 Compare October 21, 2024 17:54
@gmicol gmicol requested review from shrsr and anvitha-jain October 21, 2024 17:55
@gmicol gmicol force-pushed the mso_port_channel_interface branch 2 times, most recently from 027e458 to eb8cf29 Compare October 21, 2024 17:58
plugins/module_utils/mso.py Outdated Show resolved Hide resolved
plugins/modules/ndo_port_channel_interface.py Show resolved Hide resolved
@gmicol gmicol requested a review from samiib October 23, 2024 17:15
@gmicol gmicol force-pushed the mso_port_channel_interface branch from eb8cf29 to 108465f Compare October 23, 2024 17:38
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

Attention: Patch coverage is 14.87603% with 103 lines in your changes missing coverage. Please review.

Project coverage is 17.75%. Comparing base (f65f3d0) to head (763afc2).

Files with missing lines Patch % Lines
plugins/modules/ndo_port_channel_interface.py 16.32% 82 Missing ⚠️
plugins/module_utils/mso.py 5.55% 17 Missing ⚠️
plugins/module_utils/template.py 20.00% 4 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (f65f3d0) and HEAD (763afc2). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (f65f3d0) HEAD (763afc2)
integration 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #539       +/-   ##
===========================================
- Coverage   72.58%   17.75%   -54.83%     
===========================================
  Files          55       93       +38     
  Lines        5504     8826     +3322     
  Branches     1462     2329      +867     
===========================================
- Hits         3995     1567     -2428     
- Misses       1174     7258     +6084     
+ Partials      335        1      -334     
Flag Coverage Δ
integration ?
sanity 17.75% <14.87%> (?)

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 previously approved these changes Oct 24, 2024
Copy link
Collaborator

@samiib samiib 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 Oct 24, 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

shrsr
shrsr previously approved these changes Dec 9, 2024
Copy link
Collaborator

@shrsr shrsr 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 force-pushed the mso_port_channel_interface branch from 4316408 to c5fd99a Compare December 10, 2024 09:38
@gmicol gmicol requested a review from shrsr December 10, 2024 09:38
…nel_interface module. Modify Documentation, Example sections and code logic for interface_descriptions attribute.
@gmicol gmicol force-pushed the mso_port_channel_interface branch from c5fd99a to 263ef9a Compare December 10, 2024 13:12
shrsr
shrsr previously approved these changes Dec 10, 2024
Copy link
Collaborator

@shrsr shrsr left a comment

Choose a reason for hiding this comment

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

LGTM

sajagana
sajagana previously approved these changes Dec 11, 2024
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!

akinross
akinross previously approved these changes Dec 11, 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

@gmicol gmicol dismissed stale reviews from akinross, sajagana, and shrsr via 227262d December 11, 2024 09:57
@gmicol gmicol force-pushed the mso_port_channel_interface branch 2 times, most recently from b2d05b4 to 9f40b3b Compare December 11, 2024 10:20
sajagana
sajagana previously approved these changes Dec 12, 2024
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 requested review from akinross and shrsr December 12, 2024 09:12
shrsr
shrsr previously approved these changes Dec 13, 2024
Copy link
Collaborator

@shrsr shrsr 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 previously approved these changes Dec 13, 2024
Copy link
Collaborator

@samiib samiib 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 Dec 13, 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

anvitha-jain
anvitha-jain previously approved these changes Dec 17, 2024
Copy link
Collaborator

@anvitha-jain anvitha-jain 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 dismissed stale reviews from anvitha-jain, akinross, samiib, shrsr, and sajagana via af83ff4 December 17, 2024 11:27
@gmicol gmicol force-pushed the mso_port_channel_interface branch from 9f40b3b to af83ff4 Compare December 17, 2024 11:27
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 b76ca6a into CiscoDevNet:master Dec 17, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request jira-sync Sync this issue to Jira
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: New module for Port Channel Interface in Fabric Resource Policies Template (DCNE-52)
7 participants