Skip to content

chore: regenerate dps control plane sdk on 2026-06-01-preview#848

Merged
cheatsheet1999 merged 1 commit into
yuelinzhao/DPS-RBAC-supportfrom
yuelinzhao/generate-dps-sdk-0601
Jun 30, 2026
Merged

chore: regenerate dps control plane sdk on 2026-06-01-preview#848
cheatsheet1999 merged 1 commit into
yuelinzhao/DPS-RBAC-supportfrom
yuelinzhao/generate-dps-sdk-0601

Conversation

@cheatsheet1999

@cheatsheet1999 cheatsheet1999 commented Jun 30, 2026

Copy link
Copy Markdown
Member

Summary

Generate IoT DPS control plane (mgmt) SDK on API version 2026-06-01-preview. The new API Version includes

"disableLocalAuth": {
          "type": "boolean",
          "description": "Disables all authentication methods other than Azure RBAC.",
          "default": false
}

for feature to use

Changes

  • Auto-generated (TypeSpec): everything under  azext_iot/sdk/dps/mgmt/  was regenerated via the TypeSpec
  • Manual (not generated) - 2 files:
      - azext_iot/_factory.py : changed the  IotDpsClient(...)  call from  endpoint=  to  base_url= .
      - azext_iot/tests/test_factory_unit.py : updated the matching assertion ( endpoint  ->  base_url ).

Why  _factory.py  had to change

The regenerated  IotDpsClient  renamed its constructor parameter from  endpoint  to  base_url  (and added  cloud_setting ). Since the constructor also takes  **kwargs , leaving endpoint  does not raise - it's silently dropped, and  base_url  falls back to the default public ARM endpoint. The client would then ignore  _get_arm_endpoint(cli_ctx) and always target public-cloud ARM, breaking sovereign clouds (Fairfax/Mooncake) and the canary endpoint used for preview testing. Switching to  base_url=  restores correct endpoint resolution (verified:  endpoint=  ->  management.azure.com ;  base_url=  -> canary).

Integration test result

image

@cheatsheet1999 cheatsheet1999 changed the title chore: regenerate dps sdk on 2026-06-01-preview chore: regenerate dps control plane sdk on 2026-06-01-preview Jun 30, 2026
@cheatsheet1999 cheatsheet1999 marked this pull request as ready for review June 30, 2026 05:34
@cheatsheet1999 cheatsheet1999 requested a review from digimaun as a code owner June 30, 2026 05:34
@ketkimnaik

Copy link
Copy Markdown
Contributor

Have we run integration tests for it to make sure no regressions?

@cheatsheet1999

Copy link
Copy Markdown
Member Author

Have we run integration tests for it to make sure no regressions?

Yes, let me run it one more time and attach a result screenshot. Thanks for the remainder.

@cheatsheet1999

Copy link
Copy Markdown
Member Author

@HangyiWang @ketkimnaik , updated int test result, will merge it in a moment. Thank you!

@cheatsheet1999 cheatsheet1999 merged commit 1f39d36 into yuelinzhao/DPS-RBAC-support Jun 30, 2026
46 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.

4 participants