Skip to content

[MySQL] Fix flexible server AD admin creation - #33838

Open
Aditya Pujara (a0x1ab) with Copilot wants to merge 2 commits into
devfrom
copilot/fix-ad-admin-creation-error
Open

[MySQL] Fix flexible server AD admin creation#33838
Aditya Pujara (a0x1ab) with Copilot wants to merge 2 commits into
devfrom
copilot/fix-ad-admin-creation-error

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Related command

az mysql flexible-server ad-admin create

Description

AD admin creation fails after the MySQL Flexible Server SDK upgrade because plain dictionaries no longer receive SDK REST serialization.

  • Use AzureADAdministrator for the request payload.
  • Restores nested properties serialization and REST field names:
models.AzureADAdministrator(
    tenant_id=get_tenant_id(),
    identity_resource_id=identity
)

Testing Guide

Run az mysql flexible-server ad-admin create with a user-assigned identity.

History Notes

[MySQL] az mysql flexible-server ad-admin create: Fix AD admin creation


@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI linked an issue Aug 4, 2026 that may be closed by this pull request
@microsoft-github-policy-service microsoft-github-policy-service Bot added Auto-Assign Auto assign by bot ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group act-identity-squad Managed Identity For `az identity` only Backup act-observability-squad labels Aug 4, 2026
@azure-client-tools-agent
azure-client-tools-agent Bot marked this pull request as ready for review August 4, 2026 09:52
Copilot AI lite review requested due to automatic review settings August 4, 2026 09:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: a0x1ab <59631311+a0x1ab@users.noreply.github.com>
Copilot AI requested a review from a team as a code owner August 4, 2026 09:58
Copilot AI changed the title [WIP] Fix internal server error when creating AD Admin on MySQL Flexible Server [MySQL] Fix flexible server AD admin creation Aug 4, 2026
@azure-client-tools-agent

Copy link
Copy Markdown

Live test skipped

⏭️ Skipping the live test for this revision because no new test file was added (tests/**/test_*.py).

The live-test pipeline runs only the test files a PR changes, so there is nothing to execute for this commit. This is informational — a regression test is encouraged where it makes sense, but not required. If a test file is added in a later commit, the live test will run automatically.


Posted by agent-assist (autonomous bug-fix pipeline).

@yonzhan

Copy link
Copy Markdown
Collaborator

MySQL

@a0x1ab

Copy link
Copy Markdown
Member

Copilot please review and implement all unaddressed human feedback through Yong Zhang (@yonzhan)'s latest comment at #33838 (comment), then push the fix to this PR.

Human review feedback to implement:
MySQL


Posted by agent-assist (autonomous bug-fix pipeline).

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Copilot please review and implement all unaddressed human feedback through Yong Zhang (@yonzhan)'s latest comment at #33838 (comment), then push the fix to thi...

The referenced human comment contains only “MySQL” and does not specify a requested change. Please provide the concrete feedback to implement.

@a0x1ab

Copy link
Copy Markdown
Member

Copilot please review and implement all unaddressed human feedback through Yong Zhang (@yonzhan)'s latest comment at #33838 (comment), then push the fix to this PR.

Human review feedback to implement:
MySQL


Posted by agent-assist (autonomous bug-fix pipeline).

@a0x1ab

Copy link
Copy Markdown
Member

Copilot please review and implement all unaddressed human feedback through Yong Zhang (@yonzhan)'s latest comment at #33838 (comment), then push the fix to this PR.

Human review feedback to implement:
MySQL


Posted by agent-assist (autonomous bug-fix pipeline).

@azure-client-tools-agent azure-client-tools-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot finished without pushing a fix after 3 automated attempts for the feedback at #33838 (comment). Leaving this PR for manual follow-up.


Posted by agent-assist (autonomous bug-fix pipeline).

@a0x1ab Aditya Pujara (a0x1ab) added agent-assist On-demand trigger for the agent-assist autonomous bug-fix pipeline azure-client-tools-agent Pull request commented on or reviewed by Azure Client Tools Agent and removed agent-assist On-demand trigger for the agent-assist autonomous bug-fix pipeline labels Aug 6, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

🔔 Routing this PR to @Azure/act-codegen-extensibility-squad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

act-codegen-extensibility-squad ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group Auto-Assign Auto assign by bot azure-client-tools-agent Pull request commented on or reviewed by Azure Client Tools Agent Backup Managed Identity For `az identity` only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Release 2.89.0 breaks creating an AD Admin on a MySQL Flexible Server

5 participants