Bug 1539211 - oc adm diagnostic does not report error after removing endpoint for logging-kibana-ops
Summary: oc adm diagnostic does not report error after removing endpoint for logging-k...
Keywords:
Status: CLOSED DUPLICATE of bug 1538922
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Logging
Version: 3.9.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 3.9.z
Assignee: Jan Wozniak
QA Contact: Junqi Zhao
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-01-26 23:12 UTC by Peter Ruan
Modified: 2018-02-05 12:34 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-02-05 12:34:57 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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 ***


Note You need to log in before you can comment on or make changes to this bug.