[fix][build] Fix skipTag and use explicit tag for image name (apache#24168)

Signed-off-by: Zixuan Liu <nodeces@gmail.com>
(cherry picked from commit eb2020c)
(cherry picked from commit 876ece4)

Author: nodece

docker/pulsar-all/pom.xml

@@ -154,11 +154,11 @@
                 <configuration>
                   <images>
                     <image>
-                      <name>${docker.organization}/${docker.image}-all</name>
+                      <name>${docker.organization}/${docker.image}-all:${docker.tag}</name>
                       <build>
                         <contextDir>${project.basedir}</contextDir>
+                        <skipTag>${docker.skip.tag}</skipTag>
                         <tags>
-                          <tag>${docker.tag}</tag>
                           <tag>${project.version}-${git.commit.id.abbrev}</tag>
                         </tags>
                         <args>

docker/pulsar/pom.xml

@@ -77,7 +77,7 @@
                 <configuration>
                   <images>
                     <image>
-                      <name>${docker.organization}/${docker.image}</name>
+                      <name>${docker.organization}/${docker.image}:${docker.tag}</name>
                       <build>
                         <args>
                           <PULSAR_TARBALL>target/pulsar-server-distribution-${project.version}-bin.tar.gz</PULSAR_TARBALL>
@@ -86,8 +86,8 @@
                           <IMAGE_JDK_MAJOR_VERSION>${IMAGE_JDK_MAJOR_VERSION}</IMAGE_JDK_MAJOR_VERSION>
                         </args>
                         <contextDir>${project.basedir}</contextDir>
+                        <skipTag>${docker.skip.tag}</skipTag>
                         <tags>
-                          <tag>${docker.tag}</tag>
                           <tag>${project.version}-${git.commit.id.abbrev}</tag>
                         </tags>
                         <buildx>
@@ -127,6 +127,7 @@
       <id>docker-push</id>
       <properties>
         <docker.skip.push>false</docker.skip.push>
+        <docker.skip.tag>true</docker.skip.tag>
         <docker.platforms>linux/amd64,linux/arm64</docker.platforms>
       </properties>
     </profile>

pom.xml

@@ -102,6 +102,7 @@ flexible messaging model and an intuitive client API.</description>
     -->
     <docker.platforms></docker.platforms>
     <docker.skip.push>true</docker.skip.push>
+    <docker.skip.tag>false</docker.skip.tag>
 
     <!-- required for running tests on JDK11+ -->
     <test.additional.args>

tests/docker-images/java-test-image/pom.xml

@@ -148,16 +148,12 @@
                 <configuration>
                   <images>
                     <image>
-                      <name>${docker.organization}/java-test-image</name>
+                      <name>${docker.organization}/java-test-image:${docker.tag}</name>
                       <build>
                         <args>
                           <PULSAR_IMAGE>${docker.organization}/${docker.image}:${project.version}-${git.commit.id.abbrev}</PULSAR_IMAGE>
                         </args>
                         <contextDir>${project.basedir}</contextDir>
-                        <tags>
-                          <tag>${docker.tag}</tag>
-                          <tag>${project.version}</tag>
-                        </tags>
                         <noCache>true</noCache>
                       </build>
                     </image>

tests/docker-images/latest-version-image/pom.xml

@@ -152,16 +152,12 @@
                 <configuration>
                   <images>
                     <image>
-                      <name>${docker.organization}/${docker.image}-test-latest-version</name>
+                      <name>${docker.organization}/${docker.image}-test-latest-version:${docker.tag}</name>
                       <build>
                         <contextDir>${project.basedir}</contextDir>
                         <args>
                           <PULSAR_ALL_IMAGE>${docker.organization}/${docker.image}-all:${project.version}-${git.commit.id.abbrev}</PULSAR_ALL_IMAGE>
                         </args>
-                        <tags>
-                          <tag>${docker.tag}</tag>
-                          <tag>${project.version}</tag>
-                        </tags>
                         <noCache>true</noCache>
                         <buildx>
                           <platforms>