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.

Sign up for GitHub

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

Jump to bottom

Changes in workflow manager module #229

Closed
wants to merge 81 commits into from
Closed

Changes in workflow manager module #229

wants to merge 81 commits into from

Conversation

madhansansel
Copy link
Contributor

@madhansansel madhansansel commented Jan 24, 2025 •
edited
Loading

Description

Please include a summary of the changes and the related issue. Also, include relevant motivation and context.

Type of Change

  • [ Yes] Bug fix

  • New feature

  • Breaking change

  • Documentation update

      - Changes in device_credential_workflow_manager module
  - Changes in inventory_workflow_manager module
  - Changes in ise_radius_integration_workflow_manager module
  - Changes in network_settings_workflow_manager module
  - Enhancements in provision_workflow_manager module
  - Bug fixes in sda_fabric_devices_workflow_manager
  - Bug fixes in sda_fabric_transits_workflow_manager
  - Bug fixes in sda_fabric_transits_workflow_manager module
  - Bug fixes in site_workflow_manager module
  - Bug fixes in swim_workflow_manager module
  - Bug fixes in template_workflow_manager module
  - Bug fixes in user_role_workflow_manager module
  - sda_fabric_devices_workflow_manager - attribute 'route_distribution_protocol' was removed

Screenshots (if applicable)

Notes to Reviewers

JosePabloOcampo1212 and others added 30 commits December 18, 2024 08:17
@JosePabloOcampo1212
Merge pull request cisco-en-programmability#223 from cisco-en-program…
430ef36 
…mability/develop

Changes in workflow manager modules
@JosePabloOcampo1212
Update docs.yml
e36f9d6 
update docs
@JosePabloOcampo1212
Update docs.yml
9882bb2 
update workflows
@madhansansel
Merge from Mainline (#77)
db9df21
@syed-khadeerahmed
examppple added for un-tag and tag a image
74b9eba
@syed-khadeerahmed
review comments compleated
35572a9
@syed-khadeerahmed
review comments fixed
0aa0c49
@syed-khadeerahmed
review comments fixed
af66bfb
@madhansansel
examppple added for un-tag and tag a image (#78)
166390f
@JosePabloOcampo1212
Merge pull request cisco-en-programmability#225 from cisco-en-program…
2b4b54b 
…mability/develop

add compatibility with cisco version 2.3.7.9
@Kandarimegha
sync bug: no device available
f8c4476
@syed-khadeerahmed
inventory bug fixed
5856f2b
@syed-khadeerahmed
inventory bug fixed
017857e
@syed-khadeerahmed
inventory bug fixed
6dcba64
Provided the bulk input to the add and update fabric devices APIs in …
efb3035 
…a batch of 40
@Kandarimegha
Merge branch 'main' of https://github.com/cisco-en-programmability/ca…
dd4868f 
…talyst-center-ansible-intg into credentials_module
@Kandarimegha
Bug fix: Sync Device Credentials to Site devices failed
77a87aa
@Kandarimegha
Bug fix: Sync Device Credentials to Site devices failed
202a8b3
@Kandarimegha
Bug fix: Sync Device Credentials to Site devices failed
3ac6f6d
@madhansansel
Merge branch 'main' into main
9987b91
@madhansansel
Merge from mainline (#82)
9baf953
@Kandarimegha
Telemetry settings bug fix
2cf31ec
@MUTHU-RAKESH-27
Merge branch 'cisco-en-programmability:main' into main
7f42ed1
@Kandarimegha
Addressed Review comment
2bcfa13
Addressed the review comments
86c0420
@syed-khadeerahmed
review comments fixed
36dcaf0
@syed-khadeerahmed
fixed review comments
019554b
@madhansansel
Changes in template workflow manager module
806330a
@Kandarimegha
Address review comments
2456594
@Kandarimegha
Address Review comment
d774031
syed-khadeerahmed and others added 24 commits January 16, 2025 17:29
@syed-khadeerahmed
minor fix
ad98fff
@syed-khadeerahmed
review comments fixed
11a0603
@Kandarimegha
address Review comments
42b8e30
@Kandarimegha
address Review comments
708dea3
@MUTHU-RAKESH-27
The telemetry update workflow alignment with the API design (#86)
d5aa9fd
@MUTHU-RAKESH-27
Merge branch 'cisco-en-programmability:main' into main
9189cc5
Added the intergration tests for Fabric Transit and Fabric Devices mo…
45ad211 
…dule.
@Kandarimegha
Bug Fix: should return the error attribute instead of an unknown error
52d9620
@Kandarimegha
Bug Fix: should return the error attribute instead of an unknown error
6272afd
@syed-khadeerahmed
bugs fixed
ab03c39
@syed-khadeerahmed
sanity fixed
953957e
@MUTHU-RAKESH-27
return the error attribute instead of an unknown error for invalid cl…
e763324 
…i username (#92)
@MUTHU-RAKESH-27
Merge branch 'cisco-en-programmability:main' into main
56571cb
Removed the fabric devices and fabric transits
66009ed
@syed-khadeerahmed
addressed review comments
f666f7d
@madhansansel
bug fix in provision WLC (#91)
de32d27
@madhansansel
New inventory workflow feature (#93)
b1fac58
Addressed the review comments
188c03d
Addressed the review comments
307b8da
@madhansansel
Removed the business API from the SDA devices workflow (#90)
7a3a0e0
@ajithandrewj
User and Role Bug Fixed
2efb33f
@madhansansel
User and Role Bug Fixed (#96)
2c44910
@madhansansel
Merge from mainline (#102)
96e12d4
@madhansansel
Changes in workflow manager module
bd010a6
@madhansansel
Copy link
Contributor Author

@JosePabloOcampo1212, can you review and merge the changes?

@madhansansel
Copy link
Contributor Author

Need additional changes. Closing this request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JosePabloOcampo1212 JosePabloOcampo1212 Awaiting requested review from JosePabloOcampo1212

@fmunozmiranda fmunozmiranda Awaiting requested review from fmunozmiranda

@bvargasre bvargasre Awaiting requested review from bvargasre

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@madhansansel @JosePabloOcampo1212 @syed-khadeerahmed @Kandarimegha @MUTHU-RAKESH-27 @Abhishek-121 @ajithandrewj