Skip to content

Add filelock for git operations#28

Open
therazix wants to merge 3 commits into
mainfrom
fvagner-filelock
Open

Add filelock for git operations#28
therazix wants to merge 3 commits into
mainfrom
fvagner-filelock

Conversation

@therazix
Copy link
Copy Markdown
Contributor

This PR wraps all git operations in get_git_repository with a per-repository FileLock to prevent concurrent tasks from potentially corrupting a shared local clone. Also, if fetching from the remote fails, the repository is automatically wiped and re-cloned to ensure a clean state.

@therazix therazix requested review from janhavlin and psss May 13, 2026 15:20
@happz happz added this to planning May 20, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning May 20, 2026
@happz happz added the bug Something isn't working label May 20, 2026
@happz happz moved this from backlog to review in planning May 20, 2026
Comment thread src/tmt_web/utils/git_handler.py Outdated
Comment thread src/tmt_web/utils/git_handler.py Outdated
Comment thread src/tmt_web/utils/git_handler.py Outdated
Base automatically changed from fvagner-update-tmt-version to main May 22, 2026 06:39
@psss
Copy link
Copy Markdown
Member

psss commented May 25, 2026

Testing the story point sync, please ignore.

@happz
Copy link
Copy Markdown

happz commented May 26, 2026

@therazix please, rebase the PR - I can't see the button, and it seems to include some older changes.

@LecrisUT
Copy link
Copy Markdown
Member

@therazix please, rebase the PR - I can't see the button, and it seems to include some older changes.

Button is added now with the branch ruleset. It is a copy from tmt's

@therazix therazix force-pushed the fvagner-filelock branch from 218af45 to 19d0bec Compare May 26, 2026 08:16
Comment thread src/tmt_web/utils/git_handler.py Outdated
@therazix therazix force-pushed the fvagner-filelock branch from b1dddc2 to 5aaaa61 Compare May 27, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

4 participants