Skip to content

Commit

Permalink
(build) fix docker artifacts test
Browse files Browse the repository at this point in the history
  • Loading branch information
arturcic committed Apr 14, 2023
1 parent 6b3ff65 commit 0d5c064
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions build/artifacts/Tasks/ArtifactsNativeTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,13 @@ public override void Run(BuildContext context)
{
if (context.SkipImageForArtifacts(dockerImage)) continue;

var runtime = dockerImage.Architecture == Architecture.Amd64 ? "linux-x64" : "linux-arm64";
var runtime = "linux";
if (dockerImage.Distro.StartsWith("alpine"))
{
runtime = "linux-musl-x64";
runtime += "-musl";
}
runtime += dockerImage.Architecture == Architecture.Amd64 ? "-x64" : "-arm64";


var cmd = $"{rootPrefix}/scripts/test-native-tool.sh --version {version} --repoPath {rootPrefix}/repo --runtime {runtime}";

Expand Down
2 changes: 1 addition & 1 deletion build/common/Utilities/Constants.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public class Constants

public static readonly Architecture[] ArchToBuild = { Architecture.Amd64, Architecture.Arm64 };
public static readonly string[] VersionsToBuild = { Version60, Version70 };
public static readonly string[] DistrosToSkipForArtifacts = { Alpine316, Alpine317, Centos7, Ubuntu2204 };
public static readonly string[] DistrosToSkipForArtifacts = { Centos7 };
public static readonly string[] DistrosToSkipForDocker = { Centos7 };

public const string DockerBaseImageName = "gittools/build-images";
Expand Down

0 comments on commit 0d5c064

Please sign in to comment.