nick-boss-tech commented on PR #5268: URL: https://github.com/apache/solr/pull/5268#issuecomment-6539462490
Hi @sudip-ibm, 🤖 *AI text below* 🤖 *(reviewed by Nick Shanin)* I prepared a [follow-up patch on top of this PR](https://github.com/nick-boss-tech/solr/compare/705142f6ff0ef6e61c5db747a7a30c12d2fc4ed0...a18d913f2908f0ddf244b43384bd3caecbe9bc6e) to refresh the Testcontainers dependency locks, preserve unrelated pull errors/timeouts/interruptions as test failures, and apply configured image substitution before probing s390x support. <details> <summary>Validation (Docker integration unverified)</summary> At `a18d913f29`, validation on Linux aarch64 passed: the daemon-free helper tests (9/9), API serialization tests (13/13), framework compilation, `tidy`, changelog validation, and `check -x test`. A synthetic mutation restoring the previous error handling and image selection produced exactly the seven expected test failures. Docker and Podman are unavailable in my validation environment, so `DeleteByQueryToIdTest` and `S3OutputStreamTest` were skipped and remain unverified. All of the non-Docker validation listed above passed; there is no native s390x validation yet. </details> The tested head does not yet include the Tika rule migration or clarification of the UI source-build comment raised in the review. To take these changes on top of the current PR head: ```sh git switch testcontainers-util git fetch https://github.com/nick-boss-tech/solr.git pr-5268-container-platform-fixes git merge --ff-only a18d913f2908f0ddf244b43384bd3caecbe9bc6e ``` -- 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]
