Bug 2383751

The etcekeeper-dnf5 plugin is no longer able to stop dnf5 transaction when it notices that /etc directory is not clean.

Reproducible
============

Always

Version
=======

```
$ rpm -qa | egrep '(etckeeper|dnf5)'
libdnf5-5.2.13.1-1.fc42.x86_64
libdnf5-cli-5.2.13.1-1.fc42.x86_64
dnf5-5.2.13.1-1.fc42.x86_64
dnf5-plugins-5.2.13.1-1.fc42.x86_64
dnf5daemon-server-5.2.13.1-1.fc42.x86_64
python3-libdnf5-5.2.13.1-1.fc42.x86_64
libdnf5-plugin-expired-pgp-keys-5.2.13.1-1.fc42.x86_64
libdnf5-plugin-actions-5.2.13.1-1.fc42.x86_64
etckeeper-dnf-1.18.22-3.fc42.noarch
etckeeper-1.18.22-3.fc42.noarch
etckeeper-dnf5-1.18.22-3.fc42.noarch
```

Steps to Reproduce
==================

1. Install and initialize etckeeper to track /etc via git
2. Configure `AVOID_COMMIT_BEFORE_INSTALL=1` in /etc/etckeeper/etckeeper.conf
3. Make a change in /etc so that git working three there will not be clean
4. Try to install some package, eg `dnf install caddy` assuming you don't have it installed

Actual Results
==============

The etcekeeper-dnf5 plugin notices the problem, but the error is not propagated:

```
# dnf install caddy

** etckeeper detected uncommitted changes in /etc prior to dnf run
** Aborting dnf run. Manually commit and restart.

Updating and loading repositories:
Repositories loaded.
Package                        Arch     Version                        Repository            Size
Installing:
 caddy                         x86_64   2.10.0-1.fc42                  updates           46.1 MiB
Installing dependencies:
 fedora-logos-httpd            noarch   42.0.1-1.fc42                  fedora            12.1 KiB
Installing weak dependencies:
 julietaula-montserrat-fonts   noarch   1:9.000-2.fc42                 updates            5.6 MiB

Transaction Summary:
 Installing:         3 packages

Total size of inbound packages is 14 MiB. Need to download 14 MiB.
After this operation, 52 MiB extra will be used (install 52 MiB, remove 0 B).
Is this ok [y/N]:
```

Expected Results
================

The etcekeeper-dnf5 plugin notices the problem and stops the dnf transaction:

```
# dnf install caddy

** etckeeper detected uncommitted changes in /etc prior to dnf run
** Aborting dnf run. Manually commit and restart.

File "/etc/dnf/libdnf5-plugins/actions.d/etckeeper.actions" on line 3: Exit code: 1
# 
```

Additional Information
======================

It looks like this is caused by adding the `log.INFO=` line in the etckeeper output as was proposed here:

https://bugzilla.redhat.com/show_bug.cgi?id=2326283#c8

When I drop these, the raise error works again.

```
# dnf install caddy

** etckeeper detected uncommitted changes in /etc prior to dnf run
** Aborting dnf run. Manually commit and restart.

File "/etc/dnf/libdnf5-plugins/actions.d/etckeeper.actions" on line 3: Exit code: 1
```

I suspect that when using the log.INFO convention, one can no longer simply raise error via sheer return code, but will need to report it as an error or via other  means.