eurec4a / eurec4a-intake

Intake catalogue for EUREC4A field campaign datasets
17 stars 19 forks source link

fix weekly tests #136

Closed observingClouds closed 9 months ago

observingClouds commented 1 year ago

This PR does

d70-t commented 1 year ago

ipfsspec v0.4 is released. Would this be an option for a more permanent fix?

d70-t commented 1 year ago

I'm wondering why the current test fails. Line 1 of the install_ipfs script wants to run /bin/bash... Does the error mean, that /bin/bash is not found?

observingClouds commented 1 year ago

ipfsspec v0.4 is released. Would this be an option for a more permanent fix?

That solves this fix, yes! Probably we have to exclude the newest netCDF4 library again as well. That's what I'll test next. Thanks for doing the release!

observingClouds commented 1 year ago

I'm wondering why the current test fails. Line 1 of the install_ipfs script wants to run /bin/bash... Does the error mean, that /bin/bash is not found?

Seems like that was temporary

observingClouds commented 1 year ago

Unfortunately, this does not fix the weekly test entirely.

observingClouds commented 1 year ago

@d70-t any idea how to figure out which tests fail even in this case where the CI times out and the log file is not uploaded?

observingClouds commented 9 months ago

Now look at this. All checks have passed and even the weekly check passes. I started a plan with Piñata and pinned the IPFS content there for now as the retrieval has worked reliably from their service so far. Time will tell and I need to find some longer term funding for this. But I would vote for merging this now. This would make a good start for version 1.0.0