Project

General

Profile

action #111215

QA - coordination #99303: [saga][epic] Future improvements for SUSE Maintenance QA workflows with fully automated testing, approval and release

coordination #99306: [epic] Future improvements: Make reviewing openQA results per squad easier

Various improvements for email notification about unreviewed jobs size:M

Added by tinita about 1 month ago. Updated 27 days ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Feature requests
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Observations

  1. Expanding a message creates a lot of vertical white space.
  2. The job group ID is not very useful - better use the group name
  3. The email sender name is quite long, it could be shorter, e.g. "OSD review"
  4. Links are not clickable

Acceptance criteria

  • AC1: URLs are clickable
  • AC2: Human-readable group names are visible
  • AC3: Retain plain text email content

Suggestions

  1. This seems to be a Slack problem - the original emails don't have extra trailing lines
  2. Should be very easy
  3. Try if URLs can be clickable in subject or if there is any formatting other than HTML that would work
  4. We need to send HTML emails instead, then links are clickable (tested)
  5. See also #111135
  6. Send a multi-part email with markdown and HTML

Related issues

Related to openQA Project - action #111135: Enhance email notification message content for about failed and unreviewed jobs New2022-05-16

History

#1 Updated by tinita about 1 month ago

  • Related to action #111135: Enhance email notification message content for about failed and unreviewed jobs added

#2 Updated by tinita about 1 month ago

  • Description updated (diff)

#3 Updated by okurz about 1 month ago

  • Target version set to Ready

#4 Updated by okurz about 1 month ago

I would be happy if we could avoid sending HTML emails. Maybe there is a special format, maybe markdown?, that would make links clickable? Maybe if the URL is even in the subject line?

#5 Updated by mkittler about 1 month ago

  • Subject changed from Various improvements for email notification about unreviewed jobs to Various improvements for email notification about unreviewed jobs size:M
  • Description updated (diff)
  • Status changed from New to Workable

#6 Updated by okurz about 1 month ago

13286

I found that in my Android Slack client the URLs are already clickable, see

photo_2022-05-20_19-36-53.jpg

#7 Updated by tinita about 1 month ago

  • Status changed from Workable to In Progress
  • Assignee set to tinita

#8 Updated by tinita about 1 month ago

https://github.com/os-autoinst/scripts/pull/164 Add dependency Text::Markdown (merged)

https://github.com/os-autoinst/os-autoinst-common/pull/21 Add test for make update-deps (merged)
https://github.com/os-autoinst/scripts/pull/165 Add test for make update-deps (merged)

#9 Updated by tinita about 1 month ago

https://github.com/os-autoinst/openQA/pull/4675 Allow markdown in apparmor (merged)

#10 Updated by tinita about 1 month ago

https://github.com/os-autoinst/scripts/pull/166 Send multipart email to improve formatting

Still blocked by the above PRs

#11 Updated by tinita 28 days ago

  • Status changed from In Progress to Feedback

https://github.com/os-autoinst/scripts/pull/166 merged

I added a trigger to automatically rebuild https://build.opensuse.org/package/show/devel:openQA/os-autoinst-scripts on push, which updates os-autoinst-scripts-deps

#12 Updated by tinita 28 days ago

https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/689 by okurz (for installing/updating os-autoinst-scripts-deps on osd) was merged.

#13 Updated by tinita 27 days ago

  • Status changed from Feedback to Resolved

Deployed and everything looks good so far.

Also available in: Atom PDF