crossplane-contrib / provider-keycloak

Apache License 2.0
14 stars 11 forks source link

ExecutionConfig did not properly set defaultProvider when using executionIdRef #121

Open fujin opened 6 days ago

fujin commented 6 days ago

I had to delete my Executions and ExecutionConfigs for the Execution to recreate, to propagate into the ExecutionConfig.

Breee commented 6 days ago

hi, can you give an example?

Resolving only once is in fact by design of crossplane and the default behaviour to reduce overhead.

If you want to update references continously, you need a:

[...]
executionIdRef: 
  name: the-execution
  policy:
    resolve: Always
[...]

Can you try if that helps you please?

fujin commented 6 days ago

thanks! I'll try to reproduce, and try that, probably what we were missing.

fujin commented 5 days ago

Does the same behavior (resolving only once) apply to e.g. those using Selectors vs IdRefs?

Breee commented 5 days ago

Can you Provide a minimal example for me?

blakebarnett commented 4 days ago

More background in this PR and the linked issues: https://github.com/crossplane/crossplane-runtime/pull/328

It's a bit unintuitive that a Selector doesn't set policy.resolve to Always automatically, but seems it's necessary.