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

Improve support for hooks in @hashed storage #429

Merged
merged 3 commits into from
Feb 15, 2024

Conversation

yakatz
Copy link
Contributor

@yakatz yakatz commented Feb 13, 2024

Pull Request (PR) description

Add hashed repository storage support for custom_hook. Also add tests for custom_hook.

This Pull Request (PR) fixes the following issues

Partial solution to #322

@yakatz yakatz marked this pull request as draft February 13, 2024 22:37
@yakatz yakatz force-pushed the hashed_repos branch 3 times, most recently from e4e12b8 to 1aeddb2 Compare February 13, 2024 23:17
@yakatz yakatz marked this pull request as ready for review February 13, 2024 23:43
@yakatz
Copy link
Contributor Author

yakatz commented Feb 14, 2024

This might still only work for repos created before Gitlab 15.3. There is now a command line to create hooks - need to investigate that further: https://docs.gitlab.com/ee/administration/server_hooks.html#server-hooks-on-a-gitaly-cluster
This change is still useful.

@yakatz yakatz changed the title Hashed repos Improve support for hooks in @hashed storage Feb 15, 2024
manifests/custom_hook.pp Outdated Show resolved Hide resolved
@bastelfreak bastelfreak added the enhancement New feature or request label Feb 15, 2024
@yakatz yakatz force-pushed the hashed_repos branch 2 times, most recently from b282b85 to 457d6e5 Compare February 15, 2024 20:09
@bastelfreak bastelfreak merged commit 49f1f60 into voxpupuli:master Feb 15, 2024
29 checks passed
@yakatz yakatz deleted the hashed_repos branch April 17, 2024 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants