Bug 1489902

Summary:	Request host reinstall if kdump fencing integration is enabled after registration
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Marina Kalinin <mkalinin>
Component:	ovirt-engine	Assignee:	Martin Perina <mperina>
Status:	CLOSED DUPLICATE	QA Contact:	meital avital <mavital>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	4.1.4	CC:	gveitmic, lsurette, mkalinin, mperina, rbalakri, Rhev-m-bugs, srevivo, ykaul
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-09-11 07:42:05 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Marina Kalinin 2017-09-08 15:43:39 UTC

The documentation [1] states, that if fencing and kdump integration are enabled on an existing host, a host reinstall is required.
However, there is no indication for this in UI.

Assuming documentation is correct, we should let the user know that host reinstall is required to enjoy the kdump intergration functionality. It can be a warning sign and a tool tip with explanation. Current behavior confuses the users and prevents us from getting the required information from the first attempt.

[1] 
https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.1/html/administration_guide/sect-host_tasks#Configuring_Host_Power_Management_Settings

Comment 1 Martin Perina 2017-09-08 17:04:08 UTC

(In reply to Marina from comment #0)
> The documentation [1] states, that if fencing and kdump integration are
> enabled on an existing host, a host reinstall is required.
> However, there is no indication for this in UI.
> 
> Assuming documentation is correct, we should let the user know that host
> reinstall is required to enjoy the kdump intergration functionality. It can
> be a warning sign and a tool tip with explanation. Current behavior confuses
> the users and prevents us from getting the required information from the
> first attempt.
> 
> [1] 
> https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.1/
> html/administration_guide/sect-
> host_tasks#Configuring_Host_Power_Management_Settings

This is not true, below warning is save in audit log and showed in Events tab on

1. Each activation of the host
2. If you enable Kdump integration in Power Management tab when you edit a host

Please take a look at customer logs, they are plenty of those warnings:

2017-09-07 07:26:23,745-04 WARN  [org.ovirt.engine.core.dal.dbbroker.auditloghandling.AuditLogDirector] (DefaultQuartzScheduler5) [7834553c] EVENT_ID: KDUMP_DETECTION_NOT_CONFIGURED_ON_VDS(617), Correlation ID: 7834553c, Call Stack: null, Custom Event ID: -1, Message: Kdump integration is enabled for host host1.example.com, but kdump is not configured properly on host.

Anyway it seems that customers completely ignores warnings, so we would need to prioritize BZ1480433.

Comment 2 Marina Kalinin 2017-09-08 20:38:01 UTC

Martin, indeed, I see this warning in the events log:
"Kdump integration is enabled for host rhevh-26.gsslab.rdu2.redhat.com, but kdump is not configured properly on host."

However:
1. The warning does not say that restart is required.
2. No indication on the host itself. 
3. No one except support look into the logs. Read: customers would not go to logs to see if operation performed in UI is successful. It should result in a feedback in the same place it was issued, i.e. the UI.

Maybe we should close this bug as duplicate of the one you pointed to.
Or, we can keep this one to request to adjust the text of the warning in the Events log, to make it more clear that reinstall of the host is needed. (reinstall, really? not restart?)

Comment 3 Marina Kalinin 2017-09-08 20:39:56 UTC

*** Bug 1480433 has been marked as a duplicate of this bug. ***

Comment 4 Germano Veit Michel 2017-09-10 23:16:10 UTC

Marina, kdump is not the only configuration that needs the host to be re-installed, there is also the iommu stuff for example. I think bugzilla #1480433 was a more general approach. This current BZ is just for kdump, so subset of the other BZ. We should close this one as DUP and keep the other, the other way around, to get a more comprehensive solution instead of opening BZs for every single setting that needs a host to be re-installed.... What do you think?

Comment 5 Martin Perina 2017-09-11 07:40:55 UTC

(In reply to Marina from comment #2)
> Martin, indeed, I see this warning in the events log:
> "Kdump integration is enabled for host rhevh-26.gsslab.rdu2.redhat.com, but
> kdump is not configured properly on host."
> 
> However:
> 1. The warning does not say that restart is required.

Not restart, but Reinstall is required, because we need to add parameters to kdump and restart the service.

> 2. No indication on the host itself. 
> 3. No one except support look into the logs. Read: customers would not go to
> logs to see if operation performed in UI is successful. It should result in
> a feedback in the same place it was issued, i.e. the UI.

As I said this is a WARNING in audit log and it is displayed in Events tab in UI. It's not our fault that customers are ignoring warnings

Comment 6 Martin Perina 2017-09-11 07:42:05 UTC

(In reply to Germano Veit Michel from comment #4)
> Marina, kdump is not the only configuration that needs the host to be
> re-installed, there is also the iommu stuff for example. I think bugzilla
> #1480433 was a more general approach. This current BZ is just for kdump, so
> subset of the other BZ. We should close this one as DUP and keep the other,
> the other way around, to get a more comprehensive solution instead of
> opening BZs for every single setting that needs a host to be
> re-installed.... What do you think?

Completely agree with the above, I've just reopened BZ1480433 and closing this one as duplicate of BZ1480433.

*** This bug has been marked as a duplicate of bug 1480433 ***