-
Notifications
You must be signed in to change notification settings - Fork 418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create the root directory if it doesn't exist #383
Conversation
The `git clone` command will create the root directory if it doesn't exist, but if `git clone` fails, the root directory needs to be present so that we can write the error to a file under the directory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Can you port to master
branch too ?
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nan-yu, thockin The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Sure. Please check #385. Can we have a release for the fix? |
https://github.com/kubernetes/git-sync/releases/tag/v3.3.1 - waiting for the image to be promoted. |
Thanks @thockin . I did an integration test with the latest image. Unfortunately, we need another fix for the error file's read permission. I pushed up two PRs: #389 and #390 for review. I verified this is the last fix to make the error file readable to other containers. I'm sorry for not capturing them earlier. I should have run the test before the release. |
The
git clone
command will create the root directory if it doesn'texist, but if
git clone
fails, the root directory needs to be presentso that we can write the error to a file under the directory.