search

EarthWorksOrg / EarthWorks

Other
3 stars 3 forks source link

NVIDIA Problematic SVN certificate with repo #10

Closed gdicker1 closed 1 year ago

gdicker1 commented 1 year ago

Original issue: https://github.com/ESCOMP/EarthWorks/issues/2

Description by @cponder

This setup command

./manage_externals/checkout_externals -v gives these failure messages

svn checkout --quiet https://svn-ccsm-models.cgd.ucar.edu/tools/proc_atm/chem_proc/release_tags/chem_proc5_0_04 /lustre/fsw/devtech/hpc-devtech/cponder/EarthWorks/2023-03-08/CESM/components/cam/chem_proc

ERROR:root:Command '['svn', 'checkout', '--quiet', 'https://svn-ccsm-models.cgd.ucar.edu/tools/proc_atm/chem_proc/release_tags/chem_proc5_0_04', '/lustre/fsw/devtech/hpc-devtech/cponder/EarthWorks/2023-03-08/CESM/components/cam/chem_proc']' returned non-zero exit status 1. ERROR:root:Failed with output: svn: E170013: Unable to connect to a repository at URL 'https://svn-ccsm-models.cgd.ucar.edu/tools/proc_atm/chem_proc/release_tags/chem_proc5_0_04' svn: E230001: Server SSL certificate verification failed: issuer is not trusted

ERROR: In directory /lustre/fsw/devtech/hpc-devtech/cponder/EarthWorks/2023-03-08/CESM Process did not run successfully; returned status 1: svn checkout --quiet https://svn-ccsm-models.cgd.ucar.edu/tools/proc_atm/chem_proc/release_tags/chem_proc5_0_04 /lustre/fsw/devtech/hpc-devtech/cponder/EarthWorks/2023-03-08/CESM/components/cam/chem_proc See above for output from failed command.

cponder commented 1 year ago

I see all the same problems when I clone this repo. If you're not going to fix it, can you give me some instructions that will work?

cponder commented 1 year ago

Try updating the config-file in the repo. The clone-edit-clone doesn't work because of the inconsistency that it creates. You could do this on a branch and then I could test it that way.

gdicker1 commented 1 year ago

There's coming work to update the EarthWorkOrg/CAM to a newer tag from ESCOMP/CAM. For now you can try creating a branch based off EarthWorksOrg/CAM and then merging in a newer ESCOMP/CAM tag (or head of develop) to address this for yourself. If it goes well, I would more than welcome a PR in EarthWorksOrg/CAM that updates it.

cponder commented 1 year ago

Can you give me the commands to do this? I've manages to mess this up in the past. The Externals_CAM.cfg file isn't in this repo so there's some indirection to deal with.

supreethms1809 commented 1 year ago

@cponder Thanks for reporting this. Long term fix: Update Externals_CAM.cfg to use the git protocal and point chem proc to git repo. This has been done in the latest version of CESM beta tags and this will eventually gets merged into Earthworks as we update our CAM tags.

Short term fix:

Please let me know which is convenient for you.

cponder commented 1 year ago

@supreethms1809 -- yeah -- could you please create the branch? Also that way i can test the updated code to see if it has any issues before the merge happens.

supreethms1809 commented 1 year ago

@cponder I created a new branch for you to test. And here are some instructions to get it.

git clone git@github.com:EarthWorksOrg/EarthWorks.git
cd Earthworks
git checkout update_component_chem_proc
./manage_externals/checkout_externals -v

Please let me know how this turns out.

cponder commented 1 year ago

No errors in the checkout, thanks! Dunno if we should close this now, or wait until things get merged into the trunk.

supreethms1809 commented 1 year ago

@cponder thanks for testing it. Lets keep this open till we merge. We need to check if you can build and run the compsets without errors as expected. I’ll submit a PR for this change.

gdicker1 commented 1 year ago

I think this issue has been resolved or at least gone stale, so I'm closing it for now.