Skip to content

BeyondTrust PM Cloud - Updated connector template to utilize a pre-built ZIP for more seamless deployment#13581

Merged
v-atulyadav merged 10 commits intoAzure:masterfrom
jamos-bt:master
Feb 23, 2026
Merged

BeyondTrust PM Cloud - Updated connector template to utilize a pre-built ZIP for more seamless deployment#13581
v-atulyadav merged 10 commits intoAzure:masterfrom
jamos-bt:master

Conversation

@jamos-bt
Copy link
Contributor

@jamos-bt jamos-bt commented Feb 5, 2026

Change(s):

  • Updated the main ARM template so the Function App references a pre-built ZIP for the functions source code
  • Added this pre-build ZIP to the Solutions/BeyondTrustPMCloud/Data Connectors folder
  • Updated instructions in multiple files to correctly guide creation of the API account in PM Cloud

Reason for Change(s):

  • The initial submission required admins to download the data connector source (a .NET app), build it, package it, and deploy it to the Function App created by the template. The approach has been updated to utilize a pre-built ZIP that is referenced directly from the official repository. This matches what other similar solutions / connectors are doing.
  • There was also a misleading statement in a few locations, including the data connector's landing page, around the creation of the API account in PM Cloud ... this was corrected.

Version Updated:

  • No

Testing Completed:

  • Yes

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

  • Yes

…ctor

- Add conditional deployment script for FC1 to upload function package to blob storage
- Upload package as 'released-package.zip' with kudu-state.json metadata (matches Azure Functions deployment pattern)
- Configure functionAppConfig for FC1 with blob container deployment storage
- Add managed identity and Storage Blob Data Contributor RBAC for deployment script
- Conditional logic ensures zero impact on Y1/EP1/EP2/EP3 hosting plans (still use WEBSITE_RUN_FROM_PACKAGE)
- FC1 uses BeyondTrust__ prefix for app settings, Y1/EP uses BeyondTrust: prefix
- Tested and verified: Functions load correctly, data flows to Log Analytics
- Clarify that customers can create OAuth API credentials directly in their PM Cloud instance
- Remove reference to contacting BeyondTrust support for API credentials
- Improve self-service documentation for data connector setup
- Update remaining references to API credential creation process
- Ensure consistent messaging about self-service credential creation
@jamos-bt jamos-bt marked this pull request as ready for review February 5, 2026 20:40
@jamos-bt jamos-bt requested review from a team as code owners February 5, 2026 20:40
@v-shukore v-shukore self-assigned this Feb 6, 2026
@v-shukore v-shukore added the Solution Solution specialty review needed label Feb 6, 2026
@v-shukore
Copy link
Contributor

Hi @jamos-bt, since you’ve already updated the main template, please update the ZIP package file as well. Thanks!

@jamos-bt
Copy link
Contributor Author

@v-shukore - Sorry about that. Forgot to repackage that. All done now, and I also synced with the upstream repo while I was at it.

@v-shukore
Copy link
Contributor

v-shukore commented Feb 16, 2026

Hi @jamos-bt,
could you please share a screenshot of the running connector so we can proceed with the merge?
Also, update the release notes for these changes.
Thanks!

@jamos-bt
Copy link
Contributor Author

Hi @v-shukore ... here are a few screenshots from the workbook that queries the data ingested into Sentinel by the data connector:

image image

With regard to the release notes, there is nothing to update. This is still the initial release. We simply changed the deployment method for the Azure Function App code to be more user-friendly as soon as we noticed that during the marketplace listing approval process. Nothing has been published yet though. :)

@v-shukore
Copy link
Contributor

Hi @jamos-bt,
As this is a Function App–based data connector, please ensure that all required files are added and the correct folder structure for a Function App connector is followed.
Also, please update the name of the newly created ZIP file it should not contain a dot (.) in the file name.
You may refer to the following solution, which already includes a properly structured Function App data connector, for guidance:
https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Cloudflare/Data%20Connectors
Thanks!

@jamos-bt
Copy link
Contributor Author

@v-shukore I've renamed the zip and updated the reference to it in the ARM template. However, I'm not sure what else would need to change. I see other files in the directory for the connector you referenced, but I don't understand what those files are or whether each of them are required.

I modeled my own after another solution which utilizes an Azure Function App but that is .NET based just like my own. This one: https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/VaronisSaaS/Data%20Connectors

As you can see, my structure and content matches what is present for that connector. In fact, that connector uses a . in the ZIP file name which is why I thought that was acceptable. If there are other changes I need to make, can you please point me to documentation which clearly states what is required? There's so much variation in the content and structure of these various solutions that there's really no way (without official documentation / guidelines) to know what are the actual requirements.

@jamos-bt
Copy link
Contributor Author

@v-shukore Are we good now that the zip is renamed and we match the structure of other existing .NET Azure Function App connectors? Or are there other specific changes you're requesting?

If there are other changes, please see my previous comment and provide a link to documentation that clearly describes the requirements so we can ensure meet them. Thanks!

@v-shukore
Copy link
Contributor

Hi @jamos-bt, as you confirmed this is .net based connector then its fine. We can move forward with this. Thanks

@v-atulyadav v-atulyadav merged commit a74578a into Azure:master Feb 23, 2026
33 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.

4 participants