Skip to content

improve: event recorder correctness - #3665

Draft
csviri wants to merge 1 commit into
nextfrom
fix/event-recorder-correctness
Draft

csviri wants to merge 1 commit into
nextfrom
fix/event-recorder-correctness

Conversation

@csviri

@csviri csviri commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

…namespace

- aggregating onto an event with an explicit name also updates type, reason and
  action, and rejects a change of them when the event carries an eventTime, which
  the API server treats as immutable
- supplied names longer than the Kubernetes limit are shortened with a hash of the
  full name instead of plainly truncated, so distinct names stay distinct
- events about cluster scoped resources in an overridden namespace carry an
  eventTime (and an action), as the API server refuses them otherwise
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 27, 2026
@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@csviri
csviri changed the base branch from main to next October 27, 2026 17:07

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant