enh(docker::local): Make compatible with Podman containers

[jensrudolf]

Community contributors

Description

Modify the cloud::docker::local::plugin to be compatible with Podman container engine.

Podman uses two whitespaces at minimum to seperate its cli output colums whereas Docker uses three. This reduces the required whichtespaces in the regex filter of the command output accordingly.

Type of change

• Patch fixing an issue (non-breaking change)
• New functionality (non-breaking change)
• Functionality enhancement or optimization (non-breaking change)
• Breaking change (patch or feature) that might cause side effects breaking part of the Software

How this pull request can be tested ?

Tested locally on RHEL8 with Icinga, Podman and Docker containers.

Checklist

• I have followed the coding style guidelines provided by Centreon
• I have commented my code, especially hard-to-understand areas of the PR.
• I have rebased my development branch on the base branch (develop).
• I have provide data or shown output displaying the result of this code in the plugin area concerned.

[lucie-dubrunfaut]

Hello :)

Thank you for your contribution. We plan to work very soon on a dedicated Podman plugin (you can vote on the idea related in TheWatch). We would be delighted to discuss your specific requirements for a Podman connector with you in more detail.

[jensrudolf]

Hi Lucie,

thanks for your reply, A dedicated Podman plugin sounds fantastic. Currently, we only monitor whether containers are running or not. This is already possible with this docker plugin if you provide a softlink docker -> podman and apply the patch in this MR to adapt the regex for the number of whitespaces podman uses to separate its output.

Another approach might be to replace the docker plugin with a more generic container plugin that supports both docker and podman via CLI or the REST API preferably with the local socket so the Docker/Podman API does not have to be exposed outside the monitored host.

What do you think?

BR
Jens

[lucie-dubrunfaut]

Hello @jensrudolf :)

We'll try to take your suggestion into account, but I think that the Docker and Podman packs will be separated for maintenance reasons in the future if there's too much divergence between them. However, we have planned a common code base (as in the case of the snmp_standard plugin, which is called in a large number of plugins).
About protocols, for the time being our development team will address the need for a Podman RestAPI plugin. It will be possible to adapt Cli afterwards (our plugins are designed to manage several protocols, generally via custom modes) but this is not planned for the time being (however, you can help us prioritise this need by creating a TheWatch idea on the subject).

Best regards.

[lucie-dubrunfaut]

Hello :)

We just released the new podman plugin with this PR #5427
Feel free to test it and give us feedbacks about it :)