Latest container does not work

Open-EO / openeo-web-editor

An interactive and easy to use web-based editor for the OpenEO API.

https://editor.openeo.org

Apache License 2.0

12 stars 17 forks source link

Latest container does not work #287

Closed danielFlemstrom closed 1 year ago

danielFlemstrom commented 1 year ago

mundialis/openeo-web-editor:latest generates an error for the process (pseudo form) loadcollection->save (geotiff)

None of the available hashes between the one used in https://editor.openeo.org(0.11.4) and 78257122a486ca19ca46c1944432d9d9afa0532c gives that error so i am not sure why :latestcomplains..

Is there a way to see which commit hash :latest corresponds to?

m-mohr commented 1 year ago

This is weird, doesn't look like it should be docker related. It basically says it can't access the GeoTiff file, which can have a couple of reasons:

General connectivity issues of the local computer
Network access through docker doesn't work correctly
CORS issues on the server side
Some dependency of the Web Editor has a malfunction
A bug in the Web Editor itself.
...

I'm not a Docker user myself and the Docker image is maintained by @mmacata so I can't really look into the issue.

Does it work in the most recent deployed version at editor.openeo.org?

danielFlemstrom commented 1 year ago

Yes and all check-ins after that version. So with a bit of luck, re-generating the :latest would fix it. I used the same EO-Server and the same process tree to test.

mmacata commented 1 year ago

Hi @danielFlemstrom, the latest tag is updated for every release which was v0.11.0 ~ six month ago. So the same error should appear for mundialis/openeo-web-editor:v0.11.0 and should already have been there this long. So to be sure what's inside, it might makes sense to pin version either to a commit hash or to a released version.

danielFlemstrom commented 1 year ago

OK, my bad. I assumed latest was the automatic build of the latest commit, or at least labelled commit. Maybe tag the "latest"s corresponding commit with latest would help as well ? Anyway it seems all after 0.11.4 works fine when i pulled using the commit hash. @mmacata just say if you want me to test anything!

m-mohr commented 1 year ago

The issue you are seeing is likely fixed by #267. And it's probably time for a 0.12.0 release :)

mmacata commented 1 year ago

We pass the decision to what latest should be to a github action called docker/metadata-action and simply use the default behaviour of it.

So I also think that it would be time for a new release (:

m-mohr commented 1 year ago

Fixed the remaining bugs for 0.12.0 and released it. The container should be updated in some minutes.

mmacata commented 1 year ago

mundialis/openeo-web-editor:latest LAST PUSHED 19 hours ago :+1: and hash is identical to v0.12.0