Red Hat Bugzilla – Bug 1559690
If import cluster failed, the cluster global details status should be set as unhealthy
Last modified: 2018-09-04 03:04:26 EDT
Description of problem: If there is a failed import cluster earlier for a cluster, the global details of cluster should set cluster status as unhealthy while sync. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
It this downstream item for this upstream issue? https://github.com/Tendrl/ui/issues/827
Also based on Luboš's description, I'm not sure if this behavior is correct: > If there is a failed import cluster earlier for a cluster, > the global details of cluster should set cluster status as unhealthy > while sync. Would it make more sense to have such cluster reported as unmanaged and import-failed?
>> It this downstream item for this upstream issue? >> https://github.com/Tendrl/ui/issues/827 Not exactly, but if import has failed say due to a situation where one nodes was not able to install tendrl-gluster-integration or unable to start tendrl-gluster-integration, just because other nodes are running tendrl-gluster-integration successfully we should not mark the cluster as healthy at least. This BZ for handling that scenario actually. >> Would it make more sense to have such cluster reported as unmanaged and >> import-failed? Yes with latest changes now import cluster job waits for all the nodes to finish their first round of gluster integration sync and if the same does not happen within defined time (each node we wait for around ~6 mins which is quite a huge and reasonable time) the import job is marked as failed. But if somebody looks at REST output and global details of the cluster says healthy, that also is not correct, so we are marking the cluster global details status as unhealthy here. Hope this clarifies.
This bug was discussed during "RHGS WA Team status meeting" on 2018-03-29. While we should provide "unknown" state in scenario described in the BZ instead of "unhealthy", the idea here is to improve the current implementation without introducing another state into code, using "unhealthy" over "healthy" (as reported now) would align better with "import-failed,unmanaged" state in tendrl ui. Dev team also highlighted that when import fails, the state in tendrl ui should be reported as "?", grafana should not be avaialble. QE team agrees that this is not a good final solution, but agrees to verify this improvement under assumptions described above.
`status` in `globaldetails` is marked as `unhealthy` for failed import flows when import times out or installation repositories are unavailable. --> VERIFIED Tested with: tendrl-ansible-1.6.3-5.el7rhgs.noarch tendrl-api-1.6.3-4.el7rhgs.noarch tendrl-api-httpd-1.6.3-4.el7rhgs.noarch tendrl-commons-1.6.3-8.el7rhgs.noarch tendrl-grafana-plugins-1.6.3-6.el7rhgs.noarch tendrl-grafana-selinux-1.5.4-2.el7rhgs.noarch tendrl-monitoring-integration-1.6.3-6.el7rhgs.noarch tendrl-node-agent-1.6.3-8.el7rhgs.noarch tendrl-notifier-1.6.3-4.el7rhgs.noarch tendrl-selinux-1.5.4-2.el7rhgs.noarch tendrl-ui-1.6.3-6.el7rhgs.noarch
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