-
Notifications
You must be signed in to change notification settings - Fork 6.4k
feat: add Actions for PipelineRollout to allow/disallow data loss #20298
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: add Actions for PipelineRollout to allow/disallow data loss #20298
Conversation
Signed-off-by: Julie Vogelman <julievogelman0@gmail.com>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
Signed-off-by: Julie Vogelman <julievogelman0@gmail.com>
Signed-off-by: Julie Vogelman <julievogelman0@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #20298 +/- ##
==========================================
+ Coverage 55.92% 55.96% +0.04%
==========================================
Files 322 322
Lines 44673 44701 +28
==========================================
+ Hits 24985 25019 +34
+ Misses 17094 17082 -12
- Partials 2594 2600 +6 ☔ View full report in Codecov by Sentry. |
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.
lgtm, thanks!
…goproj#20298) * add Actions for PipelineRollout to allow/disallow data loss Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: test, add codegen generated docs Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix name Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: make actions more robust to handle empty metadata Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * spacing Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> --------- Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> Signed-off-by: alnoor <alihamzanoor99@gmail.com>
…goproj#20298) * add Actions for PipelineRollout to allow/disallow data loss Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: test, add codegen generated docs Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix name Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: make actions more robust to handle empty metadata Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * spacing Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> --------- Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> Signed-off-by: austin5219 <3936059+austin5219@users.noreply.github.com>
…goproj#20298) * add Actions for PipelineRollout to allow/disallow data loss Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: test, add codegen generated docs Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix name Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * fix: make actions more robust to handle empty metadata Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> * spacing Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> --------- Signed-off-by: Julie Vogelman <julievogelman0@gmail.com> Signed-off-by: Adrian Aneci <aneci@adobe.com>
The Open Source Project github.com/numaproj/numaplane is a Control Plane for Numaflow Pipelines.
PipelineRollout
is a CR which wraps around NumaflowPipeline
to manage its state.Numaplane has a strategy for safely updating a Numaflow Pipeline to eliminate data loss by initially pausing it.
The "allow-data-loss" Action is a solution to the problem that a Pipeline may get stuck in "pausing" for whatever reason, so the user needs a way to stop the pausing and complete the update. This is done through the use of an annotation on the object.
Checklist: