Last modified: 2014-09-26 22:19:39 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 T66371, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64371 - Update CodeSniffer rules automatically on machines running tests
Update CodeSniffer rules automatically on machines running tests
Status: NEW
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-24 16:15 UTC by Siebrand Mazeland
Modified: 2014-09-26 22:19 UTC (History)
4 users (show)

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


Attachments

Description Siebrand Mazeland 2014-04-24 16:15:34 UTC
Gerrit change #127863 was merged 48 hours ago, but the test runners are still using an older rule set, as is clear from https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-strict-HEAD/9608/console on Gerrit change #129442.

Please make the CodeSniffer rule set update automatically on machines running tests.
Comment 1 Antoine "hashar" Musso (WMF) 2014-04-24 20:15:32 UTC
As I stated on the Gerrit change:

[..]. Whenever this is merged, one will have to update the related submodule in integration/jenkins.git (path: tools/mwcodesniffer ).


Then use git-deploy on tin.eqiad.wmnet to refresh integration/jenkins.git on the production servers in  /srv/deployment/integration/slave-scripts
Comment 2 Chad H. 2014-04-24 20:32:59 UTC
I did do that.
Comment 3 Antoine "hashar" Musso (WMF) 2014-04-24 21:00:47 UTC
Bah the job uses a different git-deploy target sorry. phpcs has for option:

--standard=/srv/deployment/integration/mediawiki-tools-codesniffer/MediaWiki


Should probably remove the submodule from integration/jenkins.git if it is not used :)


I have refreshed the minions, that should fix the issue.

git-deploy log:


hashar@tin:/srv/deployment/integration/mediawiki-tools-codesniffer$ git deploy start
giDeployment started.
hashar@tin:/srv/deployment/integration/mediawiki-tools-codesniffer$ git pull
remote: Counting objects: 11, done
remote: Finding sources: 100% (4/4)
remote: Getting sizes: 100% (1/1)
remote: Total 4 (delta 2), reused 4 (delta 2)
Unpacking objects: 100% (4/4), done.
From https://gerrit.wikimedia.org/r/mediawiki/tools/codesniffer/
   0bebf0f..bf82117  master     -> origin/master
Updating 0bebf0f..bf82117
Fast-forward
 MediaWiki/ruleset.xml |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
hashar@tin:/srv/deployment/integration/mediawiki-tools-codesniffer$ git deploy sync
Repo: integration/mediawiki-tools-codesniffer
Tag: integration/mediawiki-tools-codesniffer-sync-20140424-205855

2/2 minions completed fetch
Continue? ([d]etailed/[C]oncise report,[y]es,[n]o,[r]etry): y
Repo: integration/mediawiki-tools-codesniffer
Tag: integration/mediawiki-tools-codesniffer-sync-20140424-205855

2/2 minions completed checkout
Continue? ([d]etailed/[C]oncise report,[y]es,[n]o,[r]etry): y
Deployment finished.
hashar@tin:/srv/deployment/integration/mediawiki-tools-codesniffer$
Comment 4 Antoine "hashar" Musso (WMF) 2014-04-24 21:02:48 UTC
And the job pass now! https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-strict-HEAD/9654/console

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


Navigation
Links