[Bug 46723] Jenkins merged a faulty change

2014-03-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2014-03-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #23 from Gerrit Notification Bot  ---
Change 117045 merged by jenkins-bot:
Parsoid: uses ZUUL_COMMIT as a git refspec to build

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2014-03-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Gerrit Notification Bot  changed:

   What|Removed |Added

 Status|RESOLVED|PATCH_TO_REVIEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2014-03-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #22 from Gerrit Notification Bot  ---
Change 117045 had a related patch set uploaded by Hashar:
Parsoid: uses ZUUL_COMMIT as a git refspec to build

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #21 from Antoine "hashar" Musso  ---
I have verified the jobs triggered over the past few days. Seems to work fine
now :-)  The root cause was using ZUUL_BRANCH as a branch specifier instead of
ZUUL_COMMIT.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #20 from Antoine "hashar" Musso  ---
hashar@gallium:~$ ./checkbug46723.py mediawiki-core-phpunit-api --filter
2013-04-16*
Found 0 mismatches in 29 log files.
hashar@gallium:~$ ./checkbug46723.py mediawiki-core-phpunit-misc --filter
2013-04-16*
Found 0 mismatches in 29 log files.
$

Seems it got fixed :-]  Will verify again during the week, but so far that
looks good.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

   Priority|Immediate   |Highest
 Status|NEW |ASSIGNED
   Assignee|wikibugs-l@lists.wikimedia. |has...@free.fr
   |org |

--- Comment #19 from Antoine "hashar" Musso  ---
Will monitor over the next few days. Lowering priority for now.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #18 from Antoine "hashar" Musso  ---
https://gerrit.wikimedia.org/r/#/c/58865/ has been deployed.

I am now manually updating the jobs which are not under JJB:

analytics-libanon
analytics-udp-filters
analytics-webstatscollector
analytics-wikistats
mwext-PoolCounter-pep8
mwext-VisualEditor-docgen
operations-debs-python-voluptuous-debbuild
parsoid-parse-tool-check
parsoid-roundtrip-test-check
parsoid-runTests
test-mediawiki-merge

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #17 from Gerrit Notification Bot  ---
https://gerrit.wikimedia.org/r/58865 (Gerrit Change
Iafebfffe480886fc8956e56517291b1b3b1fc0cc) | change APPROVED and MERGED [by
Hashar]

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

 CC||s...@ludd.net

--- Comment #16 from Antoine "hashar" Musso  ---
*** Bug 47208 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #15 from Liangent  ---
(In reply to comment #13)
> Related URL: https://gerrit.wikimedia.org/r/58865 (Gerrit Change
> Iafebfffe480886fc8956e56517291b1b3b1fc0cc)

Why is this comment duplicated?

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #14 from Antoine "hashar" Musso  ---
I have updated mediawiki-core-whitespaces job to use ZUUL_COMMIT as a refspec
specifier. The job is non voting so that is not going to do any harm.

The experimental change is https://gerrit.wikimedia.org/r/58865

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #12 from Gerrit Notification Bot  ---
Related URL: https://gerrit.wikimedia.org/r/58865 (Gerrit Change
Iafebfffe480886fc8956e56517291b1b3b1fc0cc)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #13 from Gerrit Notification Bot  ---
Related URL: https://gerrit.wikimedia.org/r/58865 (Gerrit Change
Iafebfffe480886fc8956e56517291b1b3b1fc0cc)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #11 from Antoine "hashar" Musso  ---
The branch specifier in the git plugin is set to ZUUL_BRANCH which is 'master'. 

In the git plugin (at
git-plugin/src/main/java/hudson/plugins/git/util/DefaultBuildChooser.java ),
the getCandidateRevisions() will recognize whether the branch looks like a sha1
(if it matches /[0-9a-f]{6,40}/) and in such a case will create a detached
branch using that commit.

Seems the Jenkins job macro should then use ZUUL_COMMIT as a branch specifier.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #10 from Antoine "hashar" Musso  ---
ZUUL_COMMIT=76cb37f0c69dcd69884fc6e66681e77c8045a08e

but it fetched origin/master instead :-(

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #9 from Antoine "hashar" Musso  ---
Created attachment 12084
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=12084&action=edit
Console output for
https://integration.wikimedia.org/ci/job/mediawiki-core-phpunit-parser/5386/console

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #8 from Antoine "hashar" Musso  ---
We are lacking the git plugin verbose mode in production due to a bug. It is
corrected with https://gerrit.wikimedia.org/r/58489  . That will help find out
what the plugin is doing internally.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #7 from Antoine "hashar" Musso  ---
I have traced the issue as far as mediawiki-core-lint build #19 from made on
November 22nd 2012).


MISMATCH in /var/lib/jenkins/jobs/mediawiki-core-lint/builds/19/log
Pipeline: gate
Zuulcommit: 76606b66b006ac0e62087e6d00b1e4bdd56fff09
Checkedout: 232e34733fc68739ba9631d3ff88f9484a23

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #6 from Antoine "hashar" Musso  ---
Extract for the two builds referenced somewhere above:

Verifying /var/lib/jenkins/jobs/mediawiki-core-phpunit-misc/builds/5222/log
Zuulcommit: 8cc0b601aa2db6db09ac0e4d70847293d75875aa
Checkedout: 8cc0b601aa2db6db09ac0e4d70847293d75875aa
Verifying /var/lib/jenkins/jobs/mediawiki-core-phpunit-misc/builds/5223/log
Zuulcommit: 7dd3356a51951f8cdfe463552b5e5aae272e8e60
Checkedout: 4c69569db71d149feff6c4b10ea7a493425d67fd (MISMATCH)


We can see that build 5223 did not used the proper commit :-]


I suspect git plugin does not fetch the proper references / can't find it. That
result internally in an unknown sha1 and then git plugin fallback to master or
something else.

I will try to reproduce the issue in labs with git plugin set to verbose. That
needs to start Jenkins with -Dhudson.plugins.git.GitSCM.verbose=true

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #5 from Antoine "hashar" Musso  ---
Created attachment 12066
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=12066&action=edit
output of checkbug46723.py

The result script output highlight that some builds are not testing what they
should be testing because they check out a parent commit.  By looking at the
Jenkins Git plugin source code, it seems that whenever the reference is not
parseable (aka: git rev-parse $ZUUL_REF), the plugin fallback to use master or
some parent commit.

I need to improve the script to find out if that happens in a specific pipeline
or for some specific refs.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #4 from Antoine "hashar" Musso  ---
Created attachment 12065
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=12065&action=edit
python script parsing build logs to find Zuul commit vs Git plugin checkout

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #3 from Antoine "hashar" Musso  ---
build #5223 


Workspace did get wiped:
 02:46:53 Wiping out workspace first.

It check out the revision:
 02:46:56 Checking out Revision 4c69569db71d149feff6c4b10ea7a493425d67fd
(origin/master)

That is the master revision NOT the change. The commit should have been
7dd3356a51951f8cdfe463552b5e5aae272e8e60



The related merge job
https://integration.wikimedia.org/ci/job/mediawiki-core-merge/11333/console

02:44:17 Commencing build of Revision 7dd3356a51951f8cdfe463552b5e5aae272e8e60
(origin/master)
02:44:17 Checking out Revision 7dd3356a51951f8cdfe463552b5e5aae272e8e60
(origin/master)



The ZUUL_REF has probably not been resolved properly and the git plugin
fallback to master.

There is also the possibility that the mediawiki-core-phpunit-misc job was
using ZUUL_COMMIT as a refspec instead of ZUUL_REF. That might prevent the
plugin from fetching the revision.   The job history is no more accessible due
to an unexpected upgrade (see bug 47040).

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

   Priority|High|Immediate

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #2 from Antoine "hashar" Musso  ---
From bug 47031 : https://gerrit.wikimedia.org/r/#/c/57436/ has been merged
although it is faulty.

The unit tests ran on patchset upload did catch the issue:

https://integration.wikimedia.org/ci/job/mediawiki-core-phpunit-misc/5222/console
: FAILURE

But the gating run after CR+2 did not catch it:

https://integration.wikimedia.org/ci/job/mediawiki-core-phpunit-misc/5223/console
: SUCCESS


The root cause is that despite the ZUUL_REF points to the proper merge commit,
the Jenkins Git plugin seems to use the current origin/master to build.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

--- Comment #1 from Gerrit Notification Bot  ---
Related URL: https://gerrit.wikimedia.org/r/58283 (Gerrit Change
I4b3fadccaae9c35964a0c47d63b22c4f35148a24)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Ċ½eljko Filipin  changed:

   What|Removed |Added

 CC|zfili...@wikimedia.org  |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-04-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Andre Klapper  changed:

   What|Removed |Added

   Priority|Unprioritized   |High

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-03-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

jeremyb  changed:

   What|Removed |Added

 CC||bugzilla+org.wikimedia@tuxm
   ||achine.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-03-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Alex Monk (Krenair)  changed:

   What|Removed |Added

 CC||kren...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-03-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Liangent  changed:

   What|Removed |Added

 CC||liang...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 46723] Jenkins merged a faulty change

2013-03-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46723

Antoine "hashar" Musso  changed:

   What|Removed |Added

 CC||aschulz4...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l