Last modified: 2014-02-15 23:36:20 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 T54016, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52016 - gerrit stream-events reports older subject, not changed subject, when a new patchset is uploaded that changes the subject
gerrit stream-events reports older subject, not changed subject, when a new p...
Status: RESOLVED WORKSFORME
Product: Wikimedia
Classification: Unclassified
Git/Gerrit (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: upstream
: 54372 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-25 08:43 UTC by MZMcBride
Modified: 2014-02-15 23:36 UTC (History)
7 users (show)

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


Attachments

Description MZMcBride 2013-07-25 08:43:34 UTC
There's a minor bug in grrrit-wm that goes something like this:

* commit message is "foo"
* user edits and submits commit message to be "bar"
* when grrrit-wm reports new patchset, it reports "foo"

Yuvi says this is an upstream bug in Gerrit. This needs further investigation.
Comment 1 MZMcBride 2013-07-25 08:49:05 UTC
From Yuvi:

{
    "type": "patchset-created",
    "change": {
        "project": "mediawiki/extensions/MassMessage",
        "branch": "master",
        "topic": "apiedit",
        "id": "Ib51f2f4a65a45d70e278c13c198733fd47bc3229",
        "number": "75299",
        "subject": "[WIP] Edit via the API",
        "owner": {
            "name": "Legoktm",
            "email": "legoktm.wikipedia@gmail.com",
            "username": "legoktm"
        },
        "url": "https://gerrit.wikimedia.org/r/75299",
        "status": "NEW"
    },
    "patchSet": {
        "number": "4",
        "revision": "5a10eb75f79055afca9f23f4195d068236d64515",
        "parents": [
            "3890db98317a855bca2645e5730f9e9404493f6b"
        ],
        "ref": "refs/changes/99/75299/4",
        "uploader": {
            "name": "MZMcBride",
            "email": "w@mzmcbride.com",
            "username": "mzmcbride"
        },
        "createdOn": 1374741950,
        "author": {
            "name": "Legoktm",
            "email": "legoktm.wikipedia@gmail.com",
            "username": "legoktm"
        },
        "sizeInsertions": 23,
        "sizeDeletions": -10
    },
    "uploader": {
        "name": "MZMcBride",
        "email": "w@mzmcbride.com",
        "username": "mzmcbride"
    }
}{
    "type": "comment-added",
    "change": {
        "project": "mediawiki/extensions/MassMessage",
        "branch": "master",
        "topic": "apiedit",
        "id": "Ib51f2f4a65a45d70e278c13c198733fd47bc3229",
        "number": "75299",
        "subject": "[WIP] Edit via the MediaWiki Web API",
        "owner": {
            "name": "Legoktm",
            "email": "legoktm.wikipedia@gmail.com",
            "username": "legoktm"
        },
        "url": "https://gerrit.wikimedia.org/r/75299",
        "status": "NEW"
    },
    "patchSet": {
        "number": "4",
        "revision": "5a10eb75f79055afca9f23f4195d068236d64515",
        "parents": [
            "3890db98317a855bca2645e5730f9e9404493f6b"
        ],
        "ref": "refs/changes/99/75299/4",
        "uploader": {
            "name": "MZMcBride",
            "email": "w@mzmcbride.com",
            "username": "mzmcbride"
        },
        "createdOn": 1374741950,
        "author": {
            "name": "Legoktm",
            "email": "legoktm.wikipedia@gmail.com",
            "username": "legoktm"
        },
        "sizeInsertions": 23,
        "sizeDeletions": -10
    },
    "author": {
        "name": "jenkins-bot",
        "username": "jenkins-bot"
    },
    "approvals": [
        {
            "type": "Verified",
            "description": "Verified",
            "value": "1"
        }
    ],
    "comment": "Patch Set 4: Verified+1\n\nBuild succeeded.\n\n- https://integration.wikimedia.org/ci/job/mwext-MassMessage-lint/38/console : SUCCESS in 0s"
}
Comment 2 MZMcBride 2013-07-25 08:49:49 UTC
(In reply to comment #1)
comment-added subject is fine; patchset-created subject is wrong.
Comment 3 MZMcBride 2014-02-15 23:33:56 UTC
Bug 54372 is likely a duplicate of this bug report. This issue may be fixed.
Comment 4 Bartosz Dziewoński 2014-02-15 23:36:20 UTC
*** Bug 54372 has been marked as a duplicate of this bug. ***

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


Navigation
Links