docs: give ci-bot organization-template-admin; tick #208 OpenSpec tasks (#209) [skip ci] - #209
Merged
Merged
Conversation
ci-show-provisioner-errors.sh needs to read provisioner jobs, which the site-wide template-admin role doesn't allow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017LNV7pCH3fCUkVkhto13Ev
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017LNV7pCH3fCUkVkhto13Ev
|
Member
Author
|
The staging Coder server couldn't download the This is the same intermittent Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Issue
Follow-up to #208; these two commits were ready but missed that merge.
#208 added
scripts/ci-show-provisioner-errors.sh, which runs after everycoder createin the integration workflows. When the server-side Terraform step fails, the Coder CLI often prints onlyinitialize terraform: exit status 1, and this script prints the real error from the failed provisioner job's logs. To read those logs, the CI user needs theorganization-template-adminorg role. The site-widetemplate-adminrole alone can't list provisioner jobs, and without the org role the script can only print a warning. The staging setup docs didn't mention the org role.How This PR Solves The Issue
docs/admin/server-setup.md: "Create the CI bot user on staging" now includescoder organizations members edit-roles ci-bot organization-template-admin, with a short explanation of why it's needed.ci-botalready has this role on staging-coder.ddev.com (granted 2026-09-24); the command was run there as written.openspec/changes/add-user-startup-hook/tasks.md: ticks task 2.3 (the integration test passed on staging) and adds 2.4 (the manual end-to-end check on staging: a~/.coder-startup.shthat starts a Claude Code self-hosted runner in tmux brought it up after a workspace restart with no terminal open, and a session ran on it).Manual Testing Instructions
None needed: this changes a doc and a task list. To check the role on a server:
coder organizations members listshould showci-botwith[Organization Template Admin].Automated Testing Overview
No code changes.
openspec validate add-user-startup-hook --strictpasses.Release/Deployment Notes
On any Coder server the integration tests run against, give the CI user the org role once. On staging-coder.ddev.com this is already done.
🤖 Generated with Claude Code
https://claude.ai/code/session_017LNV7pCH3fCUkVkhto13Ev