Skip to content
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

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

raghavendra-talur
Copy link
Member

  • Updated the go.mod file to use the newer version of the Recipe API
  • Added the updated Recipe CRD to the test directory
  • Replaced references to non-existent workflows (CaptureWorkflow and RecoverWorkflow) with the appropriate backup and restore workflow names.
  • Updated the timeout attribute type from a string to an integer
  • Changed the command attribute type from []string to string

Fixed tests.

Co-Authored-by: @asn1809

Copy link
Contributor

@asn1809 asn1809 left a 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>
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>
Command run: `go get -u github.com/ramendr/ramen/api/v1alpha1`

Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
@BenamarMk
Copy link
Member

LGTM.
The only question is why we have a new Recipe repo outside of Ramen repo?

@ShyamsundarR
Copy link
Member

LGTM. The only question is why we have a new Recipe repo outside of Ramen repo?

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.

@ShyamsundarR ShyamsundarR merged commit ba56704 into RamenDR:main Sep 26, 2024
19 of 20 checks passed
BenamarMk added a commit to red-hat-storage/ramen that referenced this pull request Oct 10, 2024
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