CoderMCPServer delete and RBAC test follow-ups
メンテナーはふだん 1 日以内に返信
@ThomasK33 がすでに取り組んでいます。
2026年10月14日 から。
評価
この issue はまだ評価されていません。
説明
Follow-ups from the final assessments of #368 (CoderMCPServer delete path), #376 (test S9) and #377 (test S11). None blocks the activation. Line numbers refer to main after #368.
Delete path
- A marker-only object waits forever when its pinned Coder organization is deleted while the control plane lives. With only a create marker (no
status.configID), the delete reads the organization's MCP server configs (findMCPServerSlug,internal/controller/codermcpserver_controller.go:492). Any list error, including Coder's own 404 for the deleted organization, givesCoderUnavailable, so the finalizer stays. Fix: treat Coder's own 404 on that list (isCoderResourceNotFound) as absence. A proxy 404 must still wait. - Missing tests:
- No operator access on delete: the wait branch at
internal/controller/codermcpserver_delete.go:61. - A marker-only delete when the operator is not a site owner:
internal/controller/codermcpserver_delete.go:107. - A marker-only delete when the config list fails.
- No operator access on delete: the wait branch at
- A manual finalizer removal leaves the object's
sentCredentialsentry in memory until the operator restarts. The entry holds only hashes, not values. The release path removes it, but a hand-removed finalizer skips that path.
RBAC test S9 (#376)
- S9 reads only
config/rbac/role.yaml. It must scan everyconfig/rbac/*.yamlfor rules that reach core Secrets, and also pinconfig/rbac/clusterrolebinding.yaml, so a new Role or binding cannot widen Secret access without a golden-copy change.
Admission test S11 (#377)
- The comment of
TestCoderMCPServerStatusPolicysays the policy "denies the status writes of every other test". It denies only status writes ofCoderMCPServerobjects. Correct the comment.
Refs #155
Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high • Cost: $220.71
- 主要言語
- Go
- スター
- 5
- フォーク
- 3
- 平均マージ
- 3時間
- マージ済み PR（30日）
- 192
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/coder-k8s のほかの issue
-
CoderWorkspacePoolMember checkCreate: trust only Coder's own 404 on the name lookup (before PR 12)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
TestReconcile_LicenseRotationUploadsNewSecretValue fails at -count=3 (reuses a control plane name)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
CoderMCPServer Kind E2E follow-ups対応中かも @ThomasK33 が 4 日前に担当しました。 オープン
coder/coder-k8s#390 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Ownership proof for Coder IDs recorded in controller status (D11 B)対応中かも @ThomasK33 が 4 日前に担当しました。 オープン
coder/coder-k8s#372 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Namespace deletion leaves AI Gateway keys and AI providers in Coder対応中かも @ThomasK33 が 4 日前に担当しました。 オープン
coder/coder-k8s#353 · コメント 4 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
coder/coder-k8s の issue をすべて見る
似ている issue
-
[Bug]: message-level cache_control on chat messages is dropped, so Anthropic prompt caching never triggers対応中かも @Atharva-Kanherkar が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
メンテナーはふだん 1 日以内に返信
-
documentation good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
eugenioenko/ttt#1000 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Billionmail/BillionMail#389 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Gentleman-Programming/gentle-ai#5603 ·
メンテナーはふだん 1 日以内に返信
-
good first issue help wanted v3
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
oras-project/oras-go#1621 ·
メンテナーはふだん 1 日以内に返信