Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

exclude RoleDefinitionID and RoleDefinitionName from RoleAssignment l… #201

Open
wants to merge 1 commit into
base: release-0.12-preview
Choose a base branch
from

Conversation

ctkeyser
Copy link

this fixes a similar issue to the one described here #106 and fixed here #107

when creating a RoleAssignment and filling in either roleDefinitionId or roleDefinitionName, the opposite field is automatically filled in and results in the error

Warning  CannotObserveExternalResource  25s (x3 over 34s)  managed/authorization.azure.jet.crossplane.io/v1alpha1, kind=roleassignment  cannot run refresh: refresh failed: Conflicting configuration arguments: "role_definition_id": conflicts with role_definition_name: File name: main.tf.json
Conflicting configuration arguments: "role_definition_name": conflicts with role_definition_id: File name: main.tf.json

i would have opened against main branch, but the roleassignment files are missing there (this one for example https://github.com/crossplane-contrib/provider-jet-azure/blob/release-0.12-preview/apis/authorization/v1alpha1/zz_roleassignment_types.go). hopefully it's ok to open against this branch

@ctkeyser ctkeyser force-pushed the release-0.12-preview branch from ed6b5a9 to f710c82 Compare August 17, 2022 14:51
…ate-initialization

Signed-off-by: Casey Keyser <ctkeyser@gmail.com>
@ctkeyser ctkeyser force-pushed the release-0.12-preview branch from f710c82 to a37e632 Compare August 17, 2022 14:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants