Skip to content

chore(docs): fix inconsistencies in external provisioner docs #15416

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.

Sign up for GitHub

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
merged 7 commits into from
Nov 7, 2024
Merged

chore(docs): fix inconsistencies in external provisioner docs #15416

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f2680df
chore(docs): fix inconsistencies in external provisioner docs
johnstcn Nov 6, 2024
919aaaf
fixup! chore(docs): fix inconsistencies in external provisioner docs
johnstcn Nov 6, 2024
8c1964f
fixup! fixup! chore(docs): fix inconsistencies in external provisione…
johnstcn Nov 6, 2024
633899b
fixup! fixup! fixup! chore(docs): fix inconsistencies in external pro…
johnstcn Nov 6, 2024
938be05
fixup! fixup! fixup! fixup! chore(docs): fix inconsistencies in exter…
johnstcn Nov 6, 2024
f600308
make fmt
johnstcn Nov 6, 2024
f66a877
Update docs/admin/provisioners.md
bpmct Nov 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update docs/admin/provisioners.md
  • Loading branch information
@bpmct
bpmct authored Nov 7, 2024
commit f66a87747ea24122fde6175aff4edab2c116edce
2 changes: 1 addition & 1 deletion docs/admin/provisioners.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ will use in concert with the Helm chart for deploying the Coder server.
value: "https://coder.example.com"
replicaCount: 10
provisionerDaemon:
# NOTE: in older versions of the Helm chart, it is required to set this to an empty string.
# NOTE: in older versions of the Helm chart (2.17.0 and below), it is required to set this to an empty string.
pskSecretName: ""
Copy link
Member Author

Choose a reason for hiding this comment

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

review: PR to address this workaround is here: #15417

keySecretName: "coder-provisioner-keys"
keySecretKey: "my-cool-key"
Expand Down
Loading