Skip to content

fix: align custom application hooks and private validation fixtures - #773

Merged
Paulo Lacerda (placerda) merged 19 commits into
developfrom
feature/refresh-private-network-hook-fixtures
Oct 9, 2026
Merged

Paulo Lacerda (placerda) merged 19 commits into
developfrom
feature/refresh-private-network-hook-fixtures

Conversation

@placerda

@placerda Paulo Lacerda (placerda) commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Changes

  • Normalize directory/file application-definition selections in PowerShell and Bash.
  • Deploy each declared service from the definition folder's azure.yaml; retain source-commit verification in its source folder and reuse the azd environment.
  • Select the declared hosted greeting protocol and explicitly target the service. Validate raw Responses JSON/SSE without weakening bundled Invocations completion checks.
  • Make the hosted sample return a completed assistant message.
  • Narrow VM SKU preflight requests to virtualMachines.
  • Preserve the ACR Task pool guard; positive fixtures provide a pool and negative cases cover missing pools and local builds.

Evidence

  • python -m pytest -q tests: 386 passed, 4 skipped, 462 subtests passed.
  • python -m unittest discover -s tests: 355 passed, 2 skipped.
  • Infrastructure preflight: 73 of 75 assertions passed; both failures also reproduce on unchanged HEAD (empty-Boolean parameter expansion). The two new SKU assertions pass.
  • The consolidation merge preserved the exact tested source tree.
  • Central docs: docs: clarify custom application project and smoke contracts AI-Landing-Zones#148.

Azure acceptance status

These fixes do not retroactively validate v4.2.2. T035/T085/T098/T093 remain open. Classic foundation provisioned but post-provision governance failed because public Key Vault access was disabled and no private endpoint exists for that vault. Hosted retry and private-foundation evaluation continue separately. No security policy was disabled.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Resolve directory selections in both lifecycle variants, deploy named services from the application definition project, and validate hosted greetings using the declared protocol. Narrow VM SKU preflight queries to virtual machines.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@placerda Paulo Lacerda (placerda) changed the title test: align private network hook fixtures with ACR pool guard fix: align custom application hooks and private validation fixtures Oct 9, 2026
Paulo Lacerda (placerda) and others added 15 commits October 9, 2026 08:55
Correct sample paths and binding negative checks; distinguish runtime failures from offline regression evidence.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Share conservative existing-allocation verification across both preflight gates, bind discovery to the selected environment, and retain availability and new-capacity checks.

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

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Discover actual Container App client IDs when no explicit identity map is supplied. Preserve the map override and fail before publication for unresolved identities. Record live private publication and repeated child runtime evidence.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Publish enabled foundation outputs, recover verified quota credit for older blank outputs, and reconcile declared hosted profiles against the routed runtime identity.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@placerda
Paulo Lacerda (placerda) merged commit 61ae8f7 into develop Oct 9, 2026
4 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.

1 participant