Skip to content

[carbon_black_cloud] fix pagination logic for asset_vulnerability_summary data stream - #16564

Merged
brijesh-elastic merged 3 commits into
elastic:mainfrom
brijesh-elastic:carbon_black_cloud-4.0.1
Dec 16, 2025
Merged

[carbon_black_cloud] fix pagination logic for asset_vulnerability_summary data stream#16564
brijesh-elastic merged 3 commits into
elastic:mainfrom
brijesh-elastic:carbon_black_cloud-4.0.1

Conversation

@brijesh-elastic

Copy link
Copy Markdown
Contributor

Proposed commit message

carbon_black_cloud: fix pagination logic for asset_vulnerability_summary data stream

For CEL input, it correctly sets the `cursor.processed_num_rows` to 0 when it's the last page.

For HTTPJSON input, it updates the `body.rows` to 1000 and correctly updates the `body.start`
to fetch the next page. It also adds `do_not_log_failure: true` in set processors to avoid
updating fleet health status to degraded.

This change bumps minimum Kibana version to ^8.19.4 || ~9.0.7 || ^9.1.4

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

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/carbon_black_cloud directory.
  • Run the following command to run tests.

elastic-package test

Related issues

@brijesh-elastic brijesh-elastic self-assigned this Dec 15, 2025
@brijesh-elastic
brijesh-elastic requested a review from a team as a code owner December 15, 2025 15:10
@brijesh-elastic brijesh-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:carbon_black_cloud VMware Carbon Black Cloud bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Dec 15, 2025
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

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

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @brijesh-elastic

@ShourieG ShourieG 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

@brijesh-elastic
brijesh-elastic merged commit 7b64280 into elastic:main Dec 16, 2025
8 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

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

Labels

bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:carbon_black_cloud VMware Carbon Black Cloud Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants