cmogstored dev/user discussion/issues/patches/etc
 help / color / mirror / code / Atom feed
From: Eric Wong <e@80x24.org>
To: mogile@googlegroups.com
Cc: cmogstored-public@bogomips.org
Subject: [PATCH (cmogstored)] doc: document CMOGSTORED_FDS in the manpage
Date: Wed, 11 Nov 2015 22:55:35 +0000	[thread overview]
Message-ID: <20151111225535.GA25851@dcvr.yhbt.net> (raw)
In-Reply-To: <CABJfL5iZOM3T_pVsqevFx=Q1cFXgaxJjKQhb2=TfGHbwaekevQ@mail.gmail.com>

David Birdsong <david.birdsong -AT- gmail -DOT- com> wrote:
> On Wed, Nov 11, 2015 at 2:41 PM, Eric Wong <e@80x24.org> wrote:
> > It can't set environment variables?  I suppose you can use env(1) from
> > the command-line to launch cmogstored, as it always supported the
> > CMOGSTORED_FD environment.  Unlike LISTEN_FDS, CMOGSTORED_FD
> > does not validate $$ == LISTEN_PID like LISTEN_FDS does.  It
> > is comma-delimited, so it can use non-contiguous FDs:
> >
> >    env CMOGSTORED_FD=3,4 cmogstored ...
> >
> >
> it can set env, but i was thinking that the templating from the socket
> support doesn't extend into env. either way, a simple shell shm could do
> it, so yeah that's probably fine.

The env(1) command-line would be templated, actually::

	env CMOGSTORED_FD=%d,%d cmogstored ...

Just pushed the following out to cmogstored.git, too.  Thanks.

Subject: [PATCH] doc: document CMOGSTORED_FDS in the manpage

This has always been supported internally, and we can't stop
supporting it since we'll be supporting upgrades from old versions
indefinitely.  So document it, as it has some minor advantages over
the LISTEN_{FDS,PID} environment handling of systemd.
---
 cmogstored.x | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/cmogstored.x b/cmogstored.x
index a6b96e6..20eeb36 100644
--- a/cmogstored.x
+++ b/cmogstored.x
@@ -22,6 +22,12 @@ different sockets, the command line or config file must still specify
 which addresses to listen on.  This feature is available in cmogstored
 1.5.0 and later.
 
+CMOGSTORED_FD - a comma-delimited list of integer file descriptors.
+This is used internally for SIGUSR2 upgrades, but may be specified
+at startup (e.g. via env(1)) as it is not tied to LISTEN_PID
+like LISTEN_FDS is.  Unlike LISTEN_FDS, CMOGSTORED_FD does not rely
+on inherited file descriptors being contiguous.
+
 See MALLOC TUNING for environment variables which may affect memory
 usage.
 
-- 
EW

      parent reply	other threads:[~2015-11-11 22:55 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-11 22:05 [ANN] cmogstored 1.5.0rc1 - a mogstored alternative Eric Wong
     [not found] ` <CABJfL5iEz47zinp0rbOanrSN=1ubRFAa18wHaBnKSiTc+-Zpew@mail.gmail.com>
2015-11-11 22:41   ` Eric Wong
     [not found]     ` <CABJfL5iZOM3T_pVsqevFx=Q1cFXgaxJjKQhb2=TfGHbwaekevQ@mail.gmail.com>
2015-11-11 22:55       ` Eric Wong [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://yhbt.net/cmogstored/README

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151111225535.GA25851@dcvr.yhbt.net \
    --to=e@80x24.org \
    --cc=cmogstored-public@bogomips.org \
    --cc=mogile@googlegroups.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://yhbt.net/cmogstored.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).