[PATCH] spec: kdump/ppc64: make servicelog_notify silent when there are no errors

[Lichen Liu]

There is confusing message in /var/log/anaconda/packaging.log when installing
kexec-tools during the system installation on ppc64le:

	Event Notification Registration successful (id: 1)

Make servicelog_notify slient when there are no erros.

Signed-off-by: Lichen Liu <lichliu(a)redhat.com&gt;
---
 kexec-tools.spec | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kexec-tools.spec b/kexec-tools.spec
index d7d2947..51a16ba 100644
--- a/kexec-tools.spec
+++ b/kexec-tools.spec
@@ -268,7 +268,7 @@ touch /etc/kdump.conf
 
 %ifarch ppc64 ppc64le
 servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
2>/dev/null
-servicelog_notify --add --command=/usr/lib/kdump/kdump-migrate-action.sh
--match='refcode="#MIGRATE" and serviceable=0' --type=EVENT
--method=pairs_stdin
+servicelog_notify --add --command=/usr/lib/kdump/kdump-migrate-action.sh
--match='refcode="#MIGRATE" and serviceable=0' --type=EVENT
--method=pairs_stdin >/dev/null
 %endif
 
 # This portion of the script is temporary.  Its only here
@@ -299,7 +299,7 @@ fi
 
 %preun
 %ifarch ppc64 ppc64le
-servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
+servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
>/dev/null
 %endif
 %systemd_preun kdump.service
 
-- 
2.40.1

[Coiby Xu]

On Mon, Jun 12, 2023 at 05:17:43PM +0800, Lichen Liu wrote:
> There is confusing message in /var/log/anaconda/packaging.log when
> installing
> kexec-tools during the system installation on ppc64le:
>
> 	Event Notification Registration successful (id: 1)
>
> Make servicelog_notify slient when there are no erros.
>
> Signed-off-by: Lichen Liu <lichliu(a)redhat.com&gt;
> ---
>  kexec-tools.spec | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/kexec-tools.spec b/kexec-tools.spec
> index d7d2947..51a16ba 100644
> --- a/kexec-tools.spec
> +++ b/kexec-tools.spec
> @@ -268,7 +268,7 @@ touch /etc/kdump.conf
>
>  %ifarch ppc64 ppc64le
>  servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
> 2>/dev/null
> -servicelog_notify --add --command=/usr/lib/kdump/kdump-migrate-action.sh
> --match='refcode="#MIGRATE" and serviceable=0' --type=EVENT
> --method=pairs_stdin
> +servicelog_notify --add --command=/usr/lib/kdump/kdump-migrate-action.sh
> --match='refcode="#MIGRATE" and serviceable=0' --type=EVENT
> --method=pairs_stdin >/dev/null
>  %endif
>
>  # This portion of the script is temporary.  Its only here
> @@ -299,7 +299,7 @@ fi
>
>  %preun
>  %ifarch ppc64 ppc64le
> -servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
> +servicelog_notify --remove --command=/usr/lib/kdump/kdump-migrate-action.sh
> >/dev/null
>  %endif
>  %systemd_preun kdump.service
>
> -- 
> 2.40.1
>  
Patch merged, thanks!

Reviewed-by: Coiby Xu <coxu(a)redhat.com&gt;

-- 
Best regards,
Coiby