-
Notifications
You must be signed in to change notification settings - Fork 495
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
facter fail on vservers after upgrade to puppet7 (facter 4.6.1) #2695
Comments
@aqueos thanks for the bug report, we're looking into this. |
Migrated issue to FACT-3464 |
@aqueos can you try a more recent version of facter? Please list which facts are resolving differently and their before and after values. Also can you try installing |
hi, i tried, the facter version is the one in the puppet7 packages.
installation do not change the issue:
|
ah interesting, so:
Is coming from
Could you try running
|
Here it is:
as you know perhaps vserver containers do not let you access thoses files. It worked fine on older facter versions :) |
Describe the Bug
After upgrading puppet 6.x to puppet 7.X facter failed in various ways:
Expected Behavior
the puppet working on this container type (vservers) worked perfectly for many years but this upgrade it seems facter is a lot more brittle and need some files to be here even in container and chroot :)
Steps to Reproduce
enter a vserver container, launch facter 4.6.1
Environment
Additional Context
the issue arrived after upgrading from puppet 6.x to 7.29
the first error seems to be in resolvers/containers.rb see farther for why.
the second one is lscpu in a container returning an error because the containers have an empty /sys directory
the third is access to /sys/block as there is no /sys/block file at all in resolvers/partitions.rb and resolvers/disks.rb, i think a little check could be missing for this case ?
the first is generated because the container system show the /proc/1/environ file as here and readable but it block the access, so it is readable but not really. On previous factor it was not an issue this was skipped for vserver container but not here.
The second error is that lscpu is using /sys that is not present in the container (or chroot) and therefor the facts linked to it fail in error, /proc/cpuinfo is here and readable but not /sys/.
best regards,
Ghislain.
The text was updated successfully, but these errors were encountered: