Skip to content
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

Several OS files under version 4.2 are missing the mountpoints fact #200

Closed
michael-riddle opened this issue Oct 29, 2021 · 5 comments
Closed

Comments

@michael-riddle
Copy link
Contributor

michael-riddle commented Oct 29, 2021

Hi Folks,

Our spec tests for our internal projects leverage the mountpoints fact and we noticed recently (within the last month) that the spec tests were failing due to mountpoints not actually being set. Looking at the following files in the 4.2 directory, we found many of them were missing that fact. Here's the list:

ubuntu-20.04-x86_64.facts
ubuntu-18.04-x86_64.facts
redhat-8-x86_64.facts
freebsd-13-x86_64.facts
freebsd-12-x86_64.facts
freebsd-11-x86_64.facts
fedora-34-x86_64.facts
fedora-33-x86_64.facts
debian-9-x86_64.facts
debian-11-x86_64.facts
centos-9-x86_64.facts
@bastelfreak
Copy link
Member

I am not sure which part creates that fact

  • maybe facter was installed but puppet wasnt
  • maybe a regression in facter 4.2

do you have facter 4.2 machines where this fact is present?

@michael-riddle
Copy link
Contributor Author

I do not currently, this popped up during spec tests, we specifically saw the issue in rhel 8 and that caused us to poke around in the other facterdb facts to see if it was actually missing from the fact files or not, that's where the list above came from.

I could fairly easily spin up a rhel 8 box, but the other OS's in the list might be a bit more difficult since we don't currently have an automated process to spin those up.

@bastelfreak
Copy link
Member

every factset is a good step forward, so even a PR for RHEL8 is helpful :) This repository contains also a Vagrant file for most of the operating systems.

michael-riddle added a commit to michael-riddle/facterdb that referenced this issue Nov 4, 2021
…ntpoints

- Added functionality to get_facts.sh to iterate through the version 7 puppet agents and their facter versions for Debian, Fedora, and RedHat OS families.
- Generated more complete facts for redhat 8
- Generated more complete facts for debian 11
- Generated facter 3.14 facts for fedora 33
- Generated more complete facts for fedora 34

Ref voxpupuli#200
@michael-riddle
Copy link
Contributor Author

I created a PR to cover as many of these OS's as I could using the Vagrantfile within facts. There are a few outstanding ones that I couldn't figure out how to get the facts for using the tools provided in the repo as it exists now, but those should be a good start.

bastelfreak added a commit that referenced this issue Nov 5, 2021
(#200) Updated get_facts.sh and several os facts missing mountpoints
@hbrown-uiowa
Copy link
Contributor

#206 should fix the issue going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants