Hi Sero,
> openSUSE and SLE don't support username begin with digit, so it will
> skip test generic/597 and generic/598 by lack of 123456-fsgqa user.
> generic/597 and 598 are not test username begin with digit on purpose
> (different with generic/381). It's will be helpful to use an username
> begin with non-digit in this case.
> Signed-off-by: Sun Yong <yosun@xxxxxxxx>
> ---
> README | 1 +
> tests/generic/597 | 4 ++--
> tests/generic/598 | 4 ++--
> 3 files changed, 5 insertions(+), 4 deletions(-)
> diff --git a/README b/README
> index 094a7742..cffa0bc6 100644
> --- a/README
> +++ b/README
> @@ -22,6 +22,7 @@ _______________________
> - create fsgqa test user ("sudo useradd -m fsgqa")
> - create fsgqa group ("sudo groupadd fsgqa")
> - create 123456-fsgqa test user ("sudo useradd 123456-fsgqa")
I thought fsgqa-123456 is replacement for 123456-fsgqa. Thus I'd delete this
line.
> +- create fsgqa-123456 test user ("sudo useradd fsgqa-123456")
Also test tests/generic/381 still uses 123456-fsgqa. Shouldn't be replaced as
well?
> ______________________
> USING THE FSQA SUITE
> diff --git a/tests/generic/597 b/tests/generic/597
> index 1d87a23a..c4d524b7 100755
> --- a/tests/generic/597
> +++ b/tests/generic/597
> @@ -43,11 +43,11 @@ _require_sysctl_variable fs.protected_symlinks
> _require_sysctl_variable fs.protected_hardlinks
> # su in _require_user prints warnings about user name starts with a digit,
> # discard the warning
> -_require_user 123456-fsgqa >/dev/null 2>&1
> +_require_user fsgqa-123456 >/dev/null 2>&1
I that redirection to /dev/null was due errors caused by starting with
digits. As other _require_user calls don't have it, I'd remove the
redirection. + remove that comment "su in _require_user prints warnings about
user name starts with a digit," ... which is now useless/invalid.
Eric, was there a reson to use user name which starts with digit?
> # Do this SECOND so that qa_user is fsgqa, and _user_do uses that account
> _require_user fsgqa
> -OWNER=123456-fsgqa
> +OWNER=fsgqa-123456
> OTHER=fsgqa
> # Save current system state to reset when done
> diff --git a/tests/generic/598 b/tests/generic/598
> index 998b62cf..1cf9a3d1 100755
> --- a/tests/generic/598
> +++ b/tests/generic/598
> @@ -43,11 +43,11 @@ _require_sysctl_variable fs.protected_regular
> _require_sysctl_variable fs.protected_fifos
> # su in _require_user prints warnings about user name starts with a digit,
> # discard the warning
> -_require_user 123456-fsgqa >/dev/null 2>&1
> +_require_user fsgqa-123456 >/dev/null 2>&1
> # Do this SECOND so that qa_user is fsgqa, and _user_do uses that account
> _require_user fsgqa
> -USER1=123456-fsgqa
> +USER1=fsgqa-123456
> USER2=fsgqa
> # Save current system state to reset when done
Kind regards,
Petr
