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

Inventory OS column blank on all nodes running Centos 7 #485

Closed
ebdavison opened this issue Nov 15, 2018 · 3 comments · Fixed by #712
Closed

Inventory OS column blank on all nodes running Centos 7 #485

ebdavison opened this issue Nov 15, 2018 · 3 comments · Fixed by #712

Comments

@ebdavison
Copy link

I have 7 nodes running CentOS and 1 running Ubuntu and none of the CentOS nodes are displaying a value in the OS column on the inventory screen.
Puppet: 6.0
PuppetDB: 6.0
puppetboard: 0.3.0, installed manually and running on Apache with Mod-WSGI

@ebdavison
Copy link
Author

ebdavison commented Nov 16, 2018

More digging shows the values for OS that is being used is the fact lsbdistdescription, which for CentOS does not seem to be populated.

Maybe something like this would work (better?):

app.py
273c273
<                      ('OS', "%s %s" % (operatingsystem, operatingsystemrelease)),
---
>                      ('OS', 'lsbdistdescription'),

@jrau9115
Copy link

Adding my own information to this bug as we recently started using Puppetboard and I noticed the same issue. The OS column appears to work for Debian based operating systems because of what facts it's trying to pull to get the OS. RedHat, CentOS, and Fedora don't show up in the list at all.

While looking into this myself I can't seem to find where this is getting set, as modifying app.py doesn't seem to change anything, so I'm guessing it's getting overloaded deeper into the program. Regardless, I'll try taking a stab at getting this working, as my OCD acts up seeing only half our inventory showing the proper data in the OS column.

@ffalor
Copy link

ffalor commented Aug 13, 2021

Looks like it's being set here

I haven't tried it, but based on the code and readme we can overwrite it by providing a value for INVENTORY_FACTS

The readme just says to provide a list of tuples, if you want to see how they are generating it you can view the code here

Also os.description fact should have the similar/same output as lsbdistdescription and work for all OS's. Perhaps that should be the default.

melck added a commit to melck/puppetboard that referenced this issue Sep 12, 2022
melck added a commit to melck/puppetboard that referenced this issue Sep 12, 2022
melck added a commit to melck/puppetboard that referenced this issue Sep 12, 2022
gdubicki pushed a commit that referenced this issue Sep 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants