Skip to content

Fix workingDir permissions issue by creating it within init container#283

Merged
nikola-jokic merged 3 commits intoactions:mainfrom
zarko-a:zarko-a/fix-workdir-permissions
Nov 26, 2025
Merged

Fix workingDir permissions issue by creating it within init container#283
nikola-jokic merged 3 commits intoactions:mainfrom
zarko-a:zarko-a/fix-workdir-permissions

Conversation

@zarko-a
Copy link
Copy Markdown
Contributor

@zarko-a zarko-a commented Nov 21, 2025

Fixes #282

@nikola-jokic . Please see the linked issue. This fixes the problem, I've tested it in our ARC deployment.

Though, I'm not sure there was a more elegant way to solve it.

Copilot AI review requested due to automatic review settings November 21, 2025 18:57
@zarko-a zarko-a requested a review from a team as a code owner November 21, 2025 18:57
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

This PR addresses a permissions issue by creating a working directory within the init container. The change extracts the working directory path from the GITHUB_WORKSPACE environment variable and ensures it's created with proper permissions during pod initialization.

  • Adds logic to parse the working directory path from GITHUB_WORKSPACE
  • Creates the working directory in the init container to avoid permission issues

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

Comment thread packages/k8s/src/k8s/index.ts Outdated
Comment thread packages/k8s/src/k8s/index.ts Outdated
zarko-a and others added 2 commits November 21, 2025 14:34
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@zarko-a
Copy link
Copy Markdown
Contributor Author

zarko-a commented Nov 25, 2025

Bump.

@nikola-jokic any chance you can review this PR and cut a release?

Copy link
Copy Markdown
Collaborator

@nikola-jokic nikola-jokic left a comment

Choose a reason for hiding this comment

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

Thank you @zarko-a for your awesome contributions!

@nikola-jokic nikola-jokic merged commit 011ffb2 into actions:main Nov 26, 2025
5 checks passed
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.

Permissions denied on workingDir (__w/repo)

4 participants