ptupitsyn commented on code in PR #5869:
URL: https://github.com/apache/ignite-3/pull/5869#discussion_r2102736961


##########
migration-tools/modules/migration-tools-commons-tests/src/main/java/org/apache/ignite/migrationtools/tests/containers/Ignite3ClusterContainer.java:
##########
@@ -183,4 +181,15 @@ public IgniteClient.Builder clientBuilder() {
     public IgniteClient buildClient() {
         return clientBuilder().build();
     }
+
+    /**
+     * Retrieves the ignite 3 docker image name which should be used in the 
tests.
+     *
+     * @return The docker image name.
+     */
+    public static String dockerImageName() {
+        String imageName = System.getProperty("ignite3.docker.image");
+        assert imageName != null : "ignite3.docker.image property must be 
defined";

Review Comment:
   Ah, I've noticed `src/main` but not on the module name. Does not matter then.



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to