GeoTools / GeoServer PMC meeting - 2024-07-30Attending

   -

   Torben Barsballe
   -

   Jody Garnett
   -

   Peter Smythe
   -

   Jukka Rahkonnen
   -

   Gabriel Roldan
   -

   Kevin Smith
   -

   Andrea Aime


Actions from prior meetings:

   -

   [DONE] Torben: @ignore failing ogr-jni and vsi tests
   -

   [DONE] Torben: Update supported GDAL version in docs
   -

   [TODO] Add flag to fail imageio-ext-gdal tests if GDAL bindings aren’t
   found
   -

   Jody: request or setup discourse channels (for screenshots)

Agenda

   -

   Discourse Transition
   -

   Build stability
   -

   FOSS4G
   -

   PRs

Actions

   -

   Please help review PRs everyone!

Discourse Transition

Discourse is live, please test the instructions:

   -

   Instructions:
   https://discourse.osgeo.org/t/welcome-to-geoserver-user-forum/31009
   -

   Category: https://discourse.osgeo.org/c/geoserver/geoserver-users/51


Is the email sorted:

   -

   join the group: https://discourse.osgeo.org/g/geoserver-users


   -

   geoserver-u...@discourse.osgeo.org


2 Open PRs:

   -

   Documentation: https://github.com/geoserver/geoserver/pull/7796
   -

   Website: https://github.com/geoserver/geoserver.github.io/pull/208
   -

   Changes to Jira
   -

   Changes to Discourse


Feedback:

   -

   Can we correct
   https://discourse.osgeo.org/t/about-the-geoserver-category/31008

Build stability

PRs for security vulnerabilities remain a trouble due to lack of QA
automations.

   -

   This means some open PRs are failuring QA tests and need to be rebased
   -

   I think we need to manually merge these, or just not use that private
   repository feature?
   -

   See
   
https://docs.geoserver.org/latest/en/developer/policies/security.html#working-with-vulnerability-reports
   -

   https://github.com/github/roadmap/issues/627


So until further notice, use public PRs, or manually merge by hand.

MS SQL is Failing / Auto-canceling as well

   -

   mssql_online.yml
   
<https://github.com/geotools/geotools/blob/main/.github/workflows/mssql_online.yml>
   -

   failing to setup docker image also?
   -

      changed docker image <https://hub.docker.com/r/microsoft/mssql-server>
      and automation we rely on is not there
      (directory changed)
      -

      action: Andrea is going to try and pin on older version
      <https://github.com/geotools/geotools/pull/4848>


See PRs for examples:

   -

   https://github.com/geotools/geotools/pull/4847
   -

   https://github.com/geotools/geotools/pull/4808


This seems to have started happening in the last week or so.
FOSS4G & FOSS4GNA

foss4gna: Andrea and turingtestfail are attending :)

   -

   Andrea is going: couple of workshops: ogcapi and vector tiles, and some
   presentations
   -

   Number of PRs around vector tiles ahead of the workshop (see GWC PRs),
   and lots of fun ideas (use transform to cut down labels and apply
   functions).
   -

      this is painful, some ideas on how to make it easier
      -

      using layer group, and pregeneralize, and so on …
      -

      having pregeneralize use zoom tools
      -

   It would be nice to do a BOF and promote supporting the project etc…


foss4g:

   -

   Jody and Gabe can do “State of GeoServer”
   -

   Jody can submit some workshops etc…
   -

   Jody will also be doing some OSGeo talks etc…
   -

   Gabe will do a talk on GeoServer Cloud


PRs

GeoServer

   -

   [GEOS-11271] Upgrade spring-security to 5.8
   <https://github.com/geoserver/geoserver/pull/7792>
   -

      so good to see this, needs to be rebased to address build stability..
      -

   Attempt updating to latest Marlin renderer
   <https://github.com/geoserver/geoserver/pull/7760>
   -

      workin, but not on windows
      -

      needs the “opens” to work on windows…
      -

      need someone (Peter) with access to a windows machine, and ability to
      edit + test batch file + it may be just adjusting quotes?
      -

   [GEOS-11379] Refactor inline JavaScript in the OGC API modules
   <https://github.com/geoserver/geoserver/pull/7587>
   -

      Needs manual testing please! A/B testing - just to see all the same
      functionality exists on the page (does not need to be pretty just not
      missing stuff)
      -

   [GEOS-11399] Use Catalog streaming API in LayerGroupPage
   <https://github.com/geoserver/geoserver/pull/7638>
   -

      This is open since may, any update Gabe?
      -

      After rebase I cannot break it anymore…
      -

      Ready for review :) Get it in now (in August) ahead of the release
      cycle…
      -

         Test with the tools to generate duplicate workspaces again..
         -

         Or use Chat GPT to write a python REST API script
         -

   Bump mapfish print 2.3.0 -> 2.3.1
   <https://github.com/geoserver/geoserver/pull/7794>
   -

      This is now just a simple update, can we merge?
      -

      Additive change only, includes docs, …
      -

      Backport or not? Not ..
      -

      This has a minor breaking change due to change of yaml library
      -

          … a mandatory space is required between key and value
         -

         See https://github.com/mapfish/mapfish-print-v2/pull/23
         -

         This is in the update instructions for geoserver
         -

      The rest of the codebase should be fine, ysld uses snakeyaml and is
      not affected
      -

   Build GDAL from source for OSX Github build
   <https://github.com/geotools/geotools/pull/4808>
   -

      jody to review, squash and merge :)
      -

   Pinning MSSQL 2022 tag to avoid changes in tools directories.
   <https://github.com/geotools/geotools/pull/4848>
   -

      this is the fix from above, thanks Andrea
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to