Skip to content

[release-6.7] LOG-10325: match structured log levels case-insensitively - #3573

Merged
openshift-merge-bot[bot] merged 2 commits into
openshift:release-6.7from
openshift-cherrypick-robot:cherry-pick-3525-to-release-6.7
Nov 9, 2026
Merged

openshift-merge-bot[bot] merged 2 commits into
openshift:release-6.7from
openshift-cherrypick-robot:cherry-pick-3525-to-release-6.7

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown

This is an automated cherry-pick of #3525

/assign jcantrill

Uppercase tokens such as "level":"INFO" missed the structured matcher,
so the keyword scan tagged the line from an earlier severity word in
the message. Keep klog prefixes case-sensitive.

Signed-off-by: kchawlani19 <kchawlan@redhat.com>
Match the existing unit-test style so the case-insensitive level cases run as a DescribeTable.
@openshift-ci-robot

Copy link
Copy Markdown

@openshift-cherrypick-robot: Ignoring requests to cherry-pick non-bug issues: LOG-10325

Details

In response to this:

This is an automated cherry-pick of #3525

/assign jcantrill

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.

@coderabbitai

coderabbitai Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review

@openshift-ci
openshift-ci Bot requested review from alanconway and cahartma November 7, 2026 00:44
@openshift-ci

openshift-ci Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

@openshift-cherrypick-robot: all tests passed!

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

@jcantrill

Copy link
Copy Markdown
Contributor

/label tide/merge-method-squash

@jcantrill

Copy link
Copy Markdown
Contributor

/approve
/lgtm
/label verified

@openshift-ci openshift-ci Bot added tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. verified Signifies that the PR passed pre-merge verification criteria labels Nov 9, 2026
@jcantrill

Copy link
Copy Markdown
Contributor

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Nov 9, 2026
@openshift-ci

openshift-ci Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcantrill, openshift-cherrypick-robot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 9, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit 9770b49 into openshift:release-6.7 Nov 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release/6.7 tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants