Bug 1444038
| Summary: | Chargeback for container images report editor filter tab produces an error if there are too many images in the database | ||
|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Satoe Imaishi <simaishi> |
| Component: | Reporting | Assignee: | Ari Zellner <azellner> |
| Status: | CLOSED ERRATA | QA Contact: | Einat Pacifici <epacific> |
| Severity: | high | Docs Contact: | |
| Priority: | urgent | ||
| Version: | 5.7.0 | CC: | agrare, azellner, fsimonce, jhardy, mtayer, obarenbo, simaishi, slukasik |
| Target Milestone: | GA | ||
| Target Release: | 5.8.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | container:chargeback | ||
| Fixed In Version: | 5.8.0.12 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1442158 | Environment: | |
| Last Closed: | 2017-05-31 14:55:47 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | Container Management | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1442158 | ||
| Bug Blocks: | |||
|
Comment 2
CFME Bot
2017-04-20 22:13:19 UTC
New commit detected on ManageIQ/manageiq/fine: https://github.com/ManageIQ/manageiq/commit/71fa007377196b7a97c941c79692d8c049df129d commit 71fa007377196b7a97c941c79692d8c049df129d Author: Šimon Lukašík <isimluk> AuthorDate: Thu Apr 20 10:00:47 2017 +0200 Commit: Satoe Imaishi <simaishi> CommitDate: Thu Apr 20 18:29:50 2017 -0400 Merge pull request #14816 from zeari/fix_nil_images_in_chargeback group chargeback with unknown image under 'unknown image' (cherry picked from commit 993774e1eb0d36a800ea60d7ed04eb1be2b38c8f) https://bugzilla.redhat.com/show_bug.cgi?id=1444038 app/models/chargeback_container_image.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Ari, should this be on POST ? Also needs https://github.com/ManageIQ/manageiq/pull/14811 listed in the opening message since https://github.com/ManageIQ/manageiq/pull/14811 is also merged to fine can we move this to POST? right, i didnt see the commit message here. moving to POST. Ari, since this BZ requires a DB with 40K+ images (which is something we don't have in QE) can you please provide some details on how to setup/reproduce this? Einat, we can generate fake images that would be 'good enough' to try to blow up the UI.
1) open rails console with 'rails c'
2) paste in: (1..40000).each {|x| ContainerImage.create(:name => x.to_s, :ext_management_system => ExtManagementSystem.first)}
3) in Chargeback based on container images report editor, go to the filter tab
4) under provider, choose the first one there.
5) at this point the UI should error or let you choose between 40K images under the 'Container Image' dropdown
Verified. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2017:1367 |