Skip to content

Build azure#12949

Merged
v-atulyadav merged 3 commits into
Azure:masterfrom
prajval-um:build-azure
Oct 15, 2025
Merged

Build azure#12949
v-atulyadav merged 3 commits into
Azure:masterfrom
prajval-um:build-azure

Conversation

@prajval-um
Copy link
Copy Markdown
Contributor

Pull Request Description
Change(s):

Fixed incorrect template files (createUiDefinition.json and mainTemplate.json) in CiscoDuoSecurity solution that contained

Reason for Change(s):

Ensures proper Azure deployment functionality for CiscoDuoSecurity solution
Resolves template configuration mismatch that prevented successful deployments

Version Updated:

Yes, solution version updated to 3.0.5
Template files now contain correct CiscoDuo-specific configurations
Testing Completed:

Yes, validated that template files now contain correct CiscoDuo-specific content
Verified Azure deployment templates are properly configured for CiscoDuoSecurity
Confirmed removal of all Tomcat-specific references
Tested template validation and deployment functionality
Checked that the validations are passing and have addressed any issues that are present:

Yes, all template validation checks are now passing
Azure ARM template validation successful
No remaining Tomcat references found in CiscoDuo solution files

@prajval-um prajval-um requested review from a team as code owners October 13, 2025 03:45
@v-shukore v-shukore self-assigned this Oct 13, 2025
@v-shukore v-shukore added the Solution Solution specialty review needed label Oct 13, 2025
@v-shukore
Copy link
Copy Markdown
Contributor

Hi @prajval-um, do you also want to update the other files in this PR? Thanks!

@prajval-um
Copy link
Copy Markdown
Contributor Author

prajval-um commented Oct 14, 2025

Hi @prajval-um, do you also want to update the other files in this PR? Thanks!

I checked all the files, and this is the only one file with that name, so I changed just this file. Which file do you want to update?

v-shukore
v-shukore previously approved these changes Oct 15, 2025
@v-atulyadav v-atulyadav merged commit a216fa3 into Azure:master Oct 15, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Solution Solution specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants