Skip to content

Upgrade TrafficManager SDK to API version 2024-04-01-preview - #29711

Merged
Thomas Temby (thomas-temby) merged 3 commits into
Azure:Az.TrafficManager-previewfrom
ransinha1989:TrafficManager-Preview-Powershell-Sdk
Jun 9, 2026
Merged

Upgrade TrafficManager SDK to API version 2024-04-01-preview#29711
Thomas Temby (thomas-temby) merged 3 commits into
Azure:Az.TrafficManager-previewfrom
ransinha1989:TrafficManager-Preview-Powershell-Sdk

Conversation

@ransinha1989

@ransinha1989 ransinha1989 commented May 29, 2026

Copy link
Copy Markdown
Member
  • Regenerate TrafficManager.Management.Sdk from 2024-04-01-preview swagger
  • Fix AutoRest README.md directive to use name-based parameter selector
  • Update test recordings for new API version (RequestUri and EncodedRequestUri)
  • Fix RecordType mapping in TrafficManagerClient.GetPowershellTrafficManagerProfile
  • Add RecordType round-trip test (TestProfileCrudWithRecordType)
  • Add RecordType immutability test (TestProfileChangeRecordTypeShouldFail)
  • All 50 tests passing

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

- Regenerate TrafficManager.Management.Sdk from 2024-04-01-preview swagger
- Fix AutoRest README.md directive to use name-based parameter selector
- Update test recordings for new API version (RequestUri and EncodedRequestUri)
- Fix RecordType mapping in TrafficManagerClient.GetPowershellTrafficManagerProfile
- Add RecordType round-trip test (TestProfileCrudWithRecordType)
- Add RecordType immutability test (TestProfileChangeRecordTypeShouldFail)
- All 50 tests passing

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-client-tools-bot-prd

Copy link
Copy Markdown
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@thomas-temby

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

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

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

Comment thread src/TrafficManager/.github/copilot-instructions.md Outdated
Comment thread src/TrafficManager/TrafficManager.Management.Sdk/README.md Outdated
Comment thread src/TrafficManager/TrafficManager.Management.Sdk/README.md Outdated
Comment thread src/TrafficManager/TrafficManager.Management.Sdk/README.md Outdated
Comment thread src/TrafficManager/TrafficManager/Models/TrafficManagerProfile.cs Outdated
- Remove copilot-instructions.md from commit
- Remove comment after commit hash in README.md
- Remove trailing whitespace in README.md directive
- Remove commented-out input-file line in README.md
- Remove redundant RecordType null-check (already set in constructor)

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

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

@thomas-temby
Thomas Temby (thomas-temby) merged commit ed910c8 into Azure:Az.TrafficManager-preview Jun 9, 2026
12 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