esm-tools / esm_tools

Simple Infrastructure for Earth System Simulations
https://esm-tools.github.io/
GNU General Public License v2.0
25 stars 12 forks source link

Failing CI after Release 6 #533

Closed pgierz closed 1 year ago

pgierz commented 2 years ago

Hello,

This issue is for discussion and does not really report a "problem", but maybe something we should be aware of. The following models and components are currently failing the compile tests. In some cases, it is because my CI dummy account doesn't have access everywhere, in others, I am a bit confused:

Full Setups

AWIESM

Standalone Components

ECHAM

PISM:

FESOM

General ideas

One idea. The following:

python/2020.2.902/lib/python3.7/subprocess.py", line 512, in run
119
    output=stdout, stderr=stderr)
120
subprocess.CalledProcessError: Command '['git', 'clone', '-b', 'oifs-deck', 'https://github.com/FESOM/fesom2.git', 'fesom-2.0-o']' returned non-zero exit status 128.
121
Error: Process completed with exit code 1.

is super unhelpful. It just says "ok, you got back first a 128 and then a 1". I have absolutely no way of seeing what didn't work without more specific info. I guess we should make esm-master spit out something more sensible.

pgierz commented 2 years ago

Here are also links to the various CI jobs:

AWIESM: https://github.com/esm-tools/esm_tools/actions/runs/1591738747

ECHAM: https://github.com/esm-tools/esm_tools/actions/runs/1591738748

PISM: https://github.com/esm-tools/esm_tools/actions/runs/1591738749

FESOM: https://github.com/esm-tools/esm_tools/runs/4559260911

mandresm commented 2 years ago

AWIESM

Many of the AWIESM versions use an ECHAM version that is retrieved from the gitlab.awi instead of the gitlab.dkrz, and when you manually install, you not only need to provide the password for awi (as you'll do for dkrz), but also your awi user (ESM-Tools only asks you for your dkrz user during installation and it never saves your awi-user).

pgierz commented 2 years ago

I think we should at some point entirely remove this esmtoolsrc file and get everything from env variables if (and only if) it does not exist. @seb-wahl will hopefully approve.

I need to test by hand, but I have the feeling that something during git clone messes up with tokens as a replacement for the password. The failing AWI-ESM test never happens to anyone in real life, but I think it would still be good to have it automatically.

nwieters commented 1 year ago

Hi @pgierz, was this only an issue on ollie? Or does it also apply to other machines and needs to kept open?

pgierz commented 1 year ago

Can be closed :-)