Bug #22090


Datasource broken b/c version of dependencyManagement provided XML lib is not enforced in children projects

Added by François ARMAND over 1 year ago. Updated 7 months ago.

Plugins management
Target version:
UX impact:
User visibility:
Effort required:
Name check:
To do
Fix check:
To do


Trying to build datasource plugin in rudder 7.3 leads to an error:

Parsing: Unexpected <SECTIONS> child element in technique rpmPackageInstallation/7.0: #REM

After some decyphering, it means that the XML parsing failed because it found a comment. Which it does not on rudder-core, where the same XML are provided for test config-repo techniques.

After some head banging, it appears that maven is not using the correct scala-xml lib, which we chose to avoid because of such changes.
It seems that somewhere, we are loosing the dependency versions we tried to enforce by declaring them in parent-pom <dependencyManagement> (@mvn dependency:tree)

Other dependencies have the correct version but are omitted and that one is chosen. It seems to be that reported maven bug:

So, we need to enforce again at each level the version we want.

The correct solution seems to be:

- create a new sub-project in rudder, named something like rudder-libs-bom
- create a script that can be run from rudder-web (our final project) and autogenerate the list of all dependencies with their version, if possible using the properties declared in rudder parent for that when available,
- then, in at least plugin-commons, use the <dependencyManagement>... <dependency>...<artifactid>rudder-libs-bom<artifactid><scope>... directive.

It would provide a correct matrix of libs/version for plugins.
Since it's a big change, a first step that should allow to correct XML version (and all other lib in the same case) is to add the version in our currect dependencyManagement in plugin-commons.


clipboard-202211171305-6tsce.png (10.3 KB) clipboard-202211171305-6tsce.png François ARMAND, 2022-11-17 13:05
Actions #1

Updated by François ARMAND over 1 year ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions #2

Updated by François ARMAND over 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Alexis Mousset
  • Pull Request set to
Actions #3

Updated by Anonymous over 1 year ago

  • Status changed from Pending technical review to Pending release
Actions #4

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from master to 8.0
Actions #5

Updated by Alexis Mousset 7 months ago

  • Status changed from Pending release to Released
  • Target version changed from 8.0 to 7.3

Also available in: Atom PDF