Skip to content
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

Implement several documentation improvements to the Agent (for the Buildkite Docs). #3043

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

gilesgas
Copy link
Contributor

@gilesgas gilesgas commented Oct 17, 2024

Description

This PR addresses several docs improvements:

  • Adds more info regarding 'environment propagation' for JWKS files.
  • Improve the description for the 'redactor' command to make it behave like the others for the docs.

Context

The purpose of this PR is to address some long-standing issues with the Buildkite Docs that can only be addressed through the Agent codebase.

Changes

Testing

  • Tests have run locally (with go test ./...). Buildkite employees may check this if the pipeline has run automatically.
  • Code is formatted (with go fmt ./...)

@gilesgas gilesgas marked this pull request as draft October 17, 2024 03:19
clicommand/agent_start.go Outdated Show resolved Hide resolved
Comment on lines 57 to 74
Description: `Usage:

buildkite-agent redactor add [options...]

Description:

This command may be used to parse a file for values to redact from a
running job's log output. If you dynamically fetch secrets during a job,
it is recommended that you use this command to ensure they will be
redacted from subsequent logs. Secrets fetched with the builtin
′secret get′ command do not require the use of this command, they will
be redacted automatically.

Example:

$ buildkite-agent redactor add ...

Redacts something.`,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hello @wolfeidau ,

Could you (or would you know anyone who could) please provide an example of how this command is used? Just a typical example using some of the options would be great, and then I can provide a better explanation than what's currently provided in the agent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI, I fixed the spacing in the Description section only, which is why this comment appears outdated, but it's actually not.

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