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.

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

Unable to package solution using v3 createsolution.ps1 #11512

Open
ashwinvenkatesha opened this issue Dec 2, 2024 · 5 comments
Open

Unable to package solution using v3 createsolution.ps1 #11512

ashwinvenkatesha opened this issue Dec 2, 2024 · 5 comments
Assignees
Labels
Solution Solution specialty review needed

Comments

@ashwinvenkatesha
Copy link
Contributor

Describe the bug
Unable to package solution using v3 createsolution.ps1

To Reproduce
After adding playbook content

  1. Am running createSolution.ps1 with data path as input
    /Users/<username>/Azure-Sentinel/Solutions/IllumioSaaS/data

This leads to,
Generating Playbook using Playbooks/Illumio-Port-Blocking-Switch/azuredeploy.json
Error occured in catch of createSolutionV3 file Error details are Exception calling "Add" with "2" argument(s): "Value cannot be null. (Parameter 'key')"

Expected behavior
Solution package should work as expected.

@v-visodadasi v-visodadasi self-assigned this Dec 2, 2024
@v-visodadasi v-visodadasi added the Solution Solution specialty review needed label Dec 2, 2024
@v-visodadasi
Copy link
Contributor

Hi @ashwinvenkatesha , Thanks for flagging this issue, we will investigate this issue and get back to you with some updates. Thanks!

@ashwinvenkatesha
Copy link
Contributor Author

https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/common/commonFunctions.ps1#L1047
this line seems like the culprit, I changed that to
if($IsFunctionAppResource)
and now solution zip is building.
However, the content built for playbooks is incorrect
Here is an example,
"name": "[[variables('hostingPlanName')]", "location": "[[variables('location')]",
The braces are not balanced anymore.

This is holding up our release and we need this to be fixed asap.
cc: @v-atulyadav

@v-prasadboke
Copy link
Contributor

Hello @ashwinvenkatesha, As discussed we will be working on this and try to resolve it from our.

Will be checking with MS team if needed

@ashwinvenkatesha
Copy link
Contributor Author

Here is the fix which helped unblock me
d40340c

@v-visodadasi
Copy link
Contributor

Hi @ashwinvenkatesha , I wanted to check in and see if the issue has been resolved from your end. Please let me know at your earliest convenience.

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

No branches or pull requests

3 participants