Skip to content

workaround - ManagedDevopsPools identity field#5242

Merged
sreallymatt merged 4 commits intohashicorp:mainfrom
QixiaLu:add-workaround-adopool-identity
Mar 12, 2026
Merged

workaround - ManagedDevopsPools identity field#5242
sreallymatt merged 4 commits intohashicorp:mainfrom
QixiaLu:add-workaround-adopool-identity

Conversation

@QixiaLu
Copy link
Copy Markdown
Contributor

@QixiaLu QixiaLu commented Mar 2, 2026

Community Note

  • Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
  • Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

Add workaround for Azure/azure-rest-api-specs#40905 where wrong type is used for identity

PR Checklist

  • I have checked to ensure there aren't other open Pull Requests for the same update/change.
  • I have checked if my changes close any open issues. If so, please include appropriate closing keywords below.
  • I have used a meaningful PR title to help maintainers and other users understand this change and help prevent duplicate work.

Changes to Resource Config

Related Issue(s)

AI Assistance Disclosure

  • AI Assisted - This contribution was made by, or with the assistance of, AI/LLMs

Note

If this PR changes meaningfully during the course of review, please update the title and description as required.

@QixiaLu QixiaLu marked this pull request as ready for review March 3, 2026 01:06
Copy link
Copy Markdown
Contributor

@sreallymatt sreallymatt left a comment

Choose a reason for hiding this comment

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

Thanks @QixiaLu - just one small suggestion

@QixiaLu QixiaLu requested a review from sreallymatt March 12, 2026 01:14
@sreallymatt sreallymatt force-pushed the add-workaround-adopool-identity branch from 7db3093 to 5184a8e Compare March 12, 2026 14:29
Copy link
Copy Markdown
Contributor

@sreallymatt sreallymatt left a comment

Choose a reason for hiding this comment

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

Thanks @QixiaLu - LGTM ✅ (note: I've rebased this PR to resolve conflicts with main)

@sreallymatt sreallymatt merged commit 6f6463d into hashicorp:main Mar 12, 2026
4 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.

2 participants