Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Sep 14, 2021

Changes

Prior to this commit we primarily documented use of termination messages
through a lens of Task Results, but the mechanism is used more broadly
than this.

This commit adds notes to our developer docs explaining use of pod termination
messages, giving examples of the data included, how the data is distributed to
other parts of the Pipelines machinery and describing the lack of guarantees
around the message's content.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

Release Notes

Our developer docs now include more details on how Tekton Pipelines uses Pod termination messages to communicate information between Task pods and the Tekton Pipelines controller.

Prior to this commit we primarily documented use of termination messages
through a lens of Task Results, but the mechanism is used more broadly
than this.

This commit adds notes to our developer docs explaining use of pod termination
messages, giving examples of the data included, how the data is distributed to
other parts of the Pipelines machinery and describing the lack of guarantees
around the message's content.
@ghost ghost added the kind/documentation Categorizes issue or PR as related to documentation. label Sep 14, 2021
@ghost ghost requested a review from afrittoli September 14, 2021 14:47
@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Sep 14, 2021
@tekton-robot tekton-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 14, 2021
@ghost ghost mentioned this pull request Sep 14, 2021
5 tasks
@tekton-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 14, 2021
step began, contents of task results, contents of pipeline resource
results.

The contents and format of the termination message can change. At time
Copy link
Member

Choose a reason for hiding this comment

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

thanks a bunch @sbwsg for adding this.

NIT: I would add a little note here saying its not recommended to consume the termination message.

@pritidesai
Copy link
Member

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 14, 2021
@tekton-robot tekton-robot merged commit 78d6d75 into tektoncd:main Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/documentation Categorizes issue or PR as related to documentation. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants