From 35e00100c2865918ea217c196c687a60b333ec4c Mon Sep 17 00:00:00 2001 From: Chidozie Ononiwu Date: Thu, 7 Oct 2021 12:39:09 -0700 Subject: [PATCH 1/2] Fix up identity resolver to make it more reliable, publish notification configuration as a tool --- eng/common/pipelines/templates/steps/get-pr-owners.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/steps/get-pr-owners.yml b/eng/common/pipelines/templates/steps/get-pr-owners.yml index bf2ee00ffc9e4..6e8a1af3d2083 100644 --- a/eng/common/pipelines/templates/steps/get-pr-owners.yml +++ b/eng/common/pipelines/templates/steps/get-pr-owners.yml @@ -17,7 +17,7 @@ steps: --kusto-url-var KUSTO_URL ` --kusto-database-var KUSTO_DB ` --kusto-table-var KUSTO_TABLE ` - --identity "$(Build.QueuedBy)" ` + --identity "$(Build.RequestedForEmail)" ` --targetvar "${{ parameters.TargetVariable }}" displayName: 'Resolving Queuing User' continueOnError: true From bf87e462fbd8e976394b259210a60d4ea9feb126 Mon Sep 17 00:00:00 2001 From: Chidozie Ononiwu Date: Thu, 14 Oct 2021 13:01:02 -0700 Subject: [PATCH 2/2] Use both username and email to resolve queuing user --- eng/common/pipelines/templates/steps/get-pr-owners.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/steps/get-pr-owners.yml b/eng/common/pipelines/templates/steps/get-pr-owners.yml index 6e8a1af3d2083..019bf5aa023e8 100644 --- a/eng/common/pipelines/templates/steps/get-pr-owners.yml +++ b/eng/common/pipelines/templates/steps/get-pr-owners.yml @@ -17,7 +17,8 @@ steps: --kusto-url-var KUSTO_URL ` --kusto-database-var KUSTO_DB ` --kusto-table-var KUSTO_TABLE ` - --identity "$(Build.RequestedForEmail)" ` + --identity-name "$(Build.QueuedBy)" ` + --identity-email "$(Build.RequestedForEmail)" ` --targetvar "${{ parameters.TargetVariable }}" displayName: 'Resolving Queuing User' continueOnError: true