Bug 1518538
Summary: | Wrong project names on Ad Hoc matrics page cause to internal server error | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Shalom Naim <snaim> | ||||||
Component: | Providers | Assignee: | Yaacov Zamir <yzamir> | ||||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Einat Pacifici <epacific> | ||||||
Severity: | high | Docs Contact: | |||||||
Priority: | urgent | ||||||||
Version: | 5.9.0 | CC: | cpelland, epacific, gblomqui, jfrey, jhardy, lavenel, obarenbo, simaishi, yzamir | ||||||
Target Milestone: | GA | Keywords: | TestOnly | ||||||
Target Release: | 5.10.0 | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Whiteboard: | containers | ||||||||
Fixed In Version: | 5.10.0.0 | Doc Type: | If docs needed, set a value | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | |||||||||
: | 1520614 1523256 1523774 (view as bug list) | Environment: | |||||||
Last Closed: | 2018-06-21 21:15:57 UTC | Type: | Bug | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | Container Management | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Bug Depends On: | |||||||||
Bug Blocks: | 1520614, 1523774 | ||||||||
Attachments: |
|
Description
Shalom Naim
2017-11-29 07:04:55 UTC
How reproducible: 100% a. The fix for the UI is trivial b. The underlying bug is in Hawkualr (*) and the UI is correctly showing the wrong data it gets from Hakular's API. We need PM advice: Do we fix the UI and hide the underliying bug in Hawkular ? Or Do we wait for a fix in Hawkular, that may not be in time for users using the buggy versions Hawkular. (*) according to Hawkular API documentation the query should return a list of "tenant_names", the "tenants_name:hash" returned value not as advertised. http://www.hawkular.org/docs/rest/rest-metrics.html#_tenant Opened a POC on how we can work around this in the UI: https://github.com/ManageIQ/manageiq-ui-classic/pull/2860 this patch is not for use, just to show we can work around this if we need to. submitted upstream (a fix on manageiq side) https://github.com/ManageIQ/manageiq-ui-classic/pull/2860 merged upstream (a fix on manageiq side) https://github.com/ManageIQ/manageiq-ui-classic/pull/2860 Created attachment 1375148 [details]
bug verification
|