Skip to content

[Cloud Security] Add elastic connector id to azure - #15326

Merged
seanrathier merged 2 commits into
elastic:mainfrom
seanrathier:azure-cloud-connector-id
Sep 17, 2025
Merged

[Cloud Security] Add elastic connector id to azure#15326
seanrathier merged 2 commits into
elastic:mainfrom
seanrathier:azure-cloud-connector-id

Conversation

@seanrathier

Copy link
Copy Markdown
Contributor

Proposed commit message

Add the cloud_connector_id field to the CSPM and Cloud Asset Inventory packages for Azure cloud connectors.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

Screenshots

Screenshot 2025-09-15 at 6 33 11 PM
@seanrathier seanrathier self-assigned this Sep 15, 2025
@seanrathier
seanrathier requested a review from a team as a code owner September 15, 2025 22:34
@seanrathier seanrathier added enhancement New feature or request Team:Cloud Security Cloud Security team [elastic/cloud-security-posture] labels Sep 15, 2025
@seanrathier
seanrathier requested a review from Copilot September 15, 2025 22:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the Elastic Cloud Connector ID field to Azure cloud connectors in both the Cloud Security Posture Management (CSPM) and Cloud Asset Inventory packages. This enables proper identification and configuration of Azure cloud connectors using Elastic's cloud connector infrastructure.

  • Added cloud_connector_id field configuration to Azure credential settings
  • Updated package versions to reflect the enhancement
  • Added changelog entries documenting the new functionality

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/cloud_security_posture/manifest.yml Version bump to 3.1.0-preview05
packages/cloud_security_posture/data_stream/findings/manifest.yml Added cloud_connector_id field to Azure credentials for both single and organization account types
packages/cloud_security_posture/changelog.yml Added changelog entry for cloud_connector_id enhancement
packages/cloud_asset_inventory/manifest.yml Version bump to 1.1.4
packages/cloud_asset_inventory/data_stream/asset_inventory/manifest.yml Added cloud_connector_id field to Azure credentials and new organization account cloud connectors configuration
packages/cloud_asset_inventory/changelog.yml Added changelog entry for Azure cloud connector support

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread packages/cloud_asset_inventory/changelog.yml Outdated
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

Comment thread packages/cloud_asset_inventory/data_stream/asset_inventory/manifest.yml Outdated
show_user: true
secret: true
description: Required when using Service Principal with Client Certificate
- name: azure.credentials.cloud_connector_id

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

azure_credentials_cloud_connector_id

@andrewkroh andrewkroh added Integration:cloud_asset_inventory Cloud Asset Discovery Integration:cloud_security_posture Security Posture Management labels Sep 16, 2025
@seanrathier
seanrathier enabled auto-merge (squash) September 16, 2025 14:32
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @seanrathier

@elastic-sonarqube

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube

@Omolola-Akinleye Omolola-Akinleye left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@seanrathier
seanrathier merged commit 70b658c into elastic:main Sep 17, 2025
8 of 9 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package cloud_asset_inventory - 1.1.4 containing this change is available at https://epr.elastic.co/package/cloud_asset_inventory/1.1.4/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package cloud_security_posture - 3.1.0-preview05 containing this change is available at https://epr.elastic.co/package/cloud_security_posture/3.1.0-preview05/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package cloud_security_posture - 3.1.0 containing this change is available at https://epr.elastic.co/package/cloud_security_posture/3.1.0/

tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:cloud_asset_inventory Cloud Asset Discovery Integration:cloud_security_posture Security Posture Management Team:Cloud Security Cloud Security team [elastic/cloud-security-posture]

5 participants