Skip to content

docs: document FromEnvironmentFieldPath rerun on updated value - #129

Merged
phisco merged 2 commits into
crossplane-contrib:mainfrom
tenstad:from-environment-field-path
Aug 3, 2026
Merged

phisco merged 2 commits into
crossplane-contrib:mainfrom
tenstad:from-environment-field-path

Conversation

@tenstad

@tenstad tenstad commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description of your changes

Documents #119 (specifically tenstad#1)

The fix recomputes the requirements against the merged environment before
returning. Requirements that grew are how a Function asks to be called again, so
Crossplane fetches the newly selected EnvironmentConfigs and calls us once more;
they stabilise as soon as a pass selects nothing new.

Two limits worth knowing, both of which I think are fine to document rather than
solve here:

Required still fails on the first call, since the environment genuinely is
empty then — chained matchers have to be Optional. That is a sharp edge given
Required is the default, so it probably deserves a line in the field docs.
Each link in the chain costs roughly two iterations of Crossplane's
MaxRequirementsIterations (5), capping chains at about two levels.

Found it difficult to describe. Open for suggestions.

I have:

Signed-off-by: Amund Tenstad <github@amund.io>
@tenstad
tenstad requested a review from phisco as a code owner July 31, 2026 09:28
Signed-off-by: Amund Tenstad <github@amund.io>
@phisco
phisco merged commit 2eb03e1 into crossplane-contrib:main Aug 3, 2026
6 checks passed
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