Jenkins should fail verification of a puppet patch touching modules/admin/data/data.yaml if it doesn't contain a reference to a Phabricator task via a "Bug: T($ID)" line.
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T142815 Enhance account handling (meta bug) | |||
| Declined | None | T142827 Enforce reference to Phabricator task for all commits to modules/admin/data/data.yaml |
Event Timeline
Comment Actions
Nowaday this can be done directly in the puppet repository. There is an admin testenv (tox -e admin) which runs: pytest modules/admin/data I guess the logic can be implemented there.
Comment Actions
Mid-term data.yaml will be generated via the IDM which will include proper history/revision handling, and there's no need to still work on it (the vast majority of changes uses Phab IDs anyway). Marking as declined