public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [Patch RFC 14/15] network startup: Start local and uplink network independent
Date: Wed, 24 May 2023 10:00:51 +0100	[thread overview]
Message-ID: <FA45630B-1306-426D-B16F-20673AEE702A@ipfire.org> (raw)
In-Reply-To: <20230523172314.7826-15-jonatan.schlag@ipfire.org>

[-- Attachment #1: Type: text/plain, Size: 9764 bytes --]

I agree with the essence of this change.

But it should not rely on confusing symlinks and instead have a simple script that calls another one.

> On 23 May 2023, at 18:23, Jonatan Schlag <jonatan.schlag(a)ipfire.org> wrote:
> 
> This allows us several things:
> 
> 1. We can start apache and sshd earlier. This makes debugging and
>   troubleshooting easier. When the startup hangs at red an admin can
>   now at least login and search for what is wrong.
> 2. We can change the implementation of local and uplink network easier
>   as both are a little bit more independent as before.
> 
> Fixes: #11502
> 
> Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
> ---
> config/rootfiles/common/aarch64/initscripts | 27 ++++++++++---------
> config/rootfiles/common/riscv64/initscripts | 27 ++++++++++---------
> config/rootfiles/common/x86_64/initscripts  | 27 ++++++++++---------
> lfs/initscripts                             | 29 ++++++++++++---------
> 4 files changed, 61 insertions(+), 49 deletions(-)
> 
> diff --git a/config/rootfiles/common/aarch64/initscripts b/config/rootfiles/common/aarch64/initscripts
> index 739503ae6..b3187fb45 100644
> --- a/config/rootfiles/common/aarch64/initscripts
> +++ b/config/rootfiles/common/aarch64/initscripts
> @@ -110,8 +110,9 @@ etc/rc.d/rc0.d/K77conntrackd
> etc/rc.d/rc0.d/K78suricata
> etc/rc.d/rc0.d/K79depends-on-network
> etc/rc.d/rc0.d/K79leds
> -etc/rc.d/rc0.d/K80network
> -etc/rc.d/rc0.d/K82wlanclient
> +etc/rc.d/rc0.d/K80network-uplink
> +etc/rc.d/rc0.d/K81wlanclient
> +etc/rc.d/rc0.d/K82network-local
> #etc/rc.d/rc0.d/K85messagebus
> etc/rc.d/rc0.d/K86unbound
> etc/rc.d/rc0.d/K87acpid
> @@ -130,14 +131,15 @@ etc/rc.d/rc3.d/S15fireinfo
> #etc/rc.d/rc3.d/S15messagebus
> #etc/rc.d/rc3.d/S18cpufreq
> etc/rc.d/rc3.d/S19smartenabler
> -etc/rc.d/rc3.d/S19wlanclient
> -etc/rc.d/rc3.d/S20network
> -etc/rc.d/rc3.d/S21depends-on-network
> -etc/rc.d/rc3.d/S21leds
> -etc/rc.d/rc3.d/S22conntrackd
> -etc/rc.d/rc3.d/S24cyrus-sasl
> -etc/rc.d/rc3.d/S30sshd
> -etc/rc.d/rc3.d/S32apache
> +etc/rc.d/rc3.d/S20network-local
> +etc/rc.d/rc3.d/S21sshd
> +etc/rc.d/rc3.d/S22apache
> +etc/rc.d/rc3.d/S23wlanclient
> +etc/rc.d/rc3.d/S24network-uplink
> +etc/rc.d/rc3.d/S25depends-on-network
> +etc/rc.d/rc3.d/S25leds
> +etc/rc.d/rc3.d/S30conntrackd
> +etc/rc.d/rc3.d/S35cyrus-sasl
> etc/rc.d/rc3.d/S40fcron
> #etc/rc.d/rc3.d/S66client175
> etc/rc.d/rc3.d/S98rc.local
> @@ -161,8 +163,9 @@ etc/rc.d/rc6.d/K77conntrackd
> etc/rc.d/rc6.d/K78suricata
> etc/rc.d/rc6.d/K79depends-on-network
> etc/rc.d/rc6.d/K79leds
> -etc/rc.d/rc6.d/K80network
> -etc/rc.d/rc6.d/K82wlanclient
> +etc/rc.d/rc6.d/K80network-uplink
> +etc/rc.d/rc6.d/K81wlanclient
> +etc/rc.d/rc6.d/K82network-local
> #etc/rc.d/rc6.d/K85messagebus
> etc/rc.d/rc6.d/K86unbound
> etc/rc.d/rc6.d/K87acpid
> diff --git a/config/rootfiles/common/riscv64/initscripts b/config/rootfiles/common/riscv64/initscripts
> index ed1eae904..cbb3f9118 100644
> --- a/config/rootfiles/common/riscv64/initscripts
> +++ b/config/rootfiles/common/riscv64/initscripts
> @@ -109,8 +109,9 @@ etc/rc.d/rc0.d/K77conntrackd
> etc/rc.d/rc0.d/K78suricata
> etc/rc.d/rc0.d/K79depends-on-network
> etc/rc.d/rc0.d/K79leds
> -etc/rc.d/rc0.d/K80network
> -etc/rc.d/rc0.d/K82wlanclient
> +etc/rc.d/rc0.d/K80network-uplink
> +etc/rc.d/rc0.d/K81wlanclient
> +etc/rc.d/rc0.d/K82network-local
> #etc/rc.d/rc0.d/K85messagebus
> etc/rc.d/rc0.d/K86unbound
> etc/rc.d/rc0.d/K87acpid
> @@ -129,14 +130,15 @@ etc/rc.d/rc3.d/S15fireinfo
> #etc/rc.d/rc3.d/S15messagebus
> #etc/rc.d/rc3.d/S18cpufreq
> etc/rc.d/rc3.d/S19smartenabler
> -etc/rc.d/rc3.d/S19wlanclient
> -etc/rc.d/rc3.d/S20network
> -etc/rc.d/rc3.d/S21depends-on-network
> -etc/rc.d/rc3.d/S21leds
> -etc/rc.d/rc3.d/S22conntrackd
> -etc/rc.d/rc3.d/S24cyrus-sasl
> -etc/rc.d/rc3.d/S30sshd
> -etc/rc.d/rc3.d/S32apache
> +etc/rc.d/rc3.d/S20network-local
> +etc/rc.d/rc3.d/S21sshd
> +etc/rc.d/rc3.d/S22apache
> +etc/rc.d/rc3.d/S23wlanclient
> +etc/rc.d/rc3.d/S24network-uplink
> +etc/rc.d/rc3.d/S25depends-on-network
> +etc/rc.d/rc3.d/S25leds
> +etc/rc.d/rc3.d/S30conntrackd
> +etc/rc.d/rc3.d/S35cyrus-sasl
> etc/rc.d/rc3.d/S40fcron
> #etc/rc.d/rc3.d/S66client175
> etc/rc.d/rc3.d/S98rc.local
> @@ -160,8 +162,9 @@ etc/rc.d/rc6.d/K77conntrackd
> etc/rc.d/rc6.d/K78suricata
> etc/rc.d/rc6.d/K79depends-on-network
> etc/rc.d/rc6.d/K79leds
> -etc/rc.d/rc6.d/K80network
> -etc/rc.d/rc6.d/K82wlanclient
> +etc/rc.d/rc6.d/K80network-uplink
> +etc/rc.d/rc6.d/K81wlanclient
> +etc/rc.d/rc6.d/K82network-local
> #etc/rc.d/rc6.d/K85messagebus
> etc/rc.d/rc6.d/K86unbound
> etc/rc.d/rc6.d/K87acpid
> diff --git a/config/rootfiles/common/x86_64/initscripts b/config/rootfiles/common/x86_64/initscripts
> index ed1eae904..cbb3f9118 100644
> --- a/config/rootfiles/common/x86_64/initscripts
> +++ b/config/rootfiles/common/x86_64/initscripts
> @@ -109,8 +109,9 @@ etc/rc.d/rc0.d/K77conntrackd
> etc/rc.d/rc0.d/K78suricata
> etc/rc.d/rc0.d/K79depends-on-network
> etc/rc.d/rc0.d/K79leds
> -etc/rc.d/rc0.d/K80network
> -etc/rc.d/rc0.d/K82wlanclient
> +etc/rc.d/rc0.d/K80network-uplink
> +etc/rc.d/rc0.d/K81wlanclient
> +etc/rc.d/rc0.d/K82network-local
> #etc/rc.d/rc0.d/K85messagebus
> etc/rc.d/rc0.d/K86unbound
> etc/rc.d/rc0.d/K87acpid
> @@ -129,14 +130,15 @@ etc/rc.d/rc3.d/S15fireinfo
> #etc/rc.d/rc3.d/S15messagebus
> #etc/rc.d/rc3.d/S18cpufreq
> etc/rc.d/rc3.d/S19smartenabler
> -etc/rc.d/rc3.d/S19wlanclient
> -etc/rc.d/rc3.d/S20network
> -etc/rc.d/rc3.d/S21depends-on-network
> -etc/rc.d/rc3.d/S21leds
> -etc/rc.d/rc3.d/S22conntrackd
> -etc/rc.d/rc3.d/S24cyrus-sasl
> -etc/rc.d/rc3.d/S30sshd
> -etc/rc.d/rc3.d/S32apache
> +etc/rc.d/rc3.d/S20network-local
> +etc/rc.d/rc3.d/S21sshd
> +etc/rc.d/rc3.d/S22apache
> +etc/rc.d/rc3.d/S23wlanclient
> +etc/rc.d/rc3.d/S24network-uplink
> +etc/rc.d/rc3.d/S25depends-on-network
> +etc/rc.d/rc3.d/S25leds
> +etc/rc.d/rc3.d/S30conntrackd
> +etc/rc.d/rc3.d/S35cyrus-sasl
> etc/rc.d/rc3.d/S40fcron
> #etc/rc.d/rc3.d/S66client175
> etc/rc.d/rc3.d/S98rc.local
> @@ -160,8 +162,9 @@ etc/rc.d/rc6.d/K77conntrackd
> etc/rc.d/rc6.d/K78suricata
> etc/rc.d/rc6.d/K79depends-on-network
> etc/rc.d/rc6.d/K79leds
> -etc/rc.d/rc6.d/K80network
> -etc/rc.d/rc6.d/K82wlanclient
> +etc/rc.d/rc6.d/K80network-uplink
> +etc/rc.d/rc6.d/K81wlanclient
> +etc/rc.d/rc6.d/K82network-local
> #etc/rc.d/rc6.d/K85messagebus
> etc/rc.d/rc6.d/K86unbound
> etc/rc.d/rc6.d/K87acpid
> diff --git a/lfs/initscripts b/lfs/initscripts
> index e8a2e4c6e..711f29632 100644
> --- a/lfs/initscripts
> +++ b/lfs/initscripts
> @@ -104,9 +104,10 @@ $(TARGET) :
> ln -sf ../init.d/conntrackd /etc/rc.d/rc0.d/K77conntrackd
> ln -sf ../init.d/suricata /etc/rc.d/rc0.d/K78suricata
> ln -sf ../init.d/depends-on-network /etc/rc.d/rc0.d/K79depends-on-network
> - ln -sf ../init.d/leds /etc/rc.d/rc0.d/K79leds
> - ln -sf ../init.d/network /etc/rc.d/rc0.d/K80network
> - ln -sf ../init.d/wlanclient /etc/rc.d/rc0.d/K82wlanclient
> + ln -sf ../init.d/leds         /etc/rc.d/rc0.d/K79leds
> + ln -sf ../init.d/network-uplink /etc/rc.d/rc0.d/K80network-uplink
> + ln -sf ../init.d/wlanclient   /etc/rc.d/rc0.d/K81wlanclient
> + ln -sf ../init.d/network-local /etc/rc.d/rc0.d/K82network-local
> ln -sf ../init.d/messagebus /etc/rc.d/rc0.d/K85messagebus
> ln -sf ../init.d/unbound /etc/rc.d/rc0.d/K86unbound
> ln -sf ../init.d/sysklogd /etc/rc.d/rc0.d/K90sysklogd
> @@ -123,14 +124,15 @@ $(TARGET) :
> ln -sf ../init.d/messagebus /etc/rc.d/rc3.d/S15messagebus
> ln -sf ../init.d/cpufreq /etc/rc.d/rc3.d/S18cpufreq
> ln -sf ../init.d/smartenabler /etc/rc.d/rc3.d/S19smartenabler
> - ln -sf ../init.d/wlanclient /etc/rc.d/rc3.d/S19wlanclient
> - ln -sf ../init.d/network /etc/rc.d/rc3.d/S20network
> - ln -sf ../init.d/depends-on-network     /etc/rc.d/rc3.d/S21depends-on-network
> - ln -sf ../init.d/leds /etc/rc.d/rc3.d/S21leds
> - ln -sf ../init.d/conntrackd /etc/rc.d/rc3.d/S22conntrackd
> - ln -sf  ../init.d/cyrus-sasl /etc/rc.d/rc3.d/S24cyrus-sasl
> - ln -sf ../init.d/sshd /etc/rc.d/rc3.d/S30sshd
> - ln -sf ../init.d/apache /etc/rc.d/rc3.d/S32apache
> + ln -sf ../init.d/network-local /etc/rc.d/rc3.d/S20network-local
> + ln -sf ../init.d/sshd         /etc/rc.d/rc3.d/S21sshd
> + ln -sf ../init.d/apache       /etc/rc.d/rc3.d/S22apache
> + ln -sf ../init.d/wlanclient   /etc/rc.d/rc3.d/S23wlanclient
> + ln -sf ../init.d/network-uplink /etc/rc.d/rc3.d/S24network-uplink
> + ln -sf ../init.d/depends-on-network     /etc/rc.d/rc3.d/S25depends-on-network
> + ln -sf ../init.d/leds         /etc/rc.d/rc3.d/S25leds
> + ln -sf ../init.d/conntrackd   /etc/rc.d/rc3.d/S30conntrackd
> + ln -sf ../init.d/cyrus-sasl /etc/rc.d/rc3.d/S35cyrus-sasl
> ln -sf ../init.d/fcron /etc/rc.d/rc3.d/S40fcron
> ln -sf ../../sysconfig/rc.local /etc/rc.d/rc3.d/S98rc.local
> ln -sf ../init.d/sslh /etc/rc.d/rc3.d/S98sslh
> @@ -152,8 +154,9 @@ $(TARGET) :
> ln -sf ../init.d/suricata /etc/rc.d/rc6.d/K78suricata
> ln -sf ../init.d/depends-on-network     /etc/rc.d/rc6.d/K79depends-on-network
> ln -sf ../init.d/leds /etc/rc.d/rc6.d/K79leds
> - ln -sf ../init.d/network /etc/rc.d/rc6.d/K80network
> - ln -sf ../init.d/wlanclient /etc/rc.d/rc6.d/K82wlanclient
> + ln -sf ../init.d/network-uplink /etc/rc.d/rc6.d/K80network-uplink
> + ln -sf ../init.d/wlanclient   /etc/rc.d/rc6.d/K81wlanclient
> + ln -sf ../init.d/network-local /etc/rc.d/rc6.d/K82network-local
> ln -sf ../init.d/messagebus /etc/rc.d/rc6.d/K85messagebus
> ln -sf ../init.d/unbound /etc/rc.d/rc6.d/K86unbound
> ln -sf ../init.d/sysklogd /etc/rc.d/rc6.d/K90sysklogd
> -- 
> 2.30.2
> 


  reply	other threads:[~2023-05-24  9:00 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-23 17:23 Jonatan Schlag
2023-05-23 17:23 ` [Patch RFC 01/15] Remove ipsec interface creation from network startup Jonatan Schlag
2023-05-24  8:59   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 02/15] Remove Start/Stop links for client175 Jonatan Schlag
2023-05-23 17:23 ` [Patch RFC 03/15] Use bash as shebang in network initscripts Jonatan Schlag
2023-05-24  8:59   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 04/15] network initscripts: check if the zone in the current config exists Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-08-18 12:55     ` Jonatan Schlag
2023-08-21  9:40       ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 05/15] network initscripts: Remove code for old zone scheme Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 06/15] network scripts: remove check for AUTOCONNECT Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 07/15] network startup: Reload routing informations for every interface Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 08/15] network startup: Always cleanup before red gets started Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 09/15] network startup: check for correct action at start Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-08-18 10:23     ` Jonatan Schlag
2023-05-23 17:23 ` [Patch RFC 10/15] network startup: Refactor how cmd args are processed Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 11/15] network startup: Clean up duplicated Code Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 12/15] network script: add extra scripts for action that depend on a network Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 13/15] network startup: Add scripts for local and uplink Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-23 17:23 ` [Patch RFC 14/15] network startup: Start local and uplink network independent Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer [this message]
2023-05-23 17:23 ` [Patch RFC 15/15] network startup: Only work with configured zones Jonatan Schlag
2023-05-24  9:00   ` Michael Tremer
2023-05-24  8:59 ` Start local and uplink network independent Michael Tremer
2023-08-18 10:30   ` Jonatan Schlag

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=FA45630B-1306-426D-B16F-20673AEE702A@ipfire.org \
    --to=michael.tremer@ipfire.org \
    --cc=development@lists.ipfire.org \
    /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