davsclaus commented on PR #29360: URL: https://github.com/apache/camel/pull/29360#issuecomment-6696892684
CI fails in the new `ScpProducerChecksumFileAlgorithmTest` (`hello.txt.md5 must exist on the server`, and the same for `subdir/bye.txt.md5`). It is not flaky, and the diff is an exact copy of #29317. On camel-4.22.x, `GenericFileProducer.writeChecksumFile` writes the checksum file only when the `CamelFileChecksum` header is already set, and only the local `FileOperations` set it. The change that makes the remote file producers compute the checksum themselves is CAMEL-25906 (#28430), which is on main only. So on this branch `checksumFileAlgorithm=md5` alone never calls the scp `storeFileDirectly`, and the test cannot pass. Two ways forward: - keep the backport and have the test set the `CamelFileChecksum` header on the message, so it exercises the fixed `storeFileDirectly` path, or - drop it: CAMEL-26672 has fixVersion 4.24.0 only, and without CAMEL-25906 the bug is reachable on the LTS branches only when an earlier file producer left that header on the message. _Claude Code on behalf of davsclaus_ -- 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]
