Closed forsyth2 closed 6 months ago
Strangely, this error does in fact occur on Chrysalis using Unified 1.9.2rc3. I know I tested that successfully though. Therefore, something has changed to affect past versions.
I will try on Perlmutter too.
@chengzhuzhang Interestingly, I don't actually see this error on Perlmutter. Since Perlmutter is the primary machine people use bundles on, I suppose we can mark this lower priority.
@forsyth2 and @chengzhuzhang, I've been keeping an eye on this. Is this something you expect to have diagnosed and fixed soon? @wlin7 found another bug in MPAS-Analysis, https://github.com/MPAS-Dev/MPAS-Analysis/issues/981, that will require another bug-fix release of E3SM-Unified. I could include a fix for this if need be.
@xylar I think we decided it's lower priority. I'm not quite sure what would cause this issue.
Okay, just wanted to check.
@forsyth2 I think there are two issues (https://github.com/E3SM-Project/zppy/discussions/544 and https://github.com/E3SM-Project/zppy/issues/546 )we should perhaps consider to figure out, if they are user errors or need a fix in zppy.
that will require another bug-fix release of E3SM-Unified
@xylar Do you have a timeline/expected deadline for this?
For reference, our prioritized list for zppy:
I have already tested 1.9.3rc1. It fixed the MPAS-Analysis issue it was meant to fix.
I would be willing to wait until early next week and then make a second and hopefully final rc but I don't want a process that snowballs and takes 2 months like 1.9.2 did (which was partly because of the holidays).
@chengzhuzhang How urgent are we deeming the above issues? I think it's unlikely they could all be fixed by next week.
@forsyth2 https://github.com/E3SM-Project/zppy/pull/424 (I need to run the test suite and make fixes) and https://github.com/E3SM-Project/zppy/pull/548 are ready to review. Please help review and integrate.
Have you had a chance to look at the other two (#544 and #546)? If not I will try to look into both and see if quick fixes are possible.
And I don't think https://github.com/E3SM-Project/zppy/issues/543 is a priority.
@xylar thanks for the heads-up. e3sm_diags will have a new release as well. I will work with @tomvothecoder to have the release candidate ready by this week.
424 (I need to run the test suite and make fixes)
and #548 are ready to review. Please help review and integrate.
I will test/code-review those tomorrow morning, I'm out-of-office this afternoon.
Have you had a chance to look at the other two (#544 and #546)? If not I will try to look into both and see if quick fixes are possible.
Not yet. I will try to take a look at those tomorrow too.
And I don't think #543 is a priority.
Sounds good.
Okay, I'll expect zppy and e3sm_diags RCs by sometime next week and I can make an E3SM-Unified rc2 after that.
In testing #424, I got the bundles test passing. I think it was a combination of two issues 1) cannot stat
error happens on two land variables, so I removed those, 2) it looks like at some point I accidentally updated the expected bundles files to be a non-merged PR's output, so I updated the expected files.
In any case, since bundles is passing, I'm closing this.
What happened?
I was running the "c. test final Unified" steps of https://e3sm-project.github.io/zppy/_build/html/main/dev_guide/release_testing.html, for Unified 1.9.2 (that is, testing what was actually released).
I see the following in the output file:
However, this appears to happen elsewhere without causing complete failures:
What machine were you running on?
Chrysalis
Environment
E3SM Unified 1.9.2 (zppy v2.3.0)
What command did you run?
Copy your cfg file
What jobs are failing?
What stack trace are you encountering?
No response