Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs
@ 2020-11-08 23:00 Romain Naour
  2020-11-09 20:11 ` Peter Korsgaard
  2020-11-13 22:36 ` Peter Korsgaard
  0 siblings, 2 replies; 4+ messages in thread
From: Romain Naour @ 2020-11-08 23:00 UTC (permalink / raw)
  To: buildroot

When running the TestInitSystemSystemdRwIfupdown test, the rootfs must
be in read-write mode. The commit log [1] introducing systemd tests say
so:

"basic systemd, read-write, network w/ ifupdown"

With systemd 246.5, the service systemd-update-done return an error code
when it can't write on the filesystem (/etc)

[1] 117835d5fcd508f301d62dd08ee658c1982c7fa7
[2] https://github.com/systemd/systemd/commit/8019995e9af9c6d7b5985198cedccd24eda3e26e

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/830981813

Signed-off-by: Romain Naour <romain.naour@gmail.com>
Cc: Yann E. MORIN <yann.morin.1998@free.fr>
---
Note: systemd-update-done will continue to fail for other systemd tests
when the rootfs is in read-only mode.
---
 support/testing/tests/init/test_systemd.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/support/testing/tests/init/test_systemd.py b/support/testing/tests/init/test_systemd.py
index 65c06d21b7..c0ef5e1202 100644
--- a/support/testing/tests/init/test_systemd.py
+++ b/support/testing/tests/init/test_systemd.py
@@ -91,7 +91,6 @@ class TestInitSystemSystemdRwIfupdown(InitSystemSystemdBase):
         """
         BR2_SYSTEM_DHCP="eth0"
         # BR2_PACKAGE_SYSTEMD_NETWORKD is not set
-        # BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW is not set
         BR2_TARGET_ROOTFS_EXT2=y
         """
 
-- 
2.25.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs
  2020-11-08 23:00 [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs Romain Naour
@ 2020-11-09 20:11 ` Peter Korsgaard
  2020-11-09 20:25   ` Romain Naour
  2020-11-13 22:36 ` Peter Korsgaard
  1 sibling, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2020-11-09 20:11 UTC (permalink / raw)
  To: buildroot

>>>>> "Romain" == Romain Naour <romain.naour@gmail.com> writes:

 > When running the TestInitSystemSystemdRwIfupdown test, the rootfs must
 > be in read-write mode. The commit log [1] introducing systemd tests say
 > so:

 > "basic systemd, read-write, network w/ ifupdown"

 > With systemd 246.5, the service systemd-update-done return an error code
 > when it can't write on the filesystem (/etc)

 > [1] 117835d5fcd508f301d62dd08ee658c1982c7fa7
 > [2] https://github.com/systemd/systemd/commit/8019995e9af9c6d7b5985198cedccd24eda3e26e

 > Fixes:
 > https://gitlab.com/buildroot.org/buildroot/-/jobs/830981813

 > Signed-off-by: Romain Naour <romain.naour@gmail.com>
 > Cc: Yann E. MORIN <yann.morin.1998@free.fr>
 > ---

Committed, thanks.

 > Note: systemd-update-done will continue to fail for other systemd tests
 > when the rootfs is in read-only mode.

What is the impact of that? Anything we can do about it?

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs
  2020-11-09 20:11 ` Peter Korsgaard
@ 2020-11-09 20:25   ` Romain Naour
  0 siblings, 0 replies; 4+ messages in thread
From: Romain Naour @ 2020-11-09 20:25 UTC (permalink / raw)
  To: buildroot

Hello Peter,

Le 09/11/2020 ? 21:11, Peter Korsgaard a ?crit?:
>>>>>> "Romain" == Romain Naour <romain.naour@gmail.com> writes:
> 
>  > When running the TestInitSystemSystemdRwIfupdown test, the rootfs must
>  > be in read-write mode. The commit log [1] introducing systemd tests say
>  > so:
> 
>  > "basic systemd, read-write, network w/ ifupdown"
> 
>  > With systemd 246.5, the service systemd-update-done return an error code
>  > when it can't write on the filesystem (/etc)
> 
>  > [1] 117835d5fcd508f301d62dd08ee658c1982c7fa7
>  > [2] https://github.com/systemd/systemd/commit/8019995e9af9c6d7b5985198cedccd24eda3e26e
> 
>  > Fixes:
>  > https://gitlab.com/buildroot.org/buildroot/-/jobs/830981813
> 
>  > Signed-off-by: Romain Naour <romain.naour@gmail.com>
>  > Cc: Yann E. MORIN <yann.morin.1998@free.fr>
>  > ---
> 
> Committed, thanks.
> 
>  > Note: systemd-update-done will continue to fail for other systemd tests
>  > when the rootfs is in read-only mode.
> 
> What is the impact of that? Anything we can do about it?
> 

The systemd-update-done service can't work for read-only filesystem even if
previously the service didn't failed in such case.

I'll take a look if we can disable it at build time.

Best regards,
Romain

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs
  2020-11-08 23:00 [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs Romain Naour
  2020-11-09 20:11 ` Peter Korsgaard
@ 2020-11-13 22:36 ` Peter Korsgaard
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2020-11-13 22:36 UTC (permalink / raw)
  To: buildroot

>>>>> "Romain" == Romain Naour <romain.naour@gmail.com> writes:

 > When running the TestInitSystemSystemdRwIfupdown test, the rootfs must
 > be in read-write mode. The commit log [1] introducing systemd tests say
 > so:

 > "basic systemd, read-write, network w/ ifupdown"

 > With systemd 246.5, the service systemd-update-done return an error code
 > when it can't write on the filesystem (/etc)

 > [1] 117835d5fcd508f301d62dd08ee658c1982c7fa7
 > [2] https://github.com/systemd/systemd/commit/8019995e9af9c6d7b5985198cedccd24eda3e26e

 > Fixes:
 > https://gitlab.com/buildroot.org/buildroot/-/jobs/830981813

 > Signed-off-by: Romain Naour <romain.naour@gmail.com>
 > Cc: Yann E. MORIN <yann.morin.1998@free.fr>
 > ---
 > Note: systemd-update-done will continue to fail for other systemd tests
 > when the rootfs is in read-only mode.

Committed to 2020.02.x and 2020.08.x, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-11-13 22:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-08 23:00 [Buildroot] [PATCH] support/testing: TestInitSystemSystemdRwIfupdown test expect a RW rootfs Romain Naour
2020-11-09 20:11 ` Peter Korsgaard
2020-11-09 20:25   ` Romain Naour
2020-11-13 22:36 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox