You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently the Workflow State resource document is only additive, using a Painless script during provisioning to append to a list. This prevented iteratively updating it during deprovisioning, leading to open questions like #691.
Following #778 we will have the ability to update any individual element of the Workflow State which will bring these benefits:
Allow for incremental addition to the resources created without using Painless, improving portability
Allow for the easy removal of the "error" field on subsequent successful provisioning
What solution would you like?
Replace the painless script with appropriate code from #778. Use ifSeqNo() and ifPrimaryTerm() on the update requests to enforce concurrency, and retry if it fails.
The text was updated successfully, but these errors were encountered:
Coming from #763 (comment), there's a lot of repetitive boilerplate used in Workflow Steps for exception handling during resource updates. As part of this refactoring, the "failed" update code should be de-duplicated.
dbwiddis
added
v2.18.0
Issues targeting release v2.18.0
and removed
backlog
Good to have functionality not critical for next release
labels
Sep 18, 2024
Is your feature request related to a problem?
Currently the Workflow State resource document is only additive, using a Painless script during provisioning to append to a list. This prevented iteratively updating it during deprovisioning, leading to open questions like #691.
Following #778 we will have the ability to update any individual element of the Workflow State which will bring these benefits:
What solution would you like?
Replace the painless script with appropriate code from #778. Use
ifSeqNo()
andifPrimaryTerm()
on the update requests to enforce concurrency, and retry if it fails.The text was updated successfully, but these errors were encountered: