Skip to content

Conversation

@marquiz
Copy link
Contributor

@marquiz marquiz commented Aug 7, 2025

Also add minimal readme for the template and wasm plugins.

The default-validator plugin is left out as it cannot be built as an external plugin (without changes).

@marquiz
Copy link
Contributor Author

marquiz commented Aug 7, 2025

@klihub klihub requested review from chrishenzie and mikebrow August 7, 2025 08:11
@klihub klihub changed the title readme: list all reference plugins readme: list all sample plugins Aug 7, 2025
@klihub
Copy link
Member

klihub commented Aug 7, 2025

@marquiz I replaced 'reference plugin' with 'sample plugin' in the PR title, which is the term we usually use in this repo to refer to the plugins here (trying to avoid confusion wrt. to the more complex ones in the NRI Ref. Plugins repo).

Also add minimal readme for the template and wasm plugins.

The default-validator plugin is left out as it cannot be built as an
external plugin (without changes).

Signed-off-by: Markus Lehtonen <markus.lehtonen@intel.com>
@marquiz
Copy link
Contributor Author

marquiz commented Aug 7, 2025

I replaced 'reference plugin' with 'sample plugin'

K, I adjusted the commit message accordingly

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebrow mikebrow merged commit bfdf5c2 into containerd:main Aug 7, 2025
7 checks passed
@marquiz marquiz deleted the devel/readme branch August 7, 2025 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants