Skip to content

Conversation

@snguyen64
Copy link
Contributor

@snguyen64 snguyen64 commented Dec 5, 2025

Continuation of optional refs pr part 2:

  • Add Optional field to Ref type in Synthesizer CRD
  • Executor skips missing optional inputs without error
  • Pass optional metadata via FunctionConfig.optionalRefs
  • Function framework checks FunctionConfig for optional status
  • Watch controller handles optional ref deletion correctly
  • Add comprehensive tests for optional refs functionality

We have previously added this field in the CRD and controller/watcher. These parts function as expected, but eno has the Executor and Synthesizer/Function which was missing the optional refs.

Testing Done:

  • unit tests
  • standalone testing

Manual Testing Steps

  • Build eno controller/reconciler
  • Deploy onto a standalone environment
  • Deploy synthesizer with optional input that does not exist - confirm no errors
  • Update so input exists - confirm synthesizer reads optional input
  • Delete optional input - confirm synthesizer still synthesizes without the input

…sing optional fields- Pass optional metadata to FunctionConfig.optionalRefs- Function framework checks FunctionConfig for optional status
@snguyen64 snguyen64 marked this pull request as ready for review December 5, 2025 01:48
@ruinan-liu ruinan-liu merged commit c0a5e77 into Azure:main Dec 10, 2025
32 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