Skip to content

fix: use python user in skills-init container#1762

Merged
jmhbh merged 2 commits intokagent-dev:mainfrom
jmunozro:jmunozro/skill-init-user
Apr 27, 2026
Merged

fix: use python user in skills-init container#1762
jmhbh merged 2 commits intokagent-dev:mainfrom
jmunozro:jmunozro/skill-init-user

Conversation

@jmunozro
Copy link
Copy Markdown
Contributor

fixes #1761

Copilot AI review requested due to automatic review settings April 27, 2026 12:15
@github-actions github-actions Bot added the bug Something isn't working label Apr 27, 2026
Signed-off-by: Jesus Munoz <jesus.munoz@solo.io>
@jmunozro jmunozro force-pushed the jmunozro/skill-init-user branch from 2a95e7b to 023a6df Compare April 27, 2026 12:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes an init-container permissions bug by ensuring the skills-init image runs as the same non-root user as the main agent container, so cloned skill repos are readable from the main container.

Changes:

  • Create a python user/group (UID/GID 1001) inside the skills-init image.
  • Run the skills-init container as UID/GID 1001:1001 to align file ownership on the shared /skills volume.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker/skills-init/Dockerfile Outdated
Comment thread docker/skills-init/Dockerfile Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Jesus Munoz <jesus.munoz@solo.io>
@jmhbh jmhbh merged commit 7b7c8b6 into kagent-dev:main Apr 27, 2026
23 checks passed
@jmunozro jmunozro deleted the jmunozro/skill-init-user branch April 27, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] skills-init is using root user to clone

3 participants