Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

This project is now read‑only. Starting Monday, February 2, please use https://ibm-ceph.atlassian.net/ for all bug tracking management.

Bug 1190870

Summary:	ceph is not built with yasm
Product:	[Red Hat Storage] Red Hat Ceph Storage	Reporter:	Ken Dreyer (Red Hat) <kdreyer>
Component:	Build	Assignee:	Ken Dreyer (Red Hat) <kdreyer>
Status:	CLOSED CURRENTRELEASE	QA Contact:	ceph-qa-bugs <ceph-qa-bugs>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.2.2	CC:	branto, dmick
Target Milestone:	pre-dev-freeze
Target Release:	1.2.3
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	ceph-0.80.8-4.el7cp	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-10 01:31:51 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 Ken Dreyer (Red Hat) 2015-02-09 20:03:24 UTC

Description of problem:
ceph-0.80.8-3 is not built with yasm

Version-Release number of selected component (if applicable):
ceph-0.80.8-3.el6cp
ceph-0.80.8-3.el7cp

Additional info:
yasm was initially disabled in the downstream packaging in order to solve an SELinux issue (bug 1118504). Ceph v0.80.8 contains an upstream fix that should make a yasm build operate properly with SELinux.

Comment 1 Ken Dreyer (Red Hat) 2015-02-09 20:04:12 UTC

Dan mentioned to me a while back that disabling yasm impedes Ceph's performance, so it would be ideal if we re-enabled yasm. It's not clear to me how bad the performance impact would be if we continued to disable yasm - Dan knows more about this than me.

Since ICE 1.2.2 shipped with yasm enabled in the build, and product management has clarified that SELinux support is not a blocker for Red Hat Ceph 1.2.3, I think we should re-enable yasm. Boris, do you agree ?

Comment 2 Dan Mick 2015-02-09 20:39:16 UTC

Argh, yes, we definitely want it on.  CRC and erasure coding benefit greatly from assembly.

Comment 3 Boris Ranto 2015-02-09 20:45:29 UTC

Hmm, I thought this already was enabled in RHC builds. I will have to enable it in fedora as well once I will have some time to rebase the ceph packages there.

AFAIK, it is not that important for client stuff and that is why I just disabled it in base RHEL.

Comment 4 Ken Dreyer (Red Hat) 2015-02-09 21:09:17 UTC

Thanks Boris and Dan for helping me understand this issue.

I'll re-enable the BuildRequires: yasm in the 0.80.8 packages and get a new RHC build attached to the Errata today.

So that this doesn't get lost for Fedora, I've filed https://bugzilla.redhat.com/show_bug.cgi?id=1190882 in order to track that separately.