Skip to content

Commit

Permalink
chore: include variant in ng installer error reporting (#25544)
Browse files Browse the repository at this point in the history
Co-authored-by: julien-lebot <julien.lebot@datadoghq.com>
  • Loading branch information
spencergilbert and julien-lebot committed May 14, 2024
1 parent 46f1f3e commit 2cacfc1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ Mock<ISession> sessionMock
{ "version", CiInfo.PackageVersion },
{ "log", string.Empty },
{ "email", email },
{ "apikey", apiKey }
{ "apikey", apiKey },
{ "variant", "windows_installer" }
};
httpClientMock.Verify(c => c.Post(
$"https://api.{site}/agent_stats/report_failure",
Expand Down Expand Up @@ -82,7 +83,8 @@ public void Flare_Should_Post_Empty_Logs_If_MsiLogFileLocation_Empty(
{ "version", CiInfo.PackageVersion },
{ "log", string.Empty },
{ "email", email },
{ "apikey", apiKey }
{ "apikey", apiKey },
{ "variant", "windows_installer" }
};
httpClientMock.Verify(c => c.Post(
$"https://api.{site}/agent_stats/report_failure",
Expand Down Expand Up @@ -131,7 +133,8 @@ Mock<ISession> sessionMock
{ "version", CiInfo.PackageVersion },
{ "log", log },
{ "email", email },
{ "apikey", apiKey }
{ "apikey", apiKey },
{ "variant", "windows_installer" }
};
httpClientMock.Verify(c => c.Post(
$"https://api.{site}/agent_stats/report_failure",
Expand Down
3 changes: 2 additions & 1 deletion tools/windows/DatadogAgentInstaller/CustomActions/Flare.cs
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,8 @@ public void Send(string email)
{ "version", agentVersion },
{ "log", log },
{ "email", email },
{ "apikey", apikey }
{ "apikey", apikey },
{ "variant", "windows_installer" }
};

_client.Post(uri, payload, new Dictionary<string, string>());
Expand Down

0 comments on commit 2cacfc1

Please sign in to comment.