I am not sure who maintains Gerrit bot, it seems it always use the commit author as the owner and the committer field for the by action. That does not match the reality though :]

Im currently the maintainer of GerritBot source code (its-phabricator) the template your seeing is https://github.com/wikimedia/puppet/blob/production/modules/gerrit/files/homedir/review_site/etc/its/templates/PatchSetCreated.soy

Commit also has "owner" field and I noted that gerritbot provides different owner in this comment.

If gerritbot can read author and committer field and "owner" is taken from author field, then I suppose it would be less confusing and sufficient if in this template "owner" was changed to "author".

See https://phabricator.wikimedia.org/source/operations-puppet/browse/production/modules/gerrit/templates/its/

The template has:

Change {$changeNumber} had a related patch set uploaded (by {$uploaderName ? $uploaderName : $uploaderUsername}; owner: {$authorName ? $authorName : $authorUsername}):{\n}

The change owner is EBernhardson since he has uploaded.

The "owner" in the message is actually the author which leads to the confusion, so I guess we can just change that to use "author".

So tentatively instead of:

Change 512231 had a related patch set uploaded (by EBernhardson; owner: DCausse):

Use:

Change 512231 had a related patch set uploaded (by EBernhardson; author: DCausse):

There are other fields available and described at https://gerrit.wikimedia.org/r/plugins/its-phabricator/Documentation/config-rulebase-common.md#common-properties-for-events-on-a-change , which get us:

ownerName
name of the change's owner.

ownerUsername
username of the change's owner.

But I don't think we want to list all three fields (uploader, author, owner).

Change 674552 had a related patch set uploaded (by Hashar; owner: Hashar):
[operations/puppet@production] gerrit: reported owner is actually patchset author

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

Change 674552 merged by Dzahn:
[operations/puppet@production] gerrit: reported owner is actually patchset author

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

Change 674810 had a related patch set uploaded (by Hashar; author: Pikne):
[test/gerrit-ping@master] Test its-phabricator messages

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

Change 674810 abandoned by Hashar:
[test/gerrit-ping@master] Test its-phabricator messages

Reason:
Shown as Change 674810 had a related patch set uploaded (by Hashar; author: Pikne): ! :)

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