pangeo-forge / C-iTRACE-feedstock

Pangeo Forge feedstock for C-iTRACE.
https://pangeo-forge.org
Apache License 2.0
0 stars 0 forks source link

Upgrade to recipes 0.9.4, and use kerchunk to open inputs #5

Open cisaacstern opened 1 year ago

cisaacstern commented 1 year ago

@jordanplanders https://github.com/pangeo-forge/pangeo-forge-recipes/pull/383 was merged this week, so upgrading this feedstock's required pangeo-forge-recipes version may likely resolve the root issue #3 we've been fighting all along. I'll run a test from this PR to see if that's the case. 🤞

cisaacstern commented 1 year ago

/run citrace_tracer

cisaacstern commented 1 year ago

@jordanplanders the test errored because of a bug in the 0.9.3 release. I'm fixing that in https://github.com/pangeo-forge/pangeo-forge-recipes/pull/466, and will retry the test here once that fix makes it up to Pangeo Forge Cloud. Should be today.

jordanplanders commented 1 year ago

@cisaacstern Sounds good. I was surprised that it was still "in progress" this morning, but figured it might just be a lot of bytes to process. Thanks for your persistence!

cisaacstern commented 1 year ago

Actually still waiting for this bug fix to propagate up to Pangeo Forge Cloud, but going to make a few adjustments here in the meantime... I'll explain after committing.

cisaacstern commented 1 year ago

Two changes just made:

Now just waiting for the new version of the cloud application to build, after which I'll re-trigger the test here.

cisaacstern commented 1 year ago

Ok I think everything's fixed 🤞 so will re-run test now.

cisaacstern commented 1 year ago

/run citrace_tracer

pangeo-forge[bot] commented 1 year ago

The test failed, but I'm sure we can find out why!

Pangeo Forge maintainers are working diligently to provide public logs for contributors. That feature is not quite ready yet, however, so please reach out on this thread to a maintainer, and they'll help you diagnose the problem.

jordanplanders commented 1 year ago

@cisaacstern ah, the plot thickens! Let me know what I can probe.

cisaacstern commented 1 year ago

Thanks for the ping, @jordanplanders. I believe this is a versioning issue on the backend, which if I'm correct, should be solved by https://github.com/pangeo-forge/pangeo-forge-orchestrator/pull/210. I'll re-trigger the test once that goes through. As noted in the comments in that PR thread, this is certainly helping to highlight the necessity of some broader infrastructure upgrades we're working on (which should hopefully prevent or greatly minimize this type of problem in the future).

cisaacstern commented 1 year ago

Ok I think this problem is solved... re-testing this recipe now.

cisaacstern commented 1 year ago

/run citrace_tracer

cisaacstern commented 1 year ago

Just going to get out ahead of the upcoming automated failure comment and say... this is going to fail! 😆 Hoping https://github.com/pangeo-forge/pangeo-forge-orchestrator/pull/212 will fix the underlying issue (finally!).

pangeo-forge[bot] commented 1 year ago

The test failed, but I'm sure we can find out why!

Pangeo Forge maintainers are working diligently to provide public logs for contributors. That feature is not quite ready yet, however, so please reach out on this thread to a maintainer, and they'll help you diagnose the problem.

cisaacstern commented 1 year ago

/run citrace_tracer

cisaacstern commented 1 year ago

https://github.com/pangeo-forge/pangeo-forge-orchestrator/pull/212 did not fix this 😮‍💨 🤦 ... going to try another approach.

jordanplanders commented 1 year ago

@cisaacstern Oh gracious! As always, if there is something I can do (investigate something about the files? I'm not sure how to contribute at this point 😬), let me know! 🤞

cisaacstern commented 1 year ago

@jordanplanders thanks for asking. At this point this is totally on me to fix what has ended up being a far more tangled backend service issue than I'd anticipated. I'll keep truckin' on that and ping you again here when it's (actually!) fixed.