[v2] Core Update 169: Delete unused initrd on 32-bit ARM to save space in /boot

Message ID 20220707135805.1597569-1-peter.mueller@ipfire.org
State Accepted
Commit f26f08b464287f91ff6df849b9e77d354e72d4cb
Headers
Series [v2] Core Update 169: Delete unused initrd on 32-bit ARM to save space in /boot |

Commit Message

Peter Müller July 7, 2022, 1:58 p.m. UTC
  On 64-bit ARM, this is still needed for booting. Since the initrds were
already shipped with the updater, there is no need to regenerate them
locally again.

https://community.ipfire.org/t/again-with-the-file-system-full-core-169/8186

Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
---
 config/rootfiles/core/169/update.sh | 7 +++++++
 1 file changed, 7 insertions(+)
  

Comments

Michael Tremer July 7, 2022, 2:02 p.m. UTC | #1
Why do we ship this in the first place, then?

> On 7 Jul 2022, at 15:58, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> On 64-bit ARM, this is still needed for booting. Since the initrds were
> already shipped with the updater, there is no need to regenerate them
> locally again.
> 
> https://community.ipfire.org/t/again-with-the-file-system-full-core-169/8186
> 
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
> config/rootfiles/core/169/update.sh | 7 +++++++
> 1 file changed, 7 insertions(+)
> 
> diff --git a/config/rootfiles/core/169/update.sh b/config/rootfiles/core/169/update.sh
> index 3902e2d45..dc7a2a18a 100644
> --- a/config/rootfiles/core/169/update.sh
> +++ b/config/rootfiles/core/169/update.sh
> @@ -150,6 +150,13 @@ ldconfig
> # Apply sysctl changes
> /etc/init.d/sysctl start
> 
> +# Delete unused initrd on 32-bit ARM to save space in /boot
> +case "$(uname -m)" in
> +	armv*)
> +		rm /boot/initramfs-${KVER}-ipfire.img
> +		;;
> +esac
> +
> # Start services
> telinit u
> /etc/init.d/firewall restart
> -- 
> 2.30.2
>
  
Peter Müller July 7, 2022, 2:05 p.m. UTC | #2
This is a question for Arne, I'm afraid...

> Why do we ship this in the first place, then?
> 
>> On 7 Jul 2022, at 15:58, Peter Müller <peter.mueller@ipfire.org> wrote:
>>
>> On 64-bit ARM, this is still needed for booting. Since the initrds were
>> already shipped with the updater, there is no need to regenerate them
>> locally again.
>>
>> https://community.ipfire.org/t/again-with-the-file-system-full-core-169/8186
>>
>> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
>> ---
>> config/rootfiles/core/169/update.sh | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/config/rootfiles/core/169/update.sh b/config/rootfiles/core/169/update.sh
>> index 3902e2d45..dc7a2a18a 100644
>> --- a/config/rootfiles/core/169/update.sh
>> +++ b/config/rootfiles/core/169/update.sh
>> @@ -150,6 +150,13 @@ ldconfig
>> # Apply sysctl changes
>> /etc/init.d/sysctl start
>>
>> +# Delete unused initrd on 32-bit ARM to save space in /boot
>> +case "$(uname -m)" in
>> +	armv*)
>> +		rm /boot/initramfs-${KVER}-ipfire.img
>> +		;;
>> +esac
>> +
>> # Start services
>> telinit u
>> /etc/init.d/firewall restart
>> -- 
>> 2.30.2
>>
>
  
Michael Tremer July 7, 2022, 2:11 p.m. UTC | #3
Okay, I merged this into master and core169.

> On 7 Jul 2022, at 16:05, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> This is a question for Arne, I'm afraid...
> 
>> Why do we ship this in the first place, then?
>>> On 7 Jul 2022, at 15:58, Peter Müller <peter.mueller@ipfire.org> wrote:
>>> 
>>> On 64-bit ARM, this is still needed for booting. Since the initrds were
>>> already shipped with the updater, there is no need to regenerate them
>>> locally again.
>>> 
>>> https://community.ipfire.org/t/again-with-the-file-system-full-core-169/8186
>>> 
>>> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
>>> ---
>>> config/rootfiles/core/169/update.sh | 7 +++++++
>>> 1 file changed, 7 insertions(+)
>>> 
>>> diff --git a/config/rootfiles/core/169/update.sh b/config/rootfiles/core/169/update.sh
>>> index 3902e2d45..dc7a2a18a 100644
>>> --- a/config/rootfiles/core/169/update.sh
>>> +++ b/config/rootfiles/core/169/update.sh
>>> @@ -150,6 +150,13 @@ ldconfig
>>> # Apply sysctl changes
>>> /etc/init.d/sysctl start
>>> 
>>> +# Delete unused initrd on 32-bit ARM to save space in /boot
>>> +case "$(uname -m)" in
>>> +	armv*)
>>> +		rm /boot/initramfs-${KVER}-ipfire.img
>>> +		;;
>>> +esac
>>> +
>>> # Start services
>>> telinit u
>>> /etc/init.d/firewall restart
>>> -- 
>>> 2.30.2
>>>
  

Patch

diff --git a/config/rootfiles/core/169/update.sh b/config/rootfiles/core/169/update.sh
index 3902e2d45..dc7a2a18a 100644
--- a/config/rootfiles/core/169/update.sh
+++ b/config/rootfiles/core/169/update.sh
@@ -150,6 +150,13 @@  ldconfig
 # Apply sysctl changes
 /etc/init.d/sysctl start
 
+# Delete unused initrd on 32-bit ARM to save space in /boot
+case "$(uname -m)" in
+	armv*)
+		rm /boot/initramfs-${KVER}-ipfire.img
+		;;
+esac
+
 # Start services
 telinit u
 /etc/init.d/firewall restart