ptupitsyn commented on code in PR #5869:
URL: https://github.com/apache/ignite-3/pull/5869#discussion_r2101847398
##########
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:
Is this supposed to be seen by the users? I think an exception might be
better than an assertion.
--
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
