Merge pull request #12 from yrodiere/moretags

More tags

Author: yrodiere

src/main/java/org/hibernate/search/develocity/plugins/CompilerConfiguredPlugin.java

@@ -4,6 +4,7 @@
 
 import org.hibernate.search.develocity.GoalMetadataProvider;
 import org.hibernate.search.develocity.SimpleConfiguredPlugin;
+import org.hibernate.search.develocity.scan.BuildScanMetadata;
 import org.hibernate.search.develocity.util.JavaVersions;
 
 public class CompilerConfiguredPlugin extends SimpleConfiguredPlugin {
@@ -28,5 +29,6 @@ private static void configureCompile(GoalMetadataProvider.Context context) {
 					context.configuration().getBoolean( "skip" ),
 					JavaVersions::forJavacExecutable );
 		} );
+		BuildScanMetadata.addCompilerMetadata( context );
 	}
 }

src/main/java/org/hibernate/search/develocity/scan/BuildScanMetadata.java

@@ -38,6 +38,14 @@ public static void addMainMetadata(BuildScanApi buildScanApi) {
 		buildScanApi.tag( "hibernate-search" );
 	}
 
+	public static void addCompilerMetadata(GoalMetadataProvider.Context context) {
+		var buildScanApi = context.buildScan();
+		String compilerId = context.configuration().getString( "compilerId" );
+		if ( !Strings.isBlank( compilerId ) ) {
+			buildScanApi.tag( "compiler-%s".formatted( compilerId ) );
+		}
+	}
+
 	public static void addJavaExecutableVersion(GoalMetadataProvider.Context context,
 			String javaExecutable, String javaVersion,
 			boolean canCacheExactVersion) {
@@ -79,8 +87,8 @@ public static void addFailsafeMetadata(GoalMetadataProvider.Context context) {
 					buildScanApi.tag( "h2" );
 				}
 				else {
-					addDockerfileShortImageRef( context,
-							"database/%s.Dockerfile".formatted( dbKind ), null );
+					addDockerfileMetadata( context,
+							"database/%s.Dockerfile".formatted( dbKind ), dbKind, null, false );
 				}
 			}
 		}
@@ -101,17 +109,22 @@ public static void addFailsafeMetadata(GoalMetadataProvider.Context context) {
 				&& ( explicitBackend == null || "elasticsearch".equals( explicitBackend ) )
 				&& !context.properties().getBoolean( "test.elasticsearch.skip" ) ) {
 			var distribution = context.properties().getString( "test.elasticsearch.distribution" );
-			addDockerfileShortImageRef( context,
+			addDockerfileMetadata( context,
 					"search-backend/%s.Dockerfile".formatted( distribution ),
-					context.properties().getString( "test.elasticsearch.version" ) );
+					"elastic".equals( distribution ) ? "elasticsearch" : distribution,
+					context.properties().getString( "test.elasticsearch.version" ), true );
 		}
 	}
 
-	private static void addDockerfileShortImageRef(GoalMetadataProvider.Context context,
-			String dockerfileRelativePath, String versionOverride) {
+	private static void addDockerfileMetadata(GoalMetadataProvider.Context context,
+			String dockerfileRelativePath, String tag, String versionOverride, boolean tagVersion) {
 		var buildScanApi = context.buildScan();
 		var path = Path.of( context.metadata().getSession().getExecutionRootDirectory(),
 				"build/container", dockerfileRelativePath );
+		if ( tag != null ) {
+			// Tag without version
+			buildScanApi.tag( tag );
+		}
 		try {
 			String ref;
 			try ( var stream = Files.lines( path ) ) {
@@ -131,15 +144,19 @@ private static void addDockerfileShortImageRef(GoalMetadataProvider.Context cont
 			if ( !Strings.isBlank( versionOverride ) ) {
 				ref = ref.substring( 0, ref.lastIndexOf( ':' ) + 1 ) + versionOverride;
 			}
-			String shortImageRef = toShortImageRef( ref );
-			buildScanApi.tag( shortImageRef.replace( ':', '-' ) );
-			buildScanApi.value(
-					shortImageRef.substring( 0, shortImageRef.lastIndexOf( ':' ) ),
-					ref.substring( ref.lastIndexOf( ':' ) + 1 )
-			);
+			context.buildScanDeduplicatedValue( "Container", ref );
+
+			// Tag with version
+			if ( tag != null && tagVersion ) {
+				String shortImageRef = toShortImageRef( ref );
+				int colonIndex = shortImageRef.indexOf( ':' );
+				if ( colonIndex >= 0 ) {
+					buildScanApi.tag( tag + "-" + shortImageRef.substring( colonIndex + 1 ) );
+				}
+			}
 		}
 		catch (RuntimeException | IOException e) {
-			Log.warn( "Unable to add tag from Dockerfile at %s: %s".formatted( path, e.getMessage() ) );
+			Log.warn( "Unable to add metadata from Dockerfile at %s: %s".formatted( path, e.getMessage() ) );
 		}
 	}