Skip to content

Composite roles: make it possible to clear them again#426

Draft
TomBillietKlarrio wants to merge 1 commit intocrossplane-contrib:mainfrom
klarrio-kpn-dsh:composite-clients
Draft

Composite roles: make it possible to clear them again#426
TomBillietKlarrio wants to merge 1 commit intocrossplane-contrib:mainfrom
klarrio-kpn-dsh:composite-clients

Conversation

@TomBillietKlarrio
Copy link
Copy Markdown

@Breee
Copy link
Copy Markdown
Collaborator

Breee commented Nov 5, 2025

Hm no, we cannot edit the autogenerated files but have to do that probably in the config Part

@TomBillietKlarrio
Copy link
Copy Markdown
Author

could you give me some pointers?
I was also a bit confused since it looked to me I saw modification to those files before. You're saying they're fully autogenerated?

@TomBillietKlarrio
Copy link
Copy Markdown
Author

I suppose

p.AddResourceConfigurator("keycloak_role", func(r *config.Resource) {
is what you're referring to? I've no clue on how to continue here to be honest.

@Breee
Copy link
Copy Markdown
Collaborator

Breee commented Nov 6, 2025

could you give me some pointers? I was also a bit confused since it looked to me I saw modification to those files before. You're saying they're fully autogenerated?

Exactly, this provider is automatically generated using the upjet framework, which is essentially generating types + crds using the terraform providers definitions.

Most of the things need to be configured here:

https://github.com/crossplane-contrib/provider-keycloak/tree/main/config

here is a doc if you want to check yourself and learn: https://github.com/crossplane/upjet/blob/main/docs/configuring-a-resource.md

you then have to run "make generate" to run the code generation

@TomBillietKlarrio
Copy link
Copy Markdown
Author

Looking into it a bit, I can't find a way how to get this working. This actually starts to look like a bug in upjet to me. Unless you have another clue, should I open a ticket on the upjet bugtracker?

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