Red Hat Bugzilla – Bug 1291445
CFME 5.3.3 Zone Level Log collection incorrectly references and uses the appliance level log depot instead of zone level specified depot
Last modified: 2016-02-10 10:24:49 EST
Implemented upstream in https://github.com/ManageIQ/manageiq/pull/5864, fixing: https://github.com/ManageIQ/manageiq/issues/1962
Libor, can you see how hard it is to backport this to 5.5.z? From your changes, I don't see a lot of conflicts happening but you never know until you try.
New commit detected on cfme/5.5.z:
Merge: 73e18b0 f454638
Author: Joe Rafaniello <firstname.lastname@example.org>
AuthorDate: Wed Jan 20 14:25:31 2016 -0500
Commit: Joe Rafaniello <email@example.com>
CommitDate: Wed Jan 20 14:25:31 2016 -0500
Merge branch 'bz_1291445' into '5.5.z'
Change logic of choosing depot for collection log
It was clean cherry-pick expect to some specs(diagnostic_specs.rb, log_management_spec.rb), but there were just few conflicts about
See merge request !736
app/controllers/ops_controller/diagnostics.rb | 3 +-
app/helpers/application_helper/toolbar_builder.rb | 12 +-
app/models/miq_server/log_management.rb | 17 ++-
app/models/zone.rb | 5 +-
app/views/ops/_logs_selected.html.haml | 2 +-
.../controllers/ops_controller/diagnostics_spec.rb | 10 +-
.../application_helper/toolbar_builder_spec.rb | 111 +++++++++++++-
spec/models/miq_server/log_management_spec.rb | 163 +++++++++++++++++++--
8 files changed, 290 insertions(+), 33 deletions(-)
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.