kevinjqliu opened a new pull request, #126: URL: https://github.com/apache/terraform-provider-iceberg/pull/126
Follow up to apache/iceberg#18246. This brings the RustFS setup here in line with the other Iceberg repos (iceberg, iceberg-rust, iceberg-go, iceberg-cpp) so they all behave the same way. - Healthcheck on `/health/ready` instead of `/health`. `/health` is just liveness and can return 200 before storage/IAM are ready, so the `aws-cli` bucket creation can race it. - Set `RUSTFS_OBS_LOG_STDOUT_ENABLED=true`. It defaults to false and the image logs to `/logs`, so `docker logs` is empty otherwise, which makes CI failures hard to debug. - Drop `RUSTFS_VOLUMES`, `RUSTFS_ADDRESS`, `RUSTFS_CONSOLE_ENABLE`, and `RUSTFS_CONSOLE_ADDRESS`. They're already the defaults in 1.0.0. Tested with `docker compose config` and by running `rustfs/rustfs:1.0.0` with this config: `/health/ready` returns 200 and 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
