Page MenuHomePhabricator

remove all tab characters from DNS repository, then enable jenkins check
Closed, ResolvedPublic

Description

clean the DNS repo of all tab characters,
then enable jenkins voting on it (just like in operations/puppet)


Version: unspecified
Severity: normal

Details

Reference
bz68769

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:30 AM
bzimport set Reference to bz68769.

I have added a job template in Jenkins Job Builder which let us check for tabs in puppet manifest. Have to refactor the template so it can be applied on other repositories such as operations/dns.git

Change 150067 had a related patch set uploaded by Dzahn:
mediawiki.org - retab

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

Change 150068 had a related patch set uploaded by Dzahn:
0.6.8.0.0.0.0.0.0.2.6.2.ip6.arpa - retab

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

Change 150068 merged by BBlack:
0.6.8.0.0.0.0.0.0.2.6.2.ip6.arpa - retab

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

Change 150095 had a related patch set uploaded by Dzahn:
wmnet - retab only

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

all merged, all clean, plus this one that fixes the templates:

https://gerrit.wikimedia.org/r/#/c/150397/

and done :)

~/wmf/dns/templates$ grep -l -P '\t' *.*
~/wmf/dns/templates$

bah, besides config-geo one directory up :)

eh, reopening for hashar to make the jenkins change, wasn't sure if that should have been a separate bug now

Change 151064 had a related patch set uploaded by Hashar:
Refactor tabs detector to job template

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

Change 151064 merged by jenkins-bot:
Refactor tabs detector to job template

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

The tab check for operations/puppet.git only lint .pp files and only look for tabs at the beginning of the line.

I believe operations/dns.git would want to prevent all tabs, regardless of their position.

We would need two different templates. Ie -tabs and -leadingtabs.

This is now completed. Another bug tracked the addition of the Jenkins job.