-
-
Notifications
You must be signed in to change notification settings - Fork 103
tmc meeting minutes 20251016
(07:32) <julianzz98> Bom dia!
(07:41) <copierrj> I started preparing a list of dependencies requiring our attention: https://github.com/deegree/deegree3/issues/1884
(07:41) <log-bot> deegree3#1884 Dependencies requiring attention
(07:41) <log-bot> deegree3#1884 copierrj opened this issue on 2025-10-16 07:41 * updated 2025-10-16 07:41 * 0 comments
(07:45) <StephanR> Hi
(08:00) <dstenger> Welcome to the second day of the deegree code sprint
(08:02) <copierrj> 🎉
(08:36) <tfr42> Hi and welcome to the 2nd day of the deegree code sprint
(08:40) <tfr42> There is one dependency used by deegree which is an obstacle to fully support the Java module system (JDK 9+): Axiom
(08:40) <tfr42> https://github.com/deegree/deegree3/issues/770
(08:40) <log-bot> deegree3#770 Get rid of Axiom [enhancement,contributions welcome,hard,funding welcome]
(08:40) <log-bot> deegree3#770 tfr42 opened this issue on 2016-09-15 12:31 * updated 2025-10-16 08:35 * 1 comments
(08:40) <tfr42> Added it to the task list for the code sprint
(08:46) <dstenger> [ERROR] Failed to execute goal on project deegree-core-cs: Could not resolve dependencies for project org.deegree:deegree-core-cs:jar:3.6.2-SNAPSHOT
(08:46) <dstenger> [ERROR] dependency: it.geosolutions.jgridshift:jgridshift-core:jar:1.3.1 (compile)
(08:46) <dstenger> [ERROR] it.geosolutions.jgridshift:jgridshift-core:jar:1.3.1 was not found in https://repo.deegree.org/content/groups/public/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of deegree-repo has elapsed or updates are forced
(08:46) <dstenger> [ERROR] it.geosolutions.jgridshift:jgridshift-core:jar:1.3.1 was not found in https://repo.maven.apache.org/maven2 during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of central has elapsed or updates are forced
(08:47) <dstenger> We have to build PR https://github.com/deegree/deegree3/pull/1885 with -U option.
(08:47) <log-bot> deegree3#1885 Remove references to external repositories [CI and build]
(08:47) <log-bot> deegree3#1885 dstenger wants to merge 2 commits (1 files) from deegree:limitReferencesToExternalRepositories-1876
(08:47) <log-bot> deegree3#1885 build error at 2025-10-15 18:18 see https://buildserver.deegree.org/job/pull-request/job/limitReferencesToExternalRepositories-1876/2/display/redirect
(08:47) <log-bot> deegree3#1885 build error at 2025-10-16 08:18 see https://buildserver.deegree.org/job/pull-request/job/PR-1885/1/display/redirect
(09:04) <dstenger> Problem was solved by cleaning up local Maven repo
(10:49) <dstenger> I propose to have a session to take a look at and merge pull requests.
(10:51) <dstenger> Let's start with https://github.com/deegree/deegree3/pull/1889
(10:52) <log-bot> deegree3#1889 Switch to ANTLR4 for parsing sql mapping expressions
(10:52) <log-bot> deegree3#1889 copierrj wants to merge 5 commits (8 files) from IDgis:antlr4_sql_mapping_expression
(10:55) <dstenger> We decided to postpone the pull request because it needs further specific testing.
(10:55) <lyngo> mapping="CASE WHEN diameter IS NULL THEN TRUE ELSE FALSE END"
(10:55) <dstenger> https://github.com/deegree/deegree3/pull/1887
(10:55) <log-bot> deegree3#1887 Fix missing mapping of required attributes if property is nilled [bug,tools]
(10:55) <log-bot> deegree3#1887 lgoltz wants to merge 1 commits (3 files) from lat-lon:fix/missingRequiredAttributes-907
(10:55) <log-bot> deegree3#1887 build success at 2025-10-16 10:32 see https://buildserver.deegree.org/job/pull-request/job/PR-1887/1/display/redirect
(10:56) <lyngo> mapping="('https://example.org/' || value || '.txt)"
(11:01) <StephanR> +1 for #1887
(11:01) <dstenger> +1
(11:01) <copierrj> 0
(11:01) <lyngo> +1
(11:02) <julianzz98> +1
(11:03) <tfr42> +1
(11:04) <tfr42> Merged
(11:04) <dstenger> Next PR: https://github.com/deegree/deegree3/pull/1886
(11:04) <log-bot> deegree3#1886 NOT-MERGEABLE Upgrade dependencies
(11:04) <log-bot> deegree3#1886 copierrj wants to merge 5 commits (3 files) from IDgis:upgrade_dependencies
(11:04) <log-bot> deegree3#1886 build success at 2025-10-16 10:48 see https://buildserver.deegree.org/job/pull-request/job/PR-1886/1/display/redirect
(11:05) <copierrj> +1
(11:08) <dstenger> +1
(11:08) <julianzz98> +1
(11:08) <lyngo> +1
(11:10) <tfr42> +1
(11:10) <StephanR> +1
(11:11) <tfr42> merged
(11:11) <dstenger> https://github.com/deegree/deegree3/pull/1885
(11:11) <log-bot> deegree3#1885 Remove references to external repositories [CI and build]
(11:11) <log-bot> deegree3#1885 dstenger wants to merge 2 commits (1 files) from deegree:limitReferencesToExternalRepositories-1876
(11:11) <log-bot> deegree3#1885 build error at 2025-10-15 18:18 see https://buildserver.deegree.org/job/pull-request/job/limitReferencesToExternalRepositories-1876/2/display/redirect
(11:11) <log-bot> deegree3#1885 build success at 2025-10-16 09:44 see https://buildserver.deegree.org/job/pull-request/job/PR-1885/3/display/redirect
(11:14) <copierrj> +1
(11:23) <tfr42> I would like to have a short description in our private https://github.com/deegree/infrastructure as a Markdown file like the System Setup MD.
(13:03) <dstenger> New PR for documentation: https://github.com/deegree/infrastructure/pull/112
(13:06) <copierrj> +1
(13:10) <dstenger> +1
(13:11) <StephanR> +1
(13:11) <tfr42> +1
(13:11) <lyngo> +1
(13:12) <tfr42> merged
(13:12) <dstenger> Now, votes for https://github.com/deegree/deegree3/pull/1885
(13:12) <log-bot> deegree3#1885 Removed references to external repositories from Maven POM [CI and build]
(13:12) <log-bot> deegree3#1885 dstenger wants to merge 2 commits (1 files) from deegree:limitReferencesToExternalRepositories-1876
(13:12) <log-bot> deegree3#1885 build error at 2025-10-15 18:18 see https://buildserver.deegree.org/job/pull-request/job/limitReferencesToExternalRepositories-1876/2/display/redirect
(13:12) <log-bot> deegree3#1885 build pending at 2025-10-16 13:00 see https://buildserver.deegree.org/job/pull-request/job/PR-1885/4/display/redirect
(13:14) <dstenger> +1
(13:16) <StephanR> +1
(13:16) <lyngo> +1
(13:21) <tfr42> +1
(13:22) <copierrj> +1
(13:22) <tfr42> merged
(13:23) <dstenger> https://github.com/deegree/deegree3/pull/1883
(13:23) <log-bot> deegree3#1883 Extended API of Cql2FilterParser for use in OGC API features [OGCAPI,core]
(13:23) <log-bot> deegree3#1883 lgoltz wants to merge 1 commits (5 files) from lat-lon:feature/prepareCql2FilterToUseInOgcApiFeatures-176
(13:23) <log-bot> deegree3#1883 build pending at 2025-10-16 13:15 see https://buildserver.deegree.org/job/pull-request/job/PR-1883/2/display/redirect
(13:27) <dstenger> +1
(13:31) <StephanR> +1
(13:32) <copierrj> 0
(13:32) <tfr42> +1
(13:33) <tfr42> merged
(13:33) <dstenger> https://github.com/deegree/deegree3/pull/1882
(13:33) <log-bot> deegree3#1882 Introduced reduced variants of deegree webservice web app for secure deployments [enhancement]
(13:33) <log-bot> deegree3#1882 stephanr wants to merge 2 commits (13 files) from gritGmbH:enhancement/webservice-variants
(13:33) <log-bot> deegree3#1882 build pending at 2025-10-16 13:30 see https://buildserver.deegree.org/job/pull-request/job/PR-1882/3/display/redirect
(13:36) <copierrj> +1
(13:38) <dstenger> +1
(13:38) <StephanR> +1
(13:38) <lyngo> +1
(13:39) <tfr42> +1
(13:43) <tfr42> merged
(13:43) <dstenger> https://github.com/deegree/deegree3/pull/1881
(13:43) <log-bot> deegree3#1881 Added cyclonedx-maven-plugin to generate SBOM artifacts [CI and build]
(13:43) <log-bot> deegree3#1881 copierrj wants to merge 1 commits (1 files) from IDgis:add_sbom_artifacts
(13:43) <log-bot> deegree3#1881 build pending at 2025-10-16 12:17 see https://buildserver.deegree.org/job/pull-request/job/PR-1881/display/redirect
(13:46) <dstenger> +1
(13:46) <copierrj> +1
(13:47) <StephanR> +1
(13:47) <lyngo> +1
(13:48) <tfr42> There is a plugin for Jenkins CI
(13:48) <tfr42> https://plugins.jenkins.io/dependency-track/
(13:48) <tfr42> I would like to install
(13:48) <tfr42> +1
(14:03) <copierrj> +1
(15:05) <tfr42> merged
(15:10) <dstenger> https://github.com/deegree/deegree3/pull/1880
(15:10) <log-bot> deegree3#1880 NOT-MERGEABLE Moved dependencyManagement section from root POM into a separate deegree-bom module [CI and build]
(15:10) <log-bot> deegree3#1880 copierrj wants to merge 3 commits (3 files) from IDgis:add_deegree_bom_module
(15:10) <log-bot> deegree3#1880 build success at 2025-10-15 16:50 see https://buildserver.deegree.org/job/pull-request/job/PR-1880/3/display/redirect
(15:10) <tfr42> Increased the number of build processors on our Jenkins CI from 1 to 3
(15:11) <copierrj> 🎉
(17:30) <dstenger> https://github.com/deegree/deegree3/pull/1880
(17:30) <log-bot> deegree3#1880 NOT-MERGEABLE Moved dependencyManagement section from root POM into a separate deegree-bom module [CI and build]
(17:30) <log-bot> deegree3#1880 copierrj wants to merge 3 commits (3 files) from IDgis:add_deegree_bom_module
(17:30) <log-bot> deegree3#1880 build success at 2025-10-15 16:50 see https://buildserver.deegree.org/job/pull-request/job/PR-1880/3/display/redirect
(17:47) <StephanR> As discussed we came to the solution to try the maven flatten plugin instead and postpone this pr
(17:48) <dstenger> https://github.com/deegree/deegree3/pull/1849
(17:48) <log-bot> deegree3#1849 Fixes nil properties in GeoJSON (3.6) [bug,WFS,OGCAPI,core]
(17:48) <log-bot> deegree3#1849 lgoltz wants to merge 2 commits (2 files) from lat-lon:fix/nilPropertiesGeoJson-512-3.6
(17:48) <log-bot> deegree3#1849 build success at 2025-10-16 16:45 see https://buildserver.deegree.org/job/pull-request/job/PR-1849/13/display/redirect
(17:48) <dstenger> and backport https://github.com/deegree/deegree3/pull/1850
(17:48) <log-bot> deegree3#1850 Fixes nil properties in GeoJSON (3.5) [bug,under review,WFS,OGCAPI,core]
(17:48) <log-bot> deegree3#1850 lgoltz wants to merge 2 commits (2 files) from lat-lon:fix/nilPropertiesGeoJson-512-3.5
(17:48) <log-bot> deegree3#1850 build success at 2025-09-17 17:39 see https://buildserver.deegree.org/job/pull-request/job/PR-1850/7/display/redirect
(18:03) <dstenger> Let's start with the votes for 3.6: https://github.com/deegree/deegree3/pull/1849
(18:03) <dstenger> +1
(18:03) <StephanR> +1
(18:03) <julianzz98> +1
(18:05) <copierrj> 0
(18:05) <lyngo> +1
(18:06) <StephanR> merged
(18:06) <dstenger> Now, let's decide how to handle: https://github.com/deegree/deegree3/pull/1850
(18:06) <log-bot> deegree3#1850 Fixes nil properties in GeoJSON (3.5) [bug,under review,WFS,OGCAPI,core]
(18:06) <log-bot> deegree3#1850 lgoltz wants to merge 2 commits (2 files) from lat-lon:fix/nilPropertiesGeoJson-512-3.5
(18:06) <log-bot> deegree3#1850 build success at 2025-09-17 17:39 see https://buildserver.deegree.org/job/pull-request/job/PR-1850/7/display/redirect
(18:08) <tfr42> 0
(18:08) <lyngo> 0
(18:08) <julianzz98> 0
(18:08) <StephanR> 0
(18:08) <dstenger> 0
(18:09) <dstenger> PR was not accepted as 3.5 line is running out and is closed with a comment.
(18:10) <StephanR> closed with comment as in our templates
(19:12) <julianzz98> hi
(19:12) <copierrj> 👋
(19:13) <dstenger> https://github.com/deegree/deegree3/pull/1892
(19:13) <log-bot> deegree3#1892 Migrated and upgraded of commons-collections4 to version 4.5.0 and of commons-beanutils2 to version 2.0.0-M2 [dependencies]
(19:13) <log-bot> deegree3#1892 dstenger wants to merge 1 commits (6 files) from lat-lon:upgradeCommonsCollections-1891
(19:13) <log-bot> deegree3#1892 build success at 2025-10-16 17:48 see https://buildserver.deegree.org/job/pull-request/job/PR-1892/1/display/redirect
(19:14) <copierrj> +1
(19:15) <dstenger> +1
(19:15) <julianzz98> +1
(19:15) <lyngo> +1
(19:15) <StephanR> +1
(19:15) <tfr42> +1
(19:16) <copierrj> merged
(19:16) <dstenger> https://github.com/deegree/deegree3/pull/1834
(19:16) <log-bot> deegree3#1834 Fixes GFI on a group layers if one of the sub layers is not queryable (3.5) [bug,needs discussion,TMC discussion,under review,WMS]
(19:16) <log-bot> deegree3#1834 lgoltz wants to merge 2 commits (1 files) from lat-lon:fix/gfiOnGroupLayer-539-1822
(19:16) <log-bot> deegree3#1834 build success at 2025-09-17 17:29 see https://buildserver.deegree.org/job/pull-request/job/PR-1834/12/display/redirect
(19:17) <dstenger> https://github.com/deegree/deegree3/pull/1757
(19:17) <log-bot> deegree3#1757 Fixed recognization of deprecated FeatureCollections in deegree (3.5) [needs discussion,TMC discussion,under review,tools,core]
(19:17) <log-bot> deegree3#1757 lgoltz wants to merge 4 commits (17 files) from lat-lon:feature/recognitionDeprecatedTypes-3.5-10004-1752
(19:17) <log-bot> deegree3#1757 build success at 2025-09-17 18:05 see https://buildserver.deegree.org/job/pull-request/job/PR-1757/27/display/redirect
(19:17) <dstenger> https://github.com/deegree/deegree3/pull/1730
(19:17) <log-bot> deegree3#1730 Enhanced bbox requests without geometry property to consider all geometry properties (3.5) [enhancement,needs discussion,TMC discussion,under review,WFS]
(19:17) <log-bot> deegree3#1730 lgoltz wants to merge 1 commits (13 files) from lat-lon:feature/multipleGeomProperties-9885-32
(19:17) <log-bot> deegree3#1730 build success at 2025-09-17 18:15 see https://buildserver.deegree.org/job/pull-request/job/PR-1730/28/display/redirect
(19:17) <dstenger> https://github.com/deegree/deegree3/pull/1729
(19:17) <log-bot> deegree3#1729 Added calculation of the bounding box across multiple geometry columns for PostgreSQL (3.5) [enhancement,needs discussion,TMC discussion,under review,breaking]
(19:17) <log-bot> deegree3#1729 julianzz98 wants to merge 2 commits (6 files) from lat-lon:feature/calvEnvelopeMultipleBbox-9727-30
(19:17) <log-bot> deegree3#1729 build success at 2025-09-17 18:56 see https://buildserver.deegree.org/job/pull-request/job/PR-1729/31/display/redirect
(19:18) <dstenger> All 4 pull requests are targeting 3.5.
(19:18) <tfr42> -1
(19:19) <dstenger> My proposal is to close all 4 of them with a comment that 3.5 line is running out. In all cases, the 3.6 PRs were merged.
(19:19) <dstenger> +1
(19:19) <tfr42> +1
(19:19) <lyngo> +1
(19:19) <StephanR> +1
(19:20) <copierrj> +1
(19:20) <julianzz98> +1
(19:21) <StephanR> all four where closed with comment
(19:22) <tfr42> https://github.com/deegree/deegree-ogcapi/issues/149
(19:22) <log-bot> deegree-ogcapi#149 Rename deegree OGC API - Features in documentation to deegree ogcapi [documentation]
(19:22) <log-bot> deegree-ogcapi#149 tfr42 opened this issue on 2025-03-21 19:59 * updated 2025-10-16 09:44 * 2 comments
(19:23) <tfr42> +1
(19:23) <tfr42> https://github.com/deegree/deegree-ogcapi/pull/178#
(19:23) <log-bot> deegree-ogcapi#178 Renamed deegree OAF to deegree ogcapi in documentation [documentation]
(19:23) <log-bot> deegree-ogcapi#178 tfr42 wants to merge 1 commits (6 files) from lat-lon:fix/updateDocumentation-149
(19:23) <log-bot> deegree-ogcapi#178 build success at 2025-10-15 22:50 see https://buildserver.deegree.org/job/ogcapi-pr/job/PR-178/1/display/redirect
(19:23) <julianzz98> +1
(19:24) <lyngo> +1
(19:24) <StephanR> +1
(19:25) <dstenger> +1
(19:25) <copierrj> +1
(19:25) <tfr42> +1
(19:26) <copierrj> merged
(19:32) <StephanR> Discussion of topic https://github.com/deegree/deegree3/issues/1874
(19:32) <log-bot> deegree3#1874 Introduce automated mechanism closing stuck pull requests
(19:32) <log-bot> deegree3#1874 dstenger opened this issue on 2025-10-08 16:31 * updated 2025-10-08 16:33 * 1 comments
(19:35) <StephanR> My proposal would be to assign each pull request with no activity for 3 months (no comments, label or commits) to assign the label stuck automatically.
(19:36) <copierrj> +1
(19:36) <lyngo> +1
(19:37) <StephanR> And as a futher step to close all pull request that are labeld stuck or needs_rebase without 3 months of activity (no comments, label or comment) to close automatically