You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "kevinjqliu (via GitHub)" <gi...@apache.org> on 2026/09/25 04:08:44 UTC
[PR] ci: enable RustFS stdout logging in docker compose [iceberg-rust]
kevinjqliu opened a new pull request, #3279:
URL: https://github.com/apache/iceberg-rust/pull/3279
## Which issue does this PR close?
Follow up to apache/iceberg#18246.
## What changes are included in this PR?
This sets `RUSTFS_OBS_LOG_STDOUT_ENABLED=true` on the object store, matching the other Iceberg repos (iceberg, iceberg-go, iceberg-cpp). It defaults to false and the image logs to `/logs`, so `docker logs` is empty otherwise, which makes CI failures hard to debug. Everything else here already lines up.
## Are these changes tested?
Tested with `docker compose config` and by running `rustfs/rustfs:1.0.0` with this setting: logs show up in `docker logs`.
## AI Disclosure
- Model: Claude Opus 5.5
- Platform/Tool: GitHub Copilot
- Human Oversight: [unknown - human to fill in]
- Prompt Summary: Align RustFS docker compose config with the other Iceberg repos
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] ci: standardize RustFS setup across Iceberg repos [iceberg-rust]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu merged PR #3279:
URL: https://github.com/apache/iceberg-rust/pull/3279
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] ci: standardize RustFS setup across Iceberg repos [iceberg-rust]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on code in PR #3279:
URL: https://github.com/apache/iceberg-rust/pull/3279#discussion_r4328595202
##########
dev/docker-compose.yaml:
##########
@@ -28,10 +28,12 @@ services:
# Object store - S3-compatible storage backed by RustFS (shared by all tests)
# =============================================================================
object-store:
- image: rustfs/rustfs:1.0.0
+ image: rustfs/rustfs:1.0.3
environment:
- RUSTFS_ACCESS_KEY=admin
- RUSTFS_SECRET_KEY=password
+ - RUSTFS_OBS_LOG_STDOUT_ENABLED=true
Review Comment:
show logs in CI
##########
dev/docker-compose.yaml:
##########
@@ -28,10 +28,12 @@ services:
# Object store - S3-compatible storage backed by RustFS (shared by all tests)
# =============================================================================
object-store:
- image: rustfs/rustfs:1.0.0
+ image: rustfs/rustfs:1.0.3
environment:
- RUSTFS_ACCESS_KEY=admin
- RUSTFS_SECRET_KEY=password
+ - RUSTFS_OBS_LOG_STDOUT_ENABLED=true
+ - RUSTFS_CONSOLE_ENABLE=false
Review Comment:
removes console port 9001, we dont need it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org