[BREAKING CHANGE] update for onedir salt package

[didiermfb]

PR progress checklist (to be filled in by reviewers)

• Changes to documentation are appropriate (or tick if not required)
• Changes to tests are appropriate (or tick if not required)
• Reviews completed

---

What type of PR is this?

Primary type

• [build] Changes related to the build system
• [chore] Changes to the build process or auxiliary tools and libraries such as documentation generation
• [ci] Changes to the continuous integration configuration
• [feat] A new feature
• [fix] A bug fix
• [perf] A code change that improves performance
• [refactor] A code change that neither fixes a bug nor adds a feature
• [revert] A change used to revert a previous commit
• [style] Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)

Secondary type

• [docs] Documentation changes
• [test] Adding missing or correcting existing tests

Does this PR introduce a BREAKING CHANGE?

Yes.
The new version is working only with onedir package. Do not use this version with non onedir salt versions.

Related issues and/or pull requests

#76

Describe the changes you're proposing

Make this formula compatible with onedir salt versions.

Pillar / config required to test the proposed changes

Need onedir Salt version.

Debug log showing how the proposed changes work

N/A

Documentation checklist

• Updated the README (e.g. Available states).
• Updated pillar.example.

Testing checklist

• Included in Kitchen (i.e. under state_top).
• Covered by new/existing tests (e.g. InSpec, Serverspec, etc.).
• Updated the relevant test pillar.

Additional context

[didiermfb]

Hello, I work only on debian so I tried to update packages for other OS but I'm not 100% sure that they are the correct ones.

[didiermfb]

I still work on this PR to make the CI run properly.

[didiermfb]

I don't understand what's happening with the CI. It's seems that some master images are using salt 3007, 3006.6 or 3006RC1 and some are not with salt onedir.
For example AlmaLinux master and CentOS 8 master shouldn't return the error with cp437 encoding as it should use python embedded with salt.
Debian 10 master is using Salt 3006rc1.
I don't know what can I do to resolve this.

[sylvainfaivre]

The default-debian-10-master-py3 box is outdated, see https://gitlab.com/saltstack-formulas/libvirt-formula/-/jobs/6085940290 :

You asked for latest and you have 3006.0+0na.a350c44 installed, sweet!

It refers to this commit, which is more than one year old :
saltstack/salt@a350c44

The CentOS and Alma boxes are current, though.