-
Notifications
You must be signed in to change notification settings - Fork 20
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] - add new etcd-disk flavor #258
Conversation
ac904f3
to
2bda4bf
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #258 +/- ##
=======================================
Coverage 53.83% 53.83%
=======================================
Files 27 27
Lines 1566 1566
=======================================
Hits 843 843
Misses 673 673
Partials 50 50 ☔ View full report in Codecov by Sentry. |
local: | ||
dataDir: /var/lib/etcd_data/etcd | ||
extraArgs: | ||
quota-backend-bytes: "8589934592" |
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.
do we still need this section if it is in the default flavor?
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.
We technically don't but I wanted to leave it in for clarity. I'm fine with removing it
3375f17
to
b54ac50
Compare
b54ac50
to
a8cd5bc
Compare
What type of PR is this?
/kind feature
What this PR does / why we need it: This creates a new flavor to configure etcd to be on a separate disk
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
TODOs: