Skip to content

Fix filtered subnet ID expansion for existing VNets#3856

Merged
polatengin merged 14 commits into
mainfrom
polatengin/2159-the-arguments-for-filter-are-not-in-the-expected-format-or-type
Jul 7, 2026
Merged

Fix filtered subnet ID expansion for existing VNets#3856
polatengin merged 14 commits into
mainfrom
polatengin/2159-the-arguments-for-filter-are-not-in-the-expected-format-or-type

Conversation

@polatengin

@polatengin polatengin commented Jul 6, 2026

Copy link
Copy Markdown
Member

PR Summary

Fixes Bicep template expansion for filtered runtime resource property collections, including existing virtual network subnets.

This supports expressions like:

filter(vnet.properties.subnets, s => s.name == subnetName)[0].id

Closes #2159

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
  • Rule changes
    • Unit tests created/ updated
    • Rule documentation created/ updated
    • Link to a filed issue
    • Change log has been updated with change under unreleased section
  • Other code changes
    • Unit tests created/ updated
    • Link to a filed issue
    • Change log has been updated with change under unreleased section

polatengin added 13 commits June 17, 2026 23:12
@polatengin
Add handling for existing resources and subnet filtering in Functions
16bfa22
@polatengin
Add MockResourceProperties and related classes for enhanced resource …
c959ca8 
…property handling
@polatengin
Refactor filter handling to use IMockResourceCollection for improved …
ce2c82c 
…resource management
@polatengin
Enhance string binding and literal string handling in ExpressionHelpers
75b9049
@polatengin
Add test case for subnet filtering in function app deployment
818dc71
@polatengin
Add Bicep template for function app deployment with virtual network i…
2b3f71a 
…ntegration
@polatengin
Fix missing newline at end of file in Bicep test for function app dep…
56beaf5 
…loyment
@polatengin
Add output for vnetLiteralId to filter subnets by name
d0a67f3
@polatengin
Update Bicep template API version and metadata for subnet filtering
70a07f9
@polatengin
Add test case to validate subnet ID inference for existing VNet subnets
c4039a9
@polatengin
Add Bicep test case for Tests.Bicep.43.json with output directory pre…
4de8ec5 
…servation
@polatengin
Add support for resolving filtered subnet IDs from existing virtual n…
4eb1990 
…etworks during Bicep expansion
@polatengin
Merge remote-tracking branch 'origin/main' into polatengin/2159-the-a…
7b5ce75 
…rguments-for-filter-are-not-in-the-expected-format-or-type
@polatengin polatengin added this to the v2.0.0 milestone Jul 6, 2026
@polatengin polatengin self-assigned this Jul 6, 2026
@polatengin polatengin requested a review from a team as a code owner July 6, 2026 23:07
@polatengin polatengin added the enhancement New feature or request label Jul 6, 2026
@polatengin polatengin linked an issue Jul 6, 2026 that may be closed by this pull request
The arguments for 'Filter' are not in the expected format or type #2159
Closed
BernieWhite
BernieWhite previously approved these changes Jul 7, 2026
View reviewed changes

@BernieWhite BernieWhite left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me. Nice work @polatengin.

Comment thread docs/changelog.md Outdated
@polatengin polatengin merged commit 9859203 into main Jul 7, 2026
14 checks passed
@polatengin polatengin deleted the polatengin/2159-the-arguments-for-filter-are-not-in-the-expected-format-or-type branch July 7, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BernieWhite BernieWhite BernieWhite approved these changes

Assignees

@polatengin polatengin

Labels

enhancement New feature or request

Projects

None yet

Milestone

v2.0.0

Development

Successfully merging this pull request may close these issues.

The arguments for 'Filter' are not in the expected format or type

2 participants

@polatengin @BernieWhite