-
-
Notifications
You must be signed in to change notification settings - Fork 103
tmc meeting minutes 20250416
(17:00) <StephanR>Hi
(17:00) <dstenger>Hi
(17:00) <julianzz98>hi
(17:00) <StephanR>https://meet.google.com/nhd-phdu-tzs
(17:02) <julianzz98>Google meets is still loading...
(17:06) <dstenger>Hello everyone, welcome to the TMC meeting!
(17:06) <dstenger>Are there any left overs from last meeting?
(17:07) <StephanR>Meeting minutes from last meeting available at https://github.com/deegree/deegree3/wiki/tmc-meeting-minutes-20250402
(17:09) <dstenger>Last time, we discuessed to go through all issues/ PRs of milestone 3.6: https://github.com/deegree/deegree3/milestone/68
(17:09) <StephanR>Infrastructure pull request for ogcri updates already implemented https://github.com/deegree/infrastructure/pull/104
(17:10) <dstenger>Any topics for this meeting?
(17:10) <dstenger>Status of reference implementations
(17:12) - copierrj joined
(17:12) <copierrj>hi
(17:12) <StephanR>https://meet.google.com/nhd-phdu-tzs
(17:12) <StephanR>Hi
(17:14) <tfr42>PR for infrastructure https://github.com/deegree/infrastructure/pull/105
(17:15) <StephanR>Any other topics ?
(17:15) <dstenger>Lets start with status of reference implementations
(17:16) <StephanR>see infrastructure pr https://github.com/deegree/infrastructure/pull/104
(17:17) <dstenger>New URLs:
(17:17) <dstenger>- https://cite.deegree.org/deegree-webservices/
(17:17) <dstenger>- https://cite.deegree.org/deegree-ogcapi/datasets/kitaeinrichtung
(17:18) <copierrj>+1
(17:18) <StephanR>+1
(17:18) <tfr42>+1
(17:19) <dstenger>+1
(17:19) <julianzz98>+1
(17:19) <copierrj>merged
(17:19) <copierrj>https://github.com/deegree/infrastructure/pull/105
(17:20) <tfr42>Important upgrade of Nexus and Jenkins
(17:20) <StephanR>We are palning to update the demo instances as discussed at laste meetin before the next tmc meeting.
(17:20) <copierrj>+1
(17:20) <dstenger>+1
(17:20) <StephanR>+1
(17:20) <tfr42>+1
(17:21) <copierrj>merged
(17:23) <dstenger>Then let's talk about issues/PRs of milestone 3.6: https://github.com/deegree/deegree3/milestone/68
(17:23) <dstenger>https://github.com/deegree/deegree3/pull/1728 and https://github.com/deegree/deegree3/pull/1726
(17:23) <log-bot>deegree3#1728 Enhance SqlFeatureStoreConfigCreator to apply useRefDataProps to complex xml elements (3.6) [enhancement,tools]
(17:23) <log-bot>deegree3#1728 lgoltz wants to merge 1 commits (2 files) from lat-lon:fix/ignoreXSElementDeclaration-9914-3.6
(17:23) <log-bot>deegree3#1728 build success at 2025-04-10 18:23 see https://buildserver.deegree.org/job/pull-request/job/PR-1728/19/display/redirect
(17:23) <log-bot>deegree3#1726 Enhance SqlFeatureStoreConfigCreator to apply useRefDataProps to complex xml elements (3.5) [enhancement,tools]
(17:23) <log-bot>deegree3#1726 lgoltz wants to merge 1 commits (2 files) from lat-lon:fix/ignoreXSElementDeclaration-9914
(17:23) <log-bot>deegree3#1726 build success at 2025-04-03 12:38 see https://buildserver.deegree.org/job/pull-request/job/PR-1726/14/display/redirect
(17:27) <julianzz98>https://github.com/deegree/deegree3/issues/1727
(17:27) <log-bot>deegree3#1727 Apply useRefDataProps=true to substitution [enhancement,tools]
(17:27) <log-bot>deegree3#1727 lgoltz opened this issue on 2024-08-06 08:14 * updated 2024-08-06 08:14 * 0 comments
(17:29) <julianzz98>Why is this issue marked as enhancement, when, as I understand, its more like a "bug"?
(17:30) <StephanR>Regarding the issue of jenkins: WARNING h.i.i.InstallUncaughtExceptionHandler#handleException: Caught unhandled exception with ID d139a7d3-1c2e-4334-a604-feda6ed72e34
(17:30) <StephanR>jenkins-1 | java.lang.AbstractMethodError: Receiver class hudson.plugins.audit_trail.AuditTrailFilter does not define or inherit an implementation of the resolved method 'abstract boolean handle(jakarta.servlet.http.HttpServletRequest, jakarta.servlet.http.HttpServletResponse)' of interface jenkins.util.HttpServletFilter.
(17:33) <StephanR>artifactId=audit-trail
(17:33) <StephanR>groupId=org.jenkins-ci.plugins
(17:33) <StephanR>version=393.v6a_3d1e251274
(17:36) <StephanR>touched the file to disable the plugin
(17:36) <StephanR>-la /opt/jenkins/jenkins-home/plugins/aud*
(17:36) <StephanR>-rw-r--r-- 1 reichhelm tmc 104702 Oct 31 11:58 /opt/jenkins/jenkins-home/plugins/audit-trail.bak
(17:36) <StephanR>-rw-r--r-- 1 reichhelm tmc 104669 Mar 17 15:50 /opt/jenkins/jenkins-home/plugins/audit-trail.jpi
(17:36) <StephanR>-rw-r--r-- 1 reichhelm tmc 0 Apr 16 17:34 /opt/jenkins/jenkins-home/plugins/audit-trail.jpi.disabled
(17:36) <dstenger>https://download.deegree.org/documentation/3.5.11/html/#_usage_of_option_userefdataprops
(17:39) <julianzz98>https://github.com/deegree/deegree3/issues/1727
(17:40) <dstenger>Let's vote for https://github.com/deegree/deegree3/pull/1728 and https://github.com/deegree/deegree3/pull/1726
(17:40) <dstenger>+1
(17:40) <copierrj>+1
(17:40) <tfr42>+1
(17:41) <StephanR>+
(17:41) <StephanR>+1
(17:41) <julianzz98>+1
(17:41) <copierrj>both merged
(17:42) <tfr42>https://github.com/deegree/deegree3/pull/1799
(17:42) <log-bot>deegree3#1799 NOT-MERGEABLE Upgraded slf4j to latest bugfix version and maven plugins [dependencies,CI and build]
(17:42) <log-bot>deegree3#1799 lgoltz wants to merge 3 commits (1 files) from lat-lon:fix/upgradeDependencies-3.6-XPLANBOX-3568
(17:42) <log-bot>deegree3#1799 build success at 2025-04-10 17:28 see https://buildserver.deegree.org/job/pull-request/job/PR-1799/8/display/redirect
(17:42) <dstenger>+1
(17:42) <tfr42>+1
(17:42) <copierrj>+1
(17:43) <StephanR>+1
(17:43) <copierrj>merged
(17:43) <tfr42>https://github.com/deegree/deegree3/pull/1805 and https://github.com/deegree/deegree3/pull/1806
(17:43) <log-bot>deegree3#1805 Added http notation to EPSG:5554,5555,5556 (3.6) [crs]
(17:43) <log-bot>deegree3#1805 lgoltz wants to merge 1 commits (1 files) from lat-lon:feature/addHttpUrlForEpsg5554_5555_5556-343-6.0
(17:43) <log-bot>deegree3#1805 build success at 2025-04-14 15:08 see https://buildserver.deegree.org/job/pull-request/job/PR-1805/1/display/redirect
(17:43) <log-bot>deegree3#1806 Added http notation to EPSG:5554,5555,5556 (3.5) [crs]
(17:43) <log-bot>deegree3#1806 lgoltz wants to merge 1 commits (1 files) from lat-lon:feature/addHttpUrlForEpsg5554_5555_5556-343-3.5
(17:43) <log-bot>deegree3#1806 build success at 2025-04-14 15:19 see https://buildserver.deegree.org/job/pull-request/job/PR-1806/1/display/redirect
(17:43) <dstenger>+1
(17:44) <copierrj>+1
(17:44) <tfr42>+1
(17:44) <StephanR>+1
(17:44) <julianzz98>+1
(17:45) <tfr42>+1
(17:46) <copierrj>merged
(17:46) <dstenger>Let's continue with https://github.com/deegree/deegree3/pull/1758 and https://github.com/deegree/deegree3/pull/1757
(17:46) <log-bot>deegree3#1758 NOT-MERGEABLE Fix recognization of deprecated FeatureCollections in deegree (3.6) [tests failing,CI failing,tools,core]
(17:46) <log-bot>deegree3#1758 lgoltz wants to merge 3 commits (18 files) from lat-lon:feature/recognitionDeprecatedTypes-3.6-10004-1752
(17:46) <log-bot>deegree3#1758 build error at 2025-04-10 17:37 see https://buildserver.deegree.org/job/pull-request/job/PR-1758/16/display/redirect
(17:46) <log-bot>deegree3#1757 NOT-MERGEABLE Fix recognization of deprecated FeatureCollections in deegree (3.5) [tools,core]
(17:46) <log-bot>deegree3#1757 lgoltz wants to merge 3 commits (17 files) from lat-lon:feature/recognitionDeprecatedTypes-3.5-10004-1752
(17:46) <log-bot>deegree3#1757 build success at 2025-04-03 12:07 see https://buildserver.deegree.org/job/pull-request/job/PR-1757/11/display/redirect
(17:48) <tfr42>Let's postpone it until build job finished
(17:48) <dstenger>Let's continue with https://github.com/deegree/deegree3/pull/1731 and https://github.com/deegree/deegree3/pull/1729
(17:49) <log-bot>deegree3#1731 Added calculation of the bounding box across multiple geometry columns for PostgreSQL (3.6) [enhancement,WFS]
(17:49) <log-bot>deegree3#1731 lgoltz wants to merge 2 commits (6 files) from lat-lon:feature/calvEnvelopeMultipleBbox-9727-30-3.6
(17:49) <log-bot>deegree3#1731 build success at 2025-04-10 18:08 see https://buildserver.deegree.org/job/pull-request/job/PR-1731/21/display/redirect
(17:49) <log-bot>deegree3#1729 Added calculation of the bounding box across multiple geometry columns for PostgreSQL (3.5) [enhancement,needs discussion]
(17:49) <log-bot>deegree3#1729 julianzz98 wants to merge 2 commits (6 files) from lat-lon:feature/calvEnvelopeMultipleBbox-9727-30
(17:49) <log-bot>deegree3#1729 build success at 2025-04-03 12:28 see https://buildserver.deegree.org/job/pull-request/job/PR-1729/15/display/redirect
(17:51) <dstenger>There was a similar discussion in https://github.com/deegree/deegree3/pull/1732
(17:51) <log-bot>deegree3#1732 Enhance bbox requests without geometry property to consider all geometry properties (3.6) [enhancement,needs discussion,breaking]
(17:51) <log-bot>deegree3#1732 lgoltz wants to merge 1 commits (13 files) from lat-lon:feature/multipleGeomProperties-9885-32-3.6
(17:51) <log-bot>deegree3#1732 build success at 2025-04-10 17:52 see https://buildserver.deegree.org/job/pull-request/job/PR-1732/20/display/redirect
(18:06) - copierrj quit (Quit: Client closed)
(18:06) - copierrj joined
(18:09) <copierrj>After some discussion the TMC concluded that this change in behavior (= how deegree deals with multiple geometry attributes) is sensible but might result in surprising (to the user) results. Therefore we would like to put this change behind a tunable parameter.
(18:11) <copierrj>In this backport we would expect the default value of this tunable to be false.
(18:13) <dstenger>Same applies to https://github.com/deegree/deegree3/pull/1730 and https://github.com/deegree/deegree3/pull/1732
(18:13) <log-bot>deegree3#1730 Enhance bbox requests without geometry property to consider all geometry properties (3.5) [enhancement]
(18:13) <log-bot>deegree3#1730 lgoltz wants to merge 1 commits (13 files) from lat-lon:feature/multipleGeomProperties-9885-32
(18:13) <log-bot>deegree3#1730 build success at 2025-04-03 12:18 see https://buildserver.deegree.org/job/pull-request/job/PR-1730/13/display/redirect
(18:13) <log-bot>deegree3#1732 Enhance bbox requests without geometry property to consider all geometry properties (3.6) [enhancement,needs discussion,breaking]
(18:13) <log-bot>deegree3#1732 lgoltz wants to merge 1 commits (13 files) from lat-lon:feature/multipleGeomProperties-9885-32-3.6
(18:13) <log-bot>deegree3#1732 build success at 2025-04-10 17:52 see https://buildserver.deegree.org/job/pull-request/job/PR-1732/20/display/redirect
(18:14) <tfr42>Build for Fix recognization of deprecated FeatureCollections in deegree (3.6) (#1758) still fails https://buildserver.deegree.org/view/All/job/pull-request/view/change-requests/job/PR-1758/
(18:23) <tfr42>I would like to build a new bugfix version 3.5.12 with the PR we have included today
(18:23) <copierrj>+1
(18:23) <tfr42>+1
(18:23) <dstenger>+1
(18:23) <StephanR>+1
(18:23) <julianzz98>+1
(18:25) <tfr42>I will prepare the bugfix version
(18:25) <tfr42>All other open PR marked fot 3.5.12 have been moved to 3.5.13
(18:26) <tfr42>They still need to be discussed
(18:27) <tfr42>#1757 Fix recognization of deprecated FeatureCollections in deegree (3.5)
(18:27) <tfr42>#1730 Enhance bbox requests without geometry property to consider all geometry properties (3.5)
(18:27) <tfr42>#1729 Added calculation of the bounding box across multiple geometry columns for PostgreSQL (3.5)
(18:27) <tfr42>Next TMC meeting?
(18:28) <copierrj>2025-04-30?
(18:30) <tfr42>15:00 CEST
(18:32) <copierrj>let's do the regular time: 17:00h?
(18:32) <StephanR>+1
(18:32) <copierrj>+1
(18:32) <tfr42>Yes, w stay with the regular schedule and time
(18:32) <tfr42>+1
(18:32) <dstenger>0
(18:33) <tfr42>Ok, thank you and talk to you soon...
(18:35) <julianzz98>+1
(18:35) <tfr42>bye
(18:35) <julianzz98>bye
(18:35) <copierrj>bye