Bug 1291445
| Summary: | CFME 5.3.3 Zone Level Log collection incorrectly references and uses the appliance level log depot instead of zone level specified depot | ||
|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | John Prause <jprause> |
| Component: | Appliance | Assignee: | Libor Pichler <lpichler> |
| Status: | CLOSED ERRATA | QA Contact: | Kyrylo Zvyagintsev <kzvyahin> |
| Severity: | medium | Docs Contact: | |
| Priority: | high | ||
| Version: | 5.3.0 | CC: | abellott, bmorriso, cpelland, dajohnso, jdeubel, jhardy, jkrocil, jocarter, jprause, jrafanie, kzvyahin, mfeifer, obarenbo, thenness |
| Target Milestone: | GA | Keywords: | ZStream |
| Target Release: | 5.5.2 | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1197869 | Environment: | |
| Last Closed: | 2016-02-10 15:24:49 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1197869 | ||
| Bug Blocks: | |||
|
Comment 2
Joe Rafaniello
2016-01-19 14:28:14 UTC
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. Thanks! New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3b2299330c6aea11cb1d9142654f8ebc2c29ee84 commit 3b2299330c6aea11cb1d9142654f8ebc2c29ee84 Merge: 73e18b0 f454638 Author: Joe Rafaniello <jrafanie> AuthorDate: Wed Jan 20 14:25:31 2016 -0500 Commit: Joe Rafaniello <jrafanie> 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 https://bugzilla.redhat.com/show_bug.cgi?id=1291445 https://github.com/ManageIQ/manageiq/pull/5864 It was clean cherry-pick expect to some specs(diagnostic_specs.rb, log_management_spec.rb), but there were just few conflicts about converting matchers: XXXX.should to XXX.to 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. https://access.redhat.com/errata/RHBA-2016:0159 |