Remove cloudtest notifications sent to lowdev#40046
Remove cloudtest notifications sent to lowdev#40046OneBlue wants to merge 6 commits intofeature/wsl-for-appsfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Azure Pipelines CloudTest job configuration to stop attributing scheduled build requests to the "lowdev" alias, aligning notifications/requester identity with the actual build requester.
Changes:
- Removed
scheduleBuildRequesterAlias: "lowdev"from the CloudTestServerBuildTask inputs. - Kept
notificationSubscribersset to$(Build.RequestedForEmail)so notifications go to the requesting user.
| - script: echo "Build requested for email - $(Build.RequestedForEmail)" | ||
| displayName: "Log build requested for email" |
There was a problem hiding this comment.
This step logs $(Build.RequestedForEmail) directly into build logs. Email addresses are PII and can end up retained/shared beyond the original audience of the notification. If the intent is troubleshooting, consider removing this log or masking the value (e.g., log only the domain or a hashed/obfuscated form), or use a non-PII identifier instead.
| - script: echo "Build requested for email - $(Build.RequestedForEmail)" | |
| displayName: "Log build requested for email" | |
| - script: echo "Build requested for - $(Build.RequestedFor)" | |
| displayName: "Log build requested for user" |
|
Hey @OneBlue 👋 — Following up on this draft PR. CI is green, but there's 1 unresolved review thread about a PII concern — \\\ being logged in the pipeline. Could you take a look at that? Is this draft still planned for merge? |
Summary of the Pull Request
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed