Bug 1394746

Summary: Schedule report sends mail to maximum 5 email IDs
Product: Red Hat CloudForms Management Engine Reporter: Sachin <sacpatil>
Component: ReportingAssignee: Joe Rafaniello <jrafanie>
Status: CLOSED NOTABUG QA Contact: Jiri Stefanisin <jistefan>
Severity: high Docs Contact:
Priority: unspecified    
Version: 5.6.0CC: gtanzill, jhardy, jistefan, jrafanie, myoder, obarenbo, sacpatil
Target Milestone: GAKeywords: WorkAround
Target Release: cfme-future   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: report
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-09 11:07:52 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Work in progress patch to remove grouping of emails none

Description Sachin 2016-11-14 11:35:26 UTC 
Description of problem:

When an external smtp server is set, schedule report is able to send report to maximum number of 5 email.


Version-Release number of selected component (if applicable):
5.6.2.2


How reproducible:

Always reproducible


Steps to Reproduce:

Set external SMTP server

1. Go to 'Setting' > Configuration > Setting accordian -> zone(server EVM) > Outgoing SMTP E-mail Server 
2.Schedule a report and add more than 5 emails


Actual results:

Report is sent to max. 5 emails(not it order)

Expected results:

Should be able to sent to as many people as possible


Additional info:

This works file for default(localhost) SMTP server
Comment 13 Joe Rafaniello 2016-11-21 20:54:49 UTC 
Created attachment 1222482 [details]
Work in progress patch to remove grouping of emails

The attached patch is very rough and untested but would give us the ability to not group emails and instead try to send with as many recipients as possible.  It's unclear if this would introduce a problem for some mail servers/clients but any feedback from a test environment would be greatly appreciated.
Comment 14 Joe Rafaniello 2016-11-21 20:55:58 UTC 
The above patch should apply to 5.6+ version of cfme.
Comment 15 Sachin 2016-11-29 10:02:11 UTC 
Let me test the patch on my end and provide feedback
Comment 16 Dave Johnson 2016-12-06 19:19:54 UTC 
Reducing severity, it doesn;t fit urgent criteria and there is a workaround... use a mailing list.
Comment 17 Sachin 2016-12-14 09:39:38 UTC 
Dave,

I idea of using a mailing list is not feasible for CU as the group may have few people in some case. It is not possible for them to create mailing list involving few number of people.

@Jiri: Any idea about testing of this patch? I can manually patch this in CU environment provided this does not effect other functionality within cloudforms.