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

Fix #299: Updated Troubleshooting docs #300

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

HardikGoyal2003
Copy link
Member

@HardikGoyal2003 HardikGoyal2003 commented Apr 1, 2024

@HardikGoyal2003 HardikGoyal2003 marked this pull request as draft April 1, 2024 17:37
@@ -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)
Copy link
Member Author

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?

Copy link
Member

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

Copy link
Member Author

@HardikGoyal2003 HardikGoyal2003 Apr 1, 2024

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!

@HardikGoyal2003 HardikGoyal2003 marked this pull request as ready for review April 1, 2024 19:12
@HardikGoyal2003
Copy link
Member Author

@Ash-2k3 PTAL! Thanks!

Copy link
Member

@Ash-2k3 Ash-2k3 left a 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):
Copy link
Member

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.

Copy link
Member Author

Choose a reason for hiding this comment

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

@Ash-2k3 The error log I pasted here is from this comment. Can you please tell me if there is a way to reproduce these errors? I am not sure about the exact error logs I encountered as it has been a month and I can't recall them precisely.

Copy link
Member

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

Copy link
Member Author

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?

Copy link
Member

Choose a reason for hiding this comment

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

Sounds good, thanks!

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

@HardikGoyal2003
Copy link
Member Author

@Ash-2k3 PTAL! Thanks!

Copy link
Member

@Ash-2k3 Ash-2k3 left a 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

```
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)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> 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)

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

@Ash-2k3
Copy link
Member

Ash-2k3 commented Apr 2, 2024

Assigning @seanlip for a second pass, Thanks.

@HardikGoyal2003 HardikGoyal2003 removed their assignment Apr 2, 2024
Copy link
Member

@seanlip seanlip left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@seanlip seanlip merged commit 0faf305 into oppia:develop Apr 3, 2024
3 checks passed
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.

Add solution for subprocess.CalledProcessError in troubleshooting docs
3 participants