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

sync: prepare troubleshooting guide for Too many open files error #871

Closed
pared opened this issue Dec 17, 2019 · 9 comments · Fixed by #875
Closed

sync: prepare troubleshooting guide for Too many open files error #871

pared opened this issue Dec 17, 2019 · 9 comments · Fixed by #875
Assignees
Labels
A: docs Area: user documentation (gatsby-theme-iterative)

Comments

@pared
Copy link
Contributor

pared commented Dec 17, 2019

Related:
iterative/dvc#2866
iterative/dvc#2473

Need to prepare short information about why Too many open files happens and how to solve it, as this information is too long for exception message.

After this issue is solved, need to improve log message to include a link to the explanation

@pared pared self-assigned this Dec 17, 2019
@efiop efiop changed the title Prepar troubleshooting guide for Too many open files error Prepare troubleshooting guide for Too many open files error Dec 17, 2019
@shcheklein shcheklein added A: docs Area: user documentation (gatsby-theme-iterative) user-guide labels Dec 17, 2019
@shcheklein shcheklein changed the title Prepare troubleshooting guide for Too many open files error prepare troubleshooting guide for Too many open files error Dec 17, 2019
@jorgeorpinel
Copy link
Contributor

Where would you put this info? If it will fit in a single paragraph I recommend using existing docs, probably some of the command references. It can be repeated in a few ones wherever it's appropriate (e.g. remote and push perhaps)

@pared
Copy link
Contributor Author

pared commented Dec 18, 2019

@jorgeorpinel I guess push and pull are the most "threatened" operations, maybe there?

@jorgeorpinel
Copy link
Contributor

Sounds good. Maybe also checkout since it can include pull. Consider adding the note in the remote index page too. Thanks

@pared
Copy link
Contributor Author

pared commented Dec 19, 2019

@jorgeorpinel remote seems to be about remote command, not remote as entity per se, so I am not sure we should put some note there, what do you think?

@shcheklein
Copy link
Member

@jorgeorpinel @pared I think with these errors that are edge cases in a regular workflow we should docs "event-driven". Event happens -> link -> troubleshoot section that help to understand and overcome. We don't need to put a lot of links and duplicate this around. It is not scalable.

I suggest we actually introduce error.dvc.org/{code} shortcut and use then in DVC. They should redirect to specific Troubleshooting sections.

@jorgeorpinel
Copy link
Contributor

Event happens -> link -> troubleshoot section that help to understand and overcome. ...
I suggest we actually introduce error.dvc.org/{code} shortcut and use then in DVC.

Great idea!

So no links at all in this case even from pull or push cmd refs?

@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Dec 19, 2019

remote seems to be about remote command

@pared actually https://dvc.org/doc/command-reference/remote is an index page to list the sub commands and we also use it to explain what DVC remotes are – which by the way may be a bad thing and we should introduce a user guide about DVC remotes instead... Not sure. But anyway, sounds like we don't want links from any other docs.

@shcheklein
Copy link
Member

Agreed to create an issue in the DVC core project and link it in the message.

@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Jan 6, 2020

I think this is still open, will be closed by an ongoing PR (#875).

@jorgeorpinel jorgeorpinel reopened this Jan 6, 2020
@jorgeorpinel jorgeorpinel changed the title prepare troubleshooting guide for Too many open files error sync: prepare troubleshooting guide for Too many open files error Jan 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants