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

Daemon don't pull GIT repository before commit #72

Closed
Dragonn opened this issue Jan 29, 2019 · 2 comments
Closed

Daemon don't pull GIT repository before commit #72

Dragonn opened this issue Jan 29, 2019 · 2 comments

Comments

@Dragonn
Copy link
Contributor

Dragonn commented Jan 29, 2019

I am running katafygio version 0.7.3 in container from docker-hub (slightly modified container and almost sample config file).

When I am running katafygio and make single push into GIT repository manually (e.g. add README.md in my case), daemon stops backing up and only solution is to purge working dir (or recreate container in my case).

Daemon is just printing errors about unable to push because there is another commit in remote master (reformated for better reading):

failed to git push: git push failed with code exit status 1:
To gitlab.<redacted>:k8s-config-backup.git
 ! [rejected]        master -> master (fetch first)
error: failed to push some refs to 'git@gitlab.<redacted>:k8s-config-backup.git'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

I suppose this can be relative easily fixed by pulling repository before commiting with git pull -s recursive -X ours to automaticaly resolve possible conflicts.

@Dragonn Dragonn changed the title Daemon dont pull GIT repository before commit Daemon don't pull GIT repository before commit Jan 29, 2019
@Dragonn
Copy link
Contributor Author

Dragonn commented Feb 1, 2019

Okay, i dived into code and tried to fix this. Please check my changes, merging will be appreciated. Thx

@bpineau
Copy link
Owner

bpineau commented Jul 20, 2019

Closing, as your PR #73 fixed this (thanks!).

@bpineau bpineau closed this as completed Jul 20, 2019
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

No branches or pull requests

2 participants