Skip to content

bitdefender: migrate push_configuration and push_statistics data streams to CEL#14787

Merged
efd6 merged 1 commit intoelastic:mainfrom
efd6:14759-bitdefender
Aug 5, 2025
Merged

bitdefender: migrate push_configuration and push_statistics data streams to CEL#14787
efd6 merged 1 commit intoelastic:mainfrom
efd6:14759-bitdefender

Conversation

@efd6
Copy link
Copy Markdown
Contributor

@efd6 efd6 commented Aug 4, 2025

Proposed commit message

See title.

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

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@efd6 efd6 self-assigned this Aug 4, 2025
@efd6 efd6 added enhancement New feature or request Integration:bitdefender BitDefender (Community supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Aug 4, 2025
@efd6 efd6 force-pushed the 14759-bitdefender branch from ddfeb41 to ae78062 Compare August 4, 2025 06:13
@efd6 efd6 mentioned this pull request Jul 31, 2025
Open
17 tasks
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

elastic-vault-github-plugin-prod Bot commented Aug 4, 2025

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@efd6 efd6 marked this pull request as ready for review August 4, 2025 06:42
@efd6 efd6 requested a review from a team as a code owner August 4, 2025 06:42
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Aug 4, 2025

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

ShourieG
ShourieG reviewed Aug 4, 2025
View reviewed changes
Comment thread packages/bitdefender/data_stream/push_configuration/sample_event.json
@ShourieG
Copy link
Copy Markdown
Contributor

ShourieG commented Aug 4, 2025

Minor clarification - Currently the httpjson state is backwards compatible with CEL, which is why this is not marked as a breaking-change ?

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Aug 4, 2025
@efd6 efd6 force-pushed the 14759-bitdefender branch from ae78062 to e6857e8 Compare August 4, 2025 21:22
@efd6
Copy link
Copy Markdown
Contributor Author

efd6 commented Aug 4, 2025 •
edited
Loading

Currently the httpjson state is backwards compatible with CEL, which is why this is not marked as a breaking-change ?

That is correct.

The input is disabled though to let the user add the secrets that are unfortunately not translated across (determined by experiment).

@efd6 efd6 force-pushed the 14759-bitdefender branch from e6857e8 to ae78062 Compare August 4, 2025 22:01
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Aug 4, 2025

💚 Build Succeeded

History

cc @efd6

@elastic-sonarqube
Copy link
Copy Markdown

elastic-sonarqube Bot commented Aug 4, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
81.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@efd6 efd6 merged commit 1262596 into elastic:main Aug 5, 2025
13 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

elastic-vault-github-plugin-prod Bot commented Aug 5, 2025

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

@chrisberkhout chrisberkhout mentioned this pull request Aug 6, 2025
Open
robester0403 pushed a commit to robester0403/integrations that referenced this pull request Aug 14, 2025
@efd6 @robester0403
bitdefender: migrate push_configuration and push_statistics data stre…
e017555 
…ams to CEL (elastic#14787)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ShourieG ShourieG ShourieG approved these changes

Assignees

@efd6 efd6

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:bitdefender BitDefender (Community supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@efd6 @elasticmachine @ShourieG @andrewkroh