Skip to content

custom-init-containers-command-validation#14

Open
akurinnoy wants to merge 4 commits into
mainfrom
supervisor/f968f4d7-d0c5-4a19-b8d2-e93e4535b9f2
Open

custom-init-containers-command-validation#14
akurinnoy wants to merge 4 commits into
mainfrom
supervisor/f968f4d7-d0c5-4a19-b8d2-e93e4535b9f2

Conversation

@akurinnoy

Copy link
Copy Markdown
Owner

Summary

Automated implementation via supervisor execution pipeline.

Run ID: f968f4d7-d0c5-4a19-b8d2-e93e4535b9f2
Plan ID: custom-init-containers-command-validation
Waves completed: 4/4

Tasks

  • t1: Clone repo, verify build, and map exact change surface
  • t2: Implement command validation in EnsureHomeInitContainerFields
  • t4: Add unit tests for EnsureHomeInitContainerFields command validation
  • t5: Run full test suite, verify build, commit, and file PR

Integration Waves

  • Wave 1: merged
  • Wave 2: merged t2
  • Wave 3: merged t4
  • Wave 4: merged t5

akurinnoy and others added 4 commits June 30, 2026 11:28
Add command validation using reflect.DeepEqual. If a user-supplied command
is non-empty and does not equal [/bin/sh, -c], return an error. If empty,
set it to the default [/bin/sh, -c] as before.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
…tion

Adds TestEnsureHomeInitContainerFields covering four cases: no command
(sets default [/bin/sh, -c] and VolumeMounts), exact valid command accepted,
wrong binary returns error, and missing -c argument returns error. Each test
asserts both error return and container field state.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant