Last modified: 2013-05-21 08:26:14 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50590, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48590 - Jenkins Puppet linting for operations/puppet/* repositories
Jenkins Puppet linting for operations/puppet/* repositories
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Normal minor (vote)
: ---
Assigned To: Antoine "hashar" Musso (WMF)
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-17 20:42 UTC by Andrew Otto
Modified: 2013-05-21 08:26 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Andrew Otto 2013-05-17 20:42:09 UTC
We're starting to create puppet module repositories under operations/puppet/*.  We should enable automatic linting for repositories.  This should work the same way it does for the operations/puppet repository.

While we're at it, what about running puppet unit tests?  The operations/puppet/cdh4 repository has them!  (Thanks Alex!)
Comment 1 Antoine "hashar" Musso (WMF) 2013-05-17 20:54:40 UTC
Need to do something like the erb linter. Working on it.
Comment 2 Gerrit Notification Bot 2013-05-17 20:57:12 UTC
Related URL: https://gerrit.wikimedia.org/r/64427 (Gerrit Change I66fca52afc87b00b7d77ff4adead2547211a39db)
Comment 3 Antoine "hashar" Musso (WMF) 2013-05-17 21:03:44 UTC
INFO:jenkins_jobs.builder:Creating jenkins job operations-puppet-cdh4-erblint-HEAD
INFO:jenkins_jobs.builder:Creating jenkins job operations-puppet-cdh4-pplint-HEAD
INFO:jenkins_jobs.builder:Creating jenkins job operations-puppet-kafka-erblint-HEAD
INFO:jenkins_jobs.builder:Creating jenkins job operations-puppet-kafka-pplint-HEAD
Comment 4 Gerrit Notification Bot 2013-05-17 21:06:37 UTC
Related URL: https://gerrit.wikimedia.org/r/64432 (Gerrit Change I0cb06d78188f82cfad25477c86e94660954f3061)
Comment 5 Gerrit Notification Bot 2013-05-17 21:32:52 UTC
Related URL: https://gerrit.wikimedia.org/r/64438 (Gerrit Change If043e1db3d470cf326a66776113c03c4a056e42d)
Comment 6 Antoine "hashar" Musso (WMF) 2013-05-17 21:35:26 UTC
Of course I made some tiny mistakes that would have been catched by a proper "pre commit" review workflow.   But got them fixed and that seems to work now.

I have triggered the jobs on cdh4 and hadoop modules by filling in Gerrit a 'recheck' comment.  Tests changes:

Kafka: https://gerrit.wikimedia.org/r/50385

Cdh4: https://gerrit.wikimedia.org/r/61710
Comment 7 Andrew Otto 2013-05-17 21:36:37 UTC
Looks great!  We should get unit testing in too sometime.  Will resolve this for now.
Comment 8 Antoine "hashar" Musso (WMF) 2013-05-17 21:37:40 UTC
On second thought, I forgot to create the '-merge' jobs and make them the first one in the template.  Will poke that monday (or later next week).
Comment 9 Gerrit Notification Bot 2013-05-21 08:16:50 UTC
Related URL: https://gerrit.wikimedia.org/r/64783 (Gerrit Change I60530800226f7d9aecf61c3e53db055d26178f66)
Comment 10 Gerrit Notification Bot 2013-05-21 08:18:37 UTC
Related URL: https://gerrit.wikimedia.org/r/64784 (Gerrit Change Ic7cc463b5518ae40f29b4a077f2342186cb64470)
Comment 11 Antoine "hashar" Musso (WMF) 2013-05-21 08:26:14 UTC
Well merge jobs are not really needed since the test jobs are very fast. I reverted the two above changes.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links