Skip to content

Removed event.original processors in webserver/proxy related integrations #13519

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Jun 13, 2025
Merged

Removed event.original processors in webserver/proxy related integrations #13519

merged 6 commits into from
Jun 13, 2025

Conversation

Alphayeeeet
Copy link
Contributor

Label as enhancement

Proposed commit message

Remove event.original removal processors

Related issues

Review #10072 for additional info

@Alphayeeeet Alphayeeeet requested review from a team as code owners April 12, 2025 09:41
@andrewkroh andrewkroh added Integration:iis IIS Integration:haproxy HAProxy Integration:nginx Nginx Integration:php_fpm PHP-FPM Integration:tomcat Tomcat NetWitness Logs (Deprecated) Integration:envoyproxy Envoyproxy Integration:nginx_ingress_controller Nginx Ingress Controller Logs Integration:traefik Traefik Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Apr 14, 2025
@botelastic
Copy link

botelastic bot commented May 14, 2025

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label May 14, 2025
@Alphayeeeet
Copy link
Contributor Author

+1

@botelastic botelastic bot removed the Stalled label May 16, 2025
@gizas
Copy link
Contributor

gizas commented May 21, 2025

Approved for nginx_ingress_controller part

@Alphayeeeet
Copy link
Contributor Author

Same here. Are there any updates on the review of obs-infraobs?

@Alphayeeeet
Copy link
Contributor Author

@andrewkroh Same here, any updates on the review by obs-infraobs team?

ishleenk17
ishleenk17 reviewed Jun 13, 2025
View reviewed changes
@ishleenk17
Copy link
Member

@Alphayeeeet : 1 nit. Else looks good.

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 changelog has been fixed. Please run CI

@Alphayeeeet Alphayeeeet requested a review from ishleenk17 June 13, 2025 10:21
@ishleenk17
Copy link
Member

/test

@ishleenk17 ishleenk17 enabled auto-merge (squash) June 13, 2025 10:47
@elastic-vault-github-plugin-prod
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@ishleenk17 ishleenk17 merged commit d976a34 into elastic:main Jun 13, 2025
5 checks passed
@elasticmachine
Copy link

💚 Build Succeeded

@elastic-sonarqube Elastic SonarQube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

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

See analysis details on SonarQube

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

@Alphayeeeet Alphayeeeet deleted the remove-event.original3 branch July 14, 2025 06:30
@elastic-vault-github-plugin-prod
Copy link

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

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

@gizas gizas gizas approved these changes

@ishleenk17 ishleenk17 ishleenk17 approved these changes

Assignees
No one assigned
Labels
Integration:envoyproxy Envoyproxy Integration:haproxy HAProxy Integration:iis IIS Integration:nginx_ingress_controller Nginx Ingress Controller Logs Integration:nginx Nginx Integration:php_fpm PHP-FPM Integration:tomcat Tomcat NetWitness Logs (Deprecated) Integration:traefik Traefik Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Alphayeeeet @gizas @ishleenk17 @elasticmachine @andrewkroh