-
Notifications
You must be signed in to change notification settings - Fork 152
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
Move kopia storage consts to a secrets package #2059
Conversation
Thanks for submitting this pull request 🎉. The team will review it soon and get back to you. If you haven't already, please take a moment to review our project contributing guideline and Code of Conduct document. |
d2a3d67
to
63e856b
Compare
63e856b
to
4a1fa5a
Compare
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, but this PR does not seem to handle refactor for other constants. Would there be a separate PR for these? Respective comment: #1940 (comment)
f3157b1
to
a8ee0de
Compare
4e3bc64
to
5b63e96
Compare
Change Overview
Move the kopia location secret consts to
secrets
package so that they can be reused. We cannot use the consts from storage package in secrets since it creates cyclic dependency. Hence moving it to secrets package. This PR would be the base PR for validation of kopia repository server secrets PR - #1940Pull request type
Please check the type of change your PR introduces:
Issues
Test Plan