Bug 1424817
Summary: | Fix wrong operators, found by coverty | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Michael S. <misc> |
Component: | core | Assignee: | M. Scherer <mscherer> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | mainline | CC: | bugs |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.12.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-09-05 17:25:47 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Michael S.
2017-02-19 16:22:56 UTC
REVIEW: https://review.gluster.org/16676 (Fix wrong operators) posted (#1) for review on master by Michael Scherer (misc) REVIEW: https://review.gluster.org/16676 (Fix wrong operators) posted (#2) for review on master by Michael Scherer (misc) COMMIT: https://review.gluster.org/16676 committed in master by Jeff Darcy (jeff.us) ------ commit 5c68937ef943a5ed8179cb19fef833f2be790588 Author: Michael Scherer <misc> Date: Sun Feb 19 17:19:33 2017 +0100 Fix wrong operators Coverty rightfully note that if we verify that A =! C or A != B, it will always be true. In one case, that prevent healing from continuing. In the other, that trigger useless logs. Fixing this bug also show that ENOSPC shouldn't abort the rebalance operation, as seen during the review of the first patch on https://review.gluster.org/#/c/16676/1/xlators/cluster/dht/src/dht-rebalance.c Change-Id: I93c4df43b880b211da202a7e49cef6b1ce7ab68f BUG: 1424817 Signed-off-by: Michael Scherer <misc> Reviewed-on: https://review.gluster.org/16676 Smoke: Gluster Build System <jenkins.org> Reviewed-by: Jeff Darcy <jeff.us> Tested-by: Jeff Darcy <jeff.us> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Amar Tumballi <amarts> This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.12.0, please open a new bug report. glusterfs-3.12.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://lists.gluster.org/pipermail/announce/2017-September/000082.html [2] https://www.gluster.org/pipermail/gluster-users/ |