Skip to content

fix(operate): version bump + factual corrections for PR #1637 - #2025

Merged
Paul Yuknewicz (paulyuk) merged 1 commit into
microsoft:pass-equity-gap-3from
simonjj:fix/pr-1637-operate
Apr 24, 2026
Merged

fix(operate): version bump + factual corrections for PR #1637#2025
Paul Yuknewicz (paulyuk) merged 1 commit into
microsoft:pass-equity-gap-3from
simonjj:fix/pr-1637-operate

Conversation

@simonjj

Copy link
Copy Markdown
Contributor

Fixes applied to PR #1637 (Container Apps Operate)

5 corrections across 3 files:

SKILL.md

  • Version bump 1.1.8 → 1.1.9

networking.md

  1. IP restriction mixing — removed incorrect warning that Allow and Deny rules cannot be mixed; they CAN and are evaluated by priority
  2. Subnet delegation — corrected to show \Microsoft.App/environments\ delegation is required for BOTH workload-profiles AND consumption-only environments
  3. Fully-private topology — split into two sub-modes (VNet-wide vs env-only) for accuracy

revisions.md

  1. First-deployment tip — clarified that the first deployment creates the initial revision automatically

Model-Matrix Test Results — 15/15 PASS

Model Cosmos RBAC KEDA Identity KQL+Redis
GPT-5 mini
GPT-5.3-Codex
Claude Haiku 4.5
Claude Sonnet 4.6
Claude Opus 4.6

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

- Bump azure-prepare version 1.1.8 → 1.1.9 (fixes Skill Structure CI check)
- Fix incorrect claim that Allow/Deny IP rules cannot be mixed
- Fix subnet delegation for consumption-only environments (requires Microsoft.App/environments)
- Clarify 'fully private' topology: VNet-wide vs env-only based on ingress external flag
- Clarify first-deployment revision tip

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the azure-prepare skill’s Container Apps “operate” reference docs with factual corrections and bumps the skill version to reflect the change set.

Changes:

  • Bumped azure-prepare skill version 1.1.8 → 1.1.9.
  • Corrected Container Apps networking guidance (subnet delegation, IP restriction behavior, and clarified fully-private topology modes).
  • Clarified revisions guidance for first deployment behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
plugin/skills/azure-prepare/SKILL.md Patch version bump to reflect reference updates.
plugin/skills/azure-prepare/references/services/container-apps/networking.md Corrected/clarified VNet delegation, IP restriction evaluation, and fully-private topology rows.
plugin/skills/azure-prepare/references/services/container-apps/revisions.md Clarified first-deployment revision tip wording.

@paulyuk

Copy link
Copy Markdown
Member

Thanks Simon — merging this! Subnet delegation fix and the fully-private topology split are spot on.

One heads-up: JonG flagged that the priority field on ipSecurityRestrictions doesn't exist in the Container Apps API (2024-03-01) — rules are evaluated in array order, not by priority. Your warning text update references priority-based evaluation, so I'll adjust that in a follow-up commit along with removing the priority: 100/200 fields from the Bicep snippet.

Remaining JonG items (8 total) I'll fix directly on the branch after merge.

@paulyuk
Paul Yuknewicz (paulyuk) merged commit 79e80cd into microsoft:pass-equity-gap-3 Apr 24, 2026
6 of 7 checks passed
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.

3 participants