Skip to content
This repository was archived by the owner on Mar 20, 2026. It is now read-only.

Use k8s recorder for events#1242

Merged
dvaldivia merged 1 commit intominio:masterfrom
reivaj05:use-recorder-for-events
Aug 17, 2022
Merged

Use k8s recorder for events#1242
dvaldivia merged 1 commit intominio:masterfrom
reivaj05:use-recorder-for-events

Conversation

@reivaj05
Copy link
Copy Markdown
Contributor

@reivaj05 reivaj05 commented Aug 16, 2022

Use k8s recorder implementation to create and update events.

Closes: https://github.com/miniohq/engineering/issues/705

It will better handle the creation of events.

  • It will reduce the number of API calls to k8s
  • It will increment the count for duplicated messages

First events were created with previous implementation and last line with the change
Screen Shot 2022-08-16 at 15 54 50

Copy link
Copy Markdown
Contributor

@cniackz cniackz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@pjuarezd pjuarezd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
Screen Shot 2022-08-16 at 4 40 30 PM

Copy link
Copy Markdown
Contributor

@Alevsk Alevsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳 🥳 🥳

@dvaldivia dvaldivia merged commit c94dfd9 into minio:master Aug 17, 2022
@dvaldivia
Copy link
Copy Markdown
Collaborator

It was that easy?!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants