-
Notifications
You must be signed in to change notification settings - Fork 95
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
Submodule pointer for post_gtg.fd and/or UPP_GTG repo doesn't exist when cloning the develop branch of UPP #425
Comments
GTG repo was changed from a manage_externals to a submodule recently for GFSv16, it looks like this was just merged into develop but I have not been able to test. GTG repo is a private protected repo of source code. I don't think we want UFS to clone it as it's meant just for operational implementation. @JeffBeck-NOAA - just to confirm, you're not really try to access the GTG code, right? It's just that the UPP clone fails because you have the submodule flag? If that's the case, @WenMeng-NOAA - we might need to have a way to control which submodules are cloned. |
@fossell, yep, I'm just trying to clone the UPP develop branch, not trying to access the GTG code, so it's failing when trying to access the submodule (the SRW App does a recursive clone). |
@JeffBeck-NOAA - thanks for confirming. It looks like there will be an access issue. Is it possible to modify the clone for UPP repo to not be recursive as a short term fix while we assess the best path forward? |
@fossell, actually, we're not using the HEAD of the UPP develop branch in the SRW App just yet. We're still on an older hash that works, but I was just testing things out on all the develop branches briefly. So, we should be OK for now! |
@JeffBeck-NOAA - sounds good, thanks for bringing this to our attention! We'll work on finding a solution. |
@WenMeng-NOAA - we'll need to address this submodule issue as it may cause issues for other users too. I think the quickest things we can recommend is just to have users clone without using recurse-submodules. For future more permanent and sustainable fix, I'd like to revisit the potential of using cmake options to control which external repos are cloned. Both Sam and Rahul mentioned and recommended this. If you agree, let's touch base with them and see if we can expedite this development. Another note - the GTG Access page on the wiki needs to be updated as well. |
@fossell Thanks for suggestions on this issue. I agree with you. Right now post_gtg.fd is only submodule in UPP. The solution would be non-recursively cloning for UPP code checking out. |
@WenMeng-NOAA - we had removed any submodule references from procedures in our user documentation a couple weeks ago so we should be ok from that angle. I can reach out to Sam and/or Rahul after the holidays and see what it might take to get that underway. We might also be able to ask Mike on our DTC UPP team as he might have experience with the cmake options as well. |
@fossell An announcement for checking out general UPP code was just sent out. I also updated the wiki GTG Access. Please let me know if DTC have more solutions/suggestions. Thanks! |
@fossell, @WenMeng-NOAA, just a quick update on this issue in another location: I have a series of PRs open in ufs-weather-model and their related submodules to add SPP to FV3. When I issue a "git submodule --init --update --recursive" to update the pointers of the submodules in the ufs-weather-model PR, I also get the error about https://github.com/NCAR/UPP_GTG not being found. In order to avoid this, I have to implement this rather complicated git command to avoid trying to update the UPP pointer: git -c submodule."upp".update=none submodule update --init --force --remote --recursive |
@JeffBeck-NOAA Could you provide me your UFS PR location? We will work with UFS team for adding the issue since UPP is a submodule of FV3 and GTG is a submodule of UPP. Thanks! |
@WenMeng-NOAA, sure! My ufs-weather-model PR is here. |
Thanks @DomHeinzeller for providing the solution of skipping gtg code checkout. I submitted his fix in PR #427. |
@WenMeng-NOAA, unfortunately, I still get an error: Checking status of externals: upp, |
@JeffBeck-NOAA Did you test in your ufs-weather-model clone or UPP clone? |
Sorry, @WenMeng-NOAA, my mistake. It is working now! |
@JeffBeck-NOAA Great. Thanks for testing. |
…es_cube (NOAA-EMC#425) in make_ics and make_lbcs.
When cloning the UPP develop branch with manage_externals through the SRW App, the pointer for post_gtg.fd and/or the UPP_GTG repo is not found.
Error from manage_externals:
ERROR:root:Failed with output:
Submodule 'sorc/post_gtg.fd' (https://github.com/NCAR/UPP_GTG) registered for path 'sorc/post_gtg.fd'
remote: Repository not found.
fatal: repository 'https://github.com/NCAR/UPP_GTG/' not found
@WenMeng-NOAA @hertneky @kayeekayee @fossell
The text was updated successfully, but these errors were encountered: