New commit detected on ManageIQ/manageiq-ui-classic/fine: https://github.com/ManageIQ/manageiq-ui-classic/commit/644a960dd8288d4c408debdac275b9b742e918fc commit 644a960dd8288d4c408debdac275b9b742e918fc Author: Martin Povolny <mpovolny> AuthorDate: Thu Apr 20 10:49:38 2017 +0200 Commit: Satoe Imaishi <simaishi> CommitDate: Thu Apr 20 18:06:59 2017 -0400 Merge pull request #1077 from isimluk/rhbz#1442158 Do not store openshift env in the controller, use lookup helper instead (cherry picked from commit 287dc37c723d86d1e4fe6657a98e9dc29d3980d0) https://bugzilla.redhat.com/show_bug.cgi?id=1444038 .../report_controller/reports/editor.rb | 8 -------- app/helpers/report_helper.rb | 2 ++ app/helpers/report_helper/editor.rb | 12 ++++++++++++ app/views/report/_form_filter_chargeback.html.haml | 3 +-- spec/controllers/report_controller_spec.rb | 22 ---------------------- 5 files changed, 15 insertions(+), 32 deletions(-) create mode 100644 app/helpers/report_helper/editor.rb
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