Skip to content

Add force replacement for SqlResourceSqlRoleAssignment#3708

Closed
otahirs wants to merge 2 commits into
pulumi:masterfrom
otahirs:patch-1
Closed

Add force replacement for SqlResourceSqlRoleAssignment#3708
otahirs wants to merge 2 commits into
pulumi:masterfrom
otahirs:patch-1

Conversation

@otahirs

@otahirs otahirs commented Nov 16, 2024

Copy link
Copy Markdown
Contributor

force replacement for documentdb.SqlResourceSqlRoleAssignment if principalId or scope changes

fix #3701

error: Code="BadRequest" Message="Updating SQL Role Assignment Principal ID is not permitted. You may only update the associated Role Definition.\r\nActivityId: 8a163189-0c65-45c0-9969-d09f2047f41d, Microsoft.Azure.Documents.Common/2.14.0"

@github-actions

Copy link
Copy Markdown
Contributor

Hi there 👋 Thank you for submitting a pull request.

Acceptance tests must be run by a maintainer. If a maintainer hasn't picked this up after 7 days, please do reach out to a maintainer via our community Slack.

Maintainers: Please review changes then run tests by adding the comment:

/run-acceptance-tests

@thomas11

Copy link
Copy Markdown
Contributor

Awesome, thank you for this contribution @otahirs!

Were you able to confirm locally that the change has the intended effect? If not you can build via make provider and then run PATH=$your_clone/bin:$PATH pulumi up.

@otahirs

otahirs commented Nov 16, 2024

Copy link
Copy Markdown
Contributor Author

you can build via make provider and then run PATH=$your_clone/bin:$PATH pulumi up.

thanks for the guidance. I can now confirm that after the change the role gets successfully replaced.

@thomas11 thomas11 added resolution/duplicate This issue is a duplicate of another issue and removed resolution/duplicate This issue is a duplicate of another issue labels Nov 18, 2024
@thomas11

Copy link
Copy Markdown
Contributor

Superceded by #3711 due to CI permission issues. The commit will still have you as author.

@thomas11 thomas11 closed this Nov 18, 2024
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.

change of principalId in documentdb:SqlResourceSqlRoleAssignment fails to update resource

2 participants