Skip to content

Remove dead dn-bot-dnceng-build-rw-code-rw-release-rw vault entry (WI 10134)#6485

Merged
mmitche merged 1 commit intomainfrom
dev/copilot/remove-dead-build-rw-code-rw-release-rw-10134
Mar 26, 2026
Merged

Remove dead dn-bot-dnceng-build-rw-code-rw-release-rw vault entry (WI 10134)#6485
mmitche merged 1 commit intomainfrom
dev/copilot/remove-dead-build-rw-code-rw-release-rw-10134

Conversation

@missymessa
Copy link
Member

Summary

Removes the dn-bot-dnceng-build-rw-code-rw-release-rw PAT definition from dotneteng-status-secrets.yaml.

Investigation

This PAT (scopes: build_execute, code_write, release_execute) is defined in the secret-manager manifest but never consumed by the dotneteng-status web app:

  • settings.json has no [vault(dn-bot-dnceng-build-rw-code-rw-release-rw)] reference
  • The app configures only two AzDO clients:
  • Code search across all repos in dnceng/internal confirmed zero [vault(...)] consumers

Change

Remove the dead vault entry from .vault-config/shared/dotneteng-status-secrets.yaml to stop unnecessary PAT generation/rotation by the secret manager.

Related

… 10134)

This PAT is defined in the secret-manager manifest but never consumed
by the dotneteng-status app. settings.json has no [vault(...)] reference
to this secret. The app only uses:
- dn-bot-dnceng-workitems-rw (work item ops, migrated in WI 10113)
- dn-bot-dnceng-build-r-code-r-project-r-profile-r (build monitor, WI 10136)

Removing the dead entry stops unnecessary PAT generation/rotation.
@mmitche mmitche merged commit d0ff328 into main Mar 26, 2026
5 of 6 checks passed
@mmitche mmitche deleted the dev/copilot/remove-dead-build-rw-code-rw-release-rw-10134 branch March 26, 2026 14:37
missymessa added a commit that referenced this pull request Mar 26, 2026
Both dn-bot-dnceng-workitems-rw (this PR) and
dn-bot-dnceng-build-rw-code-rw-release-rw (PR #6485) are removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants