-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
stripe cleanup: Remove the option from create and add-brick cmds #1812
Conversation
/run regression |
@@ -5841,8 +5833,6 @@ glusterd_get_state(rpcsvc_request_t *req, dict_t *dict) | |||
|
|||
fprintf(fp, "Volume%d.snap_count: %" PRIu64 "\n", count, | |||
volinfo->snap_count); | |||
fprintf(fp, "Volume%d.stripe_count: %d\n", count, | |||
volinfo->stripe_count); |
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.
this removal may result in checksum mismatch during update... can you check if we need to write it regardless.
@mohit84 please confirm.
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.
4a7e9db
to
709b130
Compare
/run regression |
1 similar comment
/run regression |
Hi @mohit84, Review approval please if you don't see any other issue with this patch. |
Yes sheetal please do it |
/run regression |
/recheck smoke |
This patch aims to remove the code for stripe option instead of keeping a default values of stripe/stripe-count variables and setting and getting dict options and similar redundant operations. Also removing tests for stripe volumes that have been already marked bad. Updates: gluster#1000 Change-Id: Ic2b3cabd671f0c8dc0521384b164c3078f7ca7c6 Signed-off-by: Sheetal Pamecha <spamecha@redhat.com>
tests/000-flaky/basic_changelog_changelog-snapshot.t was failing due to 0 return value Change-Id: I8ea0443669c63768760526db5aa1f205978e1dbb Signed-off-by: Sheetal Pamecha <spamecha@redhat.com>
Change-Id: I49f3da4f106c55f9da20d0b0a299275a19daf4ba
9ef5745
to
947926e
Compare
Change-Id: I83bae85d10c8c5b3c66f56c9f8de1ec81d0bbc95
/run regression |
…mds (gluster#1812)" This reverts commit 1d74669. Signed-off-by: Shwetha K Acharya <sacharya@redhat.com>
This patch aims to remove the code for stripe option instead
of keeping a default values of stripe/stripe-count variables and
setting and getting dict options and similar redundant operations.
Also removing tests for stripe volumes that have been already
marked bad.
Updates: #1000
Change-Id: Ic2b3cabd671f0c8dc0521384b164c3078f7ca7c6
Signed-off-by: Sheetal Pamecha spamecha@redhat.com