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

There was a problem with Read the Docs while building your documentation. Please report this to us #4071

Closed
jorisvandenbossche opened this issue May 9, 2018 · 29 comments
Assignees
Labels
Needed: more information A reply from issue author is required Support Support question

Comments

@jorisvandenbossche
Copy link

Details

I am not fully sure why it is failing, I don't find something in the different log parts that fail, but it just stops after the "cat doc/source/conf.py" step.
And since it asked to report this, that's what I did.

@humitos humitos added Support Support question Needed: replication Bug replication is required labels May 9, 2018
@humitos
Copy link
Member

humitos commented May 9, 2018

Hi @jorisvandenbossche! I'm just adding some debug information to take a look later:

Sentry issues:

The build failed at when trying to save the BuildCommandResult over the API. It server returns 500 --I don't know why yet.

@pagreene
Copy link

pagreene commented May 11, 2018

We have been experiencing a potentially similar failure for several weeks:

As with the above, our automatic builds seem to fail directly after the conf step, however we have found that manually triggered builds continue and apparently succeed, except that they still throw the same error message.

@jorisvandenbossche
Copy link
Author

I also did a manual trigger of the build, but that failed as well (and also didn't update the docs online)

@begriffs
Copy link

Experiencing this as well. My builds have been failing for days starting at build 7213846.

@stsewd
Copy link
Member

stsewd commented May 22, 2018

Hi @begriffs, think your build is failing because your repo has a unicode branch https://github.com/citusdata/citus_docs/tree/metrics-%E2%88%82eco%E2%88%82e, rtd doesn't support unicode names for now (we are planning to support them soon #4052)

@begriffs
Copy link

@stsewd thanks for the quick response! That was exactly the issue. I deleted the offending remote branch and builds are back to normal.

@jorisvandenbossche
Copy link
Author

Any updates on this?
I just triggered a build for latest, and still have the same problem: https://readthedocs.org/projects/geopandas/builds/7289654/

(although in the meantime, there also has been a build that went further than the "cat doc/source/conf.py" step, but that one failed at the time due to an error in one of the our dependencies: https://readthedocs.org/projects/geopandas/builds/7259769/)

@humitos
Copy link
Member

humitos commented Jun 4, 2018

but that one failed at the time due to an error in one of the our dependencies: readthedocs.org/projects/geopandas/builds/7259769

Where you able to solve this? This one in particular doesn't seem to be a problem in RTD but in your project configuration.

In case you need to mock a C library, you can read the FAQ at http://docs.readthedocs.io/en/latest/faq.html#i-get-import-errors-on-libraries-that-depend-on-c-modules

@jorisvandenbossche
Copy link
Author

Where you able to solve this? This one in particular doesn't seem to be a problem in RTD but in your project configuration.

Yes, that was a problem on the Anaconda side, which has been fixed in the meantime. So therefore I triggered a new build to check if everything was OK now, but now having the original reported issue again (https://readthedocs.org/projects/geopandas/builds/7289654/, stopped at the "cat doc/source/conf.py" step)

After I commented here, I triggered yet another build. And now I also get the message "There was a problem with Read the Docs while building your documentation. Please report ...", but, it seems to be failing at another stage (now it just stops somewhere during building the docs): https://readthedocs.org/projects/geopandas/builds/7289777/

@jorisvandenbossche
Copy link
Author

Any update on this?
The latest triggered build of yesterday is still failing with "There was a problem with Read the Docs while building your documentation. Please report this to us with your build id (7442873)." (https://readthedocs.org/projects/geopandas/builds/7442873/)

@humitos
Copy link
Member

humitos commented Jul 10, 2018

@jorisvandenbossche hi! Your two latest builds passed. Do you still have this problem or you were able to solve it? Thanks!

@jorisvandenbossche
Copy link
Author

Yes, it seems to be working now. Thanks!

@jorisvandenbossche
Copy link
Author

The last build today seems to be failing again: https://readthedocs.org/projects/geopandas/builds/7486083/

@bgyori
Copy link

bgyori commented Jul 14, 2018

Following up on @pagreene's report from the indra project:

All the automatically triggered builds are still failing. In addition, the situation has gotten worse, since recently the manually triggered builds are failing too so we haven't been able to update our public documentation for a while. Any help would be very much appreciated!

Error
There was a problem with Read the Docs while building your documentation. Please report this to us with your build id (7486455).

git clone --recursive https://github.com/sorgerlab/indra.git .
git checkout --force origin/master
git clean -d -f -f
git branch -r
python3.5 -mvirtualenv --no-site-packages --no-download /home/docs/checkouts/readthedocs.org/user_builds/indra/envs/latest
python /home/docs/checkouts/readthedocs.org/user_builds/indra/envs/latest/bin/pip install --upgrade --cache-dir /home/docs/checkouts/readthedocs.org/user_builds/indra/.cache/pip Pygments==2.2.0 setuptools<40 docutils==0.13.1 mock==1.0.1 pillow==2.6.1 alabaster>=0.7,<0.8,!=0.7.5 commonmark==0.5.4 recommonmark==0.4.0 sphinx<1.8 sphinx-rtd-theme<0.5 readthedocs-sphinx-ext<0.6
python /home/docs/checkouts/readthedocs.org/user_builds/indra/envs/latest/bin/pip install --exists-action=w --cache-dir /home/docs/checkouts/readthedocs.org/user_builds/indra/.cache/pip -rdoc/requirements.txt
cat doc/conf.py

@stsewd
Copy link
Member

stsewd commented Jul 16, 2018

@jorisvandenbossche looks like a problem with memory limits, same for @bgyori 's project

@bgyori
Copy link

bgyori commented Jul 16, 2018

Thanks @stsewd, is this something we can solve by changing the project config? Or does this need to be solved on the server side?

@stsewd
Copy link
Member

stsewd commented Jul 16, 2018

@bgyori you can check https://docs.readthedocs.io/en/latest/guides/build-using-too-many-resources.html for some tips. If that doesn't solve the problem you need to ask a maintainer to increase the memory limits for your project.

@jorisvandenbossche
Copy link
Author

Ah, yes, we are probably hitting the 15 min limit (it says the (failed) build run for 14.7 min).

@humitos geopandas has quite some heavy dependencies to install, but still normally it doesn't take that long to create the environment locally.
Does readthedocs do any caching of the conda packages?

@humitos
Copy link
Member

humitos commented Jul 16, 2018

Does readthedocs do any caching of the conda packages?

Unfortunately, no. That's something that needs to be implemented: #3261

Does geopandas to install all of those dependencies to build the docs, or they are needed only to use geopandas? (I assume they are needed because you have the reference documentation also)

I double your time limit and triggered a new build. It should finish successfully. Let me know otherwise.

@jorisvandenbossche
Copy link
Author

I double your time limit and triggered a new build. It should finish successfully. Let me know otherwise.

Thanks for doing that!
However, now there is another error "Problem parsing YAML configuration. Invalid "requirements_file": path requirements_rtd.txt does not exist", but we didn't change anything in the yaml config file, and there is also no requirements_rtd.txt mentioned in there.

@jorisvandenbossche
Copy link
Author

Does geopandas to install all of those dependencies to build the docs, or they are needed only to use geopandas? (I assume they are needed because you have the reference documentation also)

Some of them are needed because we run some examples in the docs (running them itself does not take a lot of time), but the most heavy dependencies (eg gdal and all its dependencies) are not optional.

@stsewd
Copy link
Member

stsewd commented Jul 18, 2018

@jorisvandenbossche

However, now there is another error "Problem parsing YAML configuration. Invalid "requirements_file":

Sorry about that, it was a bug and it was fixed today, if you rebuild your docs, the error should be gone.

@jorisvandenbossche
Copy link
Author

if you rebuild your docs, the error should be gone.

It still seems to give the same error now.

@humitos
Copy link
Member

humitos commented Jul 18, 2018

@jorisvandenbossche the problem is that you have set the file requirements_rtd.txt as the requirements file to be installed under Admin -> Advanced Settings and that file doesn't exist in your repo. Remove it from there and build it again.

@jorisvandenbossche
Copy link
Author

Thanks for the quick answer!
I didn't change anything there, so until now the readthedocs.yaml was apparently overruling that.

@humitos
Copy link
Member

humitos commented Jul 18, 2018

Following up on @pagreene's report from the indra project:

Please, @bgyori open a new issue for this. It seems to be a different problem.

I imported this project in my local instance and the build is killed for a memory consumption.

Link this comment in the new issue and, if you can, please research a little about why it's consuming too much memory for the build. You can get some ideas from: https://docs.readthedocs.io/en/latest/guides/build-using-too-many-resources.html

Since the OOM is tricky to catch always properly, the message is sometimes not accurate.

@humitos humitos removed the Needed: replication Bug replication is required label Jul 18, 2018
@jorisvandenbossche
Copy link
Author

@humitos sorry, the build is still failing (now again the original error, after adjusting the advanced settings). Any pointers on what this is related to ? (Memory, time,...)

@humitos
Copy link
Member

humitos commented Aug 20, 2018

@jorisvandenbossche do you still have this issue?

We changed servers during the weekend and new ones are faster. So, I think you should be able to build without any problem now. If so, please close this issue.

I just triggered a build and it passed: https://readthedocs.org/projects/geopandas/builds/7662770/

@stsewd stsewd added the Needed: more information A reply from issue author is required label Aug 20, 2018
@jorisvandenbossche
Copy link
Author

Yes, it seems to work now. Thanks, closing then!

(back in July, I still saw some intermittent failures, but at least when re-triggering a build it sometimes passed. However, I am not sure it was related to a time-out, because when it passed, it was not close to the limit)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: more information A reply from issue author is required Support Support question
Projects
None yet
Development

No branches or pull requests

6 participants