[Buildbot-devel] email lookup is broken?

[Dustin J. Mitchell]

Well, that should be specifying src='git', but now that I look at the
user support in
https://github.com/buildbot/buildbot/blob/master/master/buildbot/process/users/users.py
it doesn't look like that properly breaks down the authorship
information, so that would need to be fixed as well.

Dustin

On Sat, Feb 21, 2015 at 9:20 PM, Vitali Lovich <vlovich at gmail.com> wrote:
> Perhaps it’s a bug in the change hook for github?
>
>  89                 change = {
>  90                     'author': '%s <%s>' % (
>  91                         commit['author']['name'],
> commit['author']['email']
>  92                     ),
>  93                     'files': files,
>  94                     'comments': commit['message'],
>  95                     'revision': commit['id'],
>  96                     'when_timestamp': when_timestamp,
>  97                     'branch': branch,
>  98                     'revlink': commit['url'],
>  99                     'repository': repo_url,
> 100                     'project': project
> 101                 }
> 102
>
> Or am I looking into the wrong part of the codebase?
> Is there something else that then decodes ‘author’ from the change
> & recovers the e-mail?
>
> -Vitali
>
> On Feb 21, 2015, at 6:18 PM, Dustin J. Mitchell <dustin at v.igoro.us> wrote:
>
> The git changesource should be setting the email attribute to the
> email portion of the git name.  If it's not, that's a bug!
>
> Dustin
>
> On Thu, Feb 19, 2015 at 2:45 PM, Vitali Lovich <vlovich at gmail.com> wrote:
>
> Hi,
>
> I’m trying to use the mail notifier, but it’s complaining that:
> Unable to find any of ['email'] for uid: 2
>
> All my changes have an ‘author’ field that is the username & email.
>
> My sqlite database schemas look like:
>
> select * from users;
> uid         identifier                         bb_username  bb_password
> ----------  ---------------------------------  -----------  -----------
> 2           Vitali Lovich <email address>   NULL         NULL
>
> select * from users_info;
> uid         attr_type   attr_data
> ----------  ----------  ---------------------------------
> 2           git         Vitali Lovich <email address>
>
> I don’t see any column named e-mail.  Is this a bug?
> My understanding was that the MailNotifier could be used without needing a
> custom lookup.
>
> Thanks,
> Vitali
> ------------------------------------------------------------------------------
> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
> from Actuate! Instantly Supercharge Your Business Reports and Dashboards
> with Interactivity, Sharing, Native Excel Exports, App Integration & more
> Get technology previously reserved for billion-dollar corporations, FREE
> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
> _______________________________________________
> Buildbot-devel mailing list
> Buildbot-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/buildbot-devel
>
>