Project

General

Profile

Backport #19098

RGW/openssl fix for autoconf logic problem in Ubuntu Xenial

Added by Thomas Serlin about 2 months ago. Updated 28 days ago.

Status:
In Progress
Priority:
Normal
Assignee:
Target version:
-
Release:
jewel

History

#1 Updated by Thomas Serlin about 2 months ago

  • Backport set to Jewel

#2 Updated by Ken Dreyer about 2 months ago

  • Assignee set to Marcus Watts
  • Backport changed from Jewel to jewel

#3 Updated by Ken Dreyer about 2 months ago

  • Release deleted (jewel)

#4 Updated by Nathan Cutler about 2 months ago

  • Tracker changed from Bug to Backport
  • Description updated (diff)
  • Target version deleted (v10.2.7)
  • Release jewel added

Description

Marcus Watts has a fix for this BZ:

RGW service fails to start with SSL configured on Ubuntu
https://bugzilla.redhat.com/show_bug.cgi?id=1425710

The fix is here:

https://github.com/mdw-at-linuxbox/ceph-1/commit/78c8be7a0df3d1c669f8a2a8fd7a5676d0823209

This was cherry-picked downstream for the RHCS 2.2 release. We want to get it in future upstream Jewel releases.

#5 Updated by Nathan Cutler about 2 months ago

Thomas/Marcus: Please open a PR at ceph/ceph.git against the "jewel" branch. The end of the PR description could look like this:

Fixes: http://tracker.ceph.com/issues/19098
Signed-off-by: Marcus Watts . . . 

Note: this autotools-specific fix is not cherry-picked from master because master uses cmake.

Thanks!

#6 Updated by Nathan Cutler about 2 months ago

  • Project changed from Ceph to rgw

#7 Updated by Marcus Watts about 2 months ago

I'll make a PR for master that fixes cmake. Is it worth trying to fix the cmake for this in jewel also or do we just ignore it?

#8 Updated by Nathan Cutler about 2 months ago

Marcus Watts wrote:

Is it worth trying to fix the cmake for this in jewel also or do we just ignore it?

cmake in jewel is not production-ready so we are not backporting cmake fixes to jewel.

#9 Updated by Nathan Cutler about 2 months ago

@Marcus: This issue (#19096) is for your autotools fix - are you going to open a PR against jewel for that, at or around the same time that you open the master PR with the cmake version?

#10 Updated by Marcus Watts about 2 months ago

Well, I think I have good news. Turns out the cmake logic already gets the right soname in xenial. So, nothing to fix there.

So this is only a problem if using autoconf/configure to do builds. I believe we no longer support that in master. Do we in kraken?

#11 Updated by Nathan Cutler about 2 months ago

I believe we no longer support that in master. Do we in kraken?

Nope. Jewel is the last release that uses autotools, and Kraken is the first that uses cmake. (There is some cmake stuff in Jewel, but it is not supported.)

#12 Updated by Nathan Cutler about 2 months ago

  • Status changed from New to Need More Info

#13 Updated by Marcus Watts 28 days ago

Since Nathan suggested making another PR to address this I've created https://github.com/ceph/ceph/pull/14215 with the required change. Looks like that's going to incorporate and supersede PR # 12917.

#14 Updated by Nathan Cutler 28 days ago

  • Description updated (diff)
  • Status changed from Need More Info to In Progress

Also available in: Atom PDF