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_changes] Adding new module for physical interface (DCNE-50) #560

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

Conversation

anvitha-jain
Copy link
Collaborator

(object: interfaceProfiles)
solves #392

it required ndo_interface_seeting module #540

@anvitha-jain anvitha-jain added enhancement New feature or request jira-sync Sync this issue to Jira labels Oct 30, 2024
@github-actions github-actions bot changed the title [minor_changes] Adding new module for physical interface [minor_changes] Adding new module for physical interface (DCNE-50) Oct 30, 2024
@akinross
Copy link
Collaborator

sanity fails

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.

Sanity tests failed

plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
@anvitha-jain
Copy link
Collaborator Author

The sanity fails because of a function which doesn't exist, since this function is being added in another PR #539

plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Show resolved Hide resolved
@anvitha-jain anvitha-jain force-pushed the physical_interface branch 2 times, most recently from 54cabb7 to a710854 Compare November 12, 2024 18:32
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
plugins/modules/ndo_physical_interface.py Show resolved Hide resolved
gmicol
gmicol previously approved these changes Nov 19, 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

plugins/modules/ndo_physical_interface.py Outdated Show resolved Hide resolved
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

<<: *template_absent
state: present

# - name: Create interface policy group (physical policy)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are you planning to uncomment this when #540 is merged in?

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 Physical Interface in Fabric Resource Policies Template (DCNE-50)
5 participants