@ju We can show this "If un-manage fails, resolve the issue and then try un-manage cluster again" tooltip text for only "Unmanage Failed" description and for "Ready to Use", can show "The cluster is successfully imported for viewing monitoring data and metrics". Also, do we need to show tooltip for all other description like Import failed, Expansion failed etc?
+1 on showing tooltips with more details in context to what the message shows. When unmanage fails, infotip to show "If unmanage fails, resolve the issue and then try unmanage cluster again." For Ready to Use, infotip to show "The cluster is successfully imported for viewing monitoring data and metrics." When Import fails, infotip to show "If import cluster fails, resolve the issue before performing an unmanage cluster and then import cluster." If Expand cluster fails, infotip to show "If cluster expansion fails, check that tendrl-ansible ran without issues and that agents are properly configured." @rghatvis Please feel free to comment / modify the infotips.
Ju, nice suggestions. I just have some minor tweaks. (In reply to Ju Lim from comment #5) > +1 on showing tooltips with more details in context to what the message > shows. > > > > When unmanage fails, infotip to show "If unmanage fails, resolve the issue > and then try unmanage cluster again." Remove the adverb "then" and modify to: "If unmanage fails, resolve the issue and reinitiate unmanage cluster." > > > > For Ready to Use, infotip to show "The cluster is successfully imported for > viewing monitoring data and metrics." LGTM > > > > When Import fails, infotip to show "If import cluster fails, resolve the > issue before performing an unmanage cluster and then import cluster." Modifty to: "If import cluster fails, resolve the issue before performing an unmanage cluster and reinitiate import." > > > > If Expand cluster fails, infotip to show "If cluster expansion fails, check > that tendrl-ansible ran without issues and that agents are properly > configured." Modify to: "If cluster expansion fails, check if tendrl-ansible was executed successfully and ensure the node agents are correctly configured." > > > > > > @rghatvis Please feel free to comment / modify the infotips.
Created attachment 1458736 [details] infotip: Import Failed
Created attachment 1458737 [details] infotip: Import Passed
Created attachment 1458750 [details] infotip: Unmanage Failed
Created attachment 1459058 [details] infotip: Expand Failed
All four infotips: * Import Passed - attachment 1458737 [details] * Import Failed - attachment 1458736 [details] * Unmanage Failed - attachment 1458750 [details] and * Expand Failed - attachment 1459058 [details] are correct from the point of view, that they correspond to the suggestions from Comment 6. But the last one for "Failed Expand" is not correct from the content point of view: > If cluster expansion fails, check if tendrl-ansible was executed > successfully and ensure the node agents are correctly configured. There is no information what to do next, to fix the state (once you resolve the root cause of the failure). This issue was described with more details in Bug 1596655 and corresponding Doc bug 1596660. Once the correct steps will be at least documented (Bug 1596660), we should review the infotip covered by this bug and accordingly update it. >> ASSIGNED
What is your expectation here?
It depends on the resolution of Doc bug 1596660 (or Bug 1596655, if it will be part of the upcoming release). Until that, it's not possible to decide what should be written in the infotip.
@dahorak Bug bz#1596655 is resolved and UI do allows to trigger expand flow, if expansion fails with the infotip as "If cluster expansion fails, check if tendrl-ansible was executed successfully and ensure the node agents are correctly configured. If cluster expansion failed due to errors, resolve the errors on affected nodes and re-initiate the Expand Cluster action." for "Expand Failed".
Created attachment 1476364 [details] infotip: Expand Failed - updated content
The infotip for Failed Expand was updated accordingly to bug 1596655 comment 7. See attachment 1476364 [details]. Version: tendrl-ui-1.6.3-10.el7rhgs.noarch
Verifying based on comment 12 and comment 17. >> VERIFIED
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/RHSA-2018:2616