trt-1117: Test for update lease errors #28020
Conversation
openshift-ci-robot
added
the
jira/valid-reference
|
@neisw: This pull request references trt-1117 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade 10 |
|
@neisw: trigger 1 job(s) for the /payload-(job|aggregate) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/8d54eb10-191e-11ee-80f0-5ac2cac2cc78-0 |
|
/lgtm |
|
/retest-required |
|
@neisw: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
| var failedLeaseUpdateErrorRegex = regexp.MustCompile(`failed to update lease, error: Put \"(?P<URL>[a-z0-9.-:\/\-\?\=]+)\": (?P<MSG>[^\"]+)`) | ||
|
|
||
| // upper 'F'ailed and 'err' | ||
| var failedLeaseUpdateErrRegex = regexp.MustCompile(`Failed to update lease\" err\=\"Put \\\"(?P<URL>[a-z0-9.-:\/\-\?\=]+)\\\": (?P<MSG>[^\"]+)`) |
There was a problem hiding this comment.
Ah I see they're different in other places
There was a problem hiding this comment.
The err vs. error string is a bit different as well but if you see a common regex for them both that would work for me as well.
There was a problem hiding this comment.
No it's fine, 2 simpler regex is better than 1 unreadable one.
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: neisw, stbenjam, xueqzhan 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 |
openshift-ci
bot
added
the
approved
|
Job Failure Risk Analysis for sha: b820bee
|
openshift-merge-robot
merged commit acd302f
into
openshift:master
Looks for matches indicating 'Failed to update lease' in journal logs.
Creates a new flake test case. We see update lease failures early on during the install so a grace period of 30 minutes is added from the first event time.