Command plug-in only works for pre-commit verification

Bug #1100353 reported by Diogo Matsubara
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Tarmac
Confirmed
Medium
Unassigned

Bug Description

Use case:

After Tarmac merges a branch, we want it to trigger an arbitrary command (in our case `juju set charmworld revno=$(bzr revno lp:charmworld 2>/dev/null)`) to update the staging instance of charmworld.

I spoke to Dobey on IRC and his suggestion was to have the command.py refactored so it could be used as a pre commit hook and a post commit hook.

As a workaround I'm going to copy the current command.py plugin, change the class name, the hook and the plugin name to act as the post commit hook command.

Changed in tarmac:
status: New → Confirmed
dobey (dobey)
Changed in tarmac:
importance: Undecided → Medium
summary: - Refactor command.py to support post commit hook
+ Command plug-in only works for pre-commit verification
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.