Bug 1878007 - Clean up invalid operator/v1 (Operator)LogLevels for all operators and tighten validation
Summary: Clean up invalid operator/v1 (Operator)LogLevels for all operators and tighte...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: config-operator
Version: 4.5
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 4.5.z
Assignee: Venkata Siva Teja Areti
QA Contact: Xingxing Xia
URL:
Whiteboard:
Depends On: 1877408
Blocks: 1866791
TreeView+ depends on / blocked
 
Reported: 2020-09-11 04:33 UTC by Venkata Siva Teja Areti
Modified: 2020-11-10 14:54 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1877408
Environment:
Last Closed: 2020-11-10 14:53:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift cluster-config-operator pull 150 0 None closed Bug 1878007: Add Controller to Normalize the Operator Log Level Values 2021-02-02 06:44:05 UTC
Github openshift library-go pull 890 0 None closed Bug 1878007: Validate operatorSpec.logLevel 2021-02-02 06:44:06 UTC
Red Hat Product Errata RHBA-2020:4425 0 None None None 2020-11-10 14:54:10 UTC

Comment 10 Xingxing Xia 2020-10-26 07:30:03 UTC
I'm following the issue DPTP-660 to do the pre-merge verifying for QE pre-merge verification goal of issue OCPQE-815. Below are steps:
Use the bot to launch cluster including the still open but approved PR(s).
Then run:
$ oc patch openshiftapiserver cluster -p '{"spec": {"operatorLogLevel": "oastest111"}}' --type merge
$ oc patch openshiftapiserver cluster -p '{"spec": {"logLevel": "oastest222"}}' --type merge

$ oc get openshiftapiserver cluster -o=jsonpath='{.spec.operatorLogLevel}{"; "}{.spec.logLevel}{"\n"}'
oastest111; oastest222

$ while true; do date; oc get openshiftapiserver cluster -o=jsonpath='{.spec.operatorLogLevel}{"; "}{.spec.logLevel}{"\n"}'; sleep 10; done
Mon Oct 26 14:45:00 CST 2020
...
Mon Oct 26 14:48:40 CST 2020
Normal; Normal
^C
$ oc get event -n openshift-config-operator
...
4m39s       Normal   OperatorLogLevelChange        deployment/openshift-config-operator              "spec.operatorLogLevel" changed from "oastest111" to "Normal"
4m39s       Normal   OperatorLogLevelChange        deployment/openshift-config-operator              "spec.logLevel" changed from "oastest222" to "Normal"

The invalid log levels are normalized, expected by the PR. Notes, per comment 0, 4.5 fix only normalizes, not validates.
So the bug is pre-merge verified. After the PR gets merged, the bug will be moved to VERIFIED by the bot automatically or, if not working, by me manually.

Comment 12 Xingxing Xia 2020-10-29 10:38:34 UTC
Latest payload 4.5.0-0.nightly-2020-10-29-070117 still does not yet include the PR. Thus, per DPTP-660's requirement "a nightly build containing the fix is marked as accepted" for "bugs will be moved to verified", will wait more time to until such a build is there.

Comment 15 errata-xmlrpc 2020-11-10 14:53:52 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.5.18 bug fix update), 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:4425


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