Skip to content

v1.3 backport: save null module outputs in state#32584

Merged
jbardin merged 1 commit intov1.3from
jbardin/store-null-module-outputs-v1.3
Jan 26, 2023
Merged

v1.3 backport: save null module outputs in state#32584
jbardin merged 1 commit intov1.3from
jbardin/store-null-module-outputs-v1.3

Conversation

@jbardin
Copy link
Copy Markdown
Member

@jbardin jbardin commented Jan 26, 2023

Although they are not serialized to the final stored state, all module outputs must be saved in the state for evaluation. There is no defined schema which is used to identify the overall type of module outputs, so all outputs must exist in the state to build the correct type for proper evaluation.

Manual backport of #32583

Although they are not serialized to the final stored state, all module
outputs must be saved in the state for evaluation. There is no defined
schema which is used to identify the overall type of module outputs, so
all outputs must exist in the state to build the correct type for proper
evaluation.
@jbardin jbardin changed the title save null module outputs in state v1.3 backport: save null module outputs in state Jan 26, 2023
@jbardin jbardin merged commit c22135e into v1.3 Jan 26, 2023
@jbardin jbardin deleted the jbardin/store-null-module-outputs-v1.3 branch January 26, 2023 20:36
@github-actions
Copy link
Copy Markdown
Contributor

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 26, 2023
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