Bug 1878040

Summary:	No validation if set invalid value to spec.logLevel and spec.operatorLogLevel in configs.imageregistry.operator.openshift.io
Product:	OpenShift Container Platform	Reporter:	Wenjing Zheng <wzheng>
Component:	Image Registry	Assignee:	Ricardo Maraschini <rmarasch>
Status:	CLOSED ERRATA	QA Contact:	Wenjing Zheng <wzheng>
Severity:	high	Docs Contact:
Priority:	high
Version:	4.6	CC:	aos-bugs, obulatov, sttts
Target Milestone:	---
Target Release:	4.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1883689 1883691 1883692 1883698 1883769 (view as bug list)		Environment:
Last Closed:	2020-10-27 16:40:10 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 Wenjing Zheng 2020-09-11 07:14:52 UTC

Description of problem:
When set invalid value to spec.logLevel  and spec.operatorLogLevel in configs.imageregistry.operator.openshift.io, no warning appears.

Version-Release number of selected component (if applicable):
4.6.0-0.nightly-2020-09-10-195619

How reproducible:
Always

Steps to Reproduce:
1.Change loglevel and operatorLogLevel with invalid values:
  logLevel: abc
  logging: 0
  managementState: Managed
  observedConfig: null
  operatorLogLevel: abc
2.
3.

Actual results:
No warning appears.

Expected results:
There should be warning appears.

Additional info:

Comment 2 Stefan Schimanski 2020-09-29 08:07:38 UTC

Increasing priority. We should have landed the API change 17 days ago when the controller merged. We have to vendor it into all the operators with logLevels.

Comment 3 Stefan Schimanski 2020-09-29 08:08:01 UTC

Increasing priority. We should have landed the API change 17 days ago when the controller merged. We have to vendor it into all the operators with logLevels.

Comment 6 Wenjing Zheng 2020-10-09 02:32:07 UTC

Verified on 4.6.0-0.nightly-2020-10-08-182439.
configs.imageregistry.operator.openshift.io "cluster" was not valid:
# * spec.operatorLogLevel: Unsupported value: "abc": supported values: "", "Normal", "Debug", "Trace", "TraceAll"

Comment 8 errata-xmlrpc 2020-10-27 16:40:10 UTC

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 (OpenShift Container Platform 4.6 GA Images), 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/RHBA-2020:4196