Remove datawarehouse views #1502
Conversation
|
Travis failure seems related. |
|
This pull request is not mergeable. Please rebase and repush. |
|
@moolitayer : what is the status of this PR? Do you need any assistance? Thx! |
martinpovolny
changed the title
|
@martinpovolny I will get back to working on this as soon as I can |
|
@moolitayer : ping? |
moolitayer
force-pushed
the
remove_datawarehouse_views
branch
from
444acc6
to
6f22bf7
Compare
|
@martinpovolny please review. Should I remove the locale stuff as well? |
moolitayer
changed the title
|
@miq-bot add_label gaprindashvili/no |
|
@miq-bot remove_label wip |
moolitayer
force-pushed
the
remove_datawarehouse_views
branch
2 times, most recently
from
4480f1a
to
b0731a4
Compare
|
@martinpovolny I'll do another run on the original PR tomorrow to make sure I didn't forget stuff |
|
@martinpovolny ping |
|
This pull request is not mergeable. Please rebase and repush. |
|
Please, rebase and ping me. Thx! |
moolitayer
force-pushed
the
remove_datawarehouse_views
branch
from
b0731a4
to
b35b48b
Compare
|
Checked commit moolitayer@b35b48b with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
|
This pull request is not mergeable. Please rebase and repush. |
moolitayer
force-pushed
the
remove_datawarehouse_views
branch
from
b35b48b
to
8d9a0fa
Compare
moolitayer
force-pushed
the
remove_datawarehouse_views
branch
from
8d9a0fa
to
8cd69e6
Compare
|
Rebased, fixed tests & checked ManageIQ/manageiq#12205 to make sure nothing is missing. Ping @martinpovolny |
We are moving away from using the datawarehouse manager as an external provider to integrating it in container providers. Removing all screens added in ManageIQ/manageiq#12205
Needed for ManageIQ/manageiq#15295