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
Scan Storage via EMS not Host #3166
Scan Storage via EMS not Host #3166
Conversation
Remove the requirement for hosts with valid credentials to be able to run storage smartstate. The valid unsupported reasons will be moved to the Storage model. https://bugzilla.redhat.com/show_bug.cgi?id=1529725
Checked commit agrare@5dcd9e3 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
I'm guessing spec failures will clean up once the backend PRs are merged? |
Exactly I moved the supported checks to the storage model (where they should be IMO) so once that's in this will pass. |
Backend PRs merged. Bounced this PR. |
Hm the javascript tests timed out, kicking the tests again |
Looks like the same issue on master https://travis-ci.org/ManageIQ/manageiq-ui-classic/jobs/329142187#L2755-L2758 |
@Fryguy travis is green |
…a_ems Scan Storage via EMS not Host (cherry picked from commit 805fae2) https://bugzilla.redhat.com/show_bug.cgi?id=1534753
Gaprindashvili backport details:
|
Remove the requirement for hosts with valid credentials to be able to
run storage smartstate. The valid unsupported reasons will be moved to
the Storage model.
Depends ManageIQ/manageiq#16743
https://bugzilla.redhat.com/show_bug.cgi?id=1529725