Skip to content

Commit

Permalink
It works
Browse files Browse the repository at this point in the history
  • Loading branch information
fadnavistanmay committed Oct 3, 2023
1 parent 42669db commit 5139251
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions src/Agent.Plugins/Artifact/PipelineArtifactConstants.cs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

namespace Agent.Plugins
{
// Use PipelineArtifactContants.cs from ADO, once the latest libs are available.
public class PipelineArtifactConstants
{
public const string AzurePipelinesAgent = "AzurePipelinesAgent";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,7 @@ private DedupManifestArtifactClientFactory()
}

traceOutput($"Max dedup parallelism: {maxParallelism}");
traceOutput($"DomainId: {domainId}");

ArtifactHttpClientFactory factory = new ArtifactHttpClientFactory(
connection.Credentials,
Expand Down Expand Up @@ -176,14 +177,13 @@ private DedupManifestArtifactClientFactory()
});

var telemetry = new BlobStoreClientTelemetry(tracer, dedupStoreHttpClient.BaseAddress);
traceOutput($"Hashtype: {this.HashType.Value}");

this.HashType = GetClientHashType(clientSettings, context, tracer);

if (this.HashType == BuildXL.Cache.ContentStore.Hashing.HashType.Dedup1024K)
{
dedupStoreHttpClient.RecommendedChunkCountPerCall = 10; // This is to workaround IIS limit - https://learn.microsoft.com/en-us/iis/configuration/system.webserver/security/requestfiltering/requestlimits/
}
traceOutput($"Hashtype: {this.HashType.Value}");

var dedupClient = new DedupStoreClientWithDataport(dedupStoreHttpClient, new DedupStoreClientContext(maxParallelism), this.HashType.Value);
return (new DedupManifestArtifactClient(telemetry, dedupClient, tracer), telemetry);
Expand Down

0 comments on commit 5139251

Please sign in to comment.