search

earthobservations / wetterdienst

Open weather data for humans.
https://wetterdienst.readthedocs.io/
MIT License
358 stars 55 forks source link

CI: Test suite fails every other time #730

Closed amotl closed 2 years ago

amotl commented 2 years ago

Hi there,

while submitting #729 and #731, I found CI croaks like [1]:

Run if [ "$RUNNER_OS" != "Windows" ]; then
The virtual environment found in /home/runner/work/wetterdienst/wetterdienst/.venv seems to be broken.
Recreating virtualenv wetterdienst in /home/runner/work/wetterdienst/wetterdienst/.venv
Command not found: poe
Error: Process completed with exit code 1.

Do you know how to fix it, @gutzbenj?

With kind regards, Andreas.

[1] https://github.com/earthobservations/wetterdienst/actions/runs/3120098807/jobs/5060465902 [2] https://github.com/earthobservations/wetterdienst/actions/runs/3120160432/jobs/5060548390

amotl commented 2 years ago

Observation

It looks like there is a caching issue on CI. With 2c9e15ed3, the tests proceeded, until it hit a test anomaly, which had to be fixed with c1a7133f7. After that, it failed again with the same error. 72b06d46d finally makes the test suite proceed to success on CI [1].

[1] https://github.com/earthobservations/wetterdienst/actions/runs/3122893943

Thoughts

The virtual environment found in /home/runner/work/wetterdienst/wetterdienst/.venv seems to be broken.

I think when caching the .venv directory, such errors are predictable.

https://github.com/earthobservations/wetterdienst/blob/4a4a9cf3ecfc4f95226c4accf6856734722e4748/.github/workflows/tests.yml#L40-L48