Skip to content

Better logging from the entrypoint and apache - #6

Merged
crwood merged 5 commits into
mainfrom
better-logs
Sep 16, 2026
Merged

crwood merged 5 commits into
mainfrom
better-logs

Conversation

@btlogy

@btlogy btlogy commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

Part 1 of #5

This change tries to present the logs generated by entrypoint,sh and apache2 (error and access) in a similar format than the supervisor in the docker logs:

suitecrm-1  | 2026-09-11 21:42:44,152 INFO entrypoint: Initialization process started
suitecrm-1  | 2026-09-11 21:42:44,529 INFO entrypoint: Version 7.15.1 detected
suitecrm-1  | 2026-09-11 21:42:44,530 INFO entrypoint: Nothing to upgrade
suitecrm-1  | 2026-09-11 21:42:44,723 INFO Set uid to user 0 succeeded
suitecrm-1  | 2026-09-11 21:42:44,725 INFO supervisord started with pid 1
suitecrm-1  | 2026-09-11 21:42:45,731 INFO spawned: 'apache' with pid 20
suitecrm-1  | 2026-09-11 21:42:45,738 INFO spawned: 'cron' with pid 21
suitecrm-1  | 2026-09-11 21:42:45.824244 notice apache2/mpm_prefork: AH00163: Apache/2.4.68 (Debian) PHP/8.3.33 configured -- resuming normal operations
suitecrm-1  | 2026-09-11 21:42:45.824287 notice apache2/core: AH00094: Command line: 'apache2 -D FOREGROUND'
suitecrm-1  | 2026-09-11 21:42:46,826 INFO success: apache entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
suitecrm-1  | 2026-09-11 21:42:46,826 INFO success: cron entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
suitecrm-1  | 2026-09-11 21:42:49.366307 info apache2/access: [localhost:-] 127.0.0.1 - - "GET /index.php HTTP/1.1" 301 - "-" "curl/7.88.1"
suitecrm-1  | 2026-09-11 21:42:49.992451 info apache2/access: [localhost:-] 127.0.0.1 - - "GET /index.php?action=Login&module=Users HTTP/1.1" 200 12005 "-" "curl/7.88.1"

This change adds a new log file in the state directory of SuiteCRM, but at least, it will be rotated: access.log.X
I'm working on an other PR to collect all log files, including this new one, in a dedicated folder.

Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
@btlogy btlogy self-assigned this Sep 11, 2026
@btlogy
btlogy requested a review from crwood September 11, 2026 21:48
Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
@btlogy

btlogy commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

@chris: what do you think about this change?

@btlogy
btlogy marked this pull request as ready for review September 11, 2026 22:03
Comment thread src/apache.conf Outdated
Comment thread src/entrypoint.sh Outdated
Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
@btlogy
btlogy requested a review from crwood September 16, 2026 10:17

@crwood crwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@crwood
crwood merged commit b0b9c4d into main Sep 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants