Skip to content

[Cloud Security] Fix cluster_id missing error in the Ingest Pipeline - #9434

Merged
opauloh merged 4 commits into
mainfrom
csp/fix-cluster-id-missing
Mar 28, 2024
Merged

[Cloud Security] Fix cluster_id missing error in the Ingest Pipeline#9434
opauloh merged 4 commits into
mainfrom
csp/fix-cluster-id-missing

Conversation

@opauloh

@opauloh opauloh commented Mar 22, 2024

Copy link
Copy Markdown
Contributor

Summary

This PR Fixes the Ingest Pipeline Error, this error was due to a missing check for the presence of the cluster_id field used in the copy_from.

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.

Author's Checklist

  • no pipeline errors in the latest findings docs

How to test this PR locally

Follow the guide described here to test this PR locally.

To collect the necessary data you can execute the following Reindex command in the dev tools (The pipeline is the important part!), point host and credentials to one of the long-running environments

Related issues

@opauloh opauloh added bug Something isn't working, use only for issues Team:Cloud Security Cloud Security team [elastic/cloud-security-posture] labels Mar 22, 2024
@opauloh
opauloh requested a review from a team as a code owner March 22, 2024 23:47
@opauloh opauloh changed the title [Cloud Security] Fix cluster_id missing error in the Ingest Pipeline Mar 23, 2024
@opauloh opauloh changed the title [Cloud Security] Fix cluster_id missing error in the Ingest Pipeline and add remove empty cloud.account.name fields Mar 23, 2024
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

@elastic-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@opauloh
opauloh merged commit 8d519cf into main Mar 28, 2024
@elasticmachine

Copy link
Copy Markdown

Package cloud_security_posture - 1.9.0-preview02 containing this change is available at https://epr.elastic.co/search?package=cloud_security_posture

@elasticmachine

Copy link
Copy Markdown

Package cloud_security_posture - 1.9.0 containing this change is available at https://epr.elastic.co/search?package=cloud_security_posture

@andrewkroh andrewkroh added the Integration:cloud_security_posture Security Posture Management label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues Integration:cloud_security_posture Security Posture Management Team:Cloud Security Cloud Security team [elastic/cloud-security-posture]

5 participants