update sbetr to remove redundant print to log

[jinyuntang]

This avoids redundant output to log file.
Also suspend unit testing until pfunit fixed.
Update sbetr submodules to use ssh. Get ready to fix obsolete 3rd party libs

[BFB]

[github-actions]

PR Preview Action v1.4.8
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6740/
on branch gh-pages at 2024-11-08 17:17 UTC

[jinyuntang]

@ndkeen I checked the test ERS.f19_g16.I1850ELM.pm-cpu_gnu.elm-betr, it no longer produce the large log file. Please review. Thanks.

[ndkeen]

I see you removed the debug print. It kinda looks like other things were also changed? I might ask others involved to verify those changes are inert -- or would benefit from a different PR -- or at least update to the title.

[rljacob]

The current version of sbetr being used is 66260f4991d6. You should make a branch from that, fix just the debug statement, and then update the submodule to point to that branch. Otherwise this is a much bigger "update sbetr" PR.

[rljacob]

The other updates looks like its just testing, not code, so maybe this is fine as is.

[ndkeen]

Yes, I also see they are testing related -- I just don't know if it was intentional, related, important, etc. So if you think it's ok, I will approve.

[rljacob]

Wait for @peterdschwartz or @bishtgautam to approve.

[ndkeen]

Can we merge this?

[rljacob]

@ndkeen go ahead

[ndkeen]

Can we find someone more familiar with working with submodules?