1996fanrui opened a new pull request, #29894: URL: https://github.com/apache/flink/pull/29894
## What is the purpose of the change Backport of #29340 to release-1.20. #29340 is the release-2.3 backport of #29324 (5c56320f1ce). It carries the net effect of the FLINK-40099 / FLINK-40109 changes on master (#28687, #28694, #29324), so it is picked as one commit instead of the three master commits. release-1.20 now has log4j 2.25.3 (FLINK-39146, 5cf2d5e1d15), and the distribution's `log4j.properties` uses `OnStartupTriggeringPolicy`, so container logs can roll on startup here too, and the keytab check misses the login line in `<file>.1`. `YARNSessionFIFOSecuredITCase` and `YarnTestBase.verifyStringsInNamedLogFiles` on this branch have the same code as release-2.3 before the fix, so the test is flaky here in the same way. ## Verifying this change - `./mvnw -pl flink-yarn-tests -am -Pskip-webui-build -Dfast -DskipTests install`, then `./mvnw -pl flink-yarn-tests surefire:test@integration-tests -Dtest='YARNSessionFIFOSecuredITCase' -Dsurefire.failIfNoSpecifiedTests=false` (JDK 11): `Tests run: 3, Failures: 0, Errors: 0, Skipped: 0` in the `integration-tests` execution, `BUILD SUCCESS` - `./mvnw -q -pl flink-yarn-tests spotless:check checkstyle:check` (JDK 11): pass ## Difference from the original | Original commit | Conflict | Byte-identical | Changes | | --- | --- | --- | --- | | 9ce420f91b7 | No | Yes | None | ## Notes - The test ran once; it was not repeated. ##### Was generative AI tooling used to co-author this PR? - [X] Yes (please specify the tool below) Generated-by: Claude Code (Claude Opus 5.5) -- 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]
