Skip to content

Revert "Update BigID package and add stepId variable"#13617

Merged
v-dvedak merged 1 commit intoAzure:masterfrom
mhebrard-bigid:bigid-stepid-fixes
Feb 17, 2026
Merged

Revert "Update BigID package and add stepId variable"#13617
v-dvedak merged 1 commit intoAzure:masterfrom
mhebrard-bigid:bigid-stepid-fixes

Conversation

@mhebrard-bigid
Copy link
Contributor

@mhebrard-bigid mhebrard-bigid commented Feb 11, 2026

This reverts commit d3ddbc8.

Proper fix for TTK validation exception is in https://github.com/Azure/arm-ttk/pull/794/changes

Support URL changed to a valid one

@mhebrard-bigid mhebrard-bigid requested review from a team as code owners February 11, 2026 18:41
@v-shukore v-shukore added the Solution Solution specialty review needed label Feb 12, 2026
@v-maheshbh
Copy link
Contributor

v-maheshbh commented Feb 12, 2026

Hi @mhebrard-bigid
There appear to be issues with the PR. It seems incomplete, as it only includes changes to the package (zip) without corresponding updates to the mainTemplate.json and createUIDefinition.json files.

Kindly resolve the branch conflicts with the latest master/main and update the PR.

Thanks!

@mhebrard-bigid
Copy link
Contributor Author

changes properly reverted @v-maheshbh

@mhebrard-bigid mhebrard-bigid marked this pull request as draft February 12, 2026 13:44
This reverts commit d3ddbc8.

Changed support url to a valid one
@v-maheshbh
Copy link
Contributor

Hi @mhebrard-bigid

The PR is currently in draft state. Please let me know once it is ready for review.

Thanks!

@mhebrard-bigid mhebrard-bigid marked this pull request as ready for review February 13, 2026 13:36
@mhebrard-bigid
Copy link
Contributor Author

@v-maheshbh can you retrigger the arm-ttk checks ? thanks

@mhebrard-bigid
Copy link
Contributor Author

@v-maheshbh please approve this. Thanks

@v-maheshbh
Copy link
Contributor

Hi @mhebrard-bigid

Reverting these changes results in the ARM-TTK validation error shown below for the stepId variable.

image

Thanks!

@mhebrard-bigid
Copy link
Contributor Author

@v-maheshbh , you retriggered the ttk and it's now working https://github.com/Azure/Azure-Sentinel/actions/runs/21951881686

This is because https://github.com/Azure/arm-ttk/pull/794/changes got merged

So now everything is green !

@v-dvedak v-dvedak merged commit 34b871f into Azure:master Feb 17, 2026
33 of 34 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.

5 participants