Project

General

Profile

Actions

action #30124

closed

don't set mail adress for system user

Added by lnussel over 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2018-01-10
Due date:
% Done:

0%

Estimated time:

Description

The system user in openqa has a mail address noemail@open.qa set. That results in a rather ugly gravatar that cannot be changed as the address goes to /dev/null:
https://openqa.opensuse.org/tests/576994#comments

The ttm user on the other hand does not have a mail address which results in a better icon (maybe openqa could avoid asking gravatar in the first place for those). So I'd suggest to unset the mail address for the system user too.

Actions #1

Updated by coolo almost 6 years ago

  • Category set to Feature requests
  • Target version set to Ready
  • Difficulty set to easy
Actions #2

Updated by lnussel over 5 years ago

  • Assignee set to okurz
Actions #3

Updated by coolo over 5 years ago

okurz, handling this?

Actions #4

Updated by coolo over 5 years ago

  • Target version changed from Ready to Current Sprint
Actions #5

Updated by okurz over 5 years ago

  • Status changed from New to In Progress

This email adress nomail@openqa.qa is neither set in openQA source code nor salt pillars nor states. I assume either set locally in config on osd/o3 or manually in the database on at least o3, maybe osd as well. will check.

Actions #6

Updated by coolo over 5 years ago

that's just a typo of lnussel:

dbicdh/_common/upgrade/49-50/01-add-system-user.pl:            email    => 'noemail@open.qa',
dbicdh/_common/upgrade/50-51/01-add-system-user.pl:            email    => 'noemail@open.qa',
lib/OpenQA/Schema.pm:                email    => 'noemail@open.qa',
t/06-users.t:    is($system_user->email, 'noemail@open.qa', 'system user`s email uses open.qa domain');
Actions #7

Updated by okurz over 5 years ago

  • Description updated (diff)

It even was a double "typo" mistake :/

ok, I called update users set email = null where username = 'system'; for the database on o3 and the icon e.g. in https://openqa.opensuse.org/tests/576994#comments looks the same now for both ttm and system. Now the icons are not unique anymore. I am not convinced this is better. I guess we could create unique email adresses and register them properly with gravatar and select an appropriate icon for each. I like the github https://uxdesign.cc/design-avatars-that-make-sense-and-be-more-inclusive-in-the-process-d4dd6a486ea6 more than the gravatar avatars when one does not provide a custom picture. Or we try to find a nicer looking gravatar pic by playing around with the fake email adress we give to "system". This is what I did now with the SQL command update users set email = 'system@open.qa' where username = 'system'; which brings us this which I think looks nicer by chance. If you like it as well then I can provide a PR to change the email adress field in source code. I doubt anyone would write an email to "system@open.qa" who would also not by mistake write to "noemail@open.qa". However, what about changing the type of the non-custom icons to something different, e.g. "identicon"? This would give us this for "system@open.qa" or this for "noemail@open.qa"

Actions #8

Updated by coolo over 5 years ago

We can't change the gravatar for email adresses we can't receive email for. So leave it with system if you like that better.

Actions #9

Updated by coolo over 5 years ago

Build service uses a different default which results in and

Both look a lot like system users :)

Actions #10

Updated by coolo over 5 years ago

So that sounds like an easy fix: stick to noemail@ but use wavatar instead of monsterid. All monsters on openqa.suse.de look silly, while the wavatars look reasonable.

Actions #11

Updated by okurz over 5 years ago

  • Status changed from In Progress to Feedback

Well, as it was your decision to come up with monsters I do not see a problem to change that ;) -> https://github.com/os-autoinst/openQA/pull/1822

Actions #12

Updated by coolo over 5 years ago

Just as it was my decision to use wavatar for OBS: https://github.com/openSUSE/open-build-service/commit/c2e2304b655bfc6bc592b88d988027c6b16beccb

No progress without mistakes :)

Actions #13

Updated by okurz over 5 years ago

  • Status changed from Feedback to Resolved

I meant: As you already commented that you prefer the new way I do not need to ask someone else

Actions #14

Updated by lnussel over 5 years ago

of the three choices identicon looks least childish.

Actions #15

Updated by coolo over 5 years ago

  • Target version changed from Current Sprint to Done

Well, seriousness wasn't the goal :)

Actions

Also available in: Atom PDF