Fire update hook from GitLab
- Zendesk ticket: https://gitlab.zendesk.com/agent/tickets/10049
- Dev link: None
- GitLab version: 8.0
Expected behavior
Custom update hooks should be fired by GitLab when a commit is made via the UI.
Observed behavior
Only pre-receive and post-receive hooks are executed.
Next steps
Add support for update hook execution. @dzaporozhets and I discussed this already.
/cc @JobV
Activity
Customer wonders if this can be patched. It is a regression since prior to satellite removal this hook would have been executed. They have an update hook that does strict commit message checking to align with a tracking system they use. They are concerned that if invalid commit messages are entered between now and when we fix this in a later feature release they will have problems.
Yes I think we should release a patch for this. cc @dzaporozhets
Path for 8.1? cc @rspeicher
@JobV Not sure what you're asking. If we have a patch ready I'll include it.
@DouweM or @rspeicher Do you have time to look at this for an 8.1 patch release? This is a regression from 8.0 that we really need to get in.
Should we do this fix a a patch for 8.1, or just schedule it for 8.2?
/cc @JobV
This is an 8.0 regression so we should strive for a patch release before 8.2.
Edited by Drew Blessing@vsizov Can you please pick up this issue? It's a regression we've had since 8.0 and we're about to ship 8.2.
Reassigned to @vsizov
Added promised regression labels
mentioned in issue #3530 (closed)
merged. @drew Please report back.
Reassigned to @dblessing
Reported to user. Thanks @vsizov
mentioned in commit 338eb2c4
mentioned in commit 5e6a5270
mentioned in commit 13c6096e
mentioned in commit tnir/gitlab-ce@13c6096e
Mentioned in commit pfjason/gitlab-ce@13c6096e
Mentioned in commit pfjason/gitlab-ce@49db8b79
mentioned in merge request !14785
