Make event types lowercase

[rh-amarin]

Make CloudEvent type property lowercase

Summary by CodeRabbit

• Bug Fixes

  • CloudEvent type values now consistently use lowercase resource kinds for reliable event formatting.

• Tests

  • Updated test expectations to reflect the normalized CloudEvent type format.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign xueli181114 for approval. For more information see the Code Review Process.

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

Details

Needs approval from an approver in each of these files:

• OWNERS

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e0b49e1e-ea47-4a3a-b020-8bb62a48e9ad

📥 Commits

Reviewing files that changed from the base of the PR and between 02e3467 and 3035603.

📒 Files selected for processing (2)

• internal/sentinel/sentinel.go
• internal/sentinel/sentinel_test.go

✅ Files skipped from review due to trivial changes (1)

• internal/sentinel/sentinel.go

🚧 Files skipped from review as they are similar to previous changes (1)

• internal/sentinel/sentinel_test.go

---

Walkthrough

This change normalizes CloudEvent type string construction by ensuring the resource kind is lowercased before inclusion. The modification adds a strings.ToLower() call to the event type building logic in sentinel.go, and the corresponding test expectation is updated to reflect the lowercase format in the emitted CloudEvent type.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Make event types lowercase' directly summarizes the main change: normalizing CloudEvent type values to lowercase.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}