Harden deployer private networking and add optional AMPLS for Commercial and Government
维护者通常 1 天内回复
@paullizer 已经在做这个了。
开始于 2026年10月14日。
评估
这个 Issue 还没有评估数据。
描述
Summary
Close private-networking gaps in SimpleChat deployers and provide an administrator opt-in for Azure Monitor Private Link Scope (AMPLS) in Azure Commercial and Azure Government.
Original Gaps
- Redis deployment does not create a private endpoint; Azure Managed Redis is explicitly left public even with private networking enabled.
- Video Indexer private endpoints are associated with the generic Cognitive Services DNS zone instead of privatelink.api.videoindexer.ai in Commercial.
- Application Insights and Log Analytics do not have an opt-in AMPLS deployment path.
- Government Key Vault uses privatelink.vaultcore.azure.us instead of privatelink.vaultcore.usgovcloudapi.net.
Implemented Behavior
- Create Redis private endpoints and correct cloud/offering-specific DNS when private networking is enabled. Preserve bootstrap access needed for deployment configuration, then close public access only after private connectivity is available.
- Use a dedicated Video Indexer DNS zone only in clouds/API profiles that support its private endpoint.
- Add a default-off deployment option for AMPLS, requiring private networking. Connect deployed Application Insights and Log Analytics, create one AMPLS private endpoint, and configure Commercial or Government DNS zones with existing-zone reuse support.
- Document subnet sizing, DNS forwarding, deployment-runner connectivity, and the effect of private-only monitoring on other resources sharing DNS.
Acceptance Criteria
- Managed and classic Redis private endpoints use the correct resource type, group ID, and DNS zones; public deployments remain unchanged.
- Commercial Video Indexer uses privatelink.api.videoindexer.ai; unsupported Government private-endpoint creation remains disabled.
- Government Key Vault DNS is corrected in each supported deployer.
- Administrators can opt into AMPLS in either cloud; default-off and invalid combinations are covered.
- AMPLS connects Application Insights and Log Analytics, supports correct cloud DNS and existing-zone reuse, and accounts for multiple private IPs.
- Focused regression tests and IaC validation pass; docs and deployer version are updated.
Notes
Primary areas: deployers/bicep/main.bicep, modules/privateNetworking.bicep, modules/privateDNSZones.json, modules/redisCache.bicep, AZD parameter and lifecycle wiring; equivalent Terraform/Azure CLI surfaces as applicable.
Microsoft DNS reference: https://learn.microsoft.com/azure/private-link/private-endpoint-dns
AMPLS reference: https://learn.microsoft.com/azure/azure-monitor/fundamentals/private-link-configure
Priority: P1. Size: L. Assignee: paullizer.
Implementation Update
Implementation is committed and ready for review in PR #1767: https://github.com/microsoft/simplechat/pull/1767
- Head:
paullizer/private-networking-ampls-1765; base:paullizer-react-v2-ui. Only the feature branch was pushed. The base branch was not directly pushed or merged. - Commit: https://github.com/microsoft/simplechat/commit/62d4b0210c2e052d7903c8f56e2ef3685e1bdc1c
- Application version 0.261.333; deployer 1.0.34.
- Redis private endpoints and final AZD lockdown implemented, Commercial Video Indexer DNS corrected, and Government Key Vault DNS corrected across deployers.
- Optional AMPLS added for both clouds in Bicep/AZD, Terraform, and CLI. Default off, with private-only monitoring, per-zone reuse, shared Blob DNS, and separate Government overflow-zone handling.
- Updated both Visio files in
artifacts/, covering ACR, Key Vault, platform subnet, compact subnet sizing, monitoring DNS, and cloud-specific service/Redis labels. Added a diagram companion and Commercial/Government fresh deployment versus compact existing-network tables. - Release notes include the implementation and the wider Visio updates. Regenerated previously stale release pages without losing any of the 304 historical version sections.
Validation
- 19 focused tests passed across private DNS/AMPLS wiring, Video Indexer region/permissions, and AZD runner-IP prerequisites.
- Bicep compiles and both generated ARM templates match fresh builds.
- Terraform 1.12.2 validation and networking formatting passed using locked providers and no backend.
- CLI and all four Windows AZD hooks parse; mocked CLI tests cover failure/success sequencing and existing endpoint DNS reconciliation.
- Documentation coverage 7/7 and site-quality checks 6/6 passed.
- Release-note integrity confirms all 304 sections; generator freshness and staged whitespace checks passed.
Remaining Review And Deployment Checks
- No live Azure deployment was performed. Validate private DNS, Redis connectivity, monitoring ingestion, and queries in the intended cloud.
- Government's Visio page still contains the Commercial Video Indexer private-zone label. Review/correct that diagram label; the Government API profile intentionally does not provision a Video Indexer PE.
- Separate
azd provision/azd deployworkflows require equivalent final Redis lockdown because they do not runpostup; reprovisioning reopens Managed Redis bootstrap access until lockdown runs again. - The compact five-subnet enterprise reference requires explicit endpoint placement/customization; existing-VNet reuse still supplies one shared PE subnet.
This issue remains open pending PR review and deployment validation.
- 主要语言
- Python
- 星标
- 155
- 派生
- 116
- 平均合并
- 5 小时 3 分钟
- 30 天内合并 PR
- 206
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
microsoft/simplechat 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
microsoft/simplechat#1491 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
microsoft/simplechat#1370 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 8/100
microsoft/simplechat#1770 ·
维护者通常 1 天内回复
-
V2 bootstrap takes ~5s: optimize /api/v2/bootstrap backend and first paint可能已有人在做 @paullizer 于 8 天前认领。 未关闭enhancement P1 python
microsoft/simplechat#1725 · 已指派 1 人 ·
维护者通常 1 天内回复
-
Serve the new UI at the main URLs, move the legacy UI under /legacy, and let admins control interface access可能已有人在做 @paullizer 于 8 天前认领。 未关闭Created By Copilot enhancement P1
microsoft/simplechat#1723 · 已指派 1 人 ·
维护者通常 1 天内回复
查看 microsoft/simplechat 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 74/100
cellrishi-code/claude-skills-atlas#165 ·
维护者通常 1 天内回复
-
bug confirmed issue
难度 2/5 1-3 小时 新手友好度 72/100
open-webui/open-webui#32635 · 1 条评论 ·
维护者通常 1 天内回复
-
Filters: ?field:ne=null matches NULL rows instead of non-NULL ones可能已有人在做 @rshdhere 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
Netflix/metaflow-service#504 ·
-
needs-triage
难度 2/5 1-3 小时 新手友好度 84/100
gke-labs/kube-agents#3394 ·
维护者通常 1 天内回复
-
A2A audit timestamps end in +00:00Z (invalid ISO 8601)可能已有人在做 @O1sumitkumar 今天认领。 未关闭bug good first issue help wanted python
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复