-
Notifications
You must be signed in to change notification settings - Fork 145
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(backupbackingimage): add parameters to backup backing image create proto #2974
feat(backupbackingimage): add parameters to backup backing image create proto #2974
Conversation
975985a
to
4330231
Compare
This pull request is now in conflict. Could you fix it @ChanYiLin? 🙏 |
3cae5d3
to
cc57b6d
Compare
Hi @derekbit |
This pull request is now in conflict. Could you fix it @ChanYiLin? 🙏 |
a404dab
to
7576ddc
Compare
Fixed. |
Hi @c3y1huang |
This pull request is now in conflict. Could you fix it @ChanYiLin? 🙏 |
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
…te proto ref: longhorn/longhorn 8884 Signed-off-by: Jack Lin <jack.lin@suse.com>
7576ddc
to
d2967c6
Compare
Fixed the conflicts |
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
ref: longhorn/longhorn#8884
update
backupBackingImage
status CRD to addsecret
andsecretNamespace
secret
andsecretNamespace
tobacking-image-manager
through grpc usingparameters
backupBackingImage
api model to pass thesecret
andsecretNamespace
to UI