Skip to content

Conversation

@shuagarw
Copy link
Contributor

Role commands bugbash fixes

@azurecla
Copy link

Hi @shuagarw, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (shuagarw). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@ogail
Copy link
Contributor

ogail commented Oct 21, 2015

Is it guranteed that the name will match with one role def only?

@shuagarw
Copy link
Contributor Author

Name can match with multiple role definitions(edge case), but when deleting we ensure that if only 1 role is matched, then only delete, otherwise an error is thrown asking user to specify role by Id, as multiple roles with same name are found.
So yes, when deleting a role, name will match to 1 role def only, otherwise it will be an error.

ogail added a commit that referenced this pull request Oct 21, 2015
@ogail ogail merged commit 50c3fc7 into Azure:dev Oct 21, 2015
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