Skip to content

Add examples for getting secrets for VMs#56

Merged
BernieWhite merged 13 commits into
Azure:mainfrom
BernieWhite:bewhite/add-keyvault-secret-example
Jan 13, 2026
Merged

Add examples for getting secrets for VMs#56
BernieWhite merged 13 commits into
Azure:mainfrom
BernieWhite:bewhite/add-keyvault-secret-example

Conversation

@BernieWhite
Copy link
Copy Markdown
Collaborator

PR Summary

Add new examples for working with secrets in the pipeline and key vault.

PR Checklist

  • PR has a meaningful title
  • Summarized changes
  • Change is not breaking
  • This PR is ready to merge and is not Work in Progress

@BernieWhite BernieWhite requested a review from a team as a code owner March 11, 2025 03:48
@BernieWhite
Copy link
Copy Markdown
Collaborator Author

Please review @Tatsinnit. Thanks.

Copy link
Copy Markdown
Member

@Tatsinnit Tatsinnit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Thank you so much for the kind ping! I've added a few thoughts—though I'm not entirely sure about their full relevance, I do feel it might be helpful to note whether these files are generated by a tool or adapted from somewhere else.

Also, regarding the constants and GUIDs in use—it would be great to document the purpose behind them, especially since they may potentially be referenced by end users. Highlighting any possible downsides or considerations could be valuable too. 🙏

Comment thread modules/virtual-machine-windows/v1/.bicep/rbac.bicep
Comment thread modules/virtual-machine-windows/v1/main.bicep
@Tatsinnit Tatsinnit assigned BernieWhite and unassigned Tatsinnit Apr 4, 2025
@BernieWhite BernieWhite merged commit e93279a into Azure:main Jan 13, 2026
3 checks passed
@BernieWhite BernieWhite deleted the bewhite/add-keyvault-secret-example branch January 13, 2026 04:38
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