Skip to content

Add filelock for git operations#28

Merged
therazix merged 4 commits into
mainfrom
fvagner-filelock
Jun 1, 2026
Merged

Add filelock for git operations#28
therazix merged 4 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
@happz happz self-assigned this May 28, 2026
@happz happz removed their assignment Jun 1, 2026
@therazix therazix merged commit 456eb32 into main Jun 1, 2026
3 checks passed
@therazix therazix deleted the fvagner-filelock branch June 1, 2026 08:53
@github-project-automation github-project-automation Bot moved this from review to done in planning Jun 1, 2026
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: done

Development

Successfully merging this pull request may close these issues.

4 participants