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

docs: improvements to the "troubleshooting" page #1035

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

Conversation

regisb
Copy link
Contributor

@regisb regisb commented Apr 9, 2024

I tried to remove as many "you" pronouns as possible. Also, clarify instructions on the "images build" resource issue.

I tried to remove as many "you" pronouns as possible. Also, clarify
instructions on the "images build" resource issue.
@regisb regisb force-pushed the regisb/troubleshooting-is-not-about-you branch from 736a85a to ca6508a Compare April 9, 2024 09:46
@DawoudSheraz DawoudSheraz self-requested a review April 16, 2024 07:17
@@ -149,11 +149,11 @@ Launch your Open edX platform again with ``tutor local launch``. You should obse
...
Build failed running pavelib.servers.lms: Subprocess return code: 1`"

This might occur when you run a ``paver`` command. ``/dev/null`` eats the actual error, so you will have to run the command manually. Run ``tutor dev shell lms`` (or ``tutor dev shell cms``) to open a bash session and then::
This might occur when running a ``paver`` command. ``/dev/null`` eats the actual error, so we have to run the command manuallyto figure out the actual error. Run ``tutor dev shell lms`` (or ``tutor dev shell cms``) to open a bash session and then::
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: manuallyto --> manually to


All build commands should now make use of the newly configured builder. To later revert to the default builder, run ``docker buildx use default``.
All build commands should now make use of the newly configured builder. To later revert to the default builder, run ``docker buildx use default``.
Copy link
Contributor

Choose a reason for hiding this comment

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

As mentioned on #1005 (comment), to switch back to default builder, docker context use default should be used.

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

Successfully merging this pull request may close these issues.

None yet

2 participants