Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added containerd based event services #63

Merged
merged 1 commit into from
Aug 20, 2021
Merged

Conversation

richardcase
Copy link
Member

@richardcase richardcase commented Aug 20, 2021

What this PR does / why we need it:

Implemented an event service using the containerd event service. Also
moved the event definitions out of the core folder and added to the
api. Removed the old transport based event service.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #61

Special notes for your reviewer:

Checklist:

  • squashed commits
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

Release note:

Implemented a containerd based event service and removed the transport implementation.

Implemented an event service using the containerd event service. Also
moved the event definitions out of the core folder and added to the
api. Removed the old transport based event service.

Signed-off-by: Richard Case <[email protected]>
@richardcase richardcase added the kind/feature New feature or request label Aug 20, 2021
@richardcase richardcase merged commit b7fe083 into main Aug 20, 2021
@richardcase richardcase deleted the ctr_event_service branch August 20, 2021 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change event service implementation to use containerd event service
1 participant