Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1539211

Summary: oc adm diagnostic does not report error after removing endpoint for logging-kibana-ops
Product: OpenShift Container Platform Reporter: Peter Ruan <pruan>
Component: LoggingAssignee: Jan Wozniak <jwozniak>
Status: CLOSED DUPLICATE QA Contact: Junqi Zhao <juzhao>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.9.0CC: aos-bugs, jcantril, juzhao, jwozniak, mmccomas, rmeggins
Target Milestone: ---   
Target Release: 3.9.z   
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: 2018-02-05 12:34:57 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Peter Ruan 2018-01-26 23:12:34 UTC
Description of problem:
oc adm diagnostic does not report error after removing endpoint for logging-kibana-ops

Version-Release number of selected component (if applicable):
v3.9.0-0.24.0

How reproducible:
always

Steps to Reproduce:
1.
2.
3.

Actual results:
[root@host-172-16-120-128 ~]# oc delete endpoints logging-kibana-ops
endpoints "logging-kibana-ops" deleted
[root@host-172-16-120-128 ~]# oc adm diagnostics aggregatedlogging
[Note] Determining if client configuration exists for client/cluster diagnostics
Info:  Successfully read a client config file at '/root/.kube/config'
Info:  Using context for cluster-admin access: 'r1-6q/172-16-120-128:8443/system:admin'

[Note] Running diagnostic: AggregatedLogging
       Description: Check aggregated logging integration for proper configuration
       
[Note] Summary of diagnostics execution (version v3.9.0-0.24.0):
[Note] Completed with no errors or warnings seen.


Expected results:
endpoints "logging-kibana-ops" not found

Additional info:

Comment 1 Jan Wozniak 2018-02-02 13:38:52 UTC
An AssetConfig section is being removed from masterConfig and looks like this PR [1] disabled AggregatedLogging diagnostics as a side effect.

The PR creates a placeholder loggingURL [2] which was originally retrieved from masterConfig.AssetConfig.loggingPublicURL. The loggingURL is never populated so it skips in Check() [3] silently without any output.



[1] https://github.com/openshift/origin/pull/18068

[2] https://github.com/deads2k/origin/blob/04b0801be074eb957fd8f3ab09b785fdc877ee31/pkg/oc/admin/diagnostics/diagnostics/cluster/aggregated_logging/diagnostic.go#L75

[3] https://github.com/deads2k/origin/blob/04b0801be074eb957fd8f3ab09b785fdc877ee31/pkg/oc/admin/diagnostics/diagnostics/cluster/aggregated_logging/diagnostic.go#L180

Comment 2 Jan Wozniak 2018-02-05 12:34:57 UTC

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