Bug 1454942 – rgw_file: remove unlinked assertions

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1454942 - rgw_file: remove unlinked assertions

Summary:	rgw_file: remove unlinked assertions

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat Ceph Storage
Classification:	Red Hat
Component:	RGW (Show other bugs)
Sub Component:
Version:	2.2
Hardware:	All All

Priority	unspecified Severity low
Target Milestone:	rc
Target Release:	2.3
Assigned To:	Matt Benjamin (redhat)
QA Contact:	ceph-qe-bugs
Docs Contact:

URL:
Whiteboard:
Keywords:	CodeChange

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2017-05-23 16:19 EDT by Matt Benjamin (redhat)
Modified:	2017-07-30 12:00 EDT (History)
CC List:	9 users (show)

See Also:
Fixed In Version:	RHEL: ceph-10.2.7-20.el7cp Ubuntu: ceph_10.2.7-22redhat1
Doc Type:	If docs needed, set a value
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2017-06-19 09:33:35 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Ceph Project Bug Tracker	20047	None	None	None	2017-05-23 16:19 EDT
Red Hat Product Errata	RHBA-2017:1497	normal	SHIPPED_LIVE	Red Hat Ceph Storage 2.3 bug fix and enhancement update	2017-06-19 13:24:11 EDT

Groups: None (edit)

Description Matt Benjamin (redhat) 2017-05-23 16:19:05 EDT

Description of problem:
The rgw_file/RGW nfs unlink() call path contains a live assert() that successfully unlinked objects cannot be found in a subsequent lookup.  This assert could trigger in a false positive in a multi-gateway environment racing to create/unlink objects.

Comment 2 Matt Benjamin (redhat) 2017-05-23 19:06:19 EDT

Steps to Reproduce
No requirement to reproduce.  The change removes logic to assert() in a condition that could be, but never has in development, been reproduced in a multi-gateway deployment scenario where overlapping ranges of files and directories were being simultaneously created and deleted without coordination.

Actual Results
As above.

Expected Results
As above.

Comment 8 Matt Benjamin (redhat) 2017-06-05 07:36:07 EDT

(the issue was resolved, NEEDINFO should have been cleared)

Comment 10 errata-xmlrpc 2017-06-19 09:33:35 EDT

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:1497

Note You need to log in before you can comment on or make changes to this bug.