Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DateTimeParseException when trying to launch container #3054

Closed
acanda opened this issue Jul 27, 2020 · 2 comments · Fixed by #3055
Closed

DateTimeParseException when trying to launch container #3054

acanda opened this issue Jul 27, 2020 · 2 comments · Fixed by #3055

Comments

@acanda
Copy link
Contributor

acanda commented Jul 27, 2020

When Testcontainers is refreshing its local image cache, it is using the command "inspect image" and parses the timestamp of the attribute "created". If this timestamp contains a time zone offset, then the following exception is thrown and Testcontainers fails to launch the container.

org.testcontainers.containers.ContainerLaunchException: Container startup failed
Caused by: org.testcontainers.containers.ContainerFetchException: Can't get Docker image: RemoteDockerImage(imageName=docker-registry.example.org/some/image:1.0, imagePullPolicy=DefaultPullPolicy())
Caused by: java.time.format.DateTimeParseException: Text '2020-07-24T07:17:51.365190246+02:00' could not be parsed at index 29

This happend with Docker version 18.06.3 on Linux.

@stale
Copy link

stale bot commented Nov 1, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you believe this is a mistake, please reply to this comment to keep it open. If there isn't one already, a PR to fix or at least reproduce the problem in a test case will always help us get back on track to tackle this.

@stale stale bot added the stale label Nov 1, 2020
@acanda
Copy link
Contributor Author

acanda commented Nov 2, 2020

Unstale. Does the Pull Request #3055 need anything else? We are running a fork of TestContainers with this PR applied in production since I opened this issue and we did not have any problems with it so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants