CoderMCPServer delete and RBAC test follow-ups
Maintainer antworten meist innerhalb von 1 Tag
@ThomasK33 arbeitet bereits daran.
Seit 14.10.2026.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
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
- Vorherrschende Sprache
- Go
- Sterne
- 5
- Forks
- 3
- Ø Merge
- 2 Std. 37 Min.
- Gemergte PRs (30 T.)
- 186
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus coder/coder-k8s
-
CoderWorkspacePoolMember checkCreate: trust only Coder's own 404 on the name lookup (before PR 12)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
TestReconcile_LicenseRotationUploadsNewSecretValue fails at -count=3 (reuses a control plane name)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
CoderMCPServer Kind E2E follow-upsEvtl. vergeben @ThomasK33 hat das vor 9 Tagen übernommen. Offen
coder/coder-k8s#390 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Ownership proof for Coder IDs recorded in controller status (D11 B)Evtl. vergeben @ThomasK33 hat das vor 9 Tagen übernommen. Offen
coder/coder-k8s#372 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Namespace deletion leaves AI Gateway keys and AI providers in CoderEvtl. vergeben @ThomasK33 hat das vor 10 Tagen übernommen. Offen
coder/coder-k8s#353 · 4 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in coder/coder-k8s
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
JakeChampion/lang#13132 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG] deploy.restart_policy without condition drops the restart policyEvtl. vergeben @akshat-lakhera hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
docker/compose#14383 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
Azure/azure-storage-fuse#2429 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
AvengeMedia/dank-greeter#39 · 1 Kommentar ·
-
bug: dataplane client ignores SERVER_BASEPATH, so create/delete topic miss the subpathEvtl. vergeben @bhataprameya hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
redpanda-data/console#2693 ·
Maintainer antworten meist innerhalb von 2 Tagen