Page MenuHomePhabricator

Add a Jenkins check that forbids creation of /modules/role/manifests/*.pp
Closed, ResolvedPublic

Description

As per T119042, all roles must be in a subdir.

Event Timeline

Change 308666 had a related patch set uploaded (by Legoktm):
Add test to ensure there are no *.pp files in modules/role/manifests/

https://gerrit.wikimedia.org/r/308666

Change 308668 had a related patch set uploaded (by Andrew Bogott):
Add a test for unlucky .pp files that break puppet runs.

https://gerrit.wikimedia.org/r/308668

Change 308668 abandoned by Andrew Bogott:
Add a test for unlucky .pp files that break puppet runs.

Reason:
dropped in favor of 308666

https://gerrit.wikimedia.org/r/308668

Change 308666 merged by Alexandros Kosiaris:
Add test to ensure there are no *.pp files in modules/role/manifests/

https://gerrit.wikimedia.org/r/308666