Skip to content

Validate work dir during runner start up.#4227

Merged
TingluoHuang merged 1 commit intomainfrom
users/tihuang/validatedir
Feb 9, 2026
Merged

Validate work dir during runner start up.#4227
TingluoHuang merged 1 commit intomainfrom
users/tihuang/validatedir

Conversation

@TingluoHuang
Copy link
Member

@TingluoHuang TingluoHuang commented Feb 9, 2026

@TingluoHuang TingluoHuang requested a review from a team as a code owner February 9, 2026 01:35
Copilot AI review requested due to automatic review settings February 9, 2026 01:35
Copy link
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

Adds an early startup check in the listener to ensure the runner work directory exists and is accessible, so the runner can fail fast with a clear error rather than encountering later job/runtime failures.

Changes:

  • Create the runner work directory (if needed) and validate directory access during Runner.RunAsync startup.
  • Log validation progress/failure and terminate early on validation errors.
  • Remove an unused using and normalize some whitespace.

@TingluoHuang TingluoHuang force-pushed the users/tihuang/validatedir branch from ca705c9 to 8936191 Compare February 9, 2026 01:39
Copy link
Contributor

Copilot AI commented Feb 9, 2026

@TingluoHuang I've opened a new pull request, #4228, to work on those changes. Once the pull request is ready, I'll request review from you.

@TingluoHuang TingluoHuang force-pushed the users/tihuang/validatedir branch from 8936191 to 7b47280 Compare February 9, 2026 03:39
Copy link
Member Author

Choose a reason for hiding this comment

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

hide whitespace for better diff.

@TingluoHuang TingluoHuang merged commit 1b5486a into main Feb 9, 2026
12 checks passed
@TingluoHuang TingluoHuang deleted the users/tihuang/validatedir branch February 9, 2026 13:42
dawidmalina pushed a commit to dawidmalina/github-runner that referenced this pull request Feb 26, 2026
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.

4 participants