Skip to content

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

Merged
mykola-elastic merged 6 commits intoelastic:mainfrom
Alphayeeeet:allow-apache-timezone-offset
Jul 29, 2025
Merged

[Apache] Add support for apache ISO timestamp-format in errorlog#14647
mykola-elastic merged 6 commits intoelastic:mainfrom
Alphayeeeet:allow-apache-timezone-offset

Conversation

@Alphayeeeet
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@ishleenk17
Copy link
Copy Markdown
Member

/test

@Alphayeeeet
Copy link
Copy Markdown
Contributor Author

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

@mykola-elastic
Copy link
Copy Markdown
Contributor

/test

@Alphayeeeet
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@mykola-elastic
Copy link
Copy Markdown
Contributor

/test

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

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@elastic-sonarqube
Copy link
Copy Markdown

@Alphayeeeet
Copy link
Copy Markdown
Contributor Author

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

@Alphayeeeet
Copy link
Copy Markdown
Contributor Author

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

@mykola-elastic
Copy link
Copy Markdown
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
Copy link
Copy Markdown

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

@Alphayeeeet Alphayeeeet deleted the allow-apache-timezone-offset branch October 9, 2025 11:22
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