WIP: Replace brittle string split with regex for timestamp extraction for OCP-39845#3934
WIP: Replace brittle string split with regex for timestamp extraction for OCP-39845#3934MrSanketkumar wants to merge 2 commits into
Conversation
…ion and add debug logging
|
@MrSanketkumar: No Jira issue with key OCP-39845 exists in the tracker at https://redhat.atlassian.net. DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/hold |
|
/unhold |
…M:SS from machineset-controller logs
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: miyadav The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@MrSanketkumar: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
Problem
OCP-39845 test fails with error:
Root Cause:
The test parses machineset-controller logs using
.split("Watching")which assumes "Watching" keyword exists in logs. In 4.15/ARM64 builds, this keyword is missing or in a different location, causing the split to return 279 characters of log content instead of a timestamp.Solution
.split("Watching")logic with regex that directly extracts ISO8601 timestampsDebug Output
The fix includes debug logging that will appear in Prow job console: