Bug 1584355
| Summary: | Slow performance with displaying catalog order dialog | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Ryan Spagnola <rspagnol> | ||||||||
| Component: | UI - OPS | Assignee: | Tina Fitzgerald <tfitzger> | ||||||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Shveta <sshveta> | ||||||||
| Severity: | high | Docs Contact: | |||||||||
| Priority: | high | ||||||||||
| Version: | 5.9.0 | CC: | cpelland, duhlmann, hkataria, jcrumple, jprause, lavenel, mpovolny, myoder, obarenbo, rovalent, rspagnol, simaishi, sshveta, tfitzger | ||||||||
| Target Milestone: | GA | Keywords: | TestOnly, ZStream | ||||||||
| Target Release: | 5.10.0 | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | 5.10.0.4 | Doc Type: | If docs needed, set a value | ||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | |||||||||||
| : | 1591427 (view as bug list) | Environment: | |||||||||
| Last Closed: | 2019-02-11 14:09:05 UTC | Type: | Bug | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | Bug | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | CFME Core | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Bug Depends On: | |||||||||||
| Bug Blocks: | 1591427 | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Ryan Spagnola
2018-05-30 18:01:16 UTC
Hi Ryan, Can you ask the customer to provide the 5.8 and 5.9 dialogs and the Automate domain export? Thanks, Tina Hi Ryan, Thanks for the update. Can you clarify that this issue is being reported against the OPSUI and not the SUI? I see the related ticket referenced is for the SUI. Thanks, Tina Confirmed, this is occurring when we are using the OPSUI. Shveta setup appliance: https://10.8.198.204 imported the dialog and automate method created catalog item I can for her, I think. https://github.com/ManageIQ/manageiq/pull/17436 hasn't been backported and it will fix at least a good part of this issue. 17436 just got merged so this needs to be retested. Hi Ryan, During the debugging session with the customer last week, we determined that some of the performance issues the customer is experiencing are due to the large amount of data being processed by multiple dynamic dialog entries. There is one case where 4 dialog elements execute the same Automate method. We discussed 3 ways to improve performance: 1. A fix for running Automate multiples times was back ported this week. Customer can request a hot fix. https://github.com/ManageIQ/manageiq/pull/17436 - master PR https://github.com/ManageIQ/manageiq/commit/f984d7be552833dc5f03f5c5fad4982ca0a49626 - G release commit 2. Expression methods can be used in place of ruby methods for dynamic dialogs. More information here: http://talk.manageiq.org/t/automate-expression-methods/3071 3. For the method that is executed 4 times: Dialog Data Caching: Execute the method once and store the data to present in the 3 other dialog elements. Directions below: Dialog Data Caching: Attachments contain two files (Dialog and Automate-domain exports) that show a caching example. The example uses two fields. The first field creates some json data and stores it into the field value. The second field does a `JSON.load` on the data from the first field and uses one of the json keys to display in the second field. The first field holding the json data would normally be hidden but it is visible to make it easier to see when running the dialog. To test, import the two attached files into an appliance, make sure the new domain "dialog_caching" is enabled, then create a generic service that uses the dialog. Let me know if you have any questions. Thanks, Tina Created attachment 1450185 [details]
Dialog cache screen shot
Created attachment 1450186 [details]
dialog export
Created attachment 1450187 [details]
datastore export
Just kidding!! Ignore the last comment, please. https://github.com/ManageIQ/manageiq/pull/17642 https://github.com/ManageIQ/manageiq-api/pull/407 ... let's try this again. Dialog methods are run only once when the dilaog is loaded. Verified in 5.10.0.3.20180705151325_ce4d2b5 |