Skip to content

cloud_asset_inventory: Mark gcp_credentials_cloud_connector_id as secret#17377

Merged
Omolola-Akinleye merged 3 commits intomainfrom
cai-gcp-connector-secret
Feb 11, 2026
Merged

cloud_asset_inventory: Mark gcp_credentials_cloud_connector_id as secret#17377
Omolola-Akinleye merged 3 commits intomainfrom
cai-gcp-connector-secret

Conversation

@amirbenun
Copy link
Copy Markdown
Contributor

Summary

  • Mark gcp_credentials_cloud_connector_id as secret in asset_inventory manifest.
  • Bump version to 1.5.0-preview03.

Copilot AI review requested due to automatic review settings February 11, 2026 16:51
@amirbenun amirbenun requested a review from a team as a code owner February 11, 2026 16:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 11, 2026

Vale Linting Results

Summary: 3 suggestions found

💡 Suggestions (3)
File Line Rule Message
packages/cloud_asset_inventory/_dev/ci-regression-investigation.md 1 Elastic.HeadingColons Capitalize ': c'.
packages/cloud_asset_inventory/_dev/ci-regression-investigation.md 22 Elastic.HeadingColons Capitalize ': r'.
packages/cloud_asset_inventory/_dev/ci-regression-investigation.md 28 Elastic.Wordiness Consider using 'because' instead of 'since'.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the cloud_asset_inventory integration package to treat gcp_credentials_cloud_connector_id as a secret value (masked/secured in Fleet policy), along with the associated pre-release version bump.

Changes:

  • Bump cloud_asset_inventory package version to 1.5.0-preview03.
  • Mark gcp_credentials_cloud_connector_id as secret: true in the asset_inventory data stream manifest.
  • Add a 1.5.0-preview03 changelog entry (and add a new _dev investigation note).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
packages/cloud_asset_inventory/manifest.yml Bumps package version to 1.5.0-preview03.
packages/cloud_asset_inventory/data_stream/asset_inventory/manifest.yml Marks gcp_credentials_cloud_connector_id as a secret var.
packages/cloud_asset_inventory/changelog.yml Adds release note for 1.5.0-preview03.
packages/cloud_asset_inventory/_dev/ci-regression-investigation.md Adds internal CI regression investigation notes under _dev.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +10 to +13
- version: "1.5.0-preview03"
changes:
- description: Mark gcp_credentials_cloud_connector_id as secret
type: enhancement
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

The new 1.5.0-preview03 changelog entry is missing a link: field, while every other entry in this package’s changelog includes a PR link. Add the PR URL to keep the changelog consistent and to avoid breaking any tooling that expects the link metadata.

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +6
# CI regression: cloud_security_posture (9.4 stack) passed Feb 3, broke after

## What we know

- **PR 17185** (Cloud Security Posture – GCP Cloud Connectors) **CI passed on Feb 3rd** with elastic-package 0.118 and 9.4.0 stack.
- Same check (**Check integrations cloud_security_posture**) **fails now** (week of Feb 10).
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

This PR’s description is scoped to marking gcp_credentials_cloud_connector_id as secret and bumping the package version, but it also adds a CI regression investigation document about cloud_security_posture. Consider moving this note to a separate PR (or updating the PR description) to keep the change set focused and easier to track in history.

Copilot uses AI. Check for mistakes.
@andrewkroh andrewkroh added the Integration:cloud_asset_inventory Cloud Asset Discovery label Feb 11, 2026
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

@Omolola-Akinleye Omolola-Akinleye merged commit 2c4877a into main Feb 11, 2026
10 checks passed
@Omolola-Akinleye Omolola-Akinleye deleted the cai-gcp-connector-secret branch February 11, 2026 17:26
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

jakubgalecki0 pushed a commit to jakubgalecki0/integrations that referenced this pull request Feb 19, 2026
…ret (elastic#17377)

* Mark gcp_credentials_cloud_connector_id as secret (1.5.0-preview03)

* Add PR link to changelog

* Remove accidentally included _dev/ci-regression-investigation.md
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants