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] Add ndo_tenant_custom_qos_policy for tenant Custom QoS configuration. (DCNE-127) #571

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

gmicol
Copy link
Collaborator

@gmicol gmicol commented Nov 22, 2024

fixes #453

@gmicol gmicol added enhancement New feature or request jira-sync Sync this issue to Jira labels Nov 22, 2024
@gmicol gmicol self-assigned this Nov 22, 2024
@gmicol gmicol requested a review from samiib November 25, 2024 18:32
@gmicol gmicol force-pushed the mso_custom_qos_policy branch from c45f5cf to a719f0c Compare November 26, 2024 14:08
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
@gmicol gmicol force-pushed the mso_custom_qos_policy branch from 56cd354 to c0c938f Compare December 3, 2024 05:53
@gmicol gmicol requested a review from akinross December 3, 2024 05:53
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

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

plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
Copy link

codecov bot commented Dec 4, 2024

Codecov Report

Attention: Patch coverage is 73.68421% with 25 lines in your changes missing coverage. Please review.

Project coverage is 72.08%. Comparing base (f65f3d0) to head (fdbcbf2).

Files with missing lines Patch % Lines
plugins/modules/ndo_tenant_custom_qos_policy.py 71.25% 13 Missing and 10 partials ⚠️
plugins/module_utils/utils.py 83.33% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #571      +/-   ##
==========================================
- Coverage   72.58%   72.08%   -0.50%     
==========================================
  Files          55       55              
  Lines        5504     5524      +20     
  Branches     1462     1468       +6     
==========================================
- Hits         3995     3982      -13     
- Misses       1174     1196      +22     
- Partials      335      346      +11     
Flag Coverage Δ
integration 72.08% <73.68%> (-0.50%) ⬇️

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.

@gmicol gmicol force-pushed the mso_custom_qos_policy branch from fdbcbf2 to 3a931b6 Compare December 5, 2024 13:47
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/module_utils/utils.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_tenant_custom_qos_policy.py Outdated Show resolved Hide resolved
@gmicol gmicol requested a review from sajagana December 6, 2024 15:23
sajagana
sajagana previously approved these changes Dec 6, 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!

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_custom_qos_policy branch from 855d85f to 8dcb5f7 Compare January 23, 2025 16:43
@lhercot lhercot requested review from akinross and anvitha-jain and removed request for akinross January 23, 2025 17:34
akinross
akinross previously approved these changes Jan 23, 2025
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 Jan 23, 2025
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

anvitha-jain
anvitha-jain previously approved these changes Jan 23, 2025
gmicol added 11 commits January 24, 2025 17:58
…nvert pyhton list of dict into equivalent NDO API format.
… Adjust format_list_dict function in module_utils/utils.py.
…stom_qos_policy module. Modify test case, documentation and format_list_dict in utils.py.
…y ndo_tenant_custom_qos_policy module's documentation.
…affic types for attributes in ndo_tenant_custom_qos_policy.
…ly the changes to ndo_tenant_custom_qos_policy.py and constants.py files.
…t with dict comprehension in ndo_tenant_custom_qos_policy module.
@gmicol gmicol dismissed stale reviews from anvitha-jain, shrsr, and akinross via 6f08e26 January 24, 2025 22:58
@gmicol gmicol force-pushed the mso_custom_qos_policy branch from 8dcb5f7 to 6f08e26 Compare January 24, 2025 22:58
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

@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 self-requested a review January 27, 2025 16:58
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

@lhercot lhercot requested a review from anvitha-jain January 27, 2025 19:05
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 Custom QoS Policy in Tenant Tenant Policies Template (DCNE-127)
7 participants