geoserver-docker icon indicating copy to clipboard operation
geoserver-docker copied to clipboard

Upgrade pinned version of pip3

Open lilatomic opened this issue 3 years ago • 5 comments

pip3 is currently pinned to version 9.0.3 here, and has been for about 4 years. We should probably unpin it, or at least upgrade it. I'm happy to submit a patch for whichever we prefer.

lilatomic avatar May 04 '22 23:05 lilatomic

I do agree.

There has been a problem back in the days: https://github.com/GeoNode/geoserver-docker/commit/21b14c2ca15b53c882759729833f172b87954278 with an upgraded version. before unpinning it we should test this carefully.

t-book avatar May 05 '22 07:05 t-book

We could implement GitHub actions to run a docker build and check for errors in advance.

frafra avatar May 05 '22 07:05 frafra

Right. Seems like that was a fix for #20 , where the problem was pip 10.0.0 . Seems that pip 10.0.0 changed its internals; but, since it was installed by apt but then upgraded by us, that posed problems (pip warns you about doing this, although this is the first time I've seen that actually causing issues).

I tried just removing the line to upgrade pip and use the version provided by the package manager 20.3.4. It looks like everything installed fine. In general the one installed with the package manager is a bit old (about 1.3 years) but regarded as stable.

What sorts of acceptance tests should I run to qualify the image?

Using GitHub actions to verify that it still builds is a neat idea.

lilatomic avatar May 06 '22 02:05 lilatomic

  1. Build test
  2. Run the image and check if the service is reachable over HTTP within a minute

Those two very simple tests that would catch the biggest regressions, in my opinion.

frafra avatar May 06 '22 08:05 frafra

This repo will be aligned and upgraded soon https://github.com/GeoNode/geoserver-docker/issues/87

giohappy avatar Jun 01 '22 11:06 giohappy