[feature request] Improve integration test reliance on docker

[kevinjqliu]

Feature Request / Improvement

It would be nice to eliminate the sleep 10 and instead use health checks for the underlying containers.

iceberg-python/Makefile

Line 43 in 188e2c4

sleep 10

The sleep here is a proxy for waiting for REST/HMS services to be available.

[AronsonDan]

@kevinjqliu Did you guys consider using testcontainers?

[kevinjqliu]

@AronsonDan I have not heard of this before. I'll take a look, thank you!

[AronsonDan]

Would love to give a hand setting up testconteiners using pytest

[kevinjqliu]

@AronsonDan please do! Feel free to tag me or reach out with any questions

[github-actions]

This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.

[github-actions]

This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.

[kevinjqliu]

resolved by #2540