-
Notifications
You must be signed in to change notification settings - Fork 0
Provision Enhancement #111
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
base: main
Are you sure you want to change the base?
Provision Enhancement #111
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all new fields must be supported.
description: The name of the feature template. | ||
type: str | ||
additional_identifiers: | ||
description: A list of additional identifiers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make it more descriptive, give examples.
description: The site name hierarchy. | ||
type: str | ||
excluded_attributes: | ||
description: A list of attributes to be excluded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make it more descriptive, give examples.
additional_identifiers: | ||
wlan_profile_name: ARUBA_SSID_profile | ||
site_name_hierarchy: Global/USA/SAN JOSE/BLD23 | ||
excluded_attributes: [example, test] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Give example from a real test.
@@ -198,6 +200,33 @@ | |||
- Must be either 5, 15 or 25 representing | |||
the proportion of APs to reboot at once. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing support for multiple wireless fields:
- {
"type": "boolean",
"required": false,
"name": "skipApProvision",
"displayText": "Skip AP Provision",
"description": "True if Skip AP Provision is enabled, else False"
},
{
"type": "map",
"address": "uuidfa2e4f30",
"required": false,
"name": "rollingApUpgrade",
"displayText": "Rolling AP Upgrade",
"description": "Rolling AP Upgrade"
},
{
"type": "string",
"enum": [],
"sensitive": false,
"default": "",
"constraints": [],
"required": false,
"name": "apAuthorizationListName",
"displayText": "Ap Authorization List Name",
"description": "AP Authorization List name. 'Obtain the AP Authorization List names by using the API call GET: /intent/api/v1/wirelessSettings/apAuthorizationLists. During re-provision, obtain the AP Authorization List configured for the given provisioned network device Id using the API call GET: /intent/api/v1/wireless/apAuthorizationLists/{networkDeviceId}'"
},
{
"type": "boolean",
"required": false,
"name": "authorizeMeshAndNonMeshAccessPoints",
"displayText": "Authorize Mesh And Non Mesh Access Points",
"description": "True if AP Authorization List should authorize against All Mesh/Non-Mesh APs, else false if AP Authorization List should only authorize against Mesh APs (Applicable only when Mesh is enabled on sites)"
},
{
"type": "map",
"address": "uuid789146a0",
"required": false,
"name": "featureTemplatesOverridenAttributes",
"displayText": "Feature Templates Overriden Attributes"
}
I could not find all these supported on wireless controller provisioning.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More missing: {
"type": "boolean",
"required": false,
"name": "enableRollingApUpgrade",
"displayText": "Enable Rolling AP Upgrade",
"description": "True if Rolling AP Upgrade is enabled, else False"
},
{
"type": "integer",
"constraints": [],
"required": false,
"name": "apRebootPercentage",
"displayText": "AP Reboot Percentage",
"description": "AP Reboot Percentage. Permissible values - 5, 15, 25"
}
],
"uuid789146a0": [
{
"type": "array",
"arrayType": "map",
"constraints": [],
"address": "uuid7a58402c",
"required": true,
"name": "editFeatureTemplates",
"displayText": "Edit Feature Templates",
"description": "This array consists of Feature Templates that need to be overridden during the provisioning process for the current provision instance. These edits will not alter the original designs of the Feature Templates but will only apply to the values for the current provisioning instance. Note: Locked attributes cannot be edited in the Provision API. Additionally, default feature templates ('systemTemplate') cannot be included in the payload, as they are not editable."
}
],
Type of Change
Description
Added functionality:
Bug Fix: NA
Root Cause (if applicable): NA
Fix Implemented: NA
Enhancement: Deletes wireless provisioned device and a new parameter feature template added
Enhancement Description: NA
Impact Area: NA
Testing Done:
Test cases covered: [Mention test case IDs or brief points]
Checklist
Ansible Best Practices
ansible-vault
or environment variables)Documentation
Screenshots (if applicable)
Notes to Reviewers