-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add WaitForSettledState func to check Operator Config before progressing #241
Add WaitForSettledState func to check Operator Config before progressing #241
Conversation
cc78a60
to
9a0c2e7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of comments, otherwise looks good 👍
9a0c2e7
to
fe96bc0
Compare
@jhadvig changes made, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @benjaminapetersen 👍
/lgtm
/retest Please review the full test history for this PR and help us cut down flakes. |
17 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@benjaminapetersen some of the e2e tests re failing with
|
/hold |
Specifically this bunch: === RUN TestOperatorConfigBranding
--- FAIL: TestOperatorConfigBranding (28.47s)
branding_test.go:69: update operator with okd
branding_test.go:69: update operator with ocp
branding_test.go:69: update operator with online
branding_test.go:69: update operator with dedicated
branding_test.go:69: update operator with azure
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
=== RUN TestBrandingFromManagedConfigMap
--- FAIL: TestBrandingFromManagedConfigMap (22.26s)
branding_test.go:91: update data for the config map in openshift-config-managed namespace with okd
branding_test.go:91: update data for the config map in openshift-config-managed namespace with ocp
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
=== RUN TestDebugLogLevel
--- FAIL: TestDebugLogLevel (24.45s)
console-operator.go:66: console operator already in 'Managed' state
console-operator.go:161: checking if console operator LogLevel is set to 'Normal'...
console-operator.go:184: setting console operator to 'Normal' LogLevel ...
console-operator.go:184: setting console operator to 'Debug' LogLevel ...
logging_test.go:89: checking if '--log-level=*=DEBUG' flag is set on the console deployment container command...
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
console-operator.go:184: setting console operator to 'Normal' LogLevel ...
=== RUN TestTraceLogLevel
--- FAIL: TestTraceLogLevel (23.33s)
console-operator.go:66: console operator already in 'Managed' state
console-operator.go:161: checking if console operator LogLevel is set to 'Normal'...
console-operator.go:184: setting console operator to 'Trace' LogLevel ...
logging_test.go:89: checking if '--log-level=*=TRACE' flag is set on the console deployment container command...
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
console-operator.go:184: setting console operator to 'Normal' LogLevel ...
=== RUN TestTraceAllLogLevel
--- FAIL: TestTraceAllLogLevel (23.32s)
console-operator.go:66: console operator already in 'Managed' state
console-operator.go:161: checking if console operator LogLevel is set to 'Normal'...
console-operator.go:184: setting console operator to 'TraceAll' LogLevel ...
logging_test.go:89: checking if '--log-level=*=TRACE' flag is set on the console deployment container command...
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
console-operator.go:184: setting console operator to 'Normal' LogLevel ...
=== RUN TestManaged
--- FAIL: TestManaged (25.32s)
console-operator.go:66: console operator already in 'Managed' state
framework.go:33: deleting console Deployment...
framework.go:33: deleting console Service...
framework.go:33: deleting console Route...
framework.go:33: deleting console ConfigMap...
managed_test.go:26: validating that the operator recreates resources when ManagementState:Managed...
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
console-operator.go:66: console operator already in 'Managed' state
=== RUN TestEditManagedConfigMap
--- FAIL: TestEditManagedConfigMap (21.10s)
console-operator.go:66: console operator already in 'Managed' state
util.go:26: patching Data on the console ConfigMap
util.go:33: polling for patched Data on the console ConfigMap
console-operator.go:256: operator has not reached settled state: timed out waiting for the condition
console-operator.go:66: console operator already in 'Managed' state I'll increase the max again to see what we get. |
e69d0e2
to
42c29e4
Compare
42c29e4
to
6df6ac3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold cancel
/retest Please review the full test history for this PR and help us cut down flakes. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: benjaminapetersen, jhadvig, zherman0 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Add WaitForSettledState func to check Operator Config before progressing to next test.
/assign @jhadvig @zherman0