Skip to content
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

Allow using unsafe directory in container #2

Merged
merged 1 commit into from
May 10, 2022

Conversation

Jakuje
Copy link
Contributor

@Jakuje Jakuje commented Apr 13, 2022

From yesterday, I am getting failures like:

+ git submodule update --init --recursive
fatal: unsafe repository ('/github/workspace' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /github/workspace

Hopefully, this setting will solve the issue. I am frankly not sure why it popped up right yesterday as the Ubuntu git is still older than anything we have in Fedora so I will assume some bad version or configuration popped up from somewhere.

From yesterday, I am getting failures like:
```
+ git submodule update --init --recursive
fatal: unsafe repository ('/github/workspace' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /github/workspace
```
Jakuje added a commit to Jakuje/ansible-sshd that referenced this pull request May 10, 2022
Jakuje added a commit to Jakuje/check-ansible-fedora-latest-action that referenced this pull request May 10, 2022
Same patch as for other roles in roles-ansible/check-ansible-ubuntu-latest-action#2

Not sure if this is best solution, but it does its job.
Jakuje added a commit to willshersystems/ansible-sshd that referenced this pull request May 10, 2022
Copy link
Member

@DO1JLR DO1JLR left a comment

Choose a reason for hiding this comment

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

thanks

@DO1JLR DO1JLR merged commit dc809e6 into roles-ansible:master May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants