-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Document about item action plugin ordering. #6719
Conversation
Codecov Report
@@ Coverage Diff @@
## main #6719 +/- ##
=======================================
Coverage 60.67% 60.67%
=======================================
Files 249 249
Lines 26476 26476
=======================================
Hits 16064 16064
Misses 9268 9268
Partials 1144 1144 |
8867fe3
to
13cb39a
Compare
@sseago We discussed this doc change few weeks ago in the call and you suggested a change regarding plugins registered by the same binary. I made the change to include that case. Please take a look. |
@@ -59,6 +59,11 @@ Velero supports the following kinds of plugins: | |||
- **Restore Item Action** - executes arbitrary logic for individual items prior to restoring them into a cluster | |||
- **Delete Item Action** - executes arbitrary logic based on individual items within a backup prior to deleting the backup | |||
|
|||
Plugin binaries are discovered by recursively reading a directory in no particular order. Hence no guarantee is provided for the | |||
order in which item action plugins are invoked. How ever, if a single binary implements multiple item action plugins, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/How ever/However
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Made the change.
@@ -59,6 +59,11 @@ Velero supports the following kinds of plugins: | |||
- **Restore Item Action** - executes arbitrary logic for individual items prior to restoring them into a cluster | |||
- **Delete Item Action** - executes arbitrary logic based on individual items within a backup prior to deleting the backup | |||
|
|||
Plugin binaries are discovered by recursively reading a directory in no particular order. Hence no guarantee is provided for the | |||
order in which item action plugins are invoked. How ever, if a single binary implements multiple item action plugins, | |||
they may be invoked in the order in which they are registered but it is best not depend on this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/best not/best to not
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed it. Thanks.
Signed-off-by: Raghuram Devarakonda <draghuram@gmail.com>
13cb39a
to
59a666c
Compare
Thank you for contributing to Velero!
Please add a summary of your change
Added clarification that item action plugins have no order. Please feel free to suggest changes.
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.