Skip to content

[Apache] Add support for apache ISO timestamp-format in errorlog #14647

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.

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

Merged
merged 6 commits into from
Jul 29, 2025

Conversation

Alphayeeeet
Copy link
Contributor

@Alphayeeeet Alphayeeeet commented Jul 22, 2025

Please label as enhancement

Proposed commit message

Add support for apache ISO timestamp-format in errorlog

According to the doc, apache supports an ISO timestamp since version 2.4.58
https://httpd.apache.org/docs/2.4/mod/core.html#errorlogformat

This feature should be supported in a backward compatible manner.

@Alphayeeeet Alphayeeeet requested a review from a team as a code owner July 22, 2025 15:13
@andrewkroh andrewkroh added Integration:apache Apache HTTP Server Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Jul 22, 2025
@Alphayeeeet
Copy link
Contributor Author

@mykola-elastic Thanks. I forgot to add the PR link.

Copy link
Contributor

@mykola-elastic mykola-elastic left a comment

Choose a reason for hiding this comment

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

Got those new messages with

ErrorLogFormat "[%{cuz}t] [%-m:%-l] [R:%L] [C:%{C}L] %7F: %E: %M"

in the my-http.conf

% docker run --rm -v ./my-httpd.conf:/usr/local/apache2/conf/httpd.conf httpd:2.4 

Got

[2025-07-23 07:08:07.212504 +0000] [mpm_event:notice] AH00489: Apache/2.4.64 (Unix) configured -- resuming normal operations
[2025-07-23 07:08:07.212554 +0000] [core:notice] AH00094: Command line: 'httpd -D FOREGROUND'

@mykola-elastic mykola-elastic added the enhancement New feature or request label Jul 23, 2025
@Alphayeeeet
Copy link
Contributor Author

@andrewkroh or @mykola-elastic Could you please run the CI tests? Thank you

@ishleenk17
Copy link
Member

/test

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 Should been fixed. Yould you please run CI tests again?

@mykola-elastic
Copy link
Contributor

/test

@Alphayeeeet
Copy link
Contributor Author

No idea whats wrong with CI. CI fails on system test (due to failed apt upgrade), however the change pipeline test complete without fail. Any advice is appreciated.

@Alphayeeeet
Copy link
Contributor Author

@mykola-elastic Can we try running CI? Maybe those buildkite-changes on main have fixed this issue?

@mykola-elastic
Copy link
Contributor

/test

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

Copy link

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 Could you please review and in follow-up approve and merge if applicable? Thank you

@Alphayeeeet
Copy link
Contributor Author

@ishleenk17 or @mykola-elastic can we merge this as it has been approved? Thanks

@mykola-elastic
Copy link
Contributor

@Alphayeeeet thanks for the heads up

@mykola-elastic mykola-elastic merged commit 7fd3cba into elastic:main Jul 29, 2025
9 checks passed
@elastic-vault-github-plugin-prod

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:apache Apache HTTP Server Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants