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

Add more troubleshooting entry #1590

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add more troubleshooting entry #1590

wants to merge 1 commit into from

Conversation

kapelo
Copy link

@kapelo kapelo commented Oct 4, 2024

This covers cases when a user session times out during E2E test run

Description

[description]

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

This covers cases when a user session times out during E2E test run
@latin-panda
Copy link
Contributor

latin-panda commented Oct 7, 2024

Lorena and Tatiana are already reviewers, and can provide the best advice. So I will remove myself 🙂

@latin-panda latin-panda removed their request for review October 7, 2024 07:53
Copy link
Contributor

@tatilepizs tatilepizs left a comment

Choose a reason for hiding this comment

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

Thank you for being proactive about improving the documentation ⭐

We have to be very careful with those commands because they will delete everything, so can you please add more details about what will happen if someone runs them?

For example, you can explain that the error might happen because some data might be causing issues or because having a large number of images and volumes might crash docker depending on the resources assigned and that, as a solution, we recommend cleaning all the docker data to be able to start fresh and clean, but that is important to note that those commands will delete everything, and the next time that the e2e tests are run it might take a little longer because all the images need to be downloaded again.

@Benmuiruri
Copy link
Contributor

Typically for docs PR we need one reviewer from QA or a software developer for the technical part.

Sometimes you may include @esthermmoturi if perhaps you have a question about which section of the docs the content should go, or the language of the content.

Since you have Lorena and Tatiana, I'll remove myself as a reviewer not to block you.

@Benmuiruri Benmuiruri removed their request for review October 8, 2024 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants