Skip to content

Removed event original processors in database and mq integrations #13518

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
Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Removed event original processors in database and mq integrations #13518

wants to merge 12 commits into from

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:28
@andrewkroh andrewkroh added Integration:postgresql PostgreSQL Integration:oracle Oracle Integration:rabbitmq RabbitMQ Logs and Metrics Integration:mysql MySQL Integration:ibmmq IBM MQ Integration:mongodb MongoDB Integration:mysql_enterprise MySQL Enterprise Integration:redis Redis Integration:kafka Kafka Integration:nats NATS Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Team:Security-Windows Platform Security Windows Platform team [elastic/sec-windows-platform] labels Apr 14, 2025
@elasticmachine
Copy link

Pinging @elastic/sec-windows-platform (Team:Security-Windows Platform)

ishleenk17
ishleenk17 reviewed Apr 15, 2025
View reviewed changes
@Alphayeeeet Alphayeeeet requested a review from ishleenk17 April 29, 2025 10:00
@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 Please review the linked issue for the details of the kibana version support increment. In short: It is necessary because of changes in the fleet-final-pipeline introduced in Kibana 8.11.0

@botelastic
Copy link

botelastic bot commented May 29, 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 29, 2025
@botelastic botelastic bot removed the Stalled label Jun 2, 2025
@Alphayeeeet
Copy link
Contributor Author

Any updates on review of obs-infraobs team?

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 Please re-review this PR.

ishleenk17
ishleenk17 reviewed Jun 13, 2025
View reviewed changes
@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 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:46
@ishleenk17
Copy link
Member

@ishleenk17 Please run CI

Ran and enabled auto merge

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

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@Alphayeeeet
Copy link
Contributor Author

After manually running tests of mysql, I cannot locate why the tests are failing. Therefore it might not be due to the pipeline changes made.

auto-merge was automatically disabled June 18, 2025 11:15

Head branch was pushed to by a user without write access

@Alphayeeeet
Copy link
Contributor Author

@andrewkroh or @ishleenk17 Can we check again via CI? If mysql still fails (I'm not sure what causes it as the tests run through locally) can some of you check that? Thanks

@ishleenk17
Copy link
Member

/test

@elasticmachine
Copy link

elasticmachine commented Jun 18, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

@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
72.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 @andrewkroh Any updates on why mysql tests are failing? Is that maybe a bug in the CI?

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

@andrewkroh andrewkroh andrewkroh approved these changes

@ishleenk17 ishleenk17 ishleenk17 approved these changes

Assignees
No one assigned
Labels
Integration:ibmmq IBM MQ Integration:kafka Kafka Integration:mongodb MongoDB Integration:mysql_enterprise MySQL Enterprise Integration:mysql MySQL Integration:nats NATS Integration:oracle Oracle Integration:postgresql PostgreSQL Integration:rabbitmq RabbitMQ Logs and Metrics Integration:redis Redis Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Team:Security-Windows Platform Security Windows Platform team [elastic/sec-windows-platform]
Projects
None yet
Milestone
No milestone
Development

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