Development discussion of WireGuard
 help / color / mirror / Atom feed
From: "Jason A. Donenfeld" <Jason@zx2c4.com>
To: Martin Hauke <mardnh@gmx.de>
Cc: WireGuard mailing list <wireguard@lists.zx2c4.com>
Subject: Re: [PATCH] systemd: add file wireguard.target
Date: Fri, 1 May 2020 15:35:38 -0600	[thread overview]
Message-ID: <CAHmME9p--JcBcVdFKVnaexYy-NWobGudQyA7z9KX5w1KBW4cbw@mail.gmail.com> (raw)
In-Reply-To: <9d644025-9b85-51a1-58f8-07e445c61492@gmx.de>

On Fri, May 1, 2020 at 3:29 PM Martin Hauke <mardnh@gmx.de> wrote:
>
> Am 01.05.20 um 01:21 schrieb Jason A. Donenfeld:
> > On Thu, Apr 30, 2020 at 7:56 AM Martin Hauke <mardnh@gmx.de> wrote:
> >> +PartOf=wireguard.target
> >
> > This won't reset wireguard interfaces configured by systemd-networkd,
> > nor interfaces configured manually, right? So maybe it's better to
> > call this wg-quick.target?
>
> Correct, then let's call this wg-quick.target .
>
> >> +Description=Target to restart all parts of WireGuard
> >
> > Is this verbage consistent with other systemd targets? Specifically,
> > is the "to restart" part apt? Or should this say something more
> > general?
>
> How about "Description=WireGuard target allowing to start/stop all
> wg-quick@.service instances at once" ?

Based on the language pattern established by below,
"Description=WireGuard Tunnels via wg-quick(8)" seems more fitting.

$ rg -g '*.target' ^Description= /lib/systemd
/lib/systemd/system/initrd-fs.target:Description=Initrd File Systems
/lib/systemd/system/printer.target:Description=Printer
/lib/systemd/system/kexec.target:Description=Reboot via kexec
/lib/systemd/system/emergency.target:Description=Emergency Mode
/lib/systemd/system/getty-pre.target:Description=Login Prompts (Pre)
/lib/systemd/system/local-fs.target:Description=Local File Systems
/lib/systemd/system/initrd-root-device.target:Description=Initrd Root Device
/lib/systemd/system/initrd-switch-root.target:Description=Switch Root
/lib/systemd/system/basic.target:Description=Basic System
/lib/systemd/system/umount.target:Description=Unmount All Filesystems
/lib/systemd/system/rescue.target:Description=Rescue Mode
/lib/systemd/system/reboot.target:Description=Reboot
/lib/systemd/system/poweroff.target:Description=Power-Off
/lib/systemd/system/remote-fs-pre.target:Description=Remote File Systems (Pre)
/lib/systemd/system/timers.target:Description=Timers
/lib/systemd/system/network-pre.target:Description=Network (Pre)
/lib/systemd/system/final.target:Description=Final Step
/lib/systemd/system/remote-cryptsetup.target:Description=Remote
Encrypted Volumes
/lib/systemd/system/hibernate.target:Description=Hibernate
/lib/systemd/system/getty.target:Description=Login Prompts
/lib/systemd/system/slices.target:Description=Slices
/lib/systemd/system/boot-complete.target:Description=Boot Completion Check
/lib/systemd/system/nss-user-lookup.target:Description=User and Group
Name Lookups
/lib/systemd/system/nss-lookup.target:Description=Host and Network Name Lookups
/lib/systemd/system/hybrid-sleep.target:Description=Hybrid Suspend+Hibernate
/lib/systemd/system/suspend-then-hibernate.target:Description=Suspend;
Hibernate if not used for a period of time
/lib/systemd/system/sockets.target:Description=Sockets
/lib/systemd/system/paths.target:Description=Paths
/lib/systemd/system/graphical.target:Description=Graphical Interface
/lib/systemd/system/cryptsetup.target:Description=Local Encrypted Volumes
/lib/systemd/system/local-fs-pre.target:Description=Local File Systems (Pre)
/lib/systemd/system/halt.target:Description=Halt
/lib/systemd/system/nfs-client.target:Description=NFS client services
/lib/systemd/system/rpcbind.target:Description=RPC Port Mapper
/lib/systemd/system/machines.target:Description=Containers
/lib/systemd/system/multi-user.target:Description=Multi-User System
/lib/systemd/system/network.target:Description=Network
/lib/systemd/system/swap.target:Description=Swap
/lib/systemd/system/network-online.target:Description=Network is Online
/lib/systemd/system/initrd.target:Description=Initrd Default Target
/lib/systemd/system/initrd-root-fs.target:Description=Initrd Root File System
/lib/systemd/system/smartcard.target:Description=Smart Card
/lib/systemd/system/shutdown.target:Description=Shutdown
/lib/systemd/system/bluetooth.target:Description=Bluetooth
/lib/systemd/system/system-update-pre.target:Description=Offline
System Update (Pre)
/lib/systemd/system/remote-fs.target:Description=Remote File Systems
/lib/systemd/system/time-sync.target:Description=System Time Synchronized
/lib/systemd/system/suspend.target:Description=Suspend
/lib/systemd/system/blockdev@.target:Description=Block Device Preparation for %f
/lib/systemd/system/sleep.target:Description=Sleep
/lib/systemd/system/system-update.target:Description=Offline System Update
/lib/systemd/system/nvmf-connect.target:Description=All instances of
nvmf-autoconnect daemon
/lib/systemd/system/sound.target:Description=Sound Card
/lib/systemd/system/exit.target:Description=Exit the container
/lib/systemd/system/time-set.target:Description=System Time Set
/lib/systemd/system/sysinit.target:Description=System Initialization
/lib/systemd/system/sigpwr.target:Description=Power Failure
/lib/systemd/system/cryptsetup-pre.target:Description=Local Encrypted
Volumes (Pre)

      reply	other threads:[~2020-05-01 21:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-30 13:55 Martin Hauke
2020-04-30 14:01 ` Riccardo Berto
2020-04-30 23:21 ` Jason A. Donenfeld
2020-05-01 21:29   ` Martin Hauke
2020-05-01 21:35     ` Jason A. Donenfeld [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

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

  git send-email \
    --in-reply-to=CAHmME9p--JcBcVdFKVnaexYy-NWobGudQyA7z9KX5w1KBW4cbw@mail.gmail.com \
    --to=jason@zx2c4.com \
    --cc=mardnh@gmx.de \
    --cc=wireguard@lists.zx2c4.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.
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 NNTP newsgroup(s).