mattcasters opened a new pull request, #8953: URL: https://github.com/apache/hop/pull/8953
Fixes the 12 integration-test failures from Hop-integration-tests build 2346 (issue #8952). **Mail.** James WebAdmin was still binding port 8000 when `main-0001-james-rest` finished its fixed 30s delay, so the user-creation curls were refused and the later mail tests had no accounts. `0001` and `0009` now wait until `http://james:8000/domains` accepts a connection (up to 120s) and then create the domains and users. A server that is already up is not delayed. **Neo4j error lineage.** Registering a child linked both the owning workflow and the action with `EXECUTES`. `shortestPath` then skipped `error-lineage-8704-failing.hpl`. When the launcher node exists, only that node is linked. When the launcher is stored later, its `EXECUTES` edge is added and the owner shortcut is removed. `(workflow)-[:EXECUTES]->(action)` stays, because an action has no `launchedById`. **OpenSearch execution information.** A child location loaded the parent document without the OpenSearch `_id` (`internalId` is not stored in `_source`) and posted a second document. Search could then return the copy that had no execution state, so the validation filter kept 1 of 3 rows. Loads now keep `_id`. A link write that loaded the parent only to attach the child re-reads first and keeps a newer execution state, and the write uses `refresh=wait_for` so that re-read can see a parent that just finished. **Samples.** The suite was still progressing when the 3600s watchdog killed it during `write-read-huge-file`. `integration-tests-samples.yaml` sets `HOP_IT_TIMEOUT` to 5400 for that project only. The value is literal because `run-tests-docker.sh` exports `HOP_IT_TIMEOUT` before Compose interpolates the base file. `docker compose config` resolves it to 5400 while the host variable is 3600. Unit tests: `BaseCachingExecutionInfoLocationTest` (12), `CachingFileExecutionInfoLocationTest` (16), `NeoExecutionInfoLocationDeleteTest` (6), `OpenSearchExecutionInfoLocationCleanupTest` (6). Spotless and Checkstyle are clean on those modules. The nightly Docker suite was not rerun. ------------------------ Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] Run `mvn clean install apache-rat:check` to make sure basic checks pass. A more thorough check will be performed on your pull request automatically. - [x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using `git rebase -i`. - [x] Mention the appropriate issue in your description (for example: `addresses #123`), if applicable. To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0) you have to acknowledge this by using the following check-box. - [x] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0) - [ ] In any other case, please file an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). -- 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]
