FrankChen021 commented on PR #19936: URL: https://github.com/apache/druid/pull/19936#issuecomment-5225479662
Codex triage update at the verified head `dd11b30f03b21d67759a6de76b3d3ba7903925fa`. The pac4j upgrade is `5.7.10 -> 6.5.5`. The source migration and focused tests are in place, but pac4j 6 changes profile/session serialization; during a rolling upgrade, existing session cookies may become invalid and users may need to authenticate again. See the [pac4j 6 release notes](https://github.com/pac4j/pac4j/blob/pac4j-parent-6.5.5/documentation/docs/release-notes.md). The current [CI run](https://github.com/apache/druid/actions/runs/31248071833) is terminal with 25 successful checks, 1 failure, and 2 skipped checks. The failed [K/E/W/Z/Y/X unit job](https://github.com/apache/druid/actions/runs/31248071833/job/93079858846) reports an unrelated `KafkaMultiSupervisorTest` `UnknownTopicOrPartitionException`; the coverage job was skipped because of that failure. Local focused coverage passed, but this upgrade still requires a full human test in a real Druid test cluster covering OIDC/JWT callbacks, session cookies, deployment behavior, clients, and rolling upgrades. I am not approving or merging this PR, and I am stopping automated work on it pending that validation and CI disposition. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
