Skip to content

Conversation

@akinross
Copy link
Collaborator

No description provided.

@akinross akinross added the jira-sync Sync this issue to Jira label Nov 28, 2025
@github-actions github-actions bot changed the title Aci poe interface policy Aci poe interface policy (DCNE-599) Nov 28, 2025
- Default: `"30000"`
- Valid Values:
* `"15400"`, `"30000"`, `"4000"`, `"45000"`, `"60000"`, `"7000"`.
* Or a value in the range of `4000` to `60000`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this correct? Any value in the range of 4000 - 60000? That would also mean all valid values are also in this range. Should we only state the range and remove the valid values here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no was an issue in meta, there was validator in meta that should be ignored.

@akinross akinross force-pushed the aci_poe_interface_policy branch from 8bc9db7 to d1910bf Compare December 3, 2025 08:48
@akinross akinross requested a review from samiib December 3, 2025 08:48
@akinross akinross force-pushed the aci_poe_interface_policy branch from d1910bf to ef50857 Compare December 3, 2025 13:00
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


# aci_poe_interface_policy #

Manages ACI PoE Interface Policy
Copy link
Collaborator

Choose a reason for hiding this comment

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

I remember having a discussion that we would include expanding abbreviations in the docs some shape or form. Did we decide to include those in a glossary? Is an issue opened for this? @samiib @akinross

Copy link
Collaborator Author

@akinross akinross Dec 4, 2025

Choose a reason for hiding this comment

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

I think there was a discussion indeed but no issue has been tracked for this as far as I know. You want to change it to Power over Ethernet (PoE)? Or other suggestion?

@akinross akinross requested a review from shrsr December 4, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira-sync Sync this issue to Jira

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants