Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for storages under storage pods #2892

Merged
merged 2 commits into from Dec 4, 2017

Conversation

martinpovolny
Copy link

Selected records should be passed to get_view only when records cannot be specified using associations or named scopes.

Furthemore this imperfect implementation was broken in during the GTL conversion.

https://bugzilla.redhat.com/show_bug.cgi?id=1501063

Ping @lgalis, @tumido : please, review

Martin Povolny added 2 commits November 30, 2017 22:23
Selected records should be passed to get_view only when records cannot
be specified using associations or named scopes.

Furthemore this imperfect implementation was broken in during the GTL
conversion.
@miq-bot
Copy link
Member

miq-bot commented Nov 30, 2017

Some comments on commits martinpovolny/manageiq-ui-classic@d1a383c~...d000d5f

spec/controllers/storage_controller_spec.rb

  • ⚠️ - 272 - Detected expect_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.

@miq-bot
Copy link
Member

miq-bot commented Nov 30, 2017

Checked commits martinpovolny/manageiq-ui-classic@d1a383c~...d000d5f with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 1 offense detected

spec/controllers/storage_controller_spec.rb

@martinpovolny martinpovolny changed the title Fix for storages under storagage pods Fix for storages under storage pods Dec 1, 2017
@mzazrivec mzazrivec self-assigned this Dec 4, 2017
@mzazrivec mzazrivec added this to the Sprint 75 Ending Dec 11, 2017 milestone Dec 4, 2017
@mzazrivec mzazrivec merged commit fd1faec into ManageIQ:master Dec 4, 2017
simaishi pushed a commit that referenced this pull request Dec 4, 2017
@simaishi
Copy link
Contributor

simaishi commented Dec 4, 2017

Gaprindashvili backport details:

$ git log -1
commit c2332c648f2e40cdcb57cc2b0d0da1fb06ac697c
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Mon Dec 4 08:43:24 2017 +0100

    Merge pull request #2892 from martinpovolny/storage_pods_fix
    
    Fix for storages under storage pods
    (cherry picked from commit fd1faecce6ea4bfa65bc2fd8fb1c7fc345a05ea4)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1520530

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants