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

[minor_change] Added ndo_mcp_global_policy for fabric policy global MCP configuration (DCNE-70) #574

Merged
merged 5 commits into from
Dec 13, 2024

Conversation

samiib
Copy link
Collaborator

@samiib samiib commented Dec 2, 2024

Fixed #451

This MCP Global policy module is based on a 1:1 relationship with Fabric Policy templates.
A decision was made to keep this configuration in a separate module instead of adding it to ndo_template.

@github-actions github-actions bot changed the title [minor_change] Added ndo_mcp_global_policy for fabric policy global MCP configuration [minor_change] Added ndo_mcp_global_policy for fabric policy global MCP configuration (DCNE-70) Dec 2, 2024
@samiib samiib self-assigned this Dec 2, 2024
@samiib samiib added the enhancement New feature or request label Dec 2, 2024
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Show resolved Hide resolved
@samiib samiib requested a review from shrsr December 3, 2024 02:13
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_mcp_global_policy.py Outdated Show resolved Hide resolved
@samiib samiib requested a review from akinross December 3, 2024 13:19
akinross
akinross previously approved these changes Dec 3, 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 3, 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 previously approved these changes Dec 3, 2024
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

codecov bot commented Dec 3, 2024

Codecov Report

Attention: Patch coverage is 31.03448% with 40 lines in your changes missing coverage. Please review.

Project coverage is 17.84%. Comparing base (f65f3d0) to head (7d7b199).
Report is 18 commits behind head on master.

Files with missing lines Patch % Lines
plugins/modules/ndo_mcp_global_policy.py 31.03% 40 Missing ⚠️

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

HEAD has 1 upload less than BASE
Flag BASE (f65f3d0) HEAD (7d7b199)
integration 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #574       +/-   ##
===========================================
- Coverage   72.58%   17.84%   -54.74%     
===========================================
  Files          55       93       +38     
  Lines        5504     8791     +3287     
  Branches     1462     2322      +860     
===========================================
- Hits         3995     1569     -2426     
- Misses       1174     7221     +6047     
+ Partials      335        1      -334     
Flag Coverage Δ
integration ?
sanity 17.84% <31.03%> (?)

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.

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

@samiib samiib dismissed stale reviews from sajagana, gmicol, shrsr, and akinross via 8fe602d December 5, 2024 02:05
@samiib samiib force-pushed the mcp_global_policy branch from df330d3 to 8fe602d Compare December 5, 2024 02:05
@samiib samiib requested a review from shrsr December 5, 2024 02:07
@samiib samiib force-pushed the mcp_global_policy branch from 8fe602d to 2f09443 Compare December 5, 2024 02:13
sajagana
sajagana previously approved these changes Dec 5, 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!

@samiib samiib requested a review from akinross December 8, 2024 22:09
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

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

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
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

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 8fa10c2 into CiscoDevNet:master Dec 13, 2024
20 of 23 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: New module for MCP Global Policy in Fabric Fabric Policies Template (DCNE-70)
7 participants