sergehuber opened a new pull request, #759:
URL: https://github.com/apache/unomi/pull/759
## Summary
First commit of the PR #757 stack split — see
[`docs/PR-757-stack-extraction-tracker.md`](../blob/UNOMI-139-UNOMI-880-multitenancy/docs/PR-757-stack-extraction-tracker.md)
on the monolith branch.
This A0 PR migrates the `elasticsearch` profile of `itests` from the legacy
`com.github.alexcojocaru:elasticsearch-maven-plugin` (binary download + forked
JVM) to `io.fabric8:docker-maven-plugin`, so the ES integration tests now run
against
`docker.elastic.co/elasticsearch/elasticsearch:${elasticsearch.test.version}` —
matching how the `opensearch` profile already runs OpenSearch in Docker.
The change is intentionally minimal so it can be reviewed and merged
independently of the rest of #757. Once merged, `git town sync --stack`
propagates it through #754 → #756, and the remaining peeled phases (P / V / S /
T / D) sit on top of it.
## Changes
### `itests/pom.xml` (elasticsearch profile only)
- Add `<elasticsearch.port>9400</elasticsearch.port>` property, surface it
to failsafe `systemPropertyVariables`
- Replace the `elasticsearch-maven-plugin` plugin block with a
`docker-maven-plugin` block (start/stop ES container, bind mount
`target/snapshots_repository → /tmp/snapshots_repository`, wait on HTTP port)
- Add a `chmod -R ugo+rwx target/snapshots_repository` antrun step (Linux CI
hits `access_denied` because the ES container runs as UID 1000)
### `pom.xml` (root)
- Add `<docker-maven-plugin.version>0.48.0</docker-maven-plugin.version>`
property
- Add `io.fabric8:docker-maven-plugin` entry in `<pluginManagement>` so the
elasticsearch profile inherits the version
## Test plan
- [ ] CI: \`Execute integration tests (elasticsearch, 9400)\` job runs ES
via Docker and stays green
- [ ] CI: opensearch and unit-test jobs unaffected
- [ ] Local: `mvn -B verify -P elasticsearch -pl itests` runs the suite
against a Docker ES container
- [ ] Local: `mvn -B verify -P opensearch -pl itests` still passes (no OS
profile changes)
## Stack context
Part of the PR #757 split — see the tracker. Subsequent PRs in the stack
rebuild from the same `backup/pre-split-757` tag (`7d5b6a43f`).
--
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]
