weiqingy commented on issue #1008: URL: https://github.com/apache/flink-agents/issues/1008#issuecomment-5276730852
Folding this into #1007 rather than shipping it separately. Both halves land in `tools/ut.sh` alongside the change already there, and the profile addition is small enough to review in the same pass. The fix will be: - `-f` validated against the versions that actually exist, derived from the `dist/flink-*` modules rather than from a second hardcoded list, so a typo like `-f 9.9` fails fast instead of asking Maven for a module that is not there. - A `flink-2.3` profile added to `e2e-test/flink-agents-end-to-end-tests-integration/pom.xml`, so every value `-f` accepts maps to a real profile. It deliberately omits `flink.log4j2.version`: the four existing profiles pin `2.24.3` while the 2.3 path inherits `2.25.3` from `e2e-test/pom.xml`, and copying a sibling wholesale would move that job onto a different log4j2 without anyone asking for it. Closing here so there is one place to follow. Reopening if the fix ends up needing its own PR after review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
