From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sage Weil Subject: Re: [PATCH] Statically binding ports for ceph-osd Date: Tue, 10 Apr 2012 16:51:41 -0700 (PDT) Message-ID: References: Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: Received: from cobra.newdream.net ([66.33.216.30]:50739 "EHLO cobra.newdream.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759726Ab2DJXvr (ORCPT ); Tue, 10 Apr 2012 19:51:47 -0400 In-Reply-To: Sender: ceph-devel-owner@vger.kernel.org List-ID: To: Nick Bartos Cc: Greg Farnum , ceph-devel@vger.kernel.org On Tue, 10 Apr 2012, Nick Bartos wrote: > Awesome, thanks so much! Can I assume this will make it into the next > ceph stable release? I'll probably just backport it now before we > actually start using it, so I don't have to change the config later. v0.45 is out today/tomorrow, but it'll be in v0.46. sage > > On Tue, Apr 10, 2012 at 4:16 PM, Greg Farnum > wrote: > > Yep, you're absolutely correct. Might as well let users specify the whole address rather than just the port, though ? since your patch won't apply to current upstream due to some heartbeating changes I whipped up another one which adds the "osd heartbeat addr" option. It's pushed it to master in commit 6fbac10dc68e67d1c700421f311cf5e26991d39c, but you'll want to backport (easy) or carry your change until you upgrade (and remember to change the config!). :) > > Thanks for the report! > > -Greg > > > > > > On Tuesday, April 10, 2012 at 12:56 PM, Nick Bartos wrote: > > > >> After doing some more looking at the code, it appears that this option > >> is not supported. I created a small patch (attached) which adds the > >> functionality. Is there any way we could get this, or something like > >> this, applied upstream? I think this is important functionality for > >> firewalled environments, and seems like a simple fix since all the > >> other services (including ones for ceph-mon and ceph-mds) already > >> allow you to specify a static port. > >> > >> > >> On Mon, Apr 9, 2012 at 5:27 PM, Nick Bartos wrote: > >> > I'm trying to get ceph-osd's listening ports to be set statically for > >> > firewall reasons. I am able to get 2 of the 3 ports set statically, > >> > however the 3rd one is still getting set dynamically. > >> > > >> > I am using: > >> > > >> > [osd.48] > >> > host = 172.16.0.13 > >> > cluster addr = 172.16.0.13:6944 > >> > public addr = 172.16.0.13:6945 > >> > > >> > The daemon will successfully bind to 6944 and 6945, but also binds to > >> > 6800. What additional option do I need? I started looking at the > >> > code and thought "hb addr = 172.16.0.13:6946" would do it, but > >> > specifying that option seems to have no effect (or at least does not > >> > achieve the desired result). > >> > >> > >> > >> Attachments: > >> - ceph-0.41-osd_hb_port.patch > >> > > > > > > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > >