Closed fraser-langton closed 3 months ago
Found a fix, just os.environ["TC_HOST"] = "localhost"
It took me longer than it should've to find the fix because all I could see was Waiting to be ready...
- it doesn't show the exception until max retries is done and with that at 120 as a default I never waited the full amount because I had no indication of how many tries it was doing. And also maybe should log the errors each time it happens?
Where I found the fix https://github.com/testcontainers/testcontainers-python/issues/108#issuecomment-1540584987
And also maybe should log the errors each time it happens?
Hang on... in waiting_utils.py it does log, but setup_logger enforces logging config, as a package this should not be happening? Logging config should be up to application not the package
I'm slightly confused about why the TC_HOST fix was necessary despite the fix in #427
@alexanderankin version? I had 3.7.1
The v3 line is from 2022/jan 2023, the first release of v4 was this year with all new maintainers of the project
@fraser-langton if you run into difficulties with an up-to-date version, please open a new issue.
@alexanderankin what about changing the usage of setup_logger which made this issue harder to debug, worthwhile fixing? Make a different issue?
would you consider that breaking? or any ideas about how to soften the breaking of that change (config flag :((( ?) - I put it on my v5 list #631
Describe the bug
Cannot do the most basic use case
To Reproduce
I copied the example from the docs word for word
Runtime environment
Windows 11 + Python 3.8.10
systeminfo
docker info
freeze