Commit 7bc38ccb has broken libvirt-lxc containers detection

[turchanov]

Commit 7bc38cc that solves FACT-3202 has broken libvirt-lxc containers detection

Warning: Facter: Container runtime, 'lxc-libvirt', is unsupported, setting to 'container_other'

Before the commit facter code in read_cgroup matched "lxc" on a prefix match, that is any container runtime that was named with prefix "lxc" would have satisfied the test, and an environment string "container=lxc-libvirt" will be detected as a container runtime

https://github.com/puppetlabs/facter/blame/05e4387c8de085a44ff5cfbe02081703dbbf5d60/lib/facter/resolvers/containers.rb#L27

          lxc_from_environ = /container=lxc/ =~ output_environ

[joshcooper]

Thanks @turchanov @lollipopman and I were discussing whether it should be a prefix or exact match in #2741 (comment) And it sounds like prefix match is preferred. Can you describe steps to reproduce like I did in #2741 (comment)

[lollipopman]

I think adding another case statement would be best, as it is technically another type of container, this is how systemd does it:

        [VIRTUALIZATION_SYSTEMD_NSPAWN]  = "systemd-nspawn",
        [VIRTUALIZATION_LXC_LIBVIRT]     = "lxc-libvirt",
        [VIRTUALIZATION_LXC]             = "lxc",
        [VIRTUALIZATION_OPENVZ]          = "openvz",
        [VIRTUALIZATION_DOCKER]          = "docker",
        [VIRTUALIZATION_PODMAN]          = "podman",
        [VIRTUALIZATION_RKT]             = "rkt",
        [VIRTUALIZATION_WSL]             = "wsl",
        [VIRTUALIZATION_PROOT]           = "proot",
        [VIRTUALIZATION_POUCH]           = "pouch",
        [VIRTUALIZATION_CONTAINER_OTHER] = "container-other",

https://github.com/systemd/systemd/blob/49fd31df58a707e7d748581a6986164132460cf2/src/basic/virt.c#L568-L578

[turchanov]

Sorry for delay, I am struggling to provide steps to reproduce the bug... (We have a custom procedure to setup lxc containers in libvirt, and I am still looking for the solution)

[github-actions]

Migrated issue to FACT-3484