Project

General

Profile

Actions

action #111215

closed

QA (public) - 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 over 2 years ago. Updated over 2 years ago.

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

0%

Estimated time:

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

Files


Related issues 1 (1 open0 closed)

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

Actions
Actions #1

Updated by tinita over 2 years ago

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

Updated by tinita over 2 years ago

  • Description updated (diff)
Actions #3

Updated by okurz over 2 years ago

  • Target version set to Ready
Actions #4

Updated by okurz over 2 years 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?

Actions #5

Updated by mkittler over 2 years 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
Actions #6

Updated by okurz over 2 years ago

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

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

Actions #7

Updated by tinita over 2 years ago

  • Status changed from Workable to In Progress
  • Assignee set to tinita
Actions #8

Updated by tinita over 2 years 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)

Actions #9

Updated by tinita over 2 years ago

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

Actions #10

Updated by tinita over 2 years ago

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

Still blocked by the above PRs

Actions #11

Updated by tinita over 2 years 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

Actions #12

Updated by tinita over 2 years 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.

Actions #13

Updated by tinita over 2 years ago

  • Status changed from Feedback to Resolved

Deployed and everything looks good so far.

Actions

Also available in: Atom PDF