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

On prem support added to Illumio Solution #11673

Merged
merged 19 commits into from
Feb 3, 2025

Conversation

ashwinvenkatesha
Copy link
Contributor

Required items, please complete

Change(s):

  • Added new workbook and function app
  • Modified workbooks

Reason for Change(s):

  • To support onprem deployments in Illumio integration via Syslog

Version Updated:

  • Yes

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

…odified existing to handle onprem and saas pce; modified function app code
@ashwinvenkatesha ashwinvenkatesha requested review from a team as code owners January 16, 2025 22:34
@v-atulyadav v-atulyadav added the Solution Solution specialty review needed label Jan 17, 2025
"name": "group - 6"
}
],
"fromTemplateId": "sentinel-UserWorkbook",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the value of fromtemplateid

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to sentinel-OnPremHealthWorkbook

@v-prasadboke
Copy link
Contributor

You have updated the workbooks
Can you update the images located in workbook/images folder with latest workbook images

@v-prasadboke
Copy link
Contributor

Hello @ashwinvenkatesha, We are still facing the issue for connections string. I tried to create the playbook and encountered the error.

I'll try to check this from my end once.

@v-prasadboke
Copy link
Contributor

image

Still getting the error

@v-prasadboke
Copy link
Contributor

image

looks good

@v-atulyadav v-atulyadav merged commit f149b12 into Azure:master Feb 3, 2025
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.

3 participants