Skip to content

Add real on_failure page#6784

Merged
kiview merged 4 commits intomainfrom
on-failure-docs
Apr 11, 2023
Merged

Add real on_failure page#6784
kiview merged 4 commits intomainfrom
on-failure-docs

Conversation

@kiview
Copy link
Member

@kiview kiview commented Mar 16, 2023

When Testcontainers fails to automatically discover a supported container runtime environments, it will log a human readable error message with a link, pointing to https://www.testcontainers.org/supported_docker_environment/. In order to give new users with less Docker experience more suitable hints on how to fix their issue, we intend to explore ways to make the documentation more streamlined.

Copy link

@mtremsal mtremsal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions to clean-up the language a bit.

docs/_redirects Outdated
/usage/neo4j_container.html /modules/databases/neo4j/
/compatibility.html /supported_docker_environment/
/on_failure.html /supported_docker_environment/
/on_failure.html /on_failure

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's have a more descriptive url. e.g. error_missing_container_runtime_environment.html


If you ended up on this page,
it seems that either Testcontainers was not able to find a supported container runtime in your environment,
or you found page while browsing the documentation in search of information on dealing with errors regarding Testcontainers' environment discovery mechanism.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or you found this page while searching for information to deal with errors regarding the environment discovery mechanism of Testcontainers.

it seems that either Testcontainers was not able to find a supported container runtime in your environment,
or you found page while browsing the documentation in search of information on dealing with errors regarding Testcontainers' environment discovery mechanism.

Testcontainers requires a supported container runtime environment to be present in order to function.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testcontainers requires a supported container runtime environment to be present in order to execute containers.

or you found page while browsing the documentation in search of information on dealing with errors regarding Testcontainers' environment discovery mechanism.

Testcontainers requires a supported container runtime environment to be present in order to function.
Here is a list of supported container runtime environments, that can get you up to speed quickly:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is a list of supported container runtime environments:

* [Docker Engine on Linux](https://docs.docker.com/engine/install/)
* [Testcontainers Cloud](https://www.testcontainers.cloud)

For more extensive information on supported container runtime environments, and how to configure container runtime environments that are not supported by default,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For more extensive information on supported container runtime environments, as well as known limitations of alternative container runtime environments, please refer to...

@shelajev shelajev marked this pull request as ready for review April 10, 2023 16:51
@shelajev shelajev requested a review from a team April 10, 2023 16:51
Copy link

@mtremsal mtremsal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM -- thanks for the changes 👍

@kiview
Copy link
Member Author

kiview commented Apr 11, 2023

Thanks for the review @mtremsal and thanks for applying the changes @shelajev 👍

@kiview kiview merged commit 3d21747 into main Apr 11, 2023
@kiview kiview deleted the on-failure-docs branch April 11, 2023 11:25
@kiview kiview added this to the next milestone Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants