Bug 732951
Summary: | errata mailer doesn't detect and list correct number and list of affected systems | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Satellite 5 | Reporter: | Tomas Lestach <tlestach> | ||||
Component: | Server | Assignee: | Tomas Lestach <tlestach> | ||||
Status: | CLOSED DUPLICATE | QA Contact: | Red Hat Satellite QA List <satqe-list> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | 540 | CC: | cperry | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2011-10-20 19:59:33 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 677498 | ||||||
Attachments: |
|
Description
Tomas Lestach
2011-08-24 09:27:12 UTC
Created attachment 519642 [details]
Text file with lots of SQL text and results
Attaching my own quick review.
IT seems highly likely (with a slow down) that using DISTINCT with the SQL will pass to the Errata mailer a correct list of systems effected per org, per user for each errata.
Within java/code/src/com/redhat/rhn/taskomatic/task/ErrataMailer.java it seems that the getUserServers will correctly list servers for each user. So likely this distinct would indeed pass correct data to the java layer, before mailing out. We could also distinct the data in java stack - vs assuming we get correct unique data from query. Cliff The distinct reduced from 48 (8x6) results, to 16 (2x8). Since there is two users, 16 results actually is the correct looking data set to return. *** This bug has been marked as a duplicate of bug 640535 *** |