Skip to content

Backport of Update the docs for tfstack config output blocks into v1.10 #36355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

github-actions[bot]
Copy link
Contributor

Backport

This PR is auto-generated from #36339 to be assessed for backporting due to the inclusion of the label 1.10-backport.

The below text is copied from the body of the original PR.


The docs for the variant of the output block available in tfstack configurations were wrong — here's the fix.

To verify the schema for yourself, check its definition at the bottom of internal/stacks/stackconfig/output_value.go in this repo. (I also ran an experiment to make sure, and yep: sensitive is supported as optional, and type is mandatory.)

Q for education team reviewers: which branch is the web version of these docs sourced from? Does it come straight out of main?

Target Release

1.11.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

Overview of commits

@rkoron007 rkoron007 marked this pull request as ready for review May 19, 2025 19:21
@rkoron007 rkoron007 requested review from a team as code owners May 19, 2025 19:21
@rkoron007 rkoron007 requested a review from pbortnick May 19, 2025 19:21
Copy link
Contributor

@rkoron007 rkoron007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@rkoron007 rkoron007 enabled auto-merge (squash) May 19, 2025 19:21
@rkoron007 rkoron007 merged commit dbd5592 into v1.10 May 19, 2025
9 of 10 checks passed
@rkoron007 rkoron007 deleted the backport/nf/jan25-tfstack-output-docs/exactly-guided-labrador branch May 19, 2025 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants