ciscoisesdk icon indicating copy to clipboard operation
ciscoisesdk copied to clipboard

ISE Network Device Group API Wrong

Open maconrad opened this issue 2 years ago • 3 comments

Prerequisites

  • [x ] Have you tested the operation in the API directly?
  • [x ] Do you have the latest SDK version?

Describe the bug NetworkDeviceGroup.create uses wrong params in ISE 3.2. It uses othername but the API expects ndgtype (see screenshot below). This was probably changed/updated at some point.

https://ciscoisesdk.readthedocs.io/en/latest/_modules/ciscoisesdk/api/v3_1_0/network_device_group.html#NetworkDeviceGroup.create

Expected behavior Update SDK to use correct name as seen in screeshot below. New: ndgtype, old: othername

Screenshots

image image image

Environment (please complete the following information):

  • ISE version and patch: 3.2
  • Python version: 3.10.7
  • SDK version: 2.0.8
  • OS version: OSX

maconrad avatar Nov 09 '22 16:11 maconrad

Hello @maconrad, currently version 3.2 is not supported, when we receive the corresponding documentation we will release a new version that supports it and you will be notified.

bvargasre avatar Nov 25 '22 22:11 bvargasre

I'm also encountering this issue on 3.1 with Patch 4. I was on 3.1 (w/ no Patches) and I was able to create a Network Device Group with othername, but I was not able to delete the Group. I kept getting a CRUD operation exception. I applied Patch 4, which resolved the issue of not being able to delete Network Device Groups, but now I'm unable to create them using param othername and using ndgtype as a param throws up an error.

I did discover a work around however to create by wrapping everything under the payload param.

So rather than this api.network_device_group.create(name='Location#All Locations#TestSite')

I did this instead api.network_device_group.create(payload= {'NetworkDeviceGroup': {'name': 'Location#All Locations#TestSite', 'ndgtype': 'Location'}})

boomsfib avatar Nov 30 '22 00:11 boomsfib

This is a backwards incompatible change that will be rolled back. othername will be reinstated, most likely in Patch 6.

bvargasre avatar Nov 30 '22 22:11 bvargasre

Hi @maconrad

At the end this was a CiscoISE bug Here you can see the issue https://bst.cisco.com/bugsearch/bug/CSCwd24304 I'm going to close the issue Regards

bvargasre avatar Jul 16 '24 20:07 bvargasre