Skip to content

Remove unused httpjson agent file from {zeek, apache, windows, aws.cloudtrail, nginx} - #14995

Merged
vinit-chauhan merged 8 commits into
elastic:mainfrom
vinit-chauhan:remove_httpjson_template
Aug 25, 2025
Merged

Remove unused httpjson agent file from {zeek, apache, windows, aws.cloudtrail, nginx}#14995
vinit-chauhan merged 8 commits into
elastic:mainfrom
vinit-chauhan:remove_httpjson_template

Conversation

@vinit-chauhan

@vinit-chauhan vinit-chauhan commented Aug 20, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed commit message

httpjson input was removed from these integrations in this PR. However, the agent files were still there with the package.

This PR removes those unused httpjson files.

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

image image image image image

@vinit-chauhan vinit-chauhan added the bugfix Pull request that fixes a bug issue label Aug 20, 2025
@vinit-chauhan
vinit-chauhan marked this pull request as ready for review August 20, 2025 19:14
@vinit-chauhan
vinit-chauhan requested review from a team as code owners August 20, 2025 19:14
@vinit-chauhan vinit-chauhan self-assigned this Aug 20, 2025
@qcorporation qcorporation added the Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] label Aug 20, 2025
@elasticmachine

elasticmachine commented Aug 20, 2025

Copy link
Copy Markdown

Pinging @elastic/integration-experience (Team:Integration-Experience)

@vinit-chauhan vinit-chauhan added Integration:windows Windows Integration:apache Apache HTTP Server Integration:aws AWS Integration:zeek Zeek Integration:nginx Nginx Team:Elastic-Agent-Data-Plane Agent Data Plane team [elastic/elastic-agent-data-plane] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:Security-Windows Platform Security Windows Platform team [elastic/sec-windows-platform] Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] labels Aug 20, 2025
@elasticmachine

elasticmachine commented Aug 20, 2025

Copy link
Copy Markdown

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@elasticmachine

elasticmachine commented Aug 20, 2025

Copy link
Copy Markdown

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

@elasticmachine

elasticmachine commented Aug 20, 2025

Copy link
Copy Markdown

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

muthu-mps
muthu-mps reviewed Aug 21, 2025
View reviewed changes

@muthu-mps muthu-mps 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.

The enable_request_tracer is specific to httpjson which exists here. This is applicable for the AWS and Nginx integration too. Can this parameter removed from test_config? WDYT?

@efd6

efd6 commented Aug 21, 2025

Copy link
Copy Markdown
Contributor

The enable_request_tracer is specific to httpjson which exists here. This is applicable for the AWS and Nginx integration too. Can this parameter removed from test_config? WDYT?

Yeah, that should be removed.

efd6
efd6 reviewed Aug 21, 2025
View reviewed changes
Comment thread packages/apache/manifest.yml Outdated
# yvgJ38BRsFOtkRuAGSf6ZUwTO8JJRRIFnpUzXflAnGivK9M13D5GEQMmIl6U9Pvk
# sxSmbIUfc2SGJGCJD4I=
# -----END CERTIFICATE-----
# -----END CERTIFICATE-----

@efd6 efd6 Aug 21, 2025

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.

Suggested change
# -----END CERTIFICATE-----
# -----END CERTIFICATE-----

@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

elasticmachine commented Aug 21, 2025

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @vinit-chauhan

@elastic-sonarqube

elastic-sonarqube Bot commented Aug 21, 2025

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

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

See analysis details on SonarQube

efd6
efd6 approved these changes Aug 21, 2025
View reviewed changes

@efd6 efd6 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 for cert fix.

@muthu-mps muthu-mps changed the title Remove unused httpjson agent file from {zeek, apache, system, windows, aws.cloudtrail, nginx} Remove unused httpjson agent file from {zeek, apache, windows, aws.cloudtrail, nginx} Aug 22, 2025
gizas
gizas reviewed Aug 22, 2025
View reviewed changes
Comment thread packages/nginx/data_stream/access/_dev/test/system/test-default-config.yml
vars:
paths:
- "{{SERVICE_LOGS_DIR}}/access.log*"
enable_request_tracer: true

@gizas gizas Aug 22, 2025 •
edited
Loading

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.

Why is this removed? Was only for httpjson?

@vinit-chauhan vinit-chauhan Aug 22, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is httpjson specific, and used for system tests.

@vinit-chauhan
vinit-chauhan merged commit 342925a into elastic:main Aug 25, 2025
@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

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

@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

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

@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

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

@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

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

@elastic-vault-github-plugin-prod

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

Copy link
Copy Markdown
Contributor

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

@vinit-chauhan
vinit-chauhan deleted the remove_httpjson_template branch September 26, 2025 14:57
@vinit-chauhan
vinit-chauhan restored the remove_httpjson_template branch September 26, 2025 14:57
@vinit-chauhan
vinit-chauhan deleted the remove_httpjson_template branch September 26, 2025 14:57
@herrBez herrBez mentioned this pull request Nov 6, 2025
Closed
4 tasks
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
@vinit-chauhan
Remove unused httpjson agent file from {zeek, apache, windows, aws.cl…
3c1dc24 
…oudtrail, nginx} (elastic#14995)

* remove httpjson agent files

* remove httpjson files from {apache,aws.cloudtail,nginx,windows}

* added changelog entry

* revert formating in example ssl config

* remove tracer configs

* remove trailing white space from yml
@vinit-chauhan vinit-chauhan mentioned this pull request Nov 26, 2025
Merged
2 of 5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kgeller kgeller kgeller left review comments
@gizas gizas gizas approved these changes
@mjwolf mjwolf mjwolf approved these changes
@nfritts nfritts nfritts approved these changes
@andrzej-stencel andrzej-stencel andrzej-stencel approved these changes
@efd6 efd6 efd6 approved these changes
@khushijain21 khushijain21 Awaiting requested review from khushijain21 khushijain21 is a code owner automatically assigned from elastic/elastic-agent-data-plane
+1 more reviewer
@muthu-mps muthu-mps muthu-mps approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@vinit-chauhan vinit-chauhan

Labels

bugfix Pull request that fixes a bug issue Integration:apache Apache HTTP Server Integration:aws AWS Integration:nginx Nginx Integration:windows Windows Integration:zeek Zeek Team:Elastic-Agent-Data-Plane Agent Data Plane team [elastic/elastic-agent-data-plane] Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] 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] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-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.

apache,aws.cloudtrail,nginx,windows,zeek: removed input has agent template remaining

10 participants

@vinit-chauhan @elasticmachine @efd6 @gizas @mjwolf @kgeller @nfritts @andrzej-stencel @muthu-mps @qcorporation