-
Notifications
You must be signed in to change notification settings - Fork 25
Add submodule configuration #30
Comments
+1 we hit that currently and this is a blocker to us |
This adds a project feature that allows for a project to specify that they would like to skip submodule installation. Currently we are forcing all submodules to be checked out, so this fails on private submodules. Refs readthedocs/readthedocs-build#30
This adds a project feature that allows for a project to specify that they would like to skip submodule installation. Currently we are forcing all submodules to be checked out, so this fails on private submodules. Refs readthedocs/readthedocs-build#30
This adds a project feature that allows for a project to specify that they would like to skip submodule installation. Currently we are forcing all submodules to be checked out, so this fails on private submodules. Refs readthedocs/readthedocs-build#30
I think this is kind of related to readthedocs/readthedocs.org#3821 What about versioning the spec of the yaml? I have raised an issue for that readthedocs/readthedocs.org#3806 So, in that case the users without a For getting the submodules path we can parse the include: all I like the idea of the |
This adds a project feature that allows for a project to specify that they would like to skip submodule installation. Currently we are forcing all submodules to be checked out, so this fails on private submodules. Refs readthedocs/readthedocs-build#30
* Add temporary method for skipping submodule checkout This adds a project feature that allows for a project to specify that they would like to skip submodule installation. Currently we are forcing all submodules to be checked out, so this fails on private submodules. Refs readthedocs/readthedocs-build#30 * Also protect git clone * Lint fixes
Yeah, i think we need a @haubourg we have a stop gap measure in place for skipping all submodule checkout. What's your project? I can enable the feature if you don't want any submodules at all. |
@agjohnson Hi! our project is http://qwat.readthedocs.io/en/latest/. |
@haubourg I just added that flag in qwat project. Let me know if that worked for you. Thanks. |
@humitos that made the trick, many thanks ! |
@agjohnson @humitos can you also please disable submodules for us? https://readthedocs.com/dashboard/enway-gmbh-cleansquare-ros |
@bochen87 done! Let me know if it's OK :) |
@humitos works, thanks! :-) |
Are we going to support this on the v1 of the yaml file? Or maybe to do this only on the v2. |
Sounds like we should probably support it in the v1. Seems like a useful option. |
@humitos Could you disable submodule cloning on https://readthedocs.org/projects/firesim/ We have similar issues with submodules that are currently private (and also very large), but unnecessary for building our documentation. For now, I've made a branch with .gitmodules ripped out to get our documentation up, but that's not ideal. Thanks a bunch! |
@sagark done. It should skip the submodules now. |
Thanks, works for us now! |
Migrating this discussion to readthedocs/readthedocs.org#4464. We'll be implementing this feature there instead of this repo. |
@humitos Can you also disable submodules for https://github.com/keystone-enclave/keystone ? Dayeol |
@stsewd IIRC there is a setting to enable YAML v2 in some projects, right? If so, maybe this is a good opportunity to add |
I'm running into the same issue with a private submodule. However, I'm a complete newbie and just followed basic tutorial steps. My projects isn't very advanced and there could be hundreds of other things that have gone wrong, but it would be helpful to get the submodule problem out of the way to start fixing everything else that is potentially wrong - any advice would be appreciated. :-) |
@humitos yes we have submodules configuration working in v2, but the current schema will change soon, so I'm not sure if users want to keep an eye on the changes. And also the docs are not deployed yet. |
@stsewd I was thinking about them helping us to test our V2 config as "beta tester users". I don't think they will need to use many of the configs though and just probably be enough with the simple ones + the exclude submodules. I don't know, just an idea. |
That would be great to have test users. Basically, the config would be something similar to what we use on the rtd repo https://github.com/rtfd/readthedocs.org/blob/master/.readthedocs.yml version: 2
sphinx:
configuration: docs/conf.py
python:
requirements: requirements.txt Submodules are disabled by default in v2 |
I gladly help as a beta tester, @stsewd & @humitos, and try out what you propose. I probably would need a bit a guidance, though. I have now included a .readthedocs.yml based on @stsewd 's suggestion at the root of my directory (https://github.com/AdinaWagner/multimatch/blob/master/.readthedocs.yml). |
We need to activate a flag from our side to enable v2 for your project. |
sure - is there something I need to do at this point? |
Hi @AdinaWagner and @dayeol! I enabled YAML configuration V2 for your projects (
Hope it works! |
In case there is something wrong with this, open a new issue at https://github.com/rtfd/readthedocs.org/issues/new explaining this and linking this one there. |
Thanks, the docs built successfully! :-) |
Awesome! |
I'll let you know here when the schema change, here are some docs about the current implementation readthedocs/readthedocs.org#4451 |
Thx, @stsewd ! |
Hi, I added .readthedocs.yml and added version: 2 at the top, but still readthedocs is checking out all the submodules. Can you help? |
@dayeol can you please provide your project url? |
@stsewd Another problem is that SSL certificate is invalid, so the website shows red flag. |
@humitos believe you added the flag to the wrong project ( I think you have misconfigured the cname record, please see https://docs.readthedocs.io/en/latest/custom_domains.html
|
Please, open a new issue for this problem at https://github.com/rtfd/readthedocs.org/issues/new |
Added the flag to Hope it works now. |
Hi @stsewd, as discussed on Gitter, could you enable v2 config on my site https://docs.pantheon.pegasys.tech/ ? Thanks a lot. |
@NicolasMassart Done! |
Thanks I will test that and tell you. |
@NicolasMassart @AdinaWagner @dayeol Hi, we just deployed some new changes to our v2 schema, so your projects will need to update the configuration file. The main difference is in the What looks like this: python:
requirements: requirements.txt You'll need to change it to python:
install:
- requirements: requirements.txt You can see the full updated docs in https://github.com/rtfd/readthedocs.org/blob/329286f2f8c811c6bda37f664546de73945dfe37/docs/config-file/v2.rst or in the pull request readthedocs/readthedocs.org#4451 |
thanks a lot for letting me know @stsewd - I've implemented the change! |
Seems a number of projects are hitting issues with submodule clone steps, and this should be configurable in our config. Because git doesn't provide the mechanism for optional submodules, we can do something like:
as implicit default. And:
As include example.
^ is the exclude example.
Legacy projects implemented before this change would have an implicit configuration of:
It's not possible to include and exclude submodules, this is a validation error.
This would eventually alter the code that rtfd/readthedocs.org uses for submodule cloning, and would only selectively clone things? The changes there are likely moving to specific submodule clone commands.
project.Feature
, plus migration, to lock old users into the old behavior (all submodules are cloned)The text was updated successfully, but these errors were encountered: