-
Notifications
You must be signed in to change notification settings - Fork 56
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
Update to latest version of the recipe API #1573
Update to latest version of the recipe API #1573
Conversation
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.
Other than go.mod and go.sum conflicts, rest of the changes LGTM
Command run: `go get -u github.com/ramendr/recipe` Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com> Co-Authored-by: Annaraya Narasagond <annarayanarasagond@gmail.com>
e9f096d
to
ad7c3ab
Compare
Changes are: 1. Find the backup and restore workflow by name as CaptureWorkflow and RecoverWorkflow don't exist anymore. 2. Change timeout to int 3. Change command to string type Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com> Co-Authored-by: Annaraya Narasagond <annarayanarasagond@gmail.com>
4a8e36d
to
54c208a
Compare
Command run: `go get -u github.com/ramendr/ramen/api/v1alpha1` Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
54c208a
to
3b85faa
Compare
LGTM. |
This is to support packaging and providing recipe CRDs to any cluster, as the application would be created with a recipe. So the thought process is that and app bundle can be delivered to a cluster that includes the Recipe, when that app/cluster is DR protected Ramen would use the recipe. To aid app devs to craft and bundle the recipe it is provided as a separate repo. |
Bug 2280927: Backport of upstream PR RamenDR#1573
go.mod
file to use the newer version of the Recipe APICaptureWorkflow
andRecoverWorkflow
) with the appropriate backup and restore workflow names.Fixed tests.
Co-Authored-by: @asn1809