-
Notifications
You must be signed in to change notification settings - Fork 61
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
chore(backendPvcTimeout): option to configure BackendPvcTimeout using env Var #84
chore(backendPvcTimeout): option to configure BackendPvcTimeout using env Var #84
Conversation
… OPENEBS_IO_NFS_SERVER_BACKEND_PVC_TIMEOUT Signed-off-by: mayank <[email protected]>
Signed-off-by: mayank <[email protected]>
Signed-off-by: mayank <[email protected]>
@@ -143,6 +143,9 @@ spec: | |||
# leader election is enabled. | |||
#- name: LEADER_ELECTION_ENABLED | |||
# value: "true" | |||
# Set Timeout for backend PVC to bound, Default value is 60 seconds | |||
#- name: OPENEBS_IO_NFS_SERVER_BACKEND_PVC_TIMEOUT | |||
# value: "120" |
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.
n_: set this as 60 to match the default value.
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.
comment on a log... rest of the changes looks good to me
Signed-off-by: mayank <[email protected]>
54bb979
to
bf5642f
Compare
Codecov Report
@@ Coverage Diff @@
## develop #84 +/- ##
===========================================
- Coverage 46.99% 46.85% -0.14%
===========================================
Files 29 29
Lines 2349 2356 +7
===========================================
Hits 1104 1104
- Misses 1167 1174 +7
Partials 78 78
Continue to review full report at Codecov.
|
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.
changes are good
What this PR does?:
This PR adds an option to configure BackendPvcTimeout.
OPENEBS_IO_NFS_SERVER_BACKEND_PVC_TIMEOUT
can be set to configure BackendPvcTimeout in nfs-provisioner deployment.Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Checklist:
<type>(<scope>): <subject>