Skip to content

[test] migrate network#48609

Draft
weidongxu-microsoft wants to merge 29 commits intoAzure:mainfrom
weidongxu-microsoft:migrate-network
Draft

[test] migrate network#48609
weidongxu-microsoft wants to merge 29 commits intoAzure:mainfrom
weidongxu-microsoft:migrate-network

Conversation

@weidongxu-microsoft
Copy link
Copy Markdown
Member

@weidongxu-microsoft weidongxu-microsoft commented Mar 27, 2026

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

weidongxu-microsoft and others added 26 commits March 19, 2026 14:39
@weidongxu-microsoft
Generated network SDK from TypeSpec (compile errors remaining: InnerT…
e4770ff 
… bounds)

DdosProtectionPlanInner and RouteFilterInner extend ResourceWithReadOnlyID/ResourceWithRequiredLocation
instead of Resource, causing InnerT bounds mismatch with hand-written wrappers.
These need customization class to fix.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Add customization for DdosProtectionPlan/RouteFilter InnerT bounds an…
77b8856 
…d update module-info

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Enhance customization: SubnetInner→SubResource, RouteFilterInner with…
63590a3 
…Id removal

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix customization: SubResourceModel extends SubResource, remove withI…
b5b16ec 
…d/withName overrides

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix customization: no-op withId/withName, remove SubResourceModel change
e00791e 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Generated network SDK - 9 remaining type incompatibilities
5a5e834 
Remaining issues:
- SubnetInner (SubResourceModel) not assignable to SubResource (5 errors)
- StartFinalResult/StopFinalResult not convertible to Void (2 errors)
- GenerateVpnProfileFinalResult not convertible to String (2 errors)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Merge branch 'main' into migrate-network
fa97baa
@weidongxu-microsoft
Remove invalid @OverRide annotations from generated Resource subclass…
389b291 
…es via customization

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix hand-written wrapper code and regenerate network SDK
0545871 
- Fix SubnetInner/SubResource type mismatch in LoadBalancerFrontendImpl and NicIpConfigurationBaseImpl
- Fix generateVpnProfile return type (ArmAcceptedLroResponse -> two-step with getVpnProfilePackageUrl)
- Fix PacketCaptureImpl createResourceAsync return type mismatch
- Regenerate SDK with @@Usage and HTTP casing fixes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with BastionHostIpConfiguration rename
9c9071d 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix javadoc @inheritdoc warnings in customization and test SubnetInne…
46b6a6b 
…r fix

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix checkstyle: add @param and @return tags to withId/withName Javadoc
39b26a2 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with Javadoc fix applied
d512093 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with IP/BGP/IDPS/SNAT/SQL/JWT/LOA casing fixes and float32…
c29e74f 
…-as-double

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with property-level casing fixes
82a5f7e 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
revapi
cb59f97
@weidongxu-microsoft
Regenerate with fluent model renames and CustomIpPrefixes fix
23e311b 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with privateIpAddress fix and oldSuperType suppressions
071e27a 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Regenerate with unified ProvisioningState
7c2a2e5 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Suppress tagMap return type change in revapi
e5e2651 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Fix tagMap suppression format to use regex matcher
33f6e94 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Revert PacketCaptureImpl to main - specs fixed createAsync return type
4d628a7 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Resolve merge conflict in revapi.json - keep main's storage entries, …
3d5d501 
…append network

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
Revert VirtualNetworkGatewayImpl to main - specs fixed LRO return typ…
aa52065 
…es to String

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weidongxu-microsoft
regen
9c2ad69
@weidongxu-microsoft
revert main
2dcced0
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JonathanGiles JonathanGiles Awaiting requested review from JonathanGiles JonathanGiles will be requested when the pull request is marked ready for review JonathanGiles is a code owner

@alzimmermsft alzimmermsft Awaiting requested review from alzimmermsft alzimmermsft will be requested when the pull request is marked ready for review alzimmermsft is a code owner

@srnagar srnagar Awaiting requested review from srnagar srnagar will be requested when the pull request is marked ready for review srnagar is a code owner

@rujche rujche Awaiting requested review from rujche rujche will be requested when the pull request is marked ready for review rujche is a code owner

@Netyyyy Netyyyy Awaiting requested review from Netyyyy Netyyyy will be requested when the pull request is marked ready for review Netyyyy is a code owner

@saragluna saragluna Awaiting requested review from saragluna saragluna will be requested when the pull request is marked ready for review saragluna is a code owner

@moarychan moarychan Awaiting requested review from moarychan moarychan will be requested when the pull request is marked ready for review moarychan is a code owner

@haolingdong-msft haolingdong-msft Awaiting requested review from haolingdong-msft haolingdong-msft will be requested when the pull request is marked ready for review haolingdong-msft is a code owner

@XiaofeiCao XiaofeiCao Awaiting requested review from XiaofeiCao XiaofeiCao will be requested when the pull request is marked ready for review XiaofeiCao is a code owner

@ArthurMa1978 ArthurMa1978 Awaiting requested review from ArthurMa1978 ArthurMa1978 will be requested when the pull request is marked ready for review ArthurMa1978 is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Mgmt This issue is related to a management-plane library.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@weidongxu-microsoft