feat: add DOCKERHUB_REPO configure when run _install.sh, add search test

Author: WGrape

.work/config/.env.amd64

@@ -17,7 +17,7 @@ CONTAINER_NAMESPACE=test
 # never set username and password, you must login manually in your local system firstly.
 DOCKERHUB_USERNAME=xxx
 DOCKERHUB_PASSWORD=xxx
-DOCKERHUB_DOMAIN=docker.io
+# DOCKERHUB_DOMAIN=docker.io # Deprecated, it will not be used
 DOCKERHUB_REPO=docker.io/lvsid # change to your own docker hub.
 
 ### Services ##############################################

.work/config/.env.arm64

@@ -17,7 +17,7 @@ CONTAINER_NAMESPACE=test
 # never set username and password, you must login manually in your local system firstly.
 DOCKERHUB_USERNAME=xxx
 DOCKERHUB_PASSWORD=xxx
-DOCKERHUB_DOMAIN=docker.io
+# DOCKERHUB_DOMAIN=docker.io # Deprecated, it will not be used
 DOCKERHUB_REPO=docker.io/lvsid # change to your own docker hub.
 
 ### Services ##############################################

.work/include/internal/constant.sh

@@ -21,3 +21,6 @@ CONST_BASE_CONFIG_ENV_ARM64_FILE=".work/config/.env.arm64"
 CONST_BASE_CONFIG_COMPOSE_FILE=".work/config/docker-compose.yml"
 CONST_SPARROW_CONFIG_ENV_FILE=".env"
 CONST_SPARROW_CONFIG_COMPOSE_FILE="docker-compose.yml"
+
+# define some default values
+CONST_DEFAULT_DOCKERHUB_REPO="docker.io/lvsid"

.work/include/internal/dockerhub.sh

@@ -7,22 +7,26 @@ cd $SPARROW_BASE_PATH
 # login of $DOCKERHUB_DOMAIN.
 # to be safety, never allow login with account in the command, you must login manually in your local system firstly.
 # docker login -u $DOCKERHUB_USERNAME -p $DOCKERHUB_PASSWORD $DOCKERHUB_DOMAIN
-login() {
-    print_info "docker login {$DOCKERHUB_DOMAIN}..."
-    # due to the reasons mentioned above, there is no need to do login explicitly here, as the session will be retained after manual login.
-    # if ! docker login; then
-    #    print_error "docker login failed"
-    #    exit 1
-    # fi
-}
+# @Deprecated: it is not available, and not being called
+# login() {
+     # print_info "docker login {$DOCKERHUB_DOMAIN}..."
+     # due to the reasons mentioned above, there is no need to do login explicitly here, as the session will be retained after manual login.
+     # if ! docker login; then
+     #    print_error "docker login failed"
+     #    exit 1
+     # fi
+# }
 
 # search image
 search() {
     search_image=$1
     remove_prefix_search_image=$(echo "$search_image" | sed 's/^docker.io\///')
+    echo "search search_image=${search_image}, remove_prefix_search_image=${remove_prefix_search_image}"
     if docker search "$search_image" | grep -q "^$remove_prefix_search_image"; then
+        print_info "find it"
         return 0  # find
     else
+        print_warn "not find it"
         return 1  # not find
     fi
 }

.work/test/run.sh

@@ -16,11 +16,19 @@ sleep_seconds 3
 testhttp() {
     if is_mac; then
         # open websites.
-        open "http://127.0.0.1:${ETCDKEEPER_HOST_PORT}/etcdkeeper/" # etcdkeeper
-        open "http://127.0.0.1:${KAFKAUI_HOST_PORT}/" # kafkaui
+         open "http://127.0.0.1:${ETCDKEEPER_HOST_PORT}/etcdkeeper/" # etcdkeeper
+         open "http://127.0.0.1:${KAFKAUI_HOST_PORT}/" # kafkaui
     fi
 }
-testhttp
+# testhttp
+
+echo "test search"
+for service in "${ENABLE_SERVICE_LIST[@]}"; do
+    image="sparrow-basic-${service}"
+    search "$DOCKERHUB_REPO/$image"
+    image="sparrow-app-${service}"
+    search "$DOCKERHUB_REPO/$image"
+done
 
 printf "UnitTest Success"
 exit 0

_install.sh

@@ -63,6 +63,20 @@ modify_files() {
     # this operation is fine because when the sdk.sh file is included at the beginning, the /env file is created.
     print_info "set GO_PATH: before($GO_PATH), after($local_go_path)"
     awk -F= '/^GO_PATH=/{OFS="="; $2="'${local_go_path}'"}1' "$CONST_SPARROW_CONFIG_ENV_FILE" > temp && mv temp "$CONST_SPARROW_CONFIG_ENV_FILE"
+
+    # modify DOCKERHUB_REPO in /env file
+    if [ "$DOCKERHUB_REPO" == "$CONST_DEFAULT_DOCKERHUB_REPO" ]; then
+        echo "set DOCKERHUB_REPO to configure your remote repository, press enter will use the default value (default=docker.io/lvsid)"
+        printf "please input: "
+        read dockerhub_repo
+        if [ "$dockerhub_repo" == "" ]; then
+            dockerhub_repo="$CONST_DEFAULT_DOCKERHUB_REPO"
+        else
+            awk -F= '/^DOCKERHUB_REPO=/{OFS="="; $2="'${dockerhub_repo}'"}1' "$CONST_SPARROW_CONFIG_ENV_FILE" > temp && mv temp "$CONST_SPARROW_CONFIG_ENV_FILE"
+        fi
+        print_info "your DOCKERHUB_REPO config is: ${dockerhub_repo}"
+        print_warn "If you need to change the configuration in the future, you can update the DOCKERHUB_REPO variable in the /.env file."
+    fi
 }
 
 # before run install command.