-
Notifications
You must be signed in to change notification settings - Fork 99
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
Fix #299: Updated Troubleshooting docs #300
Conversation
HardikGoyal2003
commented
Apr 1, 2024
•
edited
Loading
edited
- This PR fix Add solution for subprocess.CalledProcessError in troubleshooting docs #299
- Added error and solution for that error.
Troubleshooting.md
Outdated
@@ -34,6 +34,11 @@ Here are some general troubleshooting tips for Oppia. The platform specific tips | |||
- [No Module functools\_lru\_cache](#no-module-functools_lru_cache) | |||
- [No Module appengine.api](#no-module-appengineapi) | |||
- [ModuleNotFoundError: No module named \_bz2](#modulenotfounderror-no-module-named-_bz2) | |||
- [Subprocess.CalledProcessError: Command 'yarn install --pure-lockfile' returned non-zero exit status 1](#subprocesscalledprocesserror-command-yarn-install-pure-lockfile-returned-non-zero-exit-status-1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@seanlip Please can you help with this, as this is not getting linked to the heading what am I missing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try #subprocesscalledprocesserror-command-yarn-install---pure-lockfile-returned-non-zero-exit-status-1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Ash-2k3 This works, Thanks!
@Ash-2k3 PTAL! Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @HardikGoyal2003, Left a comment, PTAL.
|
||
If you get error like this when running ``python -m scripts.start``: | ||
``` | ||
Traceback (most recent call last): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the traceback, can you add a few lines of preceding error logs as well, maybe around 6-7 lines.
Because if you see these discussions oppia/oppia#17425, oppia/oppia#18428, oppia/oppia#14814, these all have this same error but different reasons to why it occurred and thus different solutions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure what to do in this case, maybe apart from your changes, let's provide links to these issues that I mentioned and ask them to follow those, if your solution didn't work for them ? wdyt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Ash-2k3 After my solution ends I will put a note telling that this is very generalized solution, if this solution doesn't work well with folks so they can look the issues listed below!! how about this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
@Ash-2k3 PTAL! Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @HardikGoyal2003 , one suggestion mostly to fix grammar
Troubleshooting.md
Outdated
``` | ||
npm install yarn | ||
``` | ||
> Note :- This solution is generalized, if doesn't solves your problem kindly check similar discussions listed below [oppia/oppia#17425](https://github.com/oppia/oppia/discussions/17425), [oppia/oppia#18428](https://github.com/oppia/oppia/discussions/18428), [oppia/oppia#14814](https://github.com/oppia/oppia/discussions/14814) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
> Note :- This solution is generalized, if doesn't solves your problem kindly check similar discussions listed below [oppia/oppia#17425](https://github.com/oppia/oppia/discussions/17425), [oppia/oppia#18428](https://github.com/oppia/oppia/discussions/18428), [oppia/oppia#14814](https://github.com/oppia/oppia/discussions/14814) | |
> Note :- If the above solution doesn't work, please go through the following answered discussions and try the solution mentioned there. [oppia/oppia#17425](https://github.com/oppia/oppia/discussions/17425), [oppia/oppia#18428](https://github.com/oppia/oppia/discussions/18428), [oppia/oppia#14814](https://github.com/oppia/oppia/discussions/14814) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
Assigning @seanlip for a second pass, Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, LGTM