Skip to content

Redirect Docker logs to stdout/stderr and add ant build target#417

Open
sly4 wants to merge 22 commits intojeffpiazza:masterfrom
sly4:feature/docker-logging-stdout
Open

Redirect Docker logs to stdout/stderr and add ant build target#417
sly4 wants to merge 22 commits intojeffpiazza:masterfrom
sly4:feature/docker-logging-stdout

Conversation

@sly4
Copy link
Contributor

@sly4 sly4 commented Feb 15, 2026

This PR updates the Docker configuration to adhere to container logging best practices and simplifies the build process.

  • Logging: Configures Nginx and PHP-FPM to write logs to /dev/stdout and /dev/stderr.
    • This enables docker logs to capture application output, which was previously hidden in internal log files.
    • It resolves potential permission issues when running containers with read-only filesystems or specific volume mounts (e.g., NFS) where writing to /var/log might fail.
  • Build: Adds a docker-build target to build.xml to streamline creating the container image.

jeffpiazza and others added 22 commits December 11, 2025 17:06
… kiosk dashboard config to reflect slideshow's settings.
* Fix IllegalFormatPrecisionException when passing double to String.format

* Fix off-by-10 error in reported times from SuperTimer II
Undo earlier commit that tried to fill in missing decimal point for Super Timer II.
In javascript timer interface, recognize "cease" responses and stop polling.
- Fix timer-in-brower -> timer-in-browser in README.md
- Fix GRPM -> GPRM in Advanced Database Set-Up doc
…ents pages.

Check for missing photo paths in check-in card renderer.
This fixes URLs when derbynet-server is hosted on one server and served by a different (proxy) server (NGINX).
- Populate sortorder correctly even for the first scene after all deleted.
- Repopulate g_all_scenes after assigning a kiosk page, so the in-browser state matches the database.
- Confirm before deleting a scene.
to ensure current parameters are reflected correctly.
clear kiosk div when a scene has an unspecified page for a kiosk.
@jeffpiazza
Copy link
Owner

Could you please rebase to current mainline? This seems to have been branched a while ago.

From what I can tell, just 6c290b0 and c54afd3 are the intended changes.

I'm not at all deep on docker, but I'm wondering if there shouldn't be some provision for building with --platform options and maybe others. Also, I'm still not 100% clear whether 'docker build' or 'docker buildx build' is the preferred expression.

@jamestford
Copy link
Contributor

Given that DerbyNet is used on both AMD64 and ARM64 platforms I would also recommend leveraging the docker buildx build with a --platform option to properly support the multi-arch builds.

Recommendation: Add a configurable docker.platform property — like the other properties at the top of the file, defaulting to something sensible:

There is also a difference between the docker-deps vs the dist.debian dependency chain:
Looking at the dependency chains:
docker-deps depends on: generated, timer-in-browser
dist.debian depends on: generated, timer-jar, timer-in-browser, docs-dist
So docker-deps is missing timer-jar and docs-dist compared to the Debian package. So the timer jar and full docs are built for the Debian package but never make it into the Docker image because the Dockerfile doesn't copy them.

@sly4
Copy link
Contributor Author

sly4 commented Feb 17, 2026 via email

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.

5 participants

Comments